Give each rule one home - #74
Merged
Merged
Conversation
Move the rules of the bench into the file that owns them. A template README holds the facts of the code. A skill holds the procedure. The seat rules hold only what must hold with no skill open. - Shorten the scan rule of the Engineer to name the skill. - Cut the record rule of the Engineer. guide-a-build-step holds it. - Cut three duplicate house rules from the kit file. - Make the device-scan README a reference. Use "workstation". - Remove the skill name from the usb-camera README. Move the judgment of step 12 to observe-the-camera. - Keep only the command facts in step 4 of the psu README. - Name a README section in place of a step number in the error texts. - Fix a sentence in docs/templates.md. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V7z2GtopJRsfvJU9rCMuCh
The house rule "The power stays off until the checks of the build pass" has no other home that holds for the whole build, so kit.md keeps it. A seed test now pins it and the HM310P first power-on rule. The observe-the-camera skill holds the aim rule again, and its clip rules are a step of their own. The psu sensor error names the sensor step again. Two READMEs get a wrap fix and a plain paragraph. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V7z2GtopJRsfvJU9rCMuCh
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01V7z2GtopJRsfvJU9rCMuCh
andreisavu
marked this pull request as ready for review
October 4, 2026 22:00
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
2 participants
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.
Requested by Andrei · project thread
Before: eight rules sit in two to four places each: the seat rules,
kit.md, a skill, and a template README. Two copies disagree.kit.mdsays "cite the file that the script wrote", and the shared rule says "snapshot the file and cite the ref". The device-scan README says "container", and the skill says "workstation".After: a template README is the reference for its code. A skill is the procedure. The seat rules keep only the rules that hold with no skill open, plus the skill names.
kit.mdholds the facts of the kit and the bench.How:
kit.md: the datasheet, measurement, and power-off rules are cut. The shared rules and guide-a-build-step hold them.docs/templates.md: a grammar fix.Not changed:
test/live/led-sweep.tskeeps its own copy of two house rules in its eval prompt.Review: https://claude.ai/artifact/UpTRSZ8FiUUHPjwQuvghaz
🤖 Generated with Claude Code
https://claude.ai/code/session_01V7z2GtopJRsfvJU9rCMuCh
Generated by Claude Code