Skip to content

test(ci): gate high-risk coverage scopes#128

Merged
xingkaixin merged 2 commits into
mainfrom
feat/cs-48-coverage-gates
Jul 11, 2026
Merged

test(ci): gate high-risk coverage scopes#128
xingkaixin merged 2 commits into
mainfrom
feat/cs-48-coverage-gates

Conversation

@xingkaixin

Copy link
Copy Markdown
Owner

Summary

  • explicitly include high-risk Core, CLI runtime, Web hooks, and Web interaction sources in coverage collection
  • add independent line thresholds for each scope based on the measured baseline
  • run the coverage gate in the Ubuntu smoke job

Impact

Coverage regressions now fail CI in the affected scope instead of being hidden by an aggregate repository percentage. Previously unloaded workers and hooks are counted as zero coverage rather than omitted from the report.

Baselines and thresholds

Scope Baseline Threshold
Core cache/runtime 87.51% 75%
CLI runtime 79.20% 78%
Web hooks 64.86% 63%
Web interactions 77.73% 76%

Validation

  • pnpm test:coverage (662 tests)
  • temporarily removed live-scan-store.test.ts; CLI runtime coverage fell to 35.39% and the gate failed as expected
  • pnpm format
  • pnpm lint
  • pnpm format:check
  • pnpm build
  • pnpm test (662 tests)
  • pnpm test:e2e (6 tests)

Closes CS-48

@xingkaixin xingkaixin marked this pull request as ready for review July 11, 2026 13:12
@xingkaixin xingkaixin merged commit 70eabf7 into main Jul 11, 2026
7 checks passed
@xingkaixin xingkaixin deleted the feat/cs-48-coverage-gates branch July 11, 2026 13:12
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.

1 participant