refactor(chat): improve command status determination and exit code parsing

- Refine `deriveCommandStatus` logic to better classify success, error, and running states
- Consolidate exit code regex into reusable constant for `stripTrailingExitCode` parsing

Author: tyulyukov

apps/web/src/components/chat/CommandExecutionCard.tsx

@@ -16,10 +16,12 @@ const PREVIEW_MAX_HEIGHT_PX = 120;
 const MIN_OVERFLOW_PX = 24;
 
 function deriveCommandStatus(entry: WorkLogEntry, isLive: boolean): CommandStatus {
-  if (!entry.toolCompleted && entry.exitCode === undefined) return "running";
-  if (isLive && entry.exitCode === undefined) return "running";
-  if (entry.tone === "error" || (entry.exitCode !== undefined && entry.exitCode !== 0))
-    return "error";
+  if (entry.toolCompleted || entry.exitCode !== undefined) {
+    if (entry.tone === "error" || (entry.exitCode !== undefined && entry.exitCode !== 0))
+      return "error";
+    return "success";
+  }
+  if (isLive) return "running";
   return "success";
 }
 

apps/web/src/session-logic.ts

@@ -1135,14 +1135,15 @@ function extractToolResultText(payload: Record<string, unknown> | null): string
   return null;
 }
 
+const EXIT_CODE_RE =
+  /^(?<output>[\s\S]*?)(?:\s*<?(?:exited with (?:exit )?code|Exited with code) (?<code>\d+)>?)\s*$/i;
+
 function stripTrailingExitCode(value: string): {
   output: string | null;
   exitCode?: number | undefined;
 } {
   const trimmed = value.trim();
-  const match = /^(?<output>[\s\S]*?)(?:\s*<exited with exit code (?<code>\d+)>)\s*$/i.exec(
-    trimmed,
-  );
+  const match = EXIT_CODE_RE.exec(trimmed);
   if (!match?.groups) {
     return {
       output: trimmed.length > 0 ? trimmed : null,