Skip to content

test(sol): SIMD-525 devnet verification scripts - #9396

Open
bhavidhingra wants to merge 1 commit into
masterfrom
chore/simd525-verification-scripts
Open

test(sol): SIMD-525 devnet verification scripts#9396
bhavidhingra wants to merge 1 commit into
masterfrom
chore/simd525-verification-scripts

Conversation

@bhavidhingra

Copy link
Copy Markdown
Contributor

What

Verification scripts for CHALO-1077 (Solana 200ms slot / SIMD-525) analysis.

Scripts

  • simd525-hot-wallet-presign.ts — Hot wallet presign rebuild test. Prebuilds tx (captures blockhash B1), waits 120s, signs (triggers presignTransaction rebuild → fresh blockhash B2), verifies B1≠B2, broadcasts successfully. Proves hot wallets are NOT affected by SIMD-525.

  • simd525-durable-nonce-check.ts — Custodial/cold wallet durable nonce verification. Prebuilds tx, deserializes txHex, finds AdvanceNonceAccount instruction (System program, ix index 4), extracts nonceAccount + nonceAuthority. Proves custodial/cold wallets use durable nonce → bypass blockhash expiry.

  • simd525-recovery-warning.ts — Recovery warning test for PR feat(sol): warn when recovery is called without durable nonce (SIMD-525) #9372. Calls recover() with and without durableNonce, captures console.warn to verify warning fires only when durableNonce is missing.

Test Results

  • Hot wallet (120s hold): B1≠B2 ✅, broadcast succeeded ✅
  • Custodial wallet: AdvanceNonceAccount ix found ✅, nonce account + authority extracted ✅
  • Recovery warning: ready to run on fix branch

References: CSHLD-000

@bhavidhingra
bhavidhingra requested a review from a team as a code owner July 31, 2026 09:37
- Hot wallet presign rebuild test (120s hold, blockhash B1≠B2)
- Custodial wallets durable nonce check (AdvanceNonceAccount ix)
- Recovery warning test for PR #9372 (logger.warn on missing durableNonce)

References: CSHLD-000
@bhavidhingra
bhavidhingra force-pushed the chore/simd525-verification-scripts branch from ad886e1 to 52be259 Compare July 31, 2026 09:39
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