Repository navigation
docs(adr): ADR-0086 authz metadata↔config boundary + cross-package composition - #2559
Merged
Merged
Conversation
…mposition (#2557) Draws the model-wide line between authorization primitives that are package metadata (versioned, shipped, draft/published, portable) and those that are environment/system-admin config (live, env-specific, bound to real subjects), and specifies how grants from several installed packages compose. Key decisions: - D1/D2 Classification on the DEFINITION↔ASSIGNMENT axis, recorded on the existing `metadata-persistence.managedBy` provenance axis (no new axis). - D3 permission set carries owning `packageId` + per-record `managedBy`. - D4 composition = package ships its own sets (Shape B); runtime unions; shared-slice (Shape A) stays an env-admin-only construct. - D5 add `bootstrapDeclaredPermissions` to seed `stack.permissions` (closes the ADR-0078 inert-metadata gap; operationalizes ADR-0066 D5). - D6/D7 package Access door = draft/published metadata scoped to the package slice; env-admin door = live cross-package matrix + assignment. Two doors, one metadata. - Phased plan P0 (objectui scope + slice-merge), P1 (framework seeding), P2 (two doors + overlay/mute subtract). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KfK8SFcfhbfgE2WxX8JQDp
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
15 of 18 tasks
Points readers from this ADR's package-authoring/composition slice to the whole-lifecycle authorization gap map (P0–P3) tracked in #2561. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KfK8SFcfhbfgE2WxX8JQDp
os-zhuang
marked this pull request as ready for review
July 4, 2026 12:03
os-zhuang
added a commit
that referenced
this pull request
Jul 16, 2026
…hen + role-gated option (#2559) (#3006) Adds `showcase_cascade`, a served fixture that exercises the B3 server-side option-value enforcement (objectql `evaluateOptionVisibility`, #2915 / #1583): - country → province cascade — `province` declares `dependsOn: ['country']` and each option gates itself with a `visibleWhen` CEL predicate. The client narrows the offered set; the rule-validator rejects an out-of-set submit (`{ field, code: 'invalid_option' }`). - a role-gated `tier.restricted` option (`'admin' in current_user.positions`), enforced server-side the same way. `public_read_write` (no permission set → absent from the ADR-0090 access-matrix), wired into the Showcase app's Data Model nav. Verified live against a fresh backend: POST /api/v1/data/showcase_cascade rejects country=us+province=zj (400 invalid_option) and accepts country=cn+province=zj (201); GET /api/v1/meta/object/showcase_cascade serves the per-option visibleWhen to the client. Drives objectui e2e/live/cascading-options.spec.ts. Claude-Session: https://claude.ai/code/session_01S91NyYJURiQTKmF9q3AXxg Co-authored-by: Claude <noreply@anthropic.com>
This branch was successfully deployed
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.
Closes the design/ADR task in #2557.
Adds ADR-0086 (
docs/adr/0086-authz-metadata-config-boundary-and-cross-package-composition.md) — the model-wide decision on which authorization primitives are package metadata vs environment/system-admin config, and how permissions from multiple installed packages compose.What it decides
metadata-persistence.managedBy: package|platform|userprovenance axis — no new axis (PD Convert to monorepo with scoped packages #8).managedBy:'package'+ owningpackageId, seeded on install by a newbootstrapDeclaredPermissions(the missing sibling ofbootstrapDeclaredRoles) that migrates the already-declarablestack.permissionsintosys_permission_set. Runtime composes by the existing most-permissive UNION, conflict-free thanks to package-namespaced object api names. The "one shared set with disjoint slices" shape stays an env-admin-only construct. Closes the ADR-0078 inert-metadata gap and operationalizes ADR-0066 D5.Phased plan (each independently shippable)
{ packageId }+ slice-merge on save; closes the "84 objects" leak and the data-loss trap (mirrors objectui#2197).PermissionSetSchema.packageId+bootstrapDeclaredPermissionsso packages ship default access for their own objects.Scope
Docs-only (one new ADR file, 351 lines). No code, spec, or runtime changes — the phased implementation is deferred to follow-up PRs per the plan above.
🤖 Generated with Claude Code
https://claude.ai/code/session_01KfK8SFcfhbfgE2WxX8JQDp
Generated by Claude Code