Skip to content

pm-dispatch SKILL: skills updates are ADR-class — human review/merge required (maintainer ruling 2026-08-11) #7548

Description

@huangyiirene

Maintainer ruling, 2026-08-11, issued directly to the devx PM seat in chat (session session_01GZKbx4xyF7U5WXj6ch49BM), verbatim, untranslated:

所有 skills 的更新和 adr 类似,需要人工审核

What this settles

Skill updates get the same terminal-state discipline the ADR ruling (#6741) established for docs/adr/**: AI seats may draft, branch, open and review the PR, but confirming it and landing it is the maintainer's manual act — ⛔ no AI seat merges, enqueues, or arms auto-merge on a PR whose path face touches a skill.

The SKILL edit (by PR — and by this very ruling, that PR is itself human-merged)

  1. Step 7 ACCEPT path fork: extend the existing docs/adr/** fork so it also fires on any changed path under .claude/skills/** or skills/** (published skills dir) — same three-part terminal state: review lands on the issue as usual; the PR stays visibly pending for the maintainer (ready or draft, never queued, never auto-merged); the round report carries an "awaiting a human merge" line until it lands.
  2. Guardrails: add the ruling beside the ADR and release-action prohibitions, quote verbatim.
  3. One-hit-is-enough / no-proportional-read applies as with ADR (a mixed diff touching a skill file takes the fork; split the PR if the skill half should land separately).

Scope note for the implementer to verify at work time (measure, do not assume): "所有 skills" is read as both skill roots — .claude/skills/** (internal agent skills) and skills/** (published skills). If the maintainer narrows or widens this, the ruling comment on this card supersedes.

Serialization trap (recorded at filing, step-3 discipline)

Same-file with in-flight #7498 (claude/issue-7498-decision-four-lens, editing SKILL.md now): ⛔ do not dispatch this card until #7498's PR reaches terminal state; the dev must re-derive section anchors from post-#7498 origin/main, not from this card. Line ratchet: headroom was 23 lines before #7498; re-measure after it lands.

Immediate effect (already applied, ahead of the SKILL text)

Routing: filed with pm:queue + domain:devx under the maintainer direct-command channel (rule 4, standing authorization 2026-08-10 — the maintainer is directly instructing this session; quoted directive above is the authority). Dispatch tier when worked: claude-fable-5 (mandatory — SKILL surface).

Activity

  1. huangyiirene commented on Aug 11, 2026

    @huangyiirene
    CollaboratorAuthor

    Second maintainer ruling folded into this card's scope (2026-08-11, verbatim, untranslated):

    我认为所有的 skills 任务应该单独一个项目经理负责,专题和我讨论。

    What changes on this card:

    1. Ownership: this card now belongs to the skills seat (sticker [PM seat] domain:skills — ⏳ vacant #7623, created ⏳ vacant on the maintainer's instruction). The domain:devx label stays as an interim routing artifact until the PR below settles the vocabulary — but the devx seat no longer treats this card as its dispatch candidate; both seat posts record the carve-out.
    2. Scope extension: the SKILL amendment this card specifies now ALSO includes the structural half — the domain-table / seat-roster change registering the skills seat (scope: .claude/skills/** + skills/**, carved out of domain:devx's row), the 专题讨论 channel note, and the vocabulary decision (whether a domain:skills routing label is minted — decided in the PR with the triage seat's single-producer discipline in view).
    3. Serialization unchanged: same file as PR pm-dispatch SKILL: require a four-lens block on every needs-user-decision card, Blocked-by-aware batch selection, round-brief decision flags #7552 — ⛔ not dispatchable until pm-dispatch SKILL: require a four-lens block on every needs-user-decision card, Blocked-by-aware batch selection, round-brief decision flags #7552 reaches terminal state. Tier unchanged: claude-fable-5 mandatory; the PR is human-merged (per this card's own subject matter, now doubly so).

    Execution trail: seat sticker #7623 filed; #4604 index row added; devx seat post #6023 updated. Recorded by the devx seat (session session_01GZKbx4xyF7U5WXj6ch49BM) under the maintainer's direct-command channel.


    Generated by Claude Code

  2. huangyiirene commented on Aug 11, 2026

    @huangyiirene
    CollaboratorAuthor

    Transferred to the skills seat (#7623): label domain:devx → domain:skills. Executed on the maintainer's direct chat instruction to the devx seat, 2026-08-11, verbatim: 「所有现有 skills 相关卡片转到 新的项目经理」. Serialization note unchanged (behind PR #7552).


    Generated by Claude Code

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions