Skip to content

Conversation

@scillidan
Copy link
Contributor

@scillidan scillidan commented Jan 11, 2026

Closes #11665

  • Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>
  • I have read the Contributing Guide

Summary by CodeRabbit

  • New Features
    • Added Scoop package manager support, enabling Windows users to install and update Warpd through Scoop.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 11, 2026

Walkthrough

Adds a Scoop package manifest file for Warpd version 1.3.7, containing metadata including version, description, download URL, file hash, executable name, and autoupdate configuration. No functional code changes; purely manifest-driven package definition data.

Changes

Cohort / File(s) Summary
Scoop Manifest Addition
bucket/warpd.json
New manifest file with 19 lines defining Warpd v1.3.7 package metadata: version, description, homepage, license, download URL with SHA256 hash, bin executable mapping, and GitHub-based autoupdate configuration

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Suggested labels

review-needed

Suggested reviewers

  • z-Fng
  • aliesbelik

Poem

🐰 A manifest hops into the Scoop,
With Warpd dressed in JSON's loop,
Version pinned, hash verified true,
Auto-updates now have their debut!
The bucket grows, the package flies—
One file added, under the skies. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title clearly and specifically describes the main change: adding version 1.3.7 of the warpd manifest.
Description check ✅ Passed The description includes the required issue reference and confirms adherence to the conventional PR title format and contributing guide, matching template expectations.
Linked Issues check ✅ Passed The PR successfully fulfills the warpd package addition request [#11665] by creating a manifest with correct metadata including version 1.3.7, description, homepage, and download configuration.
Out of Scope Changes check ✅ Passed All changes are limited to adding the warpd.json manifest file for version 1.3.7, directly addressing the linked issue requirement with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0afc908 and 55b9394.

📒 Files selected for processing (1)
  • bucket/warpd.json
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
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.
Learnt from: Lutra-Fs
Repo: ScoopInstaller/Extras PR: 16020
File: bucket/regedix.json:6-8
Timestamp: 2025-09-26T02:55:31.132Z
Learning: For Scoop manifests, prefer using "suggest" over "depends" for .NET runtime dependencies because user machines often have externally managed .NET runtimes already installed, and using "depends" could cause duplicate installations of the same runtime.
Learnt from: Gitoffthelawn
Repo: ScoopInstaller/Extras PR: 16106
File: bucket/czkawka-gui.json:25-25
Timestamp: 2025-09-05T09:41:52.653Z
Learning: For czkawka-gui manifests in Scoop Extras, the correct upstream filename pattern for Windows GUI builds in recent versions uses "gtk46" (without underscore), not "gtk_46" (with underscore). The autoupdate URL should use "windows_czkawka_gui_gtk46.zip" format. This was confirmed by a working 10.0.0 update with valid hash.
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: WindowsPowerShell
🔇 Additional comments (4)
bucket/warpd.json (4)

2-2: No changes needed. The "-windows-alpha" suffix is the upstream release tag naming convention and is not a pre-release indicator. GitHub classifies v1.3.7-windows-alpha as a non-prerelease, and it is the only Windows binary available for this version. Stable Windows releases without this suffix do not exist.

Likely an incorrect or invalid review comment.


5-5: License field is correct. The warpd project is released under the MIT License, as specified in line 5 of the manifest.


7-7: Verify the hash matches the actual download.

Ensure that the provided SHA-256 hash matches the actual warpd.exe file from the release.

Run the following script to verify the hash:

#!/bin/bash
# Description: Download and verify hash of warpd.exe v1.3.7-windows-alpha

# Download the file and compute SHA-256
curl -sL "https://github.com/rvaiya/warpd/releases/download/v1.3.7-windows-alpha/warpd.exe" | sha256sum

16-18: No hash verification available for warpd releases.

The autoupdate block lacks hash verification, but this is not an issue that can be resolved: warpd releases do not provide checksum files or digest data. This mirrors the situation with CompactGUI, where hash verification was removed because the mechanism is not available upstream. The autoupdate URL is properly configured.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

warpd

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Check the full log for details.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Request]: warpd

1 participant