Skip to content

feat: defer tempo expiring-nonce signing until submission - #193

Merged
onbjerg merged 3 commits into
mainfrom
onbjerg/tempo-late-signing
Sep 11, 2026
Merged

onbjerg merged 3 commits into
mainfrom
onbjerg/tempo-late-signing

Conversation

@onbjerg

@onbjerg onbjerg commented Aug 28, 2026

Copy link
Copy Markdown
Member

Summary

  • add an opaque deferred-signing envelope and generic bench signer boundary
  • defer relative Tempo expiring-nonce signing until the final submission boundary
  • add optional --late-signing-spec support for NDJSON files and stdin pipes

Compatibility

Existing signed NDJSON and existing CLI invocations remain unchanged. Absolute valid_before records retain their pre-signed behavior. Deferred relative-expiry records are supported for standard and sponsored Tempo transactions; keychain-auth records retain generation-time signing.

Validation

  • cargo test --workspace
  • Rebased onto d339dba; local workspace tests: 400 passed, 1 ignored. PR CI is green.
  • Multi-region smoke run pins rebased txgen 39e130c, enables --defer-signing and --late-signing-spec, and uses the public-mix preparation from runner PR #401 on test branch. One pair, 30 seconds, 10 validators in us-east-1, 10 accounts, 100 target TPS, 1 GiB bloat: baseline 751/751 and feature 767/767 RPC-accepted, zero failures. The previous run had 73/775 and 72/765 expiry rejections. All 45 setup transactions still completed in approximately 22.5 seconds; no setup parallelization or expiry-window increase was used. This validates the submission-expiry fix for this smoke configuration, not high-load throughput or receipt-level workload success.

@onbjerg
onbjerg force-pushed the onbjerg/tempo-late-signing branch from 26f0050 to 974c431 Compare August 31, 2026 11:09
@onbjerg onbjerg changed the title Defer Tempo expiring-nonce signing until submission feat: defer tempo expiring-nonce signing until submission Aug 31, 2026
@decofe
decofe force-pushed the onbjerg/tempo-late-signing branch from 974c431 to 39e130c Compare September 10, 2026 17:33
@onbjerg
onbjerg marked this pull request as ready for review September 11, 2026 09:46
@onbjerg
onbjerg merged commit 433a219 into main Sep 11, 2026
8 checks passed
@onbjerg
onbjerg deleted the onbjerg/tempo-late-signing branch September 11, 2026 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant