Add Devin Desktop as a macOS FMA (renamed from Windsurf) - #46996
Conversation
Rename Windsurf to Devin Desktop across EE manifests and frontend assets. Bump darwin version to 3.0.28 and update installer URL, SHA256, and install/uninstall script refs and contents (ee/maintained-apps/inputs/homebrew/* and ee/maintained-apps/outputs/*). Update ee/maintained-apps/outputs/apps.json to list Devin Desktop and remove the old Windsurf entry. Add a new React SVG icon component and register it in the icons index, and add the corresponding PNG asset (frontend/pages/SoftwarePage/components/icons/DevinDesktop.tsx, frontend/pages/SoftwarePage/components/icons/index.ts, website/assets/images/*).
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #46996 +/- ##
==========================================
- Coverage 67.04% 67.04% -0.01%
==========================================
Files 2867 2868 +1
Lines 225117 225119 +2
Branches 11743 11600 -143
==========================================
+ Hits 150924 150925 +1
- Misses 60522 60524 +2
+ Partials 13671 13670 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
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.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
WalkthroughThis PR adds Devin Desktop as the macOS app identity by updating the Homebrew input manifest to replace Windsurf with Devin Desktop, registering it in the apps registry, bumping the version to 3.0.28 with new installer and uninstaller scripts that rename the app from Windsurf.app to Devin.app and update related user directories, and adding a new frontend icon component with its registration in the icon mapping. Possibly related PRs
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
This PR renames the macOS Fleet-maintained app entry from Windsurf to Devin Desktop, updates the macOS installer metadata/scripts for the new Devin DMG version, and wires up a new software icon so the UI can display the Devin Desktop logo.
Changes:
- Add and register a new
DevinDesktopicon component and map it to the"devin desktop"software name key. - Introduce
devin-desktop/darwinmaintained-app inputs/outputs and update macOS installer URL/SHA/scripts for version3.0.28. - Update
ee/maintained-apps/outputs/apps.jsonto add Devin Desktop for macOS and remove the old Windsurf macOS entry.
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 Devin Desktop icon and maps the software name to the icon component. |
| frontend/pages/SoftwarePage/components/icons/DevinDesktop.tsx | Adds the Devin Desktop icon component (SVG wrapper with embedded base64 PNG). |
| ee/maintained-apps/outputs/devin-desktop/darwin.json | Updates macOS maintained-app output (version, installer URL/SHA, install/uninstall scripts). |
| ee/maintained-apps/outputs/apps.json | Adds Devin Desktop (macOS) entry and removes Windsurf (macOS) entry. |
| ee/maintained-apps/inputs/homebrew/devin-desktop.json | Renames the Homebrew input metadata from Windsurf to Devin Desktop and updates token/slug. |
Comments suppressed due to low confidence (2)
ee/maintained-apps/outputs/devin-desktop/darwin.json:20
- In the embedded uninstall script (
cb4ce7dc),sudo rmdir '~/.codeium/windsurf'likely never removes the intended directory: (1)~won’t expand when quoted, so it won’t point at the console user’s home, and (2)rmdironly removes empty directories (config dirs are typically non-empty). Prefer using the existingtrashhelper (which also expands~to/Users/$LOGGED_IN_USER) or an explicit expanded path withrm -rfif you truly want deletion.
ee/maintained-apps/outputs/apps.json:2761 - PR description says Windsurf is renamed to Devin Desktop across EE manifests, but
apps.jsonstill contains a Windows app entry namedWindsurf(slug": "windsurf/windows"). If the Windows maintained app should also be renamed, this entry (and the corresponding outputs/inputs) needs updating; otherwise the PR description should be narrowed to macOS-only rename to avoid confusion for consumers ofapps.json.
{
"name": "Windsurf",
"slug": "windsurf/windows",
"platform": "windows",
"unique_identifier": "Windsurf",
"description": "Windsurf is an agentic IDE powered by AI Flow paradigm."
},
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Rename Windsurf to Devin Desktop across EE manifests and frontend assets. Bump darwin version to 3.0.28 and update installer URL, SHA256, and install/uninstall script refs and contents (ee/maintained-apps/inputs/homebrew/* and ee/maintained-apps/outputs/). Update ee/maintained-apps/outputs/apps.json to list Devin Desktop and remove the old Windsurf entry. Add a new React SVG icon component and register it in the icons index, and add the corresponding PNG asset (frontend/pages/SoftwarePage/components/icons/DevinDesktop.tsx, frontend/pages/SoftwarePage/components/icons/index.ts, website/assets/images/).
Summary by CodeRabbit