weml-monaco 0.4.4 → 0.5.1

package/CHANGELOG.md

@@ -8,6 +8,43 @@ This package mirrors the language logic of the VS Code extension; entries below
 track the port catching up with the VS Code provider changes (see the root
 `CHANGELOG.md` for the extension).
 
+## [0.5.1] — 2026-09-26
+
+### Changed
+- Updated the shared V2 normalizer: lowercase tag names, keep the first
+  duplicate attribute value and unify inline sentence whitespace handling.
+- Retained compatibility with node-html-parser 6.x and browser bundles.
+
+## [0.5.0] — 2026-09-09
+
+### Changed
+- **Breaking:** the Monaco type surface is imported from
+  `monaco-editor/editor/editor.api.js` instead of the `monaco-editor` root.
+  The root re-exports the TypeScript, CSS, JSON and HTML language
+  contributions, so its type forced every consumer to import Monaco whole and
+  ship all four language workers — around 11 MB of bundle nobody used. The
+  editor core carries everything this package actually touches. Passing the
+  full `monaco-editor` namespace still type-checks, since it is a superset.
+- **Breaking:** `peerDependencies.monaco-editor` raised from `>=0.34.0` to
+  `>=0.56.0`. The subpath resolves through the `exports` map introduced in
+  0.56 (`"./*": "./esm/vs/*.js"`) and needs `moduleResolution` `node16` or
+  `bundler`; on 0.55 and older the equivalent path was
+  `monaco-editor/esm/vs/editor/editor.api`.
+- Development and the CDN demo build against monaco-editor 0.56.
+
+### Fixed
+- The bundles no longer register themselves with an AMD loader. `he` (via
+  node-html-parser) ends in a UMD footer whose `define()` call is anonymous, so
+  loading `dist/index.global.js` from a `<script>` tag on a page that also runs
+  Monaco's AMD loader made the loader attribute that module to whatever it was
+  resolving, warning `Duplicate definition of module 'vs/editor/editor.main'`.
+  The build now folds `define.amd` to `false`, dropping the branch.
+
+The type-path change itself does not touch the runtime output: every Monaco
+import in `src/` is an `import type`, and only the first line of the emitted
+declarations changes. `dist/index.mjs`, `index.cjs` and `index.global.js`
+differ solely by the removed AMD footer above.
+
 ## [0.4.4] — 2026-09-07
 
 Synced with VS Code extension `0.7.12`.

package/README.md

@@ -58,7 +58,7 @@ Monaco is **passed in** at registration time, so this works no matter how
 Monaco itself is loaded (AMD/CDN, ESM bundle, or a global):
 
 ```ts
-import * as monaco from 'monaco-editor';
+import * as monaco from 'monaco-editor/editor/editor.api.js';
 import { registerWeml } from 'weml-monaco';
 
 const disposable = registerWeml(monaco);
@@ -69,6 +69,20 @@ monaco.editor.create(document.getElementById('container'), { model });
 // disposable.dispose();  // tears down every provider + diagnostics listener
 ```
 
+The import path is the editor core rather than the `monaco-editor` root on
+purpose. The root re-exports the TypeScript, CSS, JSON and HTML language
+contributions, so importing it pulls all four — and their workers — into the
+application bundle. `weml-monaco` never touches them: it only needs
+`editor`, `languages`, `Position`, `MarkerSeverity` and a handful of
+interfaces, all of which live in `editor/editor.api.js`. Passing the full
+`monaco-editor` namespace still type-checks (it is a superset), so this is a
+choice the consumer makes, not a restriction.
+
+The subpath resolves through the package's `exports` map, which requires
+**monaco-editor 0.56 or newer** and `moduleResolution` `node16`/`bundler`.
+On 0.55 and older the equivalent path was
+`monaco-editor/esm/vs/editor/editor.api`.
+
 `registerWeml(monaco, options?)` accepts:
 
 | option                 | default     | meaning                                            |

package/dist/index.cjs

@@ -267,7 +267,7 @@ var require_he = __commonJS({
         "escape": escape,
         "unescape": decode
       };
-      if (typeof define == "function" && typeof define.amd == "object" && define.amd) {
+      if (typeof define == "function" && false) {
         define(function() {
           return he;
         });
@@ -6306,7 +6306,7 @@ var require_wemlContentNormalizeV2 = __commonJS({
           }
           result.push(`<${name.toLowerCase()}${attrs} />`);
         } else {
-          result.push(input.slice(start, cursor), attrs, ">");
+          result.push(closing ? "</" : "<", name.toLowerCase(), attrs, ">");
         }
         index = end + 1;
       }
@@ -6356,62 +6356,14 @@ var require_wemlContentNormalizeV2 = __commonJS({
       }
     }
     function processInlineContainer(node) {
-      if (hasDirectSentChildren(node)) processSentContainer(node, true);
-      else processInlineContent(node, true);
-    }
-    function processSentContainer(node, trimEdges) {
-      for (const child of [...node.childNodes]) {
-        if (child.nodeType !== 1) continue;
-        const name = child.rawTagName.toLowerCase();
-        if (isInlineTextElement(child)) {
-          if (hasDirectSentChildren(child)) processSentContainer(child, false);
-          else processInlineContent(child, false);
-        } else if (BLOCK_ELEMENTS.has(name)) {
-          processBlockElement(child);
-        } else if (INLINE_CONTAINERS.has(name)) {
-          processInlineContainer(child);
-        }
-      }
-      for (const child of [...node.childNodes]) {
-        if (child.nodeType === 1 && isInlineTextElement(child)) {
-          bleedEdgeSpaces(child);
-        }
-      }
-      for (const child of [...node.childNodes]) {
-        if (child.nodeType === 3) child.rawText = collapseWhitespace(child.rawText);
-      }
-      for (const child of [...node.childNodes]) {
-        if (child.nodeType === 1 && child.rawTagName.toLowerCase() === "br") {
-          const prev = child.previousSibling;
-          if (prev && prev.nodeType === 3) prev.rawText = trimEndWhitespace(prev.rawText);
-          const next = child.nextSibling;
-          if (next && next.nodeType === 3) next.rawText = trimStartWhitespace(next.rawText);
-        }
-      }
-      if (trimEdges) trimContainerEdges(node);
-      for (const child of [...node.childNodes]) {
-        if (child.nodeType === 3 && child.rawText === "") child.remove();
-      }
-    }
-    function trimContainerEdges(node) {
-      const children = [...node.childNodes];
-      if (children.length === 0) return;
-      const first = children[0];
-      const last = children[children.length - 1];
-      if (children.length === 1) {
-        if (first.nodeType === 3) first.rawText = trimWhitespace(first.rawText);
-      } else {
-        if (first.nodeType === 3) first.rawText = trimStartWhitespace(first.rawText);
-        if (last.nodeType === 3) last.rawText = trimEndWhitespace(last.rawText);
-      }
+      processInlineContent(node, true);
     }
     function processInlineContent(node, trimEdges) {
       for (const child of [...node.childNodes]) {
         if (child.nodeType !== 1) continue;
         const name = child.rawTagName.toLowerCase();
         if (isInlineTextElement(child)) {
-          if (hasDirectSentChildren(child)) processSentContainer(child, false);
-          else processInlineContent(child, false);
+          processInlineContent(child, false);
         } else if (BLOCK_ELEMENTS.has(name)) {
           processBlockElement(child);
         } else if (INLINE_CONTAINERS.has(name)) {
@@ -6521,11 +6473,6 @@ var require_wemlContentNormalizeV2 = __commonJS({
         if (last.nodeType === 3) last.rawText = trimEndWhitespace(last.rawText);
       }
     }
-    function hasDirectSentChildren(node) {
-      return [...node.childNodes].some(
-        (child) => child.nodeType === 1 && child.rawTagName.toLowerCase() === "w-sent"
-      );
-    }
     function isInlineTextElement(node) {
       if (node.nodeType !== 1) return false;
       const name = node.rawTagName.toLowerCase();
@@ -6599,10 +6546,12 @@ var require_wemlContentNormalizeV2 = __commonJS({
             value = rawAttrs.slice(valueStart, index);
           }
         }
-        canonical.set(
-          name,
-          SPACE_SEPARATED_ATTRIBUTES.has(name) ? normalizeTokenList(value, entities2, marker) : trimWhitespace(value)
-        );
+        if (!canonical.has(name)) {
+          canonical.set(
+            name,
+            SPACE_SEPARATED_ATTRIBUTES.has(name) ? normalizeTokenList(value, entities2, marker) : trimWhitespace(value)
+          );
+        }
       }
       return [...canonical.entries()].sort(([left], [right]) => left < right ? -1 : left > right ? 1 : 0).map(([name, value]) => ` ${name}="${encodeAttr(String(value))}"`).join("");
     }