@across-protocol/sdk 4.3.3 → 4.3.5

package/src/relayFeeCalculator/chain-queries/svmQuery.ts

@@ -1,33 +1,11 @@
 import assert from "assert";
-import { SvmSpokeClient } from "@across-protocol/contracts";
-import { intToU8Array32 } from "@across-protocol/contracts/dist/src/svm/web3-v1/conversionUtils";
-import { SYSTEM_PROGRAM_ADDRESS } from "@solana-program/system";
-import { ASSOCIATED_TOKEN_PROGRAM_ADDRESS, fetchMint } from "@solana-program/token";
-import { getComputeUnitEstimateForTransactionMessageFactory } from "@solana/kit";
-import {
-  SVMProvider,
-  SolanaVoidSigner,
-  createFillInstruction,
-  getAssociatedTokenAddress,
-  getEventAuthority,
-  getFillRelayDelegatePda,
-  getFillStatusPda,
-  getStatePda,
-  toAddress,
-} from "../../arch/svm";
+import { getComputeUnitEstimateForTransactionMessageFactory, TransactionSigner } from "@solana/kit";
+import { SVMProvider, SolanaVoidSigner, getFillRelayTx } from "../../arch/svm";
 import { Coingecko } from "../../coingecko";
 import { CHAIN_IDs } from "../../constants";
 import { getGasPriceEstimate } from "../../gasPriceOracle";
 import { RelayData } from "../../interfaces";
-import {
-  Address,
-  BigNumber,
-  BigNumberish,
-  SvmAddress,
-  TransactionCostEstimate,
-  getRelayDataHash,
-  toBN,
-} from "../../utils";
+import { Address, BigNumber, BigNumberish, SvmAddress, TransactionCostEstimate, toBN } from "../../utils";
 import { Logger, QueryInterface, getDefaultRelayer } from "../relayFeeCalculator";
 import { SymbolMappingType } from "./";
 
