Skip to content

test: fix the six non-lag reds from the integrated gate (#236) - #255

Merged
chrisdpurcell merged 1 commit into
testingfrom
gate-fix
Sep 1, 2026
Merged

chrisdpurcell merged 1 commit into
testingfrom
gate-fix

Conversation

@chrisdpurcell

Copy link
Copy Markdown
Collaborator

Summary

The integrated gate on testing @ bc95622 reported ten ordinary-lane reds: four by-design reconcile-lag reds and six others, all test-side. This fixes the six: the 1.9 classifier corpus test now measures a frozen tip (4866a9fa) instead of HEAD, so legitimate T0/handoff commits after the admission floor no longer flip its zero counts; the two agent-handoff pin sites (test_agent_handoff_1_12 navigation links, test_agent_handoff_1_15 catalog row) move to 1.17; test_cut_successor loads scaffolded modules through tests.module_loading.load_module_from_path; scripts/go-verify-stamp.sh joins the post-anchor tooling allowlist; and the RepositorySnapshot.capture stub in test_spec_selected_routing accepts retain_content.

Governing work

Supporting: #236

Acceptance coverage

Verification

Targeted pytest (16 passed), ruff check, ruff format, basedpyright on the edited files. The full battery reruns on the merged tip.

…to 1.17; guard cut-successor module load; allowlist go-verify-stamp; match the snapshot stub signature

The integrated gate on bc95622 surfaced six reds outside the reconcile-lag set: the
1.9 classifier test measured HEAD so the train's first T0 commit flipped its zero count;
two agent-handoff pin sites still named 1.16; test_cut_successor ran exec_module outside
the bytecode guard; scripts/go-verify-stamp.sh was executable but unlisted; and a
RepositorySnapshot.capture stub predated retain_content.
@chrisdpurcell
chrisdpurcell marked this pull request as ready for review September 1, 2026 22:34
@chrisdpurcell
chrisdpurcell merged commit e3237c0 into testing Sep 1, 2026
1 of 2 checks passed
@chrisdpurcell
chrisdpurcell deleted the gate-fix branch September 1, 2026 22:34
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