mirror of
https://github.com/deepseek-ai/deepseek-harness
synced 2026-08-15 21:04:50 +00:00
e9ed7193d8
The coverage lane's wall clock was pinned by a few compiler- and subprocess-bound suites whose v8 instrumentation tax is a multiple of their runtime while contributing nothing the per-file thresholds need: typert generator fixtures (whole-workspace compiler analysis; its src is threshold-excluded) and three scripts/ child-process fixture suites (scripts/ sources are never coverage-measured; in-process imports are covered by their owning package tests). Split ci-coverage into two parallel gates: the instrumented run sets DSH_COVERAGE_EXEMPT_HEAVY=1 and vitest.config.ts drops the exempt suites from both projects (CLI --exclude cannot reach per-project include resolution); a second uninstrumented gate runs exactly those suites, so the aggregate still executes every test. Membership contract and the filter/exclude pairs live in scripts/coverage-exempt.ts. Local 6-worker A/B: instrumented gate 900s -> 260s wall; exempt gate 262s wall runs beside it, so the lane converges near the slower of the two (~4.4min vs ~7min single-gate). DSH_GATE_CONCURRENCY now has two schedulable gates in this lane.
42 lines
2.0 KiB
TypeScript
42 lines
2.0 KiB
TypeScript
/**
|
|
* Heavy suites the coverage aggregate runs uninstrumented in a parallel gate.
|
|
* Membership contract: a suite qualifies only when every coverage-measured
|
|
* file it executes in-process (`coverage.include` spans package src trees;
|
|
* typert generator src is threshold-excluded in vitest.config.ts) is already
|
|
* fully covered by other suites, so removing it from the instrumented run
|
|
* changes no threshold outcome. The aggregate still runs every listed suite
|
|
* plain beside the instrumented gate, so correctness signal is unchanged —
|
|
* only the v8 instrumentation tax on compiler- and subprocess-heavy fixtures
|
|
* is dropped.
|
|
*/
|
|
|
|
/** One coverage-exempt suite: a Vitest CLI filter and its exclude glob. */
|
|
export interface CoverageExemptSuite {
|
|
/** Positional file filter selecting the suite in the uninstrumented gate. */
|
|
readonly filter: string
|
|
/** Exclude glob removing the suite from the instrumented gate. */
|
|
readonly exclude: string
|
|
}
|
|
|
|
/**
|
|
* Set to `1` by the instrumented coverage gate; vitest.config.ts then drops
|
|
* the exempt suites from every project. CLI `--exclude` cannot express this:
|
|
* it does not reach per-project include resolution.
|
|
*/
|
|
export const COVERAGE_EXEMPT_ENV = 'DSH_COVERAGE_EXEMPT_HEAVY'
|
|
|
|
/** Coverage-exempt heavy suites; keep filter and exclude selecting the same files. */
|
|
export const coverageExemptHeavySuites: readonly CoverageExemptSuite[] = [
|
|
// Whole-workspace compiler analysis per case — the lane's longest tail.
|
|
// Generator src is threshold-excluded; tools-catalog's registry and
|
|
// tool-cordis imports are fully covered by those packages' own tests.
|
|
{
|
|
filter: 'packages/typert/generator/tests/',
|
|
exclude: 'packages/typert/generator/tests/**',
|
|
},
|
|
// Real child-process fixtures over scripts/ sources, which coverage never measures.
|
|
{ filter: 'scripts/install-lefthook.spec.ts', exclude: 'scripts/install-lefthook.spec.ts' },
|
|
{ filter: 'scripts/oxlint-contract.spec.ts', exclude: 'scripts/oxlint-contract.spec.ts' },
|
|
{ filter: 'scripts/change-scope.spec.ts', exclude: 'scripts/change-scope.spec.ts' },
|
|
]
|