Repository navigation
docs(skills): objectstack-ui says placedActions is drawn, with an eval (#29951) - #30127
Merged
Merged
Conversation
…s_user header action draws again (#29951) Ruling A on #29951 (record 6624172200), ui half of #30018's spec key. - PageHeaderActions (@object-ui/components) reads page:header's placedActions: true and draws every action of the page's object placed at record_header / record_more, read live, through placedHeaderActions (@object-ui/types), the one placed-header reader RecordDetailView's synthesized header now reads too. Omitted or false, an authored actions list is still the whole set (#11592). A true beside a list draws none of the object's actions and is reported once. The page:header registration publishes the input. - sys_user_detail, sys_organization_detail and sys_approval_request_detail set placedActions: true in place of the id lists #27219 froze. Tab strips unchanged. - Drop the "not drawn yet" qualifiers: the describe marker (reference regenerated), the page liveness PLANNED record (re-graded live), the registry-inputs parity exemption, pages.mdx and layout-dsl.mdx. The skills/objectstack-ui/rules/pages.md qualifier goes in a separate PR (Tier H). Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WGufzh8VRbTbBnqwnrXsSL
…ions The unconsumed #30018 changeset rides the same release as this reader, so its "declared ahead of its reader" bullet now says the console's page:header renderer reads the key and the PLANNED marker is gone. The @objectstack/skills bullet stays until the skills follow-up. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WGufzh8VRbTbBnqwnrXsSL
Check Changeset refuses an edit to a changeset already on the merge base (the foreign-changeset rule, #17712), so the reworded bullet goes back. This PR's own changeset says the console reads placedActions; the stale sentence in 30018's entry is left to the release-time notes. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WGufzh8VRbTbBnqwnrXsSL
- rules/pages.md: the page:header catalogue row drops the "not drawn yet" qualifier; the console's header renderer reads placedActions. - evals/record-page.json: a third case asks for a full record page whose header shows every action packages place on the object; the answer sets placedActions: true on page:header with no actions list beside it. The evals README names it. - Changeset: @objectstack/skills patch. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WGufzh8VRbTbBnqwnrXsSL
Contributor
Author
|
domain:ui#1 · os-support-ai · session_01WGufzjDyeFwq6AuFu3vwTd 维护者速读(席位终稿)
席位核对 · ACCEPT · head
|
This was referenced Nov 4, 2026
Closed
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.
Requested by Yibo Wang · project thread
Refs #29951
维护者速读
objectstack-ui技能的rules/pages.md里,page:header那一行去掉了placedActions: true的“not drawn yet”限定语;evals/record-page.json新增一个评测,考placedActions: true的用法。pages.md在 token 上限处,改后 5668/5669,没有变长。回滚 = revert 本 PR。skills/objectstack-ui/rules/pages.md一行,skills/objectstack-ui/evals/record-page.json新增的第 3 个用例。node scripts/check-skills-token-ratchet.mjs,预期pages.md is 5668 tokens (ceiling 5669)、整体通过。skills/**是受管文件(Tier H),请你复核后批准合并。Clause-②: no
Metadata-protocol: none
What changes
skills/objectstack-ui/rules/pages.md: the component catalogue'spage:headerrow now reads`placedActions: true` (all placed, read live)in place of(all placed; not drawn yet). The console's header renderer reads the key since fix(console): the record header reads placedActions, so a packaged sys_user header action draws again (#29951) #30097 (PageHeaderActions, throughplacedHeaderActions), so the qualifier feat(spec): page:header placedActions draws every placed header action, refused beside an actions list (#30018) #30028 wrote is dropped. This is the sixth of the six qualifiers feat(spec): page:header placedActions draws every placed header action, refused beside an actions list (#30018) #30028 listed; fix(console): the record header reads placedActions, so a packaged sys_user header action draws again (#29951) #30097 dropped the other five.skills/objectstack-ui/evals/record-page.json: a third case. The prompt asks for a customkind: 'full'account record page whose header shows every action packages place onaccount, without listing ids. The expected answer keeps the page and its tab strip and sets{ type: 'page:header', properties: { placedActions: true } }, with noactionsid list beside it (the two are refused together). Assertions: must containplacedActions: trueandpage:header; must not containslots.tabsoractions: ['.skills/objectstack-ui/evals/README.md: therecord-page.jsonline names the new case..changeset/29951-skills-placed-actions-drawn.md:@objectstack/skillspatch.Why
#30028's Skills-docs line deferred both pieces to the reader: "no eval exercises it while no renderer reads it, so the eval and the qualifier drop ride with the reader." The reader landed in #30097. The seat split this Tier H file out of #30097 so that PR could land through the queue.
Verification
node scripts/check-skills-token-ratchet.mjs: exit 0;skills/objectstack-ui/rules/pages.md is 5668 tokens (ceiling 5669; headroom 1), so the row costs one token less than before.node scripts/pm/dispatch-gates.mjs --commandson this diff: 42 commands, all run underos-verify-lock(VERDICT command-exit 0), every one exit 0;--ran: 42 derived, 42 run, 0 NOT-MEASURED, 0 UNRUN. They include the token ratchet and its self-test,check:pm-skill-id-lint,check:skill-compatibility,check:skill-frame-syncandcheck:skill-identifier-liveness.mainat a52761f (the fix(console): the record header reads placedActions, so a packaged sys_user header action draws again (#29951) #30097 squash) merged in, plus this one commit;git diff origin/mainbefore the commit was empty.Human Minimum
packages/specholds no runtime logic. (Not touched.)pnpm lintpasses. (No TypeScript changed; CI'spnpm lintcovers the tree.)main, one pull request.packages/spectouched: its generated artifacts are regenerated and committed. (Not touched.)content/docs/references/,packages/*/CHANGELOG.mdorcontent/docs/releases/.## 维护者速读block, which is written in Chinese for the maintainer.🤖 Generated with Claude Code
https://claude.ai/code/session_01WGufzh8VRbTbBnqwnrXsSL