Skip to content

[finding] check-half-states.mjs still cites SKILL.md by line number in three places (header line 109 / line 465, selfTest line 465) after the two ruled docblocks dropped theirs #16037

Description

@os-steve

Observation from #16003's flight, whose ruling deleted the SKILL.md line numbers from exactly two docblocks in scripts/pm/check-half-states.mjs (the RELEASE_COMMENT_MARKER docblock and the H47 section comment, both citing 465 and 493) and said no other sentence moves. Three citations of the same class remain in the file and were deliberately left alone:

  • the header inventory's H1 entry: (SKILL.md line 109) and SAME write (line 465);
  • selfTest's RELEASE_RULE_LINE comment: .claude/skills/pm-dispatch/SKILL.md line 465, VERBATIM.

The file's own register (the same H47 docblock) says to grep the quoted text rather than trust a number, and the numbers have already moved: after #16003's SKILL.md edit the release line sits at 464 and line 465 is the new partial-landing line, so both line 465 citations point at a different sentence than the one they quote. line 109 is still correct today and will drift the same way on the next edit above it.

Mechanical fix, same shape as the ruled two: delete the numbers, keep the quoted text. Nothing derives or compares these numbers, so no gate reds when they drift — which is why this is a finding and not a red build.

Refs (not addressed here): #16003.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions