Add tablexport extension#27468
Conversation
- fixing errors, adding images and gif - adding new files - fixing icon - add icons - first commit
|
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 adds a new
Additionally, a Confidence Score: 3/5Not safe to merge — the core clipboard-to-website handoff is broken and the README has a broken image. Two P1 issues: clipboard content is read but never forwarded to the URL, and the README demo image is broken. These affect both functionality and the store listing presentation. extensions/tablexport/src/convert.ts (clipboard data discarded) and extensions/tablexport/README.md (broken image path) Important Files Changed
Prompt To Fix All With AIThis is a comment left during a code review.
Path: extensions/tablexport/src/convert.ts
Line: 4-8
Comment:
**Clipboard data read but never sent to the website**
The extension reads the clipboard text and checks it's non-empty, but then discards it entirely — only `autoPaste=true` is passed in the URL. The website would have to re-request clipboard access via the browser's Clipboard API, which prompts the user again and may be denied. If the intent is for the website to receive the data, it should be URL-encoded and appended to the query string. If the website really does read the clipboard itself, the `Clipboard.readText()` call here is redundant (and you'd want to show a HUD or error when text is absent).
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/tablexport/README.md
Line: 2
Comment:
**Broken image reference in README**
The README links to `media/raycast_extension.gif`, but the actual file committed to the repository is `media/raycast_gif.gif`. This image will appear broken in the Raycast store listing.
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/tablexport/package.json
Line: 1-3
Comment:
**Missing `$schema` field**
The `package.json` is missing the required `$schema` reference, which enables autocomplete, validation, and IntelliSense for Raycast extension fields in your editor.
**Rule Used:** What: Require Raycast extension projects to includ... ([source](https://app.greptile.com/review/custom-context?memory=9c275ba8-ead2-4952-b909-12275352b16b))
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/tablexport/package.json
Line: 41-43
Comment:
**`@raycast/utils` listed as a dev dependency but never imported**
`@raycast/utils` appears in `devDependencies` but is not imported in any source file under `src/`. Unused dependencies bloat installations and create confusion. Remove it if it's not needed.
**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.Reviews (1): Last reviewed commit: "Add tablexport extension" | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| const text = await Clipboard.readText(); | ||
| if (text) { | ||
| await showHUD("Opening in TableXport..."); | ||
| await open("https://tablexport.com?autoPaste=true"); | ||
| } |
There was a problem hiding this comment.
The extension reads the clipboard text and checks it's non-empty, but then discards it entirely — only autoPaste=true is passed in the URL. The website would have to re-request clipboard access via the browser's Clipboard API, which prompts the user again and may be denied. If the intent is for the website to receive the data, it should be URL-encoded and appended to the query string. If the website really does read the clipboard itself, the Clipboard.readText() call here is redundant (and you'd want to show a HUD or error when text is absent).
Prompt To Fix With AI
This is a comment left during a code review.
Path: extensions/tablexport/src/convert.ts
Line: 4-8
Comment:
**Clipboard data read but never sent to the website**
The extension reads the clipboard text and checks it's non-empty, but then discards it entirely — only `autoPaste=true` is passed in the URL. The website would have to re-request clipboard access via the browser's Clipboard API, which prompts the user again and may be denied. If the intent is for the website to receive the data, it should be URL-encoded and appended to the query string. If the website really does read the clipboard itself, the `Clipboard.readText()` call here is redundant (and you'd want to show a HUD or error when text is absent).
How can I resolve this? If you propose a fix, please make it concise.| @@ -0,0 +1,23 @@ | |||
| # TableXport | |||
|  | |||
There was a problem hiding this comment.
The README links to media/raycast_extension.gif, but the actual file committed to the repository is media/raycast_gif.gif. This image will appear broken in the Raycast store listing.
Prompt To Fix With AI
This is a comment left during a code review.
Path: extensions/tablexport/README.md
Line: 2
Comment:
**Broken image reference in README**
The README links to `media/raycast_extension.gif`, but the actual file committed to the repository is `media/raycast_gif.gif`. This image will appear broken in the Raycast store listing.
How can I resolve this? If you propose a fix, please make it concise.| { | ||
| "name": "tablexport", | ||
| "title": "TableXport", |
There was a problem hiding this comment.
$schema field
The package.json is missing the required $schema reference, which enables autocomplete, validation, and IntelliSense for Raycast extension fields in your editor.
Rule Used: What: Require Raycast extension projects to includ... (source)
Prompt To Fix With AI
This is a comment left during a code review.
Path: extensions/tablexport/package.json
Line: 1-3
Comment:
**Missing `$schema` field**
The `package.json` is missing the required `$schema` reference, which enables autocomplete, validation, and IntelliSense for Raycast extension fields in your editor.
**Rule Used:** What: Require Raycast extension projects to includ... ([source](https://app.greptile.com/review/custom-context?memory=9c275ba8-ead2-4952-b909-12275352b16b))
How can I resolve this? If you propose a fix, please make it concise.| "@types/node": "20.8.10", | ||
| "@types/react": "18.2.27", | ||
| "typescript": "^5.2.2" |
There was a problem hiding this comment.
@raycast/utils listed as a dev dependency but never imported
@raycast/utils appears in devDependencies but is not imported in any source file under src/. Unused dependencies bloat installations and create confusion. Remove it if it's not needed.
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/tablexport/package.json
Line: 41-43
Comment:
**`@raycast/utils` listed as a dev dependency but never imported**
`@raycast/utils` appears in `devDependencies` but is not imported in any source file under `src/`. Unused dependencies bloat installations and create confusion. Remove it if it's not needed.
**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.Refactor command function to main and improve clipboard check.
Updated the changelog title and simplified the initial version description.
remove misplaced changelog
Initial version of TableXport to convert tables from clipboard to various formats.
2 participants
Description
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder