Code quality: remove dead code, deduplicate padRight, clean exports (#148)

- Remove extractRunNumber (always returned -1, dead code) (#130)
- Import padRight from display.ts instead of duplicating in evaluate.ts (#132)
- Make CleanOptions, CompareOptions, StatsOptions non-exported (unused externally) (#133)
- Keep ApplyOptions exported (used in tests)

Closes #130
Closes #132
Closes #133

Co-authored-by: unknown <that-github-user@github.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 3 people

src/commands/clean.ts

@@ -6,7 +6,7 @@ import { cleanupBranches, getRepoRoot, removeWorktree } from "../utils/git.js";
 
 const exec = promisify(execFile);
 
-export interface CleanOptions {
+interface CleanOptions {
   all?: boolean;
 }
 

src/commands/compare.ts

@@ -4,7 +4,7 @@ import pc from "picocolors";
 import { diffSimilarity, parseDiff } from "../scoring/diff-parser.js";
 import type { AgentResult, EnsembleResult } from "../types.js";
 
-export interface CompareOptions {
+interface CompareOptions {
   agentA: number;
   agentB: number;
 }

src/commands/evaluate.ts

@@ -3,6 +3,7 @@ import { join } from "node:path";
 import pc from "picocolors";
 import { analyzeConvergence, copelandRecommend, recommend } from "../scoring/convergence.js";
 import type { EnsembleResult } from "../types.js";
+import { padRight } from "../utils/display.js";
 import { parseAndValidateResult } from "../utils/schema.js";
 
 interface RunEvaluation {
@@ -207,13 +208,6 @@ export async function evaluate(): Promise<void> {
   console.log();
 }
 
-function padRight(str: string, len: number): string {
-  // biome-ignore lint/suspicious/noControlCharactersInRegex: intentional ANSI escape sequence matching
-  const stripped = str.replace(/\x1b\[[0-9;]*m/g, "");
-  const padding = Math.max(0, len - stripped.length);
-  return str + " ".repeat(padding);
-}
-
 function pct(n: number, total: number): string {
   return `${n}/${total} (${Math.round((n / total) * 100)}%)`;
 }

src/commands/list.test.ts

@@ -1,14 +1,7 @@
 import assert from "node:assert/strict";
 import { describe, it } from "node:test";
 import type { EnsembleResult } from "../types.js";
-import { buildRunSummary, extractRunNumber } from "./list.js";
-
-describe("extractRunNumber", () => {
-  it("returns -1 (run numbers assigned by load order, not filename)", () => {
-    assert.equal(extractRunNumber("run-2026-03-28T05-58-48-564Z.json"), -1);
-    assert.equal(extractRunNumber("latest.json"), -1);
-  });
-});
+import { buildRunSummary } from "./list.js";
 
 function makeResult(overrides: Partial<EnsembleResult> = {}): EnsembleResult {
   return {

src/commands/list.ts

@@ -14,11 +14,6 @@ export interface RunSummary {
   avgConvergence: number | null;
 }
 
-/** Run numbers are assigned by sort order (chronological), not embedded in filename */
-export function extractRunNumber(_filename: string): number {
-  return -1; // Assigned by loadAllRuns based on sort position
-}
-
 export function buildRunSummary(runNumber: number, result: EnsembleResult): RunSummary {
   const testPassRate =
     result.tests.length > 0

src/commands/stats.ts

@@ -4,7 +4,7 @@ import pc from "picocolors";
 import type { EnsembleResult } from "../types.js";
 import { parseAndValidateResult } from "../utils/schema.js";
 
-export interface StatsOptions {
+interface StatsOptions {
   model?: string;
   since?: string;
   until?: string;