skills(pm-dispatch, os-dev, AGENTS.md): finding admission threshold — only a reproducible defect, a contract violation, or a metadata-authoring trap becomes a card; everything else rides the PR's acceptance notes (maintainer-directed, both repos)

[claude]

Maintainer-directed (skills seat, 2026-09-06T15:0xZ). Governing text — the maintainer's instruction in the seat's chat, verbatim: 「这个问题我觉得也需要，两个仓库都需要：finding 准入门槛（源头减流，一张 skills 卡）：只有「可复现缺陷、契约违背、会让 AI 写错元数据」三类才立卡；纯观察、风格、文档 nit 写进 PR 验收评论即可。改 os-dev.md 与 SKILL.md 的立卡规则，受管面走草案 PR。预期入流减半。」

The measurement behind it (objectui, 2026-09-06T14:15Z, read by the seat)

reading	value
open issues	411
of which finding	189
domain:ui pm:queue	124, of which finding 79, priority:p3 56
new cards filed 09-05 / 09-06 (to 14:15Z)	56 / 83
open finding cards created 09-05 / 09-06	46 / 56
cards closed 09-05 / 09-06	78 / 102

Throughput is not the problem — inflow is, and the inflow is manufactured by the fleet itself: every dev flight files one to three out-of-scope findings, nearly all graded p3, and the queue only grows. The rule that produces this is written in five places and none of them carries an admission threshold.

Where the rule lives today (objectstack main; objectui carries NO .claude/agents/ and no finding rule of its own — its seats and devs read these same files, so one landing covers both repos)

• .claude/agents/os-dev.md — 范围 rule: 「顺路撞见的无关缺陷立成新的无 assignee issue」 and 「观察类发现(死代码、未演练漂移、外观抛光)打 finding 标签,⛔ 不打 pm:queue」— every observation becomes a card.
• .claude/skills/pm-dispatch/SKILL.md — the finding label row (「观察类记录,恒 = 待首次定级」), the 分类 line (「finding = 观察类(死代码、未演练漂移、抛光;真实但今天没有用户撞上),待首次定级」), the 发现分诊轮 block (「每 fire 定完全部未定级 finding」), and the report-reading lines (「out_of_scope_findings 应已由 dev 立成无 assignee 的卡」/「范围外发现的标签按种分:观察类打 finding」).
• references/core-rules.md — the digest line 「观察类记录打 finding 待首次定级」 and 「范围外发现须已立卡」.
• AGENTS.md Prime Directive 10 — 「File issues for out-of-scope findings — don't silently expand scope or leave them buried」.

The rule to land

1. Admission: a card is filed for exactly three classes — (a) a reproducible defect (repro or failing probe named), (b) a violation of a declared contract (the contract text cited), (c) a trap that would lead an AI authoring metadata to write something the runtime refuses or silently drops. Nothing else becomes a card.
2. Everything else rides the PR — observations, dead code, drift not yet exercised, polish, style, doc nits, naming: the dev lists them in a ## 验收备注 (acceptance notes) section of its PR body and in the report's out_of_scope_findings as noted, not filed: …; the seat reads them at ACCEPT. A noted item that is in fact class (a)–(c) is filed by the seat at ACCEPT, not by the dev.
3. The seat's grading of a filed finding outside the three classes is one stroke: close not_planned, the reason on the card; the 发现分诊轮 stops carrying observation-class cards.
4. AGENTS.md Prime Directive 10 keeps its intent (never bury a defect, never expand scope) and states the threshold in one sentence; the corollary (declared ≠ enforced) is untouched.
5. The finding label keeps its meaning for what is filed (待首次定级); the card-body requirement for class (a)–(c) is the evidence named above, so a triage seat can refuse a finding that carries none.

Acceptance

