Skip to content

[Spec 1286] Post-merge state capture: thread narrative + porch completion record - #1345

Open
waleedkadous wants to merge 8 commits into
mainfrom
builder/aspir-1286
Open

[Spec 1286] Post-merge state capture: thread narrative + porch completion record#1345
waleedkadous wants to merge 8 commits into
mainfrom
builder/aspir-1286

Conversation

@waleedkadous

Copy link
Copy Markdown
Contributor

Post-merge state capture for Spec 1286 (follow-up to #1341, which is merged).

Two files, four commits, no code:

File What
codev/projects/1286-consult-configurable-per-lane-/status.yaml records the PR-merge and the verify phase transition — the canonical completion record on main
codev/state/aspir-1286_thread.md the merge + verify narrative, which the thread-retention rule wants on main alongside codev/reviews/

These landed on the builder branch after #1341 merged, so they were stranded there. Opening this
at the architect's direction rather than leaving them behind.

What the verify notes record

The verify phase was a real pass, exercised against the freshly built merged dist/ rather than the
branch:

  • Ask 1 (per-lane models): claude → claude-opus-5, codex → gpt-5.6-sol, each returned with
    its provenance key
  • Ask 2 (lane selection): PIR → gemini, codex · SPIR → gemini, codex, claude
  • Fail-fast: a typo'd lane gives
    Invalid consultation model "codexx" in porch.consultation.byProtocol.pir.models…
  • --model-id is live in the merged CLI's --help — the check that mattered most, since that
    flag originally shipped registered, parsed, documented and inert, which is part of why Spec
    1286 exists
  • Full suite on the merged state: 4271 passed, 48 skipped, 0 failed; tsc and build clean

Note

The verify-approval gate is still open with Waleed. If it is approved while this PR is open, porch
will add one more status.yaml commit to this branch and it will join this PR — which is the
intended completion record, not drift.

No code, no behavior change, no test impact.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant