Skip to content

Isolate startup metadata and restore installed refinement guidance - #8661

Merged
cocolord merged 2 commits into
mainfrom
codex/rfc-r1-composition-20261112t201145
Nov 12, 2026
Merged

cocolord merged 2 commits into
mainfrom
codex/rfc-r1-composition-20261112t201145

Conversation

@cocolord

@cocolord cocolord commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

Goal-start help and preview loaded issue-fix execution and repository-analysis modules just to render command hints, even without selecting that workflow. Move the existing templates into issue-fix command metadata, use it from bootstrap/catalog, and keep the workflow import API and explicit candidate-preflight contract intact. Complete installed base/head output remains byte-identical; default startup loads 186→171 LoopX modules.

The installed CLI regression now covers complete context readback, partial delivery, lost ACK, unchanged/revised views, standalone PostCompact and changed-capture refusal through the existing owners. This also passes against the original core installation. The bilingual minimal-kernel RFC records the delivered boundary and remaining R1 work.

The installation gate also found an outcome-refinement supporting-example link outside the installed skill bundle. It now points to the byte-verified public document at an immutable revision, preserving the original procedure and authority clauses. This is an on-demand supporting reference; it adds no runtime fetch. The complete local installation smoke passes with the repaired link; earlier installation failures and disk-limited baseline attempts remain separate evidence.

Validation:

  • Source regression: 348 passed; one native Windows/PowerShell check skipped on macOS.
  • Final independently installed core wheel: 20 passed, including SQLite/File authority, artifact validation, process loss, lease-generation refusal, replay and context recovery.
  • Full startup/explicit-route output comparison, lint, compile and artifact/source byte readback passed.
  • Thirty alternating installed pairs per workload retain all observations. First-entry help median/p95: 314.73/362.55→181.25/303.01 ms; preview: 542.36/670.44→311.08/572.66 ms. The head preview maximum is 1,092.15 ms. These are ordinary-cache startup observations, with no full V10 or complete-cost claim.

Earlier failing import checks, premature installation checks and corrected context-test assumptions remain recorded separately. Native events are protocol fixtures; model retention, installed-host adoption, complete default Turn closure, independent Standard/workspace packaging and full original-goal regression remain open. The bounded refactor keeps the existing capability owner and can be reversed while preserving canonical task state.

Signed-off-by: lusendong.6789 <lusendong.6789@bytedance.com>
Co-authored-by: TRAE CLI <traecli@bytedance.com>
Signed-off-by: lusendong.6789 <lusendong.6789@bytedance.com>
Co-authored-by: TRAE CLI <traecli@bytedance.com>
@cocolord cocolord changed the title Defer issue-fix execution imports during goal startup Isolate startup metadata and restore installed refinement guidance Nov 12, 2026

@loopx-agent loopx-agent left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

通过代码评审(8f6fb368e4d5)。 默认启动不再载入未选择的 issue-fix 执行依赖;显式路由、旧导入和拒绝路径保持。新版还修复了安装技能的失效链接,安装检查已通过。

Reviewer: model_agent · gpt-6.1-sol · OpenAI · runtime_reported · reasoning_effort=max

动机 / Motivation

用 loopx-core 启动 Goal 的操作者会遇到这个问题;只打开帮助或预览、尚未选择 issue-fix 时也会触发。 基点为了生成命令提示就载入候选预检、工作流和仓库分析;改后默认启动只读取轻量命令元数据,显式选择 issue-fix 才读取预检契约。 独立安装的默认帮助和预览输出逐字节一致,每条默认路径载入的 LoopX 模块从 186 个降到 171 个。 这次不改资格判定、权限、持久化或 issue-fix 执行入口。 完整 V1–V5/V10、R1/M3、独立 Standard/workspace 安装和真实宿主采纳仍待验证。

本次按 docs/architecture/rfcs/monorepo-distribution-split-v0.md 的已接受版本 ce69003c73378fdb067a6b86e7b60feb50cb1b77 审查。V1 看按需加载;V3 看完整成功和失败输出;V5 看原安装入口能否完整恢复或拒绝陈旧内容。I3 的旧模块路径继续可用;这次不申报跨包搬迁或弃用完成。V10 的完整成本资格仍开放。INSTALL-GUIDANCE 核对安装后的示例可达性与原授权条款。

改动思路 / Approach

本批覆盖核心安装的默认启动、显式 issue-fix 预览、旧 Python 导入、安装后上下文恢复和 project skill 示例可达性。复用同一组命令模板,把它们放在现有 issue-fix 所有边界内;默认导入叶模块,显式预检沿原分支,workflow_plan 保留三个旧公开名称。 只延后预检导入仍会经旧模板传递加载工作流;把现有模板移到同一能力的轻量模块可同时给启动和 catalog 使用。显式选择仍走既有候选预检,预览仍要求先取得资格,成功预览不激活执行。

具体改动 / Changes

默认帮助和预览各做了八对独立安装的进程对照;退出码、标准输出和标准错误逐字节一致。显式路由的 55,322 字节预览也一致,仍有 qualification_required。catalog 条目不变,加载 25→5。旧路径对含空格和引号的两组参数生成同样的命令文本。两份安装的七类对照都未创建隔离 registry 或 runtime。

