chore: remove unused exported types (3 exports)

[sjnims]

Description

Dead code analysis (knip) identified 3 exported types that are defined and re-exported but never imported anywhere in the codebase.

Findings

Export	Type	Location	Notes
ValidatedEvalConfig	type	src/config/schema.ts:321	Zod-inferred config type, re-exported from config/index.ts
CreateRunOptions	interface	src/state/types.ts:75	State creation options, re-exported from state/index.ts
ResumeOptions	interface	src/state/types.ts:84	State resume options, re-exported from state/index.ts

Note: The ResumeOptions in src/cli/options.ts:23 is a different interface with the same name and IS used by the CLI resume command.

Verification

# Confirm these are unused
npm run knip

Suggested Fix

1. Remove ValidatedEvalConfig export from src/config/schema.ts (or keep if useful for external consumers)
2. Remove CreateRunOptions interface from src/state/types.ts
3. Remove ResumeOptions interface from src/state/types.ts
4. Remove corresponding re-exports from barrel files (config/index.ts, state/index.ts)

Priority

Low - Dead code cleanup, no functional impact.

Related

• chore: remove unused E2E test helpers (13 exports) #275 (E2E helpers cleanup) ✅ Merged
• chore: remove unused detection and utility functions #276 (detection/utils cleanup) ✅ Merged
• chore: investigate Stage 3/4 barrel file re-exports for public API decision #277 (Stage 3/4 exports) - PR chore: remove unused Stage 3 execution exports #280

---

🤖 Created with Claude Code

[claude]

Claude finished @sjnims's task in 2m 15s —— View job • claude/issue-281-20260118-2216 • Create PR ➔

---

Cleaning up unused exported types

Successfully removed all 3 unused exported types identified by knip.

Tasks Completed

• Install dependencies and verify findings with knip
• Read affected files to understand current exports
• Remove ValidatedEvalConfig from src/config/schema.ts and src/config/index.ts
• Remove CreateRunOptions from src/state/types.ts and src/state/index.ts
• Remove ResumeOptions from src/state/types.ts and src/state/index.ts
• Commit and push changes

Summary

Removed the following unused exports:

1. ValidatedEvalConfig (type) - Removed from:

   • src/config/schema.ts:321 (type definition)
   • src/config/index.ts:24 (re-export)

2. CreateRunOptions (interface) - Removed from:

   • src/state/types.ts:75-79 (interface definition)
   • src/state/index.ts:23 (re-export)

3. ResumeOptions (interface) - Removed from:

   • src/state/types.ts:84-87 (interface definition)
   • src/state/index.ts:26 (re-export)

The CLI's ResumeOptions interface in src/cli/options.ts:23 was left untouched as it's a different interface that is actively used.

Changes

Total lines removed: 28 (type definitions + re-exports)

---

Branch: claude/issue-281-20260118-2216 |

[sjnims]

PR submitted: #283