refactor: remove dead task helper and normalize internal test hooks

src/app.test.ts

@@ -1,5 +1,5 @@
 import { afterEach, describe, expect, it, vi } from "vitest";
-import { __testing as appTesting, AppContext } from "./app.js";
+import { appTesting, AppContext } from "./app.js";
 import { defaultConfig } from "./config.js";
 import { ClawpatchError } from "./errors.js";
 import type { PartitionedReviewOutput, Provider } from "./provider-types.js";

src/app.ts

@@ -410,8 +410,7 @@ function parseMapSource(flags: Record<string, string | boolean>): "heuristic" |
   );
 }
 
-// eslint-disable-next-line no-underscore-dangle
-export const __testing = {
+export const appTesting = {
   ...reviewTesting,
   reviewFlagSubset,
 };

src/mappers/task-graph.ts

@@ -40,17 +40,3 @@ export function taskGraphCommand(
     (command) => command.projectRoot === project.root && command.task === task,
   )?.command;
 }
-
-export function taskGraphProjectCommands(
-  graph: WorkspaceTaskGraph,
-  project: NodeProjectInfo,
-): Record<string, string> {
-  const commands: Record<string, string> = {};
-  for (const task of validationTaskNames) {
-    const command = taskGraphCommand(graph, project, task);
-    if (command !== undefined && command !== null) {
-      commands[task] = command;
-    }
-  }
-  return commands;
-}

src/provider.test.ts

@@ -1,11 +1,10 @@
 import { afterEach, describe, expect, it } from "vitest";
 import { ClawpatchError } from "./errors.js";
-import { __testing, extractJson, providerByName } from "./provider.js";
+import { providerTesting, extractJson, providerByName } from "./provider.js";
 import { safeProviderPreview } from "./provider-json.js";
 import { agentMapJsonSchema, reviewJsonSchema } from "./provider-schema.js";
 import { evidenceRefSchema, revalidateOutputSchema, reviewOutputSchema } from "./types.js";
 
-// eslint-disable-next-line no-underscore-dangle
 const {
   addClaudeModelArgs,
   acpxFailureMessage,
@@ -43,7 +42,7 @@ const {
   piThinkingLevel,
   providerExitCode,
   providerJsonSchema,
-} = __testing;
+} = providerTesting;
 
 function withEnv(name: string, value: string | undefined, fn: () => void): void {
   const previous = process.env[name];

src/provider.ts

@@ -33,8 +33,7 @@ export function providerByName(name: string): Provider {
   throw new ClawpatchError(`unsupported provider: ${name}`, 2, "unsupported-provider");
 }
 
-// eslint-disable-next-line no-underscore-dangle
-export const __testing = {
+export const providerTesting = {
   ...acpxTesting,
   ...claudeTesting,
   ...codexTesting,

src/providers/acpx.ts

@@ -423,8 +423,6 @@ function acpxPromptRetries(): number {
   return Number.isFinite(parsed) && parsed >= 0 ? Math.floor(parsed) : 1;
 }
 
-// eslint-disable-next-line no-underscore-dangle
-
 export const acpxTesting = {
   acpxFailureMessage,
   acpxPromptRetries,