-
Notifications
You must be signed in to change notification settings - Fork 9.9k
fix(app): terminal improvements - focus, rename, error state, CSP #9700
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(app): terminal improvements - focus, rename, error state, CSP #9700
Conversation
|
The following comment was made by an LLM, it may be inaccurate: Based on my search, I found 2 potentially related PRs that address similar issues:
These appear to be related to the CSP/WASM aspect of your PR. Check if these have already been merged or are still open, as your PR might supersede or complement them. |
|
@unitythemaker could you share some gifs/screenshots? |
Is this helpful? Edit: I recorded again. |
|
@unitythemaker please rebase and resolve conflicts and I'll get this merged! |
Sure! |
… state - Close terminal tabs automatically when shell exits - Fix freeze when closing middle tabs - Auto-close terminal panel when last tab is closed - Fix keyboard focus going to terminal instead of chat - Add terminal error state UI showing 'Connection Lost' message - Add terminal rename via double-click or context menu - Add OPENCODE_TERMINAL=1 env var for PTY sessions - Update CSP to allow wasm-unsafe-eval for ghostty terminal
18f4f8f to
583c259
Compare
|
Conflicts resolved! |
* dev: (345 commits) fix after merge tweak(session title): change prompt to have the response with user language (anomalyco#9847) fix(app): new workspace button with all languages (anomalyco#9848) chore: generate fix(app): add aria-labels to titlebar and sidebar buttons (anomalyco#9843) test(app): fix e2e wip: black fix: type error fix(app): breaking out of auto-scroll chore: generate feat(app): model tooltip metadata in chooser (per Figma request) (anomalyco#9707) chore: generate fix(app): workspaces padding wonkiness (anomalyco#9772) feat(desktop): properly integrate window controls on windows (anomalyco#9835) test(app): fix e2e fix(app): terminal improvements - focus, rename, error state, CSP (anomalyco#9700) fix(app): japanese language support fix(desktop): add workaround for nushell test(app): fix e2e fix(core): snapshot regression ...
Summary
wasm-unsafe-evaltoscript-srcanddata:toconnect-srcfor ghostty WASMnext()andprevious()methods to terminal contextIssues Fixed
exitcommand #6004 - Terminal freezes on shell exit (partially - server-side fix was in fix(app): terminal no longer hangs on exit or ctrl + D and closes the pane #9506)Files Changed
packages/app/src/components/session/session-sortable-terminal-tab.tsx- Rename functionalitypackages/app/src/components/terminal.tsx- Simplified WebSocket close handlingpackages/app/src/context/terminal.tsx- Added error field, improved close() logic, navigationpackages/app/src/pages/session.tsx- Terminal focus effect, error UI, disabled chat autofocuspackages/opencode/src/pty/index.ts- Added OPENCODE_TERMINAL=1 env varpackages/opencode/src/server/server.ts- Updated CSP for WASMTesting
Tested manually with
bun devfor both backend and frontend.