Skip to content

Fix beta CI and test shared BTC vectors - #128

Merged
benma merged 2 commits into
BitBoxSwiss:masterfrom
benma-agent:benma-agent/shared-btc-test-vectors
Aug 23, 2026
Merged

Fix beta CI and test shared BTC vectors#128
benma merged 2 commits into
BitBoxSwiss:masterfrom
benma-agent:benma-agent/shared-btc-test-vectors

Conversation

@benma-agent

@benma-agent benma-agent commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • update async-trait to 0.1.92 so Beta Clippy no longer sees generated redundant #[must_use] attributes
  • consume the canonical Bitcoin transaction vectors generated by bitbox02-firmware
  • validate PSBT signing screens, errors, signatures, and final transactions
  • support device-owned outputs from another account on firmware 9.22 and later

The fixture is byte-for-byte identical to firmware pull/master at a3467ad11daee7081cc041ba9217c81e471022e6 (SHA-256 5baf824c2f7d39ce433bef04fbad655fe415c71933b0db7ca12c8993c400d8b7).

Testing

  • cargo +stable test --tests --locked --features=simulator,tokio -- --nocapture --test-threads 1
  • cargo +stable clippy --tests --locked --features=simulator,tokio -- -D warnings -A clippy::empty-docs
  • cargo +beta test --tests --locked --features=simulator,tokio -- --nocapture --test-threads 1
  • cargo +beta clippy --tests --locked --features=simulator,tokio -- -D warnings -A clippy::empty-docs
  • SIMULATOR=tests/simulators/bitbox02-multi-v9.26.5-simulator1.0.0-linux-amd64 cargo test --test test_btc_psbt --locked --features=simulator,tokio test_btc_transaction_vectors -- --nocapture --test-threads 1

@benma-agent
benma-agent force-pushed the benma-agent/shared-btc-test-vectors branch 3 times, most recently from 674f90e to 387fa40 Compare July 21, 2026 20:53
@benma
benma requested a review from cedwies July 30, 2026 20:58
@benma
benma marked this pull request as ready for review July 30, 2026 20:58

@cedwies cedwies 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.

utACK

@benma-agent
benma-agent force-pushed the benma-agent/shared-btc-test-vectors branch 3 times, most recently from 3c20ed7 to ecf368b Compare August 23, 2026 18:31
@benma-agent benma-agent changed the title Test Bitcoin signing with shared vectors Fix beta CI and test shared BTC vectors Aug 23, 2026
Update async-trait to 0.1.92, which stops generating redundant #[must_use]
attributes on async trait methods. This keeps Beta Clippy clean without downstream
lint allowances.
Consume the PSBT transaction vectors introduced by
BitBoxSwiss/bitbox02-firmware#2017 in
simulator signing tests. The public PSBT API exercises conversion and
the low-level signing protocol without a second raw fixture.

Share stdout parsing and check versioned screens, errors, signature
insertion, and final transactions.

Teach PSBT signing to use output_script_configs for device-owned outputs
from another account on firmware 9.22 and later, while preserving the
older external-output behavior.
@benma-agent
benma-agent force-pushed the benma-agent/shared-btc-test-vectors branch from ecf368b to d865ab5 Compare August 23, 2026 18:43
@benma
benma merged commit 91e8212 into BitBoxSwiss:master Aug 23, 2026
3 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.

3 participants