pi-lens 3.8.23 → 3.8.24

package/CHANGELOG.md

@@ -4,6 +4,16 @@ All notable changes to pi-lens will be documented in this file.
 
 ## [Unreleased]
 
+## [3.8.24] - 2026-04-12
+
+### Changed
+- **Lazy bootstrap client loading** — startup now defers heavy client initialization behind a shared bootstrap promise, reducing first-turn startup overhead while preserving tool behavior.
+- **LSP config discovery scope** — `.pi-lens/lsp.json` (and related config paths) are now resolved from the current directory up through parent directories, improving nested-workspace support.
+- **Ruby server fallback chain** — Ruby LSP startup now tries `ruby-lsp`, then `solargraph`, then `rubocop --lsp` for broader environment compatibility.
+
+### Fixed
+- **LSP config activation timing** — LSP server config initialization now runs reliably at `session_start` and before LSP-backed `tool_call` operations, so server enable/disable overrides apply in one-shot and interactive sessions.
+
 ## [3.8.23] - 2026-04-12
 
 ### Added

package/clients/bootstrap.ts

@@ -0,0 +1,106 @@
+import type { AgentBehaviorClient } from "./agent-behavior-client.js";
+import type { ArchitectClient } from "./architect-client.js";
+import type { BiomeClient } from "./biome-client.js";
+import type { ComplexityClient } from "./complexity-client.js";
+import type { DependencyChecker } from "./dependency-checker.js";
+import type { GoClient } from "./go-client.js";
+import type { JscpdClient } from "./jscpd-client.js";
+import type { KnipClient } from "./knip-client.js";
+import type { MetricsClient } from "./metrics-client.js";
+import type { RuffClient } from "./ruff-client.js";
+import type { RustClient } from "./rust-client.js";
+import type { TestRunnerClient } from "./test-runner-client.js";
+import type { TodoScanner } from "./todo-scanner.js";
+import type { TypeCoverageClient } from "./type-coverage-client.js";
+
+export interface BootstrapClients {
+	ruffClient: RuffClient;
+	biomeClient: BiomeClient;
+	knipClient: KnipClient;
+	todoScanner: TodoScanner;
+	jscpdClient: JscpdClient;
+	typeCoverageClient: TypeCoverageClient;
+	depChecker: DependencyChecker;
+	testRunnerClient: TestRunnerClient;
+	metricsClient: MetricsClient;
+	complexityClient: ComplexityClient;
+	architectClient: ArchitectClient;
+	goClient: GoClient;
+	rustClient: RustClient;
+	agentBehaviorClient: AgentBehaviorClient;
+}
+
+let bootstrapPromise: Promise<BootstrapClients> | null = null;
+
+function createArchitectFallback(): ArchitectClient {
+	return {
+		loadConfig: () => false,
+		isUserDefined: () => false,
+		hasConfig: () => false,
+		getRulesForFile: () => [],
+		checkFile: () => [],
+		checkFileSize: () => null,
+		getHints: () => [],
+	} as unknown as ArchitectClient;
+}
+
+export function loadBootstrapClients(): Promise<BootstrapClients> {
+	bootstrapPromise ??= (async () => {
+		const [
+			ruffMod,
+			biomeMod,
+			knipMod,
+			todoMod,
+			jscpdMod,
+			typeCoverageMod,
+			depCheckerMod,
+			testRunnerMod,
+			metricsMod,
+			complexityMod,
+			goMod,
+			rustMod,
+			agentBehaviorMod,
+		] = await Promise.all([
+			import("./ruff-client.js"),
+			import("./biome-client.js"),
+			import("./knip-client.js"),
+			import("./todo-scanner.js"),
+			import("./jscpd-client.js"),
+			import("./type-coverage-client.js"),
+			import("./dependency-checker.js"),
+			import("./test-runner-client.js"),
+			import("./metrics-client.js"),
+			import("./complexity-client.js"),
+			import("./go-client.js"),
+			import("./rust-client.js"),
+			import("./agent-behavior-client.js"),
+		]);
+
+		let architectClient: ArchitectClient;
+		try {
+			const architectMod = await import("./architect-client.js");
+			architectClient = new architectMod.ArchitectClient();
+		} catch {
+			architectClient = createArchitectFallback();
+		}
+
+		return {
+			ruffClient: new ruffMod.RuffClient(),
+			biomeClient: new biomeMod.BiomeClient(),
+			knipClient: new knipMod.KnipClient(),
+			todoScanner: new todoMod.TodoScanner(),
+			jscpdClient: new jscpdMod.JscpdClient(),
+			typeCoverageClient: new typeCoverageMod.TypeCoverageClient(),
+			depChecker: new depCheckerMod.DependencyChecker(),
+			testRunnerClient: new testRunnerMod.TestRunnerClient(),
+			metricsClient: new metricsMod.MetricsClient(),
+			complexityClient: new complexityMod.ComplexityClient(),
+			architectClient,
+			goClient: new goMod.GoClient(),
+			rustClient: new rustMod.RustClient(),
+			agentBehaviorClient: new agentBehaviorMod.AgentBehaviorClient(),
+		};
+	})();
+
+	return bootstrapPromise;
+}

