@helping-ai-workflow/md2doc 3.10.0 → 3.12.0

package/lib/editor/convert-md.js

@@ -91,19 +91,24 @@ function stripMarker(sourceLines, blockKind) {
   if (blockKind === 'li') {
     const m = src[0].match(LI_MARKER);
     if (!m) return { content: [], ok: false };
-    // A MULTI-LINE li refuses. The previous code flattened continuations with
-    // replace(/^\s+/, ''), which is exactly the guess the code branch below
-    // refuses to make for an indented code block -- and it silently destroys
-    // whatever structure the continuation carried (a nested fence, an indented
-    // sub-block). Today the \u00a74.1 gate refuses a multi-line li as an operation
-    // target before this is ever reached, so this is defence in depth; dead
-    // code in this repo has a history of later gaining a caller, and it must
-    // not be a mine when it does.
-    if (src.length > 1) return { content: [], ok: false };
     let first = src[0].slice(m[0].length);
     const box = first.match(TASK_BOX);
     if (box) first = first.slice(box[0].length);
-    return { content: [first], ok: true };
+    // v3.11: a MULTI-LINE li (a hard-wrapped item, or a hard break) converts
+    // too. Its continuation lines belong to the item's own paragraph — any
+    // nested block would have made the run unsupported and refused before
+    // this — so each loses exactly the item's CONTENT column (the marker's
+    // width: anything deeper keeps its relative indent) or, on a lazy line
+    // indented less than that, all of its leading whitespace. That is not the
+    // old blanket replace(/^\s+/, '') the code branch below refuses to guess
+    // with: an over-indented line keeps its extra columns.
+    const col = m[0].length;
+    const rest = src.slice(1).map((l) => {
+      let k = 0;
+      while (k < col && l.charAt(k) === ' ') k++;
+      return k === col ? l.slice(col) : l.replace(/^\s+/, '');
+    });
+    return { content: [first].concat(rest), ok: true };
   }
 
   if (blockKind === 'heading') {
@@ -160,7 +165,9 @@ function emitAs(content, target, opts) {
     // A heading is one line by definition. Joining is the only lossless move
     // available; splitting would make one gesture produce two blocks.
     const hashes = '#'.repeat(Number(target.slice(1)));
-    const text = body.join(' ');
+    // A multi-line body (a converted hard-wrapped item) becomes one heading
+    // line; trailing hard-break spaces would otherwise land mid-heading.
+    const text = body.length > 1 ? body.map((l) => l.trim()).join(' ') : body.join(' ');
     // An EMPTY heading emits no trailing space, or converting '##' to a
     // heading of the same level would rewrite the line to '## '.
     return [text === '' ? prefix + hashes : prefix + hashes + ' ' + text];

package/lib/editor/inline-md.js

@@ -106,6 +106,14 @@
 
   function serializeAnchor(node, unsupported) {
     const href = node.getAttribute('href');
+    // A raw HTML anchor (`<a id="ref-x"></a>`, NO href attribute) is not a
+    // markdown link: emitting '[label]()' for it erased the anchor on the
+    // first edit of its block. Unsupported, so the block keeps its source.
+    // An empty href ('' — from '[x]()') is still a link.
+    if (href === null) {
+      unsupported.push('A');
+      return '';
+    }
     const text = node.textContent;
     // citation: <a href="#slug">[body]</a> -> [[body]]
     if (href && href.charAt(0) === '#' && /^\[.*\]$/.test(text)) {

package/lib/editor/list-md.js

@@ -450,6 +450,15 @@
   // replace from rewriting lines the user never looked at.
   function serializeBlocks(blockEls, opts) {
     const carryOver = (opts && opts.carryOver) || null;
+    // v3.11 (batch-2 §6): `opts.blankBefore(blockEl, prevItemEl)` — true when a
+    // blank line separates this item from the item emitted before it. The
+    // CALLER decides (client.js reads it off the file: a loose list keeps the
+    // blank lines it had, and a new neighbour takes the spacing of the list it
+    // joins). The blank line gets a lineMeta entry with blockId null and
+    // `blank: true`, so
+    // `lineMeta.length === md.split('\n').length` still holds.
+    const blankBefore = (opts && typeof opts.blankBefore === 'function') ? opts.blankBefore : null;
+    let prevItemEl = null;
     const unsupported = [];
     const unsupportedByLi = [];
     const multiLineBlockIds = [];
@@ -586,8 +595,16 @@
         // keep. (isBlankBlockText() stays as-is for the block's own direct
         // children, where the template's inter-element whitespace can legally
         // be a single space with no newline.)
-        const inlineKids = [];
+        let inlineKids = [];
         allChildNodes(textEl).forEach((c) => { if (!isBlankText(c)) inlineKids.push(c); });
+        // v3.11 (batch-2 §6): a LOOSE item's surface is one <p> — its content
+        // is that paragraph's, and the blank lines that made the list loose
+        // are restated between items (opts.blankBefore below), not carried by
+        // the <p>. Two or more <p> (a multi-paragraph item) still reach the
+        // inline serializer whole and stay unsupported.
+        if (inlineKids.length === 1 && inlineKids[0].nodeType === 1 && inlineKids[0].nodeName === 'P') {
+          inlineKids = allChildNodes(inlineKids[0]);
+        }
         const res = inlineMd.serializeInline({ childNodes: inlineKids });
         itemMd = res.md;
         res.unsupported.forEach((u) => innerUnsupported.push(u));
@@ -709,6 +726,14 @@
       // Round 6: join an open task prefix (above) to this line, dropping the
       // columns it already wrote; then, if THIS item is itself a content-free
       // task item, become the prefix instead of emitting.
+      // A line joined onto an open task prefix is the same physical line, so
+      // it never takes a blank in front of it.
+      const blankHere = !pending && lines.length > 0 && !!blankBefore && !!blankBefore(blockEl, prevItemEl);
+      prevItemEl = blockEl;
+      if (blankHere) {
+        lines.push('');
+        lineMeta.push({ blockId: null, indentPrefix: '', marker: '', blank: true });
+      }
       let head = indentPrefix + marker;
       // `indentPrefix` in lineMeta is the text PHYSICALLY standing before this
       // line's own marker. Normally that is the accumulated width; on a line a

package/lib/md2doc.js

@@ -360,10 +360,14 @@ function renderEditModeList(listToken, blocks, biRef, out) {
     const runStart = liRunStartsHere(biRef, b) ? ' data-run-start="1"' : '';
     const listStartAttr = listStart ? ' data-list-start="1"' : '';
     listStart = false;
+    // v3.11 (batch-2 §6): marked's own looseness for this list token, read
+    // by the lexer off the source (a blank line between items). The editor
+    // gives a NEW neighbour in this list the same spacing.
+    const looseAttr = listToken.loose ? ' data-list-loose="1"' : '';
     parts.push(
       `<div class="ed-block" data-block-id="${b.id}" data-block-type="li"` +
       ` data-list-type="${b.listType}" data-task="${b.task ? 1 : 0}" data-indent="${b.indent}"` +
-      runStart + listStartAttr + ` style="--ed-indent:${b.indent}">` +
+      runStart + listStartAttr + looseAttr + ` style="--ed-indent:${b.indent}">` +
       `<span class="ed-li-marker" aria-hidden="true"></span>` + check +
       `<div class="ed-li-text">${inner}</div>` +
       `</div>`
@@ -439,6 +443,8 @@ const READER_ICONS = {
   up: ICON_SVG_OPEN + '<path d="m18 15-6-6-6 6"/></svg>',
   down: ICON_SVG_OPEN + '<path d="m6 9 6 6 6-6"/></svg>',
   x: ICON_SVG_OPEN + '<path d="M18 6 6 18"/><path d="m6 6 12 12"/></svg>',
+  copy: ICON_SVG_OPEN + '<rect width="14" height="14" x="8" y="8" rx="2" ry="2"/><path d="M4 16c-1.1 0-2-.9-2-2V4c0-1.1.9-2 2-2h10c1.1 0 2 .9 2 2"/></svg>',
+  check: ICON_SVG_OPEN + '<path d="M20 6 9 17l-5-5"/></svg>',
 };
 
 async function renderMarkdown(mdText, srcPath, opts = {}) {
@@ -1041,9 +1047,16 @@ ${itemsHtml}
       // The info string goes into an attribute: escape it (an info string such
       // as x" onmouseover=… must stay text). Ordinary language names are unchanged.
       const langAttr = escapeHtml(lang);
-      if (coloured !== null) return `<pre><code class="language-${langAttr} hljs">${coloured}</code></pre>\n`;
-      const escaped = code.replace(/&/g,'&amp;').replace(/</g,'&lt;').replace(/>/g,'&gt;');
-      return `<pre><code class="language-${langAttr}">${escaped}</code></pre>\n`;
+      const pre = coloured !== null
+        ? `<pre><code class="language-${langAttr} hljs">${coloured}</code></pre>`
+        : `<pre><code class="language-${langAttr}">${code.replace(/&/g,'&amp;').replace(/</g,'&lt;').replace(/>/g,'&gt;')}</code></pre>`;
+      // v3.12.0: the reader's copy button. It sits on a wrapper beside the
+      // <pre>, not inside it — <pre> scrolls sideways and would carry the
+      // button off with the code. Icon only: a text label would be picked up
+      // by search and by select-all. Never in edit mode, where it would land
+      // inside the editable surface.
+      if (opts.editMode) return pre + '\n';
+      return `<div class="code-block">${pre}<button class="code-copy" type="button" aria-label="Copy code" title="Copy code">${READER_ICONS.copy}</button></div>\n`;
     };
 
     renderer.heading = function(token) {
@@ -2886,6 +2899,32 @@ const html = `<!DOCTYPE html>
     line-height: 1.45;
   }
   pre code { background: none; padding: 0; font-size: 0.875em; }
+  /* v3.12.0: the copy button renderer.code puts beside each fenced block.
+     The wrapper takes over the margin the pre used to have, so the spacing
+     around a block does not move. The button hides at rest and shows on
+     hover, on keyboard focus and after a press; a touch screen has no hover,
+     so there it always shows. Colours are existing reader literals, so the
+     theme post-pass carries them into dark mode. */
+  .code-block { position: relative; margin: 1em 0; }
+  .code-block > pre { margin: 0; }
+  .code-copy {
+    position: absolute; top: 8px; right: 8px;
+    display: inline-flex; align-items: center; justify-content: center; gap: 6px;
+    height: 30px; min-width: 30px; padding: 0 7px;
+    border: 1px solid #d0d7de; border-radius: 6px;
+    background: #ffffff; color: #57606a;
+    font: inherit; font-size: 12px; line-height: 1; cursor: pointer;
+    opacity: 0; transition: opacity 0.12s;
+  }
+  .code-block:hover .code-copy,
+  .code-copy:focus-visible,
+  .code-block.is-copied .code-copy,
+  .code-block.is-manual .code-copy { opacity: 1; }
+  .code-copy:hover { background: #eef2f6; color: #24292e; }
+  .code-copy:focus-visible { outline: 2px solid #0969da; outline-offset: 2px; }
+  .code-block.is-copied .code-copy { color: #0969da; }
+  @media (hover: none) { .code-copy { opacity: 1; } }
+  @media (prefers-reduced-motion: reduce) { .code-copy { transition: none; } }
   /* v3.10.0: fenced-code syntax colour, written at render time by
      lib/highlight.js (no script on the page). Colours are the syn-* roles. */
   .hljs-keyword, .hljs-selector-tag, .hljs-literal { color: var(--md-syn-keyword); }
@@ -3272,6 +3311,7 @@ ${usesDrawio ? `  /* v3.4.0 §4: a document with no drawio reference must not pa
     .content table tbody td:first-child,
     .content table thead th:first-child { position: static; }
     .heading-anchor { display: none; }
+    .code-copy { display: none; }
     pre  { font-size: 9pt; }
     a[href]:after { content: none; }
   }
@@ -6006,6 +6046,55 @@ ${usesDrawioMultiPage ? `  // v3.4.0 batch2 Task 6: wired through a NAMED, reusa
     var shown = lightboxCanvas.firstElementChild;
     lightboxFitZoom = computeLightboxFit(!!(shown && shown.tagName.toLowerCase() === 'svg'));
   });
+
+  // v3.12.0: copy button on fenced code (renderer.code emits it; edit mode
+  // does not). Delegated from document, so the edit page's wholesale
+  // re-render of .content needs no rebind. The clipboard API first; then
+  // execCommand over a selection of the code; and when both are refused the
+  // code stays selected and the button says which keys to press.
+  var CODE_COPY_RESET_MS = 2000;
+  var CODE_COPY_DONE = '${READER_ICONS.check}<span>Copied</span>';
+  var CODE_COPY_KEYS = /Mac|iPhone|iPad/.test(navigator.platform || '') ? 'Press Cmd+C' : 'Press Ctrl+C';
+  function showCodeCopyResult(btn, wrap, ok) {
+    if (btn.__md2docIcon === undefined) btn.__md2docIcon = btn.innerHTML;
+    clearTimeout(btn.__md2docReset);
+    wrap.classList.toggle('is-copied', ok);
+    wrap.classList.toggle('is-manual', !ok);
+    btn.innerHTML = ok ? CODE_COPY_DONE : '<span>' + CODE_COPY_KEYS + '</span>';
+    btn.setAttribute('aria-label', ok ? 'Copied' : CODE_COPY_KEYS);
+    btn.__md2docReset = setTimeout(function () {
+      wrap.classList.remove('is-copied', 'is-manual');
+      btn.innerHTML = btn.__md2docIcon;
+      btn.setAttribute('aria-label', 'Copy code');
+    }, CODE_COPY_RESET_MS);
+  }
+  function copyCodeBlock(btn) {
+    var wrap = btn.closest('.code-block');
+    var code = wrap && wrap.querySelector('pre > code');
+    if (!code) return;
+    function bySelection() {
+      var range = document.createRange();
+      range.selectNodeContents(code);
+      var sel = window.getSelection();
+      sel.removeAllRanges();
+      sel.addRange(range);
+      var ok = false;
+      try { ok = document.execCommand('copy'); } catch (e) { ok = false; }
+      if (ok) sel.removeAllRanges();
+      showCodeCopyResult(btn, wrap, ok);
+    }
+    if (navigator.clipboard && navigator.clipboard.writeText) {
+      navigator.clipboard.writeText(code.textContent).then(function () {
+        showCodeCopyResult(btn, wrap, true);
+      }, bySelection);
+    } else {
+      bySelection();
+    }
+  }
+  document.addEventListener('click', function (event) {
+    var btn = event.target.closest && event.target.closest('.code-block > .code-copy');
+    if (btn) copyCodeBlock(btn);
+  });
 })();
 </script>
 </body>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@helping-ai-workflow/md2doc",
-  "version": "3.10.0",
+  "version": "3.12.0",
   "description": "Markdown → HTML / PDF renderer with WaveDrom, Mermaid, Graphviz, and draw.io support",
   "keywords": [
     "markdown",