@monoes/monodesign 1.2.11 → 1.2.13

package/skill/scripts/live-accept.mjs

@@ -76,7 +76,7 @@ Output (JSON):
   const svelteComponentManifest = found ? null : findSvelteComponentManifest(id, process.cwd());
 
   if (!found && !svelteComponentManifest) {
-    console.log(JSON.stringify({ handled: false, error: 'Session markers not found for id: ' + id }));
+    console.log(JSON.stringify({ handled: false, error: `Session markers not found for id: ${id}` }));
     process.exit(0);
   }
 
@@ -112,7 +112,7 @@ Output (JSON):
       };
     }
     if (result.carbonize) {
-      result.todo = 'REQUIRED before next poll: carbonize cleanup in ' + result.file + '. See reference/live.md "Required after accept".';
+      result.todo = `REQUIRED before next poll: carbonize cleanup in ${result.file}. See reference/live.md "Required after accept".`;
     }
     console.log(JSON.stringify({ handled: result.handled !== false, ...result }));
     return;
@@ -155,7 +155,7 @@ Output (JSON):
     // five-step checklist lives in reference/live.md (loaded once per
     // session); repeating it per-event would waste tokens.
     if (result.carbonize) {
-      result.todo = 'REQUIRED before next poll: carbonize cleanup in ' + relFile + '. See reference/live.md "Required after accept".';
+      result.todo = `REQUIRED before next poll: carbonize cleanup in ${relFile}. See reference/live.md "Required after accept".`;
     }
     // Scrub stash entries whose text appeared inside the just-replaced
     // original wrap block. The accept embodies those manual edits (wrap was
