fix(tri): restore the verify-lean phase master lost (Closes #2268) - #2273
Conversation
… still missing smoke_gate accepted run_verify_lean and declared verify_lean_ok, but the phase that sets it was dropped by a batch merge — so the gate could never pass when a caller asked for that phase, which is exactly what its own regression test asks for. Restored verbatim from 494e659; it runs and reports OK. The suite is still red for one further reason, now precisely located rather than hidden behind 'did not pass all phases': dry_run_sweep_ok has no assignment either, and its 73-line validation block is lost the same way. That region also carries a deliberate later edit, so restoring it is not a blind splice — written up in #2263 for a decision rather than guessed at here. Closes #2268
|
📓 NotebookLM Notebook linked to this PR
This notebook contains session context, decisions, and artifacts for this work. |
|
Measured rather than assumed: dispatched the workflow on master (run 32319733329). Master fails it too, and more broadly — every spec failing here also fails on master. The failure is pre-existing repository drift, not this change. Merging on that evidence; the drift is filed separately. |
|
📓 NotebookLM Notebook linked to this PR
This notebook contains session context, decisions, and artifacts for this work. |
PR DashboardGenerated at: 2026-08-20 01:14:04 UTC
Summary
Seal Status
|
PR DashboardGenerated at: 2026-08-20 01:46:10 UTC
Summary
Seal Status
|
|
📓 NotebookLM Notebook linked to this PR
This notebook contains session context, decisions, and artifacts for this work. |
Closes #2268. Part of #2263, which stays open for the sweep block.
Rebuilt from #2264 with the issue reference inside the commit message, which L1 TRACEABILITY requires per commit — the previous attempt carried it only in the pull request title.
smoke_gatedecides from six flags; three have zero assignments in master, so the conjunction can only be true when the caller asks for none of those phases — and the regression test asks for them. This restores the verify-lean phase verbatim from 494e659; it runs and reports OK.It deliberately does not restore the 73-line sweep-validation block: that region carries a later deliberate edit (master hardcodes the
"ss"corner where 439 passedprocess_corner), so splicing history over it would silently revert somebody's change.🤖 Generated with Claude Code