skills(objectstack-data): re-lock field-types.md token ceiling, drop SKILL.md index blurb - #16601
Merged
Merged
Conversation
…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
Collaborator
Author
维护者速读(终稿 · skills 席,2026-09-07T12:1xZ)
Generated by Claude Code |
os-zhuang
approved these changes
Sep 7, 2026
os-zhuang
marked this pull request as ready for review
September 7, 2026 12:10
os-zhuang
enabled auto-merge
September 7, 2026 12:10
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #16374
What
Both PRs that were shrinking
skills/objectstack-data/rules/field-types.mdhave merged— PR #16187 (the two
treerows, −11 tokens) and PR #16369 (the ASCII decision treedeleted, −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 forskills/objectstack-data/rules/field-types.mdre-locks from3584to3032— thecount 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)
rules/field-types.md— measured tokensrules/field-types.md— ceilingrules/field-types.md— headroomSKILL.md— measured tokensSKILL.md— ceilingSKILL.md— headroomPackage-level (
skills/objectstack-data/**, sum of every tracked file's measured tokens,authored + generator-owned): 37901 → 37896 (−5, from the
SKILL.mdedit 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-testalone)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 nowprints
... 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.mjs— exit 3:skills/** ×1governs the wholePR (
skills/objectstack-data/SKILL.md); the ratchet script itself is not on theregister — one PR, one regime, as the acceptance criteria call for.
Merge-tree probe (driver-less bare clone, no
merge.os-regen.driverregistered)origin/main(de0bcdd4): clean,write-treesucceeded, noconflicts listed.
skills/objectstack-data/references/data-hooks.md— a disjoint file in the same package) vs this branch: clean,
write-treesucceeded, noconflicts listed.
Serial preconditions re-checked
treerows state the landedreferencecontract #16187 and PR docs(skills): drop the duplicated ASCII field-type decision tree from objectstack-data rules #16369 are both merged onorigin/main(read by squash subject).Blocked-by: #16324is discharged (skills(objectstack-data):rules/field-types.mdcarries a ~1800-byte ASCII decision tree that duplicates the docs' mermaid page, in a file 17 tokens under its ceiling #16324 closed with PR docs(skills): drop the duplicated ASCII field-type decision tree from objectstack-data rules #16369).treerows inrules/field-types.mdandrules/relationships.mdstate the landedreferencecontract — optional, and the object's own name when present (#14892 follow-up) #16083 (the tree-row follow-up that fed PR skills(objectstack-data): thetreerows state the landedreferencecontract #16187) — re-read: CLOSED / COMPLETED.维护者速读(草稿)
改了什么 — 把
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