Summary
TerminalPassthrough.tsx:82-98 — If getSessionMessages fails (network error), the error is silently swallowed — the .then() chain has no .catch(). loadingMessages is set to false, but the user sees no error indication and the transcript appears empty.
Files Affected
dashboard/src/components/session/TerminalPassthrough.tsx:82-98
Suggested Fix
Add a .catch() handler that sets an error state, similar to how TranscriptViewer handles it (line 58-59).
Code review finding from full codebase audit (2026-04-04)
Summary
TerminalPassthrough.tsx:82-98— IfgetSessionMessagesfails (network error), the error is silently swallowed — the.then()chain has no.catch().loadingMessagesis set to false, but the user sees no error indication and the transcript appears empty.Files Affected
dashboard/src/components/session/TerminalPassthrough.tsx:82-98Suggested Fix
Add a
.catch()handler that sets an error state, similar to howTranscriptViewerhandles it (line 58-59).Code review finding from full codebase audit (2026-04-04)