@lvce-editor/editor-worker 16.2.0 → 16.4.0

package/dist/editorWorkerMain.js

@@ -474,7 +474,7 @@ const getFirstEvent = (eventEmitter, eventMap) => {
   return promise;
 };
 const Message$1 = 3;
-const create$5$1 = async ({
+const create$5$2 = async ({
   isMessagePortOpen,
   messagePort
 }) => {
@@ -525,7 +525,7 @@ const wrap$5 = messagePort => {
 };
 const IpcParentWithMessagePort$1 = {
   __proto__: null,
-  create: create$5$1,
+  create: create$5$2,
   signal: signal$1,
   wrap: wrap$5
 };
@@ -731,7 +731,7 @@ const getErrorProperty = (error, prettyError) => {
     }
   };
 };
-const create$1$2 = (id, error) => {
+const create$1$1 = (id, error) => {
   return {
     jsonrpc: Two$1,
     id,
@@ -742,7 +742,7 @@ const getErrorResponse = (id, error, preparePrettyError, logError) => {
   const prettyError = preparePrettyError(error);
   logError(error, prettyError);
   const errorProperty = getErrorProperty(error, prettyError);
-  return create$1$2(id, errorProperty);
+  return create$1$1(id, errorProperty);
 };
 const create$a = (message, result) => {
   return {
@@ -867,14 +867,14 @@ const execute$1 = (command, ...args) => {
 };
 
 const Two = '2.0';
-const create$s = (method, params) => {
+const create$t = (method, params) => {
   return {
     jsonrpc: Two,
     method,
     params
   };
 };
-const create$r = (id, method, params) => {
+const create$s = (id, method, params) => {
   const message = {
     id,
     jsonrpc: Two,
@@ -884,14 +884,14 @@ const create$r = (id, method, params) => {
   return message;
 };
 let id = 0;
-const create$q = () => {
+const create$r = () => {
   return ++id;
 };
 
 /* eslint-disable n/no-unsupported-features/es-syntax */
 
 const registerPromise = map => {
-  const id = create$q();
+  const id = create$r();
   const {
     promise,
     resolve
@@ -909,7 +909,7 @@ const invokeHelper = async (callbacks, ipc, method, params, useSendAndTransfer)
     id,
     promise
   } = registerPromise(callbacks);
-  const message = create$r(id, method, params);
+  const message = create$s(id, method, params);
   if (useSendAndTransfer && ipc.sendAndTransfer) {
     ipc.sendAndTransfer(message);
   } else {
@@ -945,7 +945,7 @@ const createRpc = ipc => {
      * @deprecated
      */
     send(method, ...params) {
-      const message = create$s(method, params);
+      const message = create$t(method, params);
       ipc.send(message);
     }
   };
@@ -1011,13 +1011,13 @@ const createSharedLazyRpc = factory => {
     }
   };
 };
-const create$i = async ({
+const create$j = async ({
   commandMap,
   isMessagePortOpen,
   send
 }) => {
   return createSharedLazyRpc(() => {
-    return create$2$1({
+    return create$3$1({
       commandMap,
       isMessagePortOpen,
       send
@@ -1026,9 +1026,9 @@ const create$i = async ({
 };
 const LazyTransferMessagePortRpcParent = {
   __proto__: null,
-  create: create$i
+  create: create$j
 };
-const create$4$1 = async ({
+const create$5$1 = async ({
   commandMap,
   isMessagePortOpen = true,
   messagePort
@@ -1045,20 +1045,20 @@ const create$4$1 = async ({
   messagePort.start();
   return rpc;
 };
-const create$3$1 = async ({
+const create$4$1 = async ({
   commandMap,
   messagePort
 }) => {
-  return create$4$1({
+  return create$5$1({
     commandMap,
     messagePort
   });
 };
 const PlainMessagePortRpcParent = {
   __proto__: null,
-  create: create$3$1
+  create: create$4$1
 };
-const create$2$1 = async ({
+const create$3$1 = async ({
   commandMap,
   isMessagePortOpen,
   send
@@ -1068,7 +1068,7 @@ const create$2$1 = async ({
     port2
   } = new MessageChannel();
   await send(port1);
-  return create$4$1({
+  return create$5$1({
     commandMap,
     isMessagePortOpen,
     messagePort: port2
@@ -1076,9 +1076,9 @@ const create$2$1 = async ({
 };
 const TransferMessagePortRpcParent = {
   __proto__: null,
-  create: create$2$1
+  create: create$3$1
 };
-const create$1$1 = async ({
+const create$2$1 = async ({
   commandMap
 }) => {
   // TODO create a commandMap per rpc instance
@@ -1090,7 +1090,7 @@ const create$1$1 = async ({
 };
 const WebWorkerRpcClient = {
   __proto__: null,
-  create: create$1$1
+  create: create$2$1
 };
 const createMockRpc = ({
   commandMap
@@ -2619,6 +2619,108 @@ const getIncrementalEdits = async (oldState, newState) => {
   return emptyIncrementalEdits;
 };
 
+/**
+ * Gets all regex matches for a given text and regex pattern
+ * @param text The text to match against
+ * @param regex The regex pattern to use (should have global flag)
+ * @returns Array of regex matches
+ */
+const getRegexMatches = (text, regex) => {
+  return [...text.matchAll(regex)];
+};
+
+// URL matching regex pattern - matches common URL schemes
+// Supports: http://, https://, ftp://, ftps://, file://
+// Also matches URLs without explicit scheme (www.example.com)
+const URL_PATTERN = /(?:(?:https?|ftp|ftps|file):\/\/)?(?:www\.)?(?:[a-zA-Z0-9](?:[a-zA-Z0-9-]*[a-zA-Z0-9])?\.)+[a-zA-Z0-9](?:[a-zA-Z0-9-]*[a-zA-Z0-9])?(?:\/[^\s]*)?/g;
+
+// Regex to check if URL has a scheme (http://, https://, ftp://, etc.)
+const HAS_SCHEME_PATTERN = /^(?:https?|ftp|ftps|file):\/\//;
+
+// Regex to check if URL starts with www.
+const HAS_WWW_PATTERN = /^www\./;
+
+/**
+ * Detects links in a given text and returns their positions
+ * @param text The text to scan for links
+ * @returns Array of links with their start position and length
+ */
+const detectLinks = text => {
+  const matches = getRegexMatches(text, URL_PATTERN);
+  const links = [];
+  for (const match of matches) {
+    const url = match[0];
+    // Only consider as link if it has a scheme or starts with www.
+    if (HAS_SCHEME_PATTERN.test(url) || HAS_WWW_PATTERN.test(url)) {
+      links.push({
+        length: url.length,
+        start: match.index ?? 0
+      });
+    }
+  }
+  return links;
+};
+
+/**
+ * Detects all links in an editor and returns them as decorations
+ * @param editor The editor containing lines to scan
+ * @returns Flat array of decorations in format [offset, length, type, modifiers, ...]
+ */
+const detectAllLinksAsDecorations = editor => {
+  const decorations = [];
+  const {
+    lines
+  } = editor;
+  let offset = 0;
+  for (const line of lines) {
+    const links = detectLinks(line);
+    for (const link of links) {
+      const linkOffset = offset + link.start;
+      // Add link decoration: offset, length, type, modifiers
+      decorations.push(linkOffset, link.length, Link, 0);
+    }
+    offset += line.length + 1; // +1 for newline
+  }
+  return decorations;
+};
+
+/**
+ * Gets the URL text at a given offset in the editor if it's a link
+ * @param editor The editor
+ * @param offset The offset in the document
+ * @returns The URL string if the offset is on a link, or undefined
+ */
+const getUrlAtOffset = (editor, offset) => {
+  const {
+    decorations,
+    lines
+  } = editor;
+
+  // Iterate through decorations in groups of 4 (offset, length, type, modifiers)
+  for (let i = 0; i < decorations.length; i += 4) {
+    const decorationOffset = decorations[i];
+    const decorationLength = decorations[i + 1];
+    const decorationType = decorations[i + 2];
+
+    // Check if this decoration is a link and if the offset falls within it
+    if (decorationType === Link && offset >= decorationOffset && offset < decorationOffset + decorationLength) {
+      // Extract the URL text from the editor content
+      let currentOffset = 0;
+      for (const line of lines) {
+        const lineLength = line.length + 1; // +1 for newline
+        if (currentOffset + lineLength > decorationOffset) {
+          // The link starts in this line
+          const linkStartInLine = decorationOffset - currentOffset;
+          const url = line.slice(linkStartInLine, linkStartInLine + decorationLength);
+          return url;
+        }
+        currentOffset += lineLength;
+      }
+    }
+  }
+  return undefined;
+};
+
 const splitLines = lines => {
   if (!lines) {
     return [''];
@@ -3034,11 +3136,17 @@ const scheduleDocumentAndCursorsSelections = async (editor, changes, selectionCh
     selections: newSelections,
     undoStack: [...editor.undoStack, changes]
   };
-  set$6(editor.uid, editor, newEditor);
-  const incrementalEdits = await getIncrementalEdits(editor, newEditor);
-  const editorWithNewWidgets = await applyWidgetChanges(newEditor, changes);
-  const newEditor2 = {
+  // Update link decorations after text changes
+  const linkDecorations = detectAllLinksAsDecorations(newEditor);
+  const newEditorWithDecorations = {
     ...newEditor,
+    decorations: linkDecorations
+  };
+  set$6(editor.uid, editor, newEditorWithDecorations);
+  const incrementalEdits = await getIncrementalEdits(editor, newEditorWithDecorations);
+  const editorWithNewWidgets = await applyWidgetChanges(newEditorWithDecorations, changes);
+  const newEditor2 = {
+    ...newEditorWithDecorations,
     ...editorWithNewWidgets,
     incrementalEdits
   };
@@ -3308,108 +3416,6 @@ const getLanguages = async (platform, assetDir) => {
   return languages;
 };
 
-/**
- * Gets all regex matches for a given text and regex pattern
- * @param text The text to match against
- * @param regex The regex pattern to use (should have global flag)
- * @returns Array of regex matches
- */
-const getRegexMatches = (text, regex) => {
-  return [...text.matchAll(regex)];
-};
-
-// URL matching regex pattern - matches common URL schemes
-// Supports: http://, https://, ftp://, ftps://, file://
-// Also matches URLs without explicit scheme (www.example.com)
-const URL_PATTERN = /(?:(?:https?|ftp|ftps|file):\/\/)?(?:www\.)?(?:[a-zA-Z0-9](?:[a-zA-Z0-9-]*[a-zA-Z0-9])?\.)+[a-zA-Z0-9](?:[a-zA-Z0-9-]*[a-zA-Z0-9])?(?:\/[^\s]*)?/g;
-
-// Regex to check if URL has a scheme (http://, https://, ftp://, etc.)
-const HAS_SCHEME_PATTERN = /^(?:https?|ftp|ftps|file):\/\//;
-
-// Regex to check if URL starts with www.
-const HAS_WWW_PATTERN = /^www\./;
-
-/**
- * Detects links in a given text and returns their positions
- * @param text The text to scan for links
- * @returns Array of links with their start position and length
- */
-const detectLinks = text => {
-  const matches = getRegexMatches(text, URL_PATTERN);
-  const links = [];
-  for (const match of matches) {
-    const url = match[0];
-    // Only consider as link if it has a scheme or starts with www.
-    if (HAS_SCHEME_PATTERN.test(url) || HAS_WWW_PATTERN.test(url)) {
-      links.push({
-        length: url.length,
-        start: match.index ?? 0
-      });
-    }
-  }
-  return links;
-};
-
-/**
- * Detects all links in an editor and returns them as decorations
- * @param editor The editor containing lines to scan
- * @returns Flat array of decorations in format [offset, length, type, modifiers, ...]
- */
-const detectAllLinksAsDecorations = editor => {
-  const decorations = [];
-  const {
-    lines
-  } = editor;
-  let offset = 0;
-  for (const line of lines) {
-    const links = detectLinks(line);
-    for (const link of links) {
-      const linkOffset = offset + link.start;
-      // Add link decoration: offset, length, type, modifiers
-      decorations.push(linkOffset, link.length, Link, 0);
-    }
-    offset += line.length + 1; // +1 for newline
-  }
-  return decorations;
-};
-
-/**
- * Gets the URL text at a given offset in the editor if it's a link
- * @param editor The editor
- * @param offset The offset in the document
- * @returns The URL string if the offset is on a link, or undefined
- */
-const getUrlAtOffset = (editor, offset) => {
-  const {
-    decorations,
-    lines
-  } = editor;
-
-  // Iterate through decorations in groups of 4 (offset, length, type, modifiers)
-  for (let i = 0; i < decorations.length; i += 4) {
-    const decorationOffset = decorations[i];
-    const decorationLength = decorations[i + 1];
-    const decorationType = decorations[i + 2];
-
-    // Check if this decoration is a link and if the offset falls within it
-    if (decorationType === Link && offset >= decorationOffset && offset < decorationOffset + decorationLength) {
-      // Extract the URL text from the editor content
-      let currentOffset = 0;
-      for (const line of lines) {
-        const lineLength = line.length + 1; // +1 for newline
-        if (currentOffset + lineLength > decorationOffset) {
-          // The link starts in this line
-          const linkStartInLine = decorationOffset - currentOffset;
-          const url = line.slice(linkStartInLine, linkStartInLine + decorationLength);
-          return url;
-        }
-        currentOffset += lineLength;
-      }
-    }
-  }
-  return undefined;
-};
-
 const measureCharacterWidth = async (fontWeight, fontSize, fontFamily, letterSpacing) => {
   return await measureTextWidth('a', fontWeight, fontSize, fontFamily, letterSpacing, false, 0);
 };
@@ -3497,6 +3503,37 @@ const getVisibleDiagnostics = async (editor, diagnostics) => {
   return visibleDiagnostics;
 };
 
+/**
+ * Merges link decorations with diagnostic decorations
+ * Links should always be present, but we also need to include any diagnostic decorations
+ */
+const mergeLinksWithDiagnosticDecorations = (editor, diagnosticDecorations) => {
+  // Get link decorations
+  const linkDecorations = detectAllLinksAsDecorations(editor);
+
+  // Merge with diagnostic decorations
+  const allDecorations = [...linkDecorations, ...diagnosticDecorations];
+
+  // Sort by offset to maintain proper order
+  const sortedDecorations = [];
+  for (let i = 0; i < allDecorations.length; i += 4) {
+    sortedDecorations.push({
+      length: allDecorations[i + 1],
+      modifiers: allDecorations[i + 3],
+      offset: allDecorations[i],
+      type: allDecorations[i + 2]
+    });
+  }
+  sortedDecorations.sort((a, b) => a.offset - b.offset);
+
+  // Flatten back to array format
+  const result = [];
+  for (const dec of sortedDecorations) {
+    result.push(dec.offset, dec.length, dec.type, dec.modifiers);
+  }
+  return result;
+};
+
 const updateDiagnostics = async newState => {
   try {
     // TODO handle error
@@ -3517,11 +3554,13 @@ const updateDiagnostics = async newState => {
       return newState;
     }
     const visualDecorations = await getVisibleDiagnostics(latest.newState, diagnostics);
-    // Re-detect link decorations after text changes
-    const linkDecorations = detectAllLinksAsDecorations(latest.newState);
+    // Get diagnostic decorations from visual decorations (if any)
+    const diagnosticDecorations = visualDecorations.flatMap(deco => [deco.offset, deco.length, deco.type, deco.modifiers || 0]);
+    // Merge link decorations with diagnostic decorations
+    const mergedDecorations = mergeLinksWithDiagnosticDecorations(latest.newState, diagnosticDecorations);
     const newEditor = {
       ...latest.newState,
-      decorations: linkDecorations,
+      decorations: mergedDecorations,
       // Text-level decorations (flat array) for CSS classes
       diagnostics,
       visualDecorations // Visual decorations (objects with x, y, width, height) for squiggly underlines

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lvce-editor/editor-worker",
-  "version": "16.2.0",
+  "version": "16.4.0",
   "repository": {
     "type": "git",
     "url": "git@github.com:lvce-editor/editor-worker.git"