Repository navigation
docs(adr): ADR-0073 amendment and ADR-0096 row for the system flow's refused identity-assignment writes - #25587
Merged
Merged
Conversation
…refused identity-assignment writes ADR-0073 gains a dated Amendment: a runAs system run's governed writes to sys_user_position, sys_user_permission_set, sys_invitation and sys_member are refused with PERMISSION_DENIED, so M2's "stays god-mode throughout" holds with that one exception. The #3760 amendment's sys_user_position example gains a pointer to it. ADR-0096's surface table row for system flows names the carve-out. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013yig39ZouqWBxABRfM5FN4
os-zhuang
approved these changes
Oct 24, 2026
os-zhuang
marked this pull request as ready for review
October 24, 2026 02:13
os-zhuang
enabled auto-merge
October 24, 2026 02:14
Contributor
Author
维护者速读
席位核对 · ACCEPT ·
|
This was referenced Oct 24, 2026
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 Tesla · project thread
Part of #25554
The ADR half of ruling 6354966423 (letter B) on #25251, the follow-up of card #25554. #25483 made a
runAs: 'system'flow's governed writes to the four identity-assignment objects refused, and #25564 put that into theFlowSchema.runAsdescription and the ADR-0087 D3 ledger. Two ADRs still describe system flows as unconditionally elevated.docs/adr/**is a governed path, so these edits come as their own PR for a maintainer merge.维护者速读(草稿)
runAs:'system'流程对岗位持有、权限集授予、邀请、成员这四张表的写入会被拒绝(PERMISSION_DENIED),M2 里「system 始终是 god-mode」一句按这个例外来读;The #1888 user-less fail-open is wider than the lint that guards it — record-change flows fired by a system write run UNSCOPED, unlinted #3760 那节里的sys_user_position例子补一句指向新 Amendment。ADR-0096 的执行面表格里 system 流程那一行补上这个例外。What changed
## Amendment:section (2026-10-24, Security: a runAs system flow writes identity-assignment rows (sys_user_position, sys_invitation) past the delegated-administration gate #25251), placed above the The #1888 user-less fail-open is wider than the lint that guards it — record-change flows fired by a system write run UNSCOPED, unlinted #3760 amendment, following that section's form. It names what is refused (the six governed verbs onsys_user_position,sys_user_permission_set,sys_invitation,sys_member, from a context withisSystemand aflowRunId), what is unchanged (reads, other objects,runAs:'user', flow-less platform writers), and the remedies. The The #1888 user-less fail-open is wider than the lint that guards it — record-change flows fired by a system write run UNSCOPED, unlinted #3760 amendment's note, whose example is "letting them writesys_user_position", gains one trailing clause pointing at it.Flow (runAs:'system')now reads "explicit elevation, except identity assignment", naming the four objects and the ADR-0073 amendment.Verification
node scripts/pm/dispatch-gates.mjs --commandsderived 20 families for this diff. All 20 exit 0, among themcheck-adr-links,check-adr-symbol-anchors,check:adr-anchors,check:doc-authoringandcheck:pm-governed-merges.dispatch-gates --ran: 20 run, 0 NOT-MEASURED. No changeset:docs/adr/**ships in no package.🤖 Generated with Claude Code
https://claude.ai/code/session_013yig39ZouqWBxABRfM5FN4
Generated by Claude Code