@lvce-editor/editor-worker 5.2.0 → 5.3.0

package/dist/editorWorkerMain.js

@@ -1797,12 +1797,36 @@ const getDiagnosticType = diagnostic => {
 
 const getVisibleDiagnostics = (editor, diagnostics) => {
   const visibleDiagnostics = [];
+  const {
+    width,
+    rowHeight,
+    minLineY,
+    charWidth,
+    letterSpacing,
+    lines,
+    fontWeight,
+    fontSize,
+    fontFamily,
+    isMonospaceFont,
+    tabSize
+  } = editor;
   for (const diagnostic of diagnostics) {
+    const {
+      rowIndex,
+      columnIndex,
+      endColumnIndex
+    } = diagnostic;
+    const columnDelta = endColumnIndex - columnIndex;
+    const diagnosticWidth = columnDelta * charWidth;
+    const endLineDifference = 0;
+    const halfCursorWidth = 0;
+    const x = getX(lines[rowIndex], columnIndex, fontWeight, fontSize, fontFamily, isMonospaceFont, letterSpacing, tabSize, halfCursorWidth, width, charWidth, endLineDifference);
+    const y = getY(rowIndex, minLineY, rowHeight);
     visibleDiagnostics.push({
-      x: diagnostic.columnIndex * editor.columnWidth,
-      y: (diagnostic.rowIndex - editor.minLineY) * editor.rowHeight,
-      width: 20,
-      height: editor.rowHeight,
+      x,
+      y,
+      width: diagnosticWidth,
+      height: rowHeight,
       type: getDiagnosticType(diagnostic)
     });
   }
@@ -1816,7 +1840,12 @@ const updateDiagnostics = async uid => {
     const {
       newState
     } = get$6(uid);
-    // TODO request diagnostics from extension host
+
+    // TODO sync textdocument incrementally
+    // TODO sync and ask for diagnostics at the same time?
+    // TODO throttle diagnostics
+    const content = getText$1(newState);
+    await invoke$2(TextDocumentSyncFull, newState.uri, newState.id, newState.languageId, content);
     const diagnostics = await executeDiagnosticProvider(newState);
     const latest = get$6(uid);
     if (!latest) {
@@ -1831,6 +1860,10 @@ const updateDiagnostics = async uid => {
     set$6(uid, latest.oldState, newEditor);
     await invoke$3('Editor.rerender', uid);
   } catch (error) {
+    // @ts-ignore
+    if (error && error.message.includes('No diagnostic provider found')) {
+      return;
+    }
     console.error(`Failed to update diagnostics: ${error}`);
   }
 };
@@ -8377,9 +8410,6 @@ const loadFont = async (fontName, fontUrl) => {
   }
 };
 
-const load = async (fontName, fontUrl) => {
-  return loadFont(fontName, fontUrl);
-};
 const ensure = async (fontName, fontUrl) => {
   if (isLoaded(fontName)) {
     return;
@@ -8387,7 +8417,7 @@ const ensure = async (fontName, fontUrl) => {
   if (hasPending(fontName)) {
     return getPending(fontName);
   }
-  const promise = load(fontName, fontUrl);
+  const promise = loadFont(fontName, fontUrl);
   setPending(fontName, promise);
   await promise;
   removePending(fontName);
@@ -9326,6 +9356,17 @@ const renderEditor = async id => {
   return commands;
 };
 
+const editorDiagnosticEffect = {
+  isActive(oldEditor, newEditor) {
+    // TODO avoid slow comparison
+    return newEditor.diagnosticsEnabled && JSON.stringify(oldEditor.lines) !== JSON.stringify(newEditor.lines);
+  },
+  // TODO set effects delay / diagnostic delay
+  async apply(editor) {
+    await updateDiagnostics(editor.id);
+  }
+};
+
 const keep = [
 // 'ColorPicker.handleSliderPointerDown',
 // 'ColorPicker.handleSliderPointerMove',
@@ -9383,9 +9424,16 @@ const widgetCommands = {
 
 // TODO wrap commands globally, not per editor
 // TODO only store editor state in editor worker, not in renderer worker also
+
+const effects = [editorDiagnosticEffect];
 const wrapCommand = fn => async (editorUid, ...args) => {
   const oldInstance = get$6(editorUid);
   const newEditor = await fn(oldInstance.newState, ...args);
+  for (const effect of effects) {
+    if (effect.isActive(oldInstance.newState, newEditor)) {
+      effect.apply(newEditor);
+    }
+  }
   set$6(editorUid, oldInstance.newState, newEditor);
   // TODO if possible, rendering should be sync
   const commands = await renderEditor(editorUid);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lvce-editor/editor-worker",
-  "version": "5.2.0",
+  "version": "5.3.0",
   "description": "",
   "main": "dist/editorWorkerMain.js",
   "type": "module",