Update fleet-desktop and zoom-rooms casks - #45912
Conversation
Bump fleet-desktop to 1.2.1 and zoom-rooms to 7.0.5.12655. Update cask files and matching API JSONs: package URLs, filenames, version strings, SHA256 checksums, and ruby_source_checksum values to match the new upstream releases.
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
There was a problem hiding this comment.
Pull request overview
Updates the Homebrew custom tap metadata used by Fleet-maintained apps ingestion by bumping the fleet-desktop and zoom-rooms casks and their generated Homebrew API JSON to match new upstream releases.
Changes:
- Bump
zoom-roomscask version/SHA to7.0.5.12655. - Bump
fleet-desktopcask version/SHA to1.2.1and update the pkg filename in the API JSON. - Refresh corresponding
custom-tap/api/*.jsonfields (url/version/sha256 andruby_source_checksum), addingpinnedmetadata.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| ee/maintained-apps/inputs/homebrew/custom-tap/Casks/zoom-rooms.rb | Bumps Zoom Rooms cask version and SHA256. |
| ee/maintained-apps/inputs/homebrew/custom-tap/Casks/fleet-desktop.rb | Bumps Fleet Desktop cask version and SHA256. |
| ee/maintained-apps/inputs/homebrew/custom-tap/api/zoom-rooms.json | Updates generated Homebrew API JSON to match the bumped cask (url/version/sha256/source checksum). |
| ee/maintained-apps/inputs/homebrew/custom-tap/api/fleet-desktop.json | Updates generated Homebrew API JSON to match the bumped cask (url/version/sha256/pkg filename/source checksum). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| cask "zoom-rooms" do | ||
| version "7.0.0.12322" | ||
| sha256 "c35229e6066732aec7f26b762d0f2d43ab7b8270e512fbabe5bbb77a9c714bbc" | ||
| version "7.0.5.12655" | ||
| sha256 "8fb2ab355a5bdd0acdae3c6e0f10d8d41556a3011478f4faf6198dcad96dd0d1" |
| cask "fleet-desktop" do | ||
| version "1.2.0" | ||
| sha256 "d58a07e71d350b6c7fc39319080c28ad56d298f41ef45c67e0114940dce41420" | ||
| version "1.2.1" | ||
| sha256 "a6c29ee908baa46a6eae76b38141435043135919d958b2d2cf546ba72d2a8911" |
|
Caution Review failedPull request was closed or merged during review No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
WalkthroughThis PR updates two Homebrew package definitions in the custom tap. Fleet Desktop is upgraded from 1.2.0 to 1.2.1, and Zoom Rooms from 7.0.0.12322 to 7.0.5.12655. For each application, both the cask definition file ( Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
…m Rooms 7.1.0.13088 (#49563) **Related issue:** NA — routine custom-tap cask maintenance ## What Bump three custom-tap casks to their latest upstream releases and regenerate the api JSONs (`regenerate.sh`) and FMA output manifests (`go run cmd/maintained-apps/main.go --slug=<token>/darwin`): | Cask | Old | New | |------|-----|-----| | Druva inSync | 7.6.1 (r110931) | 8.1.3 (r110967) | | Fleet Desktop | 1.3.2 | 1.3.4 | | Zoom Rooms | 7.0.5.12655 | 7.1.0.13088 | XCreds 5.9 (9148) is still the latest upstream release and is unchanged. ## Why / reviewer notes - **Druva:** the 8.0.0 phased rollout never reached the public CDN (its `.dmg` 404'd since March); 8.1.3 is the first 8.x build served from `downloads.druva.com`. Druva 8.x officially supports macOS 14 (Sonoma)+ only, so `depends_on` moves from `:big_sur` to `:sonoma` (informational — the ingester doesn't read it). - **`api/xcreds.json` +2 lines with no cask change:** `regenerate.sh` rebuilds all api JSONs, and current Homebrew now emits `pinned`/`pinned_version` fields. Expected drift; anyone running the script gets the same output. - **Verification done against the downloaded installers:** - sha256 values computed locally from the actual downloads; the Druva download's SHA1 matches the checksum published on Druva's download page. - Installer internals inspected: the Druva dmg still contains `Install inSync.pkg`, and pkg receipt IDs are unchanged across all three (`com.druva.inSync.pkg` @ 8.1.3, `com.fleetdm.fleet-desktop` @ 1.3.4, `us.zoom.pkg.zp` @ 7.1.0.13088), so existing install/uninstall stanzas remain valid. - No `changes/` file, consistent with prior cask-bump PRs (#45912, #48028) and the automated FMA-ingestion PRs. # Checklist for submitter - [x] Input data is properly validated, `SELECT *` is avoided, SQL injection is prevented (using placeholders for values in statements), JS inline code is prevented especially for url redirects, and untrusted data interpolated into shell scripts/commands is validated against shell metacharacters. ## Testing - [x] `go test ./ee/maintained-apps/...` passes - [ ] QA'd all new/changed functionality manually (installer metadata, URLs, and checksums verified as above; not yet deployed through a Fleet server)
**Related issue:** NA — routine custom-tap cask maintenance # Checklist for submitter - [x] Input data is properly validated, `SELECT *` is avoided, SQL injection is prevented (using placeholders for values in statements), JS inline code is prevented especially for url redirects, and untrusted data interpolated into shell scripts/commands is validated against shell metacharacters. - [x] Timeouts are implemented and retries are limited to avoid infinite loops ## Testing - [x] `go test ./ee/maintained-apps/...` passes - [ ] QA'd all new/changed functionality manually (installer metadata, URL, and checksum verified as below; not yet deployed through a Fleet server) No `changes/` file, consistent with prior custom-tap cask-bump PRs (#49563, #48028, #45912). ## Version bump details | | Old | New | |---|---|---| | Version | 7.1.0.13088 | 7.1.5.13403 | - **Upstream source:** `https://zoom.us/client/latest/ZoomRooms.pkg` redirects to `https://cdn.zoom.us/prod/7.1.5.13403/ZoomRooms.pkg` (Zoom does not expose a parseable Zoom Rooms version feed, per the cask's `livecheck` block, so this is the standard manual-bump discovery method). - **New download URL:** `https://cdn.zoom.us/prod/7.1.5.13403/ZoomRooms.pkg` - **sha256:** `3b303bc150a3a5d639f09439abf84f2117784a2124ba660f7c73917ba5ef9ab6` - Downloaded installer verified: 587 MB, `xar archive` (matches expected `.pkg` format). **Reviewer note:** `api/zoom-rooms.json` was updated mechanically because `regenerate.sh` requires macOS. Before merging, run `ee/maintained-apps/inputs/homebrew/custom-tap/regenerate.sh` locally and confirm `git diff` is clean for `api/zoom-rooms.json`. --- _Generated by [Claude Code](https://claude.ai/code/session_01U8YEGYFy9Uc88wvyg96ySE)_ <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Updates** * Updated Zoom Rooms for macOS to version 7.1.5.13403. * Refreshed download links and package verification checksums. * Installation and uninstallation behavior remains unchanged. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: Claude <noreply@anthropic.com>
Bump fleet-desktop to 1.2.1 and zoom-rooms to 7.0.5.12655. Update cask files and matching API JSONs: package URLs, filenames, version strings, SHA256 checksums, and ruby_source_checksum values to match the new upstream releases.
Summary by CodeRabbit