feat(agents): simple redesign Connect button and share modal

[rafavalls]

• Replace lightning icon with overlapping Cursor/Claude branded icons on Connect button
• Redesign client action cards with colored backgrounds and white logos
• Replace Windsurf card with generic Copy URL action
• Make typegen section collapsible by default
• Fix mode selection highlighting with state-driven bg-accent/30
• Reduce radio card padding and gap for tighter layout

---

Summary by cubic

Redesigned the Connect button and share modal for agents to make installs clearer and faster, with branded icons and a collapsible typed client section. Improves Cursor and Claude flows, simplifies sharing, and fixes mode selection highlighting.

• New Features

  • Connect button uses overlapping Cursor + Claude icons.
  • Client action cards redesigned with colored backgrounds and white logos.
  • Replaced Windsurf card with a generic “Copy URL” action.
  • Typed client (typegen) section is now collapsible, with quick copy for command and env vars.

• Bug Fixes

  • Selected mode uses state-driven highlighting (bg-accent/30); reduced radio card padding/gap for a tighter layout.

^{Written for commit fd7f767. Summary will update on new commits.}

[github-actions]

🧪 Benchmark

Should we run the Virtual MCP strategy benchmark for this PR?

React with 👍 to run the benchmark.

Reaction	Action
👍	Run quick benchmark (10 & 128 tools)

Benchmark will run on the next push after you react.

[github-actions]

Release Options

Should a new version be published when this PR is merged?

React with an emoji to vote on the release type:

Reaction	Type	Next Version
👍	Prerelease	2.120.2-alpha.1
🎉	Patch	2.120.2
❤️	Minor	2.121.0
🚀	Major	3.0.0

Current version: 2.120.1

Deployment

• Deploy to production (triggers ArgoCD sync after Docker image is published)

[cubic-dev-ai]

1 issue found across 2 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="apps/mesh/src/web/components/details/virtual-mcp/virtual-mcp-share-modal.tsx">

<violation number="1" location="apps/mesh/src/web/components/details/virtual-mcp/virtual-mcp-share-modal.tsx:280">
P2: Collapsing the Typegen section unmounts TypegenSectionInner, which clears the generated API key state. If a user closes the section after generating a key, they permanently lose it despite the UI saying it won’t be shown again. Keep the inner section mounted and just hide it so the key isn’t discarded.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

P2: Collapsing the Typegen section unmounts TypegenSectionInner, which clears the generated API key state. If a user closes the section after generating a key, they permanently lose it despite the UI saying it won’t be shown again. Keep the inner section mounted and just hide it so the key isn’t discarded.

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At apps/mesh/src/web/components/details/virtual-mcp/virtual-mcp-share-modal.tsx, line 280:

<comment>Collapsing the Typegen section unmounts TypegenSectionInner, which clears the generated API key state. If a user closes the section after generating a key, they permanently lose it despite the UI saying it won’t be shown again. Keep the inner section mounted and just hide it so the key isn’t discarded.</comment>

<file context>
@@ -294,60 +217,78 @@ function TypegenSectionInner({ virtualMcp }: { virtualMcp: VirtualMCPEntity }) {
+          className={cn("transition-transform", open && "rotate-180")}
+        />
+      </button>
+      {open && (
+        <div className="pt-1 pb-1">
+          <Suspense
</file context>