@@ -93,7 +71,7 @@ export class SvmQuery implements QueryInterface {
     const [repaymentChainId, repaymentAddress] = [destinationChainId, relayer]; // These are not important for gas cost simulation.
     const fillRelayTx = await this.getFillRelayTx(
       { ...relayData, recipient, outputToken, exclusiveRelayer },
-      relayer,
+      SolanaVoidSigner(relayer.toBase58()),
       repaymentChainId,
       repaymentAddress
     );
@@ -139,7 +117,7 @@ export class SvmQuery implements QueryInterface {
     const [repaymentChainId, repaymentAddress] = [destinationChainId, relayer]; // These are not important for gas cost simulation.
     const fillRelayTx = await this.getFillRelayTx(
       { ...deposit, recipient, outputToken, exclusiveRelayer },
-      relayer,
+      SolanaVoidSigner(relayer.toBase58()),
       repaymentChainId,
       repaymentAddress
     );
@@ -158,75 +136,11 @@ export class SvmQuery implements QueryInterface {
       recipient: SvmAddress;
       outputToken: SvmAddress;
     },
-    relayer: SvmAddress,
+    signer: TransactionSigner,
     repaymentChainId: number,
     repaymentAddress: Address
   ) {
-    const { depositor, recipient, inputToken, outputToken, exclusiveRelayer, destinationChainId } = relayData;
-
-    // tsc appeasement...should be unnecessary, but isn't. @todo Identify why.
-    assert(recipient.isSVM(), `getFillRelayTx: recipient not an SVM address (${recipient})`);
-    assert(
-      repaymentAddress.isValidOn(repaymentChainId),
-      `getFillRelayTx: repayment address ${repaymentAddress} not valid on chain ${repaymentChainId})`
-    );
-
-    const program = toAddress(this.spokePool);
-    const _relayDataHash = getRelayDataHash(relayData, destinationChainId);
-    const relayDataHash = new Uint8Array(Buffer.from(_relayDataHash.slice(2), "hex"));
-
-    const [state, delegate] = await Promise.all([
-      getStatePda(program),
-      getFillRelayDelegatePda(relayDataHash, BigInt(repaymentChainId), toAddress(repaymentAddress), program),
-    ]);
-
-    const mint = toAddress(outputToken);
-    const mintInfo = await fetchMint(this.provider, mint);
-
-    const [recipientAta, relayerAta, fillStatus, eventAuthority] = await Promise.all([
-      getAssociatedTokenAddress(recipient, outputToken, mintInfo.programAddress),
-      getAssociatedTokenAddress(SvmAddress.from(relayer.toBase58()), outputToken, mintInfo.programAddress),
-      getFillStatusPda(program, relayData, destinationChainId),
-      getEventAuthority(program),
-    ]);
-
-    const svmRelayData: SvmSpokeClient.FillRelayInput["relayData"] = {
-      depositor: toAddress(depositor),
-      recipient: toAddress(recipient),
-      exclusiveRelayer: toAddress(exclusiveRelayer),
-      inputToken: toAddress(inputToken),
-      outputToken: mint,
-      inputAmount: relayData.inputAmount.toBigInt(),
-      outputAmount: relayData.outputAmount.toBigInt(),
-      originChainId: relayData.originChainId,
-      depositId: new Uint8Array(intToU8Array32(relayData.depositId.toNumber())),
-      fillDeadline: relayData.fillDeadline,
-      exclusivityDeadline: relayData.exclusivityDeadline,
-      message: new Uint8Array(Buffer.from(relayData.message, "hex")),
-    };
-
-    const simulatedSigner = SolanaVoidSigner(relayer.toBase58());
-    const fillInput: SvmSpokeClient.FillRelayInput = {
-      signer: simulatedSigner,
-      state,
-      delegate,
-      mint,
-      relayerTokenAccount: relayerAta,
-      recipientTokenAccount: recipientAta,
-      fillStatus,
-      tokenProgram: mintInfo.programAddress,
-      associatedTokenProgram: ASSOCIATED_TOKEN_PROGRAM_ADDRESS,
-      systemProgram: SYSTEM_PROGRAM_ADDRESS,
-      eventAuthority,
-      program,
-      relayHash: relayDataHash,
-      relayData: svmRelayData,
-      repaymentChainId: BigInt(repaymentChainId),
-      repaymentAddress: toAddress(repaymentAddress),
-    };
-    // Pass createRecipientAtaIfNeeded =true to the createFillInstruction function to create the recipient token account
-    // if it doesn't exist.
-    return createFillInstruction(simulatedSigner, this.provider, fillInput, mintInfo.data.decimals, true);
+    return await getFillRelayTx(this.spokePool, this.provider, relayData, signer, repaymentChainId, repaymentAddress);
   }
 
   /**

package/src/utils/AddressUtils.ts

@@ -90,6 +90,7 @@ export function toAddressType(address: string, chainId: number): Address {
 // The Address class can contain any address type. It is up to the subclasses to determine how to format the address's internal representation,
 // which for this class, is a bytes32 hex string.
 export abstract class Address {
+  readonly __address_type_brand = true;
   readonly rawAddress: Uint8Array;
 
   // Keep all address types in cache so that we may lazily evaluate them when necessary.
@@ -109,6 +110,10 @@ export abstract class Address {
     this.rawAddress = utils.zeroPad(_rawAddress, 32);
   }
 
+  static isAddress(obj: unknown): obj is Address {
+    return "__address_type_brand" in (obj as { __address_type_brand: boolean });
+  }
+
   // Converts the address into a bytes32 string. Note that the output bytes will be lowercase so that it matches ethers event data. This function will never
   // throw since address length validation was done at construction time.
   toBytes32(): string {

package/src/utils/DepositUtils.ts

@@ -1,7 +1,16 @@
 import assert from "assert";
 import { SpokePoolClient } from "../clients";
 import { DEFAULT_CACHING_TTL, EMPTY_MESSAGE, UNDEFINED_MESSAGE_HASH, ZERO_BYTES } from "../constants";
-import { CachingMechanismInterface, Deposit, DepositWithBlock, Fill, RelayData, SlowFillRequest } from "../interfaces";
+import {
+  CachingMechanismInterface,
+  Deposit,
+  DepositWithBlock,
+  Fill,
+  RelayData,
+  SlowFillRequest,
+  ConvertedRelayData,
+  ConvertedFill,
+} from "../interfaces";
 import { getMessageHash, isUnsafeDepositId } from "./SpokeUtils";
 import { getNetworkName } from "./NetworkUtils";
 import { bnZero } from "./BigNumberUtils";
@@ -251,3 +260,42 @@ export function resolveDepositMessage(deposit: Deposit): string {
   assert(isDefined(message)); // Appease tsc about the updatedMessage being possibly undefined.
   return message;
 }
+
+/**
+ * Converts a RelayData object with `Address` types as address fields to a `RelayData`-like object with
+ * strings as address fields.
+ * @param relayData RelayData type.
+ * @returns a RelayData-like type which has strings as fields.
+ */
+export function convertRelayDataParamsToBytes32(relayData: RelayData): ConvertedRelayData {
+  return {
+    ...relayData,
+    depositor: relayData.depositor.toBytes32(),
+    recipient: relayData.recipient.toBytes32(),
+    inputToken: relayData.inputToken.toBytes32(),
+    outputToken: relayData.outputToken.toBytes32(),
+    exclusiveRelayer: relayData.exclusiveRelayer.toBytes32(),
+  };
+}
+
+/**
+ * Converts a Fill object with `Address` types as address fields to a `RelayData`-like object with
+ * strings as address fields.
+ * @param relayData RelayData type.
+ * @returns a RelayData-like type which has strings as fields.
+ */
+export function convertFillParamsToBytes32(fill: Fill): ConvertedFill {
+  return {
+    ...fill,
+    depositor: fill.depositor.toBytes32(),
+    recipient: fill.recipient.toBytes32(),
+    inputToken: fill.inputToken.toBytes32(),
+    outputToken: fill.outputToken.toBytes32(),
+    exclusiveRelayer: fill.exclusiveRelayer.toBytes32(),
+    relayer: fill.relayer.toBytes32(),
+    relayExecutionInfo: {
+      ...fill.relayExecutionInfo,
+      updatedRecipient: fill.relayExecutionInfo.updatedRecipient.toBytes32(),
+    },
+  };
+}