Agentic UI: Add Studio CLI toggle to application settings - #4302
Merged
Conversation
…-toggle # Conflicts: # apps/ui/src/components/settings-view/style.module.css
Collaborator
📊 Performance Test ResultsComparing 9447c48 vs trunk app-size
site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff) |
…-toggle # Conflicts: # apps/ui/src/components/settings-view/index.tsx # apps/ui/src/data/core/connectors/local/index.ts
sejas
approved these changes
Jul 22, 2026
sejas
left a comment
Member
There was a problem hiding this comment.
I confirmed the toggle installs and uninstalls Studio from the desktop app. I also tested that the toggle is disabled when I install studio from curl. We could add the cursor disallow for that case.
I also noticed that the App doesn't detect it if I install it form npm install -g wp-studio@latest. I suggest creating an issue as a follow-up.
The alert about using system node vs Electron node is difficult to understand for users, but that alert already exists in the Classic UI.
14 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related issues
How AI was used in this PR
Implemented with Claude Code from a scoped brief: it ported the
StudioCliSectionfrom #3979 to the auto-save settings model, extended the connector contract across the three connectors, and wrote the tests. Two behaviors were hardened based on manual testing feedback: a "cancel behind the native dialog leaves the toggle stale" bug (fixed by refetching the installed state after saving) and standalone-CLI handling (toggle disabled with an explanatory tooltip). All code was human-reviewed and manually tested on macOS.Proposed Changes
Adds a Studio CLI section to the new (agentic UI) application settings, so users can install or uninstall the
studioterminal command without switching back to the classic UI — the last preference from the classic settings that had no equivalent in the new experience.studiocommand is a standalone (curl) install, the app never installs over or uninstalls it — previously the toggle would appear to work but silently no-op. The toggle is now disabled in that case, with a tooltip explaining that the CLI is managed outside the app and pointing atstudio uninstall. This exposes the existing externally-managed detection from the installation managers over IPC (isStudioCliExternallyManaged).studio ui/ hosted) and Windows Store builds. This introduces a minimalgetAppGlobals()connector method (platform + Windows Store flag) for that gating, since this part of Agentic UI: Application Settings #3979 hadn't landed with the earlier settings PRs.Testing Instructions
Note: this PR adds a main-process IPC handler, so a full app restart (not hot reload) is required.
~/.local/bin/studioexists (where studio).studioresolves in a new terminal and the toggle stays on.~/.local/bin/studiopointing outside the app bundle): the toggle shows on but is disabled. Hovering it shows a tooltip pointing atstudio uninstall— it should open quickly (~200ms, faster than the app's default tooltip delay, since the disabled control gives no other feedback) and wrap as a compact multi-line bubble (max width 280px) instead of one long line.Pre-merge Checklist