finding(skills): 预挂 needs:contract-review 在 2026-08-28 裁定后持续复发——两日 9 例,分诊/执行席的定级流程文本未吸收「可复审增量存在前永不挂标」

[zhuangjianguo]

Filed unassigned by 项目总监席(session session_01DxbNgzPMo4YuRBmGmCQp9m),仅记录,交 skills 车道定级。

事实

维护者 2026-08-28 对 #12887 裁定:needs:contract-review 仅在可复审增量(PR,或先到的报告)存在时挂载;队列/受阻卡的预挂废止(细则单源 .claude/skills/pm-dispatch/references/contract-review.md 「⛔ 不预挂」段)。

裁定后两日,总监席复审勤务两轮共摘除 9 例预挂(全部为定级/认领时挂标、无任何增量):

• 2026-08-30 轮:os#13347 · ui#6781 · ui#6776(认领评论自述「needs:contract-review applied」于认领时)
• 2026-08-31 轮:os#13540 · os#13539 · os#13531 · os#13405 · os#13216(分诊定级评论自述「触条款② PATH 肢 ⇒ needs:contract-review」于定级时,卡态 pm:queue)

每例摘除评论均引裁定原文并注明「PR 存在的同笔即重挂」。复发的成本:复审席每轮扫到不可审载体空耗一遍(本席两轮各花一趟排除),且标签宣称「真实待审」为假,腐蚀载体=待审的机读语义。

根因假设(待 skills 席核)

定级席与执行席的流程文本(分诊 SOP / 认领模板)在「条款②判定」步骤旁仍写着或暗示「同笔挂 needs:contract-review」——判定(裁决时点名条款②预期 yes)与挂载(增量存在时)两个动作没有在文本上分开。修法方向:在相应 SOP 的条款②段落写明「判定记录在卡,标签由 PR 创建者在开 PR 的同笔挂」,并引 2026-08-28 裁定。

⛔ 本卡不改任何 skill 文本(governed 面,归 skills 车道);仅记录实例与方向。

Refs: #12887(裁定载体)· references/contract-review.md 不预挂段 · 9 例摘除评论(各卡时间线可查)。

[zhuangjianguo]

定级(skills 车道自分诊):Task · priority:p2 · pm:queue · domain:skills

复发 9 例是对 2026-08-28 裁定的持续违反,成本为复审席每轮空耗一趟 + 标签机读语义腐蚀 ⇒ 高于 p3;无错误合并、无正确性伤害 ⇒ 非 p1。升 p1 条件:一次预挂导致真实待审载体被跳过或误处置。

采卡的根因方向:修法落在定级/认领流程文本的条款②段 —— 把「判定(裁决时点名条款②预期)」与「挂载(增量存在时,由开 PR 的同笔挂)」两个动作在文本上分开,并引 2026-08-28 裁定原文;references/contract-review.md 的「⛔ 不预挂」段为单源,只加指针不复制细则。

