Repository navigation
docs(adr): ADR-0096 D5 dated note — strict mode lands ON ahead of D2 and the telemetry gate - #22298
Merged
Merged
Conversation
…and the telemetry gate Records the maintainer's ruling Q3 on the principal-less hand-off closure: the explicit opt-in is the wire-level isSystem flag, no strict-mode switch is built, and the ADR's text is not rewritten. Claude-Session: https://claude.ai/code/session_01WkL6Eijt432S1Y7ekb6ovQ Co-authored-by: Claude <noreply@anthropic.com>
Contributor
Author
维护者速读 · PR #22298 · 席位
|
os-zhuang
approved these changes
Oct 8, 2026
os-zhuang
marked this pull request as ready for review
October 8, 2026 13:45
os-zhuang
enabled auto-merge
October 8, 2026 13:45
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.
Refs #21908
Tier H — governed surface (
docs/adr/**). This PR lands only on the maintainer's approval: an authorized APPROVED review, or the maintainer's own merge. No seat readies, queues or arms it.Summary
A dated note under ADR-0096 D5 records the maintainer's ruling Q3 (甲) on #21908 (comment 6019001864, 2026-10-06). The note says:
systemContext(reason)door and the telemetry gate;isSystem: trueflag, which is E1's own prescription;security.identityStrictswitch is built, so there is no OFF mode.The note cites what licenses that order: the published contract sentence on
ChatWithToolsOptions.toolExecutionContext("unauthenticated (RLS-on, sees-nothing)") and the maintainer's ruling 2B on objectstack-ai/cloud#2006. The ADR's own text is not rewritten, and its status staysProposed.The behaviour the note records ships in #22297, the deny round, which is a separate PR and not governed.
The diff
One blockquote note, 2 added lines, placed at the end of D5 and before D6. Nothing else in ADR-0096 changes.
维护者速读(草稿)
改了什么
在 ADR-0096 的 D5(严格模式)末尾加一段带日期的注记,不改动 ADR 原文与状态。注记记录:严格模式现在就默认开启,先于 D2 的
systemContext(reason)入口和遥测门;显式提权沿用线上已有的isSystem: true;不提供关闭开关。为什么改
你在 #21908 的 Q3 选了「甲」:只加注记,不重写 ADR。代码那一半(拒绝没有主体的数据引擎调用)在 #22297。ADR 是受管面,所以单独成 PR,由你批准。
风险与代价(含回滚)
纯文档,对运行时零影响。回滚就是撤掉这两行。真正的行为风险在 #22297(进程内未带上下文的调用会被 403 拒绝),那边有迁移说明和合同级复核。
席位意见
你要做的
读一遍注记的措辞,同意就批准(APPROVED);要改措辞就在 PR 上留言。
Evidence
dispatch-gates --commandsderived 19 families at2becf9015f. All 19 ran and exited 0.--ran: "19 derived famil(ies) accounted for — 19 run, 0 NOT-MEASURED (a DERIVED zero — all 19 recorded an exit code and none of them is 3)". The firstcheck:doc-formula-expressionsrun exited 3 (prerequisite not met:formulaandlintwere unbuilt in this worktree). They were built, and it then exited 0.check-governed-merges --branch claude/issue-21908-adr-0096-d5-note: GOVERNED, landing tier H,docs/adr/**×1 (docs/adr/0096-execution-surface-identity-admission.md), 2 changed lines.Generated by Claude Code