pi-lens 3.8.18 → 3.8.19

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 All notable changes to pi-lens will be documented in this file.
 
+## [3.8.19] - 2026-04-07
+
+### Fixed
+- **Biome autofix gating** — Biome autofix/auto-install now runs only when the project has Biome configuration (`biome.json`/`biome.jsonc`) or `@biomejs/biome` in `devDependencies`, preventing unwanted Biome installs in non-Biome JS/TS projects.
+
 ## [3.8.18] - 2026-04-07
 
 ### Changed

package/README.md

@@ -82,19 +82,25 @@ Some runners are language/config-gated and may skip when not applicable.
 
 ## Dependencies
 
-Auto-installed defaults:
-
-| Tool | Purpose | Auto-installed |
-|---|---|---|
-| `typescript-language-server` | LSP type diagnostics | Yes |
-| `pyright` | Python type diagnostics fallback | Yes |
-| `prettier` | Formatting fallback | Yes |
-| `ruff` | Python lint/format/autofix | Yes |
-| `@biomejs/biome` | JS/TS lint/format/autofix | Yes |
-| `madge` | Circular dependency analysis | Yes |
-| `jscpd` | Duplicate code detection | Yes |
-| `@ast-grep/cli` (`sg`) | AST search/replace and scans | Yes |
-| `knip` | Dead code analysis | Yes |
+Auto-install behavior depends on gate type:
+
+- **Config-gated**: installs only when project config/deps indicate usage.
+- **Flow/language-gated**: installs when the runtime path needs it for the current file/session flow.
+- **Operational prewarm**: installs during session warm scans / turn-end analysis paths.
+
+| Tool | Purpose | Auto-installed | Gate |
+|---|---|---|---|
+| `@biomejs/biome` | JS/TS lint/format/autofix | Yes | Config-gated (`biome.json`/`biome.jsonc` or `@biomejs/biome` dep) |
+| `prettier` | Formatting fallback | Yes | Config-gated (Prettier dep or `package.json#prettier`) |
+| `yamllint` | YAML linting | Yes | Config-gated (`.yamllint*` / tool section / dep hint) |
+| `sqlfluff` | SQL linting/formatting | Yes | Config-gated (`.sqlfluff` / tool section / dep hint) |
+| `ruff` | Python lint/format/autofix | Yes | Flow/language-gated (Python file paths/runners) |
+| `typescript-language-server` | Unified LSP diagnostics | Yes | Flow-gated (LSP enabled; default on unless `--no-lsp`) |
+| `pyright` | Python type diagnostics fallback | Yes | Flow/language-gated (Python fallback paths) |
+| `@ast-grep/cli` (`sg`) | AST scans/search/replace | Yes | Operational prewarm + analysis flows |
+| `knip` | Dead code analysis | Yes | Operational prewarm + turn-end flows |
+| `jscpd` | Duplicate code detection | Yes | Operational prewarm + turn-end flows |
+| `madge` | Circular dependency analysis | Yes | Turn-end analysis flow |
 
 LSP is enabled by default. pi-lens includes many language-server definitions (including up to 31+ servers), and activates them when the server is installed and the project/root detection matches the file.
 
@@ -105,5 +111,5 @@ Optional safety switch:
 
 ## Notes
 
-- Some tools are auto-installed; others are config/availability-based.
+- Not every auto-install runs in every project: gate type decides when install is attempted.
 - Rule packs are customizable via project-level rule directories.

package/clients/dispatch/runners/sqlfluff.ts

@@ -1,3 +1,5 @@
+import * as nodeFs from "node:fs";
+import * as path from "node:path";
 import { ensureTool } from "../../installer/index.js";
 import { safeSpawn } from "../../safe-spawn.js";
 import { createAvailabilityChecker } from "./utils/runner-helpers.js";
