integrate: 4.5b documents a flagless cross-reference-post.sh call; the helper requires --issue/--pr/--branch #30

Closed
opened 2026-08-13 11:18:36 +00:00 by jbr870 · 1 comment
Owner

Observation

integrate/SKILL.md step 4.5b instructs "Post the cross-reference pair — cross_reference.post(issue, pr) via bin/cross-reference-post.sh" without showing the call, but the helper hard-requires three flags: cross-reference-post.sh --issue N --pr P --branch B (usage line in the helper header; --branch feeds the pr-link body). The documented call shape is rejected as-is:

  • verity-main-4c283a2c: "integrate/SKILL.md 4.5b omits the required --branch flag for cross-reference-post.sh (helper rejects the documented call)"
  • verity-slot3-149: "cross-reference-post.sh requires an undocumented --branch flag derivable from the checkout"

Recurrence

Mechanical — verified against HEAD (b9d3cf7 + meta-lane work): 4.5b still names the helper without a fence showing the required flags. Also 2 independent runs hit it live.

Hypothesis

Give 4.5b a real fence with all required flags (--issue "$ISSUE_NUMBER" --pr "$PR_NUMBER" --branch "$FEATURE_BRANCH"), matching how sibling steps show their helper calls.

Suggested lane

meta-amend — minimal text fix, one fence.

Filed by /meta-retrospective 2026-08-13.

## Observation `integrate/SKILL.md` step 4.5b instructs "Post the cross-reference pair — `cross_reference.post(issue, pr)` via `bin/cross-reference-post.sh`" without showing the call, but the helper hard-requires three flags: `cross-reference-post.sh --issue N --pr P --branch B` (usage line in the helper header; `--branch` feeds the pr-link body). The documented call shape is rejected as-is: - verity-main-4c283a2c: "integrate/SKILL.md 4.5b omits the required --branch flag for cross-reference-post.sh (helper rejects the documented call)" - verity-slot3-149: "cross-reference-post.sh requires an undocumented --branch flag derivable from the checkout" ## Recurrence Mechanical — verified against HEAD (b9d3cf7 + meta-lane work): 4.5b still names the helper without a fence showing the required flags. Also 2 independent runs hit it live. ## Hypothesis Give 4.5b a real fence with all required flags (`--issue "$ISSUE_NUMBER" --pr "$PR_NUMBER" --branch "$FEATURE_BRANCH"`), matching how sibling steps show their helper calls. ## Suggested lane meta-amend — minimal text fix, one fence. *Filed by /meta-retrospective 2026-08-13.*
Author
Owner

Outcome — amended (commit 0aad192)

4.5b now shows the full required call in a fence: cross-reference-post.sh --issue "$ISSUE_NUMBER" --pr "$PR_NUMBER" --branch "$FEATURE_BRANCH", with the note that --branch feeds the pr-link body and comes from the checkout.

Gate: lint clean. Desk-check vs originating scenario (verity-main-4c283a2c, slot3-149): both runs failed by calling the helper as 4.5b described it — flagless; an orchestrator following the amended text now has the exact argv. Closing.

## Outcome — amended (commit 0aad192) 4.5b now shows the full required call in a fence: `cross-reference-post.sh --issue "$ISSUE_NUMBER" --pr "$PR_NUMBER" --branch "$FEATURE_BRANCH"`, with the note that `--branch` feeds the pr-link body and comes from the checkout. Gate: lint clean. Desk-check vs originating scenario (verity-main-4c283a2c, slot3-149): both runs failed by calling the helper as 4.5b described it — flagless; an orchestrator following the amended text now has the exact argv. Closing.
Sign in to join this conversation.
No description provided.