Drop incidental catalog paths from verbatim AGENTS sections - #426
Merged
Conversation
Three verbatim sections still named hub-only catalog paths as prose after the #422 genericization: the codegen snippet in Branching Model, the catalog-workflow LF aside in the Line Endings rule, and the vscode-catalog pointer in Editor and Tasks. A downstream repo carries none of catalog/, so each is a not-carried reference the carried-files rule bans and a Copilot false positive on every re-vendor. Restate the codegen bullet behaviorally and drop the two redundant catalog pointers (.editorconfig still enforces the LF pin; the extension set is already listed inline). The load-bearing cross-model repo-config/operational/develop.json reference stays. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Removes hub-only catalog/... path references from byte-carried (verbatim) sections of AGENTS.md so downstream repos that do not carry catalog/ do not inherit broken/coordination-only prose references.
Changes:
- Updates the Branching Model bot-sync bullet to refer to "the codegen workflow" instead of a
catalog/snippets/...path. - Drops a redundant
catalog/snippets/...aside from the Line Endings guidance while keeping the substantive LF/CRLF policy intact. - Removes the trailing VS Code extensions sentence that pointed at
catalog/snippets/vscode/.
ptr727
added a commit
that referenced
this pull request
Jul 25, 2026
## Promote develop to main Three governance/doc-only changes, all squashed to develop and Copilot-clean: - **#425** - Add Copilot review guidance for byte-locked carried fleet content (new `## Reviewing Carried Fleet Content` in `.github/copilot-instructions.md` + interim decline-pointer + `files.json` manifest entry). Root-cause fix for the recurring Copilot false positives PlexCleaner#880 flagged. - **#426** - Drop incidental catalog paths from verbatim AGENTS sections (restate the codegen bullet behaviorally; drop the two redundant `catalog/...` pointers). Finishes #422's genericization. - **#427** - Add verbatim `## Durable Knowledge and Self-Improvement` section (promoted from the non-carrying preamble; broadened to the self-improvement duty). Declared in `files.json` + `section-model.md`. No code or workflow changes - documentation and spec only, so no release fires on merge. ## Verification - `python3 spec/validate.py` - OK. - `python3 spec/audit.py --selftest` - PASS. 🤖 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.
What
Removes three hub-only
catalog/…references still sitting as prose inside verbatim AGENTS.md sections after the #422 genericization.Why
The "carried files carry no coordination references" rule (Documentation Style / carried-files) bans a reference inside byte-locked fleet text to a path a given repo does not carry. A downstream repo carries none of
catalog/, so each of these was both a rule violation and a guaranteed Copilot false positive on every AGENTS.md re-vendor - the same friction PlexCleaner#880 flagged. This finishes what #422 started.Changes
catalog/snippets/workflows/run-codegen-pull-request-task.yml-> "the codegen workflow" (behavioral; the codegen workflow has no carried.github/…path since it is catalog-only, unlike the dependabot/merge-bot workflows named alongside it).(and the catalog snippet workflows in catalog/snippets/workflows/*, pinned LF the same way)aside. Redundant -.editorconfigenforces the LF pin and its own comment carries the rationale.The catalog holds the full set and per-language additions: catalog/snippets/vscode/.The standard set and per-language additions are already enumerated inline in the same bullet.Kept: the load-bearing cross-model
repo-config/operational/develop.jsonreference, and the already-genericized*Reference:*codegen note (names a workflow, not a hub path). Preamble / Devcontainer / Repository Layout references are in non-verbatim sections, untouched.Verification
python3 spec/validate.py- OK (21 cataloged).python3 spec/audit.py --selftest- PASS.Downstream verbatim copies read stale until re-vendored, riding the re-vendor wave already in flight (no extra cycle).
🤖 Generated with Claude Code