diff --git a/apps/ai-game-creator-shell/src-tauri/src/agent/claude_code_cli.rs b/apps/ai-game-creator-shell/src-tauri/src/agent/claude_code_cli.rs index 827b06682..9f5b91221 100644 --- a/apps/ai-game-creator-shell/src-tauri/src/agent/claude_code_cli.rs +++ b/apps/ai-game-creator-shell/src-tauri/src/agent/claude_code_cli.rs @@ -732,7 +732,9 @@ pub(crate) async fn direct_game_creator_claude_code_chat_at( client_turn_id: Option<&str>, observer: Option<&mut (dyn FnMut(DirectCodexTurnObservation) + Send)>, ) -> Result { + direct_turn_trace("claude-executor-enter"); let (mcp_url, mcp_token) = start_external_mcp_loopback(root, llm.web_search_enabled).await?; + direct_turn_trace("claude-mcp-loopback-ready"); let _mcp_guard = ExternalMcpStopGuard; let home = crate::project::resolve_local_project_path(root, ".agent/runtime/claude-code/home")?; std::fs::create_dir_all(&home) @@ -742,6 +744,7 @@ pub(crate) async fn direct_game_creator_claude_code_chat_at( .lock() .ok() .and_then(|sessions| sessions.get(&session_key).cloned()); + direct_turn_trace("claude-sidecar-start"); let payload = serde_json::json!({ "type": "turn", "id": client_turn_id.unwrap_or("claude-direct-turn"), @@ -769,6 +772,7 @@ pub(crate) async fn direct_game_creator_claude_code_chat_at( client_turn_id, ) .await?; + direct_turn_trace("claude-sidecar-done"); if let Some(session_id) = sidecar .result .get("session_id") @@ -785,6 +789,7 @@ pub(crate) async fn direct_game_creator_claude_code_chat_at( .filter_map(|event| serde_json::to_string(&event).ok()) .collect::>(); stream.push(String::new()); + direct_turn_trace("claude-parse-start"); parse_direct_stream_result(stream.join("\n").as_bytes(), observer) } diff --git a/apps/ai-game-creator-shell/src-tauri/src/agent/direct_runtime/mod.rs b/apps/ai-game-creator-shell/src-tauri/src/agent/direct_runtime/mod.rs index f5059ed82..92d673ebb 100644 --- a/apps/ai-game-creator-shell/src-tauri/src/agent/direct_runtime/mod.rs +++ b/apps/ai-game-creator-shell/src-tauri/src/agent/direct_runtime/mod.rs @@ -4294,6 +4294,24 @@ pub(crate) fn check_direct_turn_preconditions( .map_err(|detail| DirectTurnError::InputRejected { detail })?; Ok(()) } +/// Direct 回合在 `direct_execution::begin` 之后的分段 trace。 +/// +/// 起因：2026-10-02 连续四轮回合都停在 `begin` 之后——账本 `phase=working`、 +/// `nextSequence=0`、`active={}`、没有任何错误 sidecar、stderr 无 panic、连 +/// `agent.direct_codex.*` 都没有。窗口就夹在 `begin` 与执行器调用之间，需要知道 +/// 具体停在哪一步。测试态不写，避免污染本机 AppData。 +#[cfg(not(test))] +pub(crate) fn direct_turn_trace(stage: &str) { + let _ = crate::append_application_log( + "info".to_string(), + "direct-turn".to_string(), + format!("agent.direct_turn.trace stage={stage}"), + ); +} + +#[cfg(test)] +pub(crate) fn direct_turn_trace(_stage: &str) {} + pub(crate) async fn run_direct_game_creator_turn_at_with_creation_type_and_emitter( root: &Path, prompt: &str, @@ -4401,6 +4419,7 @@ async fn run_direct_game_creator_turn_inner( })?; let execution_session = execution_guard.session(); execution_session.set_analytics_capture(capture.clone()); + direct_turn_trace("session-ready"); let started = execution_session .newly_accepted .then(std::time::Instant::now); @@ -4411,6 +4430,7 @@ async fn run_direct_game_creator_turn_inner( } } } + direct_turn_trace("run-analytics-accepted"); // 在 guard 仍存活时冻结整体结果，避免 Drop 的中断收尾覆盖真实失败原因。 let result: Result = async { if let Some(report) = super::direct_delivery::terminal_report(&execution_session) { @@ -4444,6 +4464,7 @@ async fn run_direct_game_creator_turn_inner( ); } } + direct_turn_trace("goal-analytics-accepted"); emit_direct_game_creator_progress(root, "codex.turn", "陶泥儿正在处理这条消息"); if let Some(emitter) = turn_emitter { emitter.emit("running", Some("preparing")); @@ -4460,6 +4481,7 @@ async fn run_direct_game_creator_turn_inner( DirectTurnError::turn_failed(DirectCodexFailureStage::CodeGeneration, error) }, )?; + direct_turn_trace("system-prompt-ready"); // 三维请求：把"自选三维技术栈、解除 Phaser 固定约束"的合同放在系统提示最前， // 避免被长度上限截断，也不阻断任何工具。 let engine_contract = direct_engine_three_dimensional_contract(root, prompt); @@ -4471,6 +4493,7 @@ async fn run_direct_game_creator_turn_inner( None => base_system_prompt, }; let reply = if let Some(emitter) = turn_emitter { + direct_turn_trace("executor-dispatch"); let client_turn_id = emitter.turn_id().to_string(); let emitter = emitter.clone(); let mut observer = |observation: DirectCodexTurnObservation| { diff --git a/apps/ai-game-creator-shell/src-tauri/src/agent/thread_manager/dispatch.rs b/apps/ai-game-creator-shell/src-tauri/src/agent/thread_manager/dispatch.rs index 5eabb23b6..be5bc35cb 100644 --- a/apps/ai-game-creator-shell/src-tauri/src/agent/thread_manager/dispatch.rs +++ b/apps/ai-game-creator-shell/src-tauri/src/agent/thread_manager/dispatch.rs @@ -13,13 +13,16 @@ use std::path::{Path, PathBuf}; +use futures::FutureExt; + #[cfg(test)] use crate::agent::PendingTurn; use crate::agent::{ append_direct_project_user_message_at, claim_pending_turn, complete_turn_if_reserved, - direct_codex_user_item_to_prompt, direct_now_ms, redact_agent_runtime_error, - run_direct_game_creator_turn_at_with_creation_type_and_emitter, thread_id_for_project, - DirectGameCreatorTurnUpdateEmitter, DirectTurnError, DirectTurnTerminal, DispatchedTurn, + direct_codex_user_item_to_prompt, direct_now_ms, record_direct_codex_failure, + redact_agent_runtime_error, run_direct_game_creator_turn_at_with_creation_type_and_emitter, + thread_id_for_project, DirectCodexFailureStage, DirectGameCreatorTurnUpdateEmitter, + DirectTurnError, DirectTurnTerminal, DispatchedTurn, }; /// 一次放行的占用。持有它就代表这一轮还没收口。 @@ -84,13 +87,90 @@ impl Drop for TurnReservation { fn drop(&mut self) { // 兜底：任务 panic、future 被丢弃、或今后在终态之前新增的 `?` 早退。 // 这类失败说不出原因，只给分类；能说清原因的错误必须由调用方在更早的地方显式收口。 - let _ = self.finish_if_unfinished(DirectTurnTerminal::host_dropped()); + let finalized_by_guard = self.finish_if_unfinished(DirectTurnTerminal::host_dropped()); + // 兜底一旦真的收口，就说明这一轮**从未写下终态**：深层既没成功也没失败地退出了。 + // 这条必须留应用日志，否则离线只剩一个 `phase=working` 的账本，无从判断是哪一层 + // 提前退出（真实案例：2026-10-02 连续两轮只留下 working 账本，errors/ 与 + // application.log 全空）。测试态不写，避免用例污染本机 AppData。 + #[cfg(not(test))] + if finalized_by_guard { + // 项目侧也留一份脱敏诊断：用户在项目目录里就能看到这一轮的收场， + // 不必只依赖 AppData 的应用日志。 + let failure = DirectTurnError::TurnFailed { + stage: DirectCodexFailureStage::CodeGeneration, + detail: format!( + "DirectProject 宿主任务提前结束（panic、future 被丢弃或被取消），本轮未写下终态；threadId={} turnToken={}", + self.thread_id, self.token + ), + }; + let _ = record_direct_codex_failure(Path::new(&self.thread_id), &failure, None); + let _ = crate::append_application_log( + "error".to_string(), + "direct-turn".to_string(), + format!( + "agent.direct_turn.host_dropped threadId={} turnToken={} userItemId={} panicking={}", + self.thread_id, + self.token, + self.user_item_id.as_deref().unwrap_or("-"), + // true = 这一轮是被 panic 带走的；false = future 被丢弃或取消。 + std::thread::panicking() + ), + ); + } // 占用的释放就是"这个项目腾出了跑回合的位置"：踢一脚，队里排着的消息不必等下一次用户动作。 // panic 也走这里，所以队列不会因为一个任务炸掉而永久停住。 kick_queue_dispatch(Path::new(&self.thread_id)); } } +// 运行段经 task-local 携带回合身份，panic hook 据此把 panic 位置与负载写进应用日志。 +// task-local 而非 thread-local：多线程 runtime 下 future 会跨 worker 迁移。 +tokio::task_local! { + static DIRECT_TURN_PANIC_CONTEXT: Option<(String, String)>; +} + +/// 兜底诊断：Direct 回合任务 panic 时，把位置与负载写进应用日志。 +/// +/// 没有它时，panic 只会让 `TurnReservation::drop` 把回合收成 `HostDropped`——那是"说不出原因" +/// 的分类，离线只留一个 `phase=working` 的账本，`.agent/runtime/errors/` 与 `application.log` +/// 全空，无法判断是哪一层退出的（真实案例：2026-10-02 连续两轮如此）。 +fn ensure_direct_turn_panic_hook() { + static ONCE: std::sync::Once = std::sync::Once::new(); + ONCE.call_once(|| { + let previous = std::panic::take_hook(); + std::panic::set_hook(Box::new(move |info| { + if let Ok(Some((thread_id, token))) = DIRECT_TURN_PANIC_CONTEXT.try_with(Clone::clone) { + let location = info + .location() + .map(|location| { + format!( + "{}:{}:{}", + location.file(), + location.line(), + location.column() + ) + }) + .unwrap_or_else(|| "未知位置".to_string()); + #[cfg(not(test))] + let _ = crate::append_application_log( + "error".to_string(), + "direct-turn".to_string(), + format!( + "agent.direct_turn.panic threadId={} turnToken={} location={} payload={}", + thread_id, + token, + location, + info.payload_as_str().unwrap_or("未知 panic 负载") + ), + ); + #[cfg(test)] + let _ = (thread_id, token, location); + } + previous(info); + })); + }); +} + /// 踢一脚：让队首那条待发消息有机会变成一轮真的回合。 /// /// 幂等，三个调用点：**入队之后**（队列空且没有回合在跑时，放行就是这一脚，不必再等一个调度周期）、 @@ -109,10 +189,23 @@ pub(crate) fn kick_queue_dispatch(root: &Path) { crate::platform_session::current_platform_session_write_state().identity_generation; let reservation = TurnReservation::resume(&thread_id, &dispatched); let root = root.to_path_buf(); - tauri::async_runtime::spawn(async move { + ensure_direct_turn_panic_hook(); + let panic_context = Some((thread_id.clone(), dispatched.token.clone())); + tauri::async_runtime::spawn(DIRECT_TURN_PANIC_CONTEXT.scope(panic_context, async move { run_dispatched_direct_turn(root, dispatched, reservation, release_identity_generation) .await; - }); + })); +} + +/// 把 panic 负载转成可读文本：`panic!("…")` 的负载是 `&str`，`panic!("{x}")` 是 `String`。 +fn direct_turn_panic_detail(payload: &(dyn std::any::Any + Send)) -> String { + if let Some(text) = payload.downcast_ref::<&str>() { + (*text).to_string() + } else if let Some(text) = payload.downcast_ref::() { + text.clone() + } else { + "未知 panic 负载".to_string() + } } /// 放行之后这一段：取这一轮的调用身份 → 落盘用户条目 → 下发 → 跑整轮。 @@ -149,16 +242,38 @@ async fn run_dispatched_direct_turn( crate::agent::codex_app_server::emit_direct_thread_user_item(&root, &canonical_user_item); let capture = crate::analytics::gui::capture_writer_context(); let emitter = DirectGameCreatorTurnUpdateEmitter::new(&root, turn_id); - let outcome = run_direct_game_creator_turn_at_with_creation_type_and_emitter( - &root, - &prompt, - dispatched.pending.creation_type.as_deref(), - Some(&emitter), - Some(canonical_user_item.clone()), - capture, - release_identity_generation, + // 回合体 panic 不能落到 `TurnReservation` 的 Drop 兜底：那会把这一轮收成"说不出原因"的 + // `HostDropped`，症状正是用户看到的「本轮执行已中断，请重试」，而项目侧诊断与应用日志 + // **一行都不会有**（真实案例：2026-10-02 连续两轮）。这里把 panic 转成分类失败， + // 走与正常失败同一条收口：先写脱敏诊断，再写终态。 + let outcome = match std::panic::AssertUnwindSafe( + run_direct_game_creator_turn_at_with_creation_type_and_emitter( + &root, + &prompt, + dispatched.pending.creation_type.as_deref(), + Some(&emitter), + Some(canonical_user_item.clone()), + capture, + release_identity_generation, + ), ) - .await; + .catch_unwind() + .await + { + Ok(result) => result, + Err(payload) => { + let failure = DirectTurnError::turn_failed( + DirectCodexFailureStage::CodeGeneration, + format!( + "DirectProject 宿主任务 panic：{}", + direct_turn_panic_detail(payload.as_ref()) + ), + ); + let stage = failure.turn_failure_stage(); + let detail = record_direct_codex_failure(&root, &failure, Some(turn_id.as_str())); + Err(DirectTurnError::TurnFailed { stage, detail }) + } + }; match outcome { Ok(_) => { // 深层的终态出口已经在 `run_turn` 里写出 `turn.completed`；这里只补最后一条回合更新。 diff --git a/apps/ai-game-creator-shell/src/styles.css b/apps/ai-game-creator-shell/src/styles.css index 86fb836d4..74d8e5e41 100644 --- a/apps/ai-game-creator-shell/src/styles.css +++ b/apps/ai-game-creator-shell/src/styles.css @@ -1332,6 +1332,19 @@ textarea { overflow: hidden; } +/* + * 页面高度契约：`.launcher-main` 是纵向 flex 列，页面根节点自己 `flex: 1 1 auto` 吃掉 + * 舞台剩余高度（有横幅时自动少掉横幅那一份），页面内部的 `height: 100%`、内层滚动容器 + * 才有确定高度可用。 + * + * 页面根节点上不要再写 `100vh` / `100dvh` / `calc(100vh - 32px)`：整窗高度含自绘标题栏， + * 比真实舞台高一个标题栏（窗口 100vh=800、舞台 750），底部会被裁掉，横幅也会被重复扣一次。 + * 窗口外壳用 `height: 100%` 覆盖这里的 `100dvh`，让页面贴合 `.window-chrome__content` + * 的真实舞台；没有窗口外壳时 `100dvh` 才是正确值。 + */ +.launcher-main:has(.launcher-home-page), +.launcher-main:has(.launcher-help-page), +.launcher-main:has(.launcher-template-library), .launcher-main:has(.launcher-projects-page) { display: flex; flex-direction: column; @@ -1340,6 +1353,20 @@ textarea { padding-bottom: 0; } +.launcher-main > .launcher-home-page, +.launcher-main > .launcher-help-page, +.launcher-main > .launcher-template-library { + flex: 1 1 auto; + height: auto; + min-height: 0; +} + +/* 帮助页没有内层滚动容器：占满剩余高度后自己滚动，底部留出与外壳一致的呼吸位。 */ +.launcher-main > .launcher-help-page { + overflow-y: auto; + padding-bottom: 42px; +} + .launcher-page header { display: flex; align-items: flex-end; diff --git a/apps/ai-game-creator-shell/src/view/home/index.tsx b/apps/ai-game-creator-shell/src/view/home/index.tsx index 79fbbc52a..64307288d 100644 --- a/apps/ai-game-creator-shell/src/view/home/index.tsx +++ b/apps/ai-game-creator-shell/src/view/home/index.tsx @@ -209,11 +209,7 @@ export default function HomeView({ } return ( -
+
(null); const viewportRef = useRef(null); - const sectionRef = useRef(null); - const [sectionHeight, setSectionHeight] = useState(null); const [viewportSize, setViewportSize] = useState({ width: 0, height: 0 }); const showEmptyLibrary = status === 'ready' && templates.length === 0; @@ -162,35 +160,10 @@ export default function TemplateLibraryView({ const showGrid = status === 'ready' && visibleTemplates.length > 0; /** - * 页面高度按**父级实测高度**定，不用百分比也不用 100vh。 - * - * 外壳样式 `.launcher-main > .platform-theme { height: 100% }` 特异性高于 Tailwind - * 工具类，而它的百分比在 `.launcher-shell { min-height: 100vh }` 这条链上是不定高， - * 页面会退化成内容高度（虚拟网格视口高度 0、卡片区空白）；`100vh` 又比真实舞台高 - * 一个标题栏（窗口 100vh=800、舞台 750），底部会被裁掉。这里直接量父级。 + * 页面高度不再自己量：`.launcher-main:has(.launcher-template-library)` 把外壳改成纵向 + * flex 列，`flex: 1 1 auto` 让这一页拿到确定高度（有横幅时自动少掉横幅那一份）， + * 下面的网格视口才能用 `flex-1` 量出真实尺寸。 */ - useEffect(() => { - const section = sectionRef.current; - const parent = section?.parentElement; - if (!section || !parent) { - return; - } - const apply = () => { - const height = Math.round(parent.getBoundingClientRect().height); - if (height > 0) { - setSectionHeight((current) => (current === height ? current : height)); - } - }; - apply(); - if (typeof ResizeObserver === 'undefined') { - window.addEventListener('resize', apply); - return () => window.removeEventListener('resize', apply); - } - const observer = new ResizeObserver(apply); - observer.observe(parent); - return () => observer.disconnect(); - }, []); - useEffect(() => { const element = viewportRef.current; if (!element) { @@ -279,9 +252,7 @@ export default function TemplateLibraryView({ return (
diff --git a/docs/project-memory/shared-memory/decision-log.md b/docs/project-memory/shared-memory/decision-log.md index 62d6161d0..c94736de9 100644 --- a/docs/project-memory/shared-memory/decision-log.md +++ b/docs/project-memory/shared-memory/decision-log.md @@ -6,6 +6,14 @@ - 影响范围：根开发脚本、AGC 开发启动编排、本地开发运维文档；不改变 API、schema、生产部署和独立 `npm run agc` 行为。 - 验证方式：参数/状态单测、开发栈健康端点 smoke、`.app/dev-stack.json` 身份复用检查、进程树收束检查。 +## 2026-10-02 launcher 页面高度契约：外壳分高度，页面不再自己算窗口高度 + +- 背景：PR #228（`6d2c275d3`）只给项目页补了「外壳纵向 flex + 页面 `flex: 1 1 auto`」的高度修复；其余页面仍各自算高度——帮助页没写高度也没有内层滚动容器，内容一长就被外壳 `overflow: hidden` 裁掉且无法滚动；首页用 `h-screen` / `h-[calc(100vh-32px)]`，模板库用 JS 量父级高度写内联 `height`。 +- 决策：把 #228 的修复提升为 `apps/ai-game-creator-shell/src/styles.css` 里的通用契约——`.launcher-main:has(<页面钩子>)` 改纵向 flex 列（`height: 100dvh`，窗口外壳 `height: 100%` 命中时贴合真实舞台；`min-height: 0`、`padding-bottom: 0`），`.launcher-main > <页面根节点>` 统一 `flex: 1 1 auto; height: auto; min-height: 0`；帮助页额外 `overflow-y: auto; padding-bottom: 42px`（它没有内层滚动容器）。契约钩子：`launcher-home-page`、`launcher-help-page`、`launcher-template-library`、`launcher-projects-page`。 +- 边界：页面根节点不再写 `100vh` / `100dvh` / `calc(100vh - Npx)`；横幅（`.launcher-promo`）是外壳里的真实行，有横幅就靠 flex 自动少一份，不手算偏移。工作台页（`.game-project-workbench`）在窗口外壳里已有等价契约（`.window-chrome__content ...` 那几条），移动端媒体查询里遗留的 `100dvh` / `calc(100dvh - 32px)` 由外层 `flex: 1 1 auto` 收缩兜住，本次不动。 +- 影响范围：`apps/ai-game-creator-shell/src/styles.css`、`src/view/home/index.tsx`、`src/view/template-library/index.tsx`。 +- 验证：Chromium 真机量测（1440x800 / 1440x560 / 390x844 / 390x560，带与不带横幅）——首页 / 模板库 / 项目页改动前后高度一致，帮助页从「被裁 112~150px 且无可滚动祖先」变为正好等于 `.window-chrome__content` 高度并可滚动到底；`npm run typecheck`、`npm run check:encoding`、`git diff --check`、`eslint` 通过。 + ## 2026-10-01 游戏广场评分展示边界 - 用户确认广场卡片增加一位小数的 10 分制平均分与评分人数，无有效评价显示“暂无评分”；保留现有排序、筛选、卡片打开详情及返回上下文。 diff --git a/docs/project-memory/shared-memory/pitfalls.md b/docs/project-memory/shared-memory/pitfalls.md index 59d9f8ea8..9e25578ef 100644 --- a/docs/project-memory/shared-memory/pitfalls.md +++ b/docs/project-memory/shared-memory/pitfalls.md @@ -8,6 +8,12 @@ - **原因**：Windows 的 `npm.cmd` 是批处理入口；Node `child_process.spawn('npm.cmd', args, { shell: false })` 会直接返回 `EINVAL`，还未执行根 `npm run dev`。 - **处理**：`scripts/dev-all.mjs` 在 Windows 使用 `shell: true`、`windowsHide: true` 启动 npm 子进程；POSIX 仍使用独立进程组，退出时按进程组收束。 - **验证**：Windows 实测根开发栈已启动并完成端口漂移（Web `3001`、API `8084`、worker `8085`、SpacetimeDB `3104`、后台 `3105`），之后 AGC 因当前工作区缺少 `@anthropic-ai/claude-agent-sdk` 退出；dev:all 已收束根栈进程。 +## 2026-10-02 AGC 页面在自绘标题栏外壳里自己算 `100vh`：底部被裁而且没得滚 + +- **现象**：帮助页（使用指南 / 联系客服 / 更新日志）在矮窗口里底部卡片看不到，把窗口拉高才出现；外壳 `.launcher-main { overflow: hidden }` 之下没有任何可滚动祖先，页面既滚不动也裁得干净。首页在通知横幅出现时用 `h-[calc(100vh-32px)]`，同样把窗口高度当成了舞台高度。 +- **原因**：AGC 桌面外壳是自绘标题栏（`--window-chrome-height`；窗口 100vh=800 时舞台只有 750），页面根节点写 `100vh` / `100dvh` / `calc(100vh - Npx)` 就比真实舞台高一整个标题栏，差额被外壳裁掉；横幅是 `.launcher-main` 里的真实行，再写 `-32px` 等于重复扣一次。帮助页还没有内层滚动容器，连「内容超高就在内部滚动」这条兜底也不存在。 +- **处理（现行口径）**：页面高度只由外壳分配——`apps/ai-game-creator-shell/src/styles.css` 里 `.launcher-main:has(<页面钩子>)` 是纵向 flex 列（`height: 100dvh`，窗口外壳命中 `height: 100%` 时贴合真实舞台），`.launcher-main > <页面根节点>` 统一 `flex: 1 1 auto; height: auto; min-height: 0`，帮助页这类没有内层滚动容器的再加 `overflow-y: auto`。页面根节点一律不再写 `100vh` / `100dvh` / `calc(100vh - Npx)`；有横幅就靠 flex 自动少一份，不要手算偏移。 +- **验证**：真机判据是 Vite + Chromium 量页面根节点是否正好等于 `.window-chrome__content` 的高度（1440x800 / 1440x560 / 390x844 / 390x560，带与不带横幅），帮助页应可滚动到底。 ## 2026-10-02 固定试玩误判祖先的指针穿透样式 - `pointer-events:none` 不会强制禁用整棵子树；后代显式 `auto` 可以恢复命中。控件探针只检查目标的计算样式，继承未覆盖的 `none` 仍拒绝；可见性、遮挡、disabled 与 inert 保留各自检查。