@rohal12/spindle-lsp 0.7.1 → 0.8.0

package/dist/bin.js

@@ -448,8 +448,10 @@ var init_variable_tracker = __esm({
     VariableTracker = class {
       declared = /* @__PURE__ */ new Map();
       _hasStoryVariables = false;
+      _nullDeclarations = [];
       declaredTransient = /* @__PURE__ */ new Map();
       _hasStoryTransients = false;
+      _nullTransientDeclarations = [];
       /** Per-URI list of variable usages. */
       usagesByUri = /* @__PURE__ */ new Map();
       /** Per-URI list of transient variable usages. */
@@ -458,17 +460,32 @@ var init_variable_tracker = __esm({
        * Parse the StoryVariables passage content for declarations.
        * Each line like `$name = value` becomes a declaration.
        */
-      parseStoryVariables(content) {
+      parseStoryVariables(content, contentStartLine = 0) {
         this.declared.clear();
         this._hasStoryVariables = true;
+        this._nullDeclarations = [];
         const lines = content.split("\n");
-        for (const line of lines) {
-          const trimmed = line.trim();
+        for (let i = 0; i < lines.length; i++) {
+          const trimmed = lines[i].trim();
           if (!trimmed || trimmed.startsWith("//") || trimmed.startsWith("<!--")) continue;
           const match = /^\$([A-Za-z_$][\w$]*)\s*=\s*(.*)$/.exec(trimmed);
           if (!match) continue;
           const name = match[1];
           const expr = match[2].trim();
+          if (expr === "null") {
+            const charIdx = lines[i].indexOf("null", lines[i].indexOf("="));
+            const absLine = contentStartLine + i;
+            this._nullDeclarations.push({
+              name,
+              sigil: "$",
+              range: {
+                start: { line: absLine, character: charIdx },
+                end: { line: absLine, character: charIdx + 4 }
+              }
+            });
+            this.declared.set(name, { name, sigil: "$" });
+            continue;
+          }
           const decl = { name, sigil: "$" };
           if (expr.startsWith("{")) {
             const fieldRegex = /(\w+)\s*:/g;
@@ -488,17 +505,32 @@ var init_variable_tracker = __esm({
        * Parse the StoryTransients passage content for declarations.
        * Each line like `%name = value` becomes a declaration.
        */
-      parseStoryTransients(content) {
+      parseStoryTransients(content, contentStartLine = 0) {
         this.declaredTransient.clear();
         this._hasStoryTransients = true;
+        this._nullTransientDeclarations = [];
         const lines = content.split("\n");
-        for (const line of lines) {
-          const trimmed = line.trim();
+        for (let i = 0; i < lines.length; i++) {
+          const trimmed = lines[i].trim();
           if (!trimmed || trimmed.startsWith("//") || trimmed.startsWith("<!--")) continue;
           const match = /^%([A-Za-z_$][\w$]*)\s*=\s*(.*)$/.exec(trimmed);
           if (!match) continue;
           const name = match[1];
           const expr = match[2].trim();
+          if (expr === "null") {
+            const charIdx = lines[i].indexOf("null", lines[i].indexOf("="));
+            const absLine = contentStartLine + i;
+            this._nullTransientDeclarations.push({
+              name,
+              sigil: "%",
+              range: {
+                start: { line: absLine, character: charIdx },
+                end: { line: absLine, character: charIdx + 4 }
+              }
+            });
+            this.declaredTransient.set(name, { name, sigil: "%" });
+            continue;
+          }
           const decl = { name, sigil: "%" };
           if (expr.startsWith("{")) {
             const fieldRegex = /(\w+)\s*:/g;
@@ -660,6 +692,14 @@ var init_variable_tracker = __esm({
       hasStoryTransients() {
         return this._hasStoryTransients;
       }
+      /** Get variables declared with null values in StoryVariables. */
+      getNullDeclarations() {
+        return this._nullDeclarations;
+      }
+      /** Get transient variables declared with null values in StoryTransients. */
+      getNullTransientDeclarations() {
+        return this._nullTransientDeclarations;
+      }
     };
   }
 });
@@ -1238,7 +1278,7 @@ var init_workspace_model = __esm({
               }
             }
             const content = lines.slice(contentStart, contentEnd).join("\n");
-            this.variables.parseStoryVariables(content);
+            this.variables.parseStoryVariables(content, contentStart);
           }
         }
         const storyTransients = this.passages.getStoryTransients();
@@ -1255,7 +1295,7 @@ var init_workspace_model = __esm({
               }
             }
             const content = lines.slice(contentStart, contentEnd).join("\n");
-            this.variables.parseStoryTransients(content);
+            this.variables.parseStoryTransients(content, contentStart);
           }
         }
         for (const uri of this.documents.getUris()) {
@@ -1306,6 +1346,7 @@ var init_diagnostic_codes = __esm({
       UndeclaredField: "SP201",
       NoStoryVariables: "SP202",
       UndeclaredTransient: "SP203",
+      NullVariableValue: "SP204",
       BrokenPassageLink: "SP300",
       WidgetArgCountMismatch: "SP301",
       DeadEndPassage: "SP400",
@@ -1324,10 +1365,11 @@ var init_diagnostic_codes = __esm({
       SP112: "warning",
       SP114: "error",
       SP115: "error",
-      SP200: "warning",
+      SP200: "error",
       SP201: "warning",
       SP202: "info",
       SP203: "warning",
+      SP204: "error",
       SP300: "warning",
       SP301: "warning",
       SP400: "hint",
@@ -2434,6 +2476,16 @@ function validateVariables(uri, workspace, diagnostics) {
         `Variable '$${u.name}' is not declared in StoryVariables`
       ));
     }
+    const storyVars = workspace.passages.getStoryVariables();
+    if (storyVars && storyVars.uri === uri) {
+      for (const nd of workspace.variables.getNullDeclarations()) {
+        diagnostics.push(makeDiag(
+          nd.range,
+          DiagnosticCode.NullVariableValue,
+          `Variable '$${nd.name}' is set to null. Spindle does not support null \u2014 use a valid default (number, string, boolean, array, or object).`
+        ));
+      }
+    }
   }
   if (workspace.variables.hasStoryTransients()) {
     const undeclaredTransient = workspace.variables.getUndeclaredTransient(uri);
@@ -2444,6 +2496,16 @@ function validateVariables(uri, workspace, diagnostics) {
         `Transient variable '%${u.name}' is not declared in StoryTransients`
       ));
     }
+    const storyTransients = workspace.passages.getStoryTransients();
+    if (storyTransients && storyTransients.uri === uri) {
+      for (const nd of workspace.variables.getNullTransientDeclarations()) {
+        diagnostics.push(makeDiag(
+          nd.range,
+          DiagnosticCode.NullVariableValue,
+          `Transient variable '%${nd.name}' is set to null. Spindle does not support null \u2014 use a valid default (number, string, boolean, array, or object).`
+        ));
+      }
+    }
   }
 }
 function validateLinks(text, passages, passageNames, diagnostics) {
@@ -5387,6 +5449,11 @@ function computeCodeActions(uri, diagnostics, workspace) {
         if (action) actions.push(action);
         break;
       }
+      case DiagnosticCode.NullVariableValue: {
+        const action = fixNullVariableValue(uri, diag);
+        if (action) actions.push(action);
+        break;
+      }
     }
   }
   return actions;
@@ -5443,7 +5510,7 @@ function fixUndeclaredVariable(diag, workspace) {
         start: { line: insertLine, character: 0 },
         end: { line: insertLine, character: 0 }
       },
-      newText: `$${varName} = null
+      newText: `$${varName} = 0
 `
     }]
   };
@@ -5498,11 +5565,23 @@ function fixUndeclaredTransient(diag, workspace) {
         start: { line: insertLine, character: 0 },
         end: { line: insertLine, character: 0 }
       },
-      newText: `%${varName} = null
+      newText: `%${varName} = 0
 `
     }]
   };
 }
+function fixNullVariableValue(uri, diag) {
+  return {
+    title: "Replace null with 0",
+    kind: "quickfix",
+    diagnosticCodes: [DiagnosticCode.NullVariableValue],
+    edits: [{
+      uri,
+      range: diag.range,
+      newText: "0"
+    }]
+  };
+}
 function toLspRange6(r) {
   return {
     start: { line: r.start.line, character: r.start.character },