@theupsider/lsp-mcp 0.1.1 → 0.1.4

package/dist/mcp/tools/read-tools.js

@@ -54,6 +54,7 @@ function registerReadTools(registrar, lifecycleManager, options) {
     });
     registrar.registerTool('lsp_workspace_symbols', { description: 'Search workspace symbols', inputSchema: zod_1.z.object({ query: zod_1.z.string().default('') }) }, async (args) => {
         const query = typeof args.query === 'string' ? args.query : '';
+        await lifecycleManager.ensureSeedFilesOpen();
         const results = await Promise.all(lifecycleManager.getReadyClients().map(async (client) => {
             return await client.request('workspace/symbol', { query }, 30000);
         }));
@@ -70,14 +71,24 @@ function registerReadTools(registrar, lifecycleManager, options) {
             raw: (result) => asCompletionItems(result)?.slice(0, 50) ?? null
         });
     });
-    registrar.registerTool('lsp_diagnostics', { description: 'Show diagnostics', inputSchema: zod_1.z.object({ file: zod_1.z.string().optional(), scope: zod_1.z.enum(['file', 'workspace']).default('file'), language: zod_1.z.string().optional() }) }, async (args) => {
+    registrar.registerTool('lsp_diagnostics', { description: 'Get errors and warnings. Pass a file path to trigger analysis of that file and its language server, then return diagnostics for that file (scope: file) or all files seen so far (scope: workspace). Omit file for workspace scope to query whatever has been opened previously.', inputSchema: zod_1.z.object({ file: zod_1.z.string().optional(), scope: zod_1.z.enum(['file', 'workspace']).default('file'), language: zod_1.z.string().optional() }) }, async (args) => {
+        const filePath = typeof args.file === 'string' ? args.file : '';
         const scope = args.scope === 'workspace' ? 'workspace' : 'file';
+        if (filePath) {
+            await lifecycleManager.ensureLanguageForFile(filePath);
+            const client = lifecycleManager.getClientForFile(filePath);
+            if (client) {
+                const waitPromise = client.waitForDiagnosticsPublish(filePath, 10000);
+                await client.ensureDidOpen(filePath);
+                client.notify('textDocument/didSave', { textDocument: { uri: (0, uri_1.pathToUri)(filePath) } });
+                await waitPromise;
+            }
+        }
         if (scope === 'workspace') {
             const language = typeof args.language === 'string' ? args.language : undefined;
             const diagnostics = lifecycleManager.getWorkspaceDiagnostics(language).slice(0, 200);
             return (0, shared_1.success)((0, formatters_1.formatDiagnostics)(diagnostics, 'workspace'), diagnostics);
         }
-        const filePath = typeof args.file === 'string' ? args.file : '';
         const diagnostics = lifecycleManager.getFileDiagnostics(filePath);
         return (0, shared_1.success)((0, formatters_1.formatDiagnostics)(diagnostics, 'file'), diagnostics);
     });
@@ -168,7 +179,7 @@ async function runFileRequest(options) {
     };
     const uri = (0, uri_1.pathToUri)(filePath);
     try {
-        await (0, shared_1.ensureDidOpen)(client, filePath);
+        await client.ensureDidOpen(filePath);
         const result = await client.request(options.method, options.params?.(uri, position) ?? {
             textDocument: { uri },
             position

package/dist/mcp/tools/shared.js

@@ -5,41 +5,19 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.success = success;
 exports.failure = failure;
-exports.ensureDidOpen = ensureDidOpen;
-exports.clearOpenedFiles = clearOpenedFiles;
 exports.noServerResult = noServerResult;
 exports.noProjectRootResult = noProjectRootResult;
 exports.mapToolError = mapToolError;
 exports.normalizeLocations = normalizeLocations;
 exports.normalizeSymbols = normalizeSymbols;
-const promises_1 = require("node:fs/promises");
 const node_path_1 = __importDefault(require("node:path"));
 const uri_1 = require("../../utils/uri");
-const openedFiles = new Set();
 function success(text, raw) {
     return { content: [{ type: 'text', text }], raw };
 }
 function failure(text, raw = null) {
     return { content: [{ type: 'text', text }], error: true, raw };
 }
-async function ensureDidOpen(client, filePath) {
-    if (openedFiles.has(filePath)) {
-        return;
-    }
-    const text = await (0, promises_1.readFile)(filePath, 'utf8');
-    client.notify('textDocument/didOpen', {
-        textDocument: {
-            uri: (0, uri_1.pathToUri)(filePath),
-            languageId: languageIdForFile(filePath),
-            version: 1,
-            text
-        }
-    });
-    openedFiles.add(filePath);
-}
-function clearOpenedFiles() {
-    openedFiles.clear();
-}
 function noServerResult(filePath) {
     return failure(`No language server available for ${node_path_1.default.extname(filePath) || 'unknown'} files. Run lsp_health for details.`);
 }
@@ -82,25 +60,3 @@ function normalizeSymbols(symbols) {
     }
     return normalized;
 }
-function languageIdForFile(filePath) {
-    const ext = node_path_1.default.extname(filePath).toLowerCase();
-    const languageMap = {
-        '.ts': 'typescript',
-        '.tsx': 'typescriptreact',
-        '.js': 'javascript',
-        '.jsx': 'javascriptreact',
-        '.py': 'python',
-        '.go': 'go',
-        '.rs': 'rust',
-        '.java': 'java',
-        '.cs': 'csharp',
-        '.php': 'php',
-        '.rb': 'ruby',
-        '.kt': 'kotlin',
-        '.swift': 'swift',
-        '.c': 'c',
-        '.cpp': 'cpp',
-        '.h': 'c'
-    };
-    return languageMap[ext] ?? 'plaintext';
-}

package/dist/mcp/tools/write-tools.js

@@ -21,7 +21,7 @@ function registerWriteTools(registrar, lifecycleManager) {
             return (0, shared_1.failure)('Rename is not supported by the active language server.');
         }
         try {
-            await (0, shared_1.ensureDidOpen)(client, filePath);
+            await client.ensureDidOpen(filePath);
             const edit = await client.request('textDocument/rename', {
                 textDocument: { uri: (0, uri_1.pathToUri)(filePath) },
                 position: getPosition(args),
@@ -41,7 +41,7 @@ function registerWriteTools(registrar, lifecycleManager) {
             return (0, shared_1.noServerResult)(filePath);
         }
         try {
-            await (0, shared_1.ensureDidOpen)(client, filePath);
+            await client.ensureDidOpen(filePath);
             const range = isRange(args.range)
                 ? args.range
                 : { start: getPosition(args), end: getPosition(args) };
@@ -108,7 +108,7 @@ async function runFormattingRequest(_method, args, lifecycleManager, requestEdit
         return (0, shared_1.noServerResult)(filePath);
     }
     try {
-        await (0, shared_1.ensureDidOpen)(client, filePath);
+        await client.ensureDidOpen(filePath);
         const options = await resolveFormattingOptions(filePath, args.options);
         const edits = await requestEdits(client, filePath, options);
         return await applyTextEdits(edits, [filePath], lifecycleManager, client);
@@ -126,7 +126,7 @@ async function applyWorkspaceEdit(edit, lifecycleManager, fallbackClient, verb)
     for (const [uri, edits] of changeEntries) {
         const filePath = (0, uri_1.uriToPath)(uri);
         const client = lifecycleManager.getClientForFile(filePath) ?? fallbackClient;
-        await (0, shared_1.ensureDidOpen)(client, filePath);
+        await client.ensureDidOpen(filePath);
         await applyEditsToFile(filePath, edits ?? []);
         client.notify('textDocument/didSave', { textDocument: { uri } });
         changedFiles.push(filePath);
@@ -135,7 +135,7 @@ async function applyWorkspaceEdit(edit, lifecycleManager, fallbackClient, verb)
         if ('textDocument' in change && 'edits' in change) {
             const filePath = (0, uri_1.uriToPath)(change.textDocument.uri);
             const client = lifecycleManager.getClientForFile(filePath) ?? fallbackClient;
-            await (0, shared_1.ensureDidOpen)(client, filePath);
+            await client.ensureDidOpen(filePath);
             await applyEditsToFile(filePath, change.edits);
             client.notify('textDocument/didSave', { textDocument: { uri: change.textDocument.uri } });
             changedFiles.push(filePath);

package/package.json

@@ -1,9 +1,13 @@
 {
   "name": "@theupsider/lsp-mcp",
-  "version": "0.1.1",
+  "version": "0.1.4",
   "description": "Universal LSP MCP server",
   "license": "Apache-2.0",
   "type": "commonjs",
+  "author": {
+    "name": "David Fischer",
+    "url": "https://davidfischer.dev"
+  },
   "main": "dist/index.js",
   "bin": {
     "lsp-mcp": "dist/index.js"
@@ -29,4 +33,4 @@
     "ts-jest": "latest",
     "typescript": "latest"
   }
-}
+}