Repository navigation
feat(desktop): add Ghostty terminal support - #950
Conversation
Add Ghostty as an option in the "Open in" menu alongside iTerm, Warp, and Terminal. Logo from https://github.com/ghostty-org/website/blob/28f1a1f/public/ghostty-logo.svg
📝 WalkthroughWalkthroughThis PR adds Ghostty as a new external application option for the "Open In" functionality. Changes include adding Ghostty to the APP_NAMES mapping, EXTERNAL_APPS constant array, OpenInButton component options, and corresponding test coverage. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR adds Ghostty terminal emulator as a new option in the "Open in" menu, allowing users to open workspaces and files directly in Ghostty alongside existing terminal options (iTerm, Warp, Terminal).
Changes:
- Added "ghostty" to the schema's EXTERNAL_APPS array
- Integrated Ghostty icon and menu option in the OpenInButton UI component
- Added Ghostty to backend helper mappings for macOS app opening
- Included unit test coverage for the new terminal option
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/local-db/src/schema/zod.ts | Added "ghostty" to EXTERNAL_APPS schema array |
| apps/desktop/src/renderer/components/OpenInButton/OpenInButton.tsx | Added Ghostty icon import and option to APP_OPTIONS array |
| apps/desktop/src/renderer/assets/app-icons/ghostty.svg | Added Ghostty terminal logo SVG asset |
| apps/desktop/src/lib/trpc/routers/external/helpers.ts | Mapped "ghostty" to "Ghostty" macOS application name |
| apps/desktop/src/lib/trpc/routers/external/helpers.test.ts | Added unit test for Ghostty terminal integration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Kitenite
left a comment
There was a problem hiding this comment.
This works, thanks @daohoangson !
|
Thanks for the quick merge! |
Description
Add Ghostty as an option in the "Open in" menu alongside iTerm, Warp, and Terminal.
Type of Change
Testing
Screenshots (if applicable)
Additional Notes
I downloaded the logo from https://github.com/ghostty-org/website/blob/28f1a1f/public/ghostty-logo.svg. Hope it's OK.
Summary by CodeRabbit
New Features
Tests
✏️ Tip: You can customize this high-level summary in your review settings.