@helping-ai-workflow/md2doc 3.2.0 → 3.2.1

package/lib/editor/docsource.js

@@ -11,7 +11,8 @@
 })(typeof self !== 'undefined' ? self : this, function () {
 
 // Pure state machine + lines[]<->string conversion for the "追加 4 全文原始碼
-// 逃生口" (whole-document source escape hatch): 編輯 ⇄ 原始碼 ⇄ 預覽.
+// 逃生口" (whole-document source escape hatch): 編輯 ⇄ 原始碼.
+// v3.2.1: 預覽（preview）已移除，見下方 MODES 上的說明。
 //
 // EOL representation: `eol` is always a single literal separator string --
 // '\n', '\r\n' or '\r' -- the same convention server.js already establishes
@@ -31,12 +32,21 @@
 //   2. `\r\n` is listed before the two single-character alternatives, so a
 //      CRLF pair is always consumed as ONE separator, never split into a
 //      spurious blank line between a lone \r and a lone \n.
-const MODES = ['edit', 'source', 'preview'];
+const MODES = ['edit', 'source'];
 
-// Three-state cycle: edit -> source -> preview -> edit. An unrecognized
-// mode (defensive: a caller passing something stale) falls back to the
-// first state rather than throwing, matching how the rest of this module
-// prefers a safe default over a thrown exception for malformed input.
+// v3.2.1: 兩態循環 edit <-> source。preview 移除 —— 實測它相對 edit 只做
+// 兩件事（applyPreviewEditability(false) 與兩行隱藏 ⠿／＋ 的 CSS），卻擋在
+// source 回 edit 的路上，而且 22 顆工具列按鈕中 13 顆在它底下仍然寫得進磁碟。
+//
+// 為什麼是移除而不是加閘門：一個「列舉可及面」推導出來的唯讀閘門必然漏掉
+// 一整族 —— 切換前就已經開著的 ⠿ 選單在切換後仍是 140x118、display:flex，
+// 其 16 個項目每一個都寫得進磁碟。閘門要約 8 個守衛點跨 2 個檔案才擋得住，
+// 而 preview 本身沒有任何 edit 給不了的能力。
+//
+// An unrecognized mode (defensive: a caller passing something stale) falls
+// back to the first state rather than throwing, matching how the rest of
+// this module prefers a safe default over a thrown exception for malformed
+// input.
 function next(mode) {
   const idx = MODES.indexOf(mode);
   return MODES[(idx + 1) % MODES.length];

package/lib/editor/toolbar-model.js

@@ -48,7 +48,13 @@ const BUTTON_DEFS = [
 
   // view
   { id: 'outline', group: 'view', label: '☰', icon: '☰', title: '大綱', toggle: false },
-  { id: 'preview', group: 'view', label: '👁', icon: '👁', title: '預覽', toggle: true },
+  // v3.2.1: 這顆是 edit <-> source 的兩態切換（preview 模式已移除）。id 保留
+  // 為 'preview' 是刻意的 —— 每一支選擇器、journey 案例與 runtime 測試都指名
+  // 它，改名只換得一個更好聽的字串。label / title 不是烘死值：deriveState()
+  // 每次都依當前 mode 覆寫成【下一個】狀態的名字，這裡的值只是 edit 模式的
+  // 起始值。toggle: false —— 兩態下 aria-pressed 講不出真話（按鈕講的是去處，
+  // 不是目前所在），真話由會變的 label / title 承擔。
+  { id: 'preview', group: 'view', label: 'M↓', icon: 'M↓', title: '切換到原始碼模式', toggle: false },
 ];
 
 const BUTTONS = Object.freeze(BUTTON_DEFS.map((b) => Object.freeze(Object.assign({}, b))));
@@ -76,11 +82,18 @@ function deriveState(ctx) {
     state[b.id] = { active: false, disabled: false };
   }
 
-  // preview is the tri-state edit/source/preview toggle: "pressed" whenever
-  // the current mode is not plain editing. It stays enabled in source mode
-  // via the mode==='source' override below (do not gate this on inList/
-  // hasBlock — it is the escape hatch OUT of those states).
-  state.preview.active = mode !== 'edit';
+  // v3.2.1: the mode button names the NEXT state, never the current one.
+  // Under the old tri-state cycle it was permanently labelled 「預覽」 and
+  // reported aria-pressed=true in BOTH non-edit modes, so pressing it twice
+  // landed in source with no grips and an interface that said nothing about
+  // where you were — the reported "stuck" defect. It carries no `active`
+  // now: with two states the honest signal is the label, and the current
+  // mode is spelled out in .ed-toolbar-status by the client.
+  // It stays enabled in source mode via the mode==='source' override below
+  // (do not gate this on inList/hasBlock — it is the escape hatch OUT of
+  // those states).
+  state.preview.label = mode === 'source' ? '✎' : 'M↓';
+  state.preview.title = mode === 'source' ? '切換到編輯模式' : '切換到原始碼模式';
 
   // headings: dropdown whose label tracks the current heading depth.
   state.headings.label = blockType === 'heading' ? 'H' + headingDepth : 'H';
@@ -145,8 +158,10 @@ function deriveState(ctx) {
   }
 
   // mode 'source' is the full-document raw-source escape hatch: everything
-  // is disabled except the toggle that got you there (preview). This
-  // overrides every disabled flag computed above.
+  // is disabled except the mode button that got you there (id 'preview',
+  // which in v3.2.1 is the edit <-> source toggle). This overrides every
+  // disabled flag computed above, and is the ONLY way back out — do not
+  // narrow it.
   if (mode === 'source') {
     for (const b of BUTTONS) {
       state[b.id].disabled = b.id !== 'preview';

package/lib/md2doc.js

@@ -2590,16 +2590,18 @@ ${itemsHtml}
      row's overflow-x: auto scrolls (measured identical rect at scrollLeft
      0 / mid / max, where the absolute alternative moved by the exact scroll
      delta). References no gutter-alignment custom property.
-     H2 fix: pointer-events: none below is LOAD-BEARING while this slot
-     ships empty, not decorative. At narrow viewports .ed-toolbar's own
+     H2 fix: pointer-events: none below is LOAD-BEARING, not decorative.
+     v3.2.1 filled this slot with the current mode's name ('編輯' / '原始碼'),
+     so it no longer ships empty — but that content is inert TEXT, never a
+     control, which is exactly what keeps the rule below sufficient. At narrow viewports .ed-toolbar's own
      overflow-x: auto lets the button row scroll to a rest position where
      this fixed 7rem box's footprint physically covers three real buttons
      (image / outline / preview, measured at 420px width, scrollLeft at its
      max 221) — document.elementFromPoint() over any of them resolves to
      this span, not the button underneath, and a real click on it never
      reaches the button's listener. Harmless today only because the span
-     carries no content to interact with. WHOEVER WIRES UP TWO-WAY SYNC
-     (§3) AND PUTS REAL CONTENT OR CONTROLS HERE MUST remove or scope this
+     carries nothing to interact with. WHOEVER PUTS A CONTROL HERE (or
+     wires up two-way sync, §3) MUST remove or scope this
      rule AND independently solve the overlap (e.g. reflow the slot in-flow
      at narrow widths, or shrink/hide it below some width) — simply
      deleting this line without doing that reopens the swallowed-click
@@ -2636,8 +2638,6 @@ ${itemsHtml}
     border: 1px solid #d0d7de; border-radius: 8px; resize: vertical;
     white-space: pre; overflow-wrap: normal;
   }
-  body[data-ed-mode="preview"] .ed-handle,
-  body[data-ed-mode="preview"] .ed-insert { display: none !important; }
   .ed-editing { position: relative; }
   .ed-raw {
     display: block; width: 100%; font-family: monospace; font-size: 13px;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@helping-ai-workflow/md2doc",
-  "version": "3.2.0",
+  "version": "3.2.1",
   "description": "Markdown → HTML / PDF renderer with WaveDrom, Mermaid, and Graphviz support",
   "keywords": [
     "markdown",
@@ -37,7 +37,7 @@
   },
   "scripts": {
     "preinstall": "node scripts/preinstall.js",
-    "test": "node test/md2doc.test.js && node test/blockmap.test.js && node test/images.test.js && node test/scroll-anchor.test.js && node test/lightbox.test.js && node test/lightbox-anno.test.js && node test/lightbox-anno-style.test.js && node test/reader-panels.test.js && node test/cli.test.js && node test/code-operator.test.js && node test/convert-md.test.js && node test/render-api.test.js && node test/lineops.test.js && node test/indent-clamp.test.js && node test/selection.test.js && node test/editmode-render.test.js && node test/editmode-wavedrom-reinit.test.js && node test/editor-server.test.js && node test/cli-edit.test.js && node test/open-viewer.test.js && node test/editor-client.test.js && node test/editor-client-runtime.test.js && node test/roundtrip.test.js && node test/byte-stability.test.js && node test/editor-server-throw.test.js && node test/editor-reader-rebind.test.js && node test/inline-md.test.js && node test/table-md.test.js && node test/gate-compat.test.js && node test/history.test.js && node test/list-md.test.js && node test/table-width.test.js && node test/toolbar-model.test.js && node test/docsource.test.js && node test/asset.test.js && node test/paste-md.test.js && node test/patchmap.test.js"
+    "test": "node test/md2doc.test.js && node test/blockmap.test.js && node test/images.test.js && node test/scroll-anchor.test.js && node test/lightbox.test.js && node test/lightbox-anno.test.js && node test/lightbox-anno-style.test.js && node test/reader-panels.test.js && node test/cli.test.js && node test/code-operator.test.js && node test/convert-md.test.js && node test/render-api.test.js && node test/lineops.test.js && node test/indent-clamp.test.js && node test/selection.test.js && node test/editmode-render.test.js && node test/editmode-wavedrom-reinit.test.js && node test/editor-server.test.js && node test/cli-edit.test.js && node test/open-viewer.test.js && node test/editor-client.test.js && node test/editor-client-runtime.test.js && node test/roundtrip.test.js && node test/byte-stability.test.js && node test/editor-server-throw.test.js && node test/editor-reader-rebind.test.js && node test/inline-md.test.js && node test/table-md.test.js && node test/gate-compat.test.js && node test/history.test.js && node test/list-md.test.js && node test/table-width.test.js && node test/toolbar-model.test.js && node test/docsource.test.js && node test/asset.test.js && node test/paste-md.test.js && node test/patchmap.test.js && node test/editor-journey.test.js && node test/detach-census.test.js"
   },
   "repository": {
     "type": "git",