Skip to content

feat(den): simplify MCP action cards - #2863

Merged
reachjalil merged 2 commits into
different-ai:devfrom
reachjalil:feature/mcp-config-details-disclosure
Jul 16, 2026
Merged

feat(den): simplify MCP action cards#2863
reachjalil merged 2 commits into
different-ai:devfrom
reachjalil:feature/mcp-config-details-disclosure

Conversation

@reachjalil

@reachjalil reachjalil commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

Follow-up to #2851 that keeps imported MCP recovery clear without making marketplace cards visually heavy when a plugin declares several MCP servers.

  • group multiple unresolved MCP requirements behind one compact MCPs require action disclosure
  • show a clean provider row with a small icon-led Connect action for each MCP after expansion
  • keep a single unresolved MCP visible without adding an unnecessary disclosure step
  • prevent plugin cards from stretching to match taller cards in the marketplace grid
  • collapse the plugin-declared URL and recovery explanation behind a quiet per-connection Details disclosure
  • preserve the existing admin/member actions and authentication recovery behavior from fix(den): recover imported MCP authentication setup #2851

Why

#2851 intentionally exposes the configuration provenance needed to recover imported MCP authentication. Showing every provider, URL, and explanation at once makes plugin cards grow quickly and becomes difficult to scan as the number of MCP requirements increases. This change uses progressive disclosure so the connection actions and operational details remain available without dominating the default view.

Scope

Den marketplace presentation only. This does not change MCP discovery, credentials, OAuth, connection ownership, readiness calculation, schemas, or runtime behavior.

UI proof

Multiple unresolved MCPs use one compact summary by default, and the marketplace grid no longer stretches the neighboring card.

Collapsed MCP action summary

Expanding the summary reveals a clean provider row and small icon-led Connect action for each MCP.

Expanded MCP action rows

Validation

  • pnpm --dir ee/apps/den-web exec bun test tests/marketplace-mcp-readiness.test.ts — 12 passed
  • pnpm --dir ee/apps/den-web typecheck — passed
  • git diff --check — passed

The default Den web test command was also attempted, but two unrelated observability tests could not resolve the freshly installed @openwork-ee/utils/observability build artifact. The focused MCP readiness suite above passed independently.

Manual review

  • Marketplace plugin cards remain compact and do not stretch to match taller grid neighbors.
  • Multiple unresolved MCPs are collapsed under MCPs require action by default.
  • Expanding the MCP summary reveals one compact Connect action per provider.
  • Details remains available independently for each requirement.

@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-landing Ready Ready Preview, Comment, Open in v0 Jul 16, 2026 11:55pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
openwork-diagnostics Skipped Skipped Jul 16, 2026 11:55pm

@vercel
vercel Bot temporarily deployed to Preview – openwork-diagnostics July 16, 2026 22:16 Inactive
@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

@reachjalil is attempting to deploy a commit to the Different AI Team on Vercel.

A member of the Team first needs to authorize it.

@reachjalil reachjalil changed the title feat(den): collapse MCP connection details feat(den): simplify MCP action cards Jul 16, 2026
@reachjalil
reachjalil marked this pull request as ready for review July 16, 2026 22:58
@reachjalil
reachjalil merged commit 0e86f2f into different-ai:dev Jul 16, 2026
3 of 8 checks passed
@reachjalil
reachjalil deleted the feature/mcp-config-details-disclosure branch July 16, 2026 22:58
@vercel
vercel Bot temporarily deployed to Preview – openwork-diagnostics July 16, 2026 23:48 Inactive
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.

1 participant