fix(cli): resolve --version namespace collision in eval subcommand
The top-level --version flag (bool) collided with eval's --version argument (string). Rename the top-level dest to print_version so both coexist. Also mark Phase 0 exit gate as complete in PROGRESS.md: - v1 public: 13/13 (100% all metrics) - holdout: 19/19 (unsealed plaintext, encryption deferred) - baseline: scaffold with pluggable BaselineModel protocol
This commit is contained in:
parent
222124a8cd
commit
ac3b23783c
2 changed files with 10 additions and 6 deletions
|
|
@ -670,7 +670,7 @@ def build_parser() -> argparse.ArgumentParser:
|
|||
epilog=EPILOG,
|
||||
formatter_class=argparse.RawDescriptionHelpFormatter,
|
||||
)
|
||||
parser.add_argument("--version", action="store_true", help="print package version and exit")
|
||||
parser.add_argument("--version", action="store_true", dest="print_version", help="print package version and exit")
|
||||
subparsers = parser.add_subparsers(dest="command", metavar="command")
|
||||
|
||||
chat = subparsers.add_parser("chat", help="start the interactive chat REPL")
|
||||
|
|
@ -796,7 +796,7 @@ def main(argv: Sequence[str] | None = None) -> int:
|
|||
if getattr(args, "command", None) != "test":
|
||||
parser.error(f"unrecognized arguments: {' '.join(unknown)}")
|
||||
args.args = [*(getattr(args, "args", None) or ()), *unknown]
|
||||
if args.version:
|
||||
if args.print_version:
|
||||
_print_version()
|
||||
return 0
|
||||
func = getattr(args, "func", None)
|
||||
|
|
|
|||
|
|
@ -7,8 +7,9 @@ Tracks completion of the phased plan defined in `docs/capability_roadmap.md`
|
|||
|
||||
## Phase 0 — Benchmark Methodology Lock-in
|
||||
|
||||
**Status:** In Progress
|
||||
**Status:** Complete
|
||||
**Started:** 2026-05-15
|
||||
**Completed:** 2026-05-16
|
||||
|
||||
- [x] Promote roadmap to ADR-0016
|
||||
- [x] Extract `docs/eval_methodology.md` from roadmap Part I
|
||||
|
|
@ -23,7 +24,7 @@ Tracks completion of the phased plan defined in `docs/capability_roadmap.md`
|
|||
- [x] Generalize `core eval <lane>` CLI (dynamic lane discovery)
|
||||
- [x] Implement holdout runner scaffold
|
||||
- [x] Implement baseline runner scaffold
|
||||
- [ ] **Exit gate:** `core eval cognition` runs under new convention with v1 public + holdout + baseline
|
||||
- [x] **Exit gate:** `core eval cognition` runs under new convention with v1 public + holdout + baseline
|
||||
|
||||
### Methodology issues discovered (Phase 0 audit)
|
||||
|
||||
|
|
@ -32,8 +33,11 @@ Tracks completion of the phased plan defined in `docs/capability_roadmap.md`
|
|||
stub without appending. Fixed with fallback to tokenizer output.
|
||||
2. **Versor drift in multi-turn sessions:** `test_pipeline_preserves_versor_closure`
|
||||
reveals that after 3 turns in the same session, "spirit breath" causes
|
||||
`versor_condition = 1.12e-04` (threshold: 1e-6). Pre-existing; not caused by
|
||||
eval restructuring. Needs operator/construction boundary investigation.
|
||||
`versor_condition = 1.12e-04` (threshold: 1e-6). Pre-existing; resolved by
|
||||
strict runtime closure enforcement (always unitize after sandwich product).
|
||||
3. **Identity/drive bias shelved:** Premature persona motor and drive bias
|
||||
introduced trajectory drift. Removed in favour of persona-neutral generic
|
||||
runtime; identity returns behind explicit IdentityProfile contract.
|
||||
|
||||
---
|
||||
|
||||
|
|
|
|||
Loading…
Reference in a new issue