Skip to content

fix(sse): clear release/v3.8.50 base reds — dup import, stale suppressions, skill sync, stryker testFiles - #10336

Closed
diegosouzapw wants to merge 2 commits into
release/v3.8.50from
fix/release-v3.8.50-basereds-r5
Closed

diegosouzapw wants to merge 2 commits into
release/v3.8.50from
fix/release-v3.8.50-basereds-r5

Conversation

@diegosouzapw

Copy link
Copy Markdown
Collaborator

O que

Drena os base-reds determinísticos do tip de release/v3.8.50 (266e39d36d) — round 5. Refs #9985.

Fix Arquivo Gate que valida (antes → depois, rodado local)
Remove import duplicado de sseCommentsEnabled (linha 77, clobber de merge do #9378) open-sse/utils/stream.ts check:open-sse-typecheck: TS2300 ×2 → OK (26 pré-existentes no baseline). Desbloqueia também Fast Production Build / dast-smoke / Build advisory (Turbopack "Ecmascript file had an error" apontava este arquivo)
Prune de 5 suppressions ESLint obsoletas (0 adições) config/quality/eslint-suppressions.json npm run lint:json -- --max-warnings 0: exit 2 ("suppressions left that do not occur anymore") → exit 0
Regen do SKILL.md gerado de omni-settings skills/omni-settings/SKILL.md check:agent-skills-sync: exit 2 (GENERATED: + omni-settings) → ✓ 46 unchanged
Registra tests/unit/guardrails/visionBridge-responses-9597.test.ts (do #10202) em tap.testFiles stryker.conf.json check:mutation-test-coverage --strict: 1 missing → ✓ no drift

Fora de escopo (reds da base que permanecem)

  • Falhas de unit shards (combo/priority retry, i18n denoRelayOrgDomainHint, signal-abort, poolside) — precisam de diagnóstico próprio (/sweep-reds).
  • Drift de ratchet dead-code (409 > 248) — rebaseline na release, por política.

⚠️ base-red inherited: #9985 (este PR é o fix de parte desses reds; os gates corrigidos devem passar já na rodada de CI deste PR, pois o CI roda contra o merge-ref base+PR).

@diegosouzapw

Copy link
Copy Markdown
Collaborator Author

Fechando: redundante + carrega uma regressão. A base avançou 11 commits (266e39d36d → 27e163e2c9) enquanto este PR estava na fila, e os fixes chegaram lá por outras vias:

Fix deste PR Estado na base 27e163e2c9
Import duplicado sseCommentsEnabled (open-sse/utils/stream.ts) ✅ já corrigido (diff vs meu HEAD = 0 linhas)
Prune de 5 suppressions ESLint obsoletas ✅ já aplicado (diff = 0 linhas)
visionBridge-responses-9597.test.ts em tap.testFiles ✅ já registrado (diff = 0 linhas)
Regen do skills/omni-settings/SKILL.md ❌ regressão — o regen --apply removeu o bloco curado <!-- skill:custom-start --> (thinking-budget, preservado em #10169) que a base mantém. Não deve ser mergeado.

Bônus da investigação, para quem for drenar os reds restantes: o Fast Production Build no meu HEAD morria em @omniroute/opencode-plugin/src/index.ts:5481 — um }); duplicado (mesma classe de clobber de merge do stream.ts) cujo primeiro sintoma era TS1005: ',' expected e cuja cascata gerava ~20 TS2304: Cannot find name. Também já corrigido no tip atual.

Nenhum commit deste PR deve ir para a release. Branch fix/release-v3.8.50-basereds-r5 será removida.

@diegosouzapw
diegosouzapw deleted the fix/release-v3.8.50-basereds-r5 branch August 14, 2026 04:00
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