Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/solana-transaction-encoding.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@crossmint/wallets-sdk": minor
---

Solana `serializedTransaction` values now accept base58 or base64 encoding with automatic detection. Solana web3.js transaction types are re-exported, and the `@solana/web3.js` dependency range is relaxed to reduce duplicate-copy type clashes.
2 changes: 1 addition & 1 deletion packages/wallets/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ const client = evmWallet.getViemClient();
// Solana
const solWallet = SolanaWallet.from(wallet);
const solTx = await solWallet.sendTransaction({
serializedTransaction: "<base64-encoded-transaction>",
serializedTransaction: "<base58-or-base64-encoded-transaction>",
});

// Stellar
Expand Down
2 changes: 1 addition & 1 deletion packages/wallets/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
"@crossmint/common-sdk-auth": "workspace:*",
"@crossmint/common-sdk-base": "workspace:*",
"@hey-api/client-fetch": "0.8.1",
"@solana/web3.js": "1.98.1",
"@solana/web3.js": "^1.98.1",
"abitype": "1.0.8",
"base32.js": "0.1.0",
"bs58": "5.0.0",
Expand Down
6 changes: 6 additions & 0 deletions packages/wallets/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,12 @@ export type {
RemoveSignerOptions,
} from "./wallets/types";
export type { Chain, EVMChain, SolanaChain, StellarChain } from "./chains/chains";
export type {
VersionedTransaction,
Transaction as SolanaTransaction,
Keypair as SolanaKeypair,
PublicKey as SolanaPublicKey,
} from "@solana/web3.js";

// Signer configuration types
export {
Expand Down
34 changes: 34 additions & 0 deletions packages/wallets/src/utils/solana-transaction.test.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
import { MessageV0, PublicKey, VersionedTransaction } from "@solana/web3.js";
import bs58 from "bs58";
import { describe, expect, it } from "vitest";
import { normalizeSolanaSerializedTransaction } from "./solana-transaction";

describe("normalizeSolanaSerializedTransaction", () => {
const transaction = new VersionedTransaction(
MessageV0.compile({
payerKey: PublicKey.default,
instructions: [],
recentBlockhash: PublicKey.default.toBase58(),
})
);
const serializedBytes = transaction.serialize();
const base58SerializedTransaction = bs58.encode(serializedBytes);

it("returns a base58-encoded transaction unchanged", () => {
expect(normalizeSolanaSerializedTransaction(base58SerializedTransaction)).toBe(base58SerializedTransaction);
});

it("converts a base64-encoded transaction to base58", () => {
const base64SerializedTransaction = Buffer.from(serializedBytes).toString("base64");

expect(normalizeSolanaSerializedTransaction(base64SerializedTransaction)).toBe(
bs58.encode(transaction.serialize())
);
});

it("throws for an invalid serialized transaction", () => {
expect(() => normalizeSolanaSerializedTransaction("garbage")).toThrow(
"Value must be a base58- or base64-encoded Solana transaction"
);
});
});
17 changes: 17 additions & 0 deletions packages/wallets/src/utils/solana-transaction.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
import { VersionedTransaction } from "@solana/web3.js";
import bs58 from "bs58";

export function normalizeSolanaSerializedTransaction(serialized: string): string {
try {
VersionedTransaction.deserialize(bs58.decode(serialized));
return serialized;
} catch {
try {
const bytes = Buffer.from(serialized, "base64");
VersionedTransaction.deserialize(bytes);
return bs58.encode(bytes);
} catch {
throw new Error("Value must be a base58- or base64-encoded Solana transaction");
}
}
}
9 changes: 8 additions & 1 deletion packages/wallets/src/wallets/__tests__/test-helpers.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
import { vi, type MockedFunction } from "vitest";
import { APIKeyEnvironmentPrefix } from "@crossmint/common-sdk-base";
import { MessageV0, PublicKey, VersionedTransaction } from "@solana/web3.js";
import bs58 from "bs58";
import { Wallet } from "../wallet";
import type { ApiClient } from "../../api";
import type { Chain } from "../../chains/chains";
Expand Down Expand Up @@ -110,5 +112,10 @@ export const createMockApiClient = (overrides: Partial<MockedApiClient> = {}): M
});

