Skip to content

FMA - claude-devtools - #47018

Merged
allenhouchins merged 5 commits into
mainfrom
fma-claude-devtools
Jun 7, 2026
Merged

FMA - claude-devtools#47018
allenhouchins merged 5 commits into
mainfrom
fma-claude-devtools

Conversation

@harrisonravazzolo

@harrisonravazzolo harrisonravazzolo commented Jun 7, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features
    • Claude DevTools for macOS (version 0.5.0) is now available as a managed application
    • Automatic detection, installation, and update management capabilities
    • Application icon added to the software catalog interface for easy identification
    • Full lifecycle support including installation, updates, and uninstallation

fleet-release
fleet-release previously approved these changes Jun 7, 2026
fleet-release
fleet-release previously approved these changes Jun 7, 2026
@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 (309f1e4) to head (edcec52).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...s/SoftwarePage/components/icons/ClaudeDevtools.tsx 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #47018      +/-   ##
==========================================
- Coverage   67.04%   67.04%   -0.01%     
==========================================
  Files        2877     2878       +1     
  Lines      225137   225139       +2     
  Branches    11779    11637     -142     
==========================================
+ Hits       150933   150934       +1     
- Misses      60533    60534       +1     
  Partials    13671    13671              
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 marked this pull request as ready for review June 7, 2026 17:36
@allenhouchins
allenhouchins requested a review from a team as a code owner June 7, 2026 17:36
Copilot AI review requested due to automatic review settings June 7, 2026 17:36
@fleet-release
fleet-release requested a review from eashaw June 7, 2026 17:36

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

@allenhouchins
allenhouchins merged commit ca26cf2 into main Jun 7, 2026
30 of 31 checks passed
@allenhouchins
allenhouchins deleted the fma-claude-devtools branch June 7, 2026 17:36
@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: d4a9069b-8bbb-4237-9982-8aa9a1889510

📥 Commits

Reviewing files that changed from the base of the PR and between 309f1e4 and edcec52.

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

Walkthrough

This PR introduces Claude DevTools as a managed macOS application. It adds Homebrew input metadata, creates a complete application manifest in the enterprise catalog with version detection via SQL queries and inline Bash scripts for controlled installation and uninstallation, updates the main app registry with the Claude DevTools entry, and registers a new React icon component on the frontend with integration into the software-to-icon mapping.

Possibly related PRs

  • fleetdm/fleet#46734: Adds a new macOS app definition with Homebrew input, manifest, app registry entry, and new icon component registration following the same pattern.
  • fleetdm/fleet#47010: Adds a new macOS app integration and extends the SOFTWARE_NAME_TO_ICON_MAP icon registry in the same frontend location.
  • fleetdm/fleet#46495: Adds a new maintained macOS app by extending the app catalog and icon mapping in the same files.
✨ 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 fma-claude-devtools

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

Adds a new Fleet-maintained app (FMA) entry for Claude DevTools on macOS and wires up a dedicated icon so it can render in the Software UI.

Changes:

  • Added a new claude-devtools maintained app definition (inputs + generated outputs) for darwin, including install/uninstall scripts and osquery queries.
  • Registered ClaudeDevtools icon component and mapped the software name to that icon in the Software page icon map.
  • Added the new app to the maintained apps catalog (apps.json).

Reviewed changes

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

Show a summary per file
File Description
frontend/pages/SoftwarePage/components/icons/index.ts Imports the new icon and maps a software name to it for UI rendering.
frontend/pages/SoftwarePage/components/icons/ClaudeDevtools.tsx Adds the Claude DevTools icon component (embedded image).
ee/maintained-apps/outputs/claude-devtools/darwin.json Generated FMA manifest for macOS (queries, installer URL, scripts, hash).
ee/maintained-apps/outputs/apps.json Adds Claude DevTools to the maintained apps catalog.
ee/maintained-apps/inputs/homebrew/claude-devtools.json Source input used to generate the maintained app output.

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

Comment on lines 382 to 385
"citrix workspace": CitrixWorkspace,
claude: Claude,
"claude-devtools": ClaudeDevtools,
cleanmymac_5: CleanMyMac,
"slug": "claude-devtools/darwin",
"platform": "darwin",
"unique_identifier": "com.claudecode.context",
"description": "Claude DevTools is an app to visualise and analyse Claude Code session executions."
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.

4 participants