Skip to content

feat: add an isolated deterministic trezor emulator fixture - #16

Merged
ovitrif merged 4 commits into
mainfrom
feat/trezor-emulator-fixture
Oct 4, 2026
Merged

ovitrif merged 4 commits into
mainfrom
feat/trezor-emulator-fixture

Conversation

@ovitrif

@ovitrif ovitrif commented Oct 3, 2026

Copy link
Copy Markdown
Collaborator

Description

  • Adds an optional Trezor emulator and Bridge profile with automatic deterministic setup, isolated state and a healthcheck. Includes an offline diagnostic that verifies the public test identity and regtest transaction signing through Bridge.
  • Checked: Built the named ARM64 image and started a temporary Compose project from the reported commit. It became healthy, enumerated a device, returned the expected regtest address and signed a synthetic SegWit transaction whose signature was independently verified; the diagnostic released its Bridge session. The lane check resolved the on-demand profile and refreshed capabilities showed the missing piece at the reported revision.

Out of Scope

  • None.

@ovi-reviewer ovi-reviewer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggestion: ♻️ Comment

The test run was for synonymdev/bitkit-ios#867 at head 16f0966, with the environment at commit a07496f. It ran one item, J1, and J1 failed.

The record of that failure does not say why it failed, so the run does not show whether the emulator change works or not, and it gives no ground to blame the change. Nothing else ran, so no item has yet passed on this change.

@ovi-reviewer ovi-reviewer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggestion: 👍 Approve

The test run for synonymdev/bitkit-ios#867 at head e0e44a4 used this environment change at commit a07496f. The one item that needed the Trezor emulator fixture, J1, ran on it and passed.

Every item of the run that needed the change ran and passed, so the emulator and Bridge setup worked as intended.

@ovitrif
ovitrif marked this pull request as ready for review October 4, 2026 12:03
Base automatically changed from feat/lnurl-pay-callback-fixture to main October 4, 2026 20:11
@ovitrif
ovitrif merged commit e1027e2 into main Oct 4, 2026
@ovitrif
ovitrif deleted the feat/trezor-emulator-fixture branch October 4, 2026 20:12
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