refactor(tui): choose the idle colour from the running phase

The nested ternary tested the negated phase first, which tripped
unicorn(no-negated-condition) and made `textMuted` the answer at both
ends of the chain. A small `idleColor` helper states the rule the way
the panel means it: only a running row can stall.

Adds the case that had no coverage — a suspended row keeps its muted
count however long it stays silent — so the guard now fails a test when
it is removed.

Author: elkaix

apps/pythinker-code/src/tui/components/messages/dynamic-workflow-mission-control.ts

@@ -1046,18 +1046,27 @@ function renderWorkCell(member: DynamicWorkflowMember, nowMs: number): string {
   }
   const idleMs = Math.max(0, nowMs - member.lastEventAtMs);
   const idleSeconds = Math.floor(idleMs / 1000);
-  // Only a running row can stall. A suspended one is waiting on the user by
-  // design, so it keeps the count without the alarm colours.
-  const token = member.phase !== 'running'
-    ? 'textMuted'
-    : idleMs >= DYNAMIC_WORKFLOW_RENDERING.stalledIdleMs
-      ? 'error'
-      : idleMs >= DYNAMIC_WORKFLOW_RENDERING.quietIdleMs
-        ? 'warning'
-        : 'textMuted';
+  const token = idleColor(member.phase, idleMs);
   return `${tools} ${currentTheme.fg(token, `${String(idleSeconds)}s`.padStart(4, ' '))}`;
 }
 
+/**
+ * How loud an idle age reads.
+ *
+ * Only a running row can stall. A suspended one is waiting on the user by
+ * design, so it keeps the count without the alarm colours.
+ */
+function idleColor(
+  phase: DynamicWorkflowPhase,
+  idleMs: number,
+): 'textMuted' | 'warning' | 'error' {
+  if (phase === 'running') {
+    if (idleMs >= DYNAMIC_WORKFLOW_RENDERING.stalledIdleMs) return 'error';
+    if (idleMs >= DYNAMIC_WORKFLOW_RENDERING.quietIdleMs) return 'warning';
+  }
+  return 'textMuted';
+}
+
 /**
  * The STATE cell for one row.
  *

apps/pythinker-code/test/tui/components/messages/dynamic-workflow-mission-control.test.ts

@@ -665,6 +665,42 @@ describe('DynamicWorkflowMissionControlComponent', () => {
     }
   });
 
+  it('keeps a suspended row muted however long it stays silent', () => {
+    vi.useFakeTimers();
+    vi.setSystemTime(0);
+    const previousLevel = chalk.level;
+    const previousPalette = currentTheme.palette;
+    chalk.level = 3;
+    currentTheme.setPalette(darkColors);
+
+    try {
+      const component = createComponent();
+      component.updateArgs({ items: ['Held work'] });
+      component.markInputComplete();
+      component.registerSubagent({ agentId: 'agent-1' });
+      component.markStarted('agent-1');
+      // The last event lands a minute in, so the idle age and the elapsed age
+      // read as different numbers and the assertion cannot match the wrong cell.
+      vi.setSystemTime(60_000);
+      component.recordToolCall({ agentId: 'agent-1', name: 'Bash' });
+      component.markSuspended({ agentId: 'agent-1', reason: 'Waiting for approval' });
+
+      // A suspended agent waits on the user by design, so its silence is not a
+      // stall and must never borrow the alarm colours.
+      vi.setSystemTime(60_000 + DYNAMIC_WORKFLOW_RENDERING.stalledIdleMs * 2);
+      const line = component.render(100).find(
+        (candidate) => strip(candidate).replace(/^│\s*/u, '').startsWith('001'),
+      );
+      if (line === undefined) throw new Error('Missing Dynamic Workflow member 001');
+      expect(strip(line)).toContain('1⚒ 360s');
+      expect(line).toContain(chalk.hex(darkColors.textMuted)('360s'));
+    } finally {
+      vi.useRealTimers();
+      chalk.level = previousLevel;
+      currentTheme.setPalette(previousPalette);
+    }
+  });
+
   it('never marks a row that has not started as stalled', () => {
     vi.useFakeTimers();
     vi.setSystemTime(0);