The boundary was named six lines before the page introduces it - #38
Merged
Conversation
A one-paragraph correction to 9271c4c. The fix that commit shipped for the pre-flight reading list was the drafted wording, not the amended one, and the amendment existed for a reason. WHAT WAS WRONG. "All three sit past the Part 1 boundary" pointed at a label the page had not yet used. "Part 1" appeared exactly twice in the file: there, and six lines later at "**Part 1 ends here.**", which is where it is introduced. A reader met the name in a sentence that assumed it, before the sentence that defines it. The same commit had just removed "the first half is yours" from the opening for naming that span by a proportion, so the net was two names for one boundary and the new one forward-referencing. The review that found this had flagged the collision before the commit landed, and named the fix. The fix was not applied. Five of six amendments went in and this one did not, so what shipped was the version its own reviewer had rejected. Recorded here rather than quietly corrected, because a correction that hides how it was needed teaches nothing. WHAT CHANGED. The sentence names the boundary by the section it is, linked, rather than by a label: "All three sit past [Handing this to Claude Code](...), in the half addressed to the agent." The link is the one the opening already uses for the same boundary, and the anchor is checked by test_internal_links_resolve.py. "Part 1" now occurs once in the file, at the marker that defines it. WHAT WAS DELIBERATELY NOT DONE. The Part vocabulary is not removed. HOUSE-STYLE.md pins this section as the only marker that Part 1 ends, so deleting the label costs a second file this session does not own. Naming the boundary by its section makes the label resolve where it stands, which was the defect; the label itself is not one. MEASURED. Long sentences 217 -> 217, fat cells 30 -> 30, both unchanged and at or inside baseline. 146 tests green, Word copy rebuilt with pandoc 3.10, zero non-ASCII bytes. Co-Authored-By: Claude Opus 5 <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
1 participant
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.
A one-paragraph correction to
9271c4c(#37).What was wrong. The pre-flight reading list said "All three sit past the Part 1 boundary" — pointing at a label the page had not yet used. "Part 1" appeared exactly twice in the file: there, and six lines later at "Part 1 ends here.", which is where it is introduced. The same commit had just removed "the first half is yours" from the opening for naming that span by a proportion, so the net was two names for one boundary, the new one forward-referencing.
How it got in, which is the part worth recording. The review had flagged this exact collision before the commit landed and named the fix. Five of its six amendments were applied and this one was not, so what shipped was the version its own reviewer had rejected. The failure was in applying the plan, not in making it.
What changed. The sentence now names the boundary by the section it is, linked — the same link the opening already uses for it — rather than by a label: "All three sit past Handing this to Claude Code, in the half addressed to the agent."
test_internal_links_resolve.pychecks that anchor. "Part 1" now occurs once in the file, at the marker that defines it.What was deliberately not done. The Part vocabulary is not removed.
HOUSE-STYLE.mdpins this section as the only marker that Part 1 ends, so deleting the label costs a second file this session does not own. Naming the boundary by its section makes the label resolve where it stands, which was the defect; the label itself is not one.Measured. Long sentences 217 → 217, fat cells 30 → 30, both unchanged. 146 tests green, Word copy rebuilt with pandoc 3.10, zero non-ASCII bytes.
🤖 Generated with Claude Code