@knip/language-server 1.2.2 → 2.0.0

package/package.json

@@ -1,12 +1,15 @@
 {
   "name": "@knip/language-server",
-  "version": "1.2.2",
+  "version": "2.0.0",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/webpro-nl/knip.git",
     "directory": "packages/language-server"
   },
   "type": "module",
+  "bin": {
+    "knip-language-server": "./src/cli.js"
+  },
   "exports": {
     ".": {
       "types": "./src/types.d.ts",
@@ -17,7 +20,7 @@
   "dependencies": {
     "vscode-languageserver": "^9.0.1",
     "vscode-languageserver-textdocument": "^1.0.12",
-    "knip": "^5.83.1"
+    "knip": "^5.84.0"
   },
   "engines": {
     "node": ">=18.20.0"

package/src/cli.js

@@ -0,0 +1,2 @@
+#!/usr/bin/env node
+import './index.js';

package/src/index.js

@@ -1,3 +1,6 @@
 import { LanguageServer } from './server.js';
 
+const transports = ['--stdio', '--socket', '--node-ipc', '--pipe'];
+if (!process.argv.some(arg => transports.includes(arg))) process.argv.push('--stdio');
+
 new LanguageServer();

package/src/server.js

@@ -26,6 +26,21 @@ import { issueToDiagnostic } from './diagnostics.js';
 
 const RESTART_FOR = new Set(['package.json', ...KNIP_CONFIG_LOCATIONS]);
 
+/** @type {Config} */
+const DEFAULT_CONFIG = {
+  deferSession: false,
+  editor: {
+    exports: {
+      codelens: { enabled: true },
+      hover: { enabled: true, includeImportLocationSnippet: false, maxSnippets: 10, timeout: 300 },
+      quickfix: { enabled: true },
+      highlight: { dimExports: false, dimTypes: false },
+    },
+  },
+  imports: { enabled: true },
+  exports: { enabled: true, contention: { enabled: true } },
+};
+
 /** @param {string} value */
 const toPosix = value => value.split(path.sep).join(path.posix.sep);
 
@@ -77,9 +92,16 @@ export class LanguageServer {
   /** @type TextDocuments<TextDocument> */
   documents;
 
+  /** @type {Config | undefined} */
+  initConfig;
+
   constructor() {
     this.connection = createConnection(ProposedFeatures.all);
     this.documents = new TextDocuments(TextDocument);
+    console.log = this.connection.console.log.bind(this.connection.console);
+    console.warn = this.connection.console.warn.bind(this.connection.console);
+    console.error = this.connection.console.error.bind(this.connection.console);
+    console.info = this.connection.console.info.bind(this.connection.console);
     this.setupHandlers();
     this.documents.listen(this.connection);
     this.connection.listen();
@@ -93,6 +115,8 @@ export class LanguageServer {
 
       this.cwd = fileURLToPath(uri);
 
+      this.initConfig = params.initializationOptions?.config;
+
       const capabilities = {
         codeActionProvider: {
           codeActionKinds: [CodeActionKind.QuickFix],
@@ -108,7 +132,10 @@ export class LanguageServer {
       };
     });
 
-    this.connection.onInitialized(() => {});
+    this.connection.onInitialized(async () => {
+      const config = await this.getConfig();
+      if (config?.deferSession !== true) this.start();
+    });
 
     this.connection.onRequest(REQUEST_START, () => this.start());
 
@@ -135,7 +162,11 @@ export class LanguageServer {
 
   /** @returns {Promise<Config>} */
   async getConfig() {
-    return await this.connection.workspace.getConfiguration('knip');
+    try {
+      const config = await this.connection.workspace.getConfiguration('knip');
+      if (config?.editor) return config;
+    } catch {}
+    return this.initConfig ?? DEFAULT_CONFIG;
   }
 
   /**
@@ -181,17 +212,13 @@ export class LanguageServer {
 
     try {
       const config = await this.getConfig();
-      if (!config?.enabled) return;
 
-      const configFilePath = config.configFilePath
+      const configFilePath = config?.configFilePath
         ? path.isAbsolute(config.configFilePath)
           ? config.configFilePath
           : path.resolve(this.cwd ?? process.cwd(), config.configFilePath)
         : undefined;
 
-      if (configFilePath) this.cwd = path.dirname(configFilePath);
-      if (this.cwd) process.chdir(this.cwd);
-
       this.connection.console.log('Creating options');
       const options = await createOptions({ cwd: this.cwd, isSession: true, args: { config: configFilePath } });
       this.rules = options.rules;
@@ -235,23 +262,29 @@ export class LanguageServer {
     this.packageJsonCache = undefined;
     if (!this.session) return;
 
+    const cwd = this.cwd ?? process.cwd();
+
     /** @type {{ type: "added" | "deleted" | "modified"; filePath: string }[]} */
     const changes = [];
     for (const change of params.changes) {
+      if (!change.uri.startsWith('file:')) continue;
       const filePath = fileURLToPath(change.uri);
+      if (!filePath.startsWith(cwd) || filePath.includes('/.git/')) continue;
       if (RESTART_FOR.has(path.basename(change.uri))) return this.restart();
       const type = FILE_CHANGE_TYPES.get(change.type);
       if (!type) continue;
       changes.push({ type, filePath });
     }
 
+    if (changes.length === 0) return;
+
     const result = await this.session.handleFileChanges(changes);
 
-    if (result) {
-      this.connection.console.log(
-        `Module graph updated (${Math.floor(result.duration)}ms • ${(result.mem / 1024 / 1024).toFixed(2)}M)`
-      );
-    }
+    if (!result) return;
+
+    this.connection.console.log(
+      `Module graph updated (${Math.floor(result.duration)}ms • ${(result.mem / 1024 / 1024).toFixed(2)}M)`
+    );
 
     const config = await this.getConfig();
     this.publishDiagnostics(this.buildDiagnostics(this.session.getIssues().issues, config, this.rules));

package/src/types.d.ts

@@ -1,7 +1,7 @@
 import type { Issue, IssueType } from 'knip/session';
 
 export type Config = {
-  enabled: boolean;
+  deferSession: boolean;
   configFilePath?: string;
   editor: {
     exports: {