@oh-my-pi/pi-coding-agent 13.7.5 → 13.7.6

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## [Unreleased]
 
+## [13.7.6] - 2026-03-04
+### Added
+
+- Exported `dedupeParseErrors` utility function to deduplicate parse error messages while preserving order
+
+### Fixed
+
+- Reduced duplicate parse error messages when multiple patterns fail on the same file
+- Normalized parse error output in ast-grep to remove pattern-specific prefixes and show only file-level errors
+
 ## [13.7.4] - 2026-03-04
 ### Added
 - Added `fetch.useKagiSummarizer` setting to toggle Kagi Universal Summarizer usage in the fetch tool.

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "13.7.5",
+	"version": "13.7.6",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -41,12 +41,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "^0.6",
-		"@oh-my-pi/omp-stats": "13.7.5",
-		"@oh-my-pi/pi-agent-core": "13.7.5",
-		"@oh-my-pi/pi-ai": "13.7.5",
-		"@oh-my-pi/pi-natives": "13.7.5",
-		"@oh-my-pi/pi-tui": "13.7.5",
-		"@oh-my-pi/pi-utils": "13.7.5",
+		"@oh-my-pi/omp-stats": "13.7.6",
+		"@oh-my-pi/pi-agent-core": "13.7.6",
+		"@oh-my-pi/pi-ai": "13.7.6",
+		"@oh-my-pi/pi-natives": "13.7.6",
+		"@oh-my-pi/pi-tui": "13.7.6",
+		"@oh-my-pi/pi-utils": "13.7.6",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/tools/ast-edit.ts

@@ -16,6 +16,7 @@ import type { ToolSession } from ".";
 import type { OutputMeta } from "./output-meta";
 import { hasGlobPathChars, parseSearchPath, resolveToCwd } from "./path-utils";
 import {
+	dedupeParseErrors,
 	formatCount,
 	formatEmptyMessage,
 	formatErrorMessage,
@@ -140,6 +141,7 @@ export class AstEditTool implements AgentTool<typeof astEditSchema, AstEditToolD
 				signal,
 			});
 
+			const dedupedParseErrors = dedupeParseErrors(result.parseErrors);
 			const formatPath = (filePath: string): string => {
 				const cleanPath = filePath.startsWith("/") ? filePath.slice(1) : filePath;
 				if (isDirectory) {
@@ -178,15 +180,15 @@ export class AstEditTool implements AgentTool<typeof astEditSchema, AstEditToolD
 				filesSearched: result.filesSearched,
 				applied: result.applied,
 				limitReached: result.limitReached,
-				parseErrors: result.parseErrors,
+				parseErrors: dedupedParseErrors,
 				scopePath,
 				files: fileList,
 				fileReplacements: [],
 			};
 
 			if (result.totalReplacements === 0) {
-				const parseMessage = result.parseErrors?.length
-					? `\n${formatParseErrors(result.parseErrors).join("\n")}`
+				const parseMessage = dedupedParseErrors.length
+					? `\n${formatParseErrors(dedupedParseErrors).join("\n")}`
 					: "";
 				return toolResult(baseDetails).text(`No replacements made${parseMessage}`).done();
 			}
@@ -258,8 +260,8 @@ export class AstEditTool implements AgentTool<typeof astEditSchema, AstEditToolD
 			if (result.limitReached) {
 				outputLines.push("", "Limit reached; narrow path or increase limit.");
 			}
-			if (result.parseErrors?.length) {
-				outputLines.push("", ...formatParseErrors(result.parseErrors));
+			if (dedupedParseErrors.length) {
+				outputLines.push("", ...formatParseErrors(dedupedParseErrors));
 			}
 
 			// Register pending action so `resolve` can apply or discard these previewed changes
@@ -281,13 +283,14 @@ export class AstEditTool implements AgentTool<typeof astEditSchema, AstEditToolD
 							maxFiles,
 							failOnParseError: false,
 						});
