Skip to content

Remove Dynalist macOS Fleet-maintained app (cask deleted from homebrew-cask) - #50215

Merged
allenhouchins merged 1 commit into
mainfrom
remove-dynalist-macos-fma
Jul 30, 2026
Merged

Remove Dynalist macOS Fleet-maintained app (cask deleted from homebrew-cask)#50215
allenhouchins merged 1 commit into
mainfrom
remove-dynalist-macos-fma

Conversation

@allenhouchins

@allenhouchins allenhouchins commented Jul 30, 2026

Copy link
Copy Markdown
Member

Related issue: NA — fixing a failing scheduled "Update Fleet-maintained apps" run.

Removes the macOS Dynalist Fleet-maintained app. The Windows (winget) FMA is unaffected and stays.

Why

The dynalist cask was deleted from homebrew-cask on 2026-07-30 (commit adac21ffc4), completing Homebrew's full deprecation lifecycle:

  • deprecate! 2024-07-29, because: :unmaintained
  • disable! 2025-07-29
  • cask file removed 2026-07-30

https://formulae.brew.sh/api/cask/dynalist.json now returns 404, so the scheduled ingester panics:

{"level":"INFO","msg":"ingesting homebrew app","name":"Dynalist"}
panic: ingesting homebrew app: app not found in brew API

Marking the app "frozen": true does not fix this — the ingester fetches the cask JSON before the frozen flag is consulted (cmd/maintained-apps/main.go), so removal is the only fix for a dead upstream cask.

What changed

Deletions only (39 lines, no additions):

  • ee/maintained-apps/inputs/homebrew/dynalist.json — deleted
  • ee/maintained-apps/outputs/dynalist/darwin.json — deleted
  • the dynalist/darwin entry in ee/maintained-apps/outputs/apps.json — removed; dynalist/windows remains

Deliberately kept

Because the Windows FMA still needs them:

  • ee/maintained-apps/outputs/dynalist/windows.json, ee/maintained-apps/inputs/winget/dynalist.json, and its install/uninstall scripts
  • frontend/pages/SoftwarePage/components/icons/Dynalist.tsx and its dynalist: mapping in icons/index.ts — the map is keyed by lowercased app name and is shared across platforms
  • website/assets/images/app-icon-dynalist-60x60@2x.png — keyed by slug token, still serving the Windows entry in the app library

Note for reviewers

Hosts that currently have the macOS Dynalist FMA installed will lose the maintained-app entry on the next sync. This matches the behavior of prior FMA removals (Nocturnal #50050, Dell Display Manager #47420, Messenger #46541).

Checklist for submitter

  • Changes file added for user-visible changes in changes/, orbit/changes/ or ee/fleetd-chrome/changes.
    Not applicable — consistent with prior FMA removal PRs, which do not add a changes file.

Testing

  • QA'd all new/changed functionality manually

Verified that ee/maintained-apps/outputs/apps.json still parses as valid JSON (1381 apps) and that Dynalist now resolves to only dynalist/windows. Confirmed the 404 against the brew API and read the pre-removal cask at adac21ffc4~1 to establish the deprecation reason. git diff confirms the change is deletions-only.

Summary by CodeRabbit

  • Bug Fixes
    • Removed Dynalist’s macOS app listing and installer metadata.
    • Dynalist remains available for Windows.

The dynalist cask was deleted from homebrew-cask on 2026-07-30
(commit adac21ffc4), completing Homebrew's deprecation lifecycle:
deprecate! 2024-07-29 (:unmaintained) -> disable! 2025-07-29 ->
cask file removed. formulae.brew.sh now 404s for it, so the
scheduled FMA ingester panics with "app not found in brew API".

Freezing the app does not help: the ingester fetches the cask JSON
before the frozen flag is consulted, so the only fix is removal.

Removes the homebrew input, the darwin output, and the
dynalist/darwin entry in apps.json. The Windows (winget) FMA is
unaffected and stays, along with the shared icons.
Copilot AI review requested due to automatic review settings July 30, 2026 15:26
@github-actions

Copy link
Copy Markdown
Contributor

Script Diff Results

ee/maintained-apps/inputs/homebrew/dynalist.json

Error: File 'ee/maintained-apps/inputs/homebrew/dynalist.json' does not exist

ee/maintained-apps/outputs/dynalist/darwin.json

Error: File 'ee/maintained-apps/outputs/dynalist/darwin.json' does not exist

Copilot AI 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.

Pull request overview

Removes the macOS (Homebrew/cask) Fleet-maintained app definition for Dynalist after the upstream cask was deleted, preventing the maintained-apps ingest from failing when the Brew API returns 404. The Windows (winget) Dynalist FMA remains intact.

Changes:

  • Deleted the Homebrew input definition for dynalist (macOS).
  • Deleted the generated macOS output metadata for dynalist.
  • Removed the dynalist/darwin entry from the maintained apps registry (apps.json), leaving dynalist/windows.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
ee/maintained-apps/outputs/dynalist/darwin.json Removes macOS maintained-app output metadata for Dynalist.
ee/maintained-apps/outputs/apps.json Drops the dynalist/darwin entry while keeping dynalist/windows.
ee/maintained-apps/inputs/homebrew/dynalist.json Removes the Homebrew input for the now-nonexistent cask.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@allenhouchins
allenhouchins marked this pull request as ready for review July 30, 2026 15:31
@allenhouchins
allenhouchins merged commit 344e5aa into main Jul 30, 2026
20 checks passed
@allenhouchins
allenhouchins deleted the remove-dynalist-macos-fma branch July 30, 2026 15:31
@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: d01c14ea-58a1-4c81-9030-eea4df6f4ccb

📥 Commits

Reviewing files that changed from the base of the PR and between dcefd13 and d64028a.

📒 Files selected for processing (3)
  • ee/maintained-apps/inputs/homebrew/dynalist.json
  • ee/maintained-apps/outputs/apps.json
  • ee/maintained-apps/outputs/dynalist/darwin.json

Walkthrough

Removed Dynalist’s macOS application mapping and associated macOS maintained-app configuration and output. The Windows Dynalist entry remains available.

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

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.

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.

3 participants