Skip to content

Keep context failures from poisoning account pools#14

Merged
gitcommit90 merged 1 commit into
mainfrom
fix/context-failure-status
Jul 18, 2026
Merged

Keep context failures from poisoning account pools#14
gitcommit90 merged 1 commit into
mainfrom
fix/context-failure-status

Conversation

@gitcommit90

@gitcommit90 gitcommit90 commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • classify Responses context-window failures as request errors instead of synthetic transient 502s
  • avoid temporary account locks for deterministic oversized requests
  • preserve the final request status instead of surfacing a misleading 429 after a real quota-locked account is skipped

Validation

  • node --test tests/router-fallback.test.js
  • npm test (302/302)
  • git diff --check

Open in Devin Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@gitcommit90
gitcommit90 merged commit 944db73 into main Jul 18, 2026
2 checks passed
@gitcommit90
gitcommit90 deleted the fix/context-failure-status branch July 18, 2026 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant