skill-authoring §3 + operational-rigor §2: contributing is not adopting; re-resolve citations on install; stage by pathspec - #87
Open
firaen22 wants to merge 1 commit into
Conversation
…ng; re-resolve citations on install; stage by pathspec over dirty state Three clauses from one downstream consumer's adoption pass over this repo. skill-authoring §3, on the merged-integration bullet: that bullet tracks the campaign continuing UPSTREAM. The mirror case is an author's own adoption — a merge closes the contribution while the author's always-loaded files still lack the rule, and nothing feels missing because the author has been applying it from working context. Binds the adoption debt to the same ledger row that logs the merge, since a debt parked in a plan or summary is the invalidation-clause failure this file already warns about. skill-authoring §3, new rule before the packaging trio: a distributable skill's section-number and named-anchor citations address ITS library. Installed elsewhere they still read as valid and resolve to the wrong content. Re-resolve each against the destination by grep, classify retargeted / unchanged / absent-here, and record retargets in the port note — unrecorded, the next diff-against-upstream reads them as drift and a re-sync restores the broken numbers. operational-rigor §2, on the baseline bullet: the mutation-side half of a baseline the rule already required. With pre-existing changes present that you did not author, stage by explicit pathspec; git add -A publishes edits you have never read under your message. All three ship unprobed per the covenant; Provenance entries added. 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
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.
Summary
Three clauses from one downstream consumer's adoption pass over this repo — reading rules in, not writing them.
§3resolving to a real section is more dangerous than a 404. Re-resolve every citation against the destination by grep, classify retargeted / unchanged / absent-here, and record retargets in the port note as upstream-citation → local-target pairs — unrecorded, the next diff-against-upstream reads your own retargets as drift and a re-sync silently restores the broken numbers.git add <path>), never-A/-u/.— on a tree that's entirely yours,-Ais fine; the baseline read is what tells you which case you're in.All three ship
unprobedper the covenant; Provenance entries added to both files.Note
Applying the new citation-resolution rule to this draft's own Provenance entries caught two mis-numbered section references before commit (both said §4/§3 where the rules actually landed in §3/§2) — fixed. Not a probe, just the rule firing on its own author.
Test plan
git diff --word-diffagainst branch point — zero words removed, additions only🤖 Generated with Claude Code