Skip to content

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

Closed

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. added
    pm:queueReady for the PM dispatch loop
    domain:devxDomain lane — seat card indexed by label:pm:seat
    on Aug 11, 2026
  2. 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 — 🟢 os-elon-musk · 由 os-elon-musk 账户的项目负责 #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

  3. added
    domain:skillsDomain lane — seat card indexed by label:pm:seat
    and removed
    domain:devxDomain lane — seat card indexed by label:pm:seat
    on Aug 11, 2026
  4. 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

    Labels

    domain:skillsDomain lane — seat card indexed by label:pm:seat

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions