Skip to content

Update Fleet-maintained apps#47559

Closed
fleet-release wants to merge 1 commit into
mainfrom
fma-2606130325
Closed

Update Fleet-maintained apps#47559
fleet-release wants to merge 1 commit into
mainfrom
fma-2606130325

Conversation

@fleet-release

@fleet-release fleet-release commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Automated ingestion of latest Fleet-maintained app data.

Summary by CodeRabbit

  • Chores
    • Updated version metadata for multiple applications across Windows and macOS platforms: Android Studio, AWS CLI, Codex, FireAlpaca, Nextcloud Talk, Notesnook, and NVIDIA GeForce NOW to their latest releases with corresponding installer updates and checksums.

Generated automatically with cmd/maintained-apps.
@coderabbitai

coderabbitai Bot commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

This PR updates version metadata for seven maintained applications across Windows and macOS platforms. Each update modifies the corresponding output JSON file to bump the application version, adjust the SQL version-comparison predicate for patched-state detection, point to the new installer URL, and replace the SHA256 checksum with the new artifact hash. All other configuration fields, including script references and categories, remain unchanged.

Possibly related PRs

  • fleetdm/fleet#46061: Updates version metadata in ee/maintained-apps/outputs/*/windows.json files, including version, installer URLs, and checksums for maintained apps.
  • fleetdm/fleet#46405: Updates AWS CLI Windows output definition with version metadata and patched query adjustments.
  • fleetdm/fleet#46418: Modifies AWS CLI Windows version metadata, including version threshold and installer URL updates.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is vague and generic ('Automated ingestion of latest Fleet-maintained app data'), lacking details about which apps were updated, version changes, or testing performed. Expand the description to list the apps updated (Android Studio, AWS CLI, Codex, FireAlpaca, Nextcloud Talk, Notesnook, NVIDIA GeForce Now) and their version changes, or confirm this is a standard automated workflow that doesn't require additional detail.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change—updating multiple Fleet-maintained app version metadata entries across several JSON configuration files.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fma-2606130325

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 and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)
ee/maintained-apps/outputs/firealpaca/darwin.json (1)

12-12: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

SHA256 checksum disabled for FireAlpaca.

The sha256 field is set to "no_check", which disables integrity verification of the downloaded installer. This creates a security risk as the installer cannot be verified against tampering or corruption.

Consider obtaining and using the actual SHA256 checksum for the FireAlpaca 2.16.0 installer to enable integrity verification.

🤖 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/firealpaca/darwin.json` at line 12, The json
currently disables integrity verification by setting the "sha256" field to
"no_check" for FireAlpaca 2.16.0; replace that placeholder with the actual
64-character SHA256 hex of the official installer (obtain from vendor or compute
locally) and validate it against a downloaded copy to ensure it matches, then
update the "sha256" value with the lowercase hex string (instead of "no_check")
so the installer integrity is enforced.
ee/maintained-apps/outputs/nvidia-geforce-now/darwin.json (1)

12-12: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

SHA256 checksum disabled for NVIDIA GeForce NOW.

The sha256 field is set to "no_check", which disables integrity verification of the downloaded installer. This creates a security risk as the installer cannot be verified against tampering or corruption.

Consider obtaining and using the actual SHA256 checksum for the GeForce NOW 2.0.85.135 installer to enable integrity verification.

🤖 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/nvidia-geforce-now/darwin.json` at line 12, The
"sha256" field currently set to "no_check" disables integrity verification;
replace that placeholder with the real SHA256 digest for the NVIDIA GeForce NOW
2.0.85.135 macOS installer by downloading the exact installer used in the
package, computing its SHA256 (e.g., shasum -a 256 or equivalent), and updating
the "sha256" field in darwin.json with the computed hex string (ensure the value
is the exact lowercase hex and not wrapped in any extra text).
🤖 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.

Outside diff comments:
In `@ee/maintained-apps/outputs/firealpaca/darwin.json`:
- Line 12: The json currently disables integrity verification by setting the
"sha256" field to "no_check" for FireAlpaca 2.16.0; replace that placeholder
with the actual 64-character SHA256 hex of the official installer (obtain from
vendor or compute locally) and validate it against a downloaded copy to ensure
it matches, then update the "sha256" value with the lowercase hex string
(instead of "no_check") so the installer integrity is enforced.

In `@ee/maintained-apps/outputs/nvidia-geforce-now/darwin.json`:
- Line 12: The "sha256" field currently set to "no_check" disables integrity
verification; replace that placeholder with the real SHA256 digest for the
NVIDIA GeForce NOW 2.0.85.135 macOS installer by downloading the exact installer
used in the package, computing its SHA256 (e.g., shasum -a 256 or equivalent),
and updating the "sha256" field in darwin.json with the computed hex string
(ensure the value is the exact lowercase hex and not wrapped in any extra text).

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 48bf5d8a-7811-4e9e-95a2-df101fe208a9

📥 Commits

Reviewing files that changed from the base of the PR and between 268c918 and 678f56d.

📒 Files selected for processing (7)
  • ee/maintained-apps/outputs/android-studio/windows.json
  • ee/maintained-apps/outputs/aws-cli/windows.json
  • ee/maintained-apps/outputs/codex-app/darwin.json
  • ee/maintained-apps/outputs/firealpaca/darwin.json
  • ee/maintained-apps/outputs/nextcloud-talk/darwin.json
  • ee/maintained-apps/outputs/notesnook/darwin.json
  • ee/maintained-apps/outputs/nvidia-geforce-now/darwin.json

@github-actions

Copy link
Copy Markdown
Contributor

Closing in favor of #47561.

@github-actions github-actions Bot closed this Jun 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants