Findings

• [Major] 旧 iframe 错误会泄漏到下一次生成结果中。CanvasErrorBar 现在会一直挂在预览区顶部，但这次改动没有在新一轮 sendPrompt 或新 iframe 建立时清空 iframeErrors；结果是上一个 preview 的运行时异常会继续显示在新的 loading / preview 上，用户会把旧错误误判成当前结果的问题。证据：apps/desktop/src/renderer/src/components/PreviewPane.tsx:52、apps/desktop/src/renderer/src/store.ts:116-117、apps/desktop/src/renderer/src/store.ts:155-159
  Suggested fix:

  set((s) => ({
    messages: [...s.messages, userMessage],
    isGenerating: true,
    errorMessage: null,
    iframeErrors: [],
  }));

• [Major] iframeErrors 现在是无上限追加的。window.message listener 会把每个 IFRAME_ERROR 都推入 store，而 pushIframeError 每次都复制并扩展整个数组；AI 生成的 HTML 很容易出现 setInterval/循环重试导致的重复 throw/reject，这里会持续占用内存并放大 rerender 成本。证据：apps/desktop/src/renderer/src/App.tsx:35、apps/desktop/src/renderer/src/store.ts:117
  Suggested fix:

  pushIframeError(message: string) {
    set((s) => {
      const last = s.iframeErrors[s.iframeErrors.length - 1];
      const next = last === message ? s.iframeErrors : [...s.iframeErrors, message];
      return { iframeErrors: next.slice(-5) };
    });
  }

Summary

• Review mode: initial
• 发现 2 个 Major 问题，都是这次新增的 iframe error surface 引入的：旧错误会跨 preview 泄漏，且错误缓冲区无上限增长。
• 未发现新增依赖、直接 provider SDK import 或 UI token 字面量违规。DCO / license / install-size 变化在本次 diff 中未涉及。

Testing

• Not run (automation)
• 缺少 Vitest coverage：window.postMessage -> pushIframeError、重新生成时清空旧错误、重复错误的去重/上限行为。

open-codesign Bot

Findings

• [Major] 重新生成期间，上一版 preview 仍会把旧的 IFRAME_ERROR 重新灌回这次新清空的错误条。sendPrompt() 现在会在开始时清空 iframeErrors，但 PreviewPane 只有在“没有旧 preview”时才切到 LoadingState；已有 preview 时，旧 iframe 会继续挂着并继续 postMessage。这样一来，新增的全局 listener 会在同一轮生成过程中重新写入旧错误，CanvasErrorBar 也会把它显示成当前结果的问题。证据：apps/desktop/src/renderer/src/App.tsx:32、apps/desktop/src/renderer/src/components/PreviewPane.tsx:31、apps/desktop/src/renderer/src/store.ts:159
  Suggested fix:

  const handler = (e: MessageEvent) => {
    if (!isIframeErrorMessage(e.data)) return;
    const { isGenerating, pushIframeError } = useCodesignStore.getState();
    if (isGenerating) return;
    pushIframeError(e.data.message);
  };

Summary

• Review mode: follow-up after new commits
• 还剩 1 个 Major 问题：这次虽然补上了清空和去重/上限，但旧 preview 在重新生成期间仍能继续污染新的 iframe error surface。
• 未发现这次新增/修改行引入新的依赖、直接 provider SDK import、silent fallback 或 UI token 字面量违规。DCO / license / install-size 变化未涉及本次 diff。

Testing

• Not run (automation)
• 建议补一个集成测试：在已有 previewHtml 时触发新一轮 sendPrompt()，并在 isGenerating=true 期间模拟旧 iframe 发出 IFRAME_ERROR，断言 CanvasErrorBar 不会重新出现。

open-codesign Bot

Findings

• 未发现基于本次新增/修改行、且置信度足够高的问题。

Summary

• Review mode: follow-up after new commits
• 上一轮指出的 Major 问题已经在 apps/desktop/src/renderer/src/App.tsx:13 与 apps/desktop/src/renderer/src/store.ts:159 的改动中得到处理：重新生成期间会忽略 IFRAME_ERROR，并在开始新一轮生成时清空旧的 iframeErrors；本轮未发现新的 diff-attributable 问题。
• 本次 diff 未引入新依赖，也未看到直接 provider SDK import、silent fallback 或新增 UI token 字面量违规。license / install-size 变化不涉及本次 diff。
• 残余风险是测试边界仍偏窄：当前只覆盖了辅助函数和 store action，未见覆盖 window.message 监听到 CanvasErrorBar 渲染联动的组件级测试。

Testing

• Not run (automation)
• 本环境缺少 pnpm，未执行 Vitest / Playwright。

open-codesign Bot

Findings

• [Major] 旧 iframe 的晚到错误仍会污染新预览状态。新的全局 message 监听只校验 isIframeErrorMessage，不再校验 event.source 是否仍然指向当前预览 iframe；这样一来，上一轮生成在切换前后迟到的 postMessage 仍会被 pushIframeError 收下，stale runtime error 会重新出现在新预览上，证据 apps/desktop/src/renderer/src/App.tsx:37、apps/desktop/src/renderer/src/components/PreviewPane.tsx:31。
  Suggested fix:

  if (!isTrustedPreviewMessageSource(event.source, iframeRef.current?.contentWindow)) return;
  
  const state = useCodesignStore.getState();
  if (state.isGenerating || !isIframeErrorMessage(event.data)) return;
  state.pushIframeError(formatIframeError(event.data));

• [Minor] iframe 运行时错误的定位信息被截断。readIframeErrorMessage() 现在只保留 message，把原先的 kind/source/lineno 都丢掉了；CanvasErrorBar 最终只能显示裸文本，和运行时消息契约提供的上下文不一致，证据 apps/desktop/src/renderer/src/App.tsx:13、packages/runtime/src/iframe-errors.ts:13。
  Suggested fix:

  const location = data.source && data.lineno ? ` (${data.source}:${data.lineno})` : '';
  return `${data.kind}: ${data.message}${location}`;

Summary

• Review mode: follow-up after new commits
• 本轮主要问题集中在 App.tsx 新增的 iframe 错误监听：它解决了“生成中忽略旧错误”的一部分场景，但同时丢掉了当前 iframe source 校验和错误位置信息。
• 未看到本次 diff 引入新依赖、直接 provider SDK import、silent fallback，license / install-size 变化也不涉及本次修改。

Testing

• Not run (automation)
• 按本任务的安全约束，未执行 PR 中代码，因此未跑 Vitest / Playwright。

open-codesign Bot

Findings

• [Blocker] App.tsx 新增的全局 message 监听会与 PreviewPane 现有监听并行消费同一条 IFRAME_ERROR，且这里没有 event.source 校验，导致两类回归：1) 同一错误被重复写入（kind: message + 裸 message）；2) 非当前 preview iframe 的消息也可能被接受，重新污染当前画布错误栏。证据 apps/desktop/src/renderer/src/App.tsx:39、apps/desktop/src/renderer/src/App.tsx:42、apps/desktop/src/renderer/src/App.tsx:44（对比现有 source 校验 apps/desktop/src/renderer/src/components/PreviewPane.tsx:36）。
  Suggested fix:

  // App.tsx
  // 删除 App 级 iframe 错误监听，保留 PreviewPane 内部监听（已有 source 校验 + 定位信息）
  // useEffect(() => { ...window.addEventListener('message', handler)... }, []);
  // export function readIframeErrorMessage(...) { ... }

Summary

• Review mode: follow-up after new commits
• 本轮发现 1 个阻塞问题：App 层新增监听引入重复/越权来源接收，属于本次 diff 直接引入的行为回归。
• 本次 diff 未引入新第三方依赖；未发现 direct provider SDK import。

Testing

• Not run (automation): 当前环境缺少 pnpm（pnpm: command not found），无法执行 Vitest/Playwright。

open-codesign Bot

Findings

• No Blocker/Major/Minor/Nit findings with >=80% confidence that are introduced by the current added/modified lines.

Summary

• Review mode: follow-up after new commits
• No diff-attributable issues found in the latest PR diff.
• Constraint checks in scope: no new dependencies added (license/install-size impact not introduced by this diff), no direct provider SDK imports in changed files, and no newly introduced silent fallback pattern found in changed lines.
• Residual risk/testing gap: runtime behavior was reviewed statically only; integration behavior was not executed in this review run.

Testing

• Not run (automation): PR-content code execution was intentionally not performed in this review run.

open-codesign Bot