@helping-ai-workflow/md2doc 3.4.2 → 3.6.0

package/lib/editor/client.js

@@ -17542,7 +17542,8 @@
     // An older cached page that predates the injection must degrade to "the
     // button does nothing visible", never to a throw out of a click handler.
     if (!ui || !mods || !mods['wave-store.js'] || !mods['wave-codec.js'] ||
-        !mods['wave-geometry.js']) return;
+        !mods['wave-geometry.js'] || !mods['wave-draw.js'] ||
+        !mods['wave-panels.js']) return;
     if (waveEditor !== null) return;
     hideWaveEditBtn();
 
@@ -17603,8 +17604,27 @@
         document: document,
         host: document.body,
         source: body.text,
+        // v3.6.0 Task 15: which block in this document the editor is over,
+        // for the 匯出 filename (`exportName()` in wave-ui.js). The editor
+        // cannot derive it — it is handed one block's TEXT and nothing about
+        // where that text came from — and two diagrams in one document
+        // exporting under the same filename is a silent overwrite in the
+        // user's downloads folder.
+        blockIndex: id,
+        // …and the document's own name for the other half of that filename.
+        // `baseTitle`, NOT `document.title`: this file owns the page title
+        // while a session is open and writes the dirty marker into it
+        // (`(documentIsDirty() ? '● ' : '') + baseTitle`, above). Handing over
+        // the decorated title would land a user who has actually edited with
+        // `●-doc-wave-1.svg`, and a user who has not with the clean name —
+        // i.e. it would look right in exactly the case nobody exports in.
+        // `baseTitle` is the title as it was before this file touched it,
+        // which is md2doc's `path.basename(src, '.md')`.
+        fileLabel: baseTitle,
         codec: mods['wave-codec.js'],
         geometry: mods['wave-geometry.js'],
+        draw: mods['wave-draw.js'],
+        panels: mods['wave-panels.js'],
         createStore: mods['wave-store.js'].createStore,
         // The store's own answer to「哪一條 lane 是哪一條」across one operation,
         // handed over rather than re-derived: the editor's cell cursor and its
@@ -17760,10 +17780,29 @@
 
   /** One refusal, recorded and raised. The only producer of `waveUnwrittenEdit`. */
   function waveRefuseGesture(seam, reason) {
-    // A refusal is not papered over and not rolled back. wave-ui has already
-    // said so on the status line, which is where the user's eyes are, but a
-    // status line is not a decision — this raises it as a banner naming the
-    // lines it could not write and waits for the user to acknowledge it.
+    // A refusal is not papered over and not rolled back. This raises it as a
+    // banner naming the lines it could not write, and waits for the user to
+    // acknowledge it.
+    //
+    // v3.6.0 Task 16 fix round 2 (N1) withdraws the reason that used to stand
+    // here — "wave-ui has already said so on the status line, which is where
+    // the user's eyes are, but a status line is not a decision". wave-ui does
+    // write the reason there (`afterStoreMoved`'s
+    // `這個改動沒辦法只改幾個位元組寫回去：…`), but that line is ONE
+    // `role="status"` string and the next thing the same gesture says
+    // replaces it. MEASURED on the journey fixture: the fourth Alt+↑ of the
+    // Task 16 scenario refuses — "the value at [\"signal\"] is or contains an
+    // object, so replacing it would re-serialise part of the document's
+    // structure…" — and `moveSelectedLane`'s landing line
+    // (`搬動的 lane 落在群組「bus」裡`) is written straight after it, so the
+    // sentence the user is left looking at reads like success.
+    //
+    // That is a FAMILY, not one caller: `addLaneFromToolbar` and 複製 have
+    // written over the same refusal since before Task 16, and the rail's drag
+    // drop announces nothing at all. Re-ordering those says is behaviour, and
+    // it is parked as a next-version follow-up rather than done here. So the
+    // banner below is not a nicety layered on a status line that already said
+    // it — it is the only part of the refusal that cannot be clobbered.
     seam.unwritten = true;
     waveUnwrittenEdit = true;
     setDirty();

package/lib/editor/server.js

@@ -56,7 +56,7 @@ const TURNDOWN_SRC = fs.readFileSync(
   require.resolve('turndown/lib/turndown.browser.umd.js'), 'utf8');
 const PASTE_MD_SRC = fs.readFileSync(path.join(__dirname, 'paste-md.js'), 'utf8');
 
-// v3.4.0 batch3 Task 6: the wavedrom GUI's four modules.
+// v3.4.0 batch3 Task 6: the wavedrom GUI's modules.
 //
 // wave-codec.js / wave-geometry.js / wave-store.js are plain CommonJS ON
 // PURPOSE, and rewriting them into the UMD header every other module here uses
@@ -67,11 +67,28 @@ const PASTE_MD_SRC = fs.readFileSync(path.join(__dirname, 'paste-md.js'), 'utf8'
 // the browser gets them through the small CommonJS shim below instead —
 // the module text itself is inlined byte for byte.
 //
+// v3.6.0 Task 1: wave-draw.js joins them, plain CommonJS too — a pure
+// extraction out of wave-ui.js, not test-guarded like the other three, but
+// needing the identical shim for a different reason: wave-ui.js itself is
+// injected as a bare <script> with no `require()` of its own (see its own
+// header comment), so it cannot resolve `require('./wave-draw.js')` in the
+// browser any more than the page around it could. `createWaveEditor()` takes
+// it in as `opts.draw` instead, the same way it already takes `opts.codec`
+// and `opts.geometry`.
+//
+// v3.6.0 Task 2: wave-panels.js joins them for the identical reason —
+// another pure extraction out of wave-ui.js (the toolbar, the right-hand
+// panels and the lane rail this time), needing the same shim because
+// wave-ui.js still cannot `require()` anything in the browser. Taken in as
+// `opts.panels`.
+//
 // wave-ui.js is UMD like its neighbours and needs no shim; in the page it
-// defines `window.md2docWaveUi` and takes the other three as parameters.
+// defines `window.md2docWaveUi` and takes the other five as parameters.
 const WAVE_CODEC_SRC = fs.readFileSync(path.join(__dirname, 'wave-codec.js'), 'utf8');
 const WAVE_GEOMETRY_SRC = fs.readFileSync(path.join(__dirname, 'wave-geometry.js'), 'utf8');
 const WAVE_STORE_SRC = fs.readFileSync(path.join(__dirname, 'wave-store.js'), 'utf8');
+const WAVE_DRAW_SRC = fs.readFileSync(path.join(__dirname, 'wave-draw.js'), 'utf8');
+const WAVE_PANELS_SRC = fs.readFileSync(path.join(__dirname, 'wave-panels.js'), 'utf8');
 const WAVE_UI_SRC = fs.readFileSync(path.join(__dirname, 'wave-ui.js'), 'utf8');
 
 // One CommonJS module, wrapped so a plain <script> can carry it.
@@ -585,6 +602,8 @@ async function createEditorServer({
           waveModuleTag('wave-codec.js', WAVE_CODEC_SRC) +
           waveModuleTag('wave-geometry.js', WAVE_GEOMETRY_SRC) +
           waveModuleTag('wave-store.js', WAVE_STORE_SRC) +
+          waveModuleTag('wave-draw.js', WAVE_DRAW_SRC) +
+          waveModuleTag('wave-panels.js', WAVE_PANELS_SRC) +
           `<script>${WAVE_UI_SRC}</script>\n` +
           `<script>${clientJs}</script>\n`;
         // Splice at the LAST "</body>" — the document's real closing tag.