Added REAPER to FMA

[desmonet]

Addition of REAPER to the Fleet-Maintained apps list.

Product information: https://www.reaper.fm/
Homebrew formula: https://formulae.brew.sh/cask/reaper#default

Thanks!

@allenhouchins

Summary by CodeRabbit

• New Features
  • Added REAPER audio production software support for macOS, including installation, uninstallation, and management capabilities through the platform.

[claude]

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

[codecov]

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 66.91%. Comparing base (b9e58d5) to head (b399618).
⚠️ Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
...end/pages/SoftwarePage/components/icons/Reaper.tsx	50.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #46428      +/-   ##
==========================================
+ Coverage   66.90%   66.91%   +0.01%     
==========================================
  Files        2834     2837       +3     
  Lines      224862   225011     +149     
  Branches    11512    11614     +102     
==========================================
+ Hits       150435   150562     +127     
- Misses      60772    60785      +13     
- Partials    13655    13664       +9     

Flag	Coverage Δ
frontend	56.69% <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 Sentry.
📢 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.

[coderabbitai]

Walkthrough

This PR integrates REAPER, a macOS application, into Fleet's maintained apps system. It defines Homebrew input metadata, registers REAPER in the app registry, creates a Darwin-specific installation manifest with embedded bash scripts for DMG-based installation and uninstall with GUI session awareness, and adds a corresponding frontend SVG icon component registered in the software icon map.

Possibly related PRs

• fleetdm/fleet#46264: Follows the same maintained-app pattern (registry entry and icon map registration) for a different app (GoToMeeting).
• fleetdm/fleet#45858: Similar macOS maintained-apps pipeline extension with install/uninstall scripts and icon registration (Secretive).
• fleetdm/fleet#45174: Parallel maintained app addition pattern updating the same icon map in SOFTWARE_NAME_TO_ICON_MAP.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is minimal and lacks completion of the required template checklist. No sections are checked, testing details are absent, and migration/configuration validation is missing.	Complete the PR template by checking applicable boxes in the checklist (e.g., changes files, testing, validation) and provide necessary details for review.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Added REAPER to FMA' is clear, concise, and accurately describes the main change: adding REAPER app to the Fleet-Maintained apps list.
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

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 2

🤖 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/reaper/darwin.json`:
- Line 19: The install script currently moves existing REAPER.app to a backup
(TMPDIR/REAPER.app.bkp) then blindly copies the new bundle and leaves the app
missing on copy failure; update the replacement logic around the sudo mv and
sudo cp -R calls to (1) introduce a backup_path variable (e.g.,
backup_path="$TMPDIR/REAPER.app.bkp"), (2) move APPDIR/REAPER.app to that
backup_path if present, (3) attempt sudo cp -R "$TMPDIR/REAPER.app" "$APPDIR"
and if that cp fails immediately restore the backup by moving backup_path back
to "$APPDIR/REAPER.app" and exit non-zero, and (4) on successful copy remove the
backup; also consider enabling strict failure handling (set -euo pipefail) at
the top of the script to fail fast.

In `@frontend/pages/SoftwarePage/components/icons/Reaper.tsx`:
- Around line 5-12: The SVG lacks a viewBox so changing the root size via props
can crop the inner 32×32 <image>; update the Reaper component's root <svg>
element to include viewBox="0 0 32 32" before spreading {...props} (i.e. set
viewBox on the <svg> in the Reaper function so scaling works while still
allowing width/height to be overridden by props).

🪄 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: b6e24a81-3b7c-4152-8147-a8e6fd5b0566

📥 Commits

Reviewing files that changed from the base of the PR and between b42a154 and fb02faf.

⛔ Files ignored due to path filters (1)

• website/assets/images/app-icon-reaper-60x60@2x.png is excluded by !**/*.png

📒 Files selected for processing (5)

• ee/maintained-apps/inputs/homebrew/reaper.json
• ee/maintained-apps/outputs/apps.json
• ee/maintained-apps/outputs/reaper/darwin.json
• frontend/pages/SoftwarePage/components/icons/Reaper.tsx
• frontend/pages/SoftwarePage/components/icons/index.ts

[coderabbitai]

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Restore the existing app if the replacement copy fails.

The install script moves /Applications/REAPER.app to a backup and then copies the new bundle into place without any rollback. If that copy fails, the machine is left with no app in /Applications, which turns a transient install error into a broken state. Guard the replace step with fail-fast error handling and restore the backup on failure.

Suggested hardening

set -euo pipefail

backup_path="$TMPDIR/REAPER.app.bkp"

if [ -d "$APPDIR/REAPER.app" ]; then
  sudo mv "$APPDIR/REAPER.app" "$backup_path"
fi

if ! sudo cp -R "$TMPDIR/REAPER.app" "$APPDIR"; then
  if [ -d "$backup_path" ]; then
    sudo mv "$backup_path" "$APPDIR/REAPER.app"
  fi
  exit 1
fi

if [ -d "$backup_path" ]; then
  sudo rm -rf "$backup_path"
fi

🤖 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/reaper/darwin.json` at line 19, The install script
currently moves existing REAPER.app to a backup (TMPDIR/REAPER.app.bkp) then
blindly copies the new bundle and leaves the app missing on copy failure; update
the replacement logic around the sudo mv and sudo cp -R calls to (1) introduce a
backup_path variable (e.g., backup_path="$TMPDIR/REAPER.app.bkp"), (2) move
APPDIR/REAPER.app to that backup_path if present, (3) attempt sudo cp -R
"$TMPDIR/REAPER.app" "$APPDIR" and if that cp fails immediately restore the
backup by moving backup_path back to "$APPDIR/REAPER.app" and exit non-zero, and
(4) on successful copy remove the backup; also consider enabling strict failure
handling (set -euo pipefail) at the top of the script to fail fast.

[coderabbitai]

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Add a viewBox before forwarding size props.

...props can override the root SVG size, but the inner <image> stays 32×32. Without a viewBox, any non-32 render size can crop the icon instead of scaling it.

Suggested fix

 const Reaper = (props: SVGProps<SVGSVGElement>) => (
-  <svg xmlns="http://www.w3.org/2000/svg" width={32} height={32} {...props}>
+  <svg
+    xmlns="http://www.w3.org/2000/svg"
+    width={32}
+    height={32}
+    viewBox="0 0 32 32"
+    {...props}
+  >

🤖 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 `@frontend/pages/SoftwarePage/components/icons/Reaper.tsx` around lines 5 - 12,
The SVG lacks a viewBox so changing the root size via props can crop the inner
32×32 <image>; update the Reaper component's root <svg> element to include
viewBox="0 0 32 32" before spreading {...props} (i.e. set viewBox on the <svg>
in the Reaper function so scaling works while still allowing width/height to be
overridden by props).

[allenhouchins]

@desmonet Please make the necessary changes to get this to pass the Test Fleet Maintained Apps - Darwin validation check and resubmit.