pi-lens 3.8.30 → 3.8.31

package/CHANGELOG.md

@@ -4,6 +4,17 @@ All notable changes to pi-lens will be documented in this file.
 
 ## [Unreleased]
 
+## [3.8.31] - 2026-04-23
+
+### Fixed
+- **Duplicate inline feedback on edit arrays** — `tool_result` calls for the same file are now deduplicated within a turn using a `reportedThisTurn` set on `RuntimeCoordinator`, cleared on each `turn_start`; previously pi's sequential per-hunk `tool_result` firing caused the pipeline to re-run and feedback to repeat N times per edit array
+- **Double latency logging on pipeline completion** — removed redundant `logLatency` call in `pipeline.ts`; `runtime-tool-result.ts` already logs the outer `tool_result completed` with full duration including format, autofix, and cascade phases
+- **Modified range tracking broken for 3-digit+ line numbers** — `parseDiffRanges` regex changed from `\s+` to `\s*` to handle unpadded line numbers; the diff format right-pads to the file's max digit width so e.g. line 613 in a <1000-line file has no leading space and was silently dropped
+- **Stale gleam grammar entries** — removed dead `LANGUAGE_TO_GRAMMAR` and `getExtensionsForLanguage` entries for gleam; `tree-sitter-gleam.wasm` was never published in `tree-sitter-wasms@0.1.13`
+
+### Changed
+- **TypeBox 0.34.x → 1.x migration** — updated `package.json` dependency from `@sinclair/typebox` to `typebox ^1.0.0` and updated imports in `tools/lsp-navigation.ts`, `tools/ast-grep-search.ts`, and `tools/ast-grep-replace.ts` to match pi-mono 0.69.0
+
 ## [3.8.30] - 2026-04-22
 
 ### Fixed

package/clients/pipeline.ts

@@ -1014,13 +1014,6 @@ export async function runPipeline(
 	}
 
 	phase.end("total", { hasOutput: !!output });
-	logLatency({
-		type: "tool_result",
-		toolName,
-		filePath,
-		durationMs: elapsed,
-		result: output ? "completed" : "no_output",
-	});
 
 	return {
 		output,

package/clients/runtime-coordinator.ts

@@ -22,6 +22,7 @@ export class RuntimeCoordinator {
 	private _lastImpactCascadeOutput = "";
 	private _complexityBaselines = new Map<string, FileComplexity>();
 	private _fixedThisTurn = new Set<string>();
+	private _reportedThisTurn = new Set<string>();
 	private _projectRulesScan: RuleScanResult = {
 		rules: [],
 		hasCustomRules: false,
@@ -43,6 +44,7 @@ export class RuntimeCoordinator {
 		this._lastCascadeOutput = "";
 		this._lastImpactCascadeOutput = "";
 		this._fixedThisTurn.clear();
+		this._reportedThisTurn.clear();
 		this._telemetrySessionId =
 			`lens-${Date.now().toString(36)}-${Math.random().toString(36).slice(2, 8)}`;
 		this._telemetryModel = "unknown";
@@ -81,6 +83,11 @@ export class RuntimeCoordinator {
 		this._lastImpactCascadeOutput = "";
 		this._turnIndex += 1;
 		this._writeIndex = 0;
+		this._reportedThisTurn.clear();
+	}
+
+	get reportedThisTurn(): Set<string> {
+		return this._reportedThisTurn;
 	}
 
 	nextWriteIndex(): number {

package/clients/runtime-tool-result.ts

@@ -41,7 +41,7 @@ interface ToolResultDeps {
 function parseDiffRanges(diff: string): { start: number; end: number }[] {
 	const changedLines: number[] = [];
 	for (const line of diff.split("\n")) {
-		const match = line.match(/^[+-]\s+(\d+)\s/);
+		const match = line.match(/^[+-]\s*(\d+)\s/);
 		if (match) {
 			changedLines.push(Number.parseInt(match[1], 10));
 		}
@@ -121,6 +121,14 @@ export async function handleToolResult(
 		return;
 	}
 
+	// Deduplicate sequential calls for the same file within the same turn.
+	// When pi processes an edit array serially, each hunk fires tool_result after
+	// the previous completes, bypassing the in-flight map above.
+	if (runtime.reportedThisTurn.has(filePath)) {
+		dbg(`tool_result: skipping already-reported file this turn for ${filePath}`);
+		return;
+	}
+
 	const sessionFileTime = createFileTime("default");
 	// tool_result is emitted after write/edit has already been applied.
 	// Asserting pre-write stamps here produces false positives on rapid edits.
@@ -285,6 +293,8 @@ export async function handleToolResult(
 		result: output ? "completed" : "no_output",
 	});
 
+	runtime.reportedThisTurn.add(filePath);
+
 	if (!output) return;
 
 	return {

package/clients/tree-sitter-client.ts

@@ -68,7 +68,6 @@ const LANGUAGE_TO_GRAMMAR: Record<string, string> = {
 	c: "tree-sitter-c.wasm",
 	cpp: "tree-sitter-cpp.wasm",
 	elixir: "tree-sitter-elixir.wasm",
-	gleam: "tree-sitter-gleam.wasm",
 	ruby: "tree-sitter-ruby.wasm",
 };
 
@@ -1112,7 +1111,6 @@ export class TreeSitterClient {
 			c: [".c", ".h"],
 			cpp: [".cpp", ".hpp", ".cc", ".hh"],
 			elixir: [".ex", ".exs"],
-			gleam: [".gleam"],
 			ruby: [".rb"],
 		};
 		return mapping[languageId] || [];

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lens",
-	"version": "3.8.30",
+	"version": "3.8.31",
 	"type": "module",
 	"description": "Real-time code feedback for pi — LSP, linters, formatters, type-checking, structural analysis & booboo",
 	"repository": {
@@ -74,7 +74,7 @@
 	},
 	"dependencies": {
 		"@ast-grep/napi": "^0.42.1",
-		"@sinclair/typebox": "^0.34.0",
+		"typebox": "^1.0.0",
 		"minimatch": "^10.2.5",
 		"typescript": "^5.0.0",
 		"vscode-jsonrpc": "^8.2.1"

package/tools/ast-grep-replace.js

@@ -3,7 +3,7 @@
  *
  * Extracted from index.ts for maintainability.
  */
-import { Type } from "@sinclair/typebox";
+import { Type } from "typebox";
 import { LANGUAGES } from "./shared.js";
 export function createAstGrepReplaceTool(astGrepClient) {
     return {

package/tools/ast-grep-replace.ts

@@ -4,7 +4,7 @@
  * Extracted from index.ts for maintainability.
  */
 
-import { Type } from "@sinclair/typebox";
+import { Type } from "typebox";
 import type { AstGrepClient } from "../clients/ast-grep-client.js";
 import { LANGUAGES } from "./shared.js";
 

package/tools/ast-grep-search.js

@@ -3,7 +3,7 @@
  *
  * Extracted from index.ts for maintainability.
  */
-import { Type } from "@sinclair/typebox";
+import { Type } from "typebox";
 import { LANGUAGES } from "./shared.js";
 function looksLikeRuleYamlOrPlainText(pattern) {
     const text = pattern.trim();

package/tools/ast-grep-search.ts

@@ -4,7 +4,7 @@
  * Extracted from index.ts for maintainability.
  */
 
-import { Type } from "@sinclair/typebox";
+import { Type } from "typebox";
 import type { AstGrepClient } from "../clients/ast-grep-client.js";
 import { LANGUAGES } from "./shared.js";
 

package/tools/lsp-navigation.js

@@ -6,7 +6,7 @@
 import * as nodeFs from "node:fs";
 import * as path from "node:path";
 import { pathToFileURL } from "node:url";
-import { Type } from "@sinclair/typebox";
+import { Type } from "typebox";
 import { getLSPService } from "../clients/lsp/index.js";
 import { logLatency } from "../clients/latency-logger.js";
 function operationSupportStatus(operation, support) {

package/tools/lsp-navigation.ts

@@ -7,7 +7,7 @@
 import * as nodeFs from "node:fs";
 import * as path from "node:path";
 import { pathToFileURL } from "node:url";
-import { Type } from "@sinclair/typebox";
+import { Type } from "typebox";
 import type { LSPCallHierarchyItem } from "../clients/lsp/client.js";
 import { getLSPService } from "../clients/lsp/index.js";
 import { logLatency } from "../clients/latency-logger.js";