nyrn
/
openclaw
mirror of https://github.com/openclaw/openclaw.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix: strip HTTP status code prefix from surfaced rate limit messages

This commit is contained in:
Saurabh 2026-03-25 20:54:09 +05:30 committed by Altay
parent dd19536ea6
commit e97efdd732
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/agents/pi-embedded-helpers.formatassistanterrortext.test.ts
View File

@ -147,6 +147,14 @@ describe("formatAssistantErrorText", () => {
);
});
it("strips leading HTTP status code prefix from non-JSON rate limit messages", () => {
const msg = makeAssistantError("429 Your quota has been exhausted, try again in 24 hours");
const result = formatAssistantErrorText(msg);
expect(result).toContain("try again in 24 hours");
expect(result).not.toMatch(/^⚠️ 429\b/);
expect(result).toBe("⚠️ Your quota has been exhausted, try again in 24 hours");
});
it("returns a friendly message for empty stream chunk errors", () => {
const msg = makeAssistantError("request ended without sending any chunks");
expect(formatAssistantErrorText(msg)).toBe("LLM request timed out.");

4
src/agents/pi-embedded-helpers/errors.ts
View File

@ -68,7 +68,9 @@ const RATE_LIMIT_SPECIFIC_HINT_RE =
function extractProviderRateLimitMessage(raw: string): string | undefined {
// Try to pull a human-readable message out of a JSON error payload first.
const info = parseApiErrorInfo(raw);
const candidate = info?.message ?? raw;
// When the raw string is not a JSON payload, strip any leading HTTP status
// code (e.g. "429 ") so the surfaced message stays clean.
const candidate = info?.message ?? (extractLeadingHttpStatus(raw.trim())?.rest || raw);
if (!candidate || !RATE_LIMIT_SPECIFIC_HINT_RE.test(candidate)) {
return undefined;