+						const dedupedApplyParseErrors = dedupeParseErrors(applyResult.parseErrors);
 						const appliedDetails: AstEditToolDetails = {
 							totalReplacements: applyResult.totalReplacements,
 							filesTouched: applyResult.filesTouched,
 							filesSearched: applyResult.filesSearched,
 							applied: applyResult.applied,
 							limitReached: applyResult.limitReached,
-							parseErrors: applyResult.parseErrors,
+							parseErrors: dedupedApplyParseErrors,
 							scopePath,
 							files: fileList,
 							fileReplacements,

package/src/tools/ast-grep.ts

@@ -16,6 +16,7 @@ import type { ToolSession } from ".";
 import type { OutputMeta } from "./output-meta";
 import { hasGlobPathChars, parseSearchPath, resolveToCwd } from "./path-utils";
 import {
+	dedupeParseErrors,
 	formatCount,
 	formatEmptyMessage,
 	formatErrorMessage,
@@ -133,6 +134,11 @@ export class AstGrepTool implements AgentTool<typeof astGrepSchema, AstGrepToolD
 				signal,
 			});
 
+			const normalizedParseErrors = (result.parseErrors ?? []).map(error => {
+				const parseError = error.match(/^.+: (.+: parse error \(syntax tree contains error nodes\))$/);
+				return parseError?.[1] ?? error;
+			});
+			const dedupedParseErrors = dedupeParseErrors(normalizedParseErrors);
 			const formatPath = (filePath: string): string => {
 				const cleanPath = filePath.startsWith("/") ? filePath.slice(1) : filePath;
 				if (isDirectory) {
@@ -165,15 +171,15 @@ export class AstGrepTool implements AgentTool<typeof astGrepSchema, AstGrepToolD
 				fileCount: result.filesWithMatches,
 				filesSearched: result.filesSearched,
 				limitReached: result.limitReached,
-				parseErrors: result.parseErrors,
+				parseErrors: dedupedParseErrors,
 				scopePath,
 				files: fileList,
 				fileMatches: [],
 			};
 
 			if (result.matches.length === 0) {
-				const parseMessage = result.parseErrors?.length
-					? `\n${formatParseErrors(result.parseErrors).join("\n")}`
+				const parseMessage = dedupedParseErrors.length
+					? `\n${formatParseErrors(dedupedParseErrors).join("\n")}`
 					: "";
 				return toolResult(baseDetails).text(`No matches found${parseMessage}`).done();
 			}
@@ -253,8 +259,8 @@ export class AstGrepTool implements AgentTool<typeof astGrepSchema, AstGrepToolD
 			if (result.limitReached) {
 				outputLines.push("", "Result limit reached; narrow path pattern or increase limit.");
 			}
-			if (result.parseErrors?.length) {
-				outputLines.push("", ...formatParseErrors(result.parseErrors));
+			if (dedupedParseErrors.length) {
+				outputLines.push("", ...formatParseErrors(dedupedParseErrors));
 			}
 
 			return toolResult(details).text(outputLines.join("\n")).done();

package/src/tools/render-utils.ts

@@ -533,10 +533,25 @@ export function wrapBrackets(text: string, theme: Theme): string {
 
 export const PARSE_ERRORS_LIMIT = 20;
 
+export function dedupeParseErrors(errors: string[] | undefined): string[] {
+	if (!errors || errors.length === 0) return [];
+	const seen = new Set<string>();
+	const deduped: string[] = [];
+	for (const error of errors) {
+		if (seen.has(error)) continue;
+		seen.add(error);
+		deduped.push(error);
+	}
+	return deduped;
+}
+
 export function formatParseErrors(errors: string[]): string[] {
-	if (errors.length === 0) return [];
-	const capped = errors.slice(0, PARSE_ERRORS_LIMIT);
+	const deduped = dedupeParseErrors(errors);
+	if (deduped.length === 0) return [];
+	const capped = deduped.slice(0, PARSE_ERRORS_LIMIT);
 	const header =
-		errors.length > PARSE_ERRORS_LIMIT ? `Parse issues (${PARSE_ERRORS_LIMIT} / ${errors.length}):` : "Parse issues:";
+		deduped.length > PARSE_ERRORS_LIMIT
+			? `Parse issues (${PARSE_ERRORS_LIMIT} / ${deduped.length}):`
+			: "Parse issues:";
 	return [header, ...capped.map(err => `- ${err}`)];
 }