Skip to content

croppickvoice: Add version 1.0.0#18269

Open
EvaldasVasiliauskas wants to merge 1 commit into
ScoopInstaller:masterfrom
EvaldasVasiliauskas:croppickvoice
Open

croppickvoice: Add version 1.0.0#18269
EvaldasVasiliauskas wants to merge 1 commit into
ScoopInstaller:masterfrom
EvaldasVasiliauskas:croppickvoice

Conversation

@EvaldasVasiliauskas

Copy link
Copy Markdown

New manifest for CroppickVoice — free on-device voice typing for any app, plus Crop & Speak (https://voice.croppick.com).

  • scoop install and scoop uninstall tested locally on Windows 11 x64 (hash, extract_dir, shortcut all verified)
  • checkver and autoupdate (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
  • Installer files are publisher-hosted at versioned, immutable URLs

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This PR adds a new Scoop bucket manifest file, bucket/croppickvoice.json, for the CroppickVoice application. The manifest specifies version 1.0.0, proprietary license, homepage, a 64-bit Windows download URL with SHA-256 hash, extraction directory, executable shortcut mapping, and checkver/autoupdate configuration referencing a remote latest.json endpoint for version tracking and dynamic URL/hash resolution.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Suggested labels: package-request-needed

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description summarizes the manifest and tests, but it omits the required issue link and template checkboxes. Add a Closes/Relates #XXXX reference and complete the required checklist items from the template, including the Contributing Guide confirmation.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the new CroppickVoice manifest and version added.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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.

@coderabbitai coderabbitai Bot added the package-request-needed Create a package request issue before raising PR. Check the criteria for a package to be accepted. label Jul 9, 2026

@coderabbitai coderabbitai 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.

Actionable comments posted: 2

🧹 Nitpick comments (1)
bucket/croppickvoice.json (1)

1-37: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Test 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 64bit

For 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

📥 Commits

Reviewing files that changed from the base of the PR and between 8280531 and 76263e8.

📒 Files selected for processing (1)
  • bucket/croppickvoice.json

Comment thread bucket/croppickvoice.json
Comment on lines +1 to +37
{
"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"
}
}
}
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 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

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

Labels

package-request-needed Create a package request issue before raising PR. Check the criteria for a package to be accepted.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant