Skip to content

feat(zcash): Orchard shielded signing engine - #346

Closed
BitHighlander wants to merge 1 commit into
feature/715-04-clearsign-metadatafrom
feature/715-05-zcash-orchard-engine
Closed

feat(zcash): Orchard shielded signing engine#346
BitHighlander wants to merge 1 commit into
feature/715-04-clearsign-metadatafrom
feature/715-05-zcash-orchard-engine

Conversation

@BitHighlander

Copy link
Copy Markdown
Owner

7.15 review stack position 05/17.

This PR is intentionally one coherent commit, based on feature/715-04-clearsign-metadata, for individual auditing.
The full stack is linear from the clean fork develop at 1af2ffe.

Review scope: feat(zcash): Orchard shielded signing engine.

The preserved pre-restage candidate remains available at preserve/alpha-2026-08-05.

The device-side Orchard engine behind Zcash shielded sends: Orchard key
derivation from the wallet seed, note/action handling, RedPallas signing over
the Pallas curve, and unified-address encoding (ZIP-316).

- Signing is constant-time on the secret scalar path; the Pallas arithmetic
  comes from the pinned trezor-firmware fork and is compiled in only when the
  privacy engine is enabled, so the bitcoin-only image carries none of it.
- Progress is reported to the host during action signing — multi-action
  transactions take long enough that a silent device looks hung.
- messages-zcash proto/options are registered with the transport generator.

Handlers, host-visible messages, unit tests, the constant-time CI gates and
the coin-integration docs land in the Zcash wiring branch; this commit adds the
engine and its build wiring only.
@BitHighlander

Copy link
Copy Markdown
Owner Author

Superseded by the smaller audit-ready 7.15 stack: #359 release foundation, #360 authenticated clear-signing, #361 recovery and entropy, #362 Zcash Orchard, #363 chain integrations, and #364 Bitcoin/release security. All release changes remain preserved in the replacement branches and preservation tags. The replacement aggregate was tree-verified against the prior release and passed the local CI-equivalent container suite (405 firmware, 2 board, 18 crypto, and 7 Pallas constant-time tests). GitHub-hosted checks have been explicitly dispatched and are currently queued.

@BitHighlander
BitHighlander deleted the feature/715-05-zcash-orchard-engine branch August 9, 2026 04:26
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