Skip to content

Add Groove OmniDialer as a macOS FMA - #47020

Merged
allenhouchins merged 1 commit into
mainfrom
46879-new-fma-groove-omnidialer
Jun 7, 2026
Merged

Add Groove OmniDialer as a macOS FMA#47020
allenhouchins merged 1 commit into
mainfrom
46879-new-fma-groove-omnidialer

Conversation

@allenhouchins

@allenhouchins allenhouchins commented Jun 7, 2026

Copy link
Copy Markdown
Member

Introduce Groove OmniDialer to the maintained apps dataset and UI. Adds a homebrew input (dmg) and an apps.json entry, plus a darwin-specific output with version 26.603.1017, installer URL, sha256, and embedded install/uninstall script refs. Also adds a frontend SVG icon component, registers it in the icons index, and includes the app PNG asset. Install/uninstall scripts handle quitting/relaunching the app and cleaning user data.

Summary by CodeRabbit

  • New Features
    • Added Groove OmniDialer, a communication application for macOS (version 26.603.1017), with integrated installation and uninstallation management.

Introduce Groove OmniDialer to the maintained apps dataset and UI. Adds a homebrew input (dmg) and an apps.json entry, plus a darwin-specific output with version 26.603.1017, installer URL, sha256, and embedded install/uninstall script refs. Also adds a frontend SVG icon component, registers it in the icons index, and includes the app PNG asset. Install/uninstall scripts handle quitting/relaunching the app and cleaning user data.
@allenhouchins allenhouchins linked an issue Jun 7, 2026 that may be closed by this pull request
@allenhouchins
allenhouchins marked this pull request as ready for review June 7, 2026 03:05
@allenhouchins
allenhouchins requested a review from a team as a code owner June 7, 2026 03:05
Copilot AI review requested due to automatic review settings June 7, 2026 03:05
@fleet-release
fleet-release requested a review from eashaw June 7, 2026 03:05

@claude claude 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.

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.

@codecov

codecov Bot commented Jun 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 67.04%. Comparing base (6fb22c0) to head (ef333f4).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...SoftwarePage/components/icons/GrooveOmniDialer.tsx 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #47020      +/-   ##
==========================================
- Coverage   67.04%   67.04%   -0.01%     
==========================================
  Files        2872     2873       +1     
  Lines      225127   225129       +2     
  Branches    11742    11743       +1     
==========================================
+ Hits       150929   150930       +1     
  Misses      60528    60528              
- Partials    13670    13671       +1     
Flag Coverage Δ
frontend 56.97% <50.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@allenhouchins
allenhouchins merged commit 3006d90 into main Jun 7, 2026
29 of 30 checks passed
@allenhouchins
allenhouchins deleted the 46879-new-fma-groove-omnidialer branch June 7, 2026 03:05
@coderabbitai

coderabbitai Bot commented Jun 7, 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

Run ID: 8254c273-6e23-4279-afcd-8b846f378445

📥 Commits

Reviewing files that changed from the base of the PR and between 6fb22c0 and ef333f4.

⛔ Files ignored due to path filters (1)
  • website/assets/images/app-icon-groove-omnidialer-60x60@2x.png is excluded by !**/*.png
📒 Files selected for processing (5)
  • ee/maintained-apps/inputs/homebrew/groove-omnidialer.json
  • ee/maintained-apps/outputs/apps.json
  • ee/maintained-apps/outputs/groove-omnidialer/darwin.json
  • frontend/pages/SoftwarePage/components/icons/GrooveOmniDialer.tsx
  • frontend/pages/SoftwarePage/components/icons/index.ts

Walkthrough

This PR adds complete integration for Groove OmniDialer, a macOS communication application. The change includes: (1) new homebrew input metadata defining the app's unique identifiers, installer format, and darwin slug; (2) a new entry in the apps catalog registering the application on darwin platform; (3) a darwin-specific version configuration with embedded bash scripts for installation (DMG mount/copy with app relaunching and GUI session handling) and uninstallation (app removal and user data cleanup); and (4) a new React SVG icon component wired into the frontend icon registry to display the application in the software browser.

Possibly related issues

  • fleetdm/fleet#46879: This PR implements the Groove OmniDialer Finalize Maintenance Assignment (FMA) by adding complete application metadata, installation scripts, and frontend icon support.

Possibly related PRs

  • fleetdm/fleet#46337: Both PRs add new macOS app entries by extending ee/maintained-apps/outputs/apps.json and registering icon components in the frontend icon registry.
  • fleetdm/fleet#46264: Both PRs extend the icon registry in frontend/pages/SoftwarePage/components/icons/index.ts by adding new app keys and corresponding icon components to SOFTWARE_NAME_TO_ICON_MAP.
  • fleetdm/fleet#46734: Both PRs follow the same pattern of adding installer metadata, darwin app output configuration, and frontend icon registration for new macOS applications.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 46879-new-fma-groove-omnidialer

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.

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

This PR adds Groove OmniDialer as a Fleet-maintained app (macOS), wiring it into the maintained apps dataset (inputs + outputs + apps list) and adding a corresponding Software page icon so it renders correctly in the UI.

Changes:

  • Add Groove OmniDialer to the maintained apps catalog (apps.json) and Homebrew input definition.
  • Add the macOS maintained-app output manifest (darwin.json) including exists/patched queries, installer URL+sha256, and install/uninstall script refs.
  • Add a new frontend icon component and register it in the software-name-to-icon map.

Reviewed changes

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

Show a summary per file
File Description
frontend/pages/SoftwarePage/components/icons/index.ts Registers the new Groove OmniDialer icon and maps it to the normalized software name.
frontend/pages/SoftwarePage/components/icons/GrooveOmniDialer.tsx Adds the Groove OmniDialer icon component (SVG embedding a base64 PNG).
ee/maintained-apps/outputs/groove-omnidialer/darwin.json Adds the generated maintained-app manifest for macOS, including queries, installer metadata, and script refs.
ee/maintained-apps/outputs/apps.json Adds Groove OmniDialer to the maintained apps list surfaced by Fleet.
ee/maintained-apps/inputs/homebrew/groove-omnidialer.json Adds the Homebrew input definition for generating the macOS maintained-app output.

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

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