Skip to content

skills(objectstack-data): re-lock field-types.md token ceiling, drop SKILL.md index blurb - #16601

Merged
os-zhuang merged 1 commit into
mainfrom
claude/issue-16374-field-types-token-relock
Sep 7, 2026
Merged

skills(objectstack-data): re-lock field-types.md token ceiling, drop SKILL.md index blurb#16601
os-zhuang merged 1 commit into
mainfrom
claude/issue-16374-field-types-token-relock

Conversation

@os-musk

@os-musk os-musk commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

Fixes #16374

What

Both PRs that were shrinking skills/objectstack-data/rules/field-types.md have merged
— PR #16187 (the two tree rows, −11 tokens) and PR #16369 (the ASCII decision tree
deleted, −536 tokens). This PR executes the ruling on PR #16369's second open question:
a lowered ceiling is the ratchet's ordinary direction and a seat's re-lock, not a
maintainer ruling.

  • scripts/check-skills-token-ratchet.mjs: the ceiling for
    skills/objectstack-data/rules/field-types.md re-locks from 3584 to 3032 — the
    count the gate itself measures on origin/main (de0bcdd4), not a computed number.
  • skills/objectstack-data/SKILL.md: the index line for that file drops the phrase
    "with decision tree" (three words, not four as the issue text says — the ASCII block
    it referred to no longer exists either way) so the line reads "All 49 field types and
    configs".

Nothing else in either file moves.

Line budget (measured, not computed)

file before after
rules/field-types.md — measured tokens 3032 3032 (unchanged)
rules/field-types.md — ceiling 3584 3032
rules/field-types.md — headroom 552 0
SKILL.md — measured tokens 10004 9999
SKILL.md — ceiling 10009 (unchanged) 10009
SKILL.md — headroom 5 10

Package-level (skills/objectstack-data/**, sum of every tracked file's measured tokens,
authored + generator-owned): 37901 → 37896 (−5, from the SKILL.md edit only —
field-types.md's measured count is unchanged, only its ceiling moved).

Whole-bundle total the gate prints (bundle total (whole shipped tree)):
150527 → 150522 (−5).

Acceptance (all measured on this branch, not --self-test alone)

  • node scripts/check-skills-token-ratchet.mjs --self-test — exit 0 (64 cases).
  • node scripts/check-skills-token-ratchet.mjs (live) — exit 0; the file's line now
    prints ... is 3032 tokens (ceiling 3032; headroom 0).
  • node scripts/pm/check-governed-merges.mjs --test skills/objectstack-data/SKILL.md scripts/check-skills-token-ratchet.mjsexit 3: skills/** ×1 governs the whole
    PR (skills/objectstack-data/SKILL.md); the ratchet script itself is not on the
    register — one PR, one regime, as the acceptance criteria call for.

Merge-tree probe (driver-less bare clone, no merge.os-regen.driver registered)

Serial preconditions re-checked

维护者速读(草稿)

改了什么 — 把 rules/field-types.md 的 token 上限从 3584 重锁到 3032(门禁实测值,不是算出来的),因为压缩这份文件的两张 PR(#16187#16369)都已合入,上限跟着落地字节数往下收。同一 PR 里,SKILL.md 索引那一行删掉"with decision tree"这个短语,因为它指向的 ASCII 决策树段落已经不存在了。

为什么改 — 棘轮的正常方向就是"降到落地计数",不需要维护者裁决,是席位自己就能执行的常规操作;裁决已经在 PR #16369 验收时做过一次了,这次只是照办。

风险与代价(含回滚) — 风险很低:改动只碰这两处,门禁(自测 + 实况)都绿,check-governed-merges --test 按预期红成 exit 3(governed surface,需要人工合并)。回滚方式是把这一个 PR revert 掉,上限与索引行都恢复原样,不影响其它文件。

席位意见 — (留空,由验收席位在 ACCEPT 时填写)

你要做的 — 这是 governed surface(skills/**),按规矩人工合并,不走 auto-merge / 合并队列;skip-changeset 已经打上(这个 PR 不发布任何包)。

Generated by Claude Code


Generated by Claude Code

…index blurb

The token ceiling for skills/objectstack-data/rules/field-types.md in
scripts/check-skills-token-ratchet.mjs re-locks from 3584 to 3032, the
count the gate itself measures on origin/main now that both PR #16187
(-11 tokens, the two `tree` rows) and PR #16369 (-536 tokens, the ASCII
decision tree deleted) have landed. Lowering a ceiling to the landed
count is the ratchet's ordinary, seat-executable direction.

skills/objectstack-data/SKILL.md's index line for the file drops the
four words "with decision tree" — the block it referred to no longer
exists.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018dxq7YqsLDMeZDZ5AzsgJX

os-musk commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator Author

维护者速读(终稿 · skills 席,2026-09-07T12:1xZ)


Generated by Claude Code

@os-zhuang
os-zhuang marked this pull request as ready for review September 7, 2026 12:10
@os-zhuang
os-zhuang enabled auto-merge September 7, 2026 12:10
@os-zhuang
os-zhuang added this pull request to the merge queue Sep 7, 2026
Merged via the queue into main with commit 1aaf0e7 Sep 7, 2026
40 checks passed
@os-zhuang
os-zhuang deleted the claude/issue-16374-field-types-token-relock branch September 7, 2026 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation needs-user-decision size/xs skip-changeset PR has no user-facing published change; bypasses the changeset gate

Projects

None yet

3 participants