export const createMockSolanaSerializedTransaction = (): string => {
return "AQAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAABAgEDBQrKxEIIPWsDwcGCzLQ7FGIHQ38p0dZq6bG2v2wUAUqMx3jV1jZ0";
const message = MessageV0.compile({
payerKey: PublicKey.default,
instructions: [],
recentBlockhash: PublicKey.default.toBase58(),
});
return bs58.encode(new VersionedTransaction(message).serialize());
};
13 changes: 7 additions & 6 deletions packages/wallets/src/wallets/solana.test.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
import bs58 from "bs58";
import { SolanaWallet } from "./solana";
import type { CreateTransactionSuccessResponse } from "../api";
import { TransactionNotCreatedError } from "../utils/errors";
Expand Down Expand Up @@ -31,7 +32,7 @@ describe("SolanaWallet - sendTransaction()", () => {
});

describe("success cases", () => {
it("sends transaction with serialized transaction string", async () => {
it("sends transaction with a base58 serialized transaction string", async () => {
const serializedTx = createMockSolanaSerializedTransaction();

const mockTransactionResponse = {
Expand Down Expand Up @@ -75,9 +76,9 @@ describe("SolanaWallet - sendTransaction()", () => {
);
});

it("sends transaction with serialized transaction string", async () => {
const serializedTx =
"AQAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAABAgEDBQrKxEIIPWsDwcGCzLQ7FGIHQ38p0dZq6bG2v2wUAUqMx3jV1jZ0";
it("converts a base64 serialized transaction to base58", async () => {
const base58SerializedTx = createMockSolanaSerializedTransaction();
const serializedTx = Buffer.from(bs58.decode(base58SerializedTx)).toString("base64");

const mockTransactionResponse = {
id: "txn-sol-456",
Expand All @@ -90,7 +91,7 @@ describe("SolanaWallet - sendTransaction()", () => {
"https://explorer.solana.com/tx/5VERv8NMvzbJMEkV8xnrLkEaWRtSz9CosKDYjCJjBRnbJLgp8uirBgmQpjKhoR4tjF3ZpRzrFmBV6UjKdiSZkQUW",
},
params: {
transaction: serializedTx,
transaction: base58SerializedTx,
signer: "api-key:test",
},
createdAt: Date.now(),
Expand All @@ -113,7 +114,7 @@ describe("SolanaWallet - sendTransaction()", () => {
"me:solana:smart",
expect.objectContaining({
params: expect.objectContaining({
transaction: serializedTx,
transaction: base58SerializedTx,
signer: "api-key",
}),
})
Expand Down
5 changes: 3 additions & 2 deletions packages/wallets/src/wallets/solana.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ import { TransactionNotCreatedError } from "../utils/errors";
import { SolanaExternalWalletSigner } from "@/signers/solana-external-wallet";
import type { CreateTransactionSuccessResponse } from "@/api";
import { walletsLogger } from "../logger";
import { normalizeSolanaSerializedTransaction } from "../utils/solana-transaction";

export class SolanaWallet extends Wallet<SolanaChain> {
constructor(wallet: Wallet<SolanaChain>) {
Expand Down Expand Up @@ -43,7 +44,7 @@ export class SolanaWallet extends Wallet<SolanaChain> {

/**
* Send a raw Solana transaction.
* @param params - The transaction parameters (serialized transaction or Transaction object)
* @param params - The transaction parameters. Serialized transactions may be base58- or base64-encoded.
* @returns The transaction result
*/
@WithLoggerContext({
Expand Down Expand Up @@ -110,7 +111,7 @@ export class SolanaWallet extends Wallet<SolanaChain> {
let serializedTransaction: string;

if ("serializedTransaction" in params) {
serializedTransaction = params.serializedTransaction;
serializedTransaction = normalizeSolanaSerializedTransaction(params.serializedTransaction);
} else {
serializedTransaction = bs58.encode(params.transaction.serialize());
}
Expand Down
1 change: 1 addition & 0 deletions packages/wallets/src/wallets/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,7 @@ export type SolanaTransactionInput = (
transaction: VersionedTransaction;
}
| {
/** A base58- or base64-encoded serialized Solana transaction. */
serializedTransaction: string;
}
) & {
Expand Down
2 changes: 1 addition & 1 deletion pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading