Fix StopIteration crash when lock has unmanaged slots with masked PINs

[raman325]

Proposed change

When a lock has code slots with masked PINs (all *'s) that aren't managed by Lock Code Manager, the async_get_usercodes() function would crash with RuntimeError: coroutine raised StopIteration.

This occurred because the code iterated over ALL slots from the lock's cache, but the next() call at line 434 tried to find a matching LCM config entry for each slot. Slots not managed by LCM have no corresponding config entry, causing StopIteration to be raised inside the async function, which Python 3.7+ converts to RuntimeError.

The fix adds an early check: if the slot isn't in the configured code_slots, we record an empty string and skip the config entry lookup entirely.

Also adds a regression test to prevent this from recurring.

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

Additional information

• This PR fixes or closes issue:
• This PR is related to issue:

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR fixes a RuntimeError: coroutine raised StopIteration crash that occurred when a Z-Wave JS lock has code slots with masked PINs (all *'s) that are not managed by Lock Code Manager. The bug was caused by attempting to find a matching LCM config entry for unmanaged slots using next(), which raised StopIteration when no match was found.

Key changes:

• Added early validation to skip config entry lookup for unmanaged slots with masked PINs
• Added regression test to prevent this issue from recurring
• Removed redundant conditional check around logging statement

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
custom_components/lock_code_manager/providers/zwave_js.py	Added early return logic to handle unmanaged slots with masked PINs, preventing StopIteration from being raised
tests/providers/test_zwave_js.py	Added comprehensive regression test that verifies the fix by testing both managed and unmanaged slots with masked PINs

---

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

[codecov]

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 93.13%. Comparing base (8920cb3) to head (4a0b78c).
⚠️ Report is 17 commits behind head on main.

Files with missing lines	Patch %	Lines
...components/lock_code_manager/providers/zwave_js.py	75.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #760      +/-   ##
==========================================
- Coverage   93.21%   93.13%   -0.08%     
==========================================
  Files          29       29              
  Lines        2357     2359       +2     
  Branches       77       77              
==========================================
  Hits         2197     2197              
- Misses        160      162       +2     

Flag	Coverage Δ
python	92.79% <75.00%> (-0.09%)	⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.