Skip to content

feat: expose trezor lock state#104

Merged
ovitrif merged 2 commits into
masterfrom
feat/trezor-lock-state
Jun 22, 2026
Merged

feat: expose trezor lock state#104
ovitrif merged 2 commits into
masterfrom
feat/trezor-lock-state

Conversation

@ovitrif

@ovitrif ovitrif commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Related: synonymdev/bitkit-android#1030, synonymdev/bitkit-android#998

Description

  • Expose Trezor lock state via TrezorFeatures.unlocked so mobile clients can derive pinProtection == true && unlocked == false for locked devices.
  • Add trezor_refresh_features() as an explicit one-shot feature refresh that updates cached features without adding polling.
  • Surface busy transport state as TrezorError::DeviceBusy so clients can back off while the device is busy or awaiting user action.
  • Add TrezorTransportErrorCode.DeviceBusy to native callback results so mobile transports can report busy open/read/write/message calls without relying on generic error strings.
  • Release: v0.3.2

QA Notes

  • cargo fmt --check
  • cargo test modules::trezor
  • cargo clippy
  • ./build.sh -r --patch all
  • ./build.sh all
  • Published release: v0.3.2
  • Release assets uploaded: BitkitCore.xcframework.zip, native-debug-symbols.zip

@ovitrif ovitrif changed the title Expose Trezor lock state feat: expose trezor lock state Jun 19, 2026
@ovitrif ovitrif self-assigned this Jun 19, 2026
Comment thread src/modules/trezor/errors.rs

@ben-kaufman ben-kaufman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reaudited latest head d95a061. The callback DeviceBusy issue is fixed and checks pass.

@ovitrif ovitrif merged commit ddd32bd into master Jun 22, 2026
1 check passed
@ovitrif ovitrif deleted the feat/trezor-lock-state branch June 22, 2026 08:53
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.

2 participants