Repository navigation
docs(adr-0056): AdminScopeSchema target shape lists only the live keys - #12903
Conversation
Co-authored-by: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XVp19E1MYUTFM3dnkwz8gB
…itor still carries Co-authored-by: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XVp19E1MYUTFM3dnkwz8gB
Contract reviewServed-tier: Re-review of the moved head (supersedes the record on ① Derived judgments
② Semver levelDocs-only diff: no ③ Boundary flags
Implemented-by: VERDICT: PASS Generated by Claude Code |
维护者速读 · objectui PR #12903 ·
|
Fixes #12660
Clause-②: no
ADR-0056's
AdminScopeSchemabullet listedmanageBindingsandauthorEnvironmentSetsas part of the "exact target shape" for theadmin_scopeeditor. objectstack-ai/objectstack#23488 retired both keys. This amends that bullet: it now lists the four live keys and states in one clause that the two keys were retired by that change and are refused by the spec. A second place in the same file, the P3 implementation-log entry, also named the -bindings / author-env-sets toggles; it now names only the controls the editor still carries (business-unit + subtree, manage-assignments toggle,assignablePermissionSets[]) and notes the other two were removed with objectui#12659 when the spec retired those keys (verified: #12659 removed both controls fromPermissionAdvancedFacets).Measurement
Installed
@objectstack/spec18.0.0-next.3,AdminScopeSchema.shapekeys:businessUnit,includeSubtree,manageAssignments,manageBindings,authorEnvironmentSets,assignablePermissionSets. The last-but-one pair are tombstones:safeParse({ businessUnit: 'x', manageBindings: true })and theauthorEnvironmentSetsequivalent both fail with aninvalid_type(expectednever) issue saying the key "was removed in @objectstack/spec 18"; the bare{ businessUnit: 'x' }parses. The live set is exactlybusinessUnit,includeSubtree,manageAssignments,assignablePermissionSets, with no other drift versus the bullet.Gates
node scripts/check-changeset-presence.mjs: exit 0 ("no changeset is owed"; docs-only diff, no changeset added).pnpm check:doc-example-readers: exit 0.pnpm check:new-line-citations: exit 0 (0 new citations); both re-run on head 2e2c31b, check-changeset-presence exit 0.scripts/check-nul-bytes.mjsdoes not exist in this repo (MODULE_NOT_FOUND, not a gate verdict).维护者速读(草稿)
改了什么
ADR-0056 里描述「委托管理范围(admin_scope)编辑器目标形状」的那一条,去掉了已被规格退役的两个字段(manageBindings、authorEnvironmentSets),只保留现行的四个字段,并加一句话说明这两个字段已由 objectstack#23488 退役。同一文件里的第二处(P3 实施记录里「manage-assignments / -bindings / author-env-sets 开关」的括号)也已改为只写编辑器现在实际提供的控件,并注明另两个开关随 objectui#12659 移除。
为什么改
规格已经拒收这两个字段,但 ADR 仍把它们当作「精确的目标形状」。照着 ADR 做编辑器的人(或 AI)会去实现一个规格必然拒绝的字段。
风险与代价(含回滚)
只改 ADR-0056 一个文件里的两处文字,不影响运行时与发布包,无需 changeset。回滚即还原这两处。
席位意见
你要做的
批准此 PR(approve)。
Generated by Claude Code