Context
Pre-existing CI failures on main that are cargo-culting into unrelated PRs (flagged in #168 review).
Failing jobs
build-and-test (22)
generate-benchmark-report
Root cause
@sint/bridge-mqtt is missing tsconfig.json (per @ExpertVagabond's investigation on #168). The build fails at the workspace level, which blocks any PR that triggers the build-and-test matrix — even PRs that don't touch bridge-mqtt.
Fix
- Add
packages/bridge-mqtt/tsconfig.json (extending the workspace base config, matching siblings like bridge-http)
- Verify
@sint/bridge-mqtt builds cleanly via pnpm --filter @sint/bridge-mqtt run build
- Re-run CI on
main to confirm green
Priority
Medium — not blocking any correctness concerns, but blocks PR author confidence and makes new-PR CI noise. Every reviewer currently has to mentally subtract these from the red checks to decide whether a PR is actually clean.
Context
Pre-existing CI failures on
mainthat are cargo-culting into unrelated PRs (flagged in #168 review).Failing jobs
build-and-test (22)generate-benchmark-reportRoot cause
@sint/bridge-mqttis missingtsconfig.json(per @ExpertVagabond's investigation on #168). The build fails at the workspace level, which blocks any PR that triggers the build-and-test matrix — even PRs that don't touchbridge-mqtt.Fix
packages/bridge-mqtt/tsconfig.json(extending the workspace base config, matching siblings likebridge-http)@sint/bridge-mqttbuilds cleanly viapnpm --filter @sint/bridge-mqtt run buildmainto confirm greenPriority
Medium — not blocking any correctness concerns, but blocks PR author confidence and makes new-PR CI noise. Every reviewer currently has to mentally subtract these from the red checks to decide whether a PR is actually clean.