The mechanism pull request for #1317, #1378, merged with thirteen introduced findings open, put here by the maintainer's decision after three granted rounds, since the last two recorded passes surfaced only new edges of the mechanism and no regression of a fix. Each is quoted from that pull request's read record. Fix the code items first, the near-miss marker above all, since it is the one that produces the silent hole in a skill the mechanism exists to prevent.
Code, scripts/build_dist.py
- A near-miss marker,
<!-- include A.md > B --> without the colon, or a marker line with trailing text past the close, matches nothing and is content, so the region is never filled and --check exits 0. Refuse any HTML comment line beginning <!-- include or <!-- /include that fails the marker regex.
heading_body ends a body at a level-one heading, where spec/audit.py and canonical_review.py match only ## and run through one, and the heading regex accepts a tab after the hashes where they do not. The "split alike" claim in two comments holds for level-two headings only. Decide one reading for the three tools.
- A marker may sit up to three spaces indented, but the body is written at column 0 and the closing marker keeps its indent, so a region opened inside a list item renders outside it. Either refuse an indented marker or state that the allowance covers the marker, not the region.
_unindented counts spaces only, so a tab-indented code block's doubled blank lines collapse. Narrow, since the Markdown lint keeps hard tabs out of sources.
Prose
scripts/README.md: the exit-2 cause list omits a body leaving a fence open, a region in a file the walk does not visit, and a file that is not UTF-8, which the skill lists.
AGENTS.md's closing paragraph and docs/fleet-map.md say a skill is "hand-authored" at the skills tree without the include-region qualifier scripts/README.md and .agents/skills/README.md carry.
skill-lifecycle > The Pipeline: "a skill file or source that is not UTF-8" exits 2 only for the Markdown files the walk decodes, so a non-UTF-8 references/*.txt or scripts/ file is copied into both distributions and --check exits 0.
skill-lifecycle > The Pipeline: "exit 2 rather than 1" reads as the unrenderable region's code alone, where a symlink anywhere under the skills tree and an unreadable generated file also exit 2.
skill-lifecycle > The Pipeline: a source need not be tracked, so an untracked or ignored file fills the region locally and exits 2 in CI where the file is absent.
skill-lifecycle > The Doc-Packaging Pattern: the "Included content" shape has no exemplar and no doc-side sentence exists in the tree until a per-class pull request lands the first include. Settle it in that pull request.
.github/copilot-instructions.md > Reviewing Carried Fleet Content: "that key" has no antecedent, since the appositive shows the marker text without saying that <path> > <heading> is the key.
- Same unit: the nested-region sentence names an inner key no copy shows, because the render drops the source's own markers, so the object it names exists only in the hub's authored tree.
- Same unit: "When the pull request changes the file the fix belongs in" can never be true downstream once the fix is placed in a hub file, and the unit leaves undecided whether a pull request touching the local copy of that path qualifies.
Related: #1379 (the doubled include walk in --check), #1317 (the per-class pull requests that will land the first live region).
The mechanism pull request for #1317, #1378, merged with thirteen introduced findings open, put here by the maintainer's decision after three granted rounds, since the last two recorded passes surfaced only new edges of the mechanism and no regression of a fix. Each is quoted from that pull request's read record. Fix the code items first, the near-miss marker above all, since it is the one that produces the silent hole in a skill the mechanism exists to prevent.
Code,
scripts/build_dist.py<!-- include A.md > B -->without the colon, or a marker line with trailing text past the close, matches nothing and is content, so the region is never filled and--checkexits 0. Refuse any HTML comment line beginning<!-- includeor<!-- /includethat fails the marker regex.heading_bodyends a body at a level-one heading, wherespec/audit.pyandcanonical_review.pymatch only##and run through one, and the heading regex accepts a tab after the hashes where they do not. The "split alike" claim in two comments holds for level-two headings only. Decide one reading for the three tools._unindentedcounts spaces only, so a tab-indented code block's doubled blank lines collapse. Narrow, since the Markdown lint keeps hard tabs out of sources.Prose
scripts/README.md: the exit-2 cause list omits a body leaving a fence open, a region in a file the walk does not visit, and a file that is not UTF-8, which the skill lists.AGENTS.md's closing paragraph anddocs/fleet-map.mdsay a skill is "hand-authored" at the skills tree without the include-region qualifierscripts/README.mdand.agents/skills/README.mdcarry.skill-lifecycle > The Pipeline: "a skill file or source that is not UTF-8" exits 2 only for the Markdown files the walk decodes, so a non-UTF-8references/*.txtorscripts/file is copied into both distributions and--checkexits 0.skill-lifecycle > The Pipeline: "exit 2 rather than 1" reads as the unrenderable region's code alone, where a symlink anywhere under the skills tree and an unreadable generated file also exit 2.skill-lifecycle > The Pipeline: a source need not be tracked, so an untracked or ignored file fills the region locally and exits 2 in CI where the file is absent.skill-lifecycle > The Doc-Packaging Pattern: the "Included content" shape has no exemplar and no doc-side sentence exists in the tree until a per-class pull request lands the first include. Settle it in that pull request..github/copilot-instructions.md > Reviewing Carried Fleet Content: "that key" has no antecedent, since the appositive shows the marker text without saying that<path> > <heading>is the key.Related: #1379 (the doubled include walk in
--check), #1317 (the per-class pull requests that will land the first live region).