@@ -292,29 +292,29 @@ function buildCarbonizeReplacement({
     : 'style="display: contents"';
 
   const pushCarbonizeBody = (bodyIndent) => {
-    const bodyRestored = reindentContent(restored, indent, bodyIndent + '  ');
-    lines.push(bodyIndent + commentSyntax.open + ' monodesign-carbonize-start ' + id + ' ' + commentSyntax.close);
-    lines.push(bodyIndent + '<style data-monodesign-css="' + id + '">' + (isJsx ? '{`' : ''));
+    const bodyRestored = reindentContent(restored, indent, `${bodyIndent}  `);
+    lines.push(`${bodyIndent + commentSyntax.open} monodesign-carbonize-start ${id} ${commentSyntax.close}`);
+    lines.push(`${bodyIndent}<style data-monodesign-css="${id}">${isJsx ? '{`' : ''}`);
     for (const cssLine of cssContent) {
       lines.push(bodyIndent + cssLine.trimStart());
     }
     lines.push(bodyIndent + (isJsx ? '`}</style>' : '</style>'));
     if (paramValues && Object.keys(paramValues).length > 0) {
       lines.push(
-        bodyIndent + commentSyntax.open + ' monodesign-param-values ' + id + ': ' + JSON.stringify(paramValues) + ' ' + commentSyntax.close,
+        `${bodyIndent + commentSyntax.open} monodesign-param-values ${id}: ${JSON.stringify(paramValues)} ${commentSyntax.close}`,
       );
     }
-    lines.push(bodyIndent + commentSyntax.open + ' monodesign-carbonize-end ' + id + ' ' + commentSyntax.close);
-    lines.push(bodyIndent + '<div data-monodesign-variant="' + variantNum + '" ' + variantStyleAttr + '>');
+    lines.push(`${bodyIndent + commentSyntax.open} monodesign-carbonize-end ${id} ${commentSyntax.close}`);
+    lines.push(`${bodyIndent}<div data-monodesign-variant="${variantNum}" ${variantStyleAttr}>`);
     lines.push(...bodyRestored);
-    lines.push(bodyIndent + '</div>');
+    lines.push(`${bodyIndent}</div>`);
   };
 
   if (isJsx) {
     const wrapperStyle = 'style={{ display: "contents" }}';
-    lines.push(indent + '<div data-monodesign-carbonize="' + id + '" ' + wrapperStyle + '>');
-    pushCarbonizeBody(indent + '  ');
-    lines.push(indent + '</div>');
+    lines.push(`${indent}<div data-monodesign-carbonize="${id}" ${wrapperStyle}>`);
+    pushCarbonizeBody(`${indent}  `);
+    lines.push(`${indent}</div>`);
   } else {
     pushCarbonizeBody(indent);
   }
@@ -345,7 +345,7 @@ function handleAccept(id, variantNum, lines, targetFile, paramValues) {
 
   // Extract the chosen variant's inner content
   const variantContent = extractVariant(lines, block, variantNum);
-  if (!variantContent) return { handled: false, error: 'Variant ' + variantNum + ' not found' };
+  if (!variantContent) return { handled: false, error: `Variant ${variantNum} not found` };
   const originalContent = extractOriginal(lines, block);
 
   // Extract CSS block if present
@@ -382,7 +382,7 @@ function handleAccept(id, variantNum, lines, targetFile, paramValues) {
 
 function readSourceShadowPreviewMeta(content, id) {
   const escaped = escapeRegExp(id);
-  const wrapperRe = new RegExp('<[^>]+data-monodesign-variants=(["\'])' + escaped + '\\1[^>]*>');
+  const wrapperRe = new RegExp(`<[^>]+data-monodesign-variants=(["'])${escaped}\\1[^>]*>`);
   const match = String(content || '').match(wrapperRe);
   if (!match) return null;
   const tag = match[0];
@@ -395,7 +395,7 @@ function readSourceShadowPreviewMeta(content, id) {
 }
 
 function readHtmlAttr(tag, name) {
-  const match = String(tag || '').match(new RegExp('\\s' + escapeRegExp(name) + '\\s*=\\s*(["\'])(.*?)\\1'));
+  const match = String(tag || '').match(new RegExp(`\\s${escapeRegExp(name)}\\s*=\\s*(["'])(.*?)\\1`));
   if (!match) return null;
   return decodeHtmlAttr(match[2]);
 }
@@ -419,8 +419,8 @@ function decodeHtmlAttr(value) {
 function findMarkerBlock(id, lines) {
   let start = -1;
   let end = -1;
-  const startPattern = 'monodesign-variants-start ' + id;
-  const endPattern = 'monodesign-variants-end ' + id;
+  const startPattern = `monodesign-variants-start ${id}`;
+  const endPattern = `monodesign-variants-end ${id}`;
 
   for (let i = 0; i < lines.length; i++) {
     if (start === -1 && lines[i].includes(startPattern)) start = i;
@@ -504,7 +504,7 @@ function escapeRegExp(value) {
 }
 
 function isVariantEndMarkerLine(line, id) {
-  return new RegExp('monodesign-variants-end\\s+' + escapeRegExp(id) + '(?:\\s|--|\\*/|$)').test(line);
+  return new RegExp(`monodesign-variants-end\\s+${escapeRegExp(id)}(?:\\s|--|\\*/|$)`).test(line);
 }
 
 function hasVariantWrapperAttr(line, id) {
@@ -562,7 +562,7 @@ function stripStyleAndJoin(lines, block) {
  * Returns the inner string (may be empty), or null if not found.
  */
 function extractInnerByAttr(text, attrMatch) {
-  const openerRe = new RegExp('<([A-Za-z][A-Za-z0-9]*)\\b[^>]*' + attrMatch + '[^>]*>');
+  const openerRe = new RegExp(`<([A-Za-z][A-Za-z0-9]*)\\b[^>]*${attrMatch}[^>]*>`);
   const openMatch = text.match(openerRe);
   if (!openMatch) return null;
 
@@ -571,7 +571,7 @@ function extractInnerByAttr(text, attrMatch) {
 
   // Match any opener or closer of this tag name after innerStart.
   // (Does not match self-closing <TAG … />, which doesn't contribute to depth.)
-  const tagRe = new RegExp('<(?:/)?' + tagName + '\\b[^>]*>', 'g');
+  const tagRe = new RegExp(`<(?:/)?${tagName}\\b[^>]*>`, 'g');
   tagRe.lastIndex = innerStart;
 
   let depth = 1;
@@ -606,7 +606,7 @@ function extractOriginal(lines, block) {
  */
 function extractVariant(lines, block, variantNum) {
   const text = stripStyleAndJoin(lines, block);
-  const inner = extractInnerByAttr(text, 'data-monodesign-variant="' + variantNum + '"');
+  const inner = extractInnerByAttr(text, `data-monodesign-variant="${variantNum}"`);
   if (inner === null) return null;
   const result = inner.split('\n');
   // Collapse a lone empty leading/trailing line (common after string splice).
@@ -626,7 +626,7 @@ function extractVariant(lines, block, variantNum) {
  *      the lines between them.
  */
 function extractCss(lines, block, id) {
-  const styleAttr = 'data-monodesign-css="' + id + '"';
+  const styleAttr = `data-monodesign-css="${id}"`;
   let inStyle = false;
   const content = [];
 
@@ -747,7 +747,7 @@ function detectCommentSyntax(filePath) {
 // ---------------------------------------------------------------------------
 
 function findSessionFile(id, cwd) {
-  const marker = 'monodesign-variants-start ' + id;
+  const marker = `monodesign-variants-start ${id}`;
   const searchDirs = ['src', 'app', 'pages', 'components', 'public', 'views', 'templates', '.'];
   const seen = new Set();
 

package/skill/scripts/live-browser-dom.js

@@ -5,7 +5,8 @@
  * chrome mounting, lookup, focus, and picker helpers without depending on the
  * full overlay UI bundle.
  */
-(function (root) {
+((root) => {
+  // biome-ignore lint/suspicious/noRedundantUseStrict: served to the page as a classic <script> (not an ES module despite package type=module), so this directive is what enables strict mode
   'use strict';
   if (!root) return;
 
@@ -21,11 +22,11 @@
     const tagsToSkip = skipTags || new Set();
 
     function own(el) {
-      return el && (el.id?.startsWith(prefix) || el.closest?.('[id^="' + prefix + '"]'));
+      return el && (el.id?.startsWith(prefix) || el.closest?.(`[id^="${prefix}"]`));
     }
 
     function pickable(el) {
-      if (!el || el.nodeType !== 1) return false;
+      if (el?.nodeType !== 1) return false;
       if (tagsToSkip.has(String(el.tagName || '').toLowerCase())) return false;
       if (own(el)) return false;
       const r = el.getBoundingClientRect();
@@ -35,8 +36,8 @@
     function desc(el) {
       if (!el) return '';
       let s = el.tagName.toLowerCase();
-      if (el.id) s += '#' + el.id;
-      else if (el.classList.length) s += '.' + [...el.classList].slice(0, 2).join('.');
+      if (el.id) s += `#${el.id}`;
+      else if (el.classList.length) s += `.${[...el.classList].slice(0, 2).join('.')}`;
       return s;
     }
 
@@ -45,7 +46,7 @@
     }
 
     function makeFrozenAnchor(el) {
-      if (!el || !el.getBoundingClientRect) return null;
+      if (!el?.getBoundingClientRect) return null;
       const r = el.getBoundingClientRect();
       if (!rectIsUsableAnchor(r)) return null;
       const rect = {
@@ -99,7 +100,7 @@
         if (found) return found;
       }
       if (uiRoot?.querySelector) {
-        const found = uiRoot.querySelector('#' + cssId(id));
+        const found = uiRoot.querySelector(`#${cssId(id)}`);
         if (found) return found;
       }
       return doc.getElementById(id);

package/skill/scripts/live-browser-session.js

@@ -5,16 +5,17 @@
  * booting the full overlay UI. Served before live-browser.js and attached to
  * window.__MONODESIGN_LIVE_SESSION__.
  */
-(function (root) {
+((root) => {
+  // biome-ignore lint/suspicious/noRedundantUseStrict: served to the page as a classic <script> (not an ES module despite package type=module), so this directive is what enables strict mode
   'use strict';
 
   function createLiveBrowserSessionState({ prefix, storage, idFactory }) {
     if (!prefix) throw new Error('prefix required');
     const store = storage || root.localStorage;
-    const makeId = idFactory || function () { return Math.random().toString(16).slice(2, 10); };
-    const sessionKey = prefix + '-session';
-    const handledKey = sessionKey + '-handled';
-    const scrollKey = sessionKey + '-scroll';
+    const makeId = idFactory || (() => Math.random().toString(16).slice(2, 10));
+    const sessionKey = `${prefix}-session`;
+    const handledKey = `${sessionKey}-handled`;
+    const scrollKey = `${sessionKey}-scroll`;
     let checkpointRevision = 0;
     const owner = makeId();
 
@@ -43,7 +44,7 @@
     }
 
     function saveSession(session) {
-      if (!session || !session.id) return;
+      if (!session?.id) return;
       const payload = {
         ...session,
         checkpointRevision,
@@ -92,7 +93,7 @@
       const raw = safeRead(scrollKey);
       if (raw == null) return null;
       const n = parseFloat(raw);
-      return isFinite(n) ? n : null;
+      return Number.isFinite(n) ? n : null;
     }
 
     function clearScrollY() {