fix: Add plugin document rendering and MCP preview support#938
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | b687fd4 | May 14 2026, 10:11 AM |
|
Claude finished @ved015's task in 55s —— View job Review Complete
Overview: Adds plugin document rendering support with dedicated parsers for Codex sessions/saves, Amp threads, and OpenClaw conversations, plus UI components for previews and modal content. Issues found: The previous review identified a logic bug where if (!document) return null
if (pluginDocument) {
return <PluginContent parsed={pluginDocument} />
}
if (!contentType) return nullNo additional bugs, security issues, or logic errors found. The regex patterns are safe from ReDoS (non-greedy quantifiers with clear boundaries), React's auto-escaping handles XSS concerns, and null/undefined handling is thorough throughout. Score: 9/10 Clean implementation with good separation between parsing logic and UI components. The only improvement opportunity would be adding unit tests for the parser edge cases, but that's not a blocker for shipping. |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | b687fd4 | Commit Preview URL Branch Preview URL |
May 14 2026, 10:13 AM |
There was a problem hiding this comment.
Overview: Adds plugin document rendering support with dedicated parsers for Codex sessions/saves, Amp threads, and OpenClaw conversations, plus UI components for previews and modal content.
Issues found:
- One logic bug in
DocumentContentwhere plugin documents with unrecognized base types won't render (see inline comment with fix)
Score: 8/10
The implementation is well-structured with clean separation between parsing logic and UI components. The regex patterns are efficient and non-vulnerable to ReDoS. React's auto-escaping handles XSS concerns for user content. The one issue is a straightforward fix to the conditional order.
Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Fix missing closing brace and remove duplicate code in DocumentContent - Add type assertion for summary prop to satisfy TypeScript Co-Authored-By: Vedant Mahajan <undefined@users.noreply.github.com> Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Refactor summary prop formatting for DocumentSummary component.
Implement comprehensive plugin document rendering support including MCP previews and plugin specific content handling.