Skip to content

refactor(zwave): extract node registry and projections#4735

Draft
AlCalzone wants to merge 21 commits into
alcalzone-backend-refactor-driver-lifecyclefrom
alcalzone-alcalzone-backend-refactor-node-projecti
Draft

refactor(zwave): extract node registry and projections#4735
AlCalzone wants to merge 21 commits into
alcalzone-backend-refactor-driver-lifecyclefrom
alcalzone-alcalzone-backend-refactor-node-projecti

Conversation

@AlCalzone

@AlCalzone AlCalzone commented Jul 12, 2026

Copy link
Copy Markdown
Member

Extracts node storage, projection, and socket emission from ZwaveClient.

  • Separates registry ownership, serialized node/value projection, and deferred room-aware socket events.
  • Keeps physical, virtual, fake, persisted, MQTT, and Home Assistant behavior compatible.
  • Review focus: node identity across restart, event ordering, deferred stale emissions, and exact payload shape/arity.

Part of #4722. Stacked on #4734.

@coveralls

coveralls commented Jul 12, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 29577590413

Warning

No base build found for commit 77c5432 on alcalzone-backend-refactor-driver-lifecycle.
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: 63.545%

Details

  • Patch coverage: 197 uncovered changes across 2 files (742 of 939 lines covered, 79.02%).

Uncovered Changes

File Changed Covered %
api/lib/ZwaveClient.ts 172 63 36.63%
api/lib/zwave/NodeRegistry.ts 618 530 85.76%
Total (5 files) 939 742 79.02%

Coverage Regressions

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


Coverage Stats

Coverage Status
Relevant Lines: 8452
Covered Lines: 5608
Line Coverage: 66.35%
Relevant Branches: 5121
Covered Branches: 3017
Branch Coverage: 58.91%
Branches in Coverage %: Yes
Coverage Strength: 37.11 hits per line

💛 - Coveralls

@AlCalzone
AlCalzone force-pushed the alcalzone-backend-refactor-driver-lifecycle branch 2 times, most recently from d16cc37 to b434a8c Compare July 17, 2026 06:43
@AlCalzone
AlCalzone force-pushed the alcalzone-alcalzone-backend-refactor-node-projecti branch from d9b643e to e88b74d Compare July 17, 2026 07:13
@AlCalzone
AlCalzone force-pushed the alcalzone-backend-refactor-driver-lifecycle branch from b434a8c to 77c5432 Compare July 17, 2026 11:23
AlCalzone and others added 21 commits July 17, 2026 13:29
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>
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-node-projecti branch from e88b74d to 7444653 Compare July 17, 2026 11:39
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