@solana/kora 0.2.0-beta.6 → 0.3.0-beta.0

package/dist/src/client.js

@@ -1,4 +1,4 @@
-import { assertIsAddress, createNoopSigner } from '@solana/kit';
+import { assertIsAddress, isTransactionSigner } from '@solana/kit';
 import { findAssociatedTokenPda, getTransferInstruction, TOKEN_PROGRAM_ADDRESS } from '@solana-program/token';
 import crypto from 'crypto';
 /**
@@ -303,7 +303,9 @@ export class KoraClient {
      * ```
      */
     async getPaymentInstruction({ transaction, fee_token, source_wallet, token_program_id = TOKEN_PROGRAM_ADDRESS, signer_key, sig_verify, }) {
-        assertIsAddress(source_wallet);
+        const isSigner = typeof source_wallet !== 'string' && isTransactionSigner(source_wallet);
+        const walletAddress = isSigner ? source_wallet.address : source_wallet;
+        assertIsAddress(walletAddress);
         assertIsAddress(fee_token);
         assertIsAddress(token_program_id);
         const { fee_in_token, payment_address, signer_pubkey } = await this.estimateTransactionFee({
@@ -315,7 +317,7 @@ export class KoraClient {
         assertIsAddress(payment_address);
         const [sourceTokenAccount] = await findAssociatedTokenPda({
             mint: fee_token,
-            owner: source_wallet,
+            owner: walletAddress,
             tokenProgram: token_program_id,
         });
         const [destinationTokenAccount] = await findAssociatedTokenPda({
@@ -328,7 +330,7 @@ export class KoraClient {
         }
         const paymentInstruction = getTransferInstruction({
             amount: fee_in_token,
-            authority: createNoopSigner(source_wallet),
+            authority: isSigner ? source_wallet : walletAddress,
             destination: destinationTokenAccount,
             source: sourceTokenAccount,
         });

package/dist/src/types/index.d.ts

@@ -96,8 +96,11 @@ export interface GetPaymentInstructionRequest {
     sig_verify?: boolean;
     /** Optional signer address for the transaction */
     signer_key?: string;
-    /** The wallet owner (not token account) that will be making the token payment */
-    source_wallet: string;
+    /** The wallet owner that will be making the token payment.
+     *  Accepts a plain address string or a TransactionSigner. When a TransactionSigner is provided,
+     *  it is used as the transfer authority on the payment instruction, preserving signer identity
+     *  and avoiding conflicts with other instructions that reference the same address. */
+    source_wallet: TransactionSigner | string;
     /** The token program id to use for the payment (defaults to TOKEN_PROGRAM_ID) */
     token_program_id?: string;
     /** Base64-encoded transaction to estimate fees for */

package/dist/test/unit.test.js

@@ -1,4 +1,5 @@
-import { TOKEN_PROGRAM_ADDRESS } from '@solana-program/token';
+import { getTransferInstruction, TOKEN_PROGRAM_ADDRESS } from '@solana-program/token';
+import { appendTransactionMessageInstructions, createNoopSigner, createTransactionMessage, generateKeyPairSigner, partiallySignTransactionMessageWithSigners, setTransactionMessageFeePayerSigner, setTransactionMessageLifetimeUsingBlockhash, } from '@solana/kit';
 import { KoraClient } from '../src/client.js';
 import { getInstructionsFromBase64Message } from '../src/utils/transaction.js';
 // Mock fetch globally
@@ -402,12 +403,9 @@ describe('KoraClient Unit Tests', () => {
                             role: 1, // writable
                         }), // Destination token account
                         expect.objectContaining({
-                            // readonly-signer
+                            // readonly (plain address, no signer attached)
                             address: validRequest.source_wallet,
-                            role: 2,
-                            signer: expect.objectContaining({
-                                address: validRequest.source_wallet,
-                            }),
+                            role: 0,
                         }), // Authority
                     ],
                     data: expect.any(Uint8Array),
@@ -473,6 +471,70 @@ describe('KoraClient Unit Tests', () => {
             mockFetch.mockRejectedValueOnce(new Error('Network error'));
             await expect(client.getPaymentInstruction(validRequest)).rejects.toThrow('Network error');
         });
+        it('should produce a payment instruction compatible with a real signer for the same address', async () => {
+            // Generate a real KeyPairSigner (simulates a user's wallet)
+            const userSigner = await generateKeyPairSigner();
+            // Mock estimateTransactionFee to return the user's address as source_wallet context
+            const feeEstimate = {
+                fee_in_lamports: 5000,
+                fee_in_token: 50000,
+                payment_address: 'PayKMZWkk483QoFPLRPQ2XVKB7bWnuXwSjvDE1JsWk7',
+                signer_pubkey: 'DemoKMZWkk483QoFPLRPQ2XVKB7bWnuXwSjvDE1JsWk7',
+            };
+            mockSuccessfulResponse(feeEstimate);
+            // Get payment instruction — authority is a plain address (no signer attached)
+            const result = await client.getPaymentInstruction({
+                ...validRequest,
+                source_wallet: userSigner.address,
+            });
+            // Build another instruction that references the same address with the REAL signer
+            // (simulates a program instruction like makePurchase where the user is a signer)
+            const userOwnedIx = getTransferInstruction({
+                amount: 1000n,
+                authority: userSigner, // <-- real KeyPairSigner
+                destination: 'PayKMZWkk483QoFPLRPQ2XVKB7bWnuXwSjvDE1JsWk7',
+                source: '11111111111111111111111111111111',
+            });
+            // Combine both instructions in a transaction — previously this would throw
+            // "Multiple distinct signers" because the payment instruction had a NoopSigner.
+            // Now the payment instruction uses a plain address, so no conflict.
+            const feePayer = createNoopSigner('DemoKMZWkk483QoFPLRPQ2XVKB7bWnuXwSjvDE1JsWk7');
+            const txMessage = appendTransactionMessageInstructions([userOwnedIx, result.payment_instruction], setTransactionMessageLifetimeUsingBlockhash({ blockhash: '11111111111111111111111111111111', lastValidBlockHeight: 0n }, setTransactionMessageFeePayerSigner(feePayer, createTransactionMessage({ version: 0 }))));
+            // This should NOT throw "Multiple distinct signers"
+            await expect(partiallySignTransactionMessageWithSigners(txMessage)).resolves.toBeDefined();
+        });
+        it('should accept a TransactionSigner as source_wallet and preserve signer identity', async () => {
+            const userSigner = await generateKeyPairSigner();
+            const feeEstimate = {
+                fee_in_lamports: 5000,
+                fee_in_token: 50000,
+                payment_address: 'PayKMZWkk483QoFPLRPQ2XVKB7bWnuXwSjvDE1JsWk7',
+                signer_pubkey: 'DemoKMZWkk483QoFPLRPQ2XVKB7bWnuXwSjvDE1JsWk7',
+            };
+            mockSuccessfulResponse(feeEstimate);
+            // Pass the signer directly as source_wallet
+            const result = await client.getPaymentInstruction({
+                ...validRequest,
+                source_wallet: userSigner,
+            });
+            // The authority account meta should carry the signer
+            const authorityMeta = result.payment_instruction.accounts?.[2];
+            expect(authorityMeta).toEqual(expect.objectContaining({
+                address: userSigner.address,
+                role: 2, // readonly-signer
+                signer: userSigner,
+            }));
+            // Combining with another instruction using the same signer should work
+            const userOwnedIx = getTransferInstruction({
+                amount: 1000n,
+                authority: userSigner,
+                destination: 'PayKMZWkk483QoFPLRPQ2XVKB7bWnuXwSjvDE1JsWk7',
+                source: '11111111111111111111111111111111',
+            });
+            const feePayer = createNoopSigner('DemoKMZWkk483QoFPLRPQ2XVKB7bWnuXwSjvDE1JsWk7');
+            const txMessage = appendTransactionMessageInstructions([userOwnedIx, result.payment_instruction], setTransactionMessageLifetimeUsingBlockhash({ blockhash: '11111111111111111111111111111111', lastValidBlockHeight: 0n }, setTransactionMessageFeePayerSigner(feePayer, createTransactionMessage({ version: 0 }))));
+            await expect(partiallySignTransactionMessageWithSigners(txMessage)).resolves.toBeDefined();
+        });
         it('should return correct payment details in response', async () => {
             mockFetch.mockResolvedValueOnce({
                 json: jest.fn().mockResolvedValueOnce({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@solana/kora",
-  "version": "0.2.0-beta.6",
+  "version": "0.3.0-beta.0",
   "description": "TypeScript SDK for Kora RPC",
   "main": "dist/src/index.js",
   "type": "module",