-
Notifications
You must be signed in to change notification settings - Fork 5.4k
Add kitty extension #25389
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
base: main
Are you sure you want to change the base?
Add kitty extension #25389
Conversation
|
Congratulations on your new Raycast extension! 🚀 We're currently experiencing a high volume of incoming requests. As a result, the initial review may take up to 10-15 business days. Once the PR is approved and merged, the extension will be available on our Store. |
Greptile OverviewGreptile SummaryThis PR adds a new Kitty terminal extension that provides remote control capabilities via Kitty's Unix socket protocol. The extension includes commands to create windows/tabs, search tabs, manage launch configurations, and open Finder folders in Kitty. Key changes:
Issues found:
The code quality is high with proper error handling, TypeScript types, and user-friendly setup instructions. The extension follows Raycast conventions and includes appropriate fallbacks for various error states. Confidence Score: 3/5
Important Files Changed
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
17 files reviewed, 2 comments
extensions/kitty/CHANGELOG.md
Outdated
| @@ -0,0 +1,9 @@ | |||
| # Kitty Raycast Extension Changelog | |||
|
|
|||
| ## [Initial Version] - 2026-02-12 | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Date should use {PR_MERGE_DATE} placeholder for initial version
| ## [Initial Version] - 2026-02-12 | |
| ## [Initial Version] - {PR_MERGE_DATE} |
Context Used: Rule from dashboard - What: Changelog entries must use {PR_MERGE_DATE} placeholder in titles, be placed at the top of th... (source)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
| { | ||
| "$schema": "https://www.raycast.com/schemas/extension.json", | ||
| "name": "kitty", | ||
| "title": "Kitty", | ||
| "description": "Control Kitty terminal via its remote control protocol", | ||
| "icon": "extension-icon.png", | ||
| "author": "jerome_soyer", | ||
| "categories": ["Applications", "Developer Tools"], | ||
| "license": "MIT", | ||
| "preferences": [ | ||
| { | ||
| "name": "socketPath", | ||
| "title": "Socket Path", | ||
| "description": "Path to the Kitty remote control socket. Leave empty to auto-detect /tmp/kitty-socket-{pid}", | ||
| "type": "textfield", | ||
| "required": false, | ||
| "default": "" | ||
| } | ||
| ], | ||
| "commands": [ | ||
| { | ||
| "name": "new-kitty-window", | ||
| "title": "New Kitty Window", | ||
| "subtitle": "Kitty", | ||
| "description": "Open a new Kitty OS window", | ||
| "mode": "no-view" | ||
| }, | ||
| { | ||
| "name": "new-kitty-tab", | ||
| "title": "New Kitty Tab", | ||
| "subtitle": "Kitty", | ||
| "description": "Open a new tab in the active Kitty window", | ||
| "mode": "no-view" | ||
| }, | ||
| { | ||
| "name": "search-kitty-tabs", | ||
| "title": "Search Kitty Tabs", | ||
| "subtitle": "Kitty", | ||
| "description": "Search and focus Kitty tabs across all windows", | ||
| "mode": "view" | ||
| }, | ||
| { | ||
| "name": "open-kitty-launch-config", | ||
| "title": "Open Kitty Launch Configuration", | ||
| "subtitle": "Kitty", | ||
| "description": "Manage and launch multi-tab/split configurations", | ||
| "mode": "view" | ||
| }, | ||
| { | ||
| "name": "open-with-kitty", | ||
| "title": "Open with Kitty", | ||
| "subtitle": "Kitty", | ||
| "description": "Open the selected Finder folder in Kitty", | ||
| "mode": "no-view" | ||
| } | ||
| ], | ||
| "dependencies": { | ||
| "@raycast/api": "^1.89.0", | ||
| "@raycast/utils": "^1.17.0", | ||
| "yaml": "^2.7.0" | ||
| }, | ||
| "devDependencies": { | ||
| "@raycast/eslint-config": "^1.0.11", | ||
| "@types/node": "22.13.4", | ||
| "@types/react": "19.0.8", | ||
| "eslint": "^8.57.0", | ||
| "prettier": "^3.4.2", | ||
| "typescript": "^5.7.3" | ||
| }, | ||
| "scripts": { | ||
| "build": "ray build", | ||
| "dev": "ray develop", | ||
| "fix-lint": "ray lint --fix", | ||
| "lint": "ray lint", | ||
| "prepublishOnly": "echo \"Error: no publish\" && exit 1", | ||
| "publish": "npx @raycast/api@latest publish" | ||
| } | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing metadata/ folder with screenshots - this extension has view-type commands (search-kitty-tabs and open-kitty-launch-config) and requires Raycast-styled screenshots in a metadata folder per the extension guidelines.
Context Used: Rule from dashboard - What: Extensions with view-type commands must include a metadata/ folder containing Raycast-styled... (source)
- Resize metadata screenshots to 2000x1250 - Add metadata screenshots and fix CHANGELOG date format
Description
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder