Repository navigation
Update r2-uploader extension - #32108
Conversation
- Merge branch \'contributions/merge-1791534111715\' - Pull contributions - Merge branch \'contributions/merge-1791533640227\' - Pull contributions - Fix Raycast extension author - Address Greptile review findings on the Browse/Upload PR - Recenter screenshots using the window\'s true opaque bounding box - Fix asymmetric padding in Browse R2 Files screenshots
|
Thank you for the update! 🎉 We're currently experiencing a high volume of incoming requests. As a result, the initial review may take up to 15 business days. |
|
- Update extension screenshots - Add WebP conversion support
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 3adf5bf. Configure here.
|
Confirmed. |
|
Published to the Raycast Store: |
|
🎉 🎉 🎉 We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag. |
| @@ -1,5 +1,12 @@ | |||
| # R2 Uploader Changelog | |||
|
|
|||
| ## [WebP Conversion and Metadata Updates] - 2026-10-10 | |||
There was a problem hiding this comment.
The new CHANGELOG.md heading replaces {PR_MERGE_DATE} with 2026-10-10. The repository requires the placeholder so the release gets its actual merge date. Restore {PR_MERGE_DATE} before merging.
| ## [WebP Conversion and Metadata Updates] - 2026-10-10 | |
| ## [WebP Conversion and Metadata Updates] - {PR_MERGE_DATE} |
Rule Used: What: Changelog entries must use {PR_MERGE_DATE} placeholder in titles, be placed at the top of the file, and maintain descending version order. Why: Standardized placeholders and consistent ordering ensure changelogs are maintainable and merge da... (source)
Prompt To Fix With AI
This is a comment left during a code review.
Path: extensions/r2-uploader/CHANGELOG.md
Line: 3
Comment:
**Release date is fixed early**
The new `CHANGELOG.md` heading replaces `{PR_MERGE_DATE}` with `2026-10-10`. The repository requires the placeholder so the release gets its actual merge date. Restore `{PR_MERGE_DATE}` before merging.
```suggestion
## [WebP Conversion and Metadata Updates] - {PR_MERGE_DATE}
```
**Rule Used:** What: Changelog entries must use `{PR_MERGE_DATE}` placeholder in titles, be placed at the top of the file, and maintain descending version order. Why: Standardized placeholders and consistent ordering ensure changelogs are maintainable and merge da... ([source](https://app.greptile.com/raycast/-/custom-context?memory=c2214c11-df56-490a-b1c0-09a385df481a))
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.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!

Description
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolderNote
Medium Risk
Upload now hard-fails without a configured public domain (behavior change for existing users), and image conversion shells out to external binaries.
Overview
Adds optional WebP conversion before upload via
cwebp(quality and encoder path preferences), with a guard so AVIF and WebP cannot both be enabled. WebP output is written to a temp directory and removed after upload.R2 Public Domain is now mandatory for the upload command: if the renamed preference (still
customDomaininternally) is empty, the command fails with a toast instead of falling back to the raw R2 endpoint URL. Docs and manifest copy reflect AVIF/WebP and the public-domain requirement.Minor UX/metadata: browse root drops the
"R2 Bucket"navigation title; extension author updates toypp;CLAUDE.mdremoved; changelog and README updated.Reviewed by Cursor Bugbot for commit 270876c. Bugbot is set up for automated code reviews on this repo. Configure here.