diff --git a/src/agents/pi-embedded-helpers.isbillingerrormessage.test.ts b/src/agents/pi-embedded-helpers.isbillingerrormessage.test.ts index 141368bf366..fd78372f2e0 100644 --- a/src/agents/pi-embedded-helpers.isbillingerrormessage.test.ts +++ b/src/agents/pi-embedded-helpers.isbillingerrormessage.test.ts @@ -862,15 +862,16 @@ describe("classifyFailoverReason", () => { }); it("classifies generic provider errors as timeout", () => { expect(classifyFailoverReason("An unknown error occurred")).toBe("timeout"); - expect(classifyFailoverReason("An error occurred")).toBe("timeout"); expect(classifyFailoverReason("Internal server error")).toBe("timeout"); expect(classifyFailoverReason("Service unavailable")).toBe("timeout"); // Case-insensitive expect(classifyFailoverReason("AN UNKNOWN ERROR OCCURRED")).toBe("timeout"); - expect(classifyFailoverReason("an error occurred while processing")).toBe("timeout"); // Wrapped in provider payload expect(classifyFailoverReason('{"error":{"message":"An unknown error occurred"}}')).toBe( "timeout", ); + // "an error occurred" alone should NOT match (too broad) + expect(classifyFailoverReason("An error occurred")).toBeNull(); + expect(classifyFailoverReason("A validation error occurred")).toBeNull(); }); }); diff --git a/src/agents/pi-embedded-helpers/errors.ts b/src/agents/pi-embedded-helpers/errors.ts index 99adc32149f..1b737fd4775 100644 --- a/src/agents/pi-embedded-helpers/errors.ts +++ b/src/agents/pi-embedded-helpers/errors.ts @@ -853,7 +853,6 @@ function isGenericProviderError(raw: string): boolean { const lower = raw.toLowerCase(); return ( lower.includes("an unknown error occurred") || - lower.includes("an error occurred") || lower.includes("internal server error") || lower.includes("service unavailable") );