Add hot-corners extension#27448
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 SummaryThis PR introduces a new
Confidence Score: 3/5Safe to merge after removing the unused One P1 finding (unused declared dependency violating the project rule) pulls the ceiling to 4/5; combined with two P2s and the P1 being straightforward to fix, a score of 3 reflects that the fix is required before merge.
Important Files Changed
Prompt To Fix All With AIThis is a comment left during a code review.
Path: extensions/hot-corners/package.json
Line: 43
Comment:
**Unused `@raycast/utils` dependency**
`@raycast/utils` is declared as a dependency but is never imported in any source file under `src/`. This adds unnecessary weight to the bundle and violates the project rule requiring every listed dependency to be used. Remove it from `dependencies`, or import the package if it is actually needed.
```suggestion
"@raycast/api": "^1.104.13"
```
**Rule Used:** What: Every dependency listed in package.json must... ([source](https://app.greptile.com/review/custom-context?memory=bffc60eb-f9f2-4219-b804-76e29e267d43))
How can I resolve this? If you propose a fix, please make it concise.
---
This is a comment left during a code review.
Path: extensions/hot-corners/src/toggle-hot-corners.tsx
Line: 30-70
Comment:
**Missing error handler in `try/finally`**
The outer `try` block has only a `finally` — no `catch`. If `applyHotCornerSettings` (or any other synchronous call) throws, the error propagates out of the exported function without any user-visible feedback. The `finally` still runs `syncToggleCommandSubtitle`, but the failure itself is never surfaced through a `showToast` or `showHUD`. Wrapping the body in a `try/catch/finally` ensures users see a meaningful error message rather than a silent failure or a generic Raycast error dialog.
How can I resolve this? If you propose a fix, please make it concise.
---
This is a comment left during a code review.
Path: extensions/hot-corners/src/list-presets.tsx
Line: 101
Comment:
**Action title casing inconsistency**
`"Move up"` uses sentence case while the sibling `"Move Down"` uses title case. Raycast action titles should consistently use title case.
```suggestion
title="Move Up"
```
How can I resolve this? If you propose a fix, please make it concise.Reviews (1): Last reviewed commit: "feat: add Hot Corners Raycast extension" | Re-trigger Greptile |
Description
Control macOS hot corners from Raycast: save named layouts, switch between them, or turn all corners off and back on. Changes use the same settings as System Settings and apply the Dock preferences (the Dock is restarted so updates take effect).
Commands
How to use
Platform: macOS only.
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder