CalvinBackup/shannon
1
0
Fork 0
mirror of https://github.com/KeygraphHQ/shannon.git synced 2026-02-12 17:22:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

revert: remove '402' billing pattern causing false positives

Browse Source 
Reverts 5428422 - the pattern matched tool call IDs containing "402"
This commit is contained in:
ajmallesh
2026-01-16 17:29:54 -08:00
parent 5428422b6c
commit f04244f8f9
2 changed files with 0 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/ai/message-handlers.ts
View File

@@ -60,15 +60,12 @@ export function detectApiError(content: string): ApiErrorDetection {
// When Claude Code hits its spending cap, it returns a short message like // When Claude Code hits its spending cap, it returns a short message like
// "Spending cap reached resets 8am" instead of throwing an error. // "Spending cap reached resets 8am" instead of throwing an error.
// These should retry with 5-30 min backoff so workflows can recover when cap resets. // These should retry with 5-30 min backoff so workflows can recover when cap resets.
// Also catches provider billing errors (OpenRouter 402, etc.) that appear in message content.
const BILLING_PATTERNS = [ const BILLING_PATTERNS = [
'spending cap', 'spending cap',
'spending limit', 'spending limit',
'cap reached', 'cap reached',
'budget exceeded', 'budget exceeded',
'usage limit', 'usage limit',
'402', // HTTP Payment Required (universal)
'insufficient credits', // OpenRouter
]; ];
const isBillingError = BILLING_PATTERNS.some((pattern) => const isBillingError = BILLING_PATTERNS.some((pattern) =>

2
src/error-handling.ts
View File

@@ -211,9 +211,7 @@ export function classifyErrorForTemporal(error: unknown): TemporalErrorClassific
// === BILLING ERRORS (Retryable with long backoff) === // === BILLING ERRORS (Retryable with long backoff) ===
// Anthropic returns billing as 400 invalid_request_error // Anthropic returns billing as 400 invalid_request_error
// Human can add credits OR wait for spending cap to reset (5-30 min backoff) // Human can add credits OR wait for spending cap to reset (5-30 min backoff)
// 402 is HTTP "Payment Required" - universal across providers (OpenRouter, etc.)
if ( if (
message.includes('402') ||
message.includes('billing_error') || message.includes('billing_error') ||
message.includes('credit balance is too low') || message.includes('credit balance is too low') ||
message.includes('insufficient credits') || message.includes('insufficient credits') ||