refactor(workflow): remove disabled prop from history view and add orderIndex to events

test(event-bus): add orderIndex to test event payloads
refactor(history): simplify navigation by removing disabled check
docs(steps): add debug logs for resume index calculation

Author: moazbuilds

src/cli/tui/routes/workflow/components/modals/history/index.tsx

@@ -17,7 +17,6 @@ import { AgentMonitorService } from "../../../../../../../agents/monitoring/inde
 export interface HistoryViewProps {
   onClose: () => void
   onOpenLogViewer: (monitoringId: number) => void
-  disabled?: boolean
   initialSelectedIndex?: number
   onSelectedIndexChange?: (index: number) => void
 }
@@ -63,9 +62,9 @@ export function HistoryView(props: HistoryViewProps) {
     onOpenLogViewer: props.onOpenLogViewer,
     onClearHistory: handleClearHistory,
     onScrollToIndex: handleScrollToIndex,
-    disabled: () => props.disabled ?? false,
   })
 
+
   // Sync selection when user scrolls manually with mouse
   createEffect(() => {
     const ref = scrollRef()

src/cli/tui/routes/workflow/components/modals/history/use-history-navigation.ts

@@ -16,7 +16,6 @@ export interface UseHistoryNavigationOptions {
   onOpenLogViewer: (monitoringId: number) => void
   onClearHistory?: () => void
   onScrollToIndex?: (index: number) => void
-  disabled?: () => boolean
 }
 
 export function useHistoryNavigation(options: UseHistoryNavigationOptions) {
@@ -60,8 +59,6 @@ export function useHistoryNavigation(options: UseHistoryNavigationOptions) {
   }
 
   useKeyboard((evt) => {
-    if (options.disabled?.()) return
-
     if (evt.name === "h" || evt.name === "escape") {
       evt.preventDefault()
       options.onClose()

src/cli/tui/routes/workflow/workflow-shell.tsx

@@ -396,7 +396,7 @@ export function WorkflowShell(props: WorkflowShellProps) {
 
       <Show when={modals.isHistoryActive()}>
         <box position="absolute" left={0} top={0} width="100%" height="100%" zIndex={1000} backgroundColor={themeCtx.theme.background}>
-          <HistoryView onClose={() => modals.setShowHistory(false)} onOpenLogViewer={(id) => { modals.setHistoryLogViewerMonitoringId(id); modals.setShowHistory(false) }} disabled={isCheckpointActive() || isWaitingForInput()} initialSelectedIndex={modals.historySelectedIndex()} onSelectedIndexChange={modals.setHistorySelectedIndex} />
+          <HistoryView onClose={() => modals.setShowHistory(false)} onOpenLogViewer={(id) => { modals.setHistoryLogViewerMonitoringId(id); modals.setShowHistory(false) }} initialSelectedIndex={modals.historySelectedIndex()} onSelectedIndexChange={modals.setHistorySelectedIndex} />
         </box>
       </Show>
 

src/shared/workflows/steps.ts

@@ -2,6 +2,7 @@ import { readFile, writeFile } from 'node:fs/promises';
 import { existsSync } from 'node:fs';
 import * as path from 'node:path';
 import type { StepData } from './template.js';
+import { debug } from '../logging/logger.js';
 
 const TEMPLATE_TRACKING_FILE = 'template.json';
 
@@ -301,20 +302,26 @@ export async function clearNotCompletedSteps(cmRoot: string): Promise<void> {
 export async function getResumeStartIndex(cmRoot: string): Promise<number> {
   const { data } = await readTrackingData(cmRoot);
 
+  debug('[getResumeStartIndex] data: %O', { resumeFromLastStep: data.resumeFromLastStep, notCompletedSteps: data.notCompletedSteps });
+
   // Check if resume feature is enabled
   if (!data.resumeFromLastStep) {
+    debug('[getResumeStartIndex] resumeFromLastStep is false, returning 0');
     return 0;
   }
 
   // First check for incomplete chains
   const chainResumeInfo = await getChainResumeInfo(cmRoot);
   if (chainResumeInfo) {
+    debug('[getResumeStartIndex] Found chain resume, returning %d', chainResumeInfo.stepIndex);
     return chainResumeInfo.stepIndex;
   }
 
   // Check notCompletedSteps (crash recovery)
   if (data.notCompletedSteps && data.notCompletedSteps.length > 0) {
-    return Math.min(...data.notCompletedSteps);
+    const startIndex = Math.min(...data.notCompletedSteps);
+    debug('[getResumeStartIndex] notCompletedSteps=%O, starting at %d', data.notCompletedSteps, startIndex);
+    return startIndex;
   }
 
   // Check completedSteps - if all steps done, start after the last one

src/workflows/events/__tests__/event-bus.test.ts

@@ -71,6 +71,7 @@ describe('WorkflowEventBus', () => {
           stepIndex: 0,
           totalSteps: 3,
           status: 'pending',
+          orderIndex: 0,
         },
       });
     });
@@ -201,6 +202,7 @@ describe('Integration: Event Flow', () => {
         stepIndex: 0,
         totalSteps: 3,
         status: 'pending',
+        orderIndex: 0,
       },
     });
 
@@ -213,6 +215,7 @@ describe('Integration: Event Flow', () => {
         stepIndex: 1,
         totalSteps: 3,
         status: 'pending',
+        orderIndex: 1,
       },
     });