清理随包 Codex 资源里的旧布局残留
Project CI / AI game creator shell Rust crates (push) Successful in 1m24s
Project CI / AI game creator shell Rust smoke (push) Successful in 1m54s
Project CI / Backend tests (push) Successful in 4m3s
Project CI / Frontend tests (push) Successful in 1m59s
Project CI / Native shell tests (push) Successful in 6m11s
Project CI / AI game creator shell Rust lane 2/2 (push) Has been cancelled
Project CI / AI game creator shell Rust lane 1/2 (push) Has been cancelled
Project CI / Repository checks (push) Has been cancelled
Project CI / AI game creator shell web tests (push) Has been cancelled

- src-tauri/build.rs:stage_codex_target 拷贝前调用新增的 prune_stale_codex_components,删掉不在本轮布局也不在 manifest.json/NOTICE.md 白名单里的文件并收掉空目录
- 复验:重建后 resources/codex/win-x64 根目录那份 0.147.0 codex.exe 与临时备份被清除,六个声明组件、manifest.json 与 NOTICE.md 保留;同一二进制下 loopback 夹具仍跑到 requests=3、markers.ready=true
- 文档:DirectProject 里程碑本轮核对细化第二层问题(不是 codex 版本漂移、不是 stdin;指向 Drop for ExecutionBinding 的收尾竞态,需插桩),并说明两条运行时验收本质要真实客户端窗口;pitfalls 记录该环境陷阱与修复
This commit is contained in:
kdletters
2026-09-28 22:55:41 +08:00
parent 7ec984d0d2
commit 90a414ebf3
3 changed files with 43 additions and 2 deletions
@@ -115,6 +115,10 @@ fn stage_codex_target(manifest_dir: &std::path::Path, target: &str) {
panic!("内置 Codex CLI 第三方声明缺失:{}", notice.display());
}
fs::create_dir_all(&target_dir).expect("创建内置 Codex CLI 资源目录失败");
// 这份目录是随包资源(Windows:`resources/codex/win-x64/**` → `coding-agent/win-x64/**`),
// 只能包含本轮布局声明的组件。上一版布局留下的旧二进制(例如包根目录那份 0.147.0
// `codex.exe`)会长期留在原地:既误导本地核对与夹具,也让「随包内容」与清单不一致。
prune_stale_codex_components(&target_dir, layout.files);
let mut file_hashes = serde_json::Map::new();
for relative in layout.files {
let source_path = source.join(relative);
@@ -174,6 +178,42 @@ fn stage_codex_target(manifest_dir: &std::path::Path, target: &str) {
}
}
/// 删除 `target_dir` 下不在本轮布局内的残留文件;空目录一并收掉。
fn prune_stale_codex_components(target_dir: &std::path::Path, files: &[&str]) {
const ALWAYS_KEEP: &[&str] = &["manifest.json", "NOTICE.md"];
fn walk(root: &std::path::Path, directory: &std::path::Path, files: &[&str], keep: &[&str]) {
let Ok(entries) = fs::read_dir(directory) else {
return;
};
for entry in entries.flatten() {
let path = entry.path();
let Ok(kind) = entry.file_type() else {
continue;
};
if kind.is_dir() {
walk(root, &path, files, keep);
if fs::read_dir(&path)
.map(|mut remaining| remaining.next().is_none())
.unwrap_or(false)
{
let _ = fs::remove_dir(&path);
}
continue;
}
let Ok(relative) = path.strip_prefix(root) else {
continue;
};
let relative = relative.to_string_lossy().replace('\\', "/");
if files.contains(&relative.as_str()) || keep.contains(&relative.as_str()) {
continue;
}
eprintln!("cargo:warning=清理内置 Codex 组件残留:{relative}");
let _ = fs::remove_file(&path);
}
}
walk(target_dir, target_dir, files, ALWAYS_KEEP);
}
fn seed_task_group_id(
group: &shared_contracts::game_creation_app::GameCreationAppAgentGroup,
) -> &'static str {
@@ -81,4 +81,4 @@ AGC 项目开发对话的显示与恢复只依赖两项输入:**项目对话
- **缺陷**:全新项目的第一次对话直接失败。夹具 `--cases completed` 下 AGC 进程 4 秒退出、`requests=[]`,stderr 只有 `Codex app-server JSON-RPC 失败:items must not be empty`。根因是 `thread_created` 时无条件发 `thread/inject_items`,而空项目没有 `.agent/conversations/project.jsonl`,载荷成了 `items: []`;codex app-server 0.155.1 起把空数组当协议错误。GUI 路径没暴露是因为前端会先写用户消息,CLI / 无前端宿主是裸的。
- **修复**:`build_direct_project_history_injection_params()` 在历史为空时返回 `Ok(None)`,调用方跳过注入;新增单测「空历史不得构造载荷」「有历史仍构造 1 条」。`cargo test … direct_project_history_wire`(3 passed)与 `cargo fmt --check` 通过。
- **复验**:同一条命令下夹具从「4 秒失败」变成「走到 loopback Provider:`requests=3`、`markers.ready=true`、AGC exit=0」,说明第一层已经修通。
- **仍未完成(第二层)**:同一用例仍断言失败,执行账本落在 `phase=interrupted`(期望 `completed`),stderr 为 `agent.runner.failed: Codex app-server 连接终止 … stderrBytes=1242`,CLI 回执是「执行通道已断开,不能自动重放未确认操作」。现象指向 CLI 收尾阶段的关闭竞态,需要先拿到 app-server 的 stderr 原文(当前只落字节数与 sha256)。两条运行时验收**继续未勾选**。
- **仍未完成(第二层)**:修好第一层后,夹具的**全部 6 个用例**都走到 loopback Provider(`requests=3`、`markers.ready=true`、exit=0),但统一失败在账本阶段——`phase=interrupted`(期望 `completed`/`exhausted`),stderr 为 `agent.runner.failed: Codex app-server 连接终止 … stderrBytes=1242`,CLI 回执是「执行通道已断开,不能自动重放未确认操作」。本轮排除:① 不是 codex 版本漂移(换 0.147.0 结果相同);② 不是 stdin 被忽略(改成 `pipe` 结果相同)。可疑路径是主机收尾竞态:`codex_app_server/execution.rs:70` 的 `Drop for ExecutionBinding` 在 `!background_done && !closed` 时补一次 `cancel_from_host()`(`execution.rs:706`,写的就是账本里那句「本轮已由宿主终止…」),同文件 750 行的通道断开处置又会先记一条 transport 失败。**结论:两条运行时验收本质上要真实客户端窗口**(夹具只能证历史持久化,证不了「界面不显示忙碌态」),因此这两条继续未勾选;CLI 收尾竞态作为独立问题记进 pitfalls,动它之前先插桩(抓 app-server stderr 原文 + Drop/transport 时序日志)。
@@ -6113,4 +6113,5 @@ Cocos Creator 根目录由 `package.json.creator.version` 与普通 `assets/`
- **根因**:`codex_app_server/mod.rs` 在 `thread_created` 时无条件调用 `thread/inject_items`,载荷由 `build_direct_project_history_injection_params()` 从 `.agent/conversations/project.jsonl` 构造;**全新项目该文件不存在 → `items: []`**,而 codex app-server 0.155.1 起把空数组当协议错误拒绝。GUI 路径之所以没暴露:前端会先调 `append_direct_project_conversation_message` 把用户消息写进历史,注入时至少有 1 条;`--direct-codex-chat` 这类无前端宿主(以及任何直接调用 `run_direct_game_creator_turn_at` 的夹具/CLI)没有这一步。
- **处理(2026-09-28 已修)**:`build_direct_project_history_injection_params()` 在历史为空时返回 `Ok(None)`,调用方跳过 `thread/inject_items`;新增两条单测(空历史不得构造载荷、有历史仍构造 1 条)。
- **复验**:同一条夹具命令下,AGC 进程从「4 秒失败、`requests=[]`」变成「走到 loopback Provider、`requests=3`、`markers.ready=true`、exit=0」——第一层缺陷确实修掉了。
- **仍未解决(第二层,另一个问题)**:同一个夹具的 `completed` 用例仍断言失败,原因是执行账本 `phase=interrupted`(期望 `completed`),stderr 为 `agent.runner.failed: Codex app-server 连接终止 … exitStatus=unknown;stderrBytes=1242`,CLI 回执文本也是「执行通道已断开,不能自动重放未确认操作」。看起来是 CLI 收尾阶段的关闭竞态(账本被记成宿主终止),与空历史注入不是同一件事;要动它需要先抓 app-server 的 stderr 内容(应用目前只记字节数与摘要,不落原文)。
- **仍未解决(第二层,另一个问题)**:同一个夹具的全部 6 个用例(`completed/passes/mcp/mcp-write/native/deadline`)在修好第一层后都走到 loopback Provider,但**统一**失败在账本阶段:`phase=interrupted`(期望 `completed`/`exhausted`),stderr 为 `agent.runner.failed: Codex app-server 连接终止 … exitStatus=unknown;stderrBytes=1242`,CLI 回执文本为「执行通道已断开,不能自动重放未确认操作」。本轮已排除两种可能:① 不是 codex 版本漂移——把 `resources/codex/win-x64/bin/codex.exe` 换成 0.147.0(仓库里那份 root 残留)后结果完全相同;② 不是 stdin 被忽略——把夹具的 `stdio[0]` 从 `ignore` 改成 `pipe` 后同样 `interrupted`。定位到的可疑路径是主机收尾竞态:`agent/codex_app_server/execution.rs:70` 的 `Drop for ExecutionBinding` 在 `!background_done && !closed` 时补一次 `cancel_from_host()`,而该函数(`execution.rs:706-716`)会写「本轮已由宿主终止;正在收束执行器,未完成项保留。」——账本里的 `terminalReport` 正是这句;同文件 `execution.rs:750` 的通道断开处置又会先记一条 transport 失败事实。下一步要动它得先做插桩(抓 app-server 的 stderr 原文、给 Drop 与 transport 记账加时序日志),不要在没插桩前猜着改守卫。
- **顺带记一条环境陷阱(2026-09-28 已修)**:`apps/ai-game-creator-shell/src-tauri/resources/codex/win-x64/` 下曾有两个 codex 二进制——`bin/codex.exe` 是**真正被解析**的那份(0.155.1),而包根目录那份 `codex.exe` 是 0.147.0 的旧残留(tauri 的 Windows 资源映射只引用 `bin/` 等路径),检查都查不出来,却会让本地核对误判「应用跑的是 0.147.0」。根因是 `src-tauri/build.rs` 的 `stage_codex_target()` 只按布局拷贝、从不清理目录,旧布局的组件会永久留在随包资源目录里。现在加了 `prune_stale_codex_components()`:拷贝前删掉不在本轮布局、也不在 `manifest.json`/`NOTICE.md` 白名单里的文件并收掉空目录;实测重建后根目录 `codex.exe` 被清掉、六个声明组件与清单/声明保留。