入队不即派(面持):候选落点 SKILL.md 认领模板段(被候批 PR #13505 持有)与 dispatch-runbook.md(被在飞 #13304 持有)⇒ 排两者之后;届时与同面卡按五门判折叠。

---

Generated by Claude Code

[zhuangjianguo]

Hypothesis VERIFIED — this card survives the value gate and stays queued: it is a correction of text that contradicts a standing ruling, not an addition.

Measured on origin/main 9c120f030: SKILL.md:758 reads 「派发席职责止于:卡上记一行认定、挂 needs:contract-review(标签命名审的对象,恒英文)、…」 — the dispatch seat's clause-② duty as written INSTRUCTS same-stroke hanging, exactly what the 2026-08-28 ruling abolished. references/contract-review.md:43 pulls the same direction (「条款②的 mode:subagent 派发照旧恒保留 needs:contract-review」) against that file's own :10 ⛔不预挂. The 9 measured recurrences trace to live text, not to seats ignoring text.

Scope when dispatched: in-place correction of SKILL.md:758 (判定记卡;挂载随可复审增量,由 PR 创建者同笔) plus reconciliation of contract-review.md:43 with its own :10 — word-level, net ≤0 lines. Serial: SKILL.md is held by PR #13505's chain; this card is next on that face after it clears. Explicitly NOT frozen into audit #13597 — ruling-contradiction corrections keep flowing; the freeze is for additions.

---

Generated by Claude Code

[zhuangjianguo]

Claim: PM loop round 5 — dispatch on the SKILL.md face freed by PR #13505's merge (per the verification comment above: correction class, explicitly not frozen by audit #13597)
Session: session_01EXxTW8mvPBhoHxmyPZ63de
Branch: claude/issue-13557-no-prehang-correction
Worktree: objectstack-issue-13557
Domain: domain:skills
File surface: .claude/skills/pm-dispatch/SKILL.md (the :758 派发席职责 sentence — position re-verified at 09b0d7b95, still :758 post-#13505) + .claude/skills/pm-dispatch/references/contract-review.md (:43 vs :10 reconciliation — READ first; edit only if :43 is measured to instruct label-timing rather than requirement-retention) ONLY. ⛔ scripts/pm/check-skill-line-ratchet.mjs untouched (pin moves ride audit #13597 Phase 3). Stop on breach; explain in the report.
Container & model: M, mode:subagent, model: fable — dispatch-gates --tier at 09b0d7b95: "Model tier — MANDATORY: claude-fable-5 … clause ① — the PM dispatch skill MAIN file is the lane's own operating protocol and a wrong edit propagates to every later dispatch". No exit taken: judgment-bearing protocol semantics (the grading/labeling duty sentence), not a mechanical one-liner.
Clause-②: yes | no — no (process/label text; no contract accept/reject behavior touched)
Serial constraints cleared: SKILL.md face freed by PR #13505's merge (02:44Z; this card was recorded next on that face). contract-review.md has no in-flight holder (#13417 parked pm:on-hold; its blocked branch stands but is not active). Retirement flight delivered (dispatch-runbook only); half-states PR armed (scripts only). Line budget: net ≤0 REQUIRED on both files (SKILL.md 960, this is a correction not an addition; contract-review.md 48/48 headroom 0 hard). H17: no intersection. Batch 2/3.

---

Generated by Claude Code

[zhuangjianguo]

ACCEPT — reviewer: skills lane seat, session session_01EXxTW8mvPBhoHxmyPZ63de (contract-review tier, full diff personally verified against the fetched PR diff). PR #13618, head 7e5871d7a, draft on the manual lane.

Verified: a 2-line, net-0 in-place rewrite of SKILL.md :758–759 — the duty list no longer instructs a hang at the grading stroke; needs:contract-review is attached by the PR creator in the same stroke as the reviewable increment, with the 恒英文 naming note riding the hang sentence and every surviving duty intact. The diff context confirms what the flight surfaced: the old text was SELF-contradictory (the duty list said hang-now while the very next clause said only-when-increment) — which explains the 9 measured misfires better than a simple omission would. All four needs:contract-review occurrences adjudicated (147/589/897 kept with reasons, 758 rewritten; the 889→897 position correction is honest re-measurement). contract-review.md:43 adjudicated as requirement-retention on structural evidence (section subject = 降档保险丝; contrast target = 免审跳过; timing owned by :7–11) and correctly left untouched, with the optional sharpening deferred to audit #13597 Phase 3 — the minimal-diff call the audit era wants. Acceptance table live at both refs (9 anchors, positive controls non-zero); both edited lines ≤120 bytes; ratchet lines quoted green (960/1005 · 48/48); 9 derived families + nul-bytes green; governed posture held.

Noted for the record: the 2026-08-28 ruling's carrier issue #12887 is measured UNREACHABLE (404 on issue page and REST — appears deleted). The ruling's single source in contract-review.md:10 carries it; the deleted-carrier fact goes to the audit's inputs.

Landing is the maintainer's: draft stays, hotlong requested. On merge this card closes and pm:dispatched gets stripped in the same sweep.

---

Generated by Claude Code

[zhuangjianguo]

os-dev-report

{
  "issue": 13557,
  "status": "done",
  "branch": "claude/issue-13557-no-prehang-correction",
  "pr": "https://github.com/objectstack-ai/objectstack/pull/13618",
  "premise_still_valid": true,
  "summary": "SKILL.md:758-759 duty sentence rewritten in place, net 0 lines: the dispatch seat's clause-2 duty now stops at recording the one-line determination on the card (记一行认定 survives verbatim), and needs:contract-review is attached by the PR creator in the same stroke as the reviewable contract increment (draft PR, or the report when it arrives first), with the 恒英文 label-naming note riding the hang sentence; the :760-761 abolition and open-carrier lines are untouched. contract-review.md:43 was read and adjudicated requirement-retention (no hang verb, no time point; its section is the tier fuse; carrier timing owned by :7-11 of the same file) and left untouched. All four SKILL.md label hits adjudicated in the PR body: :147 keep, :589 keep (requirement scope, not timing), :758 rewritten, :897 keep (the dispatch prompt said :889; measured position is :897, same director-seat passage — the file is byte-identical between the card ref 09b0d7b95 and base e2debee63). Card was assignee-empty at hand-off (PM-side half-state per protocol); the PM claim comment naming this session and branch was present, so the claim held.",
  "tests": "Derivation: node scripts/pm/dispatch-gates.mjs --repo objectstack-ai/objectstack — provenance line names the tree of objectstack-ai/objectstack at e2debee63; 9 families. Union re-run at final commit 7e5871d7a on a clean tree, exit codes captured before any pipe: agent-test-spelling 0 · doc-authoring 0 · doc-formula-expressions 0 (two PREREQUISITE NOT MET legs satisfied per the gate's own printed fix: built @objectstack/formula then @objectstack/lint, both through os-verify-lock) · pm-governed-merges 0 · pm-governed-prose 0 · pm-skill-id-lint 0 · pm-skill-ratchet 0 (own verdict line: '.claude/skills/pm-dispatch/SKILL.md is 960 lines (ceiling 1005; headroom 45)' — the card's at-most-960 met; and 'references/contract-review.md is 48 lines (ceiling 48; headroom 0)') · skill-frame-sync 0 · plus check:nul-bytes 0 added as the any-edit family. check-governed-queue-guard NOT MEASURED locally by its own design ('could not read GITHUB_EVENT_PATH … could not look must never exit 0 here') — it judges the CI event payload. Acceptance at both refs via git show: negative anchors 认定、挂 1→0 and 挂标仅当 1→0; positives 记一行认定 1→1, 禁止自查放行 1→1, 恒英文 2→2, 前瞻预挂已废止 1→1, 可复审契约增量 1→1, needs:contract-review 4→4; new spelling PR 创建者 0→1; every positive control non-zero at both refs (instrument proven). Both edited lines are 110 and 114 bytes under the 120-byte line rule, break placed between two CJK characters.",
  "line_budget": "SKILL.md 960 → 960 (net 0; card required at most 960). contract-review.md 48 → 48, byte-identical across refs — its 48/48 headroom-0 ceiling respected by not touching it.",
  "files_changed": [".claude/skills/pm-dispatch/SKILL.md"],
  "changeset": "none — skip-changeset applied to PR #13618 at open and read back (closed-list class: the diff is .claude/** only and publishes nothing from any package).",
  "governed_surface_posture": "draft PR only; no ready flip, no queueing, no auto-merge, no review submitted; landing is the maintainer's per Prime Directive 14. Label write: the additive REST POST is blocked at this seat's permission layer, so the measured MCP fallback ran — read current set (documentation, size/xs from the auto-labeler), whole-set union write, comparative read-back clean (union exact, nothing stripped, nothing lost). PR body read back in full: content byte-identical through the last authored line; the platform appended its own footer block after mine (known platform behavior, left alone).",
  "deviations": [
    "Commit trailer written as Co-Authored-By: Claude, without a model name — the default trailer carries one and the card's clause 5 forbids model IDs in anything pushed; the Claude-Session URL trailer is kept.",
    "The #12887 ruling thread is unreachable: issue page 404 AND repo-scoped REST 404 (appears deleted). The :43 adjudication therefore rests on the file's own section structure, the dispatch's stated fallback, not on the thread.",
    "Channel report: repo-scoped REST READS measured reachable from this seat (200 on an unauthenticated issues/13557 probe — differs from the 2026-08-25 all-403 measurement); REST WRITES via raw curl are blocked by the permission layer, so all GitHub writes went MCP.",
    "check:nul-bytes run in addition to the 9 derived families (any-edit obligation from the dev-agent definition)."
  ],
  "mcp_calls": "3 — create_pull_request, issue_write (label union), add_issue_comment (this report); every read rode the zero-quota issue-page payload or unauthenticated REST GET",
  "open_questions": [],
  "out_of_scope_findings": [
    "not filed — for PM triage: ruling-carrier issue #12887 returns 404 on both the issue page and REST (appears deleted). The 2026-08-28 ruling remains single-sourced at contract-review.md:10 per the dispatch adjudication, but the thread the mechanism hypothesis pointed at is gone.",
    "not filed — noted for audit #13597 Phase 3 (also recorded in the PR body): contract-review.md:43 「派发照旧恒保留」 could be sharpened to 恒欠 … 复审 (measured net minus-2 bytes on the line, inside its byte budget) to close the last mis-reading channel; left untouched under the minimal-governed-diff posture."
  ]
}

---

Generated by Claude Code