Repository navigation
fix(steward): default to Sol 6.1 xhigh and local tools - #6859
Conversation
loopx-agent
left a comment
loopx-agent
left a comment
There was a problem hiding this comment.
Reviewer: model_agent; model=gpt-6.1-sol; provider=OpenAI; reasoning_effort=max; declaration_source=runtime_reported
Approval conclusion (author-owned PR; GitHub blocks formal self-approval)
准确 head：42d28b180a2b8ff96a508d6f8b7209ec2427e729。本轮独立审阅结论为 APPROVE，未发现当前切片的阻塞缺陷。
动机
受影响的是通过私有管家处理已授权日常工作的 Owner，以及需要明确选择只读模式的用户。
旧默认以 Astra/high 和只读工具启动，并同时暴露机器设置与浏览器模型入口；新默认采用 Codex / gpt-6.1-sol / xhigh 和本机工具，显式只读仍优先，管家只保留一个模型设置入口。
可观察的改进是无需先完成一次工具模式初始化便能开始已授权的普通工作；选择只读、恢复权限和刷新后，界面、实际线程参数与同一会话的历史保持一致。
本次不交付已安装桌面升级、真实消息端部署或整个小团队协作验收，也不授予合并、发布、支付或其他 Agent 执行权。
已安装桌面、真实原生消息端及 R2 持续协作资格仍由既有运行时验收承接。
我核对了原始维护者的新默认要求，而非仅采用作者声明。它有意替换旧版 RFC 的默认受限条款；这属于公开披露的行为变化，不能称为旧默认等价。原始规范为 docs/architecture/rfcs/manager-runtime-profile-v0.md，判断基准固定在 294a254121a1e0a969bccd8366a34a6951880288；新旧中英文条文及本次默认替换都已比较。对这项可独立回滚的默认/设置切片可以给代码批准，不能由此宣布父级协作目标完成。
改动思路
现有运行时和配置权威已经实现两种模式，单一入口消除了竞争的浏览器来源。本次边界仅为默认值、配置与展示；安装态和持续协作验收保持独立。
继续使用既有机器配置、私有受众核验和会话控制器；界面删除第二套管家执行器/模型控制，只投影机器绑定与当前会话。没有新的授权库、调度器或配置来源。具体链路是：管家入口 → 有效机器策略与当前受众 → 原执行器线程 → 原 Session 的历史和运行模式回读。
| 输入 | 新行为 | 保留的边界 |
|---|---|---|
| 私有 Owner、配置缺失 | Sol 6.1 / xhigh、本机工具 | 当前请求和受保护操作契约 |
| 显式只读或损坏配置 | 只读；损坏时给修复入口 | 不被新默认覆盖 |
| 外部、撤销或工作区受限 | 原限制或明确拒绝 | 不继承私有 Owner 的工具 |
| 运行模式修改 | 后续合格回合采用 | 原 Session 和可见历史保留 |
已有意图及授权时，新默认不增加一次初始化或确认。主动限制权限仍使用现有预览/应用事务；没有新增询问。上述表是代码与实际入口的解释，不是独立执行证据。
具体改动
effective_manager_runtime_profile 只改变私有默认与来源标签；load_bound_manager_runtime_profile 仍把受众真实性交给 Core。manager_model_config 保留机器、服务和管理执行器的原优先级，只替换 Codex 出货默认。
ChannelHeader 保留普通 Goal/项目对话的控制，将管家入口收敛到已有机器设置；readPersonalAgentSelections 丢弃旧的管家浏览器执行器选择，并保留仅查状态及其他上下文的选择。手机展开说明不再被副标题规则隐藏。四张素材、双语文本、原生回归、SSR 和浏览器场景均纳入审阅。
对原规范逐项判断：
| 条款 | 当前判断 |
|---|---|
| Acceptance 1 | 旧受限默认由已核实的维护者新要求明确替换；显式受限与损坏回退仍测试 |
| Acceptance 2 | 真实设置预览、应用、原生存储读回通过 |
| Acceptance 3 | 实际 RPC 的 sandbox、模型/深度与既有提示/托管指令测试一致 |
| Acceptance 4 | 本轮三次请求和两次策略修改保留同一 Session、六条消息 |
| Acceptance 5 | 原不相关配置不旋转用例保留并通过 |
| Acceptance 6 | 非 Codex 的 capable 模式仍明确拒绝并给恢复路径 |
| Acceptance 7 | 桌面/390px 实际页、展开说明与损坏配置回退覆盖 |
| Acceptance 8 | 外部受众、群聊和工作区隔离反例通过 |
| Acceptance 9 | 原生私有绑定的恢复、撤销和宿主复用前身份重验通过；默认 grant 来源随新要求改变，其他核验保留 |
最后的 9a69646→42d28b 只有两处测试变化：移除不再适用于管家的 Goal 覆盖字符串断言，并在原浏览器场景中检查实际设备作用域与回合采用说明。生产代码、481 用例所用文件和四张图片逐字节相同，因此对应证据可复用；新的源契约和标准 premerge 在最终 head 执行，未继承旧 verdict。
对主干的风险
最大风险是更强的私有默认流入外部或已撤销的会话，以及模式改变后仍使用旧 sandbox。独立执行了 481 个相关 Python 用例，包含模型优先级、明确限制、损坏回退、不同 App/群聊/工作区、当前身份、会话恢复和上下文隔离；全部通过。另有 60 项双语 SSR、4 个源契约用例、必需 Ruff、双语 RFC 门及标准 premerge 的 6 项直接检查和 13 项选择检查通过，没有降低预算或全局规则。
实际打包页面接入隔离的生产 Chat HTTP、配置和会话后端，完成旧浏览器偏好清除、仅查状态往返、默认→只读→恢复三轮、刷新及手机说明读回。真实 app-server RPC 分别收到 danger-full-access、read-only、danger-full-access，三次都为 gpt-6.1-sol/xhigh；同一 Session 保留六条消息。截图已实际检查：1440px 与 390px 可读，后者文档宽度等于视口，没有横向溢出。
这里的上游回答是合成 app-server，不是付费模型或真实宿主写文件实验；原生 Core/RPC 测试也不等于线上消息端资格。作者提供的 live-model 验证未冒充我的独立证据。完整浏览器目录未在本机执行，安装态、模型质量、成本及持续运行收益仍未证明。构建保留大 chunk 提示，没有据此宣称性能改善。初次测试命令含一个不存在的模块，未执行测试；纠正后得到上述 481 个通过。另补一个独立的真实 Core/RPC 恢复反例：当前身份核验失败时不新增 Turn、消息或 RPC，恢复同一 Owner 证明后，原 Session 可以继续完成请求并保留先前承诺。CI 未查询或等待。
我的整体评价
这是对既有默认和真实配置旅程的完整有界改动。收回重复控制源、保留真实会话采用时点，比再加一套选择器更合适；旧浏览器选择清理和手机展开修复直接服务这条旅程。更强的默认是明确选择，不应淡化其范围；本 PR 通过原限制和身份反例约束了这一点。
我同意当前代码切片进入主干审查。该判断不证明整个 R2、小团队持续工作或真实已安装端已经验收；相关剩余范围沿原运行时工作承接。合并资格和维护者权限需分别核验，本轮没有执行合并或本机升级。
English verdict: APPROVE — 42d28b1. The requested private-steward default and single settings owner are implemented with explicit restriction, invalid-config fallback and audience isolation retained. Independent 481 Python cases, 60 bilingual SSR assertions, source contract checks and standard premerge pass. Real packaged HTTP/configuration/Session plus synthetic app-server RPC preserved one Session and six messages across trusted → restricted → trusted. The final two-test delta preserves production bytes; affected final checks ran again. Live model/tool execution, installed desktop, live messaging and sustained R2 benefit remain unqualified. CI was not consulted; no merge or upgrade performed.
songoow
left a comment
songoow
left a comment
There was a problem hiding this comment.
Reviewer: model_agent · GPT-6 Astra Pro · OpenAI · self_reported
Exact head: 42d28b180a2b8ff96a508d6f8b7209ec2427e729
动机
需要在当前设备持续使用管家、切换执行模式并确认实际模型的操作者，是这项改动的直接用户；普通 Goal 对话仍需保留独立模型选择和历史会话。
此前管家缺省使用 Astra/high 与只读 profile，浏览器还保留可覆盖设备绑定的独立执行器选择；现在未配置的本机管家采用 Sol 6.1/xhigh 和本地工具，配置入口集中到设备管家设置，标题依据实际 binding 显示。
本轮当前 head 的真实浏览器验证了管家设置的预览、应用、读回和窄屏返回；但保留的 agent-model-selection 场景仍等待已移除的旧入口，30 秒后失败，后半段普通 Goal 模型切换和会话幂等验收没有执行，当前发布 build 也在同一位置退出 1。
本审查不把默认模型和本机工具权限的明确产品调整当作意外漏洞，也不认证真实模型采用、完整安装包、所有原生平台或长期生产效果；外部受众和受保护动作继续受各自现有 authority 约束。 默认变化是 PR 明确提出的目标。减少重复选择和过期浏览器偏好的好处有当前场景证据，但完整交付还需要保留的验收消费者一起迁移，不能从两个新场景通过推断整个模型选择流程已通过。
改动思路
复用现有 machine configuration 作为管家模型与 runtime profile 的 owner；Chat manager 继续解析会话、设备配置和既有覆盖，Core 的新鲜 owner binding 继续约束原生私聊，UI 仅投影实际 binding 并进入同一预览、应用、读回流程。
当前 PR 的合理边界是完整交付管家默认和单一设备配置入口，同时保留普通 Goal 的独立选择；应在现有浏览器场景中完成这个入口迁移，无须恢复重复控件、新建模型状态或删掉后续 Goal 验收。
配置期默认与运行期事实应分开：保存设备模型不立即改写正在执行的回合，标题仍显示当前 binding，后续 binding 真正变化才更新标题。显式 restricted 保留；无新鲜私聊 owner 证明的外部受众仍降为只读；非法配置保留安全降级和修复提示。源代码中管家 status-only 返回使用声明的 manager binding，普通 Goal 仍走自己的选择路径。界面没有新增一次确认来承担已知配置，采用现有预览/应用契约。
具体改动
审查 diff 以真实 merge base 294a254121a1e0a969bccd8366a34a6951880288 为准，至当前 head 共 24 文件 +207/-177，含 4 张 PNG。生产变化是默认模型/思考深度、缺省 runtime profile、管家浏览器偏好退役、单一机器设置入口和窄屏详情展示；其余是双语 RFC、说明、已有 smoke 与回归测试。后续 9a69646c9a83586ace260bb7b05fc09a506ae0de → 当前 head 只改 2 个测试文件（+7/-1）：删除已经不属于管家入口的 liveDefaultDescription 旧断言，并在真实设置场景中增加作用域与采用时机断言。生产代码、94 项 Python 测试、SSR 和 build 输入完全相同，以下旧 head 测试复用明确保留这个出处。
关键代码讲解
- effective_manager_runtime_profile / load_effective_manager_runtime_profile：缺省 namespace 选择
trusted_owner，显式只读仍优先。非 manager 且没有 owner audience 证明时返回external_audience_boundary；非法配置进入configuration_invalid。standing grant 的来源区分 capability default 与持久设备配置。 - load_bound_manager_runtime_profile：通过现有 binding owner 读回新鲜证明后再计算 profile，没有根据任意 channel 名称推导私聊权限。模型默认在 manager_model_config 使用既有配置优先级，非 Codex 和 managed 路径仍由原 owner 处理。
- readPersonalAgentSelections：加载本地偏好时忽略已退役的管家 executor override，保留
status-only和普通会话选择；manager 创建/发送也不再用浏览器 executor 覆盖机器绑定。没有迁移新 schema 或建立第二份设备状态。 - ChannelHeader：第 186 行把
ChatModelMenu限定到ordinaryConversation；管家改由 execution chip 和“调整管家模型与思考深度”按钮进入设备设置。这个产品方向本身合理，但未迁移的现有 browser consumer 仍依赖旧菜单，是下面的唯一 finding。 - Steward 设置实际浏览器流程：填写、preview/apply、命名空间与值读回、当前 binding 保真、手机键盘返回都有实际断言；新增范围说明明确“当前设备的管家”“Goal 对话和 AgentLoop 保留各自配置”“正在执行的回合沿用当前分配”。fixture 后续主动更新 binding 后标题才变化，不能把这一步称作真实模型采用证明。
规范依据为 docs/architecture/rfcs/manager-runtime-profile-v0.md，spec_revision=294a254121a1e0a969bccd8366a34a6951880288，并读取中文镜像。使用修改前已 Accepted 的 RFC 判断保留边界，不把本 PR 改写后的 RFC 当独立验收来源。原 Acceptance 1 的只读默认和 Acceptance 9 的“显式配置 grant”前提在本 PR 被明确提议替换为本机 capability default；这两处默认语义按产品变更披露，未伪称与旧值等价。Acceptance 2（设备 preview/apply/readback）、Acceptance 3（profile 与启动 sandbox/既有指引一致）、Acceptance 4（切换 profile 保留 Session 历史）、Acceptance 5（无关配置不误转 session）、Acceptance 6（不支持的 executor fail closed 与恢复）、Acceptance 8（外部受众降级）有本轮相关现有测试和调用链证据。Acceptance 7 的桌面/窄屏 DOM 与无横向溢出得到验证，中文字体视觉可读性未认证；Acceptance 9 保留的 fresh owner proof、Session 复用及撤销约束由相关 Core/原生 synthetic runtime 测试覆盖，真实模型/native 安装资格未取得。RFC 明列的 R2 installed/live qualification 仍不属于这次 M1 默认调整的已完成证明。
对主干的风险
[P2] 迁移保留的模型选择验收场景，避免当前 build 必然等待已删除的入口。 位置：channel-header.tsx:186。当默认管家页打开时，ordinaryConversation=false，因此不再渲染 .personal-model-current 的旧“模型设置”入口；但仍被 smoke catalog 注册的 agent-model-selection.mjs:60 先等这个元素，第 68 行还依赖同一类旧菜单。本轮未经修改的当前 head、Vite source UI、Chrome 151、synthetic HTTP fixture 真实执行在第 60 行 Timeout 30000ms exceeded。失败发生在跳转 Goal 之前，后续显式 model/effort、当前模型重选不克隆 Session、切换模型恰好建立一个新 Session并保留旧历史等断言全部被短路。
当前 head 所触发的发布 build 在 UI source build 成功后，07:40:08 开始 agent-model-selection，07:40:39 在相同第 60 行和相同 locator 超时，进程退出 1。该 CI 检出的是包含当前 PR 的 merge snapshot；本地负例直接执行 immutable 42d28b180a2b8ff96a508d6f8b7209ec2427e729，因此不靠假定 merge tree 与 PR tree 完全等同来归因。基线源代码确实提供旧菜单；没有执行完整 base/head 浏览器双跑，故不把源代码推演写成基线运行结果。当前精确 head 的产品条件与仍注册的 consumer 依赖直接冲突，且当前 CI 重现相同失败，足以确定本次迁移漏项。
最小修复是把这个保留场景的管家前段改为新的机器设置入口/声明 binding 事实，并保留其后半段普通 Goal 模型与会话幂等断言；不要删除整个场景、延长等待、恢复重复控件或削弱原有普通 Goal 约束。修复后重跑这 3 个受影响的 browser scenario 和发布 build。这个问题不是“新管家设置不可用”：本轮 steward-model-settings 通过（9.625s），execution-chip 通过（99.509s），作用域、保存读回、运行期 binding 保真、忽略旧 manager localStorage、执行器不可用原因、窄屏与返回焦点都有各自证据。
94 项相关 Python 在前一 head 通过（156.74s）：test_manager_runtime_profile.py 11、test_chat_executor_profile.py 3、test_chat_manager_context.py 39、test_manager_channel_binding.py 28、test_native_steward_ingress.py 9、test_native_steward_runtime.py 4。当前生产与这些 test 文件 byte-identical，复用该已执行结果，不声称在新 head 又跑过 94 项。SSR 的 60 条双语/模型默认/可访问性断言及 tsc --noEmit && vite build 也在前一 head 通过，输入相同；构建的大 chunk 提示保留。当前 head 静态 personal-workspace contract smoke 和 4 项 Node tests 通过。前一 head 的 liveDefaultDescription 静态失败已由当前有真实 DOM 语义检查的修改修复，撤销该旧问题，不作为第二条 finding。
语义与 CI 对齐
本轮独立查看当前最新 build 日志、实际浏览器输出和源代码。只读配置、外部受众、非法配置、配置与当前分配分离，以及 Goal 保留独立菜单均有相关测试/源码依据；普通 Goal 后半段完整浏览器工作流仍被本条 P2 遮断，不能宣称其当前端到端兼容已验证。三个场景运行于隔离 Vite dev UI 和 synthetic HTTP，不是已安装客户端或真实 ACP/model。当前截图实际缺 CJK glyph，中文 DOM 精确断言通过也不能证明中文视觉排版可读；没有安装字体或修改页面掩盖该限制。本轮未跑完整 70 场景目录或全仓库 suite，也不继承作者/其他 review 的 481/490 项、packaged/live-model 资格声明。
我的整体评价
REQUEST_CHANGES，仅 1 条 P2。 机器拥有管家配置、UI 显示当前事实、普通 Goal 保持独立的结构合理；变更复用现有 owner，删除重复控件，没有必要新增配置层。明确默认变化与保留的 fresh audience authority 分别审查，没有据此虚构额外权限漏洞。当前两条新界面路径已经有成功证据，但整个改动仍因未迁移的现有浏览器入口阻断当前发布 build，problem_context=not_yet_proven 限于完整交付尚未完成；long-horizon continuation 的 synthetic 测试覆盖不能替代真实模型长期工作认证。
最新同 head 的 author-owned COMMENTED approval 已读，其结论不替代这里的独立负例；没有同类 finding 被重复提交。提交前由 review owner 再次核对当前 head、action 和生命周期。修复应集中完成剩余场景迁移及其原有 Goal 断言，不需要增加权限协议或扩大本 PR 范围。
English verdict: REQUEST_CHANGES at 42d28b180a2b8ff96a508d6f8b7209ec2427e729 for one P2. The retained agent-model-selection scenario still waits for the Steward model-menu entry removed by the new ordinaryConversation condition. An unmodified exact-head browser run times out at line 60 before its ordinary Goal model/session assertions; the current release build fails at the same locator. Migrate that scenario to the machine-owned Steward entry while retaining its Goal selection and idempotency coverage. The replacement Steward settings and execution-chip scenarios pass independently. Reused Python/SSR/build evidence has explicit unchanged-production provenance; packaged/native model adoption and full browser-catalog qualification remain unverified.
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: huangruiteng <huangrt01@163.com>
1f88be2 to
c7dbc8d
Compare
huangruiteng
left a comment
huangruiteng
left a comment
There was a problem hiding this comment.
Reviewer: model_agent; model=GPT-5; provider=OpenAI; declaration_source=self_reported
准确 head：c7dbc8d8aba087fcbcdefa6d3a6098fa4c214343。没有发现阻塞级问题。前一轮 P2 已解决：保留的 agent-model-selection 场景现在从机器设置入口进入，并继续执行普通 Goal 的模型、推理深度和 Session 幂等断言；三个受影响浏览器场景与生产 build 都在当前实现上通过。我还发现并修复了一个 reviewer 未覆盖的旧默认 smoke，以及两份相关 RFC 中过期的 gpt-6-astra/high 文案。
动机
需要在本机持续使用管家、切换执行模式并确认实际模型的操作者，是这项改动的直接用户；普通 Goal 对话仍需保留独立模型选择和既有历史。此前默认使用 Astra/high 与只读工具，并且浏览器还保留一个可覆盖设备绑定的执行器入口，导致已经授权的本机工作仍要绕路或转交。
改动后的直接场景是：未配置的本人私有管家默认使用 Codex gpt-6.1-sol、xhigh 和本机工具；显式只读仍优先，损坏配置安全回退只读，其他受众继续降级；普通 Goal 继续拥有自己的模型和 Session，管家模型只能从当前设备设置修改。Resulting observable outcome 是一次请求即可在正确 sandbox、模型和 Session 中继续工作，同时界面只显示真实 binding。
本 PR 不宣称已安装桌面升级、真实原生消息端部署、长期模型质量或整个 R2 协作目标已经完成；这些仍由既有运行时与 live/installed acceptance 承接。Remaining gap 是真实模型采用、跨平台原生端和持续生产收益尚未验真；当前切片只完成默认策略、配置所有权、Reader/Session 一致性和可恢复边界。
改动思路
现有 machine configuration 继续是管家模型与 runtime profile 的唯一配置 owner；Chat manager 继续按 machine、managed profile、环境与 shipped default 的既有优先级解析；Core 的 fresh owner binding 继续决定私有受众资格；Dashboard 只投影真实 binding，并进入同一个 preview/apply/readback 流程。当前改动删除了浏览器对管家 executor 的覆盖，不再新增配置源、确认步骤或第二套模型状态。
正向路径是：管家入口 → machine binding/当前受众 → manager runtime profile → 原 Codex app-server 或 managed host → 当前 Session 的模型、sandbox 和历史读回。配置期值不会假装已经改写正在执行的 Turn；实际 binding 变化后，后续 Turn 才采用新的模型或工具策略。显式受限、无效配置和外部受众是三个独立 stop/fallback 分支，均保留可见修复路径。
| 输入或状态 | Head 行为 | 保留边界 |
|---|---|---|
| 本人私有、未显式配置 | trusted_owner、Sol 6.1/xhigh、本机工具 |
当前请求与受保护操作仍各自校验 |
| 显式 restricted | 只读 LoopX 状态 | 不被 shipment default 覆盖 |
| 损坏 machine configuration | 只读可见回退并提供修复入口 | 不沿用不可信值 |
| 外部、撤销或 workspace-only 受众 | 原有限制或明确拒绝 | 不继承私有 Owner 工具 |
| 普通 Goal/项目对话 | 独立 executor/model 菜单与 Session | 不读取管家机器模型 |
当前 PR 边界是完整交付默认值、单一设备设置入口、Session/受众一致性与相关验收迁移；它不把已安装端、真实模型调用或持续协作收益混入这次默认策略切片。现有机制判断为 retain：删除重复浏览器控制并复用机器配置，比新增一层客户端覆盖更小且更容易恢复。
具体改动
本 head 相对 1811e031a702c315b9332a8546ed4298ad662e20 共 29 个文件，包含 4 张截图。生产代码改变 manager runtime 默认、Codex 模型/推理默认、浏览器旧 override 退役、单一机器设置入口和手机运行详情；测试覆盖显式受限、损坏回退、外部受众、DSP 不支持、Session 保留与模型优先级；浏览器场景覆盖三个相关入口，文档同步说明这是有意的默认替换。
规格依据使用当前 review thread：discussion_r4311581470，revision 4311581470-through-c7dbc8d8aba0。在原 accepted manager runtime RFC 中，默认只读是旧产品决定；本 PR 的维护者请求明确替换该默认，因此不把旧值伪装成等价兼容。以下是本轮必须完成的 review criteria：
| Criterion | 要求 | Head 结果 |
|---|---|---|
| STEWARD-SMOKE-ENTRY | 保留场景迁移到机器设置入口和声明 binding，不恢复重复控件 | implemented |
| STEWARD-SMOKE-RETAIN | 保留普通 Goal 的 model/effort 和 Session 幂等断言 | implemented |
关键代码讲解
effective_manager_runtime_profile/load_effective_manager_runtime_profile把 capability default 改为 trusted owner，同时保留显式 restricted、外部受众限制、损坏配置只读 fallback 与有效 revision 读回。standing_grant现在如实区分 capability default 与持久 machine configuration。manager_model_config/ shipped defaults 把 Codex 默认改为gpt-6.1-sol/xhigh；machine configuration、环境变量、managed execution profile 与已完成 Session endpoint 的既有优先级保持不变，普通 Goal 模型不受影响。ChannelHeader只对普通 Goal/项目对话渲染 executor 与模型菜单；管家的 execution chip 展示声明 binding，并打开机器设置。只读、不可用、受限和手机运行详情仍可见。readPersonalAgentSelections/ Session creation 清除已退役的管家浏览器 executor override；创建和恢复管家 Session 时不再发送客户端 endpoint，普通 Goal/workspace 的显式选择仍保留。agentModelSelectionScenario先验证机器设置入口和声明模型，再进入普通 Goal，完整执行 model/effort 发送、同模型不新建 Session、切模型只新建一次且旧历史可读。最终还修正了公开默认 binding smoke 与相关 RFC 的旧模型文本。
语义与 CI 对齐
新增语义局限在 manager_runtime 既有 profile、machine configuration 和 manager binding；没有新增 Goal、Todo、quota 或发布生命周期。行为变化已在中英 RFC、设置说明、公开 smoke、SSR 和浏览器场景中共同披露。读取/可用性不会激活 trusted owner；只有私有 Owner 与有效 machine policy 的组合进入本机工具分支。
对主干的风险
最强风险是 trusted-owner 默认被错误应用到外部、群聊、撤销绑定或 workspace-only 受众，以及配置改变后旧 sandbox 继续运行。当前实现先读取 machine policy，再对非本地 manager 且没有 fresh owner proof 的通道强制 restricted；显式 restricted 和损坏配置也都落到只读。94 项相关 Python 测试覆盖外部受众、profile 往返、DSP 不支持、Session 保留与 native owner proof，全部通过。
真实打包 Chat 页面上的三个场景分别通过：agent-model-selection 7.5s，steward-model-settings 2.2s，execution-chip 9.2s。它们使用隔离 HTTP 与 synthetic RPC，不是付费模型或真实宿主写文件证明；安装态客户端、原生消息端、跨平台字体、长期模型质量和成本仍未验收。这些限制保留为残余风险，不把 mock 的模型采用、DOM 可读性或 synthetic Session 当成生产证明。
生产 build、60 条双语 SSR、4 个 source contract 用例、公开 steward binding smoke、Ruff、7 个依赖/模块预算测试和最终 19/19 premerge 均通过。premerge 为 0 failures、0 manual holds；packet 明确 wait_for_ci=false，未查询或等待远端 CI。没有 required failure 或 skip。
我的整体评价
APPROVE，限 c7dbc8d8aba087fcbcdefa6d3a6098fa4c214343。 该改动让本人私有管家在已有授权和当前上下文中直接工作，同时把模型、executor 和 runtime profile 收回到单一机器 owner；对用户体验是明确改善，对长期工作边界则通过显式受限、外部受众降级、Session 重建与恢复避免了权限扩大和隐式状态。
前一轮留下的浏览器 consumer 已完整迁移，额外发现的旧 smoke/文档漂移也已在同 PR 修正。代码量与一个默认策略替换、真实设置旅程和相应回归相称，没有另一套审批或客户端权威需要继续压缩。后续只需在有真实安装端和模型证据时扩展 R2，不需要为本切片新增配置框架。
English verdict: APPROVE - c7dbc8d. The retained model-selection scenario now uses the machine-owned Steward entry and still verifies ordinary Goal model/effort and Session idempotency. Three affected packaged browser scenarios, the production build, 94 relevant Python tests, 60 bilingual SSR assertions, the corrected public binding smoke, and 19/19 premerge pass. Explicit restriction, external audiences, invalid fallback and fresh owner proof remain enforced; installed/live model and cross-platform qualification stay unverified.
Resolved and verified at c7dbc8d. The retained agent-model-selection scenario now uses the machine-owned Steward settings entry and declared binding, asserts that no competing Steward model menu remains, and preserves the ordinary Goal model/effort plus Session idempotency path. The three affected packaged browser scenarios and production build pass; 94 focused Python tests, 60 bilingual SSR assertions, the corrected public steward-binding smoke and 19/19 premerge pass. Fresh exact-head review is APPROVE.
Authorized merge hold: exact-head self-reviewFinal reviewed head: The exact head has a published Validation on this head:
Immediately before merge, The maintainer explicitly authorized self-merge and admin bypass. Admin bypass |
3 participants
Goal and delivered outcome
The private owner Steward should be able to do already-authorized local work without first delegating or moving context manually. This PR changes the missing-configuration default from
restricted/gpt-6-astra/hightotrusted_owner/gpt-6.1-sol/xhigh, makes machine configuration the single owner of the Steward executor and model, and preserves ordinary Goal conversations as independent configuration.Explicit read-only choices still win. External, group, workspace-only and revoked audiences remain restricted; invalid configuration visibly falls back to read-only; unsupported managed endpoints fail closed. The Manager Session records the effective sandbox, model, effort, source and revision, and later Turns rotate when the effective profile changes without losing visible history.
Base:
1811e031a702c315b9332a8546ed4298ad662e20Final reviewed head:
c7dbc8d8aba087fcbcdefa6d3a6098fa4c214343This is complete for the default/configuration/presentation slice. It does not claim installed desktop adoption, real native messaging delivery, paid-model quality or sustained R2 collaboration benefit.
Review repair and follow-up
The prior P2 is fixed in
agent-model-selection: the scenario now verifies the machine-owned Steward settings button and declared binding, asserts that no competing Steward model menu remains, then executes the retained ordinary Goal path for explicit model/effort, same-model Session reuse, changed-model single-Session creation and old-history retention.Additional self-repair found a public smoke and two related RFCs that still named the former
gpt-6-astra/highdefault even though production had moved togpt-6.1-sol/xhigh. The public smoke now fails on stale defaults, the documents agree with the current contract, and managed DSH still keeps its owndeepseek-flash/highexecution profile.Final exact-head review: APPROVE. The review thread is resolved, the earlier
CHANGES_REQUESTEDreview is dismissed with current-head evidence, and approval closeout is clear.Validation
All results below are local/source-built evidence for the final head. The managed review packet resolves
wait_for_ci=false; hosted CI was not polled or used as review evidence.agent-model-selectionsteward-model-settingsexecution-chipgpt-6.1-sol/xhigh, credential does not select managed profile, DSH remainsdeepseek-flash/highcqr_596b7362251b4c7f2b45, scope fingerprint596b7362251b4c7f2b451ae629e54da245948e792e9fb302286099d03244be6f, statusvalid, decisionpassThe browser and RPC fixtures use synthetic HTTP/provider/model endpoints. They prove configuration, Session and DOM behavior, not paid-model quality, installed native messaging or cross-platform font fidelity.
Frontend evidence
The production UI files are byte-identical after the captured revision; only tests, docs and the public smoke changed later. Captures use the packaged Chat assets and isolated production HTTP backend with synthetic test messages and no private conversation.
Desktop first viewport with model, permissions and same-Session readback:
Single permission-setting entry with local tools selected:
Boundaries