Fix the Hub Slug and the Dead Anchor in the Canonical copilot-instructions.md - #730
Merged
Conversation
…tions.md The file is carried whole by every fleet repo, so both defects reach each carrier rather than staying here. The unrecognized-shape step named the hub by slug, which the audit's carried check flags in a downstream copy: the file declares no verbatim section, so a byte-faithful carry trips it and the carrier is asked to answer for wording it did not write. Route the reader through the AGENTS.md "Fleet Bootstrap" section instead, which is byte-locked and names the hub as its whole function. The Disproved Claims section linked GOVERNANCE.md #every-finding-ends-in-an-action, a heading that left GOVERNANCE.md when the review-loop contract was packaged as the pr-review-conduct Skill, so the link resolves nowhere in the hub or in any carrier. Point it at "PR Review Etiquette", the section that routes to that Skill. A sweep of every relative Markdown anchor in the tree found no other dead one.
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the canonical .github/copilot-instructions.md to avoid downstream conformance drift and to fix a dead Markdown anchor that is carried into every fleet repository.
Changes:
- Removes the hardcoded hub slug from non-verbatim prose and instead routes readers via
AGENTS.md"Fleet Bootstrap". - Replaces a dead
GOVERNANCE.mdanchor with a link toPR Review Etiquette(and references thepr-review-conductSkill it routes to).
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
The appositive read as a compound subject against a singular verb.
ptr727
added a commit
that referenced
this pull request
Aug 15, 2026
… the Upstream Repo-Layout Segment (#734) Closes #719 Closes #720 Closes #721 Closes #722 Promotes six merged `develop` PRs to `main`: - #732 -- Reconcile the PlexCleaner registry entry and audit report after the 2026-08-15 resync (closes #719). - #731 -- Exempt hub-hosted paths from the prose gate's `dead-path` rule (closes #721). - #730 -- Fix the hub slug and the dead anchor in the canonical `copilot-instructions.md` (closes #720, #722). - #723 -- Refresh the PhotoCleaner audit report after the 0e84805 resync. - #733 -- Add an `upstream/` segment to the repo layout convention (addresses #725, not closed -- the fork-naming variant differs from the issue's proposal). - #724 -- Add the HomeAutomation-Config audit report and reconcile its registry entry. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
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.
Closes #720
Closes #722
Both defects are in
.github/copilot-instructions.md, which every fleet repo carries whole, so each one reaches every carrier rather than staying in the hub. #722 is the same dead anchor as #720's second item, filed separately from a different downstream pass.The hub slug (#720 item 1). The "A Shape Nothing Recognizes Blocks the Loop and Earns an Issue" step named the hub as
ptr727/ProjectTemplate. The file declares no verbatim section, so the audit'scarried:check flags that name in any downstream copy, and a byte-faithful carry trips it:audit.py --branch hub-resync-2026-08-15 PlexCleanerreported exactly that on ptr727/PlexCleaner#922. The rule's own exception, incomment-and-doc-style'sreferences/carried-doc-references.md, is scoped to a verbatim region, so the canonical is what needs fixing rather than the check needing an exception. The step now routes the reader through theAGENTS.md"Fleet Bootstrap" section, which is byte-locked and whose whole function is to name the hub, matching the wording PlexCleaner already reworded its copy to.The dead anchor (#720 item 2, #722). The Disproved Claims section linked
../GOVERNANCE.md#every-finding-ends-in-an-action. That heading leftGOVERNANCE.mdwhen the review-loop contract was packaged as thepr-review-conductSkill (#677), where the rule now reads "Every finding ends in one of five outcomes", so the link resolves nowhere in the hub or in any carrier. It now points at "PR Review Etiquette", the section that routes to that Skill, and names the Skill.A sweep of every relative Markdown anchor in the tree (101 files) found this as the only dead one, so the fix is complete rather than the first of a batch.
Prose gate and markdownlint clean on the file.