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).

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