feat(web): overhaul integrations page with featured hero, category rails and MCP setup modal#979
feat(web): overhaul integrations page with featured hero, category rails and MCP setup modal#979MaheshtheDev wants to merge 2 commits into
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | ebd1da2 | May 22 2026, 08:40 PM |
How to use the Graphite Merge QueueAdd the label Main to this PR to add it to the merge queue. You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
supermemory-app | ebd1da2 | May 22 2026, 08:41 PM |
…ils and MCP setup modal - Rebuild the integrations catalog as horizontally-scrollable category rails (MCP, Plugins, Knowledge bases, Apps & extensions) with header chevrons - Add an auto-rotating Featured hero with problem-led headlines, backdrop imagery and a gradient FEATURED badge - Surface all 9 MCP clients as individual cards that open a setup modal styled like the plugin connect flow (reuses MCPSteps embedded variant) - Align MCP setup steps and the Claude Desktop timeline to the design system - Merge the Integrations and Connections settings tabs into one entry
MaheshtheDev
force-pushed
the
worktree-nova-integrations-overhaul
branch
from
df1962c to
b4cae4a
Compare
Remove unused imports (Sparkles, PluginInfo), fix import type style, wrap update function in useCallback with deps, memoize counts object, and add a11y suppression for event propagation wrapper. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
Claude finished @MaheshtheDev's task in 2m 5s —— View job Code Review Complete
Overview: Overhauls the integrations page with a featured hero carousel, horizontally-scrollable category rails, MCP client setup modals, and unified plugin/connector management. Issues found: None — this looks good to ship. Score: 9/10 The code is well-structured with proper error handling, clean state management using React Query and nuqs, and correct loading states during plugin connection and OAuth flows. The refactored |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview: Overhauls the integrations page with a featured hero carousel, horizontally-scrollable category rails, MCP client setup modals, and unified plugin/connector management.
Issues found: None — this looks good to ship.
The code is well-structured with:
- Proper error handling via toast notifications and mutation error callbacks
- Clean state management using React Query and nuqs for URL state
- Defensive JSON parsing with try/catch in
connectedPluginsmemo - Correct loading states during plugin connection and OAuth flows
- Appropriate type guards before type assertions (e.g.,
item.action.typechecks)
Minor observations (not blocking):
- Empty catch block at line 1193 in
integrations-view.tsxsilently ignores malformed metadata — this is acceptable defensive parsing but could optionally log in dev mode - The
connectionsByProvidercast at line 1207 assumes provider values matchConnectorProvider— unknown providers silently drop, which is fine for extensibility
Score: 9/10
Clean implementation with solid error handling and state management. The refactored McpCodeBlock component is a nice DRY improvement over the duplicated copy-button patterns. Ready to merge.
1 participant
(MCP, Plugins, Knowledge bases, Apps & extensions) with header chevrons
imagery and a gradient FEATURED badge
styled like the plugin connect flow (reuses MCPSteps embedded variant)