Skip to content

Update r2-uploader extension - #32108

Merged
raycastbot merged 7 commits into
raycast:mainfrom
mazaoshe:ext/r2-uploader
Oct 10, 2026
Merged

raycastbot merged 7 commits into
raycast:mainfrom
mazaoshe:ext/r2-uploader

Conversation

@mazaoshe

@mazaoshe mazaoshe commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Description

Screencast

Checklist


Note

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 customDomain internally) 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 to ypp; CLAUDE.md removed; changelog and README updated.

Reviewed by Cursor Bugbot for commit 270876c. Bugbot is set up for automated code reviews on this repo. Configure here.

- 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
@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: r2-uploader Issues related to the r2-uploader extension platform: macOS OP is author The OP of the PR is the author of the extension labels Oct 9, 2026
@raycastbot

raycastbot commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

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.

@greptile-apps

greptile-apps Bot commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 4/5

[Medium impact] Adds WebP image conversion and updates extension metadata.

Restore the required changelog date placeholder before merging.

Findings

  1. P2 Release date is fixed early ▶
Fix with agent prompt
### Issue 1
extensions/r2-uploader/CHANGELOG.md:3
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}
```

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!

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

This PR adds optional WebP conversion and updates the extension’s setup and Store metadata.

  • The uploader can convert supported images to WebP before sending them to R2.
  • The uploader requires an R2 Public Domain before it starts an upload.
  • The R2 file browser leaves its root navigation title unset.
  • Raycast credits the extension to ypp and shows refreshed screenshots.
  • Internal notes and lockfile records change without changing the extension's behavior.

Reviews (7) · Last reviewed commit: "Update CHANGELOG.md and optimise images" · Reviewed by Greptile

Comment thread extensions/r2-uploader/package.json
@0xdhrv
0xdhrv marked this pull request as ready for review October 9, 2026 23:35
@0xdhrv 0xdhrv self-assigned this Oct 9, 2026
@0xdhrv
0xdhrv marked this pull request as draft October 9, 2026 23:35
- Update extension screenshots
- Add WebP conversion support
Comment thread extensions/r2-uploader/src/utils/convert.ts Outdated
Comment thread extensions/r2-uploader/src/r2-uploader.ts Outdated

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread extensions/r2-uploader/src/r2-uploader.ts
Comment thread extensions/r2-uploader/src/r2-uploader.ts Outdated
Comment thread extensions/r2-uploader/src/utils/convert.ts Outdated
Comment thread extensions/r2-uploader/src/utils/convert.ts Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ 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.

Comment thread extensions/r2-uploader/src/utils/convert.ts Outdated
@mazaoshe
mazaoshe marked this pull request as ready for review October 10, 2026 05:27
@mazaoshe

Copy link
Copy Markdown
Contributor Author

Confirmed. ypp is my Raycast account:
https://www.raycast.com/ypp

Comment thread extensions/r2-uploader/package.json

@0xdhrv 0xdhrv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, approved ✅

@raycastbot
raycastbot merged commit 017c6b7 into raycast:main Oct 10, 2026
2 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Published to the Raycast Store:
https://raycast.com/ypp/r2-uploader

@raycastbot

Copy link
Copy Markdown
Collaborator

🎉 🎉 🎉

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 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.

Suggested change
## [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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

extension fix / improvement Label for PRs with extension's fix improvements extension: r2-uploader Issues related to the r2-uploader extension OP is author The OP of the PR is the author of the extension platform: macOS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants