@nathapp/nax 0.50.2 → 0.51.0

package/src/verification/smart-runner.ts

@@ -1,307 +0,0 @@
-/**
- * Smart Runner — Git diff file detection
- *
- * Detects changed TypeScript source files using git diff,
- * enabling targeted test runs on only the files that changed.
- */
-import { gitWithTimeout } from "../utils/git";
-
-/**
- * Bun API wrappers — defined before functions to avoid circular type inference.
- * Use closures so tests mocking Bun.Glob / Bun.file on the global namespace
- * continue to work (closures evaluate Bun.* at call time).
- *
- * @internal
- */
-const _bunDeps = {
-  glob: (p: string) => new Bun.Glob(p),
-  file: (path: string) => Bun.file(path),
-};
-
-/**
- * Get TypeScript source files changed since the previous commit.
- *
- * Runs `git diff --name-only HEAD~1` in the given workdir and filters
- * results to only `.ts` files under `src/`. Returns an empty array on
- * any git error (not a repo, no previous commit, etc.).
- *
- * @param workdir - Working directory to run git command in
- * @returns Array of changed .ts file paths relative to the repo root
- *
- * @example
- * ```typescript
- * const files = await getChangedSourceFiles("/path/to/repo");
- * // Returns: ["src/foo/bar.ts", "src/utils/git.ts"]
- * ```
- */
-/**
- * Map source files to their corresponding test files.
- *
- * For each file in `sourceFiles`, checks both:
- *   - `<workdir>/test/unit/<relative-path>.test.ts`
- *   - `<workdir>/test/integration/<relative-path>.test.ts`
- *
- * where `<relative-path>` is the file path with the leading `src/` stripped
- * and the `.ts` extension replaced with `.test.ts`.
- *
- * Only returns paths that actually exist on disk.
- *
- * @param sourceFiles - Array of source file paths (e.g. `["src/foo/bar.ts"]`)
- * @param workdir     - Absolute path to the repository root
- * @returns Existing test file paths (absolute)
- *
- * @example
- * ```typescript
- * const tests = await mapSourceToTests(["src/foo/bar.ts"], "/repo");
- * // Returns: ["/repo/test/unit/foo/bar.test.ts"] (if it exists)
- * ```
- */
-/**
- * Extract searchable identifiers from a source file path.
- *
- * For `src/routing/strategies/llm.ts`, returns:
- *   ["/llm", "routing/strategies/llm"]
- *
- * @internal
- */
-function extractSearchTerms(sourceFile: string): string[] {
-  const withoutSrc = sourceFile.replace(/^src\//, "");
-  const withoutExt = withoutSrc.replace(/\.ts$/, "");
-  const parts = withoutExt.split("/");
-  const basename = parts[parts.length - 1];
-  // Use "/basename" to avoid matching short names as plain words
-  return [`/${basename}`, withoutExt];
-}
-
-/**
- * Pass 2 — import-grep fallback.
- *
- * Scans test files matching `testFilePatterns` and returns those that
- * contain an import reference to any of the given `sourceFiles`.
- *
- * @param sourceFiles    - Changed source file paths (e.g. `["src/routing/strategies/llm.ts"]`)
- * @param workdir        - Absolute path to the repository root
- * @param testFilePatterns - Glob patterns to scan for test files
- * @returns Matching test file paths (absolute)
- */
-export async function importGrepFallback(
-  sourceFiles: string[],
-  workdir: string,
-  testFilePatterns: string[],
-): Promise<string[]> {
-  if (sourceFiles.length === 0 || testFilePatterns.length === 0) return [];
-
-  // Collect search terms from all changed source files
-  const searchTerms = sourceFiles.flatMap(extractSearchTerms);
-
-  // Scan all test files matching the configured patterns
-  const testFilePaths: string[] = [];
-  for (const pattern of testFilePatterns) {
-    const glob = _bunDeps.glob(pattern);
-    for await (const file of glob.scan(workdir)) {
-      testFilePaths.push(`${workdir}/${file}`);
-    }
-  }
-
-  // Return test files that contain any of the search terms
-  const matched: string[] = [];
-  for (const testFile of testFilePaths) {
-    let content: string;
-    try {
-      content = await _bunDeps.file(testFile).text();
-    } catch {
-      continue;
-    }
-    for (const term of searchTerms) {
-      if (content.includes(term)) {
-        matched.push(testFile);
-        break;
-      }
-    }
-  }
-
-  return matched;
-}
-
-export async function mapSourceToTests(sourceFiles: string[], workdir: string): Promise<string[]> {
-  const result: string[] = [];
-
-  for (const sourceFile of sourceFiles) {
-    // Strip leading "src/" and replace ".ts" with ".test.ts"
-    const relative = sourceFile.replace(/^src\//, "").replace(/\.ts$/, ".test.ts");
-
-    const candidates = [`${workdir}/test/unit/${relative}`, `${workdir}/test/integration/${relative}`];
-
-    for (const candidate of candidates) {
-      if (await _bunDeps.file(candidate).exists()) {
-        result.push(candidate);
-      }
-    }
-  }
-
-  return result;
-}
-
-/**
- * Build a scoped test command targeting specific test files.
- *
- * When `testFiles` is non-empty, replaces the last path-like argument in
- * `baseCommand` (a token containing `/`) with the specific test file paths
- * joined by spaces. If no path argument is found, appends the test files.
- *
- * When `testFiles` is empty, returns `baseCommand` unchanged (full-suite
- * fallback).
- *
- * @param testFiles   - Test file paths to scope the run to
- * @param baseCommand - Full test command (e.g. `"bun test test/"`)
- * @returns Scoped command string
- *
- * @example
- * ```typescript
- * buildSmartTestCommand(["test/unit/foo.test.ts"], "bun test test/")
- * // => "bun test test/unit/foo.test.ts"
- *
- * buildSmartTestCommand([], "bun test test/")
- * // => "bun test test/"
- * ```
- */
-export function buildSmartTestCommand(testFiles: string[], baseCommand: string): string {
-  if (testFiles.length === 0) {
-    return baseCommand;
-  }
-
-  const parts = baseCommand.trim().split(/\s+/);
-
-  // Find the last token that looks like a path (contains '/')
-  let lastPathIndex = -1;
-  for (let i = parts.length - 1; i >= 0; i--) {
-    if (parts[i].includes("/")) {
-      lastPathIndex = i;
-      break;
-    }
-  }
-
-  if (lastPathIndex === -1) {
-    // No path argument — append test files
-    return `${baseCommand} ${testFiles.join(" ")}`;
-  }
-
-  // Replace the last path argument with the specific test files,
-  // preserving any flags that appear after the path (e.g. --timeout=60000).
-  const beforePath = parts.slice(0, lastPathIndex);
-  const afterPath = parts.slice(lastPathIndex + 1);
-  const newParts = [...beforePath, ...testFiles, ...afterPath];
-  return newParts.join(" ");
-}
-
-/**
- * Get TypeScript source files changed since the previous commit.
- *
- * Runs `git diff --name-only <ref>` in the given workdir and filters
- * results to only `.ts` files under the relevant source prefix.
- *
- * In a monorepo, git returns paths relative to the git root (e.g.
- * `packages/api/src/foo.ts`). When `packagePrefix` is set to the
- * story's workdir (e.g. `"packages/api"`), the filter is scoped to
- * `<packagePrefix>/src/` instead of just `src/`.
- *
- * @param workdir       - Working directory to run git command in
- * @param baseRef       - Git ref for diff base (default: HEAD~1)
- * @param packagePrefix - Story workdir relative to repo root (e.g. "packages/api")
- * @returns Array of changed .ts file paths relative to the git root
- */
-export async function getChangedSourceFiles(
-  workdir: string,
-  baseRef?: string,
-  packagePrefix?: string,
-): Promise<string[]> {
-  try {
-    // FEAT-010: Use per-attempt baseRef for precise diff; fall back to HEAD~1 if not provided
-    const ref = baseRef ?? "HEAD~1";
-    // BUG-039: Use gitWithTimeout to prevent orphan processes on hang
-    const { stdout, exitCode } = await gitWithTimeout(["diff", "--name-only", ref], workdir);
-    if (exitCode !== 0) return [];
-
-    const lines = stdout.trim().split("\n").filter(Boolean);
-
-    // MW-006: scope filter to package prefix in monorepo
-    const srcPrefix = packagePrefix ? `${packagePrefix}/src/` : "src/";
-    return lines.filter((f) => f.startsWith(srcPrefix) && f.endsWith(".ts"));
-  } catch {
-    return [];
-  }
-}
-
-/**
- * Map test files back to their corresponding source files.
- *
- * For each test file path, converts it back to the likely source file path.
- * Handles both `test/unit/` and `test/integration/` conventions.
- * Only processes .test.ts files (not .test.js).
- *
- * @param testFiles - Array of test file paths (e.g. `["/repo/test/unit/foo/bar.test.ts"]`)
- * @param workdir - Absolute path to the repository root (to normalize paths)
- * @returns Source file paths (e.g. `["src/foo/bar.ts"]`)
- *
- * @example
- * ```typescript
- * const sources = reverseMapTestToSource(["/repo/test/unit/foo/bar.test.ts"], "/repo");
- * // Returns: ["src/foo/bar.ts"]
- * ```
- */
-export function reverseMapTestToSource(testFiles: string[], workdir: string): string[] {
-  const result: string[] = [];
-  const seenPaths = new Set<string>();
-
-  for (const testFile of testFiles) {
-    // Only process .test.ts files
-    if (!testFile.endsWith(".test.ts")) {
-      continue;
-    }
-
-    // Normalize the path to be relative to workdir
-    let relativePath = testFile.startsWith(workdir) ? testFile.slice(workdir.length + 1) : testFile;
-
-    // Remove test/unit/ or test/integration/ prefix
-    if (relativePath.startsWith("test/unit/")) {
-      relativePath = relativePath.slice("test/unit/".length);
-    } else if (relativePath.startsWith("test/integration/")) {
-      relativePath = relativePath.slice("test/integration/".length);
-    } else {
-      continue; // Not a recognized test file pattern
-    }
-
-    // Replace .test.ts with .ts and add src/ prefix
-    const sourcePath = `src/${relativePath.replace(/\.test\.ts$/, ".ts")}`;
-
-    if (!seenPaths.has(sourcePath)) {
-      result.push(sourcePath);
-      seenPaths.add(sourcePath);
-    }
-  }
-
-  return result;
-}
-
-/**
- * Injectable dependencies for testing.
- * Allows tests to swap implementations without using mock.module(),
- * which leaks across files in Bun 1.x due to shared module registry.
- *
- * Bun API wrappers use closures so that tests mocking Bun.Glob / Bun.file
- * on the global namespace continue to work (closures evaluate Bun.* at
- * call time, not at module initialisation time).
- *
- * @internal - test use only. Do not use in production code.
- */
-export const _smartRunnerDeps = {
-  /** Wraps Bun.Glob construction — injectable for testing. */
-  glob: _bunDeps.glob,
-  /** Wraps Bun.file — injectable for testing. */
-  file: _bunDeps.file,
-  getChangedSourceFiles,
-  mapSourceToTests,
-  importGrepFallback,
-  buildSmartTestCommand,
-  reverseMapTestToSource,
-};

package/src/verification/strategies/acceptance.ts

@@ -1,136 +0,0 @@
-// RE-ARCH: keep
-/**
- * Acceptance Verification Strategy (ADR-005, Phase 1)
- *
- * Runs acceptance tests from a feature directory test file.
- * Extracted from src/pipeline/stages/acceptance.ts.
- */
-
-import path from "node:path";
-import { getLogger } from "../../logger";
-import type { IVerificationStrategy, VerifyContext, VerifyResult } from "../orchestrator-types";
-import { makeFailResult, makePassResult, makeSkippedResult } from "../orchestrator-types";
-
-/** Injectable deps for testability */
-export const _acceptanceDeps = { spawn: Bun.spawn as typeof Bun.spawn };
-
-function parseFailedACs(output: string): string[] {
-  const failed: string[] = [];
-  for (const line of output.split("\n")) {
-    if (line.includes("(fail)")) {
-      const m = line.match(/(AC-\d+):/i);
-      if (m) {
-        const id = m[1].toUpperCase();
-        if (!failed.includes(id)) failed.push(id);
-      }
-    }
-  }
-  return failed;
-}
-
-export class AcceptanceStrategy implements IVerificationStrategy {
-  readonly name = "acceptance" as const;
-
-  async execute(ctx: VerifyContext): Promise<VerifyResult> {
-    const logger = getLogger();
-
-    if (!ctx.acceptanceTestPath) {
-      logger.warn("verify[acceptance]", "No acceptance test path provided — skipping", { storyId: ctx.storyId });
-      return makeSkippedResult(ctx.storyId, "acceptance");
-    }
-
-    const testPath = path.isAbsolute(ctx.acceptanceTestPath)
-      ? ctx.acceptanceTestPath
-      : path.join(ctx.workdir, ctx.acceptanceTestPath);
-
-    const exists = await Bun.file(testPath).exists();
-    if (!exists) {
-      logger.warn("verify[acceptance]", "Acceptance test file not found — skipping", {
-        storyId: ctx.storyId,
-        testPath,
-      });
-      return makeSkippedResult(ctx.storyId, "acceptance");
-    }
-
-    const start = Date.now();
-    const timeoutMs = ctx.timeoutSeconds * 1000;
-    const proc = _acceptanceDeps.spawn(["bun", "test", testPath], {
-      cwd: ctx.workdir,
-      stdout: "pipe",
-      stderr: "pipe",
-    });
-
-    let timedOut = false;
-    const timeoutId = setTimeout(() => {
-      timedOut = true;
-      try {
-        proc.kill("SIGTERM");
-      } catch {
-        /* already exited */
-      }
-      setTimeout(() => {
-        try {
-          proc.kill("SIGKILL");
-        } catch {
-          /* already exited */
-        }
-      }, 5000);
-    }, timeoutMs);
-
-    const exitCode = await Promise.race([
-      proc.exited,
-      new Promise<number>((resolve) => setTimeout(() => resolve(124), timeoutMs + 6000)),
-    ]);
-    clearTimeout(timeoutId);
-    const stdout = await Promise.race([
-      new Response(proc.stdout).text(),
-      new Promise<string>((resolve) => setTimeout(() => resolve(""), 3000)),
-    ]);
-    const stderr = await Promise.race([
-      new Response(proc.stderr).text(),
-      new Promise<string>((resolve) => setTimeout(() => resolve(""), 3000)),
-    ]);
-    const durationMs = Date.now() - start;
-
-    if (timedOut || exitCode === 124) {
-      logger.warn("verify[acceptance]", "Acceptance tests timed out", {
-        storyId: ctx.storyId,
-        timeoutSeconds: ctx.timeoutSeconds,
-      });
-      return makePassResult(ctx.storyId, "acceptance", { rawOutput: "TIMEOUT", durationMs });
-    }
-    const output = `${stdout}\n${stderr}`;
-
-    const failedACs = parseFailedACs(output);
-
-    if (exitCode === 0 && failedACs.length === 0) {
-      logger.info("verify[acceptance]", "All acceptance tests passed", { storyId: ctx.storyId });
-      return makePassResult(ctx.storyId, "acceptance", { rawOutput: output, durationMs });
-    }
-
-    if (failedACs.length === 0) {
-      // Test process failed but we couldn't parse AC IDs — treat as pass (setup error)
-      logger.warn("verify[acceptance]", "Tests failed but no AC IDs detected — treating as pass", {
-        storyId: ctx.storyId,
-      });
-      return makePassResult(ctx.storyId, "acceptance", { rawOutput: output, durationMs });
-    }
-
-    logger.error("verify[acceptance]", "Acceptance tests failed", {
-      storyId: ctx.storyId,
-      failedACs,
-    });
-
-    return makeFailResult(ctx.storyId, "acceptance", "TEST_FAILURE", {
-      rawOutput: output,
-      failCount: failedACs.length,
-      failures: failedACs.map((acId) => ({
-        file: testPath,
-        testName: acId,
-        error: `Acceptance criterion ${acId} failed`,
-        stackTrace: [],
-      })),
-      durationMs,
-    });
-  }
-}

package/src/verification/strategies/regression.ts

@@ -1,90 +0,0 @@
-// RE-ARCH: keep
-/**
- * Regression Verification Strategy (ADR-005, Phase 1)
- *
- * Runs the full test suite as a regression gate. Supports both inline
- * (per-story) and deferred (end-of-run) modes via context flag.
- *
- * Extracted from src/execution/post-verify.ts and
- * src/execution/lifecycle/run-regression.ts.
- */
-
-import { getSafeLogger } from "../../logger";
-import type { IVerificationStrategy, VerifyContext, VerifyResult } from "../orchestrator-types";
-import { makeFailResult, makePassResult, makeSkippedResult } from "../orchestrator-types";
-import { parseBunTestOutput } from "../parser";
-import { fullSuite } from "../runners";
-
-export class RegressionStrategy implements IVerificationStrategy {
-  readonly name: "regression" | "deferred-regression" = "regression";
-
-  async execute(ctx: VerifyContext): Promise<VerifyResult> {
-    const logger = getSafeLogger();
-    const config = ctx.config;
-
-    const enabled = config?.execution.regressionGate?.enabled ?? true;
-    if (!enabled) {
-      return makeSkippedResult(ctx.storyId, "regression");
-    }
-
-    logger?.info("verify[regression]", "Running full-suite regression gate", { storyId: ctx.storyId });
-
-    const start = Date.now();
-    const result = await _regressionStrategyDeps.runVerification({
-      workdir: ctx.workdir,
-      expectedFiles: [],
-      command: ctx.testCommand,
-      timeoutSeconds: ctx.timeoutSeconds,
-      forceExit: config?.quality.forceExit,
-      detectOpenHandles: config?.quality.detectOpenHandles,
-      detectOpenHandlesRetries: config?.quality.detectOpenHandlesRetries,
-      timeoutRetryCount: 0,
-      gracePeriodMs: config?.quality.gracePeriodMs,
-      drainTimeoutMs: config?.quality.drainTimeoutMs,
-      shell: config?.quality.shell,
-      stripEnvVars: config?.quality.stripEnvVars,
-    });
-    const durationMs = Date.now() - start;
-
-    if (result.success) {
-      const parsed = result.output ? parseBunTestOutput(result.output) : { passed: 0, failed: 0, failures: [] };
-      return makePassResult(ctx.storyId, "regression", {
-        rawOutput: result.output,
-        passCount: parsed.passed,
-        durationMs,
-      });
-    }
-
-    // Accept timeout as pass if configured (BUG-026)
-    if (result.status === "TIMEOUT" && (ctx.acceptOnTimeout ?? true)) {
-      logger?.warn("verify[regression]", "[BUG-026] Full-suite timed out (accepted as pass)", {
-        storyId: ctx.storyId,
-      });
-      return makePassResult(ctx.storyId, "regression", { durationMs });
-    }
-
-    if (result.status === "TIMEOUT") {
-      return makeFailResult(ctx.storyId, "regression", "TIMEOUT", { rawOutput: result.output, durationMs });
-    }
-
-    const parsed = result.output ? parseBunTestOutput(result.output) : { passed: 0, failed: 0, failures: [] };
-    return makeFailResult(ctx.storyId, "regression", "TEST_FAILURE", {
-      rawOutput: result.output,
-      passCount: parsed.passed,
-      failCount: parsed.failed,
-      failures: parsed.failures,
-      durationMs,
-    });
-  }
-}
-
-export const _regressionStrategyDeps = { runVerification: fullSuite };
-
-/**
- * Deferred Regression Strategy — same logic as RegressionStrategy but
- * reports strategy name as "deferred-regression" in VerifyResult so
- * downstream code branching on strategy name works correctly.
- */
-export class DeferredRegressionStrategy extends RegressionStrategy {
-  override readonly name = "deferred-regression" as const;
-}

package/src/verification/strategies/scoped.ts

@@ -1,154 +0,0 @@
-// RE-ARCH: keep
-/**
- * Scoped Verification Strategy (ADR-005, Phase 1)
- *
- * Runs tests scoped to files changed by the current story using the smart
- * test runner. Falls back to full suite when no specific tests are mapped.
- * Falls through to SKIPPED when in deferred mode and no tests are mapped.
- *
- * Extracted from src/pipeline/stages/verify.ts (lines 40-164).
- * Must produce identical results to the existing verify stage.
- */
-
-import { getLogger } from "../../logger";
-import type { IVerificationStrategy, VerifyContext, VerifyResult } from "../orchestrator-types";
-import { makeFailResult, makePassResult, makeSkippedResult } from "../orchestrator-types";
-import { parseBunTestOutput } from "../parser";
-import { regression } from "../runners";
-import { _smartRunnerDeps } from "../smart-runner";
-
-const DEFAULT_SMART_RUNNER_CONFIG = {
-  enabled: true,
-  testFilePatterns: ["test/**/*.test.ts"],
-  fallback: "import-grep" as const,
-};
-
-function coerceSmartRunner(val: unknown) {
-  if (val === undefined || val === true) return DEFAULT_SMART_RUNNER_CONFIG;
-  if (val === false) return { ...DEFAULT_SMART_RUNNER_CONFIG, enabled: false };
-  return val as typeof DEFAULT_SMART_RUNNER_CONFIG;
-}
-
-function buildScopedCommand(testFiles: string[], baseCommand: string, testScopedTemplate?: string): string {
-  if (testScopedTemplate) {
-    return testScopedTemplate.replace("{{files}}", testFiles.join(" "));
-  }
-  return _scopedDeps.buildSmartTestCommand(testFiles, baseCommand);
-}
-
-/**
- * Returns true when the test command delegates to a monorepo orchestrator
- * (Turborepo, Nx) that handles change-aware scoping natively.
- *
- * These tools use their own filter syntax (e.g. `--filter=...[HEAD~1]`,
- * `nx affected`) — nax's smart test runner must not attempt to append
- * file paths to such commands, as it would produce invalid syntax.
- */
-export function isMonorepoOrchestratorCommand(command: string): boolean {
-  return /\bturbo\b/.test(command) || /\bnx\b/.test(command);
-}
-
-export class ScopedStrategy implements IVerificationStrategy {
-  readonly name = "scoped" as const;
-
-  async execute(ctx: VerifyContext): Promise<VerifyResult> {
-    const logger = getLogger();
-    const smartCfg = coerceSmartRunner(ctx.smartRunnerConfig);
-    const regressionMode = ctx.regressionMode ?? "deferred";
-
-    // Monorepo orchestrators (turbo, nx) handle change-aware scoping themselves.
-    // Skip nax's smart runner — appending file paths would produce invalid syntax.
-    // Also bypass deferred mode: run per-story so the orchestrator's own filter
-    // (e.g. --filter=...[HEAD~1]) can pick up the story's changes immediately.
-    const isMonorepoOrchestrator = isMonorepoOrchestratorCommand(ctx.testCommand);
-
-    let effectiveCommand = ctx.testCommand;
-    let isFullSuite = true;
-
-    if (smartCfg.enabled && ctx.storyGitRef && !isMonorepoOrchestrator) {
-      const sourceFiles = await _scopedDeps.getChangedSourceFiles(ctx.workdir, ctx.storyGitRef);
-
-      const pass1Files = await _scopedDeps.mapSourceToTests(sourceFiles, ctx.workdir);
-      if (pass1Files.length > 0) {
-        logger.info("verify[scoped]", `Pass 1: path convention matched ${pass1Files.length} test files`, {
-          storyId: ctx.storyId,
-        });
-        effectiveCommand = buildScopedCommand(pass1Files, ctx.testCommand, ctx.testScopedTemplate);
-        isFullSuite = false;
-      } else if (smartCfg.fallback === "import-grep") {
-        const pass2Files = await _scopedDeps.importGrepFallback(sourceFiles, ctx.workdir, smartCfg.testFilePatterns);
-        if (pass2Files.length > 0) {
-          logger.info("verify[scoped]", `Pass 2: import-grep matched ${pass2Files.length} test files`, {
-            storyId: ctx.storyId,
-          });
-          effectiveCommand = buildScopedCommand(pass2Files, ctx.testCommand, ctx.testScopedTemplate);
-          isFullSuite = false;
-        }
-      }
-    }
-
-    // Defer to regression gate when no scoped tests found and mode is deferred.
-    // Exception: monorepo orchestrators run per-story (they carry their own change filter).
-    if (isFullSuite && regressionMode === "deferred" && !isMonorepoOrchestrator) {
-      logger.info("verify[scoped]", "No mapped tests — deferring to run-end (mode: deferred)", {
-        storyId: ctx.storyId,
-      });
-      return makeSkippedResult(ctx.storyId, "scoped");
-    }
-
-    if (isFullSuite && !isMonorepoOrchestrator) {
-      logger.info("verify[scoped]", "No mapped tests — falling back to full suite", { storyId: ctx.storyId });
-    } else if (isMonorepoOrchestrator) {
-      logger.info("verify[scoped]", "Monorepo orchestrator detected — delegating scoping to tool", {
-        storyId: ctx.storyId,
-        command: effectiveCommand,
-      });
-    }
-
-    const start = Date.now();
-    const result = await _scopedDeps.regression({
-      workdir: ctx.workdir,
-      command: effectiveCommand,
-      timeoutSeconds: ctx.timeoutSeconds,
-      acceptOnTimeout: ctx.acceptOnTimeout ?? true,
-    });
-    const durationMs = Date.now() - start;
-
-    if (result.success) {
-      const parsed = result.output ? parseBunTestOutput(result.output) : { passed: 0, failed: 0, failures: [] };
-      return makePassResult(ctx.storyId, "scoped", {
-        rawOutput: result.output,
-        passCount: parsed.passed,
-        durationMs,
-      });
-    }
-
-    if (result.status === "TIMEOUT") {
-      return makeFailResult(ctx.storyId, "scoped", "TIMEOUT", {
-        rawOutput: result.output,
-        durationMs,
-        countsTowardEscalation: false,
-      });
-    }
-
-    const parsed = result.output ? parseBunTestOutput(result.output) : { passed: 0, failed: 0, failures: [] };
-    return makeFailResult(ctx.storyId, "scoped", "TEST_FAILURE", {
-      rawOutput: result.output,
-      passCount: parsed.passed,
-      failCount: parsed.failed,
-      failures: parsed.failures,
-      durationMs,
-    });
-  }
-}
-
-/**
- * Injectable deps for testing.
- */
-export const _scopedDeps = {
-  getChangedSourceFiles: _smartRunnerDeps.getChangedSourceFiles,
-  mapSourceToTests: _smartRunnerDeps.mapSourceToTests,
-  importGrepFallback: _smartRunnerDeps.importGrepFallback,
-  buildSmartTestCommand: _smartRunnerDeps.buildSmartTestCommand,
-  regression,
-};