Skip to content

Fix manager discovery in ordinary installs - #8830

Merged
cocolord merged 2 commits into
mainfrom
codex/r1-manager-discovery-package-20261115
Nov 15, 2026
Merged

cocolord merged 2 commits into
mainfrom
codex/r1-manager-discovery-package-20261115

Conversation

@cocolord

@cocolord cocolord commented Nov 15, 2026 •

Copy link
Copy Markdown
Member

Installed multi-source Agent discovery reaches a lazy TypeScript handler that the ordinary wheel does not contain. Include the existing manager projection in package data so the authorized read uses its original typed owner. Core already ships that resource; authorization and lazy dispatch remain unchanged.

RFC: selected distribution and R1 boundaries. The bilingual checkpoint records the repaired boundary. The same package-data patch applies to C1 #8266 at 1c578cfe85c23db40605843a0afc1d9d37caad39.

Validation:

  • The real setuptools regression fails with the old package data and passes when the required TS resource ships; the existing skill remains byte-identical.
  • All 70 targeted build and manager-inspection source tests pass without skips.
  • Fresh noneditable ordinary and C1-composed wheels pass source/wheel/RECORD/install readback and real manager RPC checks for fair source-qualified rows, unavailable sources, invalid bounds and recovery.
  • Separate module-resolution observation confirms unselected runtime requests keep the manager projection unloaded. The existing scoped caller checks retain authorization revocation and unknown-source behavior.

These are isolated packaging and selected-runtime checks. Complete D2/D5 closure, actual-model/context adoption, total-cost benefit, full R1/M3 acceptance and C1 default cutover remain unqualified. No UI source or permission rule changes.
Tested ordinary revision: 53a624a5b1892031fb291ed17239168346bdd4ad. Its newly built bundle and noneditable wheel are independently bound to that revision. The C1 qualification remains bound to the separate actual combination 0ec2fa78f917ad02bd9bd95a1640ade5b442bc04; no new main+C1 cutover is claimed.

cocolord and others added 2 commits November 15, 2026 23:05
Signed-off-by: lusendong.6789 <[email protected]>
Co-authored-by: TRAE CLI <[email protected]>
@cocolord
cocolord force-pushed the codex/r1-manager-discovery-package-20261115 branch from 03af518 to 53a624a Compare November 15, 2026 15:10

@cocolord cocolord left a comment •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewer: model_agent; GPT-6; OpenAI; self_reported (runtime attribution unavailable: session_not_bound)

Approval conclusion (author-owned PR; GitHub blocks formal self-approval)

动机 / Motivation

通过普通安装包查询多个已授权来源中 Agent 的用户和 manager。 此前普通安装下,一次合法发现请求因缺少随包资源而返回内部 handler 错误,无法给出登记身份;现在相同请求返回标明来源的登记结果,来源不可用仍明确显示 unknown。 已授权的发现操作可以使用原有 typed projection 完成,不再因发行包漏文件而失败。 本 PR 不宣告整个最小内核、真实模型或上下文采用、完整成本收益以及 C1 默认切换完成。 剩余缺口是完整 D2/D5 闭包、完整 R1/M3、真实模型/上下文与原十项最终回归;实际 SSH 网络未在此次隔离验证中运行。

改动思路 / Approach

普通发行包补齐原来已由 Core 和源码使用的资源即可修复,保持同一 typed owner;运行时重新实现、启动时预加载或增加 fallback 都会扩大机制和默认开销。 当前 PR 的可用交付边界是普通安装的所选多来源发现及未选择时的 lazy 保持;C1 固定版本组合单独验证,不代表 C1 默认切换。

已安装用户发起现有发现读取,caller 先校验输入和授权,收集有界来源结果后再核对同一 scope、grants 和来源配置;随后原 lazyHandler 才导入 TypeScript projection。补丁让这个已有文件随普通发行包到达现场,没有引入 Python evaluator、额外 cache、启动预加载、永久管线或公共开关。资源安装本身不执行方法、不授予权限,未选择该方法的原请求仍走原入口。

