style(tests): apply consistent formatting to diagnostic tests

- Import statement formatting (commas, line breaks)
- Numeric separators for large numbers (300_000 instead of 300000)
- Auto-formatting from Prettier/ESLint

Affected files:
- checkpoint-hash-bug.diagnostic.test.ts
- checkpoint-merge-bug.diagnostic.test.ts
- checkpoint-merge-bug.unit.test.ts
- parallel-checkpoint-merge.integration.test.ts

Author: Mearman

src/experiments/framework/executor/__tests__/checkpoint-hash-bug.diagnostic.test.ts

@@ -7,17 +7,18 @@
  * This test diagnoses and will help fix the hash matching issue.
  */
 
-import { describe, it, expect } from "vitest";
 import { createHash } from "node:crypto";
 
+import { describe, expect,it } from "vitest";
+
 describe("Checkpoint Config Hash Bug Diagnostics", () => {
 	it("diagnostic-1: identifies which properties cause hash mismatch", () => {
 		// Main process config (from evaluate.ts)
 		const mainProcessConfig = {
 			continueOnError: true,
 			repetitions: 1,
 			seedBase: 42,
-			timeoutMs: 300000,
+			timeoutMs: 300_000,
 			collectProvenance: true,
 			concurrency: 12, // This varies!
 			onProgress: undefined,
@@ -29,7 +30,7 @@ describe("Checkpoint Config Hash Bug Diagnostics", () => {
 			continueOnError: true,
 			repetitions: 1,
 			seedBase: 42,
-			timeoutMs: 300000,
+			timeoutMs: 300_000,
 			collectProvenance: true,
 		};
 
@@ -62,7 +63,7 @@ describe("Checkpoint Config Hash Bug Diagnostics", () => {
 			continueOnError: true,
 			repetitions: 1,
 			seedBase: 42,
-			timeoutMs: 300000,
+			timeoutMs: 300_000,
 			collectProvenance: true,
 			concurrency: 12,
 			onProgress: () => {},
@@ -73,7 +74,7 @@ describe("Checkpoint Config Hash Bug Diagnostics", () => {
 			continueOnError: true,
 			repetitions: 1,
 			seedBase: 42,
-			timeoutMs: 300000,
+			timeoutMs: 300_000,
 			collectProvenance: true,
 			concurrency: 4, // Different!
 			onProgress: undefined,
@@ -105,7 +106,7 @@ describe("Checkpoint Config Hash Bug Diagnostics", () => {
 			continueOnError: true,
 			repetitions: 1,
 			seedBase: 42,
-			timeoutMs: 300000,
+			timeoutMs: 300_000,
 			collectProvenance: true,
 			concurrency: 12,
 		};
@@ -115,7 +116,7 @@ describe("Checkpoint Config Hash Bug Diagnostics", () => {
 			continueOnError: true,
 			repetitions: 1,
 			seedBase: 42,
-			timeoutMs: 300000,
+			timeoutMs: 300_000,
 			collectProvenance: true,
 		};
 

src/experiments/framework/executor/__tests__/checkpoint-merge-bug.diagnostic.test.ts

@@ -8,16 +8,17 @@
  * - Main checkpoint has 123/132 runs but workers start fresh
  */
 
-import { describe, it, expect, beforeEach, afterEach } from "vitest";
-import { rmSync } from "node:fs";
 import { randomBytes } from "node:crypto";
+import { rmSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 
+import { afterEach,beforeEach, describe, expect, it } from "vitest";
+
+import type { CaseDefinition, SutDefinition } from "../../types/index.js";
 import type { EvaluationResult } from "../../types/result.js";
 import { CheckpointManager } from "../checkpoint-manager.js";
 import { FileStorage } from "../checkpoint-storage.js";
-import type { CaseDefinition, SutDefinition } from "../../types/index.js";
 
 describe("Checkpoint Integration Bug Diagnostics", () => {
 	let testDir: string;
@@ -191,13 +192,13 @@ describe("Checkpoint Integration Bug Diagnostics", () => {
 
 	it("diagnostic-5: config hash should include all executor config properties", async () => {
 		// This test verifies which properties are included in the hash
-		const crypto = require("crypto");
+		const crypto = require("node:crypto");
 
 		const config1 = {
 			continueOnError: true,
 			repetitions: 1,
 			seedBase: 42,
-			timeoutMs: 300000,
+			timeoutMs: 300_000,
 			collectProvenance: true,
 		};
 
@@ -226,38 +227,35 @@ describe("Checkpoint Integration Bug Diagnostics", () => {
 /**
  * Create a mock SUT for testing.
  */
-function createMockSut(): SutDefinition<unknown, unknown> {
-	return {
-		registration: {
-			id: "mock-sut-v1.0.0",
-			name: "Mock SUT",
-			version: "1.0.0",
-			role: "primary",
-			config: {},
-			tags: ["test"],
-		},
-		factory: () => ({
-			id: "mock-sut-v1.0.0",
-			config: {},
-			run: async () => ({ mockResult: true }),
-		}),
-	};
-}
+const createMockSut = (): SutDefinition<unknown, unknown> => ({
+	registration: {
+		id: "mock-sut-v1.0.0",
+		name: "Mock SUT",
+		version: "1.0.0",
+		role: "primary",
+		config: {},
+		tags: ["test"],
+	},
+	factory: () => ({
+		id: "mock-sut-v1.0.0",
+		config: {},
+		run: async () => ({ mockResult: true }),
+	}),
+});
 
 /**
  * Create a mock case for testing.
+ * @param id
  */
-function createMockCase(id: string): CaseDefinition<unknown, unknown> {
-	return {
-		case: {
-			caseId: id,
-			name: `Mock Case ${id}`,
-			caseClass: "test",
-			inputs: { summary: { test: id } },
-			tags: ["test"],
-			version: "1.0.0",
-		},
-		getInput: async () => ({ mockInput: true }),
-		getInputs: () => ({ mockInputs: true }),
-	};
-}
+const createMockCase = (id: string): CaseDefinition<unknown, unknown> => ({
+	case: {
+		caseId: id,
+		name: `Mock Case ${id}`,
+		caseClass: "test",
+		inputs: { summary: { test: id } },
+		tags: ["test"],
+		version: "1.0.0",
+	},
+	getInput: async () => ({ mockInput: true }),
+	getInputs: () => ({ mockInputs: true }),
+});

src/experiments/framework/executor/__tests__/checkpoint-merge-bug.unit.test.ts

@@ -5,13 +5,13 @@
  * the main checkpoint instead of combining it with shard results.
  */
 
-import { describe, it, expect, beforeEach, afterEach } from "vitest";
-import { readFileSync } from "node:fs";
-import { rmSync } from "node:fs";
 import { randomBytes } from "node:crypto";
+import { readFileSync , rmSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 
+import { afterEach,beforeEach, describe, expect, it } from "vitest";
+
 import type { EvaluationResult } from "../../types/result.js";
 import type { CheckpointData } from "../checkpoint-manager.js";
 import { CheckpointManager } from "../checkpoint-manager.js";
@@ -30,10 +30,9 @@ describe("Checkpoint Merge Bug Diagnostics", () => {
 
 	/**
 	 * Helper to read checkpoint data directly from file
+	 * @param path
 	 */
-	function readCheckpointFile(path: string): CheckpointData {
-		return JSON.parse(readFileSync(path, "utf-8"));
-	}
+	const readCheckpointFile = (path: string): CheckpointData => JSON.parse(readFileSync(path, "utf-8"));
 
 	afterEach(() => {
 		rmSync(testDir, { recursive: true, force: true });
@@ -160,29 +159,30 @@ describe("Checkpoint Merge Bug Diagnostics", () => {
 
 /**
  * Create a mock evaluation result for testing
+ * @param runId
+ * @param sut
+ * @param caseId
  */
-function createMockResult(runId: string, sut: string, caseId: string): EvaluationResult {
-	return {
-		run: {
-			runId,
-			sut,
-			sutRole: "primary" as const,
-			sutVersion: "1.0.0",
-			caseId,
-			caseClass: "test-class",
-			seed: 42,
-			repetition: 0,
-		},
-		correctness: {
-			expectedExists: false,
-			producedOutput: true,
-			valid: true,
-			matchesExpected: null,
-		},
-		outputs: { summary: {} },
-		metrics: { numeric: { test: 1 } },
-		provenance: {
-			runtime: { platform: "linux", arch: "x64", nodeVersion: "v22.0.0" },
-		},
-	};
-}
+const createMockResult = (runId: string, sut: string, caseId: string): EvaluationResult => ({
+	run: {
+		runId,
+		sut,
+		sutRole: "primary" as const,
+		sutVersion: "1.0.0",
+		caseId,
+		caseClass: "test-class",
+		seed: 42,
+		repetition: 0,
+	},
+	correctness: {
+		expectedExists: false,
+		producedOutput: true,
+		valid: true,
+		matchesExpected: null,
+	},
+	outputs: { summary: {} },
+	metrics: { numeric: { test: 1 } },
+	provenance: {
+		runtime: { platform: "linux", arch: "x64", nodeVersion: "v22.0.0" },
+	},
+});

src/experiments/framework/executor/__tests__/parallel-checkpoint-merge.integration.test.ts

@@ -9,12 +9,13 @@
  * 5. Main checkpoint should have all runs, but gets overwritten
  */
 
-import { describe, it, expect, beforeEach, afterEach } from "vitest";
-import { readFileSync, rmSync, mkdirSync } from "node:fs";
 import { randomBytes } from "node:crypto";
+import { mkdirSync,readFileSync, rmSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 
+import { afterEach,beforeEach, describe, expect, it } from "vitest";
+
 import type { EvaluationResult } from "../../types/result.js";
 import type { CheckpointData } from "../checkpoint-manager.js";
 import { CheckpointManager } from "../checkpoint-manager.js";
@@ -34,10 +35,9 @@ describe("Parallel Checkpoint Merge Integration Tests", () => {
 
 	/**
 	 * Helper to read checkpoint data directly from file
+	 * @param path
 	 */
-	function readCheckpointFile(path: string): CheckpointData {
-		return JSON.parse(readFileSync(path, "utf-8"));
-	}
+	const readCheckpointFile = (path: string): CheckpointData => JSON.parse(readFileSync(path, "utf-8"));
 
 	afterEach(() => {
 		rmSync(testDir, { recursive: true, force: true });
@@ -190,40 +190,42 @@ describe("Parallel Checkpoint Merge Integration Tests", () => {
 
 /**
  * Create a batch of mock evaluation results
+ * @param count
  */
-function createMockRunBatch(count: number): EvaluationResult[] {
+const createMockRunBatch = (count: number): EvaluationResult[] => {
 	const results: EvaluationResult[] = [];
-	for (let i = 0; i < count; i++) {
-		results.push(createMockResult(`run-${String(i).padStart(3, "0")}`, `sut-${i % 4}`, `case-${i}`));
+	for (let index = 0; index < count; index++) {
+		results.push(createMockResult(`run-${String(index).padStart(3, "0")}`, `sut-${index % 4}`, `case-${index}`));
 	}
 	return results;
-}
+};
 
 /**
  * Create a mock evaluation result
+ * @param runId
+ * @param sut
+ * @param caseId
  */
-function createMockResult(runId: string, sut: string, caseId: string): EvaluationResult {
-	return {
-		run: {
-			runId,
-			sut,
-			sutRole: "primary" as const,
-			sutVersion: "1.0.0",
-			caseId,
-			caseClass: "test-class",
-			seed: 42,
-			repetition: 0,
-		},
-		correctness: {
-			expectedExists: false,
-			producedOutput: true,
-			valid: true,
-			matchesExpected: null,
-		},
-		outputs: { summary: {} },
-		metrics: { numeric: { test: 1 } },
-		provenance: {
-			runtime: { platform: "linux", arch: "x64", nodeVersion: "v22.0.0" },
-		},
-	};
-}
+const createMockResult = (runId: string, sut: string, caseId: string): EvaluationResult => ({
+	run: {
+		runId,
+		sut,
+		sutRole: "primary" as const,
+		sutVersion: "1.0.0",
+		caseId,
+		caseClass: "test-class",
+		seed: 42,
+		repetition: 0,
+	},
+	correctness: {
+		expectedExists: false,
+		producedOutput: true,
+		valid: true,
+		matchesExpected: null,
+	},
+	outputs: { summary: {} },
+	metrics: { numeric: { test: 1 } },
+	provenance: {
+		runtime: { platform: "linux", arch: "x64", nodeVersion: "v22.0.0" },
+	},
+});