@1agh/maude 0.53.0 → 0.53.1

package/apps/studio/bin/_import-asset.mjs

@@ -360,12 +360,7 @@ export function sanitizeSvgAllowlist(svgText) {
   const window = new Window();
   const doc = new window.DOMParser().parseFromString(svgText, 'image/svg+xml');
   const root = doc.documentElement;
-  if (
-    !root ||
-    root.tagName !== 'svg' ||
-    root.namespaceURI !== SVG_NS ||
-    doc.querySelector('parsererror')
-  ) {
+  if (root?.tagName !== 'svg' || root.namespaceURI !== SVG_NS || doc.querySelector('parsererror')) {
     throw new ImportAssetError(3, 'SVG failed to parse');
   }
   walkAllowlist(root);

package/apps/studio/bin/_import-brand.mjs

@@ -355,7 +355,7 @@ export function hardenBrandLogoSvg(sanitizedSvgText) {
   const window = new Window();
   const doc = new window.DOMParser().parseFromString(sanitizedSvgText, 'image/svg+xml');
   const root = doc.documentElement;
-  if (!root || root.tagName !== 'svg' || root.namespaceURI !== SVG_NS) {
+  if (root?.tagName !== 'svg' || root.namespaceURI !== SVG_NS) {
     throw new ImportBrandError(3, 'input is not a valid sanitized SVG document');
   }
 

package/apps/studio/bin/_video-playwright.mjs

@@ -354,7 +354,11 @@ async function frameStepCapture({
     });
   }
 
-  const dump = dumpDir ? (mkdirSync(dumpDir, { recursive: true }), dumpDir) : null;
+  // A comma operator hid the side effect here; the directory has to exist
+  // before anything writes a frame into it, so the effect is the point and
+  // deserves its own line.
+  if (dumpDir) mkdirSync(dumpDir, { recursive: true });
+  const dump = dumpDir ?? null;
   const framePaths = [];
   const t0 = Date.now();