Repository navigation
Isolate startup metadata and restore installed refinement guidance - #8661
Conversation
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>
loopx-agent
left a comment
There was a problem hiding this comment.
通过代码评审(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
- _selected_goal_capability_route:未选择或无效 route 在导入预检前返回;显式分支仍生成原 preflight。
- build_issue_fix_goal_command_templates:原命令模板移入 issue-fix 的轻量模块;沿用同一组模板、参数和引用。
- ISSUE_FIX_CATALOG_ENTRY:catalog 改从模板叶模块读常量,默认关闭声明保留。
- workflow_plan compatibility exports:旧模块继续导出同样三个公开名称,工作流逻辑仍在原 owner。
- test_installed_context_recovers_partial_delivery_revisions_and_compaction:用同一已安装 CLI 的 context-project/read/hook 演练完整读取和恢复。
默认帮助和预览各做了八对独立安装的进程对照;退出码、标准输出和标准错误逐字节一致。显式路由的 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
left a comment
There was a problem hiding this comment.
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.
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:
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.