Refine Codex and OpenCode OAuth integration status#1048
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | 3d50394 | Jun 04 2026, 09:39 PM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | 3d50394 | Commit Preview URL Branch Preview URL |
Jun 04 2026, 09:40 PM |
|
Claude finished @MaheshtheDev's task in 1m 17s —— View job Reviewing PR Changes
Overview: This PR refines the Codex/OpenCode OAuth integration to show plugins as "connected" only after the API key has actually been used by the plugin, replacing the dropdown with a modal-based connection detail view. Issues found: None — this looks good to ship. The implementation is clean and correct:
Score: 10/10 |
There was a problem hiding this comment.
Overview: This PR refines the Codex OAuth integration to only show plugins as "connected" after the API key has actually been used, replacing the dropdown with a modal-based connection detail view.
Issues found: None — this looks good to ship.
The implementation is clean:
- The
lastRequestfiltering correctly ensures only keys that have been used by the plugin are shown as connected - The
enabled === falsecheck properly handles explicitly disabled keys while allowing undefined/null to pass - Type safety is well maintained with the new
ListedApiKeytype - The
normalizePluginClientIdfunction correctly maps various plugin ID formats - Error handling is adequate (try/catch for JSON.parse, proper null checks)
- The modal implementation has proper accessibility (DialogTitle, aria-labels)
Score: 10/10
- Show plugins as Active (used) vs Finish setup (key, no use) vs Connect - Active pill becomes subtle ghost status; dedicated + button connects another agent - Support multiple connections per plugin with count + manage modal - Mirror pattern for connectors: subtle connected status + add-knowledge shortcut - Move Pro to subtle accent on name row, Docs alone top-right - Wrap items into a grid for any specific category filter (rails only on All) - Keep featured hero visible on the Active filter
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>






What changed
Why
Generating a plugin key from the web UI is only setup, not a completed plugin connection. The card should not show Connected until the plugin/OAuth flow actually authenticates and uses the key.