Skip to content

fix(routing): reject empty auto streams - #169

Merged
actus7 merged 5 commits into
mainfrom
fix/auto-routing-empty-stream
Jul 29, 2026
Merged

fix(routing): reject empty auto streams#169
actus7 merged 5 commits into
mainfrom
fix/auto-routing-empty-stream

Conversation

@actus7

@actus7 actus7 commented Jul 29, 2026

Copy link
Copy Markdown
Owner

Resumo

  • exige primeiro evento útil do stream, não apenas bytes recebidos
  • trata stream 200 com frames vazios/finalização sem texto como falha
  • aplica fallback e cooldown quando o modelo não produz conteúdo útil
  • faz Cloudflare Workers AI retornar 502 quando a API responde sem texto
  • cobre stream vazio, stream silencioso e resposta Cloudflare vazia

Testes

  • pnpm test
  • pnpm typecheck
  • pnpm lint (passa com warning existente em .remember/tmp/last-ndc.ts)

Copilot AI review requested due to automatic review settings July 29, 2026 17:35
@vercel

vercel Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
modelhub Ready Ready Preview, Comment Jul 29, 2026 6:57pm

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@actus7
actus7 merged commit 131762c into main Jul 29, 2026
12 checks passed
@actus7
actus7 deleted the fix/auto-routing-empty-stream branch July 29, 2026 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Backend changes frontend Frontend changes tests Test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants