Skip to content

skills(pm-dispatch): scope-narrowing follow-up — revert the decision-card assign half per the 2026-08-19 rulings - #9994

Merged
os-zhuang merged 4 commits into
mainfrom
claude/issue-9976-notification-handover
Aug 19, 2026
Merged

skills(pm-dispatch): scope-narrowing follow-up — revert the decision-card assign half per the 2026-08-19 rulings#9994
os-zhuang merged 4 commits into
mainfrom
claude/issue-9976-notification-handover

Conversation

@os-warren

Copy link
Copy Markdown
Collaborator

Refs #9976 (already closed by the earlier PR — intentionally no closing keyword here)

Scope-narrowing follow-up. PR #9982 merged carrying edit 2 (the decision-card same-stroke assign/unassign of os-zhuang, with the inbox defined as the Assigned queue). After delivery and before merge, the maintainer narrowed the scope; this PR reverts exactly that half and keeps everything else #9982 landed.

Maintainer rulings (2026-08-19, PM chat, verbatim):

我感觉决策卡推给我太麻烦了,我需要和ai讨论才能判断,这个还是维持之前的样子。我会定期和ai讨论。

只有需要我审核的pr 推给我。

What this reverts (vs current main)

What this keeps and grounds

  • The governed-PR review-request step (four-piece ACCEPT fork) stays — explicitly confirmed by the second ruling, which is now quoted in the step as the scope grounding (push channel is PRs needing review, and only that).
  • The decision-digest retirement stays — 「我会定期和ai讨论」 means chat replaces the digest; the inbox is worked by periodic maintainer-AI discussion. The first ruling's full verbatim is recorded once, compactly, at the facets-block provenance line.

Line budget (682-ceiling shrink-only ratchet)

Gates (at head 9368fde78, fresh merge base 90f4d5dc3, exit codes captured before any pipe)

Derivation: node scripts/pm/dispatch-gates.mjs (no args) — "change set derived from git — 1 path(s) vs merge base 90f4d5d", same 7 families, tier line "Model tier — MANDATORY: claude-fable-5" (this card ran on it).

  • check:pm-skill-ratchet — "✓ check-skill-line-ratchet: .claude/skills/pm-dispatch/SKILL.md is 680 lines (ceiling 682; headroom 2)."
  • check:pm-governed-prose — "✓ check-governed-prose: 2 instruction surface(s) name all 5 registered governed surfaces (docs/adr/** · .claude/** · skills/** · AGENTS.md · CLAUDE.md) and claim no others."
  • check:pm-skill-id-lint — "✓ check-skill-id-lint: 15 file(s) clean"
  • check:skill-frame-sync — "✓ check-skill-frame-sync: 4 copies of the decision frame are structurally isomorphic across 3 files"
  • check:skill-compatibility — "✓ check-skill-compatibility-version: 11 SKILL.md file(s) reconciled against 77 workspace packages"
  • check:doc-authoring — "✓ doc authoring guard: 379 files clean — no bare metadata literals."
  • check:pm-governed-merges --self-test — "✓ 77 assertions"
  • check:doc-formula-expressions — "✓ check:doc-formula-expressions: 22 record-scoped formula example(s) across 404 files / 1414 TS blocks judged clean"

Reverse verification at this head (direction predicted first): 683 lines turns the ratchet RED — observed "is 683 lines; the ratchet ceiling is 682" (exit 1); restored, green at 680 (exit 0), worktree clean.

Merge note: the branch merged origin/main after #9982 landed; the textual merge auto-resolved the three revert hunks to main's side (they were net-unchanged vs the old base on this branch), so they were re-applied in a follow-up commit — the final file is byte-identical (git hash-object match) to the reviewed amendment commit 288a1df47.

No changeset: instruction-file-only PR, nothing published ships (skip-changeset).


Generated by Claude Code

…os-zhuang, retire decision digest
Per maintainer rulings 2026-08-19: waiting-on-maintainer states must be
GitHub-native pushable state. Governed-PR ACCEPT fork gains a step (request
review from os-zhuang on the draft PR); needs-user-decision flip assigns
os-zhuang and transcription unassigns (inbox = Assigned queue); the
per-round decision digest / batch-adjudication standing text is retired.
678/682 lines, paid by the retired digest text.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AeA3nU1B5Q2pgxqxgUrexd
…t-worked, only PRs push
Maintainer rulings 2026-08-19 (post-delivery, pre-merge): decision-card
assign/unassign of os-zhuang is reverted entirely (inbox is worked by
periodic maintainer-AI chat discussion, not pushes); the governed-PR
review-request step stays, grounded by the narrowing quote; the digest
retirement stays. 680/682 lines.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AeA3nU1B5Q2pgxqxgUrexd
…ification-handover
# Conflicts:
#	.claude/skills/pm-dispatch/SKILL.md
… the textual merge
The merge of origin/main auto-resolved three hunks to main's side (the
branch's edit-then-revert made them net-unchanged vs the old base). File
restored byte-identical to the amendment commit's version.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AeA3nU1B5Q2pgxqxgUrexd
@os-warrenos-warren added the skip-changeset PR has no user-facing published change; bypasses the changeset gate label Aug 19, 2026 — with Claude
@github-actionsgithub-actionsBot added size/s documentation Improvements or additions to documentation labels Aug 19, 2026
@os-zhuang
os-zhuang marked this pull request as ready for review August 19, 2026 13:00
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentationImprovements or additions to documentationsize/sskip-changesetPR has no user-facing published change; bypasses the changeset gate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@os-warren@os-zhuang@claude