fix(react): return empty array for empty sign batches

.changeset/strong-kids-burn.md

@@ -0,0 +1,5 @@
+---
+'@solana/react': patch
+---
+
+Return immediately when passing empty array of transactions to `useSignTransactions` and `useSignAndSendTransactions`

packages/react/src/__tests__/useSignAndSendTransaction-test.ts

@@ -129,6 +129,21 @@ describe('useSignAndSendTransaction', () => {
         });
     });
     describe('useSignAndSendTransactions', () => {
+        it('returns empty output without calling the wallet implementation when given no inputs', async () => {
+            expect.assertions(2);
+            const { result } = renderHook(() => useSignAndSendTransactions(mockUiWalletAccount, 'solana:danknet'));
+            // eslint-disable-next-line jest/no-conditional-in-test
+            if (result.__type === 'error' || !result.current) {
+                throw result.current;
+            } else {
+                const signAndSendTransactions = result.current;
+                const results = await signAndSendTransactions();
+                // eslint-disable-next-line jest/no-conditional-expect
+                expect(results).toEqual([]);
+                // eslint-disable-next-line jest/no-conditional-expect
+                expect(mockSignAndSendTransaction).not.toHaveBeenCalled();
+            }
+        });
         it("passes through multiple inputs to the wallet's `signAndSendTransaction` implementation", async () => {
             expect.assertions(3);
             mockSignAndSendTransaction.mockResolvedValueOnce([{ signature: 'abc' }, { signature: 'def' }]);

packages/react/src/__tests__/useSignTransaction-test.ts

@@ -132,6 +132,21 @@ describe('useSignTransaction', () => {
         });
     });
     describe('useSignTransactions', () => {
+        it('returns empty output without calling the wallet implementation when given no inputs', async () => {
+            expect.assertions(2);
+            const { result } = renderHook(() => useSignTransactions(mockUiWalletAccount, 'solana:danknet'));
+            // eslint-disable-next-line jest/no-conditional-in-test
+            if (result.__type === 'error' || !result.current) {
+                throw result.current;
+            } else {
+                const signTransactions = result.current;
+                const results = await signTransactions();
+                // eslint-disable-next-line jest/no-conditional-expect
+                expect(results).toEqual([]);
+                // eslint-disable-next-line jest/no-conditional-expect
+                expect(mockSignTransaction).not.toHaveBeenCalled();
+            }
+        });
         it("passes through multiple inputs to the wallet's `signTransaction` implementation", async () => {
             expect.assertions(3);
             mockSignTransaction.mockResolvedValueOnce([{ signature: 'abc' }, { signature: 'def' }]);

packages/react/src/useSignAndSendTransaction.ts

@@ -142,6 +142,9 @@ export function useSignAndSendTransactions<TWalletAccount extends UiWalletAccoun
     const account = getWalletAccountForUiWalletAccount_DO_NOT_USE_OR_YOU_WILL_BE_FIRED(uiWalletAccount);
     return useCallback(
         async (...inputs) => {
+            if (inputs.length === 0) {
+                return [];
+            }
             const inputsWithChainAndAccount = inputs.map(({ options, ...rest }) => {
                 const minContextSlot = options?.minContextSlot;
                 return {

packages/react/src/useSignTransaction.ts

@@ -138,6 +138,9 @@ export function useSignTransactions<TWalletAccount extends UiWalletAccount>(
     const account = getWalletAccountForUiWalletAccount_DO_NOT_USE_OR_YOU_WILL_BE_FIRED(uiWalletAccount);
     return useCallback(
         async (...inputs) => {
+            if (inputs.length === 0) {
+                return [];
+            }
             const inputsWithAccountAndChain = inputs.map(({ options, ...rest }) => {
                 const minContextSlot = options?.minContextSlot;
                 return {