Skip to content

feat: use wasm-dot parser in verifyTransaction for tdot#8322

Merged
lcovar merged 1 commit intomasterfrom
BTC-3161/dot-wasm-verify-transaction
Mar 20, 2026
Merged

feat: use wasm-dot parser in verifyTransaction for tdot#8322
lcovar merged 1 commit intomasterfrom
BTC-3161/dot-wasm-verify-transaction

Conversation

@lcovar
Copy link
Contributor

@lcovar lcovar commented Mar 19, 2026

Both wasm-dot and legacy now produce signing payload format for unsigned
transactions, so the format detection hack (compactByte, versionByte,
isStandardFormat) is no longer needed. Use chain === 'tdot' as the gate
for the wasm path instead.

Update WASM test fixtures from old standard extrinsic format to signing
payload format to match the new wasm-dot output.

BTC-3161

TICKET: BTC-3161

@lcovar lcovar force-pushed the BTC-3161/dot-wasm-verify-transaction branch 4 times, most recently from 2abc269 to 5f26ae1 Compare March 19, 2026 18:29
@lcovar lcovar marked this pull request as ready for review March 19, 2026 19:22
@lcovar lcovar requested review from a team as code owners March 19, 2026 19:22
@lcovar lcovar marked this pull request as draft March 19, 2026 19:39
@lcovar lcovar force-pushed the BTC-3161/dot-wasm-verify-transaction branch 3 times, most recently from 8f6a2bf to 03d9808 Compare March 19, 2026 21:44
Both wasm-dot and legacy now produce signing payload format for unsigned
transactions, so the format detection hack (compactByte, versionByte,
isStandardFormat) is no longer needed. Use chain === 'tdot' as the gate
for the wasm path instead.

Update WASM test fixtures from old standard extrinsic format to signing
payload format to match the new wasm-dot output.

BTC-3161

TICKET: BTC-3161
@lcovar lcovar force-pushed the BTC-3161/dot-wasm-verify-transaction branch from 03d9808 to 6060d22 Compare March 19, 2026 22:11
@lcovar lcovar marked this pull request as ready for review March 19, 2026 22:35
@lcovar lcovar merged commit e9ede8a into master Mar 20, 2026
21 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