fix: strip leading empty lines in sanitizeUserFacingText (#16158) (thanks @mcinteerj)

steipete · steipete · commit 5d29c8f79845 · 2026-02-14T16:33:32.000+01:00
diff --git a/src/agents/pi-embedded-helpers/errors.ts b/src/agents/pi-embedded-helpers/errors.ts
@@ -527,7 +527,10 @@ export function sanitizeUserFacingText(text: string, opts?: { errorContext?: boo
     }
   }
 
-  return collapseConsecutiveDuplicateBlocks(stripped).trimStart();
+  // Strip leading blank lines (including whitespace-only lines) without clobbering indentation on
+  // the first content line (e.g. markdown/code blocks).
+  const withoutLeadingEmptyLines = stripped.replace(/^(?:[ \t]*\r?\n)+/, "");
+  return collapseConsecutiveDuplicateBlocks(withoutLeadingEmptyLines);
 }
 
 export function isRateLimitAssistantError(msg: AssistantMessage | undefined): boolean {

Original file line number	Diff line number	Diff line change
`@@ -527,7 +527,10 @@ export function sanitizeUserFacingText(text: string, opts?: { errorContext?: boo`
`527`	`527`	`}`
`528`	`528`	`}`
`529`	`529`
`530`		`- return collapseConsecutiveDuplicateBlocks(stripped).trimStart();`
	`530`	`+ // Strip leading blank lines (including whitespace-only lines) without clobbering indentation on`
	`531`	`+ // the first content line (e.g. markdown/code blocks).`
	`532`	`+ const withoutLeadingEmptyLines = stripped.replace(/^(?:[ \t]*\r?\n)+/, "");`
	`533`	`+ return collapseConsecutiveDuplicateBlocks(withoutLeadingEmptyLines);`
`531`	`534`	`}`
`532`	`535`
`533`	`536`	`export function isRateLimitAssistantError(msg: AssistantMessage \| undefined): boolean {`