@angli/unit-test-tool 0.1.4 → 0.1.5

package/README.md

@@ -201,21 +201,30 @@ testbot schedule --project /path/to/project --interval 5m
 testbot start --project /path/to/project --schedule --guard-interval 5m
 ```
 
-### 8) Claude CLI 项目级配置
+### 8) Claude CLI 项目级配置（必填）
 
-补测执行时会优先读取“待补测项目目录”下的 `.claude/settings.local.json`，并使用其中 `env` 的 `ANTHROPIC_AUTH_TOKEN` / `ANTHROPIC_BASE_URL` 覆盖全局环境变量。
+补测执行时会优先读取**待补测项目目录**下的 `.claude/settings.local.json`，并使用其中 `env` 的 `ANTHROPIC_AUTH_TOKEN` / `ANTHROPIC_BASE_URL` 覆盖全局环境变量。
 
-示例：
+如果目标项目没有配置 `env`，且全局环境变量也不可用，**所有非平凡目标（Vue 组件、HOC 等）的 Claude CLI 调用将全部失败**，导致只能处理 `.d.ts`、`.js` 等简单文件。
+
+配置方式：在目标项目的 `.claude/settings.local.json` 中添加 `env` 字段：
 
 ```json
 {
-  "env": {
-    "ANTHROPIC_AUTH_TOKEN": "...",
-    "ANTHROPIC_BASE_URL": "..."
+  “env”: {
+    “ANTHROPIC_AUTH_TOKEN”: “sk-ant-...”,
+    “ANTHROPIC_BASE_URL”: “https://api.anthropic.com”
+  },
+  “permissions”: {
+    “allow”: [“Bash(*)”, “Edit(*)”, “Write(*)”, “Read(*)”],
+    “deny”: [],
+    “ask”: []
   }
 }
 ```
 
+> **注意**：如果全局环境变量中已正确配置 `ANTHROPIC_AUTH_TOKEN`，可以不写 `env`，工具会使用全局环境变量。
+
 ### 9) config.json 中的 promptOverrides
 
 `testbot init` 写入的 `.unit_test_tool_workspace/config.json` 支持项目级 prompt 覆盖。工具会先使用内置“通用单测策略 + 案例片段库”，再追加这里的覆盖内容。

package/dist/src/core/tools/run-loop.js

@@ -21,6 +21,23 @@ export async function runLoop(ctx, input = {}) {
         lastResult = result;
         if (result.targetFile)
             attemptedTargets.add(result.targetFile);
+        // 如果当前目标失败（CLI 失败或 no-op），且不是显式指定文件，则标记为跳过并继续下一轮
+        const isBlocked = !result.ok && !input.targetFile;
+        if (isBlocked) {
+            blockedTargets += 1;
+            await syncLoopSummary(ctx, {
+                iterationCount,
+                completedTargets,
+                blockedTargets,
+                noOpTargets,
+                lastCompletedTarget
+            });
+            if (blockedTargets >= Math.max(1, ctx.limits.maxRetryPerTask))
+                break;
+            if (iterationCount >= maxIterations)
+                break;
+            continue;
+        }
         const coverageSummary = await ctx.reportStore.loadCoverageSummary();
         const currentCoverage = coverageSummary?.coverageSnapshot?.lines;
         lastCoverage = currentCoverage;
@@ -39,12 +56,7 @@ export async function runLoop(ctx, input = {}) {
             });
             continue;
         }
-        if (result.summary.includes('no-op')) {
-            noOpTargets += 1;
-        }
-        else {
-            blockedTargets += 1;
-        }
+        noOpTargets += 1;
         await syncLoopSummary(ctx, {
             iterationCount,
             completedTargets,
@@ -54,8 +66,6 @@ export async function runLoop(ctx, input = {}) {
         });
         if (input.targetFile)
             break;
-        if (blockedTargets >= Math.max(1, ctx.limits.maxRetryPerTask))
-            break;
         if (noOpTargets >= Math.max(1, ctx.limits.loopThreshold))
             break;
         if (!result.targetFile)

package/dist/src/core/tools/run-with-claude-cli.js

@@ -118,7 +118,8 @@ export async function runWithClaudeCli(ctx, input = {}) {
             summary: `Claude CLI 执行失败：${firstRun.failureCategory ?? 'unknown'}。`
         }, 'cli_run_failed', {
             stdoutPath: firstStdoutPath,
-            stderrPath: firstStderrPath
+            stderrPath: firstStderrPath,
+            failureMessage
         });
         await markRunFailed(ctx, target.task, failureMessage);
         return {
@@ -244,7 +245,8 @@ export async function runWithClaudeCli(ctx, input = {}) {
             summary: `Claude CLI 重试失败：${retryRun.failureCategory ?? 'unknown'}。`
         }, 'cli_run_failed', {
             stdoutPath: retryStdoutSaved,
-            stderrPath: retryStderrSaved
+            stderrPath: retryStderrSaved,
+            failureMessage
         });
         await markRunFailed(ctx, target.task, failureMessage);
         return {
@@ -503,6 +505,19 @@ async function validateTestChanges(ctx, task, beforeSnapshot) {
             relatedTestFiles
         };
     }
+    // Snapshot diff 未命中时，兜底检查：是否有新创建的文件匹配目标
+    const newCandidateFiles = await detectNewTestFiles(ctx, task);
+    if (newCandidateFiles.length > 0) {
+        ctx.logger.info(`[run] step=test-change status=new-files taskId=${task.taskId} target=${task.targetFile} new=${newCandidateFiles.join(',')}`);
+        return {
+            ok: true,
+            phase: 'DONE',
+            summary: '检测到新创建的测试文件。',
+            targetFile: task.targetFile,
+            changedTestFiles: newCandidateFiles,
+            relatedTestFiles: newCandidateFiles
+        };
+    }
     ctx.logger.warn(`[run] step=test-change status=noop taskId=${task.taskId} target=${task.targetFile} changed=${changedTestFiles.join(',') || 'none'}`);
     return {
         ok: false,
@@ -518,6 +533,37 @@ async function validateTestChanges(ctx, task, beforeSnapshot) {
         relatedTestFiles
     };
 }
+async function detectNewTestFiles(ctx, task) {
+    const patterns = buildTestSnapshotPatterns(ctx.config.testDirNames);
+    const files = await fg(patterns, {
+        cwd: ctx.projectPath,
+        ignore: ctx.config.exclude,
+        onlyFiles: true,
+        absolute: false
+    });
+    const baseName = path.basename(task.targetFile, path.extname(task.targetFile));
+    const targetDir = path.dirname(task.targetFile);
+    const normalizedTargetDir = normalizePath(targetDir);
+    const candidates = files.filter((file) => {
+        const fileBaseName = path.basename(file, path.extname(file));
+        const normalizedFile = normalizePath(file);
+        // 匹配同名或同目录下的测试文件
+        return (fileBaseName === baseName ||
+            fileBaseName === `${baseName}.test` ||
+            fileBaseName === `${baseName}.spec` ||
+            normalizedFile.startsWith(`${normalizedTargetDir}/`));
+    });
+    // 只保留确实存在且非空的
+    const result = [];
+    for (const file of candidates) {
+        const fullPath = path.join(ctx.projectPath, file);
+        const content = await ctx.fileSystem.readText(fullPath);
+        if (content !== undefined && content.trim().length > 0) {
+            result.push(file);
+        }
+    }
+    return result.sort();
+}
 function findRelatedTestFiles(task, changedTestFiles) {
     if (changedTestFiles.length === 0)
         return [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angli/unit-test-tool",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "type": "module",
   "bin": {
     "testbot": "./dist/src/cli/index.js"

package/src/core/tools/run-loop.ts

@@ -40,6 +40,21 @@ export async function runLoop(
 
     if (result.targetFile) attemptedTargets.add(result.targetFile)
 
+    // 如果当前目标失败（CLI 失败或 no-op），且不是显式指定文件，则标记为跳过并继续下一轮
+    const isBlocked = !result.ok && !input.targetFile
+    if (isBlocked) {
+      blockedTargets += 1
+      await syncLoopSummary(ctx, {
+        iterationCount,
+        completedTargets,
+        blockedTargets,
+        noOpTargets,
+        lastCompletedTarget
+      })
+      if (blockedTargets >= Math.max(1, ctx.limits.maxRetryPerTask)) break
+      if (iterationCount >= maxIterations) break
+      continue
+    }
 
     const coverageSummary = await ctx.reportStore.loadCoverageSummary<{ coverageSnapshot?: { lines?: number } }>()
     const currentCoverage = coverageSummary?.coverageSnapshot?.lines
@@ -61,11 +76,7 @@ export async function runLoop(
       continue
     }
 
-    if (result.summary.includes('no-op')) {
-      noOpTargets += 1
-    } else {
-      blockedTargets += 1
-    }
+    noOpTargets += 1
 
     await syncLoopSummary(ctx, {
       iterationCount,
@@ -76,7 +87,6 @@ export async function runLoop(
     })
 
     if (input.targetFile) break
-    if (blockedTargets >= Math.max(1, ctx.limits.maxRetryPerTask)) break
     if (noOpTargets >= Math.max(1, ctx.limits.loopThreshold)) break
     if (!result.targetFile) break
   }

package/src/core/tools/run-with-claude-cli.ts

@@ -206,7 +206,8 @@ export async function runWithClaudeCli(
       summary: `Claude CLI 执行失败：${firstRun.failureCategory ?? 'unknown'}。`
     }, 'cli_run_failed', {
       stdoutPath: firstStdoutPath,
-      stderrPath: firstStderrPath
+      stderrPath: firstStderrPath,
+      failureMessage
     })
     await markRunFailed(ctx, target.task, failureMessage)
     return {
@@ -388,7 +389,8 @@ export async function runWithClaudeCli(
       summary: `Claude CLI 重试失败：${retryRun.failureCategory ?? 'unknown'}。`
     }, 'cli_run_failed', {
       stdoutPath: retryStdoutSaved,
-      stderrPath: retryStderrSaved
+      stderrPath: retryStderrSaved,
+      failureMessage
     })
     await markRunFailed(ctx, target.task, failureMessage)
     return {
@@ -742,6 +744,22 @@ async function validateTestChanges(
     }
   }
 
+  // Snapshot diff 未命中时，兜底检查：是否有新创建的文件匹配目标
+  const newCandidateFiles = await detectNewTestFiles(ctx, task)
+  if (newCandidateFiles.length > 0) {
+    ctx.logger.info(
+      `[run] step=test-change status=new-files taskId=${task.taskId} target=${task.targetFile} new=${newCandidateFiles.join(',')}`
+    )
+    return {
+      ok: true,
+      phase: 'DONE',
+      summary: '检测到新创建的测试文件。',
+      targetFile: task.targetFile,
+      changedTestFiles: newCandidateFiles,
+      relatedTestFiles: newCandidateFiles
+    }
+  }
+
   ctx.logger.warn(
     `[run] step=test-change status=noop taskId=${task.taskId} target=${task.targetFile} changed=${changedTestFiles.join(',') || 'none'}`
   )
@@ -760,6 +778,43 @@ async function validateTestChanges(
   }
 }
 
+async function detectNewTestFiles(ctx: AppContext, task: TaskItem): Promise<string[]> {
+  const patterns = buildTestSnapshotPatterns(ctx.config.testDirNames)
+  const files = await fg(patterns, {
+    cwd: ctx.projectPath,
+    ignore: ctx.config.exclude,
+    onlyFiles: true,
+    absolute: false
+  })
+
+  const baseName = path.basename(task.targetFile, path.extname(task.targetFile))
+  const targetDir = path.dirname(task.targetFile)
+  const normalizedTargetDir = normalizePath(targetDir)
+
+  const candidates = files.filter((file) => {
+    const fileBaseName = path.basename(file, path.extname(file))
+    const normalizedFile = normalizePath(file)
+    // 匹配同名或同目录下的测试文件
+    return (
+      fileBaseName === baseName ||
+      fileBaseName === `${baseName}.test` ||
+      fileBaseName === `${baseName}.spec` ||
+      normalizedFile.startsWith(`${normalizedTargetDir}/`)
+    )
+  })
+
+  // 只保留确实存在且非空的
+  const result: string[] = []
+  for (const file of candidates) {
+    const fullPath = path.join(ctx.projectPath, file)
+    const content = await ctx.fileSystem.readText(fullPath)
+    if (content !== undefined && content.trim().length > 0) {
+      result.push(file)
+    }
+  }
+  return result.sort()
+}
+
 function findRelatedTestFiles(task: TaskItem, changedTestFiles: string[]): string[] {
   if (changedTestFiles.length === 0) return []