sillyspec 3.7.2 → 3.7.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sillyspec",
-  "version": "3.7.2",
+  "version": "3.7.3",
   "description": "SillySpec CLI — 流程状态机，让 AI 严格按步骤来",
   "type": "module",
   "bin": {

package/templates/execute.md

@@ -54,6 +54,48 @@ cat .sillyspec/knowledge/INDEX.md 2>/dev/null
 
 子代理遇到不熟悉的库或 API 时，优先使用已配置的 MCP 工具（Context7 等）或 web search 查最新文档，不要凭记忆猜测用法。
 
+**编码规范扫描（主代理执行）：**
+主代理在 dispatch 子代理前，扫描项目中常见的编码规范配置文件，将关键规则注入子代理 prompt 的「编码规范约束」段。
+
+```bash
+# 检测存在的配置文件
+for f in .eslintrc .eslintrc.js .eslintrc.cjs .eslintrc.json .eslintrc.yml \
+         .prettierrc .prettierrc.js .prettierrc.json .prettierrc.yml \
+         tsconfig.json tsconfig.base.json \
+         .editorconfig \
+         .tailwind.config.js .tailwind.config.ts \
+         .stylelintrc .stylelintrc.js .stylelintrc.json \
+         CONTRIBUTING.md CODE_STYLE.md; do
+  [ -f "$f" ] && echo "=== $f ===" && cat "$f"
+done
+# 也检查 package.json 中的 lint/format 脚本
+cat package.json 2>/dev/null | grep -A5 '"lint\|"format\|"typecheck\|"type-check'
+```
+
+扫描后，主代理根据检测结果生成**编码规范摘要**（不是原文粘贴，是提炼关键约束），格式如下。如果某个类别未检测到对应配置文件，则省略该段：
+
+```
+## 编码规范约束（自动扫描）
+
+### ESLint
+{提取的关键规则，如：禁止 var、要求分号、禁止未使用变量、特定插件规则等}
+
+### Prettier
+{提取的格式化规则，如：单引号、2空格缩进、无分号、行宽 80 等}
+
+### TypeScript
+{从 tsconfig 提取的严格模式设置，如：strict: true、noUncheckedIndexedAccess 等}
+
+### Import / 命名约定
+{从 eslint/import 插件或 editorconfig 提取的导入排序、命名风格等}
+
+### 框架约定
+{检测到的框架特定约定，如：Next.js App Router、Tailwind 类名风格等}
+```
+
+将此摘要注入到每个子代理 prompt 的「项目约定」段之后，并追加一条铁律：
+> **10. 遵守编码规范：** 以上「编码规范约束」中的所有规则必须严格遵守。如规则与任务描述冲突，优先遵守规范约束并报告。
+
 **MCP 能力检测（主代理执行）：**
 ```bash
 for cfg in .claude/mcp.json .cursor/mcp.json; do
@@ -103,6 +145,9 @@ done
 ## 项目约定
 {CONVENTIONS.md 全文}
 
+## 编码规范约束（自动扫描）
+{主代理扫描项目配置文件后生成的规范摘要，见上方「编码规范扫描」步骤}
+
 ## 项目架构
 {ARCHITECTURE.md 全文}
 
@@ -132,6 +177,7 @@ done
 7. **E2E 任务：** 如果任务描述包含"E2E"或"端到端"，先 cat 相关功能代码和页面组件，理解交互逻辑。有测试框架则编写测试文件，无框架则编写 `.sillyspec/changes/<变更名>/e2e-steps.md` 结构化测试步骤（每条包含操作和断言）
 8. **暂存：** 完成后在工作目录执行 git add -A（不要 commit，由用户通过 /sillyspec:commit 统一提交）
 9. **不修改计划外的文件**，如必须修改则在报告中说明
+10. **遵守编码规范：** prompt 中「编码规范约束」段的所有规则必须严格遵守。如规范与任务描述冲突，优先遵守规范并报告
 
 ## 完成后报告（严格按此格式）
 
@@ -166,6 +212,20 @@ done
 
 ## 完成后
 
+**任务勾选自检（必须执行）：**
+所有任务完成后，主代理必须执行以下检查：
+
+```bash
+cat .sillyspec/changes/*/tasks.md 2>/dev/null
+```
+
+逐条验证：
+1. **所有返回 DONE/DONE_WITH_CONCERNS 的任务是否已勾选 `- [x]`？**
+2. **勾选的任务是否都记录了精确到秒的时间戳 `[YYYY-MM-DD HH:MM:SS]`？**
+3. **tasks.md 中是否还有未勾选 `- [ ]` 的已完成任务？**
+
+发现遗漏 → 立即补勾选 + 补时间戳，不要等用户提醒。
+
 **知识库审阅：** 检查是否有待确认的知识条目：
 ```bash
 grep -c '^\### \[待确认\]' .sillyspec/knowledge/uncategorized.md 2>/dev/null

package/templates/explore.md

@@ -29,10 +29,34 @@ $ARGUMENTS
 ```bash
 ls .sillyspec/changes/ 2>/dev/null | grep -v archive
 cat .sillyspec/{REQUIREMENTS,ROADMAP}.md 2>/dev/null
+cat .sillyspec/codebase/{CONVENTIONS,ARCHITECTURE}.md 2>/dev/null
+cat .sillyspec/knowledge/INDEX.md 2>/dev/null
 ```
 
 有进行中变更时读取其 design/tasks，自然引用。发现重要决策时可提议保存（不自动保存）。
 
+## MCP 能力（按需使用）
+
+```bash
+cat .claude/mcp.json .cursor/mcp.json 2>/dev/null
+```
+
+- 有 Context7 → 探索时查询最新文档，验证技术方案的可行性
+- 有浏览器 MCP → 可浏览相关网站、查竞品实现
+- 有搜索 MCP → 搜索技术方案、最佳实践
+- 无 MCP → 使用 web search
+
+## 话题升级提示
+
+探索过程中，当对话达到一定深度时（讨论了 5+ 轮、或涉及具体实现方案、或用户表达"试试看"/"能不能做"/"怎么搞"），主动用 AskUserQuestion 提示用户：
+
+1. **🧠 头脑风暴** — `/sillyspec:brainstorm` 深度探索需求和方案
+2. **⚡ 快速执行** — `/sillyspec:quick` 直接动手做
+3. **📋 创建规范** — `/sillyspec:propose` 生成结构化规范
+4. **🔍 继续探索** — 还没聊透，继续
+
+不需要每次都提示，只在对话**明显转向执行意图**时触发。纯讨论、提问、查资料时不要打断。
+
 ## 没有必需的结束
 
 探索可以创建变更提案、产出文档、继续探索或结束。

package/templates/plan.md

@@ -123,6 +123,30 @@ cat .claude/mcp.json .cursor/mcp.json 2>/dev/null | grep -i "browser\|chrome\|de
 - [ ] 每个 task 有具体文件路径？
 - [ ] 标注了 Wave 和依赖关系？
 - [ ] 涉及 UI 的任务是否有对应的 E2E 测试任务？
+- [ ] **design.md 中的每个功能点是否都在 tasks.md 中有对应任务？**
+
+### 6.5 设计完整性对照（必须完成）
+
+逐条检查 design.md 中的功能描述，确保每个功能点都有对应的 task。特别关注：
+
+1. **逐功能点扫描：** 将 design.md 中描述的每个功能点（含子功能）列出，与 tasks.md 逐条对照
+2. **前后端覆盖检查：** 涉及前后端协作的功能，确认前端和后端各有独立 task
+3. **遗漏项处理：** 发现未覆盖的功能点 → 追加 task 到对应 Wave，并提示用户确认
+
+**执行方式：**
+```
+从 design.md 提取功能点清单：
+  功能 A（后端接口）
+  功能 B（前端页面）
+  功能 C（前后端联动）
+
+与 tasks.md 对照：
+  ✅ 功能 A → task-3
+  ❌ 功能 B → 无对应 task → 追加
+  ⚠️ 功能 C → 只有后端 task，缺少前端 task → 追加
+```
+
+发现遗漏时用 AskUserQuestion 确认追加内容，用户确认后再写入 tasks.md。
 
 ### 7. 完成
 

package/templates/quick.md

@@ -21,12 +21,19 @@ $ARGUMENTS
 1. **解析参数：** 检查是否携带 `--change <变更名>`，确定记录方式
 2. **理解任务：** 模糊则问一个问题确认
 3. **加载上下文：** `cat .sillyspec/codebase/{CONVENTIONS,ARCHITECTURE}.md 2>/dev/null`
+3b. **编码规范扫描：** 检测项目中的编码规范配置文件（`.eslintrc*`、`.prettierrc*`、`tsconfig.json`、`.editorconfig`、`tailwind.config.*`、`CONTRIBUTING.md`），提取关键规则生成摘要。写作代码时必须严格遵守这些规则（分号/引号/缩进/命名风格等），如不确定优先遵守规范约束。
 4. **知识库查询（强制步骤）：**
    ```bash
    cat .sillyspec/knowledge/INDEX.md 2>/dev/null
    ```
    根据当前任务描述中的关键词匹配 INDEX.md 条目，命中时 `cat` 对应知识文件，将内容纳入后续开发考量。未命中则跳过。
-   **MCP 检测：** `cat .claude/mcp.json .cursor/mcp.json 2>/dev/null`，有 Context7 则用 MCP 查文档，无则用 web search。
+   **MCP 检测：** `cat .claude/mcp.json .cursor/mcp.json 2>/dev/null`，根据检测结果动态利用：
+   - 有 Context7 → 查询不熟悉库/API 的最新文档
+   - 有浏览器 MCP → 验证页面改动效果
+   - 有数据库 MCP → 查询表结构和数据验证（只读）
+   - 有搜索 MCP → 搜索最佳实践和解决方案
+   - 有其他 MCP → 按任务需要灵活使用
+   - 无 MCP → 使用 web search
 5. **先读后写：** 调用已有方法前 `cat` 源文件确认签名，`grep` 确认方法存在
 6. **数据操作安全：** 任何改变现有数据的操作（非 SELECT 的数据库操作）必须暂停并报告给用户确认，不得自动执行。新建表不受此限制
 6. **TDD 执行：**

package/templates/verify.md

@@ -47,6 +47,7 @@ else
   CHANGE_DIR=$(ls -d .sillyspec/changes/*/ 2>/dev/null | grep -v archive | tail -1)
 fi
 cat "$CHANGE_DIR"/{design,tasks}.md 2>/dev/null
+cat .sillyspec/local.yaml 2>/dev/null
 ```
 
 锚定确认实际存在的文件。
@@ -97,9 +98,49 @@ npx vitest run 2>/dev/null || npx jest 2>/dev/null
 > ⚠️ 使用 MCP 执行时 AI 判断可能不如测试框架精确。追求可靠性建议安装 Playwright。
 
 **自动修复循环（选了策略 1 或 2 时）：**
-读取 `.sillyspec/local.yaml` 中当前变更的 `fixAttempts`，对每个失败测试：
-- fixAttempts 未达上限 → 调 `/sillyspec:quick "修复 E2E 失败：<失败描述>"` → 重跑该测试 → 更新 local.yaml
-- fixAttempts 达到上限 → 停止，报告失败详情，提示人工介入
+
+必须按以下流程严格执行，不可跳过：
+
+```
+ROUND = 1
+MAX_ROUNDS = 策略1时为5，策略2时为50
+
+while ROUND <= MAX_ROUNDS:
+    1. 运行失败测试，捕获完整输出（错误信息、堆栈、期望值 vs 实际值）
+    2. 如果全部通过 → 跳出循环，标记 ✅
+    3. 读取 .sillyspec/local.yaml 中当前变更的 fixAttempts
+    4. 对每个失败测试：
+       a. 如果 fixAttempts >= MAX_ROUNDS → 跳过，标记 ❌ MAX_REACHED
+       b. 否则 → 调 /sillyspec:quick 修复，prompt 必须包含：
+          - 失败的测试文件路径和测试名
+          - 完整错误信息（含期望值 vs 实际值）
+          - 相关源文件路径
+          - "只修复这个测试失败，不要改其他代码"
+       c. 修复后重跑该测试确认是否通过
+       d. 通过 → fixAttempts 保持不变；仍失败 → fixAttempts + 1
+    5. 写入 .sillyspec/local.yaml 更新 fixAttempts
+    6. ROUND++
+    7. 如果本轮无任何修复（所有失败都已 MAX_REACHED）→ 跳出循环
+```
+
+**quick 修复 prompt 模板：**
+```
+/sillyspec:quick "修复测试失败：<测试文件路径>:<测试名>
+
+错误信息：
+<完整错误输出，包含期望值和实际值>
+
+相关文件：
+<被测源文件路径>
+
+只修复这个测试失败，不要改其他代码。修完后运行该测试确认通过。"
+```
+
+**禁止行为：**
+- ❌ 只看错误摘要就修复（必须看完整输出）
+- ❌ 跳过 fixAttempts 计数
+- ❌ 一次 quick 修复多个不相关的失败（逐个修复，每次修复后重跑确认）
+- ❌ 主代理直接修改代码（verify 阶段禁止改代码，必须通过 /sillyspec:quick）
 
 **更新测试结果到 `.sillyspec/local.yaml`（按变更名隔离，覆盖写入）：**
 ```yaml
