Skip to content

fix(readers): remove legacy pn532uart driver#607

Merged
wizzomafizzo merged 1 commit intomainfrom
fix/remove-legacy-pn532uart
Apr 3, 2026
Merged

fix(readers): remove legacy pn532uart driver#607
wizzomafizzo merged 1 commit intomainfrom
fix/remove-legacy-pn532uart

Conversation

@wizzomafizzo
Copy link
Copy Markdown
Member

  • Delete the standalone pn532uart package which had a permanent serial port blocklist (serialBlockList) that never cleared entries, causing one out of N PN532 readers to intermittently fail detection on Windows
  • Remove the legacy driver from Windows SupportedReaders — the modern pn532 driver already handles UART detection with proper ModTime-based cache eviction
  • The modern pn532 driver already accepts pn532uart as a valid ID, so existing user configs continue to work

Closes #602

Delete the standalone pn532uart package and remove it from Windows
SupportedReaders. The legacy driver had a permanent serial port
blocklist that never cleared entries, causing intermittent detection
failures when multiple PN532 readers were connected (issue #602).

The modern pn532 driver handles UART detection with proper ModTime-based
cache eviction and already accepts the legacy driver IDs for backwards
compatibility.

Closes #602
@sentry
Copy link
Copy Markdown

sentry bot commented Apr 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@wizzomafizzo wizzomafizzo merged commit f825ffd into main Apr 3, 2026
11 checks passed
@wizzomafizzo wizzomafizzo deleted the fix/remove-legacy-pn532uart branch April 3, 2026 10:19
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.

Intermittent detection issue with one out of seven drives connected

1 participant