• The four texts above say the same rule in their own registers; references/core-rules.md's digest line matches SKILL.md in the same PR (the one-rule-two-places rule).
• Line-neutral on every pinned file (SKILL.md 811, os-dev.md 403, core-rules.md 150, AGENTS.md 1058 — folds inside the same sections); rules-only register.
• check-skill-line-ratchet, check-skill-id-lint, check-skill-frame-sync, check-governed-prose, check-half-states --self-test (no H-row pins a sentence touched), check-governed-merges --test exit 3, whole-repo lint — every exit captured before any pipe.
• Governed (.claude/**, AGENTS.md): draft PR, seat review at contract tier, os-zhuang + hotlong, needs-user-decision + 维护者速读, human merge.
• Expected effect, to be re-measured one week after landing on both boards: new finding cards per day at or below half of the 09-05/09-06 rate.

Serial plan (recorded at filing)

SKILL.md + core-rules.md are held by flight A (PR #16339, at CI) and by the devx seat's PR #16271; os-dev.md is held by flight B (no PR yet). The SKILL.md / core-rules / AGENTS.md parts dispatch as one flight the moment a slot frees, with a merge-tree proof against both open PR heads (the shape #16003 used against PR #15946); the os-dev.md part rides the flight that follows flight B on that file, folded with #16262. The selection-order card filed in the same stroke folds into the SKILL.md half (same file, same register, no conflicting ruling).

Refs #16262.

---

Generated by Claude Code

[claude]

Claim: PM loop round 8 (continued) — flight I, the maintainer-directed finding admission threshold (this card's governing text is the maintainer's instruction quoted in the body), PARTIAL: the SKILL.md + references/core-rules.md + AGENTS.md half. The .claude/agents/os-dev.md half is NOT taken in this flight — that file is held by flight B (#14768) and the os-dev.md sentence rides the flight that follows B on it, folded with #16262; the PR carries Refs #16351 (never Fixes), and at its landing this seat releases the card for the remaining half. #16352 (priority in 取卡全序) is NOT folded in: its line sits in the 候选与批次 block that PR #16271 (devx) is rewriting, so it waits for that PR and PR #16339 to land.
Session: session_019RfFHiRCSs3JXLK4cwcfox
Branch: claude/issue-16351-finding-admission-threshold
Worktree: objectstack-issue-16351
Domain: domain:skills
File surface: .claude/skills/pm-dispatch/SKILL.md (811/811, pin 342 — the finding label row, the 分类 line, the 发现分诊轮 block, the two report-reading lines; line-neutral by folds in the same sections), references/core-rules.md (150/150, two digest lines), AGENTS.md (1058/1058, Prime Directive 10's first sentence). ⛔ No other file; ⛔ not the 候选与批次 block, not the six lines flight A changed (81 · 534 · 538 · 543 · 637 · 647).
Container & model: S, mode:subagent, contract tier — dispatch-gates.mjs --tier: SKILL.md hits clause ① (MANDATORY); launched at the contract tier, the measured quota exemption (fable unavailable ⇒ opus, never lower; recorded here) applies only if the launch is refused.
Clause-②: no
Serial: SKILL.md and core-rules.md are held by two open drafts at the governed terminal — PR #16339 (flight A, d88c29c76a) and PR #16271 (devx, 4d3194ba74), which conflict with each other on the 候选与批次 block; this flight edits neither's lines and must prove git merge-tree --write-tree clean against BOTH heads before opening its PR (the shape #16003 used against PR #15946). GOVERNED ⇒ draft PR, in-seat review at contract tier, os-zhuang + hotlong, human merge, skip-changeset; at ACCEPT the seat adds needs-user-decision + the 维护者速读.

---

Generated by Claude Code

[claude]

Claim: PM loop round 8 (continued) — flight M, the SECOND half of this card: the .claude/agents/os-dev.md sentence (the dev's out-of-scope rule — three classes file a card; the rest is noted, not filed in the report and listed under the PR's ## 验收备注), folded with #16262 on the same file. The first half (SKILL.md · core-rules.md · AGENTS.md) is flight I on claude/issue-16351-finding-admission-threshold, in flight, Refs #16351. This flight's PR carries Fixes #16351.
Session: session_019RfFHiRCSs3JXLK4cwcfox
Branch: claude/issue-16262-os-dev-lock-discipline-and-finding-threshold
Worktree: objectstack-issue-16262
Domain: domain:skills
File surface: .claude/agents/os-dev.md only.
Container & model: S, mode:subagent, contract tier (MANDATORY on this path).
Clause-②: no
Serial: see #16262's claim — merge-tree proof against flight B's head f5c08e9589.

---

Generated by Claude Code

[claude]

os-dev-report

{
  "issue": 16351,
  "status": "done",
  "branch": "claude/issue-16351-finding-admission-threshold",
  "pr": "https://github.com/objectstack-ai/objectstack/pull/16368",
  "premise_still_valid": true,
  "summary": "HALF DELIVERED: the SKILL.md + references/core-rules.md + AGENTS.md half of the finding admission threshold. A card is filed for exactly three classes — (a) a reproducible defect (repro or failing probe named), (b) a violation of a declared contract (contract text cited), (c) a trap that leads an AI authoring metadata to write what the runtime refuses or silently drops; everything else goes under a `## 验收备注` section of the PR body and into `out_of_scope_findings` as `noted, not filed`, the seat reads it at ACCEPT and files only what is in fact class (a)-(c), and a filed finding outside the three classes is closed not planned in one stroke by the grading seat; the `finding` label keeps its 待首次定级 meaning and a three-class card without evidence is refused. Sites (base b9a14dd64f): SKILL.md `finding` row 116, 分类 354 (1 → 2 lines, paid by the 387–388 发版板 fold: 「细则见」→「见」), 发现分诊轮 377–379 (3 → 3), 报告契约 778–779 (2 → 2); core-rules.md 89 and 150; AGENTS.md Prime Directive 10 lines 211–213 (corollary from mid-213 to 216 byte-identical). Line-neutral 811/811 (table pin 342/342), 150/150, 1058/1058; every changed line ≤120 bytes; rules-only register. HALF LEFT: `.claude/agents/os-dev.md` (held by flight B, rides the flight that follows it) — the PR carries `Refs #16351`, the card stays open. Merge-tree vs d88c29c76a (PR #16339) and 4d3194ba74 (PR #16271): both exit 0, 0 CONFLICT lines; neither head's lines touched (the four-axis frame, 候选与批次, and lines 81/534/538/543/637/647 untouched). Base moved from the brief's 3e270d4e29 to b9a14dd64f; the only drift on the three files was AGENTS.md line 758 (examples row), not PD 10. Draft PR #16368, `skip-changeset` via the additive labels endpoint with a comparison read-back (size/s + skip-changeset, nothing stripped); PR body read back byte-identical apart from the platform's appended session footer. The claim comment 5560108723 names this branch and session; no second claim posted; no assignee or label touched on the card.",
  "tests": "All at ea4cd7c0e4 (the pushed head), every exit captured to a file before any pipe, verdict lines quoted from the gates. Gate derivation: `node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstack` on the three paths → 18 commands; the no-path derivation off the merge base is identical; `--ran` → `✓ dispatch-gates --ran: 18 derived famil(ies) accounted for — 18 run, 0 NOT-MEASURED.` `pnpm check:pm-skill-ratchet` exit 0: `✓ check-skill-line-ratchet self-test: 155 cases pass.` · `SKILL.md is 811 lines (ceiling 811; headroom 0)` · `SKILL.md: widest table row is 342 bytes (pin 342; headroom 0)` · `core-rules.md is 150 lines (ceiling 150; headroom 0)` · `AGENTS.md is 1058 lines (ceiling 1058; headroom 0)`; before at base b9a14dd64f (wc -l): 811/150/1058, widest row 342. Exit 0 each: check:pm-skill-id-lint, check:skill-frame-sync, check:pm-governed-prose, check:pm-governed-merges, check:pm-half-states (`✓ check-half-states self-test: 2535 cases pass.`), check:nul-bytes, check-closing-keyword-parity (+ --self-test), check-comment-mask-corpus, check-governed-queue-guard --self-test, check:agent-test-spelling, check:watch-hint-literal, check:refd-timer-probe, check:required-contexts, check:doc-authoring, check:docs-audit-scope, check:driver-memory-census. `pnpm --filter @objectstack/lint run check:doc-formula-expressions`: first run exit 3 PREREQUISITE NOT MET (unbuilt @objectstack/formula) = NOT MEASURED; after `turbo run build --filter=@objectstack/formula --filter=@objectstack/lint` under the lock (exit 0) the re-run is exit 0 (`9 @example(s) judged clean across 1297 packages/spec/src files`). H-row sweep: 14 removed/rewritten sentences grepped fixed-string over scripts/ and .claude/hooks/ → 0 hits each; positive control `认领 >~24h` → 2 hits. `check-governed-merges.mjs --test` on the three paths → exit 3 (GOVERNED). Whole-repo `pnpm lint` (eslint . --no-inline-config) via `bash scripts/pm/os-verify-lock.sh` slot issue-16351, one acquisition (waited 0s, held 106s shared-box) → `lint exit=0`. Diff scan: 0 model identifiers, 0 control bytes. Merge-tree: `git merge-tree --write-tree ea4cd7c0e4 d88c29c76a` exit 0 / 0 CONFLICT; `… 4d3194ba74` exit 0 / 0 CONFLICT. CI on ea4cd7c0e4 (foreground poll of the head's check runs): all 29 check runs on the head completed by 15:57Z — 17 success, 12 skipped, 0 failure/cancelled/timed_out; Lint & Repo Gates (the 155-step gate farm) success; Check Changeset success under skip-changeset",
  "mcp_calls": "0 — no MCP GitHub calls this run; the card was read through the public payload channel and confirmed through REST (repo-scoped read probe HTTP 200, core 15000/15000), and the PR, label and report comment were written through REST",
  "open_questions": [],
  "out_of_scope_findings": [
    "noted, not filed: references/dispatch-runbook.md line 31 「范围外发现照旧单开」 (sweep-batch context) now reads as 'filed per the admission threshold rather than folded into the sweep PR' — a wording drift not yet exercised, outside this flight's declared file surface; not a card under this very rule",
    "noted, not filed: .claude/agents/os-dev.md 范围 rule still states the old admission (every observation becomes a card) — the other half of this card, held by flight B; not a finding"
  ]
}

---

Generated by Claude Code