@@ -121,7 +162,73 @@ grep -r "TODO\|FIXME\|HACK\|XXX" src/ lib/ app/ --include="*.ts" --include="*.ts
 
 审查 design.md「文件变更」中列出的文件：安全问题（输入校验、SQL拼接、硬编码敏感信息）、潜在 bug（空值、边界条件）、与 CONVENTIONS.md 一致性。每个问题标 🔴必须 / 🟡建议 / 🔵优化。
 
-### 6. 输出验证报告
+### 5.5 MCP 基础设施验证
+
+检测已配置的 MCP 服务，利用它们做实际验证（不只查文档）：
+
+**MCP 能力检测：**
+```bash
+cat .claude/mcp.json .cursor/mcp.json 2>/dev/null
+```
+
+**按检测结果执行对应验证：**
+
+**数据库 MCP（postgres/sqlite/mysql/redis）：**
+- 对照 design.md 中的数据模型，验证表/集合是否存在
+- 验证字段类型、约束（主键、外键、唯一索引）是否与设计一致
+- 验证新增的 API 是否能正确读写对应数据
+- ⚠️ 只执行 SELECT 查询，禁止任何写操作
+
+**浏览器 MCP（chrome-devtools/puppeteer/playwright）：**
+- 验证页面能否正常加载（无 404/500 错误）
+- 验证关键 UI 元素是否存在（导航、表单、按钮等）
+- 验证基础交互（点击、提交、跳转）
+
+**API MCP：**
+- 验证新增接口是否可达
+- 验证请求/响应格式是否与设计一致
+
+**无 MCP → 跳过此步骤，不影响验证结论。**
+
+将验证结果纳入验证报告。
+
+### 6. Lint / Format 检查
+
+自动检测并运行项目配置的 lint/format 工具，验证代码是否符合规范：
+
+```bash
+# ESLint
+if [ -f .eslintrc -o -f .eslintrc.js -o -f .eslintrc.cjs -o -f .eslintrc.json -o -f .eslintrc.yml ] || grep -q '"eslint"' package.json 2>/dev/null; then
+  echo "=== ESLint ==="
+  npx eslint . --max-warnings 0 2>&1 | tail -50
+fi
+
+# Prettier（检查而非修复）
+if [ -f .prettierrc -o -f .prettierrc.js -o -f .prettierrc.json -o -f .prettierrc.yml ] || grep -q '"prettier"' package.json 2>/dev/null; then
+  echo "=== Prettier ==="
+  npx prettier --check . 2>&1 | tail -30
+fi
+
+# TypeScript 类型检查
+if [ -f tsconfig.json ]; then
+  echo "=== TypeScript ==="
+  npx tsc --noEmit 2>&1 | tail -30
+fi
+
+# Stylelint
+if [ -f .stylelintrc -o -f .stylelintrc.js -o -f .stylelintrc.json ] || grep -q '"stylelint"' package.json 2>/dev/null; then
+  echo "=== Stylelint ==="
+  npx stylelint "**/*.{css,scss,less}" 2>&1 | tail -30
+fi
+```
+
+**处理策略（AskUserQuestion）：**
+1. **自动修复** — 对支持 `--fix` 的工具（ESLint、Prettier、Stylelint）自动修复后重跑检查，同一问题最多修复 3 次
+2. **只报告** — 仅列出所有 lint 错误，不修改代码
+
+将 lint 结果纳入验证报告（步骤 7）。
+
+### 7. 输出验证报告
 
 ```markdown
 # SillySpec 验证报告
@@ -130,6 +237,8 @@ grep -r "TODO\|FIXME\|HACK\|XXX" src/ lib/ app/ --include="*.ts" --include="*.ts
 ## 测试结果：passed N, failed N
 ## 技术债务标记
 ## 代码审查：🔴 N / 🟡 N / 🔵 N
+## Lint 检查：ESLint ✅/❌ | Prettier ✅/❌ | TypeScript ✅/❌ | Stylelint ✅/❌
+## MCP 基础设施验证：数据库 ✅/❌/跳过 | 页面 ✅/❌/跳过 | API ✅/❌/跳过
 ## E2E 测试：passed N / failed N / fixAttempts 详情
 ## 结论：✅ PASS / ⚠️ PASS WITH NOTES / ❌ FAIL
 ```