具体改动 / Changes

完整评审对应 53a624a5b1892031fb291ed17239168346bdd4ad,实际 PR base 为 08cc6ef5b9a15a5a74d3e56ba4114cb09b2033ee。规范依据是修改前的 loopx/capabilities/manager_context/README.md,不是本 PR 新增 checkpoint。该文档没有编号验收项,这里使用原文可识别短语:Agent discovery 已实现原 view=agents 路径;No new grants 保持 caller 在读取前后核对授权;Unreadable/ambiguous inventory 保持 unknown 和后续读取;Execution readiness 仍为 not_checked,登记信息不被提升为执行能力;Live multi-worker adoption 为 deferred,原会话成果与实际模型/context 仍需单独资格化。原 RFC 完整 D2/D5、R1/M3 与 C1 default cutover 继续未完成。

关键代码讲解

  • manager_context package-data 只将既有 *.ts 与原 skill 一起打包。当前包中只有原 discovery projection 这一个 TS 文件;完整普通 wheel/source/RECORD/install 字节核对证明没有替代实现或漏读资源。
  • test_manager_discovery_projection_ships_with_its_skill 从产品 pyproject.toml 读取真正 package-data,沿用现有 tiny-source fixture 和 wheel helper,由真实 setuptools 构建再查原 TS 与 skill 的字节。未把修复规则另写进测试;原配置下确实缺 TS 并失败,最终提交通过。
  • manager.discovery.project lazyHandler 与原 caller 均未改变。新的普通包让它的原动态 import 可用;单独的 module-resolution 观察显示 ping 为零、首次选择为一、重复选择不再增加,未用 instrumented 运行代替实际业务执行。
  • projectAgentDiscovery 仍由原 typed decoder 检查 bounds/shape,在多个来源间公平取有界 rows,保留来源身份、unknown、source_pages 和 next_reads。它不查真实执行就绪、不发任务,也不生成新的 grant。

本 PR 只有四文件 +65/-1:一行既有打包配置、25 行原构建测试、39 行双语 checkpoint。与 C1 #8266 1c578cfe85c23db40605843a0afc1d9d37caad39 的实际组合 0ec2fa78f917ad02bd9bd95a1640ade5b442bc04 分别绑定源码和安装验证;组合不是已合入版本,也不代表 C1 全部验收。

对主干的风险 / Risks

没有发现阻断项。最强风险是源码测试通过,真正普通安装仍缺依赖;只查 ZIP、editable 导入、预加载模块或假定 handler 返回成功,都可能掩盖这个问题。本次保留旧实际 wheel 的合法 RPC 内部错误,新的 noneditable 环境用 Python 隔离模式和真实 Node runtime 执行相同完整请求;只换安装 interpreter,不改 fixture 或输出。旧错误信息按原产品隐藏路径,未要求暴露文件名。另有真实 setuptools 原规则失败,独立证明漏打包原因。

PASS:当前 53a624a5b1892031fb291ed17239168346bdd4ad 的 70 项构建和 manager-inspection 源测试,零失败、零 skip;全新 ordinary wheel 完整产品与资源 provenance、两份 RECORD 和 installed 字节核对;未 instrument 的 12 项实际 RPC 覆盖合法结果、typed rejection 和同一 runtime 的恢复;另行三项 lazy resolution 观察。首次 rebase 后环境缺少 setuptools 导致八项 skip 的运行已保留;装好声明的构建 backend 后同一最终提交完整重跑通过,未把 skip 当成通过。

PASS:固定 C1 实际组合的完整安装来源核对,13 项真实 RPC 和三项 caller scoped/unknown/授权撤销检查。caller 验证只替换 SSH transport 为真实 installed CLI,实际 SSH 网络没有运行。PASS:最终提交标准 premerge 的 5 项直接检查及 11 项选定检查,包括双语、semantic vocabulary、编译、public boundary 和仓库 smoke。base 集成改变了三个上游产品文件,因此没有继承旧 ordinary 全 payload;本轮重新构建并核对当前 Chat bundle、wheel、源测试和 premerge。C1 证据保留原实际组合来源。

