Skip to content

Preflight aggregate mixed-input signing size - #12

Merged
rohenaz merged 1 commit into
mainfrom
codex/pushdrop-action-aggregate-preflight
Aug 31, 2026
Merged

Preflight aggregate mixed-input signing size#12
rohenaz merged 1 commit into
mainfrom
codex/pushdrop-action-aggregate-preflight

Conversation

@rohenaz

@rohenaz rohenaz commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • project every planned unlocking script at the pinned SDK safe maximum before any input-key derivation or wallet signature request
  • use TransactionInput pay-to-public-key-hash and PushDrop maximum unlocking-script constants rather than storage estimates
  • validate the completed projected transaction with serializedByteCount and caller limits
  • resolve all ordinary BRC-29 input keys before invoking any permission-token signer

Regression

A two-token action constrained to the maximum size of only one projected token now fails before either signer runs.

Tests

  • swift test --filter BRC116ActionSignerTests — 12 passed
  • swift test — 221 executed, 10 expected live-environment skips, 0 failures

Follow-up to PR #10. This PR is intentionally limited to ActionSigner and its focused tests.

@rohenaz
rohenaz force-pushed the codex/pushdrop-action-aggregate-preflight branch from 81c4900 to ab17e64 Compare August 31, 2026 15:52
@rohenaz
rohenaz merged commit 1927322 into main Aug 31, 2026
4 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.

1 participant