[sdk]: Add IntentsV2 #264
[sdk]: Add IntentsV2 #264
Conversation
royvardhan
force-pushed
the
roy/sdk-igv2-place-order
branch
from
8a40e99 to
a988a99
Compare
|
Since filler will be heavily dependent on this class. I will add cache service here when I work on the filler. |
| if (intentGatewayV2Address) { | ||
| const paramsSlot5 = pad(toHex(5n), { size: 32 }) as HexString | ||
| const currentSlot5Value = await this.dest.client.getStorageAt({ | ||
| address: intentGatewayV2Address, | ||
| slot: paramsSlot5, | ||
| }) | ||
| // Set solverSelection to 0x00 while preserving dispatcher | ||
| const currentValue = currentSlot5Value || "0x" + "0".repeat(64) | ||
| const newSlot5Value = (currentValue.slice(0, 24) + "00" + currentValue.slice(26)) as HexString | ||
| overrides.push({ | ||
| address: intentGatewayV2Address, | ||
| stateDiff: [{ slot: paramsSlot5, value: newSlot5Value }], | ||
| }) | ||
| } |
There was a problem hiding this comment.
I'm not sure why we need to query this at all before overriding it.
There was a problem hiding this comment.
Because this slot contains the dispatcher and the solver selection packed.
If we are not changing the dispatcher address then we can have the dispatcher stored locally and can override without querying first.
| // ========================================================================= | ||
|
|
||
| /** Builds state overrides for token balances and allowances to enable gas estimation */ | ||
| private async buildTokenStateOverrides( |
There was a problem hiding this comment.
this method should not be async
There was a problem hiding this comment.
Ohh yeah, missed this. It was using tracer before, now using config values.
| let protocolFeeInNativeToken = await this.quoteNative(postRequest, postRequestFeeInDestFeeToken).catch(() => | ||
| this.dest.quoteNative(postRequest, postRequestFeeInDestFeeToken).catch(() => 0n), | ||
| ) |
There was a problem hiding this comment.
A bit confusing as to why we don't just use the dest.quoteNative exclusively?
There was a problem hiding this comment.
Because direct contract call for IntentGateway.quoteNative() is just one rpc call. This will fail for some chains where we have the wrapper.
While EvmChain.quoteNative() is three rpc calls:
host.perByteFee()
host.feeToken()
uniswap.getAmountsIn()
Related to #263
Adds
IntentGatewayV2class to the SDK for interacting with the V2 intent gateway contract.preparePlaceOrder- Generates session key, stores it, and returns placeOrder calldataestimateFillOrderV2- Estimates gas used for filling an order, includes smart account verification costs.prepareSubmitBid- Constructs UserOperation for fillers to submit bids to HyperbridgesignSolverSelection- Signs EIP-712 solver selection with stored session key