PASS:另用当前 53a624a5b1892031fb291ed17239168346bdd4ad 普通安装在全新隔离目录执行同一个原 caller fixture 的三项授权检查,覆盖合法多来源、来源不可用与读取中授权撤销。三次真实 installed CLI 子进程输出经独立断言,授权撤销返回原 authorization_changed。fixture 与当前提交源码逐字一致,已有 54 份普通安装封存证据未改,补充验证分别封存;未运行实际 SSH 网络。

回滚可整体 revert,未引入持久 state/schema 或改历史 decoder。测试 runtime 使用原 helper、物理身份验证后显式清理;该清理不证明自然进程 drain。remote CI 已按当前同一 head 观察,仍有待完成项;未知原因保留 unresolved,不冒称既有失败,CI/readiness 与代码评审分别处理。

当前同一 head 的 Release browser (1/3) 与 dashboard-browser (1) 在 typed-actions.mjs:328 等待“创建 Goal 并开始首轮”30 秒超时,分别见 Release job 与 Dashboard job。静态检查显示保存 proposal 未声明 start_execution,原 UI 为 create-only;这些 mock API fixture/UI 不在本 PR diff,也不调用 manager RPC,C1 固定版本已包含该 selector 修正。CI 没有记录超时完整页面,故该解释保持静态证据边界,不伪称运行复现。稍后的 browser (2/3) 在 goal-workspace-selection.mjs:95 等待“进入新 Goal”超时,内部 apply 原因仍 unresolved;Release build 因 browser gate failure 未进入构建。本 PR 所改包装/所选 handler/授权/lazy 已有独立当前真实安装验证,没有确认这些 CI 失败由本 PR 导致;仍原样保留失败,不改阈值或宣告已修复。

语义与 CI 对齐

复用已有 typed projection、来源身份和 error 词汇。package glob 选择的是资源文件,没有参与 state、authority 或 delivery 分类。原 skill、输入规则、授权、unknown 及未选择 lazy 行为保持;checkpoint 只报告可复核增量,不改原预算、失败、stop 或目标验收条件。

我的整体评价 / Verdict

这是 justified_increment:user_experience 改善了已授权普通安装发现;long_horizon 的工作、持久决策和后续 continuation 留在原 owner,没有为修复读取增加成本机制。检查了现有 manager tool、Frontend/Lark 共享 evidence 入口和当前 bundle 来源,本 PR 只改变 package-data、构建回归与双语 checkpoint,没有渲染呈现改动。 完整 Settings/UI 旅程及真实模型 adoption 不由这次包装修复资格化。

采用同域最小修复和现有 build helper,完整当前 diff 与实际故障相称。普通发行包补齐原来已由 Core 和源码使用的资源即可修复,保持同一 typed owner;运行时重新实现、启动时预加载或增加 fallback 都会扩大机制和默认开销。 当前 PR 的可用交付边界是普通安装的所选多来源发现及未选择时的 lazy 保持;C1 固定版本组合单独验证,不代表 C1 默认切换。 原十项、完整 R1/M3、完整成本及模型/context 仍未完成。当前 exact head 的结论为 APPROVE;发表为 COMMENTED self-review fallback,完整 thread、approval closeout、readiness 与授权合入继续另行核对。

English verdict: APPROVE for the exact reviewed head. The ordinary wheel now ships the original selected manager projection; matched real installed RPC, negative/recovery cases, full package provenance and unselected lazy loading are verified. Physical SSH, full R1/C1, actual-model/context, complete-cost acceptance and CI/merge readiness remain separate limits.

@cocolord
cocolord merged commit b05ac41 into main Nov 15, 2026
34 of 49 checks passed
@cocolord
cocolord deleted the codex/r1-manager-discovery-package-20261115 branch November 15, 2026 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant