openmatrix 0.1.97 → 0.1.98

package/dist/agents/agent-runner.js

@@ -454,7 +454,7 @@ ${task.acceptanceCriteria.map((c, i) => `${i + 1}. ${c}`).join('\n')}
      */
     getTesterPrompt(task) {
         return `
-你是一个 Tester Agent，负责测试代码。
+你是一个 Tester Agent，负责编写高质量的测试代码。
 
 ## 职责
 
@@ -464,42 +464,64 @@ ${task.acceptanceCriteria.map((c, i) => `${i + 1}. ${c}`).join('\n')}
 4. 生成测试报告
 5. 验证修复效果
 
-## 测试原则
+## 🚨 测试质量核心原则
+
+### 1. 真实测试 > Mock 测试
+- **优先使用真实数据和真实依赖**，而不是过度 mock
+- 只有外部服务（API、数据库远程连接）才需要 mock
+- 内部模块之间的调用**不要 mock**，用真实的模块
+- 如果函数依赖文件系统，用临时目录（\`os.tmpdir()\`）而非 mock fs
+
+### 2. 集成测试不可少
+- 模块间调用必须测试（如 A 调用 B，测试完整链路）
+- 如果有多个模块协同工作，至少写一个集成测试验证端到端流程
+- 测试实际的导入/导出是否正常工作
+
+### 3. 运行时错误比逻辑错误更致命
+- 测试代码能否**实际运行**（导入不报错、实例化不报错）
+- 测试配置加载是否正常（读取配置文件、环境变量）
+- 测试依赖注入是否正确（不是 undefined）
+
+## 测试编写规范
 
 ### AAA 模式
 每个测试用例遵循 Arrange-Act-Assert 模式:
 \`\`\`typescript
 describe('MyFunction', () => {
   it('should do something', () => {
-    // Arrange - 准备测试数据
+    // Arrange - 使用真实数据，不要过度 mock
     const input = 'test';
 
-    // Act - 执行被测试的函数
+    // Act - 调用真实函数
     const result = myFunction(input);
 
-    // Assert - 验证结果
+    // Assert - 验证真实结果
     expect(result).toBe('expected');
   });
 });
 \`\`\`
 
-### 测试覆盖
-- **正常流程**: 主要功能路径
-- **边界情况**: 空值、极值、特殊字符
-- **异常处理**: 错误输入、网络失败
-- **并发场景**: 竞态条件 (如适用)
+### 测试覆盖优先级（从高到低）
+1. **运行时验证**: 模块能被导入、类能被实例化、配置能被加载
+2. **核心业务逻辑**: 主要功能路径的真实行为
+3. **集成链路**: 模块间协作是否正常
+4. **边界情况**: 空值、极值、特殊字符
+5. **异常处理**: 错误输入、资源不可用
+6. **状态变化**: 修改操作的前后状态
 
-### 命名规范
-- 测试文件: \`.test.ts\` 或 \`.spec.ts\`
-- 描述清晰: \`it('should return user when id exists')\`
-- 使用 describe 分组相关测试
+### 禁止的做法
+- ❌ 不要 mock 被测模块的内部依赖（那等于没测）
+- ❌ 不要写 \`expect(true).toBe(true)\` 这样的空测试
+- ❌ 不要只测 happy path，忽略错误场景
+- ❌ 不要在测试中捕获异常后忽略（\`try { ... } catch { }\`）
 
 ## 必须测试的场景
 
-1. **基本功能**: 每个公开方法至少一个测试
-2. **输入验证**: 无效输入、空值、边界值
-3. **错误处理**: 异常抛出、错误返回
-4. **状态变化**: 修改操作的前后状态
+1. **模块导入测试**: 验证所有公开 API 能被正确导入
+2. **基本功能**: 每个公开方法至少一个测试
+3. **输入验证**: 无效输入、空值、边界值
+4. **错误处理**: 异常抛出、错误返回
+5. **集成测试**: 模块间协作的端到端测试
 
 ## 输出格式
 
@@ -528,7 +550,9 @@ describe('MyFunction', () => {
 
 ## 完成检查清单
 
+- [ ] 模块导入正常（无运行时依赖缺失）
 - [ ] 所有公开方法已测试
+- [ ] 至少一个集成测试验证模块协作
 - [ ] 边界情况已覆盖
 - [ ] 异常处理已验证
 - [ ] 测试全部通过

package/dist/orchestrator/phase-executor.js

@@ -505,6 +505,81 @@ ${task.acceptanceCriteria.map((c, i) => `${i + 1}. [ ] ${c}`).join('\n')}
             parts.push(`
 ⏭️ 无验收标准定义`);
         }
+        // Gate 8: 真实运行验证
+        parts.push(`
+
+### 8. 🚀 真实运行验证 (Smoke Test)
+
+**目的**: 确保代码不仅编译通过，还能实际运行。
+
+#### 8a. 导入/模块验证
+\`\`\`bash
+# 检查模块是否可以被正确导入（无运行时依赖缺失）
+node -e "
+try {
+  // 检查主要入口文件是否能被导入
+  const fs = require('fs');
+  const path = require('path');
+  const pkg = require('./package.json');
+
+  // 如果有 main 或 exports 字段，尝试导入
+  const mainFile = pkg.main || pkg.exports?.['.']?.import || pkg.exports?.['.']?.require;
+  if (mainFile && fs.existsSync(mainFile)) {
+    require('./' + mainFile);
+    console.log('✅ Main entry imported successfully');
+  } else if (fs.existsSync('dist/index.js')) {
+    require('./dist/index.js');
+    console.log('✅ dist/index.js imported successfully');
+  } else {
+    console.log('⏭️ No main entry to import-test');
+  }
+} catch(e) {
+  console.error('❌ Import failed:', e.message);
+  process.exit(1);
+}
+"
+\`\`\`
+**失败后果**: ❌ VERIFY_FAILED (运行时依赖缺失或导入错误)
+
+#### 8b. 启动验证（如适用）
+\`\`\`bash
+# 检查是否有 start 脚本，如果有则做短暂启动测试
+if npm run | grep -q "start"; then
+  echo "Found start script, running smoke test..."
+  timeout 10 npm start -- --smoke-test 2>/dev/null || \
+  timeout 10 npm start &
+  START_PID=$!
+  sleep 5
+  # 检查进程是否还在运行（启动没有崩溃）
+  if kill -0 $START_PID 2>/dev/null; then
+    echo "✅ Application started successfully"
+    kill $START_PID 2>/dev/null
+  else
+    wait $START_PID
+    EXIT_CODE=$?
+    if [ $EXIT_CODE -eq 0 ]; then
+      echo "✅ Application ran and exited cleanly"
+    else
+      echo "❌ Application crashed with exit code $EXIT_CODE"
+      exit 1
+    fi
+  fi
+else
+  echo "⏭️ No start script found - skipping startup test"
+fi
+\`\`\`
+**失败后果**: ❌ VERIFY_FAILED (启动崩溃) / ⏭️ Skipped (无 start 脚本)
+
+#### 8c. 功能冒烟测试
+\`\`\`bash
+# 如果有 smoketest 脚本，运行它
+if npm run | grep -q "smoketest\\|smoke-test\\|test:smoke"; then
+  npm run smoketest 2>/dev/null || npm run smoke-test 2>/dev/null || npm run test:smoke
+else
+  echo "⏭️ No smoke test script - skipping"
+fi
+\`\`\`
+**失败后果**: ❌ VERIFY_FAILED / ⏭️ Skipped (无 smoke test 脚本)`);
         parts.push(`
 
 ## 📊 质量报告格式
@@ -628,6 +703,7 @@ ${task.acceptanceCriteria.map((c, i) => `${i + 1}. [ ] ${c}`).join('\n')}`);
 - [ ] 功能演示/验证
 - [ ] 测试报告 (verify-report.md 已生成)
 - [ ] 代码审查通过
