Skip to content

Merge dev to main#522

Merged
zbigniewsobiecki merged 1 commit intomainfrom
dev
Feb 24, 2026
Merged

Merge dev to main#522
zbigniewsobiecki merged 1 commit intomainfrom
dev

Conversation

@zbigniewsobiecki
Copy link
Copy Markdown
Member

Summary

Routine dev → main promotion after CI passed on dev.

🤖 Generated with Claude Code

Gemini Flash Lite via OpenRouter intermittently returns streaming
responses with zero text content (~40% failure rate in production).
Add a single retry with 1s delay in callProgressModelOnce() so the
first empty response no longer immediately falls back to the generic
template. Only the failure path adds latency.

Update existing empty-output tests to expect both attempts to fail
before throwing. Add two new tests verifying retry-then-success for
both empty-text and no-events scenarios.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
@aaight
Copy link
Copy Markdown
Collaborator

aaight commented Feb 24, 2026

🤖 Just a sec, looking into the dev to main merge

@aaight
Copy link
Copy Markdown
Collaborator

aaight commented Feb 24, 2026

✅ Dev → main promotion looks good!

The retry logic from #521 is working as expected:

  • Handles intermittent empty responses from Gemini Flash Lite
  • Retries once with 1s delay before failing
  • All tests passing with comprehensive coverage of retry scenarios

CI Status: Waiting for final check to complete. Earlier run passed successfully, so this should be good to merge once green.

Ready to merge once CI completes! 🚀

@zbigniewsobiecki zbigniewsobiecki merged commit 39eff21 into main Feb 24, 2026
10 checks passed
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.

2 participants