@ghyper9023/pi-dev-workflow 0.4.3 → 0.5.1

package/.pi-dev-output/pi-plans/20260522-113000-grill-left-arrow-fix.md

@@ -1,274 +0,0 @@
-# Grill 左方向键冲突与选项描述显示修复 — 实施计划
-
-## 概述
-
-修复 `dc0d3fa9` 提交引入的两个回归问题：
-
-1. **左方向键破坏 Input 光标左移**：`uiInput` 中将裸 `←` 拦截为"返回"，导致所有 `backable=true` 的输入场景中，左方向键的"光标左移"功能完全失效。
-2. **Grill 选项截断+description 显示混乱**：选项 label 被截断后，完整文本放入 `description` 字段，但 SelectList 的 description 列不换行，导致用户看到"左边简短被截断 + 右边灰色不换行"的混乱显示。
-
-**用户明确要求**：grill 提问环节的向左返回统一改为 `Ctrl+Shift+←`（与输入式一致），输入式的左方向键恢复为光标左移。
-
-## 根因分析
-
-### Bug 1：左方向键冲突
-
-**问题位置**：`extensions/ui-helpers.ts` 第 263-266 行
-
-`dc0d3fa9` 提交在 `uiInput.handleInput` 中添加了：
-
-```typescript
-// 左方向键 → 返回（优先于 Input 的光标左移）
-if (backable && matchesKey(data, Key.left)) {
-    done(BACK_MARKER);
-    return;
-}
-```
-
-这段拦截在 `input.handleInput(data)` 之前执行，Input 组件永远不会收到左方向键事件，导致：
-- Input 内部的 `tui.editor.cursorLeft` 绑定（左方向键）永远不会被触发
-- 用户无法在输入框中左移光标
-- 影响范围：所有 `backable=true` 的 uiInput 调用（grill 自定义输入、dev-* wizard 输入等）
-
-同时，`extensions/grill-me-agent.ts` 第 671-674 行的 `showQuestionTUI.handleInput` 也拦截了左方向键：
-
-```typescript
-// 左方向键 → 返回上一题
-if (backable && currentIndex > 1 && matchesKey(data, Key.left)) {
-    done("__BACK__");
-    return;
-}
-```
-
-两处拦截叠加，用户按 `←` 时：
-- 在 SelectList 中 → 返回上一题（正确，因为 SelectList 不处理 left 键）
-- 在 Input 中 → 返回上一题（错误，期望光标左移）
-
-### Bug 2：选项截断+description 显示混乱
-
-**问题位置**：`extensions/grill-me-agent.ts` 第 602-618 行
-
-```typescript
-const MAX_OPTION_LABEL = 50;
-const truncated = truncateToWidth(label, MAX_OPTION_LABEL, "...");
-return {
-    value: `opt-${i}`,
-    label: truncated,
-    description: truncated !== label ? opt : undefined,
-};
-```
-
-SelectList 的渲染流程：
-1. `renderItem` 对 `label` 使用 `truncateToWidth` 到主列宽度（截断为 50 字符 + "..."）
-2. 如果 `description` 存在，使用两列布局，description 列也被 `truncateToWidth` 到剩余宽度
-
-当终端宽度不足以显示完整的 description 文本时，description 列也会被截断且**不换行**，产生用户看到的"左边截断 + 右边灰色不换行"效果。
-
-## 修复方案
-
-### 方案（根据用户建议）
-
-**核心思路**：统一使用 `Ctrl+Shift+←` 作为返回键，恢复裸 `←` 的光标左移功能。
-
-**详细改动**：
-
-| 位置 | 当前行为（`dc0d3fa9`） | 修复后行为 |
-|------|----------------------|-----------|
-| `grill-me-agent.ts` handleInput | 裸 `←` 拦截为返回 | `Ctrl+Shift+←` 拦截为返回 |
-| `ui-helpers.ts` uiInput handleInput | 裸 `←` 拦截为返回（破坏光标左移） | 移除裸 `←` 拦截 |
-| `ui-helpers.ts` uiInput handleInput | `Ctrl+Shift+←` 返回（保留） | 保留 `Ctrl+Shift+←` |
-| 选项列表（SelectList） | 用户按 `←` → 返回上一题 | 用户按 `Ctrl+Shift+←` → 返回上一题 |
-| 输入框（Input） | 用户按 `←` → 返回上一题 ❌ | 用户按 `←` → 光标左移 ✅ |
-| 输入框（Input） | 用户按 `Ctrl+Shift+←` → 返回上一题 | 保留 |
-
-**Bug 2 修复**：直接移除截断+description 方案，恢复完整 label 显示。
-
-## 文件清单
-
-### 修改文件
-
-| 文件路径 | 改动描述 | 风险等级 |
-|---------|---------|---------|
-| `extensions/ui-helpers.ts` | `uiInput.handleInput` 中移除裸 `←` 拦截（第 263-266 行）；更新 JSDoc 注释 | 低 |
-| `extensions/grill-me-agent.ts` | 1. `showQuestionTUI.handleInput` 中 `Key.left` → `Key.ctrlShift("left")` 2. 更新 hint 文案 3. 移除选项截断+description 方案，恢复完整 label | 低 |
-
-### 无新增/删除文件
-
-## 实施步骤
-
-### 步骤 1：修复 `ui-helpers.ts` — 移除 uiInput 中的裸 ← 拦截
-
-- **前置条件**：无
-- **改动文件**：`extensions/ui-helpers.ts`
-- **改动内容**：
-
-**a) 删除裸 ← 拦截（约第 263-266 行）**
-
-删除以下代码块：
-
-```typescript
-// 左方向键 → 返回（优先于 Input 的光标左移）
-if (backable && matchesKey(data, Key.left)) {
-    done(BACK_MARKER);
-    return;
-}
-```
-
-保留 `Ctrl+Shift+←` 和 `Ctrl+Shift+→` 拦截不受影响。
-
-**b) 更新 JSDoc 注释（约第 220 行）**
-
-将：
-
-```
- * When backable=true, supports ← for back, Ctrl+Shift+← for back, and Ctrl+Shift+→ for submit+next.
-```
-
-改为：
-
-```
- * When backable=true, supports Ctrl+Shift+← for back and Ctrl+Shift+→ for submit+next.
-```
-
-- **验证方式**：
-  - 运行 `/dev-feat` → 进入"核心功能描述"输入框 → 按 `←` → 确认光标左移
-  - 按 `Ctrl+Shift+←` → 确认返回上一题
-  - 按 `Ctrl+Shift+→` → 确认跳过并继续
-
-### 步骤 2：修复 `grill-me-agent.ts` — 选项列表 ← 改为 Ctrl+Shift+←
-
-- **前置条件**：步骤 1 完成
-- **改动文件**：`extensions/grill-me-agent.ts`
-- **改动内容**：
-
-**a) 修改 handleInput 中的左方向键拦截（约第 671-674 行）**
-
-当前：
-
-```typescript
-// 左方向键 → 返回上一题（SelectList 不处理 left 键，需自行拦截）
-if (backable && currentIndex > 1 && matchesKey(data, Key.left)) {
-    done("__BACK__");
-    return;
-}
-```
-
-改为：
-
-```typescript
-// Ctrl+Shift+← → 返回上一题（SelectList 不处理该键，需自行拦截）
-if (backable && currentIndex > 1 && matchesKey(data, Key.ctrlShift("left"))) {
-    done("__BACK__");
-    return;
-}
-```
-
-**b) 修改 hint 提示文字（约第 660 行）**
-
-当前：
-
-```typescript
-const hint = backable && currentIndex > 1
-    ? "  ↑↓ 导航 • Enter 选择 • ← 返回上一题 • Esc 取消全部评审"
-    : "  ↑↓ 导航 • Enter 选择 • Esc 取消全部评审";
-```
-
-改为：
-
-```typescript
-const hint = backable && currentIndex > 1
-    ? "  ↑↓ 导航 • Enter 选择 • Ctrl+Shift+← 返回上一题 • Esc 取消全部评审"
-    : "  ↑↓ 导航 • Enter 选择 • Esc 取消全部评审";
-```
-
-- **验证方式**：
-  - 运行 `/dev-fix` → 进入 grill 评审
-  - 在第二个问题按 `←` → 确认**无反应**（正确，SelectList 不处理 left 键）
-  - 按 `Ctrl+Shift+←` → 确认返回到第一个问题
-  - 确认第一个问题选择的答案被保留
-
-### 步骤 3：修复 `grill-me-agent.ts` — 移除选项截断+description 方案
-
-- **前置条件**：步骤 2 完成
-- **改动文件**：`extensions/grill-me-agent.ts`
-- **改动内容**：
-
-**a) 修改选项构建逻辑（约第 602-618 行）**
-
-当前代码（`dc0d3fa9` 引入）：
-
-```typescript
-const MAX_OPTION_LABEL = 50;
-const selectItems: SelectItem[] = q.options.map((opt, i) => {
-    const prefix = `(${String.fromCharCode(97 + i)}) `;
-    const label = opt === previousAnswer
-        ? `${prefix}${opt} - 上次选择`
-        : `${prefix}${opt}`;
-    const truncated = truncateToWidth(label, MAX_OPTION_LABEL, "...");
-    return {
-        value: `opt-${i}`,
-        label: truncated,
-        description: truncated !== label ? opt : undefined,
-    };
-});
-```
-
-恢复为（还原到 `dc0d3fa9` 之前的状态）：
-
-```typescript
-const selectItems: SelectItem[] = q.options.map((opt, i) => ({
-    value: `opt-${i}`,
-    label: opt === previousAnswer
-        ? `(${String.fromCharCode(97 + i)}) ${opt} - 上次选择`
-        : `(${String.fromCharCode(97 + i)}) ${opt}`,
-}));
-```
-
-**b) 清理不再需要的 import（文件顶部约第 21-27 行）**
-
-如果 `truncateToWidth` 在文件其他地方没有使用（经 grep 检查仅在步骤 3a 处使用），则删除 import 中的 `truncateToWidth`：
-
-```typescript
-// 删除: truncateToWidth,
-import {
-    Container,
-    SelectList,
-    Text,
-    Spacer,
-    matchesKey,
-    Key,
-    type SelectItem,
-} from "@earendil-works/pi-tui";
-```
-
-- **验证方式**：
-  - 查看 grill 问题的选项 → 确认显示完整标签文本（而不是截断+description 两列显示）
-  - 当选项文本超宽时，SelectList 会自动截断到可用宽度（这是 SelectList 的内部行为，不需要手动干预）
-
-## 依赖关系
-
-- 步骤 1 和步骤 2、3 无直接依赖，但建议按顺序 1→2→3 执行，以保证逻辑一致性
-- 步骤 3 中需先确认 `truncateToWidth` 是否在其他地方使用，再决定是否清理 import
-
-## 测试策略
-
-手动测试清单：
-
-| 测试场景 | 操作 | 预期结果 |
-|---------|------|---------|
-| 输入框左方向键 | 在 dev-* 输入框中按 `←` | 光标左移 ✅（修复点） |
-| 输入框 Ctrl+Shift+← | 在 dev-* 输入框中按 `Ctrl+Shift+←` | 返回上一题 ✅ |
-| 输入框 Ctrl+Shift+→ | 在 dev-* 输入框中按 `Ctrl+Shift+→` | 跳过并继续 ✅ |
-| Grill 选项列表 ← | 在 grill 第二题按 `←` | 无反应（SelectList 不处理 left 键）✅ |
-| Grill 选项列表 Ctrl+Shift+← | 在 grill 第二题按 `Ctrl+Shift+←` | 返回上一题 ✅（修复点） |
-| Grill 自定义输入 ← | 进入自定义输入框按 `←` | 光标左移 ✅（修复点） |
-| Grill 自定义输入 Ctrl+Shift+← | 在自定义输入框按 `Ctrl+Shift+←` | 返回选项列表 ✅ |
-| Grill 选项完整显示 | 查看长选项文本 | 完整 label 显示 ✅（修复点） |
-| 原有功能 | ↑↓ Enter Esc | 正常导航/选择/取消 ✅ |
-
-## 注意事项
-
-- **最小改动**：仅修改 2 个文件中涉及的 4 个代码位置，总计约 10-15 行
-- **不修改 pi-tui 库**：所有改动在 extensions/ 目录内完成
-- **兼容性**：原有的 `Ctrl+Shift+←` 和 `Ctrl+Shift+→` 功能完全保留，新增的选项列表 `Ctrl+Shift+←` 与输入框保持一致的按键语义
-- **SelectList 的 `← 返回上一题` 选项**：注意 `showQuestionTUI` 中还有一个 `"← 返回上一题"` 的 `SelectItem`（第 637 行），这是通过选择选项来返回，而非通过按键。此功能应保留，因为用户也可以通过 ↑↓ 导航到该选项后按 Enter 返回。但需要确认它与 `Key.left` 拦截的分离——按键拦截在 handleInput 中，选项选择在 SelectList 的 onSelect 回调中，两者互不干扰。保留该选项作为额外导航方式。

