refactor(zwave_js): hoist push event handlers to private methods

[raman325]

Proposed change

The `on_credential_changed` and `on_credential_deleted` closures inside `setup_push_subscription` captured `self` implicitly via lexical scope. Hoist them to bound private methods (`self._on_credential_changed`, `self._on_credential_deleted`) so:

1. The handler surface is explicit in the class layout (visible to type checkers and to `grep`).
2. They match how the Matter provider already structures its push handlers (`_handle_lock_operation`, `_handle_lock_user_change`, `_dispatch_lock_user_change`).
3. Future test mocking / introspection can patch them by name rather than monkey-patching the parent method.

Pure refactor — handler bodies are identical, `@callback` decorator preserved. The `setup_push_subscription` body now wires the bound methods directly into `self.node.on(...)` registrations.

Type of change

• Dependency upgrade
• Bugfix (non-breaking change which fixes an issue)
• New feature (which adds functionality)
• Breaking change (fix/feature causing existing functionality to break)
• Code quality improvements to existing code or addition of tests

Test plan

• All 76 existing zwave_js provider tests pass without modification
• Full suite passes locally

🤖 Generated with Claude Code

[Copilot]

Pull request overview

Refactors the Z-Wave JS provider’s push subscription setup by hoisting inline credential event-handler closures into explicit private instance methods, aligning handler structure with other providers and making the handlers easier to locate, type-check, and mock in tests.

Changes:

• Replaces the on_credential_changed / on_credential_deleted inner @callback closures with bound methods self._on_credential_changed / self._on_credential_deleted.
• Keeps handler logic and @callback behavior the same while simplifying setup_push_subscription() wiring.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.98%. Comparing base (465678c) to head (dd2b141).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1244   +/-   ##
=======================================
  Coverage   96.98%   96.98%           
=======================================
  Files          53       53           
  Lines        6168     6168           
  Branches      461      461           
=======================================
  Hits         5982     5982           
  Misses        186      186           

Flag	Coverage Δ
python	97.52% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...components/lock_code_manager/providers/zwave_js.py	100.00% <100.00%> (ø)

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.