Add morse-code-reader extension#27476
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. |
|
I needed a morse code reader tool in my hand but I didn't found any extensions for this purpose. and it's not worthy to install a new application in my laptop so I built it. |
Greptile SummaryThis PR adds a new
Confidence Score: 3/5Not ready to merge — missing store metadata screenshots and has an unused dependency that needs to be removed Two P1s (unused dependency, missing metadata folder) and three P2s; the missing metadata/ folder is a blocking store requirement extensions/morse-code-reader/package.json (unused dependency + wrong category); extensions/morse-code-reader/ root (missing metadata/ folder with screenshots) Important Files Changed
Prompt To Fix All With AIThis is a comment left during a code review.
Path: extensions/morse-code-reader/package.json
Line: 25-26
Comment:
**Unused dependency: `@raycast/utils`**
`@raycast/utils` is declared as a dependency but is never imported in `src/morse-code-reader.tsx` (the only source file). This bloats the bundle unnecessarily.
```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/morse-code-reader/package.json
Line: 21-23
Comment:
**Missing `metadata/` folder with store screenshots**
The command uses `"mode": "view"`, so Raycast Store guidelines require a `metadata/` folder containing at least one Raycast-styled screenshot. Without it, the extension cannot be published to the store. See the [screenshots docs](https://developers.raycast.com/basics/prepare-an-extension-for-store#screenshots) for the required format.
**Rule Used:** What: Extensions with view-type commands must incl... ([source](https://app.greptile.com/review/custom-context?memory=87059ac1-c601-487f-9f1c-bce8a3cb6209))
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/morse-code-reader/package.json
Line: 12-14
Comment:
**Incorrect category for a Morse Code extension**
`"Security"` is reserved for extensions like password managers and secrets tools. A Morse Code translator fits `"Fun"` (novelty/entertainment tool) or possibly `"Developer Tools"`.
```suggestion
"categories": [
"Fun"
],
```
**Rule Used:** What: Assign at least one predefined category to e... ([source](https://app.greptile.com/review/custom-context?memory=f49debbf-b6f6-4c0d-9b35-e1927815992b))
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/morse-code-reader/src/morse-code-reader.tsx
Line: 155-160
Comment:
**Output field accepts user edits that are silently discarded**
The output `Form.TextArea` uses `onChange={() => {}}` to appear read-only, but users can still click into it and type — their input is just silently dropped. Consider using `Form.Description` for a truly non-interactive display, which is also the semantically correct component for showing computed output.
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/morse-code-reader/CHANGELOG.md
Line: 1-3
Comment:
**Informal changelog title and non-standard release label**
The file title `# Learning Changelog` appears to be a personal note rather than the standard format. The release label `[Initial Version]` also deviates from the conventional `[Initial Release]` used across extensions in this repo.
```suggestion
# Morse Code Reader Changelog
## [Initial Release] - {PR_MERGE_DATE}
```
How can I resolve this? If you propose a fix, please make it concise.Reviews (1): Last reviewed commit: "first commit" | Re-trigger Greptile |
| "@raycast/api": "^1.104.13", | ||
| "@raycast/utils": "^1.17.0" |
There was a problem hiding this comment.
Unused dependency:
@raycast/utils
@raycast/utils is declared as a dependency but is never imported in src/morse-code-reader.tsx (the only source file). This bloats the bundle unnecessarily.
| "@raycast/api": "^1.104.13", | |
| "@raycast/utils": "^1.17.0" | |
| "@raycast/api": "^1.104.13" |
Rule Used: What: Every dependency listed in package.json must... (source)
Prompt To Fix With AI
This is a comment left during a code review.
Path: extensions/morse-code-reader/package.json
Line: 25-26
Comment:
**Unused dependency: `@raycast/utils`**
`@raycast/utils` is declared as a dependency but is never imported in `src/morse-code-reader.tsx` (the only source file). This bloats the bundle unnecessarily.
```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.| "mode": "view" | ||
| } | ||
| ], |
There was a problem hiding this comment.
Missing
metadata/ folder with store screenshots
The command uses "mode": "view", so Raycast Store guidelines require a metadata/ folder containing at least one Raycast-styled screenshot. Without it, the extension cannot be published to the store. See the screenshots docs for the required format.
Rule Used: What: Extensions with view-type commands must incl... (source)
Prompt To Fix With AI
This is a comment left during a code review.
Path: extensions/morse-code-reader/package.json
Line: 21-23
Comment:
**Missing `metadata/` folder with store screenshots**
The command uses `"mode": "view"`, so Raycast Store guidelines require a `metadata/` folder containing at least one Raycast-styled screenshot. Without it, the extension cannot be published to the store. See the [screenshots docs](https://developers.raycast.com/basics/prepare-an-extension-for-store#screenshots) for the required format.
**Rule Used:** What: Extensions with view-type commands must incl... ([source](https://app.greptile.com/review/custom-context?memory=87059ac1-c601-487f-9f1c-bce8a3cb6209))
How can I resolve this? If you propose a fix, please make it concise.| "categories": [ | ||
| "Security" | ||
| ], |
There was a problem hiding this comment.
Incorrect category for a Morse Code extension
"Security" is reserved for extensions like password managers and secrets tools. A Morse Code translator fits "Fun" (novelty/entertainment tool) or possibly "Developer Tools".
| "categories": [ | |
| "Security" | |
| ], | |
| "categories": [ | |
| "Fun" | |
| ], |
Rule Used: What: Assign at least one predefined category to e... (source)
Prompt To Fix With AI
This is a comment left during a code review.
Path: extensions/morse-code-reader/package.json
Line: 12-14
Comment:
**Incorrect category for a Morse Code extension**
`"Security"` is reserved for extensions like password managers and secrets tools. A Morse Code translator fits `"Fun"` (novelty/entertainment tool) or possibly `"Developer Tools"`.
```suggestion
"categories": [
"Fun"
],
```
**Rule Used:** What: Assign at least one predefined category to e... ([source](https://app.greptile.com/review/custom-context?memory=f49debbf-b6f6-4c0d-9b35-e1927815992b))
How can I resolve this? If you propose a fix, please make it concise.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!
| <Form.TextArea | ||
| id="output" | ||
| title="Result" | ||
| value={result} | ||
| onChange={() => {}} // Read-only | ||
| /> |
There was a problem hiding this comment.
Output field accepts user edits that are silently discarded
The output Form.TextArea uses onChange={() => {}} to appear read-only, but users can still click into it and type — their input is just silently dropped. Consider using Form.Description for a truly non-interactive display, which is also the semantically correct component for showing computed output.
Prompt To Fix With AI
This is a comment left during a code review.
Path: extensions/morse-code-reader/src/morse-code-reader.tsx
Line: 155-160
Comment:
**Output field accepts user edits that are silently discarded**
The output `Form.TextArea` uses `onChange={() => {}}` to appear read-only, but users can still click into it and type — their input is just silently dropped. Consider using `Form.Description` for a truly non-interactive display, which is also the semantically correct component for showing computed output.
How can I resolve this? If you propose a fix, please make it concise.Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Description
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder