svm: bump default compute unit limit to 8_000

[notorious-d-e-v]

Description

Bump the compute unit limit from 6500 to 8000.

It seems like the previous limit of 6500 might've been too strict and doesn't match the currently observed compute units on mainnet (6592).

This PR bumps it to 8000 to keep some buffer, since we are no longer estimating the compute unit usage ahead of time.

For context: the reason we are not estimating the compute unit usage ahead of time (in the client or facilitator) is that it saves an RPC call on each, making the user experience faster. Since the transaction is quite simple (simple TransferChecked), a static limit vs 2 RPC calls seems like the right tradeoff.

This may change again in the future after discussion on #828

Tests

All tests pass as before.

Checklist

• I have formatted and linted my code
• All new and existing tests pass
• My commits are signed (required for merge) -- you may need to rebase if you initially pushed unsigned commits

[cb-heimdall]

🟡 Heimdall Review Status

Requirement	Status	More Info
Reviews	🟡 0/1	Denominator calculation Show calculation 1 if user is bot 0 1 if user is external 0 2 if repo is sensitive 0 From .codeflow.yml 1 Additional review requirements Show calculation Max 0 0 From CODEOWNERS 0 Global minimum 0 Max 1 1 1 if commit is unverified 0 Sum 1

[vercel]

@notorious-d-e-v is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.

[phdargen]

Thanks a lot @notorious-d-e-v, looks good to me!

Do you mind also updating this for the go implementation to keep them in sync?

x402/go/mechanisms/svm/constants.go

Line 24 in f14d875

DefaultComputeUnitLimit uint32 = 6500