Render Nova connector setup cards#1071
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | fd98c5e | Jun 09 2026, 01:48 PM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | fd98c5e | Commit Preview URL Branch Preview URL |
Jun 09 2026, 01:50 PM |
|
Claude finished @ishaanxgupta's task in 1m 51s —— View job Review Complete
Summary: This PR adds custom UI rendering for Nova connector setup cards in chat messages. The code is well-structured, follows established patterns for API key fetching, and handles edge cases properly (error states, image fallbacks, deduplication of connector cards). No bugs, security issues, or logic errors found. Score: 10/10 — Ready to ship. |
There was a problem hiding this comment.
Overview: Adds custom UI rendering for Nova connector tool results, displaying integration setup cards with status indicators, install steps, and API key generation.
Issues found: None — this looks good to ship.
The implementation is clean and follows established patterns:
- API key fetching (
/v3/auth/key?client=...) matches the existing pattern used inselect-spaces-modal.tsx,integrations-view.tsx, andplugins-detail.tsx - Proper error handling throughout with loading/error states
- The
shouldSkipNovaConnectorPartdeduplication logic correctly prioritizesprepareNovaPluginSetupovergetNovaConnectorSetupto avoid duplicate cards - Image fallback handling uses
data-fallbackAppliedflag to prevent infinite error loops - Accessibility is handled with
type="button",aria-expanded, and proper focus states - CSS changes improve code block contrast without breaking existing styles
Score: 10/10
Summary
/v3/auth/key?client=...endpoint from the UI