Skip to content

Add safe Solana stale-blockhash re-sign retries - #33

Merged
VickyXAI merged 1 commit into
BlockRunAI:mainfrom
KillerQueen-Z:fix/solana-safe-resign-retry
Aug 26, 2026
Merged

Add safe Solana stale-blockhash re-sign retries#33
VickyXAI merged 1 commit into
BlockRunAI:mainfrom
KillerQueen-Z:fix/solana-safe-resign-retry

Conversation

@KillerQueen-Z

Copy link
Copy Markdown
Contributor

Summary

  • add bounded fresh-quote retries for explicit verification-phase stale blockhash failures
  • keep settlement and phase-ambiguous 402 failures terminal to avoid double charge
  • cover chat, raw POST, and raw GET payment paths
  • bound payment error bodies to 64 KiB before parsing

Safety

  • retries require both a verification marker and an explicit stale signal
  • settlement failures, malformed bodies, insufficient funds, and bare simulation failures do not retry
  • at most two re-sign attempts are made with bounded backoff

Validation

  • 261 tests passed, 12 skipped across the sandbox-safe unit suite
  • isolated Solana concurrency suite: 3 passed
  • stale-blockhash safety tests: 10 passed
  • typecheck, lint, and build passed
  • cache/cost-log tests were not run because they intentionally mutate the user home ledger

VickyXAI pushed a commit that referenced this pull request Aug 26, 2026
Lands #33 (KillerQueen-Z) plus the fix that makes it functional: the re-sign
was reusing the same cached, already-expired blockhash.

Depends on BlockRunAI/blockrun@7e26ea1 for the PAYMENT_BLOCKHASH_STALE code.
@VickyXAI
VickyXAI merged commit 78dbe63 into BlockRunAI:main Aug 26, 2026
3 checks passed
VickyXAI pushed a commit that referenced this pull request Aug 26, 2026
Lands #33 plus the fix that makes it functional. Requires
BlockRunAI/blockrun@7e26ea1 for the PAYMENT_BLOCKHASH_STALE code; that
gateway change is already deployed on main.

Verified: tsc, lint, 266 tests across 26 files, build clean, zero
router-core leakage, version-consistency suite green across VERSION,
package.json, src/version.ts and CHANGELOG.md.
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.

2 participants