Skip to content

Node UX PR-1: NodeContentRenderer (readable markdown + code)#66

Merged
mvalancy merged 1 commit into
developfrom
feat/node-content-renderer
Jun 14, 2026
Merged

Node UX PR-1: NodeContentRenderer (readable markdown + code)#66
mvalancy merged 1 commit into
developfrom
feat/node-content-renderer

Conversation

@mvalancy

Copy link
Copy Markdown
Member

First slice of the "read files/diagrams at a useful scale, switch modes intuitively" rework (per the UX review).

The app had zero markdown/code rendering — a node's description only appeared as a plain <textarea> in a modal. NodeContentRenderer renders it as readable GitHub-flavored markdown with syntax-highlighted fenced code, at a fixed legible size (independent of canvas zoom).

  • Deps: react-markdown + remark-gfm + react-syntax-highlighter (Prism).
  • NodeContentRenderer.tsx — default export so callers React.lazy() it (keeps the heavy deps out of the main bundle until a node's contents are first viewed). compact flag for the on-canvas peek; dark-theme styling for headings/lists/tables/links/inline+fenced code.

Not wired to any view yet — PR-2 (docked inspector with Card·Contents·Diagram toggle) consumes it. Web build + typecheck clean.

🤖 Generated with Claude Code

…(PR-1)

First slice of the "read contents/diagrams at a useful scale" rework. The app
had NO markdown or code rendering — a node's `description` only showed as a
plain textarea in a modal. NodeContentRenderer turns it into readable,
GitHub-flavored markdown with syntax-highlighted fenced code, at a fixed legible
size (independent of canvas zoom).

- deps: react-markdown + remark-gfm + react-syntax-highlighter (Prism).
- NodeContentRenderer.tsx (default export so callers can React.lazy it → stays
  out of the main bundle until a node's contents are first viewed). compact flag
  for the on-canvas peek. Styled for the dark theme; inline vs fenced code,
  headings, lists, tables, links.

Not wired to any view yet — PR-2 (docked inspector) consumes it. Web build +
typecheck clean; renderer is lazy so the heavy deps aren't in the main chunk.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

🧪 Comprehensive Test Suite

  • Unit suites (Node 18.x & 20.x) — core, web, server, mcp-server: ✅ passed
  • Installer & deploy config: ✅ passed

Full-stack smoke gate runs in the CI workflow.

@mvalancy mvalancy merged commit 89b0a3a into develop Jun 14, 2026
16 checks passed
@mvalancy mvalancy deleted the feat/node-content-renderer branch June 14, 2026 20:41
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