Skip to content

feat: default mermaid renderer to pele - #49

Open
kooksee wants to merge 3 commits into
masterfrom
feat/pele-default-renderer
Open

kooksee wants to merge 3 commits into
masterfrom
feat/pele-default-renderer

Conversation

@kooksee

@kooksee kooksee commented Nov 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

New sessions (and any settings that never stored a renderer) now render Mermaid with Pele by default. Explicitly saved native or beautiful choices are left alone.

Follow-on to the v0.24.0 Pele option — this only flips the default, not the engine switch itself.

Test plan

  • vitest for useMermaidSettings, SettingsModal, MermaidBlock (34 passed)
  • Clear Mermaid localStorage → diagrams use Pele; settings show Pele as default
  • With renderer: "native" already stored → stays native

kooksee and others added 3 commits November 4, 2026 21:28
Fresh installs and settings without an explicit renderer now use Pele;
explicit native/beautiful choices stay put.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Hard-coded 0.22/0.23 fixtures broke after the 0.24.0 bump because
IsNewer compares against the live version constant.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Code Metrics Report

Coverage Code to Test Ratio Test Execution Time
52.9% 1:0.4 58s

Code coverage of files in pull request scope (78.0%)

Files Coverage
frontend/src/components/MarkdownViewer.tsx 78.2%
frontend/src/components/SettingsModal.tsx 48.5%
frontend/src/hooks/useMermaidSettings.ts 92.5%
frontend/src/utils/mermaidTheme.ts 80.8%

Reported by octocov

This branch has not been deployed

No deployments
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