dev → main: the meaning check reads new shapes first, and its miss withdraws an early flag (#4208) #181

Merged
bvandeusen merged 2 commits from dev into main 2026-09-22 08:14:50 -04:00
Owner

The operator explicitly requested this merge. It carries two commits, both green on dev (CI run 7253 on e2197ac).

  • 19438cc: the capped semantic pass now reads the newest unclassified shapes first. A conclusive "no canon" result now withdraws a divergence flag that was raised before the check had read the shape. On the first live refresh after PR #180, the 150-per-refresh cap was spent on old rows, and all six new shapes were flagged before the check read them.
  • e2197ac: the integration test asserts the withdrawn flag on the row itself, not on a flag count.

Server-side only.

🤖 Generated with Claude Code

https://claude.ai/code/session_01821k5B3Ysecp9fNYs92Kuy

The operator explicitly requested this merge. It carries two commits, both green on dev (CI run 7253 on `e2197ac`). - `19438cc`: the capped semantic pass now reads the newest unclassified shapes first. A conclusive "no canon" result now withdraws a divergence flag that was raised before the check had read the shape. On the first live refresh after PR #180, the 150-per-refresh cap was spent on old rows, and all six new shapes were flagged before the check read them. - `e2197ac`: the integration test asserts the withdrawn flag on the row itself, not on a flag count. Server-side only. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01821k5B3Ysecp9fNYs92Kuy
bvandeusen added 2 commits 2026-09-22 08:14:46 -04:00
fix(shapes): the capped meaning pass reads new shapes first, and its miss withdraws an early flag (#4208)
CI & Build / Python lint (push) Successful in 3s
CI & Build / Plugin hooks (push) Successful in 12s
CI & Build / integration (push) Failing after 51s
CI & Build / TypeScript typecheck (push) Successful in 53s
CI & Build / Python tests (push) Successful in 1m33s
CI & Build / Build & push image (push) Successful in 21s
19438cc894
Measured on the first live refresh after 91cde6c deployed: the version bump
queued 1,533 rows for the semantic arm, the cap read 150 of them in row
order, and all six shapes new since the previous refresh - the only rows
flag_divergence acts on, and the highest ids - were flagged before the arm
reached them. The gate silenced nothing because it never got to look, and
since a flag persists until judged, a later conclusive miss could not take
it back.

- propose_for_repo sorts the semantic todo by _semantic_priority:
  unclassified before scoped, newest first.
- flag_divergence withdraws a standing flag when the row now carries a
  conclusive miss; that evidence alone withdraws one.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01821k5B3Ysecp9fNYs92Kuy
fix(tests): assert the withdrawn flag on its row, not on a count the skew margin inflates (#4208)
CI & Build / Python lint (push) Successful in 2s
CI & Build / Plugin hooks (push) Successful in 9s
CI & Build / TypeScript typecheck (push) Successful in 52s
CI & Build / integration (push) Successful in 1m1s
CI & Build / Python tests (push) Successful in 1m34s
CI & Build / Build & push image (push) Successful in 14s
e2197ac799
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01821k5B3Ysecp9fNYs92Kuy
bvandeusen merged commit 3530ed5e87 into main 2026-09-22 08:14:50 -04:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: bvandeusen/FabledScribe#181