codemode-lsp 0.1.0 → 0.1.1

package/dist/index.js

@@ -39054,7 +39054,13 @@ function unifiedDiff(file2, original, updated) {
 
 // src/lsp-api.ts
 var import_vscode_languageserver_protocol2 = __toESM(require_api2(), 1);
-import { existsSync as existsSync2, readdirSync, readFileSync as readFileSync2, realpathSync } from "node:fs";
+import {
+  existsSync as existsSync2,
+  readdirSync,
+  readFileSync as readFileSync2,
+  realpathSync,
+  statSync
+} from "node:fs";
 import { dirname as dirname2, isAbsolute, relative, resolve, sep } from "node:path";
 import { fileURLToPath, pathToFileURL } from "node:url";
 
@@ -39067,6 +39073,10 @@ import {
   writeFileSync
 } from "node:fs";
 import { dirname } from "node:path";
+var LSP_DOCUMENT_EXTENSIONS = /\.(ts|tsx|mts|cts|js|jsx|mjs|cjs)$/i;
+function isLspDocumentPath(path) {
+  return LSP_DOCUMENT_EXTENSIONS.test(path);
+}
 
 class DeletedFileError extends Error {
   constructor(relPath) {
@@ -39102,6 +39112,9 @@ class TransactionalBuffer {
   isDeleted(absPath) {
     return this.files.get(absPath)?.state === "deleted";
   }
+  peekText(absPath) {
+    return this.files.get(absPath)?.current;
+  }
   track(absPath) {
     const existing = this.files.get(absPath);
     if (existing)
@@ -39113,10 +39126,11 @@ class TransactionalBuffer {
       state: "clean",
       original,
       current: original,
-      existedOnDisk
+      existedOnDisk,
+      lspVisible: isLspDocumentPath(absPath)
     };
     this.files.set(absPath, tracked);
-    if (existedOnDisk && original !== undefined) {
+    if (tracked.lspVisible && existedOnDisk && original !== undefined) {
       this.client.didOpen(absPath, original);
     }
     return tracked;
@@ -39126,14 +39140,18 @@ class TransactionalBuffer {
     if (tracked.state === "deleted") {
       tracked.state = tracked.existedOnDisk ? "modified" : "created";
       tracked.current = text;
-      this.client.didOpen(absPath, text);
-      this.client.didChangeFullText(absPath, text);
+      if (tracked.lspVisible) {
+        this.client.didOpen(absPath, text);
+        this.client.didChangeFullText(absPath, text);
+      }
       return;
     }
     tracked.current = text;
     if (tracked.state === "clean") {
       tracked.state = tracked.existedOnDisk ? "modified" : "created";
     }
+    if (!tracked.lspVisible)
+      return;
     if (!this.client.isOpen(absPath)) {
       this.client.didOpen(absPath, text);
     } else {
@@ -39150,7 +39168,8 @@ class TransactionalBuffer {
       throw new Error(`Cannot delete "${relPath}": the file does not exist. Use lsp.listFiles() to discover project files.`);
     }
     const wasCreatedThisScript = !tracked.existedOnDisk;
-    this.client.didClose(absPath);
+    if (tracked.lspVisible)
+      this.client.didClose(absPath);
     if (wasCreatedThisScript) {
       this.files.delete(absPath);
       return;
@@ -39167,6 +39186,8 @@ class TransactionalBuffer {
     if (tracked.state === "clean") {
       tracked.state = tracked.existedOnDisk ? "modified" : "created";
     }
+    if (!tracked.lspVisible)
+      return;
     if (!this.client.isOpen(absPath)) {
       this.client.didOpen(absPath, tracked.current);
     } else {
@@ -39235,7 +39256,7 @@ class TransactionalBuffer {
         continue;
       }
       try {
-        if (file2.state === "modified") {
+        if (!file2.lspVisible) {} else if (file2.state === "modified") {
           if (file2.original !== undefined && this.client.isOpen(absPath)) {
             this.client.didChangeFullText(absPath, file2.original);
           }
@@ -39253,6 +39274,8 @@ class TransactionalBuffer {
   }
   rollback() {
     for (const [absPath, file2] of this.files) {
+      if (!file2.lspVisible)
+        continue;
       try {
         if (file2.state === "modified") {
           if (file2.original !== undefined && this.client.isOpen(absPath)) {
@@ -39682,12 +39705,27 @@ class LspApi {
       await this.client.ensureAlive();
   }
   readTextSafe(resolved) {
-    if (this.buffer?.isDeleted(resolved.absPath))
-      return "";
+    if (this.buffer) {
+      if (this.buffer.isDeleted(resolved.absPath))
+        return "";
+      const buffered = this.buffer.peekText(resolved.absPath);
+      if (buffered !== undefined)
+        return buffered;
+    }
     try {
-      return this.readText(resolved);
+      return readFileSync2(resolved.absPath, "utf8");
     } catch {
-      return existsSync2(resolved.absPath) ? readFileSync2(resolved.absPath, "utf8") : "";
+      return "";
+    }
+  }
+  requireStrings(signature, example, args, contentArgs = []) {
+    for (const [name, value] of Object.entries(args)) {
+      const allowEmpty = contentArgs.includes(name);
+      if (typeof value === "string" && (allowEmpty || value.trim() !== "")) {
+        continue;
+      }
+      const got = value === undefined ? "nothing (missing argument)" : value === null ? "null" : Array.isArray(value) ? "an array" : `${typeof value === "object" ? "an" : "a"} ${typeof value}`;
+      throw new Error(`${signature}: "${name}" must be a ${allowEmpty ? "" : "non-empty "}string but got ${got}. Example: ${example}`);
     }
   }
   resolveWorkspacePath(file2) {
@@ -39718,11 +39756,15 @@ class LspApi {
     }
   }
   async readFile(file2) {
+    this.requireStrings("readFile(file)", 'await lsp.readFile("src/auth.ts")', {
+      file: file2
+    });
     const resolved = this.resolveWorkspacePath(file2);
     await this.ensureReadyForBuffer();
     return this.readText(resolved);
   }
   async getSymbolBody(file2, symbolPath) {
+    this.requireStrings("getSymbolBody(file, symbolPath)", 'await lsp.getSymbolBody("src/auth.ts", "AuthService/validate") — get file and symbolPath from getSymbols(file)', { file: file2, symbolPath });
     const resolved = this.resolveWorkspacePath(file2);
     await this.ensureReadyForBuffer();
     const text = this.readText(resolved);
@@ -39737,12 +39779,14 @@ class LspApi {
     return text.slice(start, end);
   }
   async getSymbols(file2) {
+    this.requireStrings("getSymbols(file)", 'await lsp.getSymbols("src/auth.ts")', { file: file2 });
     const resolved = this.resolveWorkspacePath(file2);
     await this.ensureReadyForBuffer();
     const text = this.readText(resolved);
     return buildSymbolInfoTree(await this.documentSymbols(resolved), text);
   }
   async findSymbol(query) {
+    this.requireStrings("findSymbol(query)", 'await lsp.findSymbol("AuthService")', { query });
     const symbols = await this.workspaceSymbolWithIndexWait(query);
     const mapped = [];
     for (const symbol2 of symbols) {
@@ -39768,6 +39812,7 @@ class LspApi {
     return mapped;
   }
   async findReferences(file2, symbolPath) {
+    this.requireStrings("findReferences(file, symbolPath)", 'await lsp.findReferences("src/auth.ts", "AuthService/validate") — get file and symbolPath from getSymbols(file)', { file: file2, symbolPath });
     const resolved = this.resolveWorkspacePath(file2);
     const symbol2 = resolveSymbolPath({
       file: resolved.relPath,
@@ -39797,6 +39842,7 @@ class LspApi {
     return references;
   }
   async goToDefinition(file2, symbolPath) {
+    this.requireStrings("goToDefinition(file, symbolPath)", 'await lsp.goToDefinition("src/auth.ts", "AuthService/validate")', { file: file2, symbolPath });
     const resolved = this.resolveWorkspacePath(file2);
     const symbol2 = resolveSymbolPath({
       file: resolved.relPath,
@@ -39814,6 +39860,10 @@ class LspApi {
     return this.locationToApiLocation(first);
   }
   async searchText(pattern, glob) {
+    this.requireStrings("searchText(pattern, glob?)", 'await lsp.searchText("new NotFoundError\\\\(", "src/**") — the pattern is a regex; escape metacharacters for literal text', { pattern });
+    if (glob !== undefined && glob !== null && typeof glob !== "string") {
+      throw new Error('searchText(pattern, glob?) takes a glob STRING as its second argument, e.g. searchText("TODO", "src/**"). There is no options object — filter or slice the returned array in your script instead.');
+    }
     let regex;
     try {
       regex = new RegExp(pattern, "g");
@@ -39844,8 +39894,38 @@ class LspApi {
     }
     return results;
   }
+  normalizeGlob(glob) {
+    if (glob === undefined || glob === null)
+      return "**/*";
+    if (typeof glob !== "string") {
+      throw new Error(`listFiles/searchText globs must be strings like "src/**/*.ts" (got ${typeof glob}). Omit the glob to cover every file.`);
+    }
+    let candidate = glob.trim();
+    if (candidate.startsWith("./"))
+      candidate = candidate.slice(2);
+    while (candidate.endsWith("/"))
+      candidate = candidate.slice(0, -1);
+    if (candidate === "" || candidate === ".")
+      return "**/*";
+    if (isAbsolute(candidate)) {
+      const rel = relative(this.rootDir, resolve(candidate));
+      if (rel === "")
+        return "**/*";
+      if (rel.startsWith("..") || isAbsolute(rel)) {
+        throw new Error(`Glob "${glob}" points outside the workspace root. Use a workspace-relative pattern like "src/**".`);
+      }
+      candidate = toPosixPath(rel);
+    }
+    if (!/[*?[\]{}]/.test(candidate)) {
+      const abs = resolve(this.rootDir, candidate);
+      if (existsSync2(abs) && statSync(abs).isDirectory()) {
+        return `${candidate}/**`;
+      }
+    }
+    return candidate;
+  }
   async listFiles(glob) {
-    const matcher = globToRegExp(glob ?? "**/*");
+    const matcher = globToRegExp(this.normalizeGlob(glob));
     const files = [];
     const visit = (dir) => {
       for (const entry of readdirSync(dir, { withFileTypes: true })) {
@@ -39867,8 +39947,11 @@ class LspApi {
     return files.sort();
   }
   async getDiagnostics(file2) {
-    if (file2) {
+    if (file2 !== undefined && file2 !== null) {
+      this.requireStrings("getDiagnostics(file?)", 'await lsp.getDiagnostics("src/auth.ts") — or no argument for every file touched this session', { file: file2 });
       const resolved = this.resolveWorkspacePath(file2);
+      if (!isLspDocumentPath(resolved.absPath))
+        return [];
       await this.client.ensureAlive();
       if (this.buffer)
         this.buffer.track(resolved.absPath);
@@ -39880,6 +39963,7 @@ class LspApi {
     return this.client.getTouchedUris().flatMap((uri) => this.convertDiagnosticsForUri(uri, this.client.getDiagnosticsForUris([uri])));
   }
   async renameSymbol(file2, symbolPath, newName) {
+    this.requireStrings("renameSymbol(file, symbolPath, newName)", 'await lsp.renameSymbol("src/auth.ts", "AuthService/validate", "checkToken")', { file: file2, symbolPath, newName });
     const buffer = this.requireBuffer("renameSymbol");
     const resolved = this.resolveWorkspacePath(file2);
     const symbol2 = resolveSymbolPath({
@@ -39920,12 +40004,14 @@ class LspApi {
     return { file: resolved.relPath, filesChanged, diagnostics };
   }
   async replaceSymbolBody(file2, symbolPath, newText) {
+    this.requireStrings("replaceSymbolBody(file, symbolPath, newText)", 'await lsp.replaceSymbolBody("src/auth.ts", "AuthService/validate", "validate(token: Token): boolean { … }")', { file: file2, symbolPath, newText }, ["newText"]);
     return this.editSymbolRange(file2, symbolPath, (symbol2) => ({
       range: symbol2.range,
       newText
     }));
   }
   async insertBeforeSymbol(file2, symbolPath, text) {
+    this.requireStrings("insertBeforeSymbol(file, symbolPath, text)", 'await lsp.insertBeforeSymbol("src/auth.ts", "AuthService", "// header\\n")', { file: file2, symbolPath, text }, ["text"]);
     return this.editSymbolRange(file2, symbolPath, (symbol2) => ({
       range: {
         start: { line: symbol2.range.start.line, character: 0 },
@@ -39937,6 +40023,7 @@ class LspApi {
     }));
   }
   async insertAfterSymbol(file2, symbolPath, text) {
+    this.requireStrings("insertAfterSymbol(file, symbolPath, text)", 'await lsp.insertAfterSymbol("src/auth.ts", "AuthService", "export const x = 1;\\n")', { file: file2, symbolPath, text }, ["text"]);
     return this.editSymbolRange(file2, symbolPath, (symbol2) => ({
       range: { start: symbol2.range.end, end: symbol2.range.end },
       newText: text.startsWith(`
@@ -39945,6 +40032,7 @@ ${text}`
     }));
   }
   async deleteSymbol(file2, symbolPath) {
+    this.requireStrings("deleteSymbol(file, symbolPath)", 'await lsp.deleteSymbol("src/auth.ts", "AuthService/validate")', { file: file2, symbolPath });
     const buffer = this.requireBuffer("deleteSymbol");
     const resolved = this.resolveWorkspacePath(file2);
     await this.client.ensureAlive();
@@ -39964,6 +40052,7 @@ ${text}`
     };
   }
   async writeFile(file2, content) {
+    this.requireStrings("writeFile(file, content)", 'await lsp.writeFile("src/new.ts", "export const x = 1;\\n")', { file: file2, content }, ["content"]);
     const buffer = this.requireBuffer("writeFile");
     const resolved = this.resolveWorkspacePath(file2);
     await this.client.ensureAlive();
@@ -39976,6 +40065,7 @@ ${text}`
     };
   }
   async deleteFile(file2) {
+    this.requireStrings("deleteFile(file)", 'await lsp.deleteFile("src/old.ts")', { file: file2 });
     const buffer = this.requireBuffer("deleteFile");
     const resolved = this.resolveWorkspacePath(file2);
     await this.client.ensureAlive();
@@ -40058,7 +40148,9 @@ ${text}`
     return byPath;
   }
   async collectDiagnostics(absPaths) {
-    const uris = absPaths.map((absPath) => pathToFileURL(absPath).href);
+    const uris = absPaths.filter((absPath) => isLspDocumentPath(absPath)).map((absPath) => pathToFileURL(absPath).href);
+    if (uris.length === 0)
+      return [];
     await this.client.waitForDiagnosticsForUris(uris);
     return uris.flatMap((uri) => this.convertDiagnosticsForUri(uri, this.client.getDiagnosticsForUris([uri])));
   }
@@ -46053,9 +46145,36 @@ function capText(text, cap, marker) {
     return text;
   return text.slice(0, cap) + marker;
 }
+function findThenablePath(value, path, seen) {
+  if (value === null || typeof value !== "object")
+    return null;
+  if (typeof value.then === "function")
+    return path;
+  if (seen.has(value))
+    return null;
+  seen.add(value);
+  if (Array.isArray(value)) {
+    for (let i2 = 0;i2 < value.length; i2 += 1) {
+      const found = findThenablePath(value[i2], `${path}[${i2}]`, seen);
+      if (found)
+        return found;
+    }
+    return null;
+  }
+  for (const [key, child] of Object.entries(value)) {
+    const found = findThenablePath(child, `${path}.${key}`, seen);
+    if (found)
+      return found;
+  }
+  return null;
+}
 function serializeResult(value) {
   if (value === undefined)
     return "undefined";
+  const thenableAt = findThenablePath(value, "result", new Set);
+  if (thenableAt !== null) {
+    throw new Error(`Script result contains a Promise at "${thenableAt}" — did you forget ` + "await? Every lsp.* function is async; write e.g. " + '`const files = await lsp.listFiles("src/**");`.');
+  }
   if (typeof value === "function") {
     throw new Error("Script returned a function, which cannot be serialized. Return a " + "JSON-serializable value instead (string, number, boolean, array, or " + "plain object) — e.g. call the function and return its result.");
   }
@@ -46189,7 +46308,7 @@ ${source}
 // src/lsp-types.generated.ts
 var LSP_COMMON_INTERFACES = `interface SymbolInfo {
   name: string;
-  /** Exact slash-separated path for use in other lsp.* calls. */
+  /** Slash-separated path within its file; pass it TOGETHER WITH the same \`file\` to other lsp.* calls. */
   path: string;
   /** "class" | "function" | "method" | "variable" | ... */
   kind: string;
@@ -46272,15 +46391,15 @@ var LSP_READ_OP_SIGNATURES = `  /** File contents as a raw string (no line numbe
   getSymbolBody(file: string, symbolPath: string): Promise<string>;
   /** Document symbol tree (file outline). Every path is a usable handle. */
   getSymbols(file: string): Promise<SymbolInfo[]>;
-  /** Workspace-wide symbol search; paths are best-effort (confirm with getSymbols). */
+  /** Workspace-wide symbol search; the index warms lazily, so early calls may be empty — getSymbols(file) is exhaustive. */
   findSymbol(query: string): Promise<WorkspaceSymbolInfo[]>;
   /** All references to a symbol across the workspace (incl. the declaration). */
   findReferences(file: string, symbolPath: string): Promise<Reference[]>;
   /** Jump to a symbol's definition. */
   goToDefinition(file: string, symbolPath: string): Promise<Location>;
-  /** Regex search across project files (respects .gitignore). */
+  /** Regex search across project files — escape metacharacters for literal text; optional second arg is a glob string. */
   searchText(pattern: string, glob?: string): Promise<SearchResult[]>;
-  /** Project files matching a glob (respects .gitignore); no glob = all files. */
+  /** Project files matching a glob; no glob = all files, a directory name means everything under it. */
   listFiles(glob?: string): Promise<string[]>;
   /** Diagnostics for a file, or every file touched this session (not project-wide). */
   getDiagnostics(file?: string): Promise<Diagnostic[]>;`;
@@ -46379,9 +46498,19 @@ script's last expression, JSON-serialized.
 
 - The last expression is the return value — end the script with the value you
   want back, e.g. \`({ count })\`.
+- Every \`lsp.*\` call returns a Promise — always \`await\` it. An un-awaited
+  call inside the result serializes as \`{}\`.
 - \`.filter()\`/\`.map()\` callbacks cannot be async — use \`for...of\` with \`await\`.
+- Globs match the whole workspace-relative path: \`listFiles("src/**")\` for a
+  directory (a bare directory name like \`"src"\` is treated as \`src/**\`),
+  \`listFiles()\` for every file. \`searchText\`'s second argument is a glob
+  string — there is no options object; filter results in your script.
 - Symbol paths are slash-separated (\`MyClass/myMethod\`); discover exact paths
-  with \`getSymbols(file)\` rather than guessing.
+  with \`getSymbols(file)\` rather than guessing. Symbol ops always take the
+  pair \`(file, symbolPath)\` — a \`SymbolInfo.path\` belongs to the file you
+  called \`getSymbols\` on.
+- \`searchText\` patterns are regexes — escape metacharacters for literal text:
+  \`searchText("new NotFoundError\\\\(")\`.
 - File paths are relative to the workspace root; anything outside it is
   rejected.
 - Diagnostics cover files touched this session only, never the whole project.`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codemode-lsp",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "MCP server exposing a single code-mode tool backed by LSP",
   "keywords": [
     "mcp",