Problem
The repository has no authoritative static-analysis gate:
.github/workflows/build-test.yml runs only npm ci and npm test.
npm run typecheck exists but is not run in CI.
- Both
lint and lint:staged invoke biome lint --staged, so there is no repository-wide lint script suitable for a clean CI checkout.
Current validation on master at 0ce34ae shows why this matters:
npx tsc --noEmit --noUnusedLocals --noUnusedParameters reports 22 unused-code diagnostics, including dead methods/fields such as attemptChannelRecovery, clearBuffer, handleCancelTransfer, and subBuffer.
npx biome lint src --max-diagnostics=200 reports 17 errors, 366 warnings, and 19 infos.
The normal typecheck passes only because unused-code checks are disabled, and staged lint cannot establish whole-repository cleanliness.
Acceptance criteria
Problem
The repository has no authoritative static-analysis gate:
.github/workflows/build-test.ymlruns onlynpm ciandnpm test.npm run typecheckexists but is not run in CI.lintandlint:stagedinvokebiome lint --staged, so there is no repository-wide lint script suitable for a clean CI checkout.Current validation on
masterat0ce34aeshows why this matters:npx tsc --noEmit --noUnusedLocals --noUnusedParametersreports 22 unused-code diagnostics, including dead methods/fields such asattemptChannelRecovery,clearBuffer,handleCancelTransfer, andsubBuffer.npx biome lint src --max-diagnostics=200reports 17 errors, 366 warnings, and 19 infos.The normal typecheck passes only because unused-code checks are disabled, and staged lint cannot establish whole-repository cleanliness.
Acceptance criteria
noUnusedLocalsandnoUnusedParameters, deleting or correctly wiring the 22 exposed surfaces.npm run typecheckand the full-source lint command to the GitHub Actions job alongsidenpm test.