Skip to content

chore(types): enable strict backend type checking#4741

Draft
AlCalzone wants to merge 18 commits into
alcalzone-alcalzone-backend-refactor-node-projectifrom
alcalzone-alcalzone-backend-refactor-enable-strict
Draft

chore(types): enable strict backend type checking#4741
AlCalzone wants to merge 18 commits into
alcalzone-alcalzone-backend-refactor-node-projectifrom
alcalzone-alcalzone-backend-refactor-enable-strict

Conversation

@AlCalzone

@AlCalzone AlCalzone commented Jul 12, 2026

Copy link
Copy Markdown
Member

Enables strict backend and backend-test type checking as the final type-safety gate.

  • Resolves remaining nullable and unknown-value boundaries without intended runtime changes.
  • Adds non-vacuous server/test type checks to CI and locks the extracted module boundaries.
  • Review focus: guards, defaults, and error paths that could accidentally alter compatibility.

Part of #4722. Stacked on #4735.

@coveralls

coveralls commented Jul 12, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 29582065023

Warning

No base build found for commit 7444653 on alcalzone-alcalzone-backend-refactor-node-projecti.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 62.995%

Details

  • Patch coverage: 204 uncovered changes across 5 files (99 of 303 lines covered, 32.67%).

Uncovered Changes

File Changed Covered %
api/lib/ZwaveClient.ts 179 50 27.93%
api/lib/Gateway.ts 82 22 26.83%
api/app.ts 23 12 52.17%
api/lib/ZnifferManager.ts 7 4 57.14%
api/lib/DebugManager.ts 4 3 75.0%
Total (7 files) 303 99 32.67%

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 8568
Covered Lines: 5640
Line Coverage: 65.83%
Relevant Branches: 5203
Covered Branches: 3035
Branch Coverage: 58.33%
Branches in Coverage %: Yes
Coverage Strength: 36.71 hits per line

💛 - Coveralls

AlCalzone and others added 18 commits July 17, 2026 13:49
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@AlCalzone
AlCalzone force-pushed the alcalzone-alcalzone-backend-refactor-enable-strict branch from 5486189 to f810ba6 Compare July 17, 2026 12:56
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.

2 participants