Skip to content

Restore selected #5599 evidence links in CI documentation - #5659

Merged
Mister-Meeseeks merged 1 commit into
CrocSwap:mainfrom
apoursh:docs/selected5599-ci-links
Oct 26, 2026
Merged

Mister-Meeseeks merged 1 commit into
CrocSwap:mainfrom
apoursh:docs/selected5599-ci-links

Conversation

@apoursh

@apoursh apoursh commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

Change and mathematical scope

The shared research CI planner fails test_current_ci_section_matches_selected_record after the #5599 promotion because the current section of docs/ci-verification.md omits the selected proof-review and source-manifest links. Add those two links so readers and the existing regression test can resolve the evidence named by certificates/selected-result.json.

Task and overlap

Scope unchanged: this one-file documentation repair follows aliromman's CI handoff, with coordination here. Base: canonical main df0137efd35e11b4445734bf612bdd4c7ebc330c; fresh all-state PR searches found no overlapping repair. The selected result, source evidence and workflow behavior are unchanged.

Validation

Validation on CPython 3.12.14, macOS:

python3 -B -m unittest discover -s tests -p test_research_current_status.py -v
git diff --check

Before the change, the six-test module failed exactly the review and source_manifest subcases. After the change, all six tests pass, including checks that both destinations exist and match the selected record. No scientific replay or full shared suite was run for this navigation-only repair; hosted CI remains to be checked.

Credit

Credit @aliromman for diagnosing and reporting the failure. Prepared by apoursh with substantial OpenAI Codex assistance, including separate agent checks of the diagnosis and overlap. Apache-2.0.

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.

2 participants