Repository navigation
chore(deps): bump mermaid from 11.17.2 to 12.2.0 - #3975
dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [mermaid](https://github.com/mermaid-js/mermaid) from 11.17.2 to 12.2.0. - [Release notes](https://github.com/mermaid-js/mermaid/releases) - [Commits](https://github.com/mermaid-js/mermaid/compare/mermaid@11.17.2...mermaid@12.2.0) --- updated-dependencies: - dependency-name: mermaid dependency-version: 12.2.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review completeClawSweeper finished reviewing this revision. The review result is being finalized. |
|
Found 2 test failures on Blacksmith runners: Failures
|
|
Codex review: blocked before merge. What this changesThis PR upgrades Mermaid from 11.17.2 to 12.2.0 and updates its transitive dependencies. Example: A skill contains the Mermaid diagram
Review scores
ProductKind: Maintenance · Worth it: No Merge readiness⛔ Blocked before merge - 6 items remain Keep open under the repository cleanup policy: main still uses Mermaid 11, and this PR introduces compatibility changes that conflict with recorded maintainer decisions. Priority: P2 Before merge
Findings
Provenance
Tests
Agent review detailsHow this fits togetherSkill documentation and plugin READMEs enter MarkdownPreview, which renders Mermaid fences as SVG and falls back to source text when rendering fails. flowchart TD
A[Skill documentation] --> C[MarkdownPreview]
B[Plugin README] --> C
C --> D[Mermaid renderer]
D --> E[Rendered SVG]
D --> F[Source fallback]
Technical reviewBest possible solution: Retain Mermaid 11.17.2 under the recorded browser policy and pursue any major upgrade through a separate compatibility-qualified change. Do we have a high-confidence way to reproduce the issue? The layout change follows from the unchanged initializer and candidate schema; the browser mismatch is supported by candidate parser source and the maintainer's explicitly limited earlier missing-API simulation, without a new runtime execution. Is this the best way to solve the issue? The version-only update is unsuitable under the recorded compatibility decision; pinning Dagre and classic appearance would address rendering drift but would not resolve the browser floor mismatch, and forcing a transitive KaTeX override was deliberately avoided in prior dependency maintenance. Full review comments:
Overall correctness: patch is incorrect AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against 6cc4aea3df7c. TestingProof path: unit tests only. SecurityNone. EvidenceWhat I checked:
Review metrics
LabelsLabel changes:
Label justifications:
Rating scale6/6 🦀 challenger crab · 5/6 🦞 diamond lobster · 4/6 🐚 platinum hermit · 3/6 🦐 gold shrimp · 2/6 🦪 silver shellfish · 1/6 🧂 unranked krab. Overall follows the weaker of proof and patch quality; ✨ marks media proof (a screenshot, video, or linked artifact) that directly shows the changed behavior. WorkflowClawSweeper edits this one comment on every review. Comment Reviewed October 16, 2026, 12:19 PM ET / 16:19 UTC. |
|
Closing this repeated Mermaid 12 upgrade under the existing owner decision to retain Mermaid 11 while ClawHub supports Safari 15. Mermaid 12 requires Safari 17.4+. The 12.2 usecase parser still enables Chevrotain grammar validation, whose duplicate-production check calls Object.groupBy. This dependency-only bump does not resolve the documented Safari 15 compatibility gap. This follows the prior owner decisions on #3863 and #3954. A future Mermaid 12 migration needs an explicit browser-compatibility decision and rendering proof; see the upstream migration notes. |
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
Bumps mermaid from 11.17.2 to 12.2.0.
Release notes
Sourced from mermaid's releases.
... (truncated)
Commits
1fb7869Merge pull request #8432 from mermaid-js/changeset-release/master34772caVersion Packages42292eaMerge pull request #8428 from mermaid-js/release/12.2.0e6c8059Wrong version set previously6c55f56Set mermaid version when Coloring by depth supported in Mindmap5a0d8cfFix missing parallel uploads788f396Update according to discussion with core-team members33d0a49Merge pull request #8411 from mermaid-js/renovate/npm-katex-vulnerability6538fdeUpdate dependency katex to ^0.18.0 [SECURITY]ba3022dMerge pull request #8398 from mermaid-js/pebr/12.1.0-tmpDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)