fix: address review feedback on TUI signature design

Author: elkaix

apps/pythinker-code/src/tui/components/chrome/status-bar.ts

@@ -17,6 +17,7 @@ export type StatusBarStatus = Pick<
   | 'homeDir'
   | 'permissionMode'
   | 'planMode'
+  | 'fastMode'
   | 'dynamicWorkflowMode'
 > & { readonly sessionKey: string };
 
@@ -82,7 +83,8 @@ function renderModesChip(status: StatusBarStatus): string | undefined {
   if (status.planMode) modes.push(currentTheme.fg('modePlan', 'plan'));
   if (status.permissionMode === 'auto') modes.push(currentTheme.fg('modePermission', 'auto'));
   if (status.permissionMode === 'yolo') modes.push(currentTheme.fg('modeAutoAccept', 'yolo'));
-  if (status.dynamicWorkflowMode) modes.push(currentTheme.fg('modeFast', 'workflow'));
+  if (status.fastMode) modes.push(currentTheme.fg('modeFast', '↯ fast'));
+  if (status.dynamicWorkflowMode) modes.push(currentTheme.fg('accent', 'workflow'));
   return modes.length === 0 ? undefined : chip(modes.join(' '));
 }
 

apps/pythinker-code/test/tui/components/messages/tool-call.test.ts

@@ -63,26 +63,22 @@ describe('ToolCallComponent', () => {
 
     try {
       const pending = component.render(40);
+      const pendingBody = pending.slice(1);
       expect(pending[0]).not.toContain('\u001B[48;2;29;33;41m');
-      expect(
-        pending.slice(1).every((line) => line.includes('\u001B[48;2;29;33;41m')),
-      ).toBe(true);
+      expect(pendingBody.length).toBeGreaterThan(0);
+      expect(pendingBody.every((line) => line.includes('\u001B[48;2;29;33;41m'))).toBe(true);
 
       component.setResult({ tool_call_id: 'call_tint', output: 'content', is_error: false });
-      expect(
-        component
-          .render(40)
-          .slice(1)
-          .every((line) => line.includes('\u001B[48;2;20;23;27m')),
-      ).toBe(true);
+      const success = component.render(40);
+      const successBody = success.slice(1);
+      expect(successBody.length).toBeGreaterThan(0);
+      expect(successBody.every((line) => line.includes('\u001B[48;2;20;23;27m'))).toBe(true);
 
       component.setResult({ tool_call_id: 'call_tint', output: 'failed', is_error: true });
-      expect(
-        component
-          .render(40)
-          .slice(1)
-          .every((line) => line.includes('\u001B[48;2;41;29;29m')),
-      ).toBe(true);
+      const error = component.render(40);
+      const errorBody = error.slice(1);
+      expect(errorBody.length).toBeGreaterThan(0);
+      expect(errorBody.every((line) => line.includes('\u001B[48;2;41;29;29m'))).toBe(true);
     } finally {
       chalk.level = previousLevel;
     }

apps/pythinker-code/test/tui/components/status-bar.test.ts

@@ -20,6 +20,7 @@ function status(overrides: Partial<StatusBarStatus> = {}): StatusBarStatus {
     homeDir: '/Users/test',
     permissionMode: 'auto',
     planMode: true,
+    fastMode: false,
     dynamicWorkflowMode: true,
     sessionKey: 'session-alpha',
     ...overrides,
@@ -61,7 +62,22 @@ describe('StatusBarComponent', () => {
     component.update(status());
 
     for (const width of [0, 1, 10, 25, 45, 53, 80]) {
-      expect(visibleWidth(component.render(width)[0] ?? '')).toBeLessThanOrEqual(width);
+      const lines = component.render(width);
+      expect(lines).toHaveLength(1);
+      expect(visibleWidth(lines[0]!)).toBeLessThanOrEqual(width);
+    }
+  });
+
+  it('renders fast mode', () => {
+    const previousLevel = chalk.level;
+    chalk.level = 3;
+    const component = new StatusBarComponent();
+    component.update(status({ fastMode: true }));
+
+    try {
+      expect(stripAnsi(component.render(80)[0] ?? '')).toContain('↯ fast');
+    } finally {
+      chalk.level = previousLevel;
     }
   });
 });