Correct the type of getComputeUnitEstimateForTransactionMessage (#2905

)
solana-labs · Jul 3, 2024 · 56fde06 · 56fde06
1 parent 798651e
commit 56fde06
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/.changeset/rare-goats-sell.md b/.changeset/rare-goats-sell.md
@@ -0,0 +1,5 @@
+---
+'@solana/web3.js-experimental': patch
+---
+
+Fixed the type of `config` on `getComputeUnitEstimateForTransactionMessage`. It is now optional and does not include `transactionMessage`.
diff --git a/packages/library/src/compute-limit.ts b/packages/library/src/compute-limit.ts
@@ -12,7 +12,10 @@ type ComputeUnitEstimateForTransactionMessageFactoryConfig = Readonly<{
 }>;
 type ComputeUnitEstimateForTransactionMessageFunction = (
     transactionMessage: CompilableTransactionMessage | (ITransactionMessageWithFeePayer & TransactionMessage),
-    config: Omit<Parameters<typeof getComputeUnitEstimateForTransactionMessage_INTERNAL_ONLY_DO_NOT_EXPORT>[0], 'rpc'>,
+    config?: Omit<
+        Parameters<typeof getComputeUnitEstimateForTransactionMessage_INTERNAL_ONLY_DO_NOT_EXPORT>[0],
+        'rpc' | 'transactionMessage'
+    >,
 ) => Promise<number>;
 
 export function getComputeUnitEstimateForTransactionMessageFactory({