Skip to content

Record paired device last connection time - #644

Merged
enaboapps merged 2 commits into
mainfrom
codex/643-record-device-last-connection-time
Aug 22, 2026
Merged

Record paired device last connection time#644
enaboapps merged 2 commits into
mainfrom
codex/643-record-device-last-connection-time

Conversation

@enaboapps

@enaboapps enaboapps commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Closes #643

What changed

  • turn every authenticated connection.ping into a connection event with the validated device ID and desktop timestamp
  • save lastSeenAt and an optional Remote name in one transaction before publishing either value in memory
  • reject out-of-order connection events so stale Windows tasks cannot replace newer metadata
  • handle the event through the Windows and macOS runtimes
  • show "Last connected" in the Devices view

Raw Bluetooth subscriptions and failed authentication do not update device metadata. Existing ping acknowledgements and the name_update_failed response remain compatible.

Validation

  • Node.js 24.13.0
  • Rust 1.97.1
  • npm run lint
  • npm test (74 tests)
  • npm run build
  • cargo fmt --manifest-path src-tauri/Cargo.toml --check
  • cargo clippy --manifest-path src-tauri/Cargo.toml --all-targets -- -D warnings
  • cargo test --manifest-path src-tauri/Cargo.toml (211 unit tests, 2 config tests)

@enaboapps

Copy link
Copy Markdown
Contributor Author

Independent exact-head review completed for 3852b50a74dc9c36791d87db3577b9752dbcb73f.

The reviewer confirmed the earlier out-of-order Windows persistence finding is fixed and reported no actionable findings on the updated head. Frontend, macOS app/DMG, and Windows NSIS CI all pass.

@enaboapps
enaboapps marked this pull request as ready for review August 22, 2026 10:36
@enaboapps
enaboapps merged commit 298d23b into main Aug 22, 2026
3 checks passed
@enaboapps
enaboapps deleted the codex/643-record-device-last-connection-time branch August 22, 2026 10:37
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.

Record paired device last connection time

2 participants