fix(providers): classify Cloudflare neuron 429 as quota_exhausted (#6980)

[xier2012]

Summary

Cloudflare Workers AI free-tier exhaustion (429, body: "you have used up your daily free allocation of 10,000 neurons", workers-ai error 4006) was misclassified as a transient rate_limit, so the connection was only locked for ~5s and the combo kept retrying every model against a budget that only resets at UTC midnight.

This PR:

• Adds a cloudflare-ai provider rule in open-sse/config/providerErrorRules.ts scoped connection (per-account budget, same reasoning as the existing opencode rules) that returns quota_exhausted on a 429 whose body mentions "daily free allocation".
• Adds /daily free allocation/i to QUOTA_PATTERNS in src/shared/utils/classify429.ts as defense-in-depth for the classify429FromError path.

Test plan

• New regression test tests/unit/cloudflare-ai-neuron-quota-6980.test.ts (node:test) asserts: the rule matches and yields quota_exhausted/connection, a generic rate-limit 429 does not match, and classify429 classifies the Cloudflare body as quota_exhausted.

Closes #6980

[gemini-code-assist]

Code Review

This pull request introduces error classification rules and defense-in-depth patterns to correctly identify Cloudflare Workers AI free tier neuron allocation exhaustion (429 errors) as a connection-scoped quota exhaustion rather than a transient rate limit. It also includes corresponding unit tests. The reviewer suggested replacing the top-level dynamic await import statements in the test file with standard static imports for cleaner and more idiomatic TypeScript code.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

Using top-level dynamic await import is unnecessary here and can be replaced with standard static import statements. Static imports are statically analyzable, cleaner, and more idiomatic in TypeScript.

import { getProviderErrorRuleMatch } from "../../open-sse/config/providerErrorRules.ts";
import { classify429 } from "../../src/shared/utils/classify429.ts";

[diegosouzapw]

Thank you for tracking down and fixing this — the root cause diagnosis (daily free allocation body falling through QUOTA_PATTERNS) and the fix itself are correct and match what we're merging. Two other contributors (rafaumeu #6983, maxmad64bis #6981) submitted essentially the same fix around the same time; we're merging #6983 because its test suite is broader (11 assertions covering both the provider-rule and classify429 layers, plus edge cases like case-insensitivity and the JSON vs plain-string body shapes). Closing this one as superseded, with full credit for independently catching and fixing #6980 — appreciate you filing it.

[diegosouzapw]

Thanks @xier2012! The Cloudflare neuron-429 quota-exhausted classification landed today via #6983, covering this path. Closing as superseded — your analysis of the 429 shape matched what shipped. If you find an uncovered variant, please reopen with the specific response body.