v0.19.1 — the class the umbrella had been catching for us, twice - #20
Merged
Conversation
B-126's board row shipped in v0.19.0 with nine cells against the eight its header declares, and the umbrella's validator refused the re-pin. Second time in three releases: B-124 did the same in v0.18.0, also caught there rather than here. A class seen twice becomes a script. check_ledger_tables_keep_their_shape() counts every row in docs/evidence/*.md against its header, splitting on an unescaped pipe exactly as a reader does. It found a pre-existing defect on its first run, older than either of mine: rows 015 and 016 of verification.md quote shell pipelines inside backticks and had been splitting into five cells since they were written. Both of my own breakages were the same underlying mistake and it was not markdown: v0.18.0's from a grep pattern pasted into a cell, v0.19.0's from a Python edit where adjacent string literals were written across lines without parentheses, so only the first fragment bound and the replacement left an orphaned tail carrying a bare pipe. TG-1..3. Gate: npm test rc=0. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
B-126's board row shipped in v0.19.0 with nine cells against the eight its header declares, and the umbrella's validator refused the re-pin.That is the second time in three releases.
B-124did the same in v0.18.0 — also caught bysshlg-skillsrather than here.A class seen twice becomes a script
check_ledger_tables_keep_their_shape()counts the cells of every row indocs/evidence/*.mdagainst the header its table declares, splitting on an unescaped|exactly as a reader does. The member now refuses what a sibling had been catching on its behalf.It found a pre-existing defect on its first run
Older than either of mine:
verification.mdrows 015 and 016 each quote a shell pipeline inside backticks —… \| grep -c '^def check_'— and that bare pipe had been splitting both rows into five cells since they were written. Nobody had seen them because nothing counted.The real cause was not markdown
Worth recording, because the two breakages look like the same careless paste and are not:
|. The row was malformed because the edit that wrote it was silently partial — the pipe was a symptom.Evidence
npm testrc=0—14 checks = 10 named + 4 per-skillrow has 9 cells against the 8Ledger TG-1 … TG-3.
🤖 Generated with Claude Code