Repository navigation
REQ-VISUALIZATION-007: README still describes the pre-M3 Pages state after #491 #496
Description
Activity
- addedpriority:highImportant and time-sensitive; schedule ahead of normal workImportant and time-sensitive; schedule ahead of normal worktype:bugVerified behavior differs from the intended contractVerified behavior differs from the intended contractarea:visualizationThe docs/ run viewerThe docs/ run viewerstatus:readySpecified and unblocked; safe for an agent to claimSpecified and unblocked; safe for an agent to claimqaFinding from the external QA voice, not yet turned into a work contractFinding from the external QA voice, not yet turned into a work contract
on Sep 15, 2026 Claim (AUTHOR)
- Role: AUTHOR, unattended run.
- Unit of work: this Issue, selected under AUTHOR_RUNBOOK section 2 item 4 (no open pull requests, no
status:blockedIssue whose condition is resolved; REQ-VISUALIZATION-007: README still describes the pre-M3 Pages state after #491 #496 is the highest-priority non-policystatus:readyIssue). - Intended scope:
README.mdpublic-text conformance only, plus the ledger row forREQ-VISUALIZATION-007indocs/spec/implementation_status.csvand the generateddocs/spec/IMPLEMENTATION_STATUS.md. No runtime, schema, formula or milestone-mechanics change. - Branch:
claude/issue-496-readme-m3-pages-conformance. - Known blockers: none identified at claim time.
Setting
status:in-progress.- addedstatus:in-progressClaimed work with an active branch or pull requestClaimed work with an active branch or pull requeststatus:needs-reviewImplementation complete, awaiting acceptanceImplementation complete, awaiting acceptanceand removedstatus:readySpecified and unblocked; safe for an agent to claimSpecified and unblocked; safe for an agent to claimstatus:in-progressClaimed work with an active branch or pull requestClaimed work with an active branch or pull request
on Sep 15, 2026 Handoff (AUTHOR)
- Pull request: REQ-VISUALIZATION-007: bring the README's Pages and requirement-status claims back to current evidence #499
- Branch:
claude/issue-496-readme-m3-pages-conformance - Tested revision:
538b020fe878b2bd9fafe449b4efa70f944b6645
Checks at that revision —
npm run typecheckpassed;npm testpassed (45 files, 655/655, 7 net new);npm run buildpassed;python -m pytest scripts/testspassed (529 passed, 247 subtests);python scripts/implementation_status.py --checkpassed;python scripts/status_lint.pypassed;dotnet restorepassed;dotnet build --configuration Release --no-restorepassed (0 warnings, 0 errors);dotnet test --configuration Release --no-buildpassed (45/45), so REQ-MIGRATION-003 is maintained. Nothing wasnot_runorunavailableexcept the two items recorded under Not checked in the pull request body: the live Pages deployment (all Pages assertions are against the committeddocs/index.html) and prose accuracy beyond what is mechanically decidable.All six acceptance criteria are met, with evidence per criterion in the pull request body.
Decisions taken
- Promoted the ledger row rather than leaving it PARTIAL.
REQ-VISUALIZATION-007was PARTIAL solely because the registry acceptance covers the README and its directly linked public docs, and the two linked articles carried open defects. Those were REQ-VISUALIZATION-008: local-market guide misstates expectation causality and bootstrap boundary #446 and REQ-VISUALIZATION-008: settlement guide conflates transfer conservation with later physical source/sink flows #447, merged as Issue #446: correct expectation causality and the bootstrap boundary in the local-market guide #486 and REQ-VISUALIZATION-008: separate transfer conservation from whole-tick stock flows in the settlement guide #489, andREQ-VISUALIZATION-008is now IMPLEMENTED. Every registry ACCEPTANCE item was re-checked against the text on this branch before promoting. - Cleared
MERGE_COMMITinstead of carrying REQ-VISUALIZATION-007: correct six stale M2/M3/M4-M5/M11 claims in the README #484's forward.scripts/backfill_merge_commits.pyfills blanks and never rewrites a value already present, so keepingf6894b33…would have permanently attributed this row to a different pull request's commit. This is inference from two mechanisms rather than a quoted rule, and it is flagged as such in the pull request body. - Added a regression test rather than relying on prose review.
src/diagnostics/readme-conformance.test.tspins each mechanically decidable README claim to the artifact that decides it. Mutation-proved: 3 of its 7 tests fail againstgit show master:README.mdand all 7 pass here. It is also the highest-risk file in the diff, and the pull request body says why.
What remains: nothing from this run. No new Issue was opened — no out-of-scope work was discovered. #481, named in this Issue's acceptance criterion 5, closed on 2026-09-15 via #497, so that part of the criterion is moot rather than waived.
Setting
status:needs-review. I did not approve, merge, or push tomaster.ACCEPTED and merged.
- Pull request: REQ-VISUALIZATION-007: bring the README's Pages and requirement-status claims back to current evidence #499, state
MERGED, squash commit7efd05a7c49ffea7ab4a72e5acd7f805df1179f6at 2026-09-15T05:55:10Z. - Accepted head revision:
538b020fe878b2bd9fafe449b4efa70f944b6645. Full verdict and evidence: REQ-VISUALIZATION-007: bring the README's Pages and requirement-status claims back to current evidence #499 (comment)
Evidence I observed at that head, independently of the pull request body:
- Required checks green at
538b020f…:typescript,build-and-test,policy-guard,mergeability(merges cleanly into the base branch, and the branch was 0 behindmaster),write-mergeability-status. - Re-executed both runtimes at the checked-out head:
npm run typecheckpassed;npm testpassed 655/655 across 45 files;npm run buildpassed;dotnet build --configuration Release0 errors anddotnet test45/45 with 0 skipped, so REQ-MIGRATION-003 is maintained;python scripts/implementation_status.py --checkreportedIMPLEMENTATION_STATUS.md matches 30 ledger row(s) over 49 registry row(s).python -m pytest scripts/testswasunavailablein my environment and is covered green bybuild-and-testat this SHA. - All six acceptance criteria met. Criteria 1–3 are now pinned mechanically by
src/diagnostics/readme-conformance.test.ts: I reproduced the mutation proof by restoringmaster's README over the repaired one, and 3 of the 7 tests failed (the same three named in the handoff), which also demonstrates the requirement-identifier extractor is not vacuous. Criterion 4 verified by the absence of those sentences from the diff; criterion 5 by theNot closedsection making no M3 closure claim. - README status claims checked against the ledger directly: REQ-MARKET-001..005, REQ-ACCEPTANCE-004, REQ-VISUALIZATION-006 and -008 all read
IMPLEMENTED. - Diff confined to the four declared paths; no test deleted, disabled or weakened (195 lines of test added, none removed, suite 648 → 655); no invariant touched; no secret, credential or local machine path.
REQ-VISUALIZATION-007 now reads
IMPLEMENTEDindocs/spec/implementation_status.csvwithISSUE496 andPR499. ItsMERGE_COMMITis intentionally blank and is filled byscripts/backfill_merge_commits.pyfrom the squash commit above; that is the normal path and not an outstanding gate on this Issue.This Issue closed automatically on merge with reason
COMPLETED, verified after confirming #499 is in stateMERGED. Removingstatus:needs-review— a closed Issue carries no activestatus:*label.- Pull request: REQ-VISUALIZATION-007: bring the README's Pages and requirement-status claims back to current evidence #499, state
- removedstatus:needs-reviewImplementation complete, awaiting acceptanceImplementation complete, awaiting acceptance
on Sep 15, 2026
Goal
Bring the root README's current-state and Pages wording into conformance with the now-merged M3 representation evidence, without changing runtime behavior, requirement meaning, or milestone mechanics.
Evidence
Fresh
masterafter Issue #269 / PR #491 still contains two directly contradictory public claims inREADME.md:Known scope boundariessays REQ-VISUALIZATION-006 "is not yet started". Issue REQ-VISUALIZATION-006: M3 polished LocalMarket Pages experience #269 is closed and the merged REQ-VISUALIZATION-006: publish the consolidated M3 LocalMarket Pages experience #491 implementation has already been accepted by QA as satisfying REQ-VISUALIZATION-006.The same section also calls
docs/spec/IMPLEMENTATION_STATUS.mdauthoritative. The project protocol explicitly saysdocs/spec/implementation_status.csvis the authoritative implementation-evidence ledger and the rendered Markdown is presentation only.This is a post-merge public-documentation conformance defect. It is distinct from closed #445: that repair fixed the stale MarketIntent/per-tick/M2-ledger/milestone/browser-capability wording it owned, but #491 subsequently changed the Pages completion state that README now fails to reflect.
Scope
README/public text only. Make the smallest factual repair:
docs/spec/implementation_status.csvas the authoritative per-row implementation evidence, not renderedIMPLEMENTATION_STATUS.md;Non-goals
docs/spec/mirror/.Acceptance criteria
docs/spec/implementation_status.csvas authoritative implementation evidence and does not call renderedIMPLEMENTATION_STATUS.mdauthoritative.Verification
Compare the updated README against current
masterPages output from #491, the authoritativedocs/spec/implementation_status.csv, and Handoff/11 milestone ownership. This is documentation-only conformance work; no economic behavior should change.