Merge pull request #425 from brendanator/return-parse-error

feat: return mermaid parse error in 400 response
jihchi · Nov 4, 2024 · 99eefb1 · 99eefb1
2 parents 845fd06 + fae56a6
commit 99eefb1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/node_modules/renderImgOrSvg.js b/src/node_modules/renderImgOrSvg.js
diff --git a/src/static/mermaid.mjs b/src/static/mermaid.mjs
@@ -1,7 +1,7 @@
 import mermaid from '../../node_modules/mermaid/dist/mermaid.esm.min.mjs';
 
-function isSyntaxErrorFromMermaid(code) {
-  return code.includes('Syntax error in graph') && code.includes('error-icon');
+function isUnknownDiagramError(code) {
+  return code.includes('UnknownDiagramError');
 }
 
 function setBgColor(svgElement, bgColor) {
@@ -54,10 +54,10 @@ async function render(definition, config, bgColor, size) {
     }
   } catch (error) {
     console.error('Failed to render', error);
-    if (isSyntaxErrorFromMermaid(error.toString())) {
-      throw new Error('Syntax error in graph');
+    if (isUnknownDiagramError(error.toString())) {
+      throw new Error('Unknown diagram error');
     }
-    throw error;
+    throw error.message;
   }
 }