croppickvoice: Add version 1.0.0#18269
Conversation
📝 WalkthroughWalkthroughThis PR adds a new Scoop bucket manifest file, Estimated code review effort: 1 (Trivial) | ~3 minutes Suggested labels: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
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 |
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
bucket/croppickvoice.json (1)
1-37: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winTest the manifest locally before submission.
Before submitting, please verify the manifest using these commands:
# Optional: enable debug mode and GitHub token for release API access scoop config debug true scoop config gh_token <your-github-token> # read access only # Force version detection and autoupdate .\bin\checkver.ps1 -App croppickvoice -f # Auto-format (lint) the JSON manifest .\bin\formatjson.ps1 -App croppickvoice # Test installation scoop install bucket/croppickvoice.json -a 64bitFor contribution guidelines, see: https://github.com/ScoopInstaller/.github/blob/main/.github/CONTRIBUTING.md
For the Scoop wiki on app manifests, see: https://github.com/ScoopInstaller/Scoop/wiki/App-Manifests🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@bucket/croppickvoice.json` around lines 1 - 37, The CroppickVoice manifest needs to be verified locally before submission; please run the existing validation flow against croppickvoice.json by checking version detection with checkver, formatting with formatjson, and testing a 64bit install so any URL/hash/extract_dir issues in the current manifest or autoupdate block are caught before merge.Source: Path instructions
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@bucket/croppickvoice.json`:
- Around line 1-37: The manifest in croppickvoice.json needs CRLF line endings
instead of LF. Update the file’s line endings to CRLF across the entire JSON
content, preserving the existing structure and values, and verify the saved file
still parses correctly after the conversion.
- Around line 1-37: This new manifest is missing the required linked
package-request reference; update the croppickvoice manifest’s description (or
the PR body) to include a GitHub issue number or issue URL for the package
request. Use the existing manifest entry and ensure the added reference points
to the newly created ScoopInstaller/Extras package-request issue so the new
package complies with policy.
---
Nitpick comments:
In `@bucket/croppickvoice.json`:
- Around line 1-37: The CroppickVoice manifest needs to be verified locally
before submission; please run the existing validation flow against
croppickvoice.json by checking version detection with checkver, formatting with
formatjson, and testing a 64bit install so any URL/hash/extract_dir issues in
the current manifest or autoupdate block are caught before merge.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 309cfdb6-8190-4d84-9ae7-73b513ddbfad
📒 Files selected for processing (1)
bucket/croppickvoice.json
| { | ||
| "version": "1.0.0", | ||
| "description": "Free on-device voice typing for any app, plus Crop & Speak — annotate a region of your screen and narrate it into ChatGPT or Claude.", | ||
| "homepage": "https://voice.croppick.com", | ||
| "license": { | ||
| "identifier": "Proprietary", | ||
| "url": "https://voice.croppick.com/terms" | ||
| }, | ||
| "architecture": { | ||
| "64bit": { | ||
| "url": "https://dl-voice.croppick.com/v1.0.0/CroppickVoice-windows-amd64.zip", | ||
| "hash": "18ff380a9a69c1063bf070e2c055f74824eb906795d43ddb06682206b92cadf0" | ||
| } | ||
| }, | ||
| "extract_dir": "CroppickVoice", | ||
| "shortcuts": [ | ||
| [ | ||
| "CroppickVoice.exe", | ||
| "CroppickVoice" | ||
| ] | ||
| ], | ||
| "checkver": { | ||
| "url": "https://dl-voice.croppick.com/latest.json", | ||
| "jsonpath": "$.version" | ||
| }, | ||
| "autoupdate": { | ||
| "architecture": { | ||
| "64bit": { | ||
| "url": "https://dl-voice.croppick.com/v$version/CroppickVoice-windows-amd64.zip", | ||
| "hash": { | ||
| "url": "https://dl-voice.croppick.com/latest.json", | ||
| "jsonpath": "$.assets['windows-amd64'].portable.sha256" | ||
| } | ||
| } | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Ensure CRLF line endings.
Scoop manifests must use CRLF line endings. Please verify this file uses CRLF and convert if necessary (e.g., in your editor or via git config core.autocrlf settings).
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@bucket/croppickvoice.json` around lines 1 - 37, The manifest in
croppickvoice.json needs CRLF line endings instead of LF. Update the file’s line
endings to CRLF across the entire JSON content, preserving the existing
structure and values, and verify the saved file still parses correctly after the
conversion.
Source: Path instructions
📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win
Link a package request issue for this new package.
This is a newly added manifest. Per ScoopInstaller/Extras policy, new package requests must have a linked GitHub issue (e.g., #123 or an issue URL) in the description field or PR body. No such reference is present in the manifest or PR summary.
Please create a package request issue at https://github.com/ScoopInstaller/Extras/issues/new?labels=package-request&template=package-request.yml&title=%5BRequest%5D%3A+ and link it here. Acceptance criteria: reasonably well-known/widely used (e.g., 100+ GitHub stars if applicable), stable release, and English interface/documentation.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@bucket/croppickvoice.json` around lines 1 - 37, This new manifest is missing
the required linked package-request reference; update the croppickvoice
manifest’s description (or the PR body) to include a GitHub issue number or
issue URL for the package request. Use the existing manifest entry and ensure
the added reference points to the newly created ScoopInstaller/Extras
package-request issue so the new package complies with policy.
Source: Path instructions
New manifest for CroppickVoice — free on-device voice typing for any app, plus Crop & Speak (https://voice.croppick.com).
scoop installandscoop uninstalltested locally on Windows 11 x64 (hash, extract_dir, shortcut all verified)checkverandautoupdate(with -ForceUpdate) verified against the publisher release manifest at https://dl-voice.croppick.com/latest.json — hash is extracted via JSONPath, so excavator can auto-update future versions