pi-lens 3.8.18 → 3.8.21

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/runtime-context.ts

@@ -25,3 +25,29 @@ export function consumeTurnEndFindings(
 		],
 	};
 }
+
+export function consumeSessionStartGuidance(
+	cacheManager: CacheManager,
+	cwd: string,
+): { messages: Array<{ role: "system"; content: string }> } | undefined {
+	const guidance = cacheManager.readCache<{ content: string }>(
+		"session-start-guidance",
+		cwd,
+	);
+	if (!guidance?.data?.content) return;
+
+	cacheManager.writeCache(
+		"session-start-guidance",
+		null as unknown as { content: string },
+		cwd,
+	);
+
+	return {
+		messages: [
+			{
+				role: "system",
+				content: `[pi-lens] Session guidance:\n\n${guidance.data.content}`,
+			},
+		],
+	};
+}

package/clients/runtime-coordinator.ts

@@ -3,6 +3,7 @@ import type { FileComplexity } from "./complexity-client.js";
 import type { RuleScanResult } from "./rules-scanner.js";
 import { RUNTIME_CONFIG } from "./runtime-config.js";
 import type { ProjectIndex } from "./project-index.js";
+import { normalizeMapKey } from "./path-utils.js";
 
 export interface ErrorDebtBaseline {
 	testsPassed: boolean;
@@ -10,11 +11,13 @@ export interface ErrorDebtBaseline {
 }
 
 export class RuntimeCoordinator {
-	private _projectRoot = process.cwd();
+	private _projectRoot = normalizeMapKey(process.cwd());
+	private _sessionGeneration = 0;
 	private _errorDebtBaseline: ErrorDebtBaseline | null = null;
 	private _pipelineCrashCounts = new Map<string, number>();
 	private _cachedExports = new Map<string, string>();
 	private _cachedProjectIndex: ProjectIndex | null = null;
+	private _startupScansInFlight = new Map<string, number>();
 	private _lastCascadeOutput = "";
 	private _complexityBaselines = new Map<string, FileComplexity>();
 	private _fixedThisTurn = new Set<string>();
@@ -30,10 +33,12 @@ export class RuntimeCoordinator {
 	private _gitGuardSummary = "";
 
 	resetForSession(): void {
+		this._sessionGeneration += 1;
 		this._complexityBaselines.clear();
 		this._pipelineCrashCounts.clear();
 		this._cachedExports.clear();
 		this._cachedProjectIndex = null;
+		this._startupScansInFlight.clear();
 		this._lastCascadeOutput = "";
 		this._fixedThisTurn.clear();
 		this._telemetrySessionId =
@@ -111,6 +116,29 @@ export class RuntimeCoordinator {
 		return this._turnIndex;
 	}
 
+	get sessionGeneration(): number {
+		return this._sessionGeneration;
+	}
+
+	isCurrentSession(generation: number): boolean {
+		return this._sessionGeneration === generation;
+	}
+
+	markStartupScanInFlight(name: string, generation: number): void {
+		this._startupScansInFlight.set(name, generation);
+	}
+
+	clearStartupScanInFlight(name: string, generation: number): void {
+		const owner = this._startupScansInFlight.get(name);
+		if (owner === generation) {
+			this._startupScansInFlight.delete(name);
+		}
+	}
+
+	isStartupScanInFlight(name: string): boolean {
+		return this._startupScansInFlight.has(name);
+	}
+
 	formatPipelineCrashNotice(filePath: string, err: unknown): string {
 		const key = path.resolve(filePath);
 		const count = (this._pipelineCrashCounts.get(key) ?? 0) + 1;
@@ -140,7 +168,7 @@ export class RuntimeCoordinator {
 	}
 
 	set projectRoot(value: string) {
-		this._projectRoot = value;
+		this._projectRoot = normalizeMapKey(value);
 	}
 
 	get errorDebtBaseline(): ErrorDebtBaseline | null {