Skip to content

Update rclone-raycast extension#26992

Merged
raycastbot merged 2 commits intoraycast:mainfrom
shtuseriff:ext/rclone-raycast
Apr 10, 2026
Merged

Update rclone-raycast extension#26992
raycastbot merged 2 commits intoraycast:mainfrom
shtuseriff:ext/rclone-raycast

Conversation

@shtuseriff
Copy link
Copy Markdown
Contributor

@shtuseriff shtuseriff commented Apr 7, 2026

Description

Screencast

Checklist

@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: rclone-raycast Issues related to the rclone-raycast extension platform: macOS platform: Windows labels Apr 7, 2026
@raycastbot
Copy link
Copy Markdown
Collaborator

raycastbot commented Apr 7, 2026

Thank you for your first contribution! 🎉

🔔 @FTCHD you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

📋 Quick checkout commands
BRANCH="ext/rclone-raycast"
FORK_URL="https://github.com/shtuseriff/raycast-extensions.git"
EXTENSION_NAME="rclone-raycast"
REPO_NAME="raycast-extensions"

git clone -n --depth=1 --filter=tree:0 -b $BRANCH $FORK_URL
cd $REPO_NAME
git sparse-checkout set --no-cone "extensions/$EXTENSION_NAME"
git checkout
cd "extensions/$EXTENSION_NAME"
npm install && npm run dev

We're currently experiencing a high volume of incoming requests. As a result, the initial review may take up to 10-15 business days.

@shtuseriff shtuseriff marked this pull request as ready for review April 7, 2026 22:39
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 7, 2026

Greptile Summary

Adds windowsHide: true to the spawn() call in start-daemon.tsx so that a terminal window is suppressed on Windows when the rclone daemon starts as a detached background process. The changelog is correctly formatted with the {PR_MERGE_DATE} placeholder at the top, and a contributors entry for misterK is appended to package.json.

Confidence Score: 5/5

Safe to merge — change is minimal, correct, and well-documented.

All three files are clean: the code change uses a well-defined Node.js spawn option, the changelog follows Raycast conventions, and the package.json update is valid JSON. No bugs, security issues, or rule violations were found.

No files require special attention.

Vulnerabilities

No security concerns identified.

Important Files Changed

Filename Overview
extensions/rclone-raycast/src/start-daemon.tsx Adds windowsHide: true to spawn options — correct Node.js API usage to suppress the Windows console popup for the detached daemon process
extensions/rclone-raycast/package.json Appends a contributors array with misterK; JSON remains valid and well-formed
extensions/rclone-raycast/CHANGELOG.md New entry correctly placed at the top with {PR_MERGE_DATE} placeholder, describing the windowsHide change

Reviews (1): Last reviewed commit: "Fix: Hide WinTerm window when running St..." | Re-trigger Greptile

@pernielsentikaer pernielsentikaer self-assigned this Apr 10, 2026
Copy link
Copy Markdown
Collaborator

@pernielsentikaer pernielsentikaer left a comment

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 481b5bb into raycast:main Apr 10, 2026
2 checks passed
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: rclone-raycast Issues related to the rclone-raycast extension platform: macOS platform: Windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants