-
Notifications
You must be signed in to change notification settings - Fork 1.6k
opencode-desktop: Add version 1.1.25 #16996
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
WalkthroughAdds a new Scoop-style Windows 64-bit package manifest for OpenCode Desktop, defining metadata, download URL and hash, a pre-install PowerShell cleanup, binary and Start Menu shortcut mappings, GitHub checkver, and an autoupdate URL template. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Your changes did not pass all checks. Please address the issues in the manifest and comment starting with Invalid manifests
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In @bucket/opencode-desktop.json:
- Around line 9-10: Remove the trailing comma after the "hash" value in the JSON
object (the comma following the hash string) so the file is valid JSON; locate
the "hash" key in the shown object and delete the comma after its value to fix
the "Invalid JSON in manifest" parse error.
- Around line 12-20: The installer/uninstaller scripts use unsupported NSIS
flags and reference a non-existent release; update the "installer" and
"uninstaller" script entries to only use official NSIS options (use "/S" and
"/D=..." for installer and "/S" for uninstaller) by removing "/noredist" and
"/Language=English" from the installer script and any extra args in the
uninstaller script, and then verify the package version referenced in the
manifest matches an actual GitHub release (update the version from v1.1.15 to
the latest existing release, e.g., v1.1.11, or confirm the newer tag exists)
before committing.
🧹 Nitpick comments (1)
bucket/opencode-desktop.json (1)
31-37: Consider adding hash extraction to autoupdate if available.The autoupdate section doesn't include a
hashproperty, so hash verification will need to be updated manually with each release. If the upstream releases provide checksums (e.g., in a.sha256file or release notes), consider adding hash extraction for automated updates.Based on learnings, note that some projects (like CompactGUI) have removed hash verification from autoupdate because upstream doesn't provide consistent hash files. If opencode releases don't include checksums, manual hash updates are acceptable.
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
bucket/opencode-desktop.jsonbucket/opencode.json
💤 Files with no reviewable changes (1)
- bucket/opencode.json
🧰 Additional context used
🧠 Learnings (2)
📚 Learning: 2025-10-19T13:58:23.389Z
Learnt from: o-l-a-v
Repo: ScoopInstaller/Extras PR: 16378
File: bucket/compactgui.json:20-22
Timestamp: 2025-10-19T13:58:23.389Z
Learning: In the ScoopInstaller/Extras repository, the CompactGUI manifest removed hash verification from the autoupdate block because the hash verification mechanism (scraping SHA-256 from release page HTML) is no longer available in newer CompactGUI releases. GitHub asset digests exist in beta releases but not in v3.8.0, and Scoop doesn't have built-in support for extracting from GitHub API asset digests.
Applied to files:
bucket/opencode-desktop.json
📚 Learning: 2025-11-29T22:02:34.868Z
Learnt from: Ra2-IFV
Repo: ScoopInstaller/Extras PR: 16672
File: bucket/chromium-clang.json:12-12
Timestamp: 2025-11-29T22:02:34.868Z
Learning: In the chromium-clang.json manifest (bucket/chromium-clang.json), the extract_dir value "chrome-win32" is a fixed name determined by the upstream Chromium_Clang archive structure and should not be changed, even though the manifest targets 64-bit architecture.
Applied to files:
bucket/opencode-desktop.json
🪛 Biome (2.1.2)
bucket/opencode-desktop.json
[error] 10-10: Expected a property but instead found '}'.
Expected a property here.
(parse)
🪛 GitHub Actions: Pull Requests
bucket/opencode-desktop.json
[error] 1-1: Conversion failed. Invalid JSON in manifest.
[error] 1-1: Manifest invalid: opencode-desktop. PR checks will fail until fixed.
🔇 Additional comments (1)
bucket/opencode-desktop.json (1)
21-27: LGTM!The binary and shortcuts configuration correctly separates the CLI (
opencode-cli.exe) from the GUI executable (OpenCode.exe), which aligns with the PR objective.
|
/verify |
|
All changes look good. Wait for review from human collaborators. opencode-desktop
|
z-Fng
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
/verify |
|
All changes look good. Wait for review from human collaborators. opencode-desktop
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@bucket/opencode-desktop.json`:
- Around line 20-29: The autoupdate URL is pointing to the wrong repository
(sst/opencode) while checkver uses anomalyco/opencode; update the "autoupdate"
-> "architecture" -> "64bit" -> "url" value to reference the anomalyco/opencode
releases URL (use the same repo as checkver) and keep the asset name
opencode-desktop-windows-x64.exe and the v$version token unchanged so downloads
come from anomalyco/opencode releases instead of the deprecated sst redirect.
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
|
/verify |
|
All changes look good. Wait for review from human collaborators. opencode-desktop
|
z-Fng
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution!
The manifest will update automatically as the package updates, as long as the upstream releases new packages that follow the same pattern defined in the manifest. If there are any other changes, feel free to submit a PR.
|
Thanks for your review! |
We have migrated the opencode manifest on Scoop to enable support for the desktop variant. The migration was made to not be a breaking change and wouldn't break existing installation but they do have to perform a manual reinstallation to finalize the transition to get future updates. This patch updates the relevant files to reflect the changes. refs: - ScoopInstaller/Main#7524 - ScoopInstaller/Extras#16996 - ScoopInstaller/Extras#17040 Signed-off-by: Chawye Hsu <su+git@chawyehsu.com>

This is Desktop version includes bin file
opencode-cli.exe.The CLI version will be in Main bucket.
<manifest-name[@version]|chore>: <general summary of the pull request>Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.