fix(han-planning): correct two defects in the visual-material reference material - #190
Merged
mxriverlynn merged 1 commit intoAug 20, 2026
Merged
Conversation
…nd bound the empty visual table artifact-invariants.md ended mid-clause on the Visual Reference invariant, so a run that received visual material asked the synthesizer to preserve a rule whose definition stopped after "with". Restores the missing clause from the pre-extraction SKILL.md text: an inline embed beside the prose describing each state. planning-boundary-rule.md showed the Visual Material Received section as a table with illustrative rows and never said the table is omitted when nothing arrived. A run following that shape wrote a placeholder row and the Step 9 completeness gate refused it. States that `None received` stands alone and the table is omitted, and why. Closes #187.
mxriverlynn
marked this pull request as ready for review
August 20, 2026 14:47
This was referenced Aug 20, 2026
Merged
mxriverlynn
added a commit
that referenced
this pull request
Aug 20, 2026
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.
Fixes the two defects reported in #187, both in
han-planningreference material and both about how a planning run handles visual material.The truncated invariant
plan-a-feature/references/artifact-invariants.mdended mid-clause on theVisual Referenceinvariant, stopping after the word "with". Step 8 tells the synthesizer to read that file and preserve the invariants it carries, so a run that received visual material was asked to preserve a rule whose definition was unfinished.The missing clause was lost when the file was extracted from
SKILL.mdin d01abe6. I recovered it from that commit's parent, where the same invariant reads in full at line 655. It is restored verbatim: the table lists every item the boundary record records as received, under that exact heading, with an inline embed beside the prose describing each state.The empty visual-material table
planning-boundary-rule.mdsaid the Visual Material Received section takesNone receivedwhen nothing arrived, then showed the section as a table with two illustrative rows. Nothing said the table itself goes away. A run that followed the shape it was shown wrote a placeholder row, and the Step 9 completeness gate refused it, because the gate reads every row as an item claimed on disk.The rule now says
None receivedstands alone and the table is omitted, and names the gate as the reason.Verification
npm run lintpasses. Both changes are prose in reference files with no script or schema behavior attached.Closes #187.