Repository navigation
fix(quality): complexity gate cobre bin/+electron + tracked-artifacts no pre-commit (6A.11/6A.12) - #4318
Merged
Conversation
…n fix) ESLINT_ARGS passava só 'src open-sse', mas o config eslint.complexity.config.mjs e o complexity-baseline.json já documentavam o escopo src+open-sse+electron+bin. A edição do scan nunca tinha sido aplicada: o gate alegava cobrir bin/electron e nunca os varria (fake-green — uma god-function nova em bin/ passava verde). - exporta ESLINT_ARGS + adiciona 'electron' e 'bin' (casa o config) - teste de build trava o escopo do scan - baseline 1887->1888: electron+bin medem 0 (widening 0-custo); o +1 é drift pré-existente de src/open-sse da base a23d0d6 (#4308 et al.), não do widening
O gate estava no CI + pre-push, mas faltava no pre-commit — e o incidente do symlink (artefato de build trackeado) acontece no 'git add'. Fecha o buraco no ponto mais cedo.
Contributor
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
Merged
tkgo11
pushed a commit
to tkgo11/OmniRoute
that referenced
this pull request
Sep 23, 2026
…s in pre-commit (cheaper-inference#4318) * fix(quality): complexity gate varre bin/ + electron (6A.11, fake-green fix) ESLINT_ARGS passava só 'src open-sse', mas o config eslint.complexity.config.mjs e o complexity-baseline.json já documentavam o escopo src+open-sse+electron+bin. A edição do scan nunca tinha sido aplicada: o gate alegava cobrir bin/electron e nunca os varria (fake-green — uma god-function nova em bin/ passava verde). - exporta ESLINT_ARGS + adiciona 'electron' e 'bin' (casa o config) - teste de build trava o escopo do scan - baseline 1887->1888: electron+bin medem 0 (widening 0-custo); o +1 é drift pré-existente de src/open-sse da base ecbbff1 (cheaper-inference#4308 et al.), não do widening * fix(quality): check:tracked-artifacts no pre-commit (6A.12) O gate estava no CI + pre-push, mas faltava no pre-commit — e o incidente do symlink (artefato de build trackeado) acontece no 'git add'. Fecha o buraco no ponto mais cedo.
tkgo11
pushed a commit
to tkgo11/OmniRoute
that referenced
this pull request
Sep 23, 2026
… drift) (cheaper-inference#4330) cheaper-inference#4313 (harvested features) and cheaper-inference#4323 (compression e2e audit) added new conditional branches that landed after cheaper-inference#4318 measured 1888; the release fast-path doesn't run check:complexity, so reconcile on the merged release tip (Rule cheaper-inference#9, release-volatile).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contexto
Fecha dois gaps residuais da Fase 6A do programa Quality Gate, descobertos na auditoria de conclusão dos planos
_tasks/quality/.6A.11 — complexity gate cobre
bin/+electron/(corrige fake-green)O
eslint.complexity.config.mjs(files) e ocomplexity-baseline.jsonjá documentavam o escoposrc+open-sse+electron+bin, mas oESLINT_ARGSdecheck-complexity.mjspassava apenassrc open-sse— a edição do scan nunca tinha sido aplicada (ou foi revertida num merge). Resultado: gate fantasma — o gate alegava cobrirbin//electron/e nunca os varria. Uma god-function nova embin/passaria verde.export const ESLINT_ARGS+ adicionaelectronebin(casa o config).tests/unit/build/check-complexity.test.ts) trava o escopo do scan.1887 → 1888:electron+binmedem 0 violações (electron só temtypes.d.ts, ignorado; os 4.tsdebin/estão sob os thresholds) → o widening é 0-custo.src/open-ssetrazido pela baserelease/v3.8.30(a23d0d678, merges do ciclo incl. fix(api): semantic-cache HIT bills incremental cost 0 + X-OmniRoute-Cost-Saved (PRD-2026-06-19) #4308) — não vem do widening. O fast-path do release não rodacheck:complexity, então o drift só aparece aqui. Mesma família dos rebaselines anteriores; crescimento de feature, não regressão.6A.12 —
check:tracked-artifactsno pre-commitO gate estava no CI + pre-push, mas faltava no
.husky/pre-commit— e o incidente do symlink (artefato de build trackeado) acontece nogit add. Adiciona a chamada ao hook, fechando o buraco no ponto mais cedo.Validação
npm run lint→ 0 errors.npm run check:complexity→ OK (1888/1888).npm run check:tracked-artifacts→ OK..husky/pre-commitroda ponta-a-ponta, exit 0, com[tracked-artifacts] OK.tests/unit/build/*.test.ts→ 294/294 pass.🤖 Generated with Claude Code