Skip to content

officecli: Add version 1.0.129#18249

Closed
goworm wants to merge 2 commits into
ScoopInstaller:masterfrom
goworm:officecli
Closed

officecli: Add version 1.0.129#18249
goworm wants to merge 2 commits into
ScoopInstaller:masterfrom
goworm:officecli

Conversation

@goworm

@goworm goworm commented Jul 7, 2026

Copy link
Copy Markdown

Closes ScoopInstaller/Main#8278

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

Adds a manifest for officecli — read, edit, and automate Office documents (.docx, .xlsx, .pptx) from the command line. Single self-contained binary, Apache-2.0, ~9k stars. Already in Homebrew core and on npm.

  • 64bit + arm64, hashes verified against the release's SHA256SUMS
  • checkver: github + autoupdate with per-arch hash extraction from SHA256SUMS

@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This pull request adds a new Scoop bucket manifest file, bucket/officecli.json, for OfficeCLI version 1.0.129. The manifest includes package metadata, Windows x64 and arm64 download URLs with SHA-256 hashes, the executable name, and an autoupdate configuration that uses $version substitution.

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

Related issue: #18248

Possibly related PRs

  • ScoopInstaller/Extras#17808: Also adds a Scoop bucket manifest with GitHub release downloads, architecture-specific URLs, hashes, and checkver/autoupdate fields.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the package and version added by this PR.
Description check ✅ Passed The description includes the issue link, summary, and required checklist items from the template.
Linked Issues check ✅ Passed The manifest matches the issue requirements for officecli, including latest version, x64/arm64 URLs, and SHA256SUMS-based hashes.
Out of Scope Changes check ✅ Passed The PR only adds the officecli manifest and does not introduce unrelated changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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.

@goworm

goworm commented Jul 7, 2026

Copy link
Copy Markdown
Author

/verify

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Your changes did not pass all checks.

Please address the issues in the manifest and comment starting with /verify to rerun the checks.

officecli

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate
  • Autoupdate Hash Extraction

Check the full log for details.

@goworm

goworm commented Jul 7, 2026

Copy link
Copy Markdown
Author

/verify

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

All changes look good.

Wait for review from human collaborators.

officecli

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate
  • Autoupdate Hash Extraction

Check the full log for details.

@niheaven

niheaven commented Jul 7, 2026

Copy link
Copy Markdown
Member

This could be moved to 'main' bucket

@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: 1

🤖 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/officecli.json`:
- Around line 20-29: The autoupdate block in the OfficeCLI manifest only defines
per-architecture URLs and is missing the hash extraction rule needed for the
SHA256SUMS workflow. Update the autoupdate configuration to include an
autoupdate.hash rule that points to SHA256SUMS so the manifest can resolve
checksums during updates; use the existing autoupdate and architecture entries
as the place to extend.
🪄 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: 85f916ee-9310-46c7-ad44-43fbeaf8b08c

📥 Commits

Reviewing files that changed from the base of the PR and between 51f3706 and 6a107d0.

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

Comment thread bucket/officecli.json
@goworm

goworm commented Jul 8, 2026

Copy link
Copy Markdown
Author

Sounds great — officecli is a pure CLI tool, so Main fits. Opening a PR to Main with the same manifest; will link it here.

@goworm

goworm commented Jul 8, 2026

Copy link
Copy Markdown
Author

Main PR opened: ScoopInstaller/Main#8274

@z-Fng z-Fng added the wrong-bucket This manifest belongs in a different bucket (e.g., Main, Games). Please submit it there instead. label Jul 8, 2026
@z-Fng z-Fng closed this Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wrong-bucket This manifest belongs in a different bucket (e.g., Main, Games). Please submit it there instead.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Request]: officecli

3 participants