package/clients/lsp/config.ts

@@ -55,17 +55,25 @@ const CONFIG_PATHS = [".pi-lens/lsp.json", ".pi-lens.json", "pi-lsp.json"];
  * Load LSP configuration from file
  */
 export async function loadLSPConfig(cwd: string): Promise<LSPConfig> {
-	for (const configPath of CONFIG_PATHS) {
-		const fullPath = path.join(cwd, configPath);
-		try {
-			const content = await fs.readFile(fullPath, "utf-8");
-			const config = JSON.parse(content) as LSPConfig;
-			console.error(`[lsp-config] Loaded config from ${configPath}`);
-			return config;
-		} catch {
-			// File doesn't exist or is invalid, try next
+	let dir = path.resolve(cwd);
+	while (true) {
+		for (const configPath of CONFIG_PATHS) {
+			const fullPath = path.join(dir, configPath);
+			try {
+				const content = await fs.readFile(fullPath, "utf-8");
+				const config = JSON.parse(content) as LSPConfig;
+				console.error(`[lsp-config] Loaded config from ${fullPath}`);
+				return config;
+			} catch {
+				// File doesn't exist or is invalid, try next
+			}
 		}
+
+		const parent = path.dirname(dir);
+		if (parent === dir) break;
+		dir = parent;
 	}
+
 	return {};
 }
 

package/clients/lsp/server.ts

@@ -740,7 +740,7 @@ export const RubyServer: LSPServerInfo = {
 	extensions: [".rb", ".rake", ".gemspec", ".ru"],
 	root: PriorityRoot([["Gemfile", ".ruby-version"], [".git"]]),
 	async spawn(root, options) {
-		// Try ruby-lsp first (prompts to install via gem if missing), fall back to solargraph
+		// Try ruby-lsp first, then solargraph, then rubocop --lsp
 		const proc = await spawnWithInteractiveInstall(
 			"ruby",
 			"ruby-lsp",
@@ -763,6 +763,14 @@ export const RubyServer: LSPServerInfo = {
 					}
 				}
 
+				for (const command of ["rubocop", ...rubyBinCandidates("rubocop")]) {
+					try {
+						return await launchLSP(command, ["--lsp"], { cwd: root });
+					} catch {
+						// try next rubocop candidate
+					}
+				}
+
 				throw new Error("ENOENT: command not found");
 			},
 		);

package/index.ts

@@ -4,13 +4,9 @@ import * as path from "node:path";
 import { fileURLToPath } from "node:url";
 import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
 import { isToolCallEventType } from "@mariozechner/pi-coding-agent";
-import { AgentBehaviorClient } from "./clients/agent-behavior-client.js";
-import { ArchitectClient } from "./clients/architect-client.js";
 import { AstGrepClient } from "./clients/ast-grep-client.js";
-import { BiomeClient } from "./clients/biome-client.js";
 import { CacheManager } from "./clients/cache-manager.js";
-import { ComplexityClient } from "./clients/complexity-client.js";
-import { DependencyChecker } from "./clients/dependency-checker.js";
+import { loadBootstrapClients } from "./clients/bootstrap.js";
 import { getDiagnosticTracker } from "./clients/diagnostic-tracker.js";
 import {
 	getDispatchSlopScoreLine,
@@ -20,15 +16,11 @@ import {
 import { extractFunctions } from "./clients/dispatch/runners/similarity.js";
 import { resetFormatService } from "./clients/format-service.js";
 import { evaluateGitGuard, isGitCommitOrPushAttempt } from "./clients/git-guard.js";
-import { GoClient } from "./clients/go-client.js";
 import { ensureTool } from "./clients/installer/index.js";
-import { JscpdClient } from "./clients/jscpd-client.js";
-import { KnipClient } from "./clients/knip-client.js";
+import { initLSPConfig } from "./clients/lsp/config.js";
 import { getLSPService, resetLSPService } from "./clients/lsp/index.js";
-import { MetricsClient } from "./clients/metrics-client.js";
 import { captureSnapshot } from "./clients/metrics-history.js";
 import { findSimilarFunctions } from "./clients/project-index.js";
-import { RuffClient } from "./clients/ruff-client.js";
 import { RuntimeCoordinator } from "./clients/runtime-coordinator.js";
 import {
 	consumeSessionStartGuidance,
@@ -38,11 +30,6 @@ import { handleSessionStart } from "./clients/runtime-session.js";
 import { handleToolResult } from "./clients/runtime-tool-result.js";
 import { handleTurnEnd } from "./clients/runtime-turn.js";
 import { formatRulesForPrompt } from "./clients/rules-scanner.js";
-import { RustClient } from "./clients/rust-client.js";
-import { TestRunnerClient } from "./clients/test-runner-client.js";
-import { TodoScanner } from "./clients/todo-scanner.js";
-import { TypeCoverageClient } from "./clients/type-coverage-client.js";
-import { TypeScriptClient } from "./clients/typescript-client.js";
 import { handleBooboo } from "./commands/booboo.js";
 import { createAstGrepReplaceTool } from "./tools/ast-grep-replace.js";
 import { createAstGrepSearchTool } from "./tools/ast-grep-search.js";
@@ -72,11 +59,19 @@ function dbg(msg: string) {
 
 let _verbose = false;
 const runtime = new RuntimeCoordinator();
+const _lspConfigInitializedCwds = new Set<string>();
 
 function log(msg: string) {
 	if (_verbose) console.error(`[pi-lens] ${msg}`);
 }
 
+async function ensureLSPConfigInitialized(cwd: string): Promise<void> {
+	const normalizedCwd = path.resolve(cwd);
+	if (_lspConfigInitializedCwds.has(normalizedCwd)) return;
+	await initLSPConfig(normalizedCwd);
+	_lspConfigInitializedCwds.add(normalizedCwd);
+}
+
 function updateRuntimeIdentityFromEvent(event: unknown): void {
 	const raw = event as {
 		provider?: string;
@@ -136,24 +131,10 @@ function cleanStaleTsBuildInfo(cwd: string): string[] {
 // --- Extension ---
 
 export default function (pi: ExtensionAPI) {
-	const tsClient = new TypeScriptClient();
 	const astGrepClient = new AstGrepClient();
-	const ruffClient = new RuffClient();
-	const biomeClient = new BiomeClient();
-	const knipClient = new KnipClient();
-	const todoScanner = new TodoScanner();
-	const jscpdClient = new JscpdClient();
-	const typeCoverageClient = new TypeCoverageClient();
-	const depChecker = new DependencyChecker();
-	const testRunnerClient = new TestRunnerClient();
-	const metricsClient = new MetricsClient();
-	const complexityClient = new ComplexityClient();
-	const architectClient = new ArchitectClient();
-	const goClient = new GoClient();
-	const rustClient = new RustClient();
-	const agentBehaviorClient = new AgentBehaviorClient();
 	const cacheManager = new CacheManager();
 
+
 	// --- Flags ---
 
 	pi.registerFlag("lens-verbose", {
@@ -299,8 +280,9 @@ export default function (pi: ExtensionAPI) {
 	pi.registerCommand("lens-booboo", {
 		description:
 			"Full codebase review: design smells, complexity, AI slop detection, TODOs, dead code, duplicates, type coverage. Results saved to .pi-lens/reviews/. Usage: /lens-booboo [path]",
-		handler: (args, ctx) =>
-			handleBooboo(
+		handler: async (args, ctx) => {
+			const { complexityClient, todoScanner, knipClient, jscpdClient, typeCoverageClient, depChecker, architectClient } = await loadBootstrapClients();
+			return handleBooboo(
 				args,
 				ctx,
 				{
@@ -310,11 +292,12 @@ export default function (pi: ExtensionAPI) {
 					knip: knipClient,
 					jscpd: jscpdClient,
 					typeCoverage: typeCoverageClient,
-					depChecker: depChecker,
+					depChecker,
 					architect: architectClient,
 				},
 				pi,
-			),
+			);
+		},
 	});
 
 	// DISABLED: lens-booboo-fix command - disabled per user request
@@ -482,7 +465,26 @@ pi.on("session_start", async (event, ctx) => {
 		_verbose = !!pi.getFlag("lens-verbose");
 		dbg("session_start fired");
 		updateRuntimeIdentityFromEvent(event);
+		try {
+			await ensureLSPConfigInitialized(ctx.cwd ?? process.cwd());
+		} catch (cfgErr) {
+			dbg(`lsp config init failed: ${cfgErr}`);
+		}
 
+		const {
+			metricsClient,
+			todoScanner,
+			biomeClient,
+			ruffClient,
+			knipClient,
+			jscpdClient,
+			typeCoverageClient,
+			depChecker,
+			architectClient,
+			testRunnerClient,
+			goClient,
+			rustClient,
+		} = await loadBootstrapClients();
 		await handleSessionStart({
 			ctxCwd: ctx.cwd,
 			getFlag: (name: string) => pi.getFlag(name),
@@ -546,6 +548,17 @@ pi.on("tool_call", async (event, ctx) => {
 			: path.resolve(ctx.cwd ?? runtime.projectRoot, rawFilePath)
 		: undefined;
 
+	if (pi.getFlag("lens-lsp") && !pi.getFlag("no-lsp")) {
+		try {
+			const configCwd = filePath
+				? path.dirname(filePath)
+				: (ctx.cwd ?? runtime.projectRoot ?? process.cwd());
+			await ensureLSPConfigInitialized(configCwd);
+		} catch (cfgErr) {
+			dbg(`lsp config init failed during tool_call: ${cfgErr}`);
+		}
+	}
+
 	if (!filePath) return;
 
 	dbg(
@@ -571,6 +584,7 @@ pi.on("tool_call", async (event, ctx) => {
 		}
 	}
 
+	const { complexityClient } = await loadBootstrapClients();
 	// Record complexity baseline for historical tracking (booboo/tdi).
 	// Not shown inline — just captured for delta analysis.
 	if (
@@ -711,6 +725,13 @@ pi.on("tool_call", async (event, ctx) => {
 // biome-ignore lint/suspicious/noExplicitAny: pi.on overload mismatch for tool_result event type
 (pi as any).on("tool_result", async (event: any) => {
 	updateRuntimeIdentityFromEvent(event);
+	const {
+		biomeClient,
+		ruffClient,
+		testRunnerClient,
+		metricsClient,
+		agentBehaviorClient,
+	} = await loadBootstrapClients();
 	return handleToolResult({
 		event: event as any,
 		getFlag: (name: string) => pi.getFlag(name),
@@ -747,6 +768,7 @@ pi.on("turn_start", () => {
 
 pi.on("turn_end", async (_event, ctx) => {
 	try {
+		const { jscpdClient, knipClient, depChecker } = await loadBootstrapClients();
 		await handleTurnEnd({
 			ctxCwd: ctx.cwd,
 			getFlag: (name: string) => pi.getFlag(name),

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lens",
-	"version": "3.8.23",
+	"version": "3.8.24",
 	"type": "module",
 	"description": "Real-time code feedback for pi — LSP, linters, formatters, type-checking, structural analysis & booboo",
 	"repository": {
@@ -75,6 +75,7 @@
 	"dependencies": {
 		"@sinclair/typebox": "^0.34.0",
 		"cross-spawn": "^7.0.6",
+		"minimatch": "^10.2.5",
 		"typescript": "^5.0.0",
 		"vscode-jsonrpc": "^8.2.1"
 	},