@rohal12/spindle-lsp 0.5.3 → 0.6.0

package/dist/bin.js

@@ -1193,7 +1193,8 @@ var init_diagnostic_codes = __esm({
       BrokenPassageLink: "SP300",
       WidgetArgCountMismatch: "SP301",
       DeadEndPassage: "SP400",
-      UnreachablePassage: "SP401"
+      UnreachablePassage: "SP401",
+      LineTooLong: "SP500"
     };
     severityMap = {
       SP100: "warning",
@@ -1213,7 +1214,8 @@ var init_diagnostic_codes = __esm({
       SP300: "warning",
       SP301: "warning",
       SP400: "hint",
-      SP401: "hint"
+      SP401: "hint",
+      SP500: "warning"
     };
   }
 });
@@ -2092,7 +2094,7 @@ var init_link_parser = __esm({
 });
 
 // src/plugins/diagnostics.ts
-function computeDiagnostics(uri, workspace) {
+function computeDiagnostics(uri, workspace, options) {
   if (!workspace.initialized) return [];
   try {
     const text = workspace.documents.getText(uri);
@@ -2124,6 +2126,12 @@ function computeDiagnostics(uri, workspace) {
       validateWidgetInvocations(macros, workspace, diagnostics);
     } catch {
     }
+    if (options?.maxLineLength) {
+      try {
+        validateLineLength(text, passages, options.maxLineLength, diagnostics);
+      } catch {
+      }
+    }
     return diagnostics;
   } catch {
     return [];
@@ -2342,6 +2350,26 @@ function validateWidgetInvocations(macros, workspace, diagnostics) {
     }
   }
 }
+function validateLineLength(text, passages, maxLength, diagnostics) {
+  const excludedTags = /* @__PURE__ */ new Set(["script", "stylesheet"]);
+  const excludedPassages = new Set(passages.filter((p) => p.tags?.some((t) => excludedTags.has(t))).map((p) => p.name));
+  const lines = text.split("\n");
+  for (let i = 0; i < lines.length; i++) {
+    const line = lines[i];
+    if (/^::\s+/.test(line)) continue;
+    if (/^\s*<[a-zA-Z]/.test(line)) continue;
+    if (line.length > maxLength) {
+      diagnostics.push(makeDiag(
+        {
+          start: { line: i, character: maxLength },
+          end: { line: i, character: line.length }
+        },
+        DiagnosticCode.LineTooLong,
+        `Line exceeds ${maxLength} characters (${line.length})`
+      ));
+    }
+  }
+}
 function makeDiag(range, code, message) {
   return {
     range,
@@ -2639,6 +2667,7 @@ function parseArgs(args2) {
     format: "pretty",
     severity: null,
     configPath: null,
+    maxLineLength: null,
     patterns: []
   };
   let i = 0;
@@ -2674,6 +2703,14 @@ function parseArgs(args2) {
     } else if (arg.startsWith("--config=")) {
       options.configPath = arg.slice("--config=".length);
       i++;
+    } else if (arg === "--max-line-length" && i + 1 < args2.length) {
+      const n = parseInt(args2[i + 1], 10);
+      if (!isNaN(n) && n > 0) options.maxLineLength = n;
+      i += 2;
+    } else if (arg.startsWith("--max-line-length=")) {
+      const n = parseInt(arg.slice("--max-line-length=".length), 10);
+      if (!isNaN(n) && n > 0) options.maxLineLength = n;
+      i++;
     } else if (!arg.startsWith("--")) {
       options.patterns.push(arg);
       i++;
@@ -2751,10 +2788,14 @@ async function runCheck(args2) {
   if (Object.keys(projectConfig.macros).length > 0) {
     workspace.macros.loadSupplements(projectConfig.macros);
   }
+  const diagOptions = {};
+  if (options.maxLineLength !== null) {
+    diagOptions.maxLineLength = options.maxLineLength;
+  }
   const allResults = [];
   let hasErrors = false;
   for (const uri of fileContents.keys()) {
-    let diags = computeDiagnostics(uri, workspace);
+    let diags = computeDiagnostics(uri, workspace, diagOptions);
     if (options.severity) {
       diags = filterBySeverity(diags, options.severity);
     }
@@ -3209,6 +3250,13 @@ function indentMacros(lines, isBlock, isDedenting) {
     if (openMatch && isBlock(openMatch[1])) {
       indentLevel++;
     }
+    const closeGlobal = /\{\/[A-Za-z][\w-]*\s*\}/g;
+    let cm;
+    while ((cm = closeGlobal.exec(trimmed)) !== null) {
+      if (cm.index > 0) {
+        indentLevel = Math.max(0, indentLevel - 1);
+      }
+    }
   }
   return result;
 }
@@ -5589,8 +5637,34 @@ var init_server2 = __esm({
 });
 
 // src/bin.ts
+import { createRequire as createRequire2 } from "node:module";
 var args = process.argv.slice(2);
-if (args[0] === "check") {
+var VERSION = createRequire2(import.meta.url)("../package.json").version;
+var HELP = `spindle-lsp v${VERSION}
+Language Server Protocol server for Spindle story format
+
+Usage:
+  spindle-lsp                Start the LSP server (stdio)
+  spindle-lsp check <files>  Lint .twee/.tw files
+  spindle-lsp format <files> Format .twee/.tw files in place
+  spindle-lsp mcp            Start the MCP server
+
+Options:
+  --help, -h                 Show this help message
+  --version, -v              Show version number
+
+Check options:
+  --format <pretty|json|sarif>   Output format (default: pretty)
+  --severity <level>             Minimum severity: error, warning, info, hint
+  --config <path>                Path to config file
+  --max-line-length <n>          Warn on lines exceeding n characters`;
+if (args.includes("--help") || args.includes("-h") || args.length === 0) {
+  console.log(HELP);
+  process.exit(0);
+} else if (args.includes("--version") || args.includes("-v")) {
+  console.log(VERSION);
+  process.exit(0);
+} else if (args[0] === "check") {
   const { runCheck: runCheck2 } = await Promise.resolve().then(() => (init_check(), check_exports));
   const exitCode = await runCheck2(args.slice(1));
   process.exit(exitCode);