Skip to content

Fix Android reader scan edge cases - #54

Draft
djwok wants to merge 1 commit into
mainfrom
codex/fix-android-reader-scan-edge-cases
Draft

Fix Android reader scan edge cases#54
djwok wants to merge 1 commit into
mainfrom
codex/fix-android-reader-scan-edge-cases

Conversation

@djwok

@djwok djwok commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Use the Bluetooth address when an Android scan result has no advertised or cached Reader name.
  • Reject the active scan when ACS Reader creation fails instead of silently dropping the result.

Root cause

The single filtered BLE scan path returned early for nameless results and swallowed ACS factory exceptions, leaving callers with missing Readers and no actionable failure.

Validation

  • pnpm lint
  • pnpm build
  • pnpm test -- --runInBand --no-watchman

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.

1 participant