+- [ ] **🚀 实际运行验证** (导入测试、启动测试)
 - [ ] 文档已更新 (如需要)
 - [ ] 所有验收标准已满足
 
@@ -640,9 +716,57 @@ ${task.acceptanceCriteria.map((c, i) => `${i + 1}. [ ] ${c}`).join('\n')}`);
 ### 2. 验证验收标准
 逐项检查验收标准是否满足
 
-### 3. 最终确认
+### 3. 🚀 实际运行验证 (Accept 阶段必须执行)
+
+**目的**: Verify 阶段验证了编译和测试，Accept 阶段验证代码在实际使用中是否可用。
+
+#### 3a. 运行时导入测试
+\`\`\`bash
+# 验证所有核心模块能被正确导入（无运行时错误）
+node -e "
+const modules = [
+  // 根据任务实际产出的模块填入
+  // 示例: './dist/index.js', './dist/core.js'
+];
+let failed = [];
+for (const mod of modules) {
+  try {
+    require(mod);
+    console.log('✅ Import OK:', mod);
+  } catch(e) {
+    console.error('❌ Import FAIL:', mod, e.message);
+    failed.push(mod);
+  }
+}
+if (failed.length > 0) {
+  console.error('Failed imports:', failed.join(', '));
+  process.exit(1);
+}
+console.log('All module imports passed');
+"
+\`\`\`
+
+#### 3b. CLI/入口测试 (如适用)
+\`\`\`bash
+# 如果项目提供 CLI 或 HTTP 服务，做基本调用测试
+# CLI 示例:
+if [ -f "dist/cli/index.js" ]; then
+  node dist/cli/index.js --help && echo "✅ CLI runs" || echo "❌ CLI crashed"
+fi
+
+# HTTP 示例 (如果有 start 脚本):
+# 启动 → 发送测试请求 → 验证响应 → 关闭
+\`\`\`
+
+#### 3c. 结果验证
+- 如果导入/运行成功 → ✅ 继续验收
+- 如果导入/运行失败但测试通过 → ❌ 标记为 ACCEPT_NEEDS_MODIFICATION，指出运行时错误
+- 如果项目无 entry point（纯库/工具）→ ⏭️ 跳过，基于代码审查判断
+
+### 4. 最终确认
 - 确认代码可以合并
 - 确认无遗留问题
+- **确认代码能实际运行**（不是只编译通过）
 
 ## 输出要求
 在 \`.openmatrix/tasks/${task.id}/artifacts/\` 目录下创建:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openmatrix",
-  "version": "0.1.97",
+  "version": "0.1.98",
   "description": "AI Agent task orchestration system with Claude Code Skills integration",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",