Skip to content

fix(cline): label OmniRoute health checks as internal - #10706

Merged
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.50from
arafatkatze:arafatkatze/label-omniroute-synthetic-traffic
Aug 20, 2026
Merged

diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.50from
arafatkatze:arafatkatze/label-omniroute-synthetic-traffic

Conversation

@arafatkatze

@arafatkatze arafatkatze commented Aug 19, 2026 •

Copy link
Copy Markdown
Contributor

Problema

As verificações de integridade de modelos e combos do OmniRoute já se identificam com:

X-Internal-Test: combo-health-check

Quando uma dessas verificações é encaminhada pelo Cline, o construtor de headers do Cline atualmente a identifica com o mesmo valor X-CLIENT-TYPE: omniroute usado pelo tráfego normal de usuários. Por isso, a telemetria downstream não consegue distinguir as verificações internas do OmniRoute das requisições de usuários.

Alteração

  • mapeia o marcador existente combo-health-check para X-CLIENT-TYPE: omniroute-internal-health-check
  • mantém o tráfego normal do Cline/ClinePass inalterado como X-CLIENT-TYPE: omniroute
  • preserva a identificação interna quando o executor reaplica os headers obrigatórios do Cline imediatamente antes do envio

Esta alteração não adiciona nenhum header ou campo no banco de dados e não modifica o corpo das requisições, o roteamento, a autenticação, o faturamento ou os IDs de tarefa.

Teste

node --import tsx/esm --test tests/unit/cline-workos-auth-token-shape.test.ts

O teste de regressão falhava antes da implementação (omniroute em vez de omniroute-internal-health-check) e passou depois dela (12/12).

Também passaram:

npm run typecheck:core
npx prettier --check src/shared/utils/clineAuth.ts tests/unit/cline-workos-auth-token-shape.test.ts
npx eslint src/shared/utils/clineAuth.ts tests/unit/cline-workos-auth-token-shape.test.ts --suppressions-location config/quality/eslint-suppressions.json

⚠️ Estado base-red herdado: #9985. A suíte ampla de testes unitários não é usada como evidência neste PR; caminhos não relacionados da branch base falharam localmente, enquanto o teste de regressão focado e as verificações principais acima estão passando.

@arafatkatze
arafatkatze marked this pull request as ready for review August 19, 2026 18:31
@arafatkatze

Copy link
Copy Markdown
Contributor Author

Oi @diegosouzapw! Aqui é o Ara, do time da @cline. Quando tiver um tempinho, agradeceria muito uma revisão rápida deste PR — a mudança é pequena e fácil de testar. Obrigado!

@diegosouzapw
diegosouzapw merged commit 81b0ff4 into diegosouzapw:release/v3.8.50 Aug 20, 2026
3 checks passed
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
Merged via merge-train (release/v3.8.50, batch1 2026-08-20) — static gates (typecheck/file-size/complexity/cognitive/changelog) green on the combined tree; test:unit reds observed in the boarded run were verified pre-existing on the pure release tip (unrelated flake), not caused by this PR. Thanks for the contribution!
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