Skip to content

Handle terminal cwd errors with fallback warning UI#86

Merged
BunsDev merged 1 commit into
mainfrom
okcode/terminal-error-ui
Mar 28, 2026
Merged

Handle terminal cwd errors with fallback warning UI#86
BunsDev merged 1 commit into
mainfrom
okcode/terminal-error-ui

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented Mar 28, 2026

Summary

  • Fall back to the user home directory when a terminal cwd is invalid during open/restart, instead of failing the session.
  • Preserve the resolved fallback cwd across session startup and restart paths so the terminal launches consistently.
  • Surface plain error messages from the server and render terminal warnings with a highlighted warning style in the drawer.

Testing

  • Not run (per request).
  • Verified the diff updates the terminal manager open/restart flows to retry with os.homedir() on cwd validation failure.
  • Verified WebSocket error serialization now returns Error.message when available, which feeds the warning UI path.
  • Verified the terminal drawer system message styling now renders warning text with a distinct color/icon treatment.

- fall back to the home directory when terminal cwd validation fails
- surface plain error messages from the server
- render terminal system messages as a warning line
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 28, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Mar 28, 2026 7:32pm

@github-actions github-actions Bot added size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Mar 28, 2026
@BunsDev BunsDev merged commit 142ebf1 into main Mar 28, 2026
8 of 9 checks passed
@BunsDev BunsDev deleted the okcode/terminal-error-ui branch March 31, 2026 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant