fix: stop retrying non-transient rate limits (weekly/monthly quotas)

Author: eeshm

packages/opencode/src/session/retry.ts

@@ -60,6 +60,15 @@ export function retryable(error: Err) {
     // even when the provider SDK doesn't explicitly mark them as retryable.
     if (!error.data.isRetryable && !(status !== undefined && status >= 500)) return undefined
     if (error.data.responseBody?.includes("FreeUsageLimitError")) return GO_UPSELL_MESSAGE
+    // Non-transient rate limits (weekly/monthly quotas) should not be retried
+    // they won't resolve with backoff and can take days to reset.
+    const lower = error.data.message.toLowerCase()
+    if (
+      lower.includes("weekly") ||
+      lower.includes("monthly") ||
+      lower.includes("exceeded your")
+    )
+      return undefined
     return error.data.message.includes("Overloaded") ? "Provider is overloaded" : error.data.message
   }
 

packages/opencode/test/session/retry.test.ts

@@ -230,6 +230,26 @@ describe("session.retry.retryable", () => {
     expect(retryable).toBeDefined()
     expect(retryable).toBe("Response decompression failed")
   })
+
+  test("does not retry weekly rate limit errors", () => {
+    const error = new MessageV2.APIError({
+      message: "Too Many Requests: Sorry, you've exceeded your weekly rate limit. Please review ...",
+      isRetryable: true,
+      statusCode: 429,
+    }).toObject() as MessageV2.APIError
+
+    expect(SessionRetry.retryable(error)).toBeUndefined()
+  })
+
+  test("does not retry monthly quota errors", () => {
+    const error = new MessageV2.APIError({
+      message: "You have exceeded your monthly usage quota",
+      isRetryable: true,
+      statusCode: 429,
+    }).toObject() as MessageV2.APIError
+
+    expect(SessionRetry.retryable(error)).toBeUndefined()
+  })
 })
 
 describe("session.message-v2.fromError", () => {