package/.pi-dev-output/pi-review/html/20260521-2305-review-workflow-index.html

@@ -1,196 +0,0 @@
-<!DOCTYPE html>
-<html lang="zh-CN">
-<head>
-<meta charset="UTF-8">
-<meta name="viewport" content="width=device-width, initial-scale=1.0">
-<title>Code Review Report</title>
-<style>
-*{margin:0;padding:0;box-sizing:border-box}
-body{font-family:-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,sans-serif;background:#f5f5f5;color:#333;line-height:1.6;padding:20px}
-.container{max-width:960px;margin:0 auto}
-h1{font-size:24px;margin-bottom:8px;color:#1a1a1a}
-.summary{background:#fff;border-radius:8px;padding:20px;margin-bottom:16px;box-shadow:0 1px 3px rgba(0,0,0,.1)}
-.summary-grid{display:grid;grid-template-columns:1fr 1fr;gap:12px;margin-top:12px}
-.summary-item{background:#fafafa;padding:12px;border-radius:6px}
-.summary-item .label{font-size:12px;color:#666;text-transform:uppercase}
-.summary-item .value{font-size:20px;font-weight:700;margin-top:4px}
-.bug-count .value{color:#d32f2f}
-.warn-count .value{color:#f57c00}
-.info-count .value{color:#1976d2}
-.score{font-size:36px;font-weight:700;text-align:center;padding:16px;background:#fafafa;border-radius:6px;margin-top:12px}
-.score.good{color:#2e7d32}
-.score.fair{color:#f57c00}
-.commit-msg{background:#e8f5e9;padding:12px 16px;border-radius:6px;margin-top:12px;font-family:"SF Mono","Consolas",monospace;font-size:13px;color:#2e7d32}
-.file-group{background:#fff;border-radius:8px;margin-bottom:12px;box-shadow:0 1px 3px rgba(0,0,0,.1);overflow:hidden}
-.file-header{display:flex;justify-content:space-between;align-items:center;padding:14px 20px;cursor:pointer;transition:background .15s}
-.file-header:hover{background:#f0f0f0}
-.file-header .path{font-family:"SF Mono","Consolas",monospace;font-size:14px;font-weight:600;color:#1a1a1a}
-.file-header .status{display:flex;gap:8px;align-items:center}
-.badge{display:inline-block;padding:2px 8px;border-radius:10px;font-size:11px;font-weight:600}
-.badge.bug{background:#ffebee;color:#c62828}
-.badge.warn{background:#fff3e0;color:#e65100}
-.badge.info{background:#e3f2fd;color:#1565c0}
-.file-body{display:none;padding:0 20px 20px;border-top:1px solid #eee}
-.file-body.open{display:block}
-.issue{margin:12px 0;padding:12px 16px;border-left:4px solid #ccc;background:#fafafa;border-radius:0 6px 6px 0}
-.issue.bug{border-color:#d32f2f}
-.issue.warn{border-color:#f57c00}
-.issue.info{border-color:#1976d2}
-.issue .tag{font-size:11px;font-weight:700;text-transform:uppercase;margin-bottom:4px}
-.issue.bug .tag{color:#d32f2f}
-.issue.warn .tag{color:#f57c00}
-.issue.info .tag{color:#1976d2}
-.issue .desc{font-size:14px;margin:4px 0}
-.issue .code{background:#272822;color:#f8f8f2;padding:8px 12px;border-radius:4px;font-size:13px;font-family:"SF Mono","Consolas",monospace;margin:6px 0;overflow-x:auto;white-space:pre}
-.diff-stats{font-size:12px;color:#666;margin-left:12px}
-.add{color:#2e7d32}
-.del{color:#c62828}
-.toggle-icon{color:#999;font-size:12px}
-</style>
-</head>
-<body>
-<div class="container">
-<h1>🔍 Code Review Report</h1>
-
-<div class="summary">
-<p><strong>审查范围：</strong>git diff HEAD（4 files changed, +56, -30）</p>
-
-<div class="summary-grid">
-<div class="summary-item bug-count"><div class="label">BUG</div><div class="value">2</div></div>
-<div class="summary-item warn-count"><div class="label">敏感信息</div><div class="value">1</div></div>
-<div class="summary-item info-count"><div class="label">可维护性</div><div class="value">1</div></div>
-<div class="summary-item info-count"><div class="label">规范</div><div class="value">1</div></div>
-</div>
-
-<div class="score fair">评分：55/100</div>
-<p style="margin-top:8px;font-size:14px;color:#666;text-align:center">存在 2 个 BUG（运行时错误），1 个敏感信息变更，扣分严重</p>
-
-<div class="commit-msg">
-<strong>💡 commit message 建议：</strong><br>
-<code>feat: 增加 reviewTimeoutMs 独立超时配置，修复 workflow sub-step 状态重置函数名错误</code>
-</div>
-</div>
-
-<!-- workflow-engine.ts -->
-<div class="file-group">
-<div class="file-header" onclick="this.nextElementSibling.classList.toggle('open')">
-<span>
-<span class="path">extensions/workflow-engine.ts</span>
-<span class="diff-stats"><span class="add">+27</span> <span class="del">-11</span></span>
-</span>
-<span class="status">
-<span class="badge bug">2 BUG</span>
-<span class="badge warn">1 敏感</span>
-<span class="toggle-icon">▶</span>
-</span>
-</div>
-<div class="file-body">
-<div class="issue bug">
-<div class="tag">🛑 BUG · 最高优先级</div>
-<div class="desc"><strong>调用已重命名的函数导致 ReferenceError</strong></div>
-<div class="desc">第 1193-1194 行调用 <code>setWidgetSubStepStatus()</code>，但该函数在第 783 行被重命名为 <code>resetWidgetSubStep()</code>（参数从 (status) 变为固定 "pending" 并增加重置 detail/tools/outputs 逻辑）。旧函数名已被删除，运行时将抛出 <code>ReferenceError: setWidgetSubStepStatus is not defined</code>。</div>
-<div class="code">// 第 1193-1194 行（executeLoopGroup）
-setWidgetSubStepStatus(stepIndex, step.loopAgentName!, "pending");
-setWidgetSubStepStatus(stepIndex, step.reviewAgentName!, "pending");</div>
-<div class="desc"><strong>修复：</strong>改为调用 <code>resetWidgetSubStep(stepIndex, step.loopAgentName!)</code> 等。</div>
-</div>
-
-<div class="issue bug">
-<div class="tag">🛑 BUG · 最高优先级</div>
-<div class="desc"><strong>runAgentWithProgress 中仍调用已删除的函数</strong></div>
-<div class="desc">第 1114 行仍调用 <code>setWidgetSubStepStatus(stepIndex, agentName, subStatus)</code>，该函数已被删除。执行单步 agent 完成后也会抛出 ReferenceError。</div>
-<div class="code">// 第 1114 行（runAgentWithProgress 末尾）
-setWidgetSubStepStatus(stepIndex, agentName, subStatus);</div>
-<div class="desc"><strong>修复：</strong>将旧函数逻辑恢复，或改写为直接修改 sub-step 状态。</div>
-</div>
-
-<div class="issue warn">
-<div class="tag">⚠️ 敏感信息 · 高优先级</div>
-<div class="desc"><strong>runAgentWithProgress 中 sub-step detail 硬编码中文提醒信息</strong></div>
-<div class="desc">新增的 <code>detail: "超时时间${formatTimeout(timeoutMs)}"</code> 是 UI 展示文本，在 engine 层硬编码中文。当工具函数输出被序列化或用于日志管道时，中文文本可能引起编码问题或混乱。建议使用英文 key（如 <code>timeoutText</code> 或分离 data/ui 层）。</div>
-</div>
-
-<div class="issue info">
-<div class="tag">🔧 可维护性 · 中优先级</div>
-<div class="desc"><strong>loop-group 未展示 timeoutMs 但 reviewer 仍有独立超时</strong></div>
-<div class="desc"><code>executeWorkflowBackground</code> 中 loop-group 步骤的 <code>updateWidgetStep</code> 调用传入 <code>timeoutMs: undefined</code>，这合理（loop-group 的 timeout 逻辑在内部）。但 reviewTimeoutMs 并未在 widget 中展示，建议在 sub-step detail 中体现 reviewer 的独立超时。</div>
-</div>
-</div>
-</div>
-
-<!-- dev-prompts.ts -->
-<div class="file-group">
-<div class="file-header" onclick="this.nextElementSibling.classList.toggle('open')">
-<span>
-<span class="path">extensions/dev-prompts.ts</span>
-<span class="diff-stats"><span class="add">+16</span> <span class="del">-8</span></span>
-</span>
-<span class="status">
-<span class="badge info">1 规范</span>
-<span class="toggle-icon">▶</span>
-</span>
-</div>
-<div class="file-body">
-<div class="issue info">
-<div class="tag">📐 规范 · 低优先级</div>
-<div class="desc"><strong>所有 loop-group 配置的高 timeoutMs 应与 reviewTimeoutMs 有明确注释</strong></div>
-<div class="desc">将 <code>timeoutMs</code> 从 900_000（15min）提升到 1_800_000（30min），并新增 <code>reviewTimeoutMs: 900_000</code>。改动意图明确，配置值合理。但所有步骤的 <code>timeoutMs</code> 和 <code>reviewTimeoutMs</code> 高度重复，建议提取为常量以提高可维护性。</div>
-<div class="code">// 建议提取为命名常量
-const LOOP_TIMEOUT_MS = 1_800_000;  // 30min
-const REVIEW_TIMEOUT_MS = 900_000;  // 15min
-const TRIM_TIMEOUT_MS = 1_200_000;  // 20min</div>
-</div>
-</div>
-</div>
-
-<!-- ui-helpers.ts -->
-<div class="file-group">
-<div class="file-header" onclick="this.nextElementSibling.classList.toggle('open')">
-<span>
-<span class="path">extensions/ui-helpers.ts</span>
-<span class="diff-stats"><span class="add">+1</span> <span class="del">-4</span></span>
-</span>
-<span class="status">
-<span class="badge info">1 可维护性</span>
-<span class="toggle-icon">▶</span>
-</span>
-</div>
-<div class="file-body">
-<div class="issue info">
-<div class="tag">🔧 可维护性 · 中优先级</div>
-<div class="desc"><strong>formatTimeout 改为 export，移除 running 状态下强制显示「第 1 次循环」的特判</strong></div>
-<div class="desc"><code>formatTimeout</code> 被导出供 workflow-engine 使用，改动合理。移除 isRunning 分支的「第 1 次循环」硬编码，改为由 engine 层通过 <code>updateWidgetStep</code> 及时更新 <code>loopCount</code>，设计更清晰。pending 状态仍显示「第 0 次循环」，语义明确。</div>
-</div>
-</div>
-</div>
-
-<!-- .gitignore -->
-<div class="file-group">
-<div class="file-header" onclick="this.nextElementSibling.classList.toggle('open')">
-<span>
-<span class="path">.gitignore</span>
-<span class="diff-stats"><span class="del">-4</span></span>
-</span>
-<span class="status">
-<span class="badge warn">1 敏感</span>
-<span class="toggle-icon">▶</span>
-</span>
-</div>
-<div class="file-body">
-<div class="issue warn">
-<div class="tag">⚠️ 敏感信息 · 高优先级</div>
-<div class="desc"><strong>移除 pi-review/ 和 pi-dev-output/ 的 gitignore 规则</strong></div>
-<div class="desc">提交移除了 <code>pi-review/</code> 和 <code>pi-dev-output/</code> 的 ignore 规则。这两个目录是 pi 框架的产物输出目录，包含审查报告、构建产物等。如果属于不应提交到版本控制的生成产物，应保留 ignore 规则。请确认这是有意的行为（例如要将产物纳入版本管理），或是误删。</div>
-</div>
-</div>
-</div>
-
-</div>
-<script>
-document.querySelectorAll('.file-body.open').forEach(el=>el.classList.remove('open'));
-document.querySelectorAll('.file-header').forEach(h=>{
-h.addEventListener('click',function(){const b=this.querySelector('.toggle-icon');b.textContent=b.textContent==='▶'?'▼':'▶';});
-});
-</script>
-</body>
-</html>

package/.pi-dev-output/pi-review/md/review-20260520-100000.md

@@ -1,91 +0,0 @@
-# 代码审查报告 — workflow-engine 双 Bug 修复
-
-**审查时间**: 2026-05-20  
-**审查范围**: `extensions/workflow-engine.ts` + `tests/test-workflow-engine-bugs.mjs`  
-**审查背景**: 修复 executeLoopGroup 缺少 exitCode 检查 (Bug A) 和 setTimeout cleanupWidget 竞态条件 (Bug B)
-
----
-
-## 严重等级汇总
-
-| 等级 | 数量 |
-|------|------|
-| 🔴 critical | 0 |
-| 🟡 medium | 2 |
-| 🟢 low | 2 |
-
----
-
-## 🟡 中等问题
-
-### M1. `tests/test-workflow-engine.mjs` 中 deleteCheckpointFile 导出测试失败（pre-existing）
-
-**文件**: `tests/test-workflow-engine.mjs` 第 222 行  
-**问题**: 测试断言 `source.includes("export function deleteCheckpointFile")` 但 `workflow-engine.ts` 中并没有 `deleteCheckpointFile` 函数。这不是本次修复引入的问题，而是已有测试的回归。
-
-**建议**: 
-- 要么在 workflow-engine.ts 中添加 `deleteCheckpointFile` 的导出（如果实际需要）
-- 要么从测试中移除这个断言（如果该函数已被移除或从未存在）
-- 由于本次任务是"只修 bug，不做重构"，建议作为独立 issue 处理
-
-### M2. `executeLoopGroup` 中 retry 后未递归检查 agentResult
-
-**文件**: `extensions/workflow-engine.ts` 第 1204 行  
-**问题**: 在非 full-auto 模式下，exitCode 检查分支中 retry 逻辑是：
-```typescript
-agentResult = await runAgentWithProgress(loopAgent, `[RETRY]\n\n${loopTask}`, ...);
-```
-retry 后**没有**再次检查 `agentResult.exitCode`。如果 retry 的 agent 仍然异常退出（例如退出码 1），流程会无检查地进入 review 阶段。这与超时处理的模式不一致——超时处理在 retry 后会检查 `isTimeoutResult(agentResult)` 并 fallback。
-
-**严重程度**: 中等 — 边界情况（连续两次退出码非零），但可能导致错误结果被静默接受。
-
-**建议**: retry 后添加 exitCode 检查：
-```typescript
-agentResult = await runAgentWithProgress(loopAgent, `[RETRY]\n\n${loopTask}`, ...);
-// retry 后再次检查
-if (agentResult.exitCode !== 0 && !isTimeoutResult(agentResult)) {
-  // fallback: 跳过此步骤
-  state.status = "skipped";
-  return;
-}
-```
-
----
-
-## 🟢 低优先级问题
-
-### L1. `cleanupWidget` 中的 `_workflowRunning = false` 与 `executeWorkflowBackground` 的 `_workflowRunning = false` 重复
-
-**文件**: `extensions/workflow-engine.ts` 第 803 行 和第 1450 行  
-**问题**: `executeWorkflowBackground` 在步骤循环完成后设置 `_workflowRunning = false`（第 1450 行），然后新建的定时器在 5 秒后调用 `cleanupWidget()` 再次设置 `_workflowRunning = false`（第 803 行）。虽然不会影响正确性（重复赋值相同值），但逻辑上冗余。
-
-**建议**: 可以移除 `executeWorkflowBackground` 中的 `_workflowRunning = false`，让 `cleanupWidget` 统一管理该变量，避免逻辑分散。
-
-### L2. 测试中 `simulateBugAFix` 的 retry 返回语义不精确
-
-**文件**: `tests/test-workflow-engine-bugs.mjs` 第 147-174 行  
-**问题**: `simulateBugAFix` 函数在非 full-auto 模式下返回 `"retry"`，但这个返回值没有区分"用户选择了跳过"、"用户选择了重试"、"用户选择了取消"三种情况。虽然作为模拟测试够用，但语义不够精确。
-
-**建议**: 可以用更精确的返回值（如 `"retry"`、`"skip"`、`"cancel"`），或使用对象形式返回。
-
----
-
-## 维护建议
-
-1. 修复测试 `test-workflow-engine.mjs` 中 deleteCheckpointFile 的断言——这会导致 CI 失败。
-2. 考虑在 retry 后添加二次 exitCode 检查（M2），以增强鲁棒性。
-3. 整体修复方案符合"最小化改动"要求，代码阅读性和可维护性好。
-
----
-
-## 测试结果
-
-- ✅ `node tests/test-workflow-engine-bugs.mjs` — **50/50 通过**（新测试全部通过）
-- ⚠️ `node tests/test-workflow-engine.mjs` — **73/74 通过**（1 个 pre-existing 失败，非本次改动导致）
-- ✅ `git diff HEAD` 确认变更仅限于计划中指定的范围
-
----
-
-## 审查结论
-
-本次修复正确地解决了两个 Bug。代码实现与实施计划一致，没有引入回归。存在 2 个中等问题和 2 个低优先级问题，但没有严重问题。