英中 RFC 和 core README 记录本批边界。真实已安装 CLI 上 off 保留原值;部分读取、丢失 ACK 和独立 PostCompact 触发恢复,修订与未变输入可回读完整必要内容,捕获文件变更被拒绝。新回归在 base/head 都过。 所以这项测试补的是既有协议在 wheel 上的回归覆盖。

已安装指引 的相对链接在安装后会越出 bundle;新版改指向同一固定版本的公开示例。我核对了文件字节和目标锚点,相关流程和授权句子未动。

对主干的风险 / Risks

如果延迟导入时也省掉显式分支的预检或资格字段,预览可能可以解析、却无法继续。已对照实际显式预览、无效路由、仅 core 时的工作流拒绝和旧 API;完整退出/诊断一致,未写入隔离状态。基点到第一版的两份独立 core wheel 对照保留;新头相对第一版只改了三份 Markdown,运行时、包定义和测试源码逐字节不变。新头的仓库安装、打包 CLI 和全部 standard 实际检查通过。 本地安装检查在基点和第一版曾因同一个已安装技能链接越界失败。新提交只替换该引用,保留流程、同意和检查范围条款;安装检查已在新头通过。 完整标准检查在新头跑过:6 个直接检查和 19 个选中检查均通过;含本地安装、打包 CLI、语义词汇、issue-fix 路由、public boundary 和双语 RFC。 示例现在需按需访问固定公开链接;原启动路径不增加网络请求。外部 CI 未查询。

我的整体评价 / Verdict

同意当前提交。源码聚焦套件 307 通过、1 跳过;head 独立安装 20 通过;同一上下文用例对 base 也通过。 检查的是 bootstrap CLI 和 installed core context reader;没有渲染界面改动。 新的导入边界是有意变化;此次不能据此承诺全轮次更快、原生宿主已采纳或模型保留了上下文。 当前提交的 19 个选中检查均通过;基点和早期头的失败观察分别保留。

English verdict: APPROVE. Complete installed startup and explicit-route behavior is preserved while unused Issue Fix imports are removed. Existing context recovery is covered, and the repaired pinned guidance link passes the exact-head local installer and standard checks.

@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; model=GPT-6; provider=OpenAI; self_reported (runtime observation unavailable: session_not_bound)

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

动机 / Motivation

从独立 core 启动自然语言目标,以及读取安装后恢复说明的用户和现有模型宿主。 同一目标原先为命令提示加载 issue-fix 执行模块,安装指导还指向包外文件;现在启动输出完整不变,执行依赖延后,恢复示例可从固定公开源读取。 默认启动依赖缩小,安装后的上下文恢复和支持资料链接通过真实 CLI 与安装检查。 本 PR 不完成全部最小组合、宿主采用、模型上下文保留或完整成本验收。 完整 Turn 默认闭包、双 preset、真实宿主上下文采用、独立 Standard/workspace 分包和原目标完整回归仍开放。

这不是用模块数量或测试数量代替完整产品验收。已检查真实启动、安装和上下文回读路径:原默认入口为了生成命令提示导入执行代码,安装指导还无法解析自己的支持示例。当前改动去掉这条实际依赖耦合,并保留所有原目标、权限与恢复约束。

改动思路 / Approach

命令模板保留在既有 issue-fix capability 的轻元数据层;执行、状态和权限决策继续由原 owner 负责,公开例子的内容也不复制成第二份规则。 本轮交付可逆的启动依赖隔离、安装上下文协议回归和一处安装指导链接修复;完整 R1 继续由现有 runtime/host/installation owner 推进。 单独延后 bootstrap 导入仍会留下 catalog 的 workflow 依赖;另写一套模板或复制 authority 说明会造成重复规则。这里直接移动既有纯命令生成代码,真实调用方共用一个定义,workflow 保留现有导入接口。支持说明改为经网络读回核验的固定公开源,仅在调用方按需打开时读取,不增加自动网络请求。

入口仍使用既有 registrar/handler;选定 route 的预检、TypeScript state/context、lease、validation、settlement 和 replay 没有换 owner。安装 context-project/read/hook 的正例和反例使用实际 wheel/进程及完整值 oracle,原生事件与 squares host 则明确是协议夹具。

具体改动 / Changes

评审 head:8f6fb368e4d5be79488fe6c9578350fadf6f5ac9。采用修改前的 docs/architecture/rfcs/monorepo-distribution-split-v0.md,spec revision:ce69003c73378fdb067a6b86e7b60feb50cb1b77,对照 §5、§9、§11 原验收。Bootstrap 的命令发现职责得到实际修复;V1、V2、V3、V4、V5、V10、R1、M3 保持 deferred:本轮只资格化启动/安装传输边界,完整双 preset、真实宿主采用、网络/模型与分包验收没有被新 checkpoint 改写为通过。

