Make the final self-check agree with the removal patterns - #236
Open
Scott-Emberson wants to merge 1 commit into
Open
Make the final self-check agree with the removal patterns#236Scott-Emberson wants to merge 1 commit into
Scott-Emberson wants to merge 1 commit into
Conversation
Step 3 treated any lost claim as an error, but patterns 2, 5, 6, 21, 25, 34, and 35 require removing claims, so the check rejected the skill's own model answers and made behavior inconsistent. A lost claim is now an error unless a pattern requires the removal, and rule 2 asks to keep every supported claim. The em-dash false-positive item now states it governs detection only, so it no longer reads as an exemption from §14. Refs blader#212. Co-Authored-By: Claude Fable 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
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.
The rewrite process's step 3 says "Treat any unsupported addition or lost claim as an error" with no carve-out, but seven patterns require removing claims: §2's After drops two named citations and a follower count, §25's After is "Cut the paragraph", §6's After drops "strategic location and ongoing initiatives", and §5, §21, §34, and §35 all instruct removal. Rule 2 ("Keep every claim.") had the same conflict. An agent applying step 3 literally has to reject the skill's own model answers, so behavior on those patterns becomes inconsistent across runs.
Changes:
This also addresses #212: with the carve-out explicit, patterns that are NOT on the removal list (like §10 rule-of-three and §24 hedging, the ones #212 demonstrates) can no longer read a deleted ranking or simultaneity claim as acceptable shape-editing; step 3 flags it as a lost claim.
SKILL.md stays at 456 lines and the 35 patterns are untouched. No version bump to avoid colliding with #233; happy to fold a release note into whichever lands last.
Closes #212.
🤖 Generated with Claude Code