refactor: clean up unused imports and type references

- remove unused ProgressState, getOpenCodeSettingsPath, ResumeDecision, and TemplateTracking types
- rename parseSimpleToml to _parseSimpleToml to indicate internal use
- fix optional chaining in signal handlers and type assertions

Author: moazbuilds

src/cli/tui/routes/onboard/onboard-view.tsx

@@ -392,7 +392,7 @@ export function OnboardView(props: OnboardViewProps) {
       >
         <Show when={currentStep() === 'launching' && props.service && props.eventBus}>
           <LaunchingView
-            controllerName={props.service!.getSelectedController()?.name ?? 'controller'}
+            controllerName={(props.service!.getSelectedController()?.name as string) ?? 'controller'}
             eventBus={props.eventBus!}
             service={props.service!}
           />

src/infra/engines/providers/opencode/mcp.ts

@@ -10,7 +10,6 @@ import {
   configureOpenCodeMCP,
   removeOpenCodeMCP,
   isOpenCodeMCPConfigured,
-  getOpenCodeSettingsPath,
 } from '../../../mcp/index.js';
 
 /**

src/infra/mcp/setup.ts

@@ -189,7 +189,7 @@ export function getCodexSettingsPath(
 /**
  * Parse simple TOML (enough for our config.toml needs)
  */
-function parseSimpleToml(content: string): string {
+function _parseSimpleToml(content: string): string {
   // Just return the content, we'll append to it
   return content;
 }

src/workflows/events/emitter.ts

@@ -15,7 +15,6 @@ import type {
   AgentInfo,
   SeparatorInfo,
   WorkflowEvent,
-  ProgressState,
 } from './types.js';
 import type {
   AgentStatus,

src/workflows/indexing/debug.ts

@@ -6,7 +6,7 @@
  */
 
 import { debug as baseDebug } from '../../shared/logging/logger.js';
-import type { StepData, ResumeInfo, StepLifecyclePhase, ResumeDecision } from './types.js';
+import type { StepData, ResumeInfo, StepLifecyclePhase } from './types.js';
 
 /**
  * Log a step lifecycle event

src/workflows/indexing/manager.ts

@@ -5,7 +5,7 @@
  * Manages both in-memory state and persistence to template.json.
  */
 
-import type { StepData, TemplateTracking, ResumeInfo } from './types.js';
+import type { StepData, ResumeInfo } from './types.js';
 import type { QueuedPrompt } from '../state/types.js';
 import { ResumeDecision, StepLifecyclePhase } from './types.js';
 import { readTrackingData, writeTrackingData } from './persistence.js';

src/workflows/onboarding/service.ts

@@ -9,7 +9,7 @@
 import { debug } from '../../shared/logging/logger.js';
 import type { WorkflowEventBus } from '../events/event-bus.js';
 import type { OnboardConfig, OnboardResult, OnboardStep } from '../events/types.js';
-import type { TracksConfig, ConditionGroup, ChildConditionGroup } from '../templates/types.js';
+import type { TracksConfig, ConditionGroup } from '../templates/types.js';
 import type { AgentDefinition } from '../../shared/agents/config/types.js';
 import { OnboardingEmitter } from './emitter.js';
 import { initControllerAgent } from '../../shared/workflows/controller.js';
@@ -168,7 +168,7 @@ export class OnboardingService {
    */
   selectCondition(conditionId: string): void {
     const step = this.state.currentStep;
-    const isChild = step === 'condition_child';
+    const _isChild = step === 'condition_child';
     const groupIndex = this.state.currentGroupIndex;
 
     if (step === 'condition_group') {
@@ -256,7 +256,7 @@ export class OnboardingService {
       return;
     }
 
-    const controllerName = agent.name;
+    const controllerName = agent.name as string;
     debug('[OnboardingService] launching controller: "%s"', controllerName);
 
     // Emit launching started
@@ -380,7 +380,7 @@ export class OnboardingService {
       case 'controller':
         return (this.config.controllerAgents ?? []).map((a) => [
           a.id,
-          { label: a.name, description: a.description as string | undefined },
+          { label: a.name as string, description: a.description as string | undefined },
         ]);
       default:
         return [];

src/workflows/run.ts

@@ -17,7 +17,7 @@ import {
   getSelectedTrack,
   getSelectedConditions,
 } from '../shared/workflows/index.js';
-import { StepIndexManager, ResumeDecision } from './indexing/index.js';
+import { StepIndexManager } from './indexing/index.js';
 import { registry } from '../infra/engines/index.js';
 import { MonitoringCleanup } from '../agents/monitoring/index.js';
 import { WorkflowEventBus, WorkflowEventEmitter } from './events/index.js';

src/workflows/runner/delegated.ts

@@ -6,8 +6,6 @@
  */
 
 import { debug } from '../../shared/logging/logger.js';
-import { formatUserInput } from '../../shared/formatters/outputMarkers.js';
-import { AgentLoggerService } from '../../agents/monitoring/index.js';
 import type { InputContext } from '../input/index.js';
 import { getUniqueAgentId } from '../context/index.js';
 import { runStepResume } from '../step/run.js';

src/workflows/signals/handlers/pause.ts

@@ -54,7 +54,7 @@ export async function handlePauseSignal(ctx: SignalContext): Promise<void> {
     // Continue to pause logic below (don't return early)
   }
 
-  if (ctx.machine.state === 'running' || ctx.machine.state === 'delegated') {
+  if (ctx.machine.state === 'running') {
     // Update UI status
     ctx.emitter.updateAgentStatus(stepContext.agentId, 'awaiting');