@@ -10,6 +12,41 @@ import type {
 
 const sqlfluff = createAvailabilityChecker("sqlfluff", ".exe");
 
+const SQLFLUFF_CONFIGS = [".sqlfluff", "pyproject.toml", "setup.cfg", "tox.ini"];
+
+export function hasSqlfluffConfig(cwd: string): boolean {
+	for (const cfg of SQLFLUFF_CONFIGS) {
+		const cfgPath = path.join(cwd, cfg);
+		if (!nodeFs.existsSync(cfgPath)) continue;
+		if (cfg === "pyproject.toml") {
+			try {
+				const content = nodeFs.readFileSync(cfgPath, "utf-8");
+				if (content.includes("[tool.sqlfluff]")) return true;
+			} catch {}
+			continue;
+		}
+		if (cfg === "setup.cfg" || cfg === "tox.ini") {
+			try {
+				const content = nodeFs.readFileSync(cfgPath, "utf-8");
+				if (content.includes("[sqlfluff]")) return true;
+			} catch {}
+			continue;
+		}
+		return true;
+	}
+
+	for (const depFile of ["requirements.txt", "Pipfile", "pyproject.toml"]) {
+		const depPath = path.join(cwd, depFile);
+		if (!nodeFs.existsSync(depPath)) continue;
+		try {
+			const content = nodeFs.readFileSync(depPath, "utf-8").toLowerCase();
+			if (content.includes("sqlfluff")) return true;
+		} catch {}
+	}
+
+	return false;
+}
+
 type SqlfluffJson = Array<{
 	filepath?: string;
 	violations?: Array<{
@@ -59,6 +96,10 @@ const sqlfluffRunner: RunnerDefinition = {
 
 	async run(ctx: DispatchContext): Promise<RunnerResult> {
 		const cwd = ctx.cwd || process.cwd();
+		if (!hasSqlfluffConfig(cwd)) {
+			return { status: "skipped", diagnostics: [], semantic: "none" };
+		}
+
 		let cmd: string | null = null;
 		if (sqlfluff.isAvailable(cwd)) {
 			cmd = sqlfluff.getCommand(cwd);

package/clients/dispatch/runners/yamllint.ts

@@ -1,3 +1,5 @@
+import * as nodeFs from "node:fs";
+import * as path from "node:path";
 import { ensureTool } from "../../installer/index.js";
 import { safeSpawn } from "../../safe-spawn.js";
 import { createAvailabilityChecker } from "./utils/runner-helpers.js";
@@ -10,6 +12,49 @@ import type {
 
 const yamllint = createAvailabilityChecker("yamllint", ".exe");
 
+const YAMLLINT_CONFIGS = [
+	".yamllint",
+	".yamllint.yml",
+	".yamllint.yaml",
+	"pyproject.toml",
+	"setup.cfg",
+	"tox.ini",
+];
+
+export function hasYamllintConfig(cwd: string): boolean {
+	for (const cfg of YAMLLINT_CONFIGS) {
+		const cfgPath = path.join(cwd, cfg);
+		if (!nodeFs.existsSync(cfgPath)) continue;
+		if (cfg === "pyproject.toml") {
+			try {
+				const content = nodeFs.readFileSync(cfgPath, "utf-8");
+				if (content.includes("[tool.yamllint]")) return true;
+			} catch {}
+			continue;
+		}
+		if (cfg === "setup.cfg" || cfg === "tox.ini") {
+			try {
+				const content = nodeFs.readFileSync(cfgPath, "utf-8");
+				if (content.includes("[yamllint]")) return true;
+			} catch {}
+			continue;
+		}
+		return true;
+	}
+
+	// Dependency hint fallback for Python projects.
+	for (const depFile of ["requirements.txt", "Pipfile", "pyproject.toml"]) {
+		const depPath = path.join(cwd, depFile);
+		if (!nodeFs.existsSync(depPath)) continue;
+		try {
+			const content = nodeFs.readFileSync(depPath, "utf-8").toLowerCase();
+			if (content.includes("yamllint")) return true;
+		} catch {}
+	}
+
+	return false;
+}
+
 function parseYamllintParsable(raw: string, filePath: string): Diagnostic[] {
 	const diagnostics: Diagnostic[] = [];
 	for (const line of raw.split(/\r?\n/)) {
@@ -44,6 +89,10 @@ const yamllintRunner: RunnerDefinition = {
 
 	async run(ctx: DispatchContext): Promise<RunnerResult> {
 		const cwd = ctx.cwd || process.cwd();
+		if (!hasYamllintConfig(cwd)) {
+			return { status: "skipped", diagnostics: [], semantic: "none" };
+		}
+
 		let cmd: string | null = null;
 		if (yamllint.isAvailable(cwd)) {
 			cmd = yamllint.getCommand(cwd);

package/clients/pipeline.ts

@@ -18,24 +18,30 @@ import type { BiomeClient } from "./biome-client.js";
 import { getDiagnosticLogger } from "./diagnostic-logger.js";
 import { getDiagnosticTracker } from "./diagnostic-tracker.js";
 import { dispatchLintWithResult } from "./dispatch/integration.js";
+import {
+	resolveRunnerPath,
+	toRunnerDisplayPath,
+} from "./dispatch/runner-context.js";
 import type { PiAgentAPI } from "./dispatch/types.js";
 import { detectFileKind, getFileKindLabel } from "./file-kinds.js";
 import type { FormatService } from "./format-service.js";
 import { logLatency } from "./latency-logger.js";
 import { getLSPService } from "./lsp/index.js";
 import type { MetricsClient } from "./metrics-client.js";
+import { normalizeMapKey } from "./path-utils.js";
 import type { RuffClient } from "./ruff-client.js";
 import { RUNTIME_CONFIG } from "./runtime-config.js";
 import { safeSpawnAsync } from "./safe-spawn.js";
 import { formatSecrets, scanForSecrets } from "./secrets-scanner.js";
 import type { TestRunnerClient } from "./test-runner-client.js";
-import { normalizeMapKey } from "./path-utils.js";
-import { resolveRunnerPath, toRunnerDisplayPath } from "./dispatch/runner-context.js";
 
 const LSP_MAX_FILE_BYTES = RUNTIME_CONFIG.pipeline.lspMaxFileBytes;
 const LSP_MAX_FILE_LINES = RUNTIME_CONFIG.pipeline.lspMaxFileLines;
 
-function exceedsLspSyncLimits(filePath: string, content: string): {
+function exceedsLspSyncLimits(
+	filePath: string,
+	content: string,
+): {
 	tooLarge: boolean;
 	reason: string;
 } {
@@ -140,6 +146,8 @@ function createPhaseTracker(toolName: string, filePath: string): PhaseTracker {
 
 // --- ESLint autofix helpers ---
 
+const BIOME_CONFIGS = ["biome.json", "biome.jsonc"];
+
 const ESLINT_CONFIGS = [
 	".eslintrc",
 	".eslintrc.js",
@@ -158,6 +166,19 @@ function isJsTs(filePath: string): boolean {
 	return JSTS_EXTS.has(path.extname(filePath).toLowerCase());
 }
 
+function hasBiomeConfig(cwd: string): boolean {
+	for (const cfg of BIOME_CONFIGS) {
+		if (nodeFs.existsSync(path.join(cwd, cfg))) return true;
+	}
+	try {
+		const pkg = JSON.parse(
+			nodeFs.readFileSync(path.join(cwd, "package.json"), "utf-8"),
+		);
+		if (pkg.devDependencies?.["@biomejs/biome"]) return true;
+	} catch {}
+	return false;
+}
+
 function hasEslintConfig(cwd: string): boolean {
 	for (const cfg of ESLINT_CONFIGS) {
 		if (nodeFs.existsSync(path.join(cwd, cfg))) return true;
@@ -171,7 +192,10 @@ function hasEslintConfig(cwd: string): boolean {
 	return false;
 }
 
-const _eslintCache = new Map<string, { available: boolean; bin: string | null }>();
+const _eslintCache = new Map<
+	string,
+	{ available: boolean; bin: string | null }
+>();
 
 function findEslintBin(cwd: string): string {
 	const isWin = process.platform === "win32";
@@ -331,7 +355,7 @@ export async function runPipeline(
 		const deferLspSync =
 			!getFlag("no-autofix") &&
 			(ruffClient.isPythonFile(filePath) ||
-				biomeClient.isSupportedFile(filePath) ||
+				(biomeClient.isSupportedFile(filePath) && hasBiomeConfig(cwd)) ||
 				isJsTs(filePath));
 
 		if (deferLspSync) {
@@ -386,7 +410,9 @@ export async function runPipeline(
 			!noAutofixRuff && ruffClient.isPythonFile(filePath)
 				? ruffClient.ensureAvailable()
 				: Promise.resolve(false),
-			!noAutofixBiome && biomeClient.isSupportedFile(filePath)
+			!noAutofixBiome &&
+			biomeClient.isSupportedFile(filePath) &&
+			hasBiomeConfig(cwd)
 				? biomeClient.ensureAvailable()
 				: Promise.resolve(false),
 		]);
@@ -494,12 +520,16 @@ export async function runPipeline(
 		);
 		for (const d of dispatchResult.diagnostics) {
 			const shownInline = inlineKeys.has(toKey(d));
-			logger.logCaught(d, {
-				model: ctx.telemetry?.model ?? "unknown",
-				sessionId: ctx.telemetry?.sessionId ?? "unknown",
-				turnIndex: ctx.telemetry?.turnIndex ?? 0,
-				writeIndex: ctx.telemetry?.writeIndex ?? 0,
-			}, shownInline);
+			logger.logCaught(
+				d,
+				{
+					model: ctx.telemetry?.model ?? "unknown",
+					sessionId: ctx.telemetry?.sessionId ?? "unknown",
+					turnIndex: ctx.telemetry?.turnIndex ?? 0,
+					writeIndex: ctx.telemetry?.writeIndex ?? 0,
+				},
+				shownInline,
+			);
 		}
 	}
 
@@ -552,34 +582,34 @@ export async function runPipeline(
 				target.runner,
 				target.config,
 			);
-				const testDuration = Date.now() - testStart;
-				logLatency({
-					type: "phase",
-					toolName,
-					filePath,
-					phase: "test_runner",
-					durationMs: testDuration,
-					metadata: {
-						testFile: target.testFile,
-						runner: target.runner,
-						strategy: target.strategy,
-						success: !testResult?.error,
-					},
-				});
-				if (testResult && !testResult.error) {
-					testSummary = {
-						passed: testResult.passed,
-						total: testResult.passed + testResult.failed + testResult.skipped,
-						failed: testResult.failed,
-					};
-					if (testSummary.failed > 0) {
-						hasBlockers = true;
-					}
-					const testOutput = testRunnerClient.formatResult(testResult);
-					if (testOutput) {
-						output += `\n\n${testOutput}`;
-					}
+			const testDuration = Date.now() - testStart;
+			logLatency({
+				type: "phase",
+				toolName,
+				filePath,
+				phase: "test_runner",
+				durationMs: testDuration,
+				metadata: {
+					testFile: target.testFile,
+					runner: target.runner,
+					strategy: target.strategy,
+					success: !testResult?.error,
+				},
+			});
+			if (testResult && !testResult.error) {
+				testSummary = {
+					passed: testResult.passed,
+					total: testResult.passed + testResult.failed + testResult.skipped,
+					failed: testResult.failed,
+				};
+				if (testSummary.failed > 0) {
+					hasBlockers = true;
+				}
+				const testOutput = testRunnerClient.formatResult(testResult);
+				if (testOutput) {
+					output += `\n\n${testOutput}`;
 				}
+			}
 		}
 	}
 	phase.end("test_runner", { found: testInfoFound, ran: testRunnerRan });
@@ -608,7 +638,8 @@ export async function runPipeline(
 
 			for (const [diagPath, diags] of allDiags) {
 				const normalizedDiagPath = resolveRunnerPath(cwd, diagPath);
-				if (normalizeMapKey(normalizedDiagPath) === normalizedEditedPath) continue;
+				if (normalizeMapKey(normalizedDiagPath) === normalizedEditedPath)
+					continue;
 
 				if (!nodeFs.existsSync(normalizedDiagPath)) {
 					stalePathsSkipped++;

package/clients/type-coverage-client.ts

@@ -105,7 +105,7 @@ export class TypeCoverageClient {
 		if (result.percentage >= 95) icon = "✓";
 		else if (result.percentage >= 80) icon = "⚠";
 
-		let output = `[type-coverage] ${icon} ${pct}% typed (${result.typed}/${result.total} identifiers)`;
+		let output = `[type-coverage] ${icon} ${pct}% typed (${result.typed}/${result.total} identifiers; any-typed flagged)`;
 
 		if (result.untypedLocations.length === 0) {
 			output += " — fully typed\n";

package/commands/booboo.ts

@@ -910,6 +910,8 @@ export async function handleBooboo(
 			});
 
 			let fullSection = `## Type Coverage\n\n**${tcResult.percentage.toFixed(1)}% typed** (${tcResult.typed}/${tcResult.total} identifiers)\n\n`;
+			fullSection +=
+				"Type coverage highlights identifiers that resolve to `any` (implicit or explicit). Inferred non-`any` types are treated as typed.\n\n";
 			const byFile: Record<string, number> = {};
 			for (const u of filteredLocations) {
 				byFile[u.file] = (byFile[u.file] || 0) + 1;
@@ -920,7 +922,7 @@ export async function handleBooboo(
 				.slice(0, 10);
 
 			if (sortedFiles.length > 0) {
-				fullSection += `### Top Files by Untyped Count\n\n| File | Untyped Count |\n|------|---------------|\n`;
+				fullSection += `### Top Files by Any-Typed Identifier Count\n\n| File | Any-Typed Count |\n|------|-----------------|\n`;
 				for (const [file, count] of sortedFiles) {
 					fullSection += `| ${file} | ${count} |\n`;
 				}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lens",
-	"version": "3.8.18",
+	"version": "3.8.19",
 	"type": "module",
 	"description": "Real-time code feedback for pi — LSP, linters, formatters, type-checking, structural analysis & booboo",
 	"repository": {