docs(runbooks): defer to the pull-request closing rule (#708)

[ss-o]

Summary

• runbooks/project-tracker.md said to "Link implementation pull requests with a closing keyword", unconditionally. runbooks/pull-requests.md (following ADR-0022) allows Closes #N only when the diff meets every acceptance criterion and asks for Refs #N otherwise. The tracker runbook now points to that rule instead of contradicting it.
• One sentence changes; nothing else in the runbook does.
• Left alone on purpose: runbooks/sub-issues.md asks for a closing keyword from an implementation pull request to its own sub-issue, which that pull request completes, so it agrees with the pull-request rule.

Closes #708

Instruction impact review

1. Kind: canonical-detail runbook prose. The owning rule is in runbooks/pull-requests.md (ADR-0022), unchanged.
2. Consumers: agents and humans using the tracker runbook; routed for project-tracking, and referenced by the thin project-tracker skill. org_policy_report.py impact lists 15 repositories as shared-policy review candidates; none carries a copy of this sentence (checked in every child checkout).
3. Canonical owner: unchanged; the tracker runbook now defers to it.
4. Duplication or contradiction: removes one contradiction; no new copy of the rule.
5. Manifest routes: none change.
6. Mandatory rule delivery: the rule lives in the pull-request runbook and ADR-0022, independent of this runbook.
7. Generated output and size limits: scripts/validate-agent-policy.py passes.

Verification

• python3 scripts/validate-agent-policy.py: passed.
• python3 -m unittest scripts/test_validate_agent_policy.py: OK.
• python3 scripts/org-routing.py validate: inventory valid.
• python3 scripts/org_policy_report.py impact --changed runbooks/project-tracker.md: shared-policy review, 15 candidates, no copies found.

Agent handoff

• Status: Ready for review
• Current state: One-sentence correction pushed; waiting for hosted checks and a review of record.
• Blockers: None.
• Next steps: Review of record per the pull-request runbook; merge is the maintainer's decision.
• Related tracker/issues: docs(runbooks): use the pull-request closing rule in the tracker runbook #708, part of decision: organization procedures live once, as public runbooks that thin skills route to #668 (Project 28).

[ss-o]

Copilot review request on 0ca6f21 did not register.

• Request: 2026-09-29T15:23:46Z, POST /repos/z-shell/.github/pulls/709/requested_reviewers with reviewer copilot-pull-request-reviewer[bot], base main (its ruleset has a copilot_code_review rule). The response returned HTTP success with an empty requested_reviewers.
• Timeline: the only review_requested event is the CODEOWNERS team request at 15:22:12Z, before this request. No event names Copilot, and no Copilot review has appeared.
• Cause: review request not registered; cause unconfirmed. Billing and allowance evidence was not inspected. This is the fourth unregistered request in the organization today (fix(scripts): shut down the Trunk daemon trunk-safe-check starts (#704) #705, docs(ci): align the workflow auditor with the CI guidance (#706) #707, docs(i18n): match the localization guidance to crowdin.yml (#929) wiki#930).

This pull request is blocked on a review of record until the maintainer waits for Copilot or elects the ADR-0026 fallback.

[ss-o]

Fallback review under ADR-0026: Copilot request not registered on 0ca6f21

The maintainer elected the fallback after the Copilot request on this head did not register (recorded in the comment above). Executed under .github/skills/code-review/SKILL.md against .github/instructions/code-review-generic.instructions.md.

Findings

No actionable findings.

Note

This is a maintainer-elected self-review by the author's session, not an independent review or approval to merge.

Scope: the one-file diff of #709 at 0ca6f21 against main at eeb1646, and the acceptance criteria of #708.

Validation

• Passed: 11 hosted checks on this head, including Validate Agent Instructions and links-check. Locally: validate-agent-policy.py, the policy tests, org-routing.py validate, and org_policy_report.py impact (15 review candidates, none carries a copy of the sentence).
• Skipped or unavailable: none relevant.

Checklist verdicts and supporting evidence

• Accuracy against the canonical owner: pass. The new sentence matches runbooks/pull-requests.md section 2 (Closes #N only when the diff meets every acceptance criterion, Refs #N otherwise), which follows ADR-0022 (#708 criterion 1).
• Consistency: pass. runbooks/sub-issues.md asks for a closing keyword from a pull request to its own sub-issue, which that pull request completes; it agrees with the rule and needs no change.
• Links: pass. [pull-requests.md](pull-requests.md) resolves from runbooks/; links-check passed.
• Routing and policy: pass. No manifest route changes; scripts/validate-agent-policy.py passes (#708 criterion 2).
• Security and CI compliance: not applicable. Prose only.

Limits and follow-up

• None. Merging is the maintainer's decision; the maintainer has authorized it for this head. #668 stays open; this pull request refs it.