fix(auto-reply): narrow conversation-state 400 classification

anyech · anyech · commit a67e5670eb11 · 2026-06-19T09:05:26.000Z
diff --git a/src/auto-reply/reply/provider-request-error-classifier.test.ts b/src/auto-reply/reply/provider-request-error-classifier.test.ts
@@ -26,7 +26,6 @@ describe("provider request error classifier", () => {
       "alternating role ordering mismatch",
       "messages: roles must alternate between user and assistant",
     ],
-    ["bare Responses 400", "400 status code (no body)"],
     [
       "local replay invariant guard",
       "invalid_replay_transcript: OpenAI Responses replay contains dangling_tool_call toolCallId=call_1 at message index 4",
@@ -39,6 +38,10 @@ describe("provider request error classifier", () => {
     });
   });
 
+  it("leaves bare no-body 400 provider failures unclassified", () => {
+    expect(classifyProviderRequestError(new Error("400 status code (no body)"))).toBeUndefined();
+  });
+
   it("leaves explicit HTTP 429 rate-limit failures on the existing rate-limit path", () => {
     expect(classifyProviderRequestError(new Error("429: rate limit exceeded"))).toBeUndefined();
   });
diff --git a/src/auto-reply/reply/provider-request-error-classifier.ts b/src/auto-reply/reply/provider-request-error-classifier.ts
@@ -69,7 +69,6 @@ export function isProviderConversationStateErrorMessage(message: string): boolea
     lower.includes("function call turn comes immediately after") ||
     lower.includes("incorrect role information") ||
     lower.includes("roles must alternate") ||
-    (lower.includes("400 status code") && lower.includes("no body")) ||
     lower.includes("invalid_replay_transcript")
   );
 }

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,6 @@ export function isProviderConversationStateErrorMessage(message: string): boolea`
`69`	`69`	`lower.includes("function call turn comes immediately after") \|\|`
`70`	`70`	`lower.includes("incorrect role information") \|\|`
`71`	`71`	`lower.includes("roles must alternate") \|\|`
`72`		`- (lower.includes("400 status code") && lower.includes("no body")) \|\|`
`73`	`72`	`lower.includes("invalid_replay_transcript")`
`74`	`73`	`);`
`75`	`74`	`}`