test in app builder encouraged in server card

[ignaciojimenezr]

Screen.Recording.2026-03-24.at.10.33.32.PM.mov

[railway-app]

🚅 Environment inspector-pr-1666 in triumphant-alignment has no services deployed.

1 service not affected by this PR

• mcp-inspector

[chelojimenez]

✅ Snyk checks have passed. No issues have been found so far.

Status	Scan Engine	Critical	High	Medium	Low	Total (0)
✅	Open Source Security	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 66cae5cd-9e02-44f7-b9fc-1c602aa2fc1a

📥 Commits

Reviewing files that changed from the base of the PR and between 0069520 and 6d2b333.

📒 Files selected for processing (4)

• mcpjam-inspector/client/src/components/connection/shared/AuthenticationSection.tsx
• mcpjam-inspector/client/src/components/mcp-sidebar.tsx
• mcpjam-inspector/client/src/components/sidebar/nav-main.tsx
• mcpjam-inspector/client/src/hooks/__tests__/use-ui-app-servers.test.tsx

✅ Files skipped from review due to trivial changes (1)

• mcpjam-inspector/client/src/components/connection/shared/AuthenticationSection.tsx

🚧 Files skipped from review as they are similar to previous changes (2)

• mcpjam-inspector/client/src/hooks/tests/use-ui-app-servers.test.tsx
• mcpjam-inspector/client/src/components/mcp-sidebar.tsx

---

Walkthrough

This PR introduces a new hook, useUiAppServers, which asynchronously fetches per-server tool metadata with a 5s per-server timeout to derive appServerNames and resolvedServerNames. The sidebar App Builder "bubble" feature and its related props/state were removed. ServerConnectionCard gained onOpenAppBuilder and isUiCapabilityResolved props and a conditional "Test in app builder" action. ServersTab and App were updated to pass an onOpenAppBuilder callback that selects a server and navigates to the app-builder tab. Tests for the UI-capability resolution behavior were added.

---

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]

🧹 Nitpick comments (1)

mcpjam-inspector/client/src/hooks/__tests__/use-ui-app-servers.test.tsx (1)

25-42: Migrate to the shared createServer() factory from client/src/test/factories.ts.

The test defines a local createServer() that duplicates the shared factory. Migrate to the shared factory and pass name as an override property rather than a positional parameter. Alternatively, use createConnectedServer() for test-specific defaults. This ensures fixture consistency across the codebase.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@mcpjam-inspector/client/src/hooks/__tests__/use-ui-app-servers.test.tsx`
around lines 25 - 42, Replace the local createServer factory with the shared
factory: remove the local function createServer and import the shared
createServer from the test factories file, calling it with an overrides object
that includes name (e.g. createServer({ name, ...otherOverrides })) instead of a
positional name arg; for tests that need the connected defaults use the shared
createConnectedServer helper (or pass connection-related overrides) so fixtures
remain consistent with ServerWithName and existing test factories.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@mcpjam-inspector/client/src/hooks/__tests__/use-ui-app-servers.test.tsx`:
- Around line 25-42: Replace the local createServer factory with the shared
factory: remove the local function createServer and import the shared
createServer from the test factories file, calling it with an overrides object
that includes name (e.g. createServer({ name, ...otherOverrides })) instead of a
positional name arg; for tests that need the connected defaults use the shared
createConnectedServer helper (or pass connection-related overrides) so fixtures
remain consistent with ServerWithName and existing test factories.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: df8211c0-6c1f-4f59-8673-8790fe69e23b

📥 Commits

Reviewing files that changed from the base of the PR and between 74bc308 and 0069520.

📒 Files selected for processing (8)

• mcpjam-inspector/client/src/App.tsx
• mcpjam-inspector/client/src/components/ServersTab.tsx
• mcpjam-inspector/client/src/components/connection/ServerConnectionCard.tsx
• mcpjam-inspector/client/src/components/connection/__tests__/ServerConnectionCard.test.tsx
• mcpjam-inspector/client/src/components/mcp-sidebar.tsx
• mcpjam-inspector/client/src/components/sidebar/nav-main.tsx
• mcpjam-inspector/client/src/hooks/__tests__/use-ui-app-servers.test.tsx
• mcpjam-inspector/client/src/hooks/use-ui-app-servers.ts

[prathmeshpatel]

Suggested change

	<span>Test in app builder</span>
	<span>Test in App Builder</span>