关键代码讲解

  • bootstrap_command_pack.py::_selected_goal_capability_route 在 None 返回和非法值校验之后,才为显式 issue-fix 导入 preflight metadata。它没有根据文字或 URL 推断 route,完整预检 packet 与原版相同。
  • command_templates.py::build_issue_fix_pr_lifecycle_command 保留原占位符、shell quoting、goal/agent/project 参数及 transition 命令。catalog 与 workflow 消费同一个定义,原导入名仍可用。
  • command_templates.py::build_issue_fix_goal_command_templates 只依赖 stdlib,继续给 bootstrap 返回原四条命令模板。完整 default/explicit/help 输出逐字节相同;源码和安装的反例能发现旧 workflow_plan eager import。

test_installed_core_context.py 从独立安装调用既有 CLI,检查 off 的完整原值、部分页/丢 ACK、修订后的完整约束、PostCompact、超出 JavaScript 安全整数的值和改字节拒绝。它也通过原版安装,证明是已有协议的安装回归边界;没有把捕获文件、digest 或 ACK 当作模型保留证明。技能里的支持链接只改 URI,原说明条款和固定公开源的完整字节保持一致。双语 RFC 记录同一边界,未修改冻结标准。

对主干的风险 / Risks

PASS:源码 348 项、最终独立 core 20 项、Node 不在 PATH 时的 selected help、完整启动/显式预检对照、公开源字节核验及修复后的完整本地安装 smoke。最终 premerge 的 6 个 direct、19 个 selected 检查全部通过;包含双语、compile、semantic/maintainability、安装、issue-fix、项目生命周期、content-ops 和公开边界。

负例仍要求完整交付才建立 baseline,standalone PostCompact 必须恢复,改动捕获字节或 lease generation/错误产物必须拒绝,重放不得再调用 host 或扣额。旧 eager-import 与包外链接的真实失败,以及过早启动安装测试、错误 test oracle 和受磁盘限制的基线尝试均分别保留;没有减少原断言或提高预算来消除失败。

三十对交替安装样本中,help 首次入口 median/p95 为 314.73/362.55→181.25/303.01 ms,preview 为 542.36/670.44→311.08/572.66 ms;head preview 的最大 1,092.15 ms 保留。这里是普通 OS/bytecode 缓存下的新解释器及后续 warm 路径;D6 的 selected-help p95 建议值和完整 V10 不能据此宣称通过。模型 tokens、网络 endpoint 完整性、全 Turn 闭包和完整成本仍未资格化。一个 native Windows/PowerShell case 本地跳过,远端 CI 完成度与 merge readiness 单独核对。

远端 kernel-static-checks 的 FAIL 已定位为 tests/test_lark_private_conversations.py:2816 的 ordinary 导入触发 F811。同一固定 base 和本 head 的该文件 blob 完全相同,同一 Ruff stdin-filename 检查在两份不可变源上复现同一 finding;本地完整 lint 也仅此一项。它是当前 PR 未改变的基线问题,已有 #8657 修复提案。原 FAIL、级联 checks/pytest/merge-gate 失败以及尚未完成的 build/browser 检查均保留为 diagnostic;这不使本 PR 的已通过验证失效,也不等于 merge readiness 已通过。

语义与 CI 对齐

本 PR 复用现有 route/schema/authority vocabulary,diff advisory 无支持的新 carrier;完整 semantic gate 通过,手工核对也没有新增 classifier、权限或状态 owner。完整命令文本和 actor、ordering、scope、evidence、continuation、stop 条款保持原义。HTTP 支持链接不是激活、执行或 authority,独立检查也不代表整个 Goal 完成。

我的整体评价 / Verdict

APPROVE;problem_context 为 justified_increment。long_horizon 保持原义,user_experience 得到有证据的改善:同一目标、同一完整输出和已有决策轮次,以更小的启动闭包到达既有 owner,安装指导也可读取支持源。命令模板保留在既有 issue-fix capability 的轻元数据层;执行、状态和权限决策继续由原 owner 负责,公开例子的内容也不复制成第二份规则。 本轮交付可逆的启动依赖隔离、安装上下文协议回归和一处安装指导链接修复;完整 R1 继续由现有 runtime/host/installation owner 推进。

检查了 start-goal、catalog 元数据、安装技能和既有 CLI/readback 路径;没有修改 App/Lark 的渲染界面或 CLI 开场输出。 本轮不重新资格化完整 App/Lark、实际模型 compaction 或安装宿主采用。 上下文事件仍是协议夹具,公开例子的网络可用性仍属于按需支持资料边界。当前结论批准本次可逆交付,完整 R1/原目标依旧开放。合入前仍须确认这个 unchanged head、无未解决 findings/threads、最终 readiness 及既有 maintainer authority。

English verdict: APPROVE 8f6fb36; complete startup parity, installed context/recovery and repaired guidance pass required local validation. Full minimal composition and host/model acceptance remain open.

@cocolord
cocolord merged commit 1cb6caf into main Nov 12, 2026
32 of 39 checks passed
@cocolord
cocolord deleted the codex/rfc-r1-composition-20261112t201145 branch November 12, 2026 22:19
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.

2 participants