Update Fleet-maintained apps - #45870
Conversation
Generated automatically with cmd/maintained-apps.
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.
WalkthroughThis PR updates three maintained-app JSON configuration files in Fleet. Google Chrome for macOS is bumped from version 148.0.7778.168 to 148.0.7778.179. Postman for macOS and Windows are both bumped from version 12.11.2 to 12.11.3. Each update modifies the version field, the SQL Possibly related PRs
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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 |
There was a problem hiding this comment.
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 `@ee/maintained-apps/outputs/postman/darwin.json`:
- Around line 4-7: The Postman entry currently targets an ARM-only installer
(installer_url with osx_arm64) but the manifest fields like "version": "12.11.3"
and the queries using bundle_identifier 'com.postmanlabs.mac' and
version_compare may be applied to mixed Intel/Apple Silicon fleets; confirm
whether this manifest is deliberately Apple Silicon–only and, if not, add a
separate Intel (x86_64) variant or a combined installer URL for Intel Macs,
update or add a parallel manifest/installer_url for the Intel build and ensure
any deployment selector or docs indicate the architecture-specific scope so that
the "patched" and "exists" queries correctly reflect both architectures.
🪄 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: a9b1407b-df57-4355-9625-645ff8daa4da
📒 Files selected for processing (3)
ee/maintained-apps/outputs/google-chrome/darwin.jsonee/maintained-apps/outputs/postman/darwin.jsonee/maintained-apps/outputs/postman/windows.json
| "version": "12.11.3", | ||
| "queries": { | ||
| "exists": "SELECT 1 FROM apps WHERE bundle_identifier = 'com.postmanlabs.mac';", | ||
| "patched": "SELECT 1 WHERE NOT EXISTS (SELECT 1 FROM apps WHERE bundle_identifier = 'com.postmanlabs.mac' AND version_compare(bundle_short_version, '12.11.2') < 0);" | ||
| "patched": "SELECT 1 WHERE NOT EXISTS (SELECT 1 FROM apps WHERE bundle_identifier = 'com.postmanlabs.mac' AND version_compare(bundle_short_version, '12.11.3') < 0);" |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Check for other Postman architecture variants
fd -e json postman ee/maintained-apps/outputs/ --exec echo "=== {} ===" \; --exec cat {} \; | grep -A 1 -B 1 'installer_url'
# Search for architecture handling logic
rg -n 'osx_arm64|osx_64|x86_64|universal' ee/maintained-apps/Repository: fleetdm/fleet
Length of output: 15675
🌐 Web query:
Does Postman version 12.11.3 ARM64 macOS installer support Intel Macs or is it ARM-only?
💡 Result:
No—Postman isn’t ARM-only. Postman provides separate macOS builds for Intel (“Mac Intel Chip”) and for Apple silicon (“Mac Apple Chip”).[1][2] Specifically, the Postman download/install documentation indicates Postman for macOS is available for “macOS (Intel or Apple silicon)” and instructs Apple silicon Macs to download the “Mac Apple Chip version.”[2] The Postman downloads page also shows distinct installer options for “Mac Intel Chip” vs “Mac Apple Chip,” implying Intel Macs use the Intel installer, not the ARM64 one.[1] Therefore, an ARM64 macOS installer for Postman 12.11.3 would be intended for Apple silicon Macs, while Intel Macs should use the Intel macOS installer variant (or run Postman under Rosetta if applicable), rather than assuming the ARM64 installer supports Intel natively.[1][2]
Citations:
- 1: https://www.postman.com/downloads/postman-agent/
- 2: https://learning.postman.com/docs/getting-started/installation/installation-and-updates
Confirm that Postman 12.11.3 ARM64 installer scope aligns with deployment targets.
The installer_url specifies osx_arm64, which is ARM-only (Apple Silicon only). Postman provides separate Intel and Apple Silicon installers. If Fleet's deployment includes Intel Macs, a separate configuration with the Intel installer is required. Clarify whether this configuration intentionally targets Apple Silicon Macs only.
🤖 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 `@ee/maintained-apps/outputs/postman/darwin.json` around lines 4 - 7, The
Postman entry currently targets an ARM-only installer (installer_url with
osx_arm64) but the manifest fields like "version": "12.11.3" and the queries
using bundle_identifier 'com.postmanlabs.mac' and version_compare may be applied
to mixed Intel/Apple Silicon fleets; confirm whether this manifest is
deliberately Apple Silicon–only and, if not, add a separate Intel (x86_64)
variant or a combined installer URL for Intel Macs, update or add a parallel
manifest/installer_url for the Intel build and ensure any deployment selector or
docs indicate the architecture-specific scope so that the "patched" and "exists"
queries correctly reflect both architectures.
|
Closing in favor of #45875. |
Automated ingestion of latest Fleet-maintained app data.
Summary by CodeRabbit