Repository navigation
Commit 77392e9
docs(os-dev): name the harness attribution-line exemption in 〈干净收尾〉; Check Changeset reads the PR body (#25393)
Fixes #25220
Clause-②: no
Metadata-protocol: none
`.claude/agents/os-dev.md` only, net zero lines (404 → 404; ceiling 404
untouched). One exemption line under 〈干净收尾〉's HTML-comment ban: the
harness-written attribution comment on PR body line 1 stays exactly as
the harness gives it and the dev reports it; `Fixes #N` then sits on the
first line after it; every HTML comment the dev writes itself stays
banned. Paid by deleting the `cdn.playwright.dev` 403 line (moot under
the `playwright install` ban directly above it). Carry-over from PR
#25240's contract review: the changeset-line rule now says `Check
Changeset` reads the two declaration lines off the PR body
(`scripts/check-changeset-no-major.mjs` `declarationFromPullRequest`,
`readClause2Line(pr.body)` / `readMetadataProtocolLine(pr.body)`); the
changeset-body copy is AGENTS.md Post-Task Checklist ③'s publishing rule
(`check:adr-0087-registration` reads it there). Every changed line is
≤120 bytes; no re-wrap.
## 维护者速读(草稿)
**改了什么**:`os-dev.md` 两处。① 〈干净收尾〉HTML 注释禁令下加一行例外:harness 自写的 PR
正文首行归属注释原样留、只报;`Fixes #N` 紧随其后;dev 自写的 HTML 注释仍禁。② 「两行同抄进 changeset
正文行首」一行改为写明 `Check Changeset` 读的是 PR 正文。为付行数,删掉 Toolchain traps 里
`cdn.playwright.dev` 403 那一行(上一行已禁 `playwright install`,该情形不再出现)。
**为什么改**:项目线程的 harness 要求 PR 正文第一行是一条 HTML 注释归属行,与 os-dev.md 的 HTML
注释禁令必然冲突,此前每张卡都只能记一条 deviation。② 来自 PR #25240 的合同复核:门禁实际读 PR
正文(`check-changeset-no-major.mjs`),原句写成读 changeset,会误导。
**风险与代价(含回滚)**:仅规则文本,不发布、不改脚本。风险是删掉的 403 一行所载的教训丢失(被上一行规则覆盖)。回滚 = revert
本 PR 一个 commit。
**席位意见**:
**你要做的**:席位的 `CONTRACT_REVIEW_TIER` PASS 落在本 head
之后,一句「objectstack-ai/objectstack#N 可以落地」(N 为本 PR 号)。
## Gates
Derived with `node scripts/pm/dispatch-gates.mjs --commands --repo
objectstack-ai/objectstack` at `fc7b53ff` (20 commands); all 20 ran on
the working tree that became `da46daf`, each exit code captured before
any pipe; `--ran` reconciliation: 20 derived, 20 run, 0 NOT-MEASURED, 0
UNRUN. All exit 0, including `pnpm check:pm-skill-ratchet` (`os-dev.md
is 404 lines (ceiling 404; headroom 0)`) and `pnpm
check:pm-skill-id-lint`. `check:doc-formula-expressions` exit 0 (no
prerequisite build needed). CI-only families (type-check lanes,
wide-population and roster families) are read off this PR's check-runs.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
https://claude.ai/code/session_01REMuzhz6MWRjg8uohedMnM
Co-authored-by: Claude <noreply@anthropic.com>1 parent 24bbd0b commit 77392e9
1 file changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
125 | 125 | | |
126 | 126 | | |
127 | 127 | | |
128 | | - | |
129 | 128 | | |
130 | 129 | | |
131 | 130 | | |
| |||
278 | 277 | | |
279 | 278 | | |
280 | 279 | | |
281 | | - | |
| 280 | + | |
282 | 281 | | |
283 | 282 | | |
284 | 283 | | |
| |||
321 | 320 | | |
322 | 321 | | |
323 | 322 | | |
| 323 | + | |
324 | 324 | | |
325 | 325 | | |
326 | 326 | | |
| |||
0 commit comments