Skip to content

fix: bump trezor-connect-rs to 0.2.5 (USB I/O serialization)#84

Merged
ben-kaufman merged 2 commits intomasterfrom
fix/trezor-7-usb
Apr 1, 2026
Merged

fix: bump trezor-connect-rs to 0.2.5 (USB I/O serialization)#84
ben-kaufman merged 2 commits intomasterfrom
fix/trezor-7-usb

Conversation

@ovitrif
Copy link
Copy Markdown
Collaborator

@ovitrif ovitrif commented Mar 31, 2026

Summary

  • Bumps trezor-connect-rs from 0.2.4 to 0.2.5 (upstream PR)
  • Fixes concurrent USB I/O calls interleaving and corrupting the wire protocol, causing protobuf decode errors ("invalid tag value: 0")
  • Bumps bitkitcore version 0.1.54 → 0.1.55 with regenerated bindings for all platforms (iOS, Android, Python)
  • Release: https://github.com/synonymdev/bitkit-core/releases/tag/v0.1.55

The upstream change adds a tokio::sync::Mutex per device path to serialize the full write/read sequence. Verified that there are no No API changes that we need to integrate after the update of the dependency.

Test plan

  • cargo test modules::trezor — all 42 tests pass
  • All platform bindings build successfully (iOS, Android, Python)
  • Release v0.1.55 created
  • Verify Trezor Safe 7 USB connection in synonymdev/bitkit-android##792 – @ovitrif
    • ovi: Unfortunately this isn't the only fix needed

🤖 Generated with Claude Code

Fixes concurrent USB calls interleaving and corrupting the wire protocol,
which caused protobuf decode errors ("invalid tag value: 0"). The upstream
fix adds per-device tokio::sync::Mutex to serialize write/read sequences.

Bumps bitkitcore version 0.1.54 → 0.1.55 and regenerates all bindings.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@claude

This comment has been minimized.

@ovitrif ovitrif self-assigned this Mar 31, 2026
@claude
Copy link
Copy Markdown

claude bot commented Mar 31, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@ben-kaufman ben-kaufman merged commit d3eb151 into master Apr 1, 2026
2 checks passed
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