@velocity-exchange/sdk 0.0.5 → 0.2.0

package/src/userConfig.ts

@@ -5,15 +5,13 @@ import { GrpcConfigs, UserAccountSubscriber } from './accounts/types';
 import { WebSocketProgramAccountSubscriber } from './accounts/webSocketProgramAccountSubscriber';
 import { UserAccount } from './types';
 import { grpcMultiUserAccountSubscriber } from './accounts/grpcMultiUserAccountSubscriber';
-import { AtLeastOne } from './util/deprecatedAlias';
 
 type UserConfigBase = {
 	accountSubscription?: UserSubscriptionConfig;
 	userAccountPublicKey: PublicKey;
 };
 
-export type UserConfig = UserConfigBase &
-	AtLeastOne<'velocityClient', 'driftClient', VelocityClient>;
+export type UserConfig = UserConfigBase & { velocityClient: VelocityClient };
 
 export type UserSubscriptionConfig =
 	| {

package/src/userMap/WebsocketSubscription.ts

@@ -14,7 +14,7 @@ export class WebsocketSubscription {
 	private additionalFilters?: MemcmpFilter[];
 	private decodeFn: (name: string, data: Buffer) => UserAccount;
 
-	private subscriber: WebSocketProgramAccountSubscriber<UserAccount>;
+	private subscriber?: WebSocketProgramAccountSubscriber<UserAccount>;
 
 	constructor({
 		userMap,
@@ -54,7 +54,7 @@ export class WebsocketSubscription {
 			this.subscriber = new WebSocketProgramAccountSubscriber<UserAccount>(
 				'UserMap',
 				'user',
-				this.userMap.driftClient.program,
+				this.userMap.velocityClient.program,
 				this.decodeFn,
 				{
 					filters,

package/src/userMap/grpcSubscription.ts

@@ -15,7 +15,7 @@ export class grpcSubscription {
 	private additionalFilters?: MemcmpFilter[];
 	private decodeFn: (name: string, data: Buffer) => UserAccount;
 
-	private subscriber: WebSocketProgramAccountSubscriber<UserAccount>;
+	private subscriber?: WebSocketProgramAccountSubscriber<UserAccount>;
 
 	constructor({
 		grpcConfigs,
@@ -56,7 +56,7 @@ export class grpcSubscription {
 				this.grpcConfigs,
 				'UserMap',
 				'user',
-				this.userMap.driftClient.program,
+				this.userMap.velocityClient.program,
 				this.decodeFn,
 				{
 					filters,

package/src/userMap/referrerMap.ts

@@ -29,14 +29,10 @@ export class ReferrerMap {
 	 */
 	private referrerReferrerInfoMap = new Map<string, ReferrerInfo>();
 	private velocityClient: VelocityClient;
-	/** @deprecated Use `velocityClient` instead. `driftClient` will be removed in a future major. */
-	private get driftClient(): VelocityClient {
-		return this.velocityClient;
-	}
 	private parallelSync: boolean;
 
 	private fetchPromise?: Promise<void>;
-	private fetchPromiseResolver: () => void;
+	private fetchPromiseResolver: () => void = () => {};
 
 	/**
 	 * Creates a new UserStatsMap instance.
@@ -76,12 +72,16 @@ export class ReferrerMap {
 				this.velocityClient.program.programId,
 				new PublicKey(authority)
 			);
-			const buffer = (
-				await this.velocityClient.connection.getAccountInfo(
-					userStatsAccountPublicKey,
-					'processed'
-				)
-			).data;
+			const accountInfo = await this.velocityClient.connection.getAccountInfo(
+				userStatsAccountPublicKey,
+				'processed'
+			);
+			if (!accountInfo) {
+				throw new Error(
+					`ReferrerMap: UserStats account not found for authority ${authority}`
+				);
+			}
+			const buffer = accountInfo.data;
 
 			const referrer = bs58.encode(buffer.subarray(40, 72));
 
@@ -177,7 +177,7 @@ export class ReferrerMap {
 		const rpcRequestArgs = [
 			this.velocityClient.program.programId.toBase58(),
 			{
-				commitment: this.velocityClient.opts.commitment,
+				commitment: this.velocityClient.opts?.commitment,
 				filters: [getUserStatsFilter()],
 				encoding: 'base64',
 				dataSlice: {
@@ -217,7 +217,7 @@ export class ReferrerMap {
 		const rpcRequestArgs = [
 			this.velocityClient.program.programId.toBase58(),
 			{
-				commitment: this.velocityClient.opts.commitment,
+				commitment: this.velocityClient.opts?.commitment,
 				filters: [getUserStatsFilter(), referrerFilter],
 				encoding: 'base64',
 				dataSlice: {

package/src/userMap/revenueShareEscrowMap.ts

@@ -10,14 +10,10 @@ export class RevenueShareEscrowMap {
 	 */
 	private authorityEscrowMap = new Map<string, RevenueShareEscrowAccount>();
 	private velocityClient: VelocityClient;
-	/** @deprecated Use `velocityClient` instead. `driftClient` will be removed in a future major. */
-	private get driftClient(): VelocityClient {
-		return this.velocityClient;
-	}
 	private parallelSync: boolean;
 
 	private fetchPromise?: Promise<void>;
-	private fetchPromiseResolver: () => void;
+	private fetchPromiseResolver: () => void = () => {};
 
 	/**
 	 * Creates a new RevenueShareEscrowMap instance.
@@ -134,6 +130,11 @@ export class RevenueShareEscrowMap {
 				),
 				'confirmed'
 			);
+			if (accountInfo === null) {
+				throw new Error(
+					`RevenueShareEscrow account not found for authority ${authority} during slowSync`
+				);
+			}
 			const escrowNew = (
 				this.velocityClient.program.account as any
 			).revenueShareEscrow.coder.accounts.decode(

package/src/userMap/userMap.ts

@@ -78,10 +78,6 @@ export interface UserMapInterface {
 export class UserMap implements UserMapInterface {
 	private userMap = new Map<string, DataAndSlot<User>>();
 	velocityClient: VelocityClient;
-	/** @deprecated Use `velocityClient` instead. `driftClient` will be removed in a future major. */
-	public get driftClient(): VelocityClient {
-		return this.velocityClient;
-	}
 	eventEmitter: StrictEventEmitter<EventEmitter, UserEvents>;
 	private connection: Connection;
 	private commitment: Commitment;
@@ -89,23 +85,26 @@ export class UserMap implements UserMapInterface {
 	private filterByPoolId?: number;
 	private additionalFilters?: MemcmpFilter[];
 	private disableSyncOnTotalAccountsChange: boolean;
-	private lastNumberOfSubAccounts: BN;
+	private lastNumberOfSubAccounts?: BN;
 	private subscription:
 		| PollingSubscription
 		| WebsocketSubscription
 		| grpcSubscription;
 	private stateAccountUpdateCallback = async (state: StateAccount) => {
-		if (!state.numberOfSubAccounts.eq(this.lastNumberOfSubAccounts)) {
+		if (
+			this.lastNumberOfSubAccounts === undefined ||
+			!state.numberOfSubAccounts.eq(this.lastNumberOfSubAccounts)
+		) {
 			await this.sync();
 			this.lastNumberOfSubAccounts = state.numberOfSubAccounts;
 		}
 	};
-	private decode;
+	private decode: (name: string, buffer: Buffer) => UserAccount;
 	private mostRecentSlot = 0;
 	private syncConfig: SyncConfig;
 
 	private syncPromise?: Promise<void>;
-	private syncPromiseResolver: () => void;
+	private syncPromiseResolver: () => void = () => {};
 
 	private throwOnFailedSync: boolean;
 
@@ -114,27 +113,27 @@ export class UserMap implements UserMapInterface {
 	 */
 	constructor(config: UserMapConfig) {
 		// Type-system guarantees at least one of the two is supplied.
-		this.velocityClient = (config.velocityClient ?? config.driftClient)!;
+		this.velocityClient = config.velocityClient!;
 		if (config.connection) {
 			this.connection = config.connection;
 		} else {
 			this.connection = this.velocityClient.connection;
 		}
 		this.commitment =
-			config.subscriptionConfig.type === 'websocket' ||
+			(config.subscriptionConfig.type === 'websocket' ||
 			config.subscriptionConfig.type === 'polling'
 				? config.subscriptionConfig.commitment ??
-				  this.velocityClient.opts.commitment
-				: this.velocityClient.opts.commitment;
+				  this.velocityClient.opts?.commitment
+				: this.velocityClient.opts?.commitment) ?? 'confirmed';
 		this.includeIdle = config.includeIdle ?? false;
 		this.filterByPoolId = config.filterByPoolId;
 		this.additionalFilters = config.additionalFilters;
 		this.disableSyncOnTotalAccountsChange =
 			config.disableSyncOnTotalAccountsChange ?? false;
 
-		let decodeFn;
+		let decodeFn: (name: string, buffer: Buffer) => UserAccount;
 		if (config.fastDecode ?? true) {
-			decodeFn = (name, buffer) => decodeUser(buffer);
+			decodeFn = (_name: string, buffer: Buffer) => decodeUser(buffer);
 		} else {
 			decodeFn = (
 				this.velocityClient.program.account as any
@@ -223,9 +222,15 @@ export class UserMap implements UserMapInterface {
 			},
 		});
 		await user.subscribe(userAccount);
+		const resolvedSlot = slot ?? user.getUserAccountAndSlot()?.slot;
+		if (resolvedSlot === undefined) {
+			throw new Error(
+				'UserMap.addPubkey: no slot available after subscribing user account'
+			);
+		}
 		this.userMap.set(userAccountPublicKey.toString(), {
 			data: user,
-			slot: slot ?? user.getUserAccountAndSlot()?.slot,
+			slot: resolvedSlot,
 		});
 		this.eventEmitter.emit('userUpdate', user);
 	}
@@ -263,7 +268,11 @@ export class UserMap implements UserMapInterface {
 				accountSubscription
 			);
 		}
-		return this.userMap.get(key).data;
+		const userWithSlot = this.userMap.get(key);
+		if (!userWithSlot) {
+			throw new Error(`UserMap.mustGet: no user found for ${key}`);
+		}
+		return userWithSlot.data;
 	}
 	public async mustGetWithSlot(
 		key: string,
@@ -277,12 +286,16 @@ export class UserMap implements UserMapInterface {
 				accountSubscription
 			);
 		}
-		return this.userMap.get(key);
+		const userWithSlot = this.userMap.get(key);
+		if (!userWithSlot) {
+			throw new Error(`UserMap.mustGetWithSlot: no user found for ${key}`);
+		}
+		return userWithSlot;
 	}
 
 	public async mustGetUserAccount(key: string): Promise<UserAccount> {
 		const user = await this.mustGet(key);
-		return user.getUserAccount();
+		return user.getUserAccountOrThrow();
 	}
 
 	/**
@@ -295,7 +308,7 @@ export class UserMap implements UserMapInterface {
 		if (!user) {
 			return undefined;
 		}
-		return user.data.getUserAccount().authority;
+		return user.data.getUserAccount()?.authority;
 	}
 
 	/**
@@ -379,15 +392,15 @@ export class UserMap implements UserMapInterface {
 		filterCriteria?: UserFilterCriteria
 	): PublicKey[] {
 		const usersMeetingCriteria = Array.from(this.values()).filter((user) => {
-			let pass = true;
+			const userAccount = user.getUserAccountOrThrow();
 			if (filterCriteria && filterCriteria.hasOpenOrders) {
-				pass = pass && user.getUserAccount().hasOpenOrder;
+				return userAccount.hasOpenOrder;
 			}
-			return pass;
+			return true;
 		});
 		const userAuths = new Set(
 			usersMeetingCriteria.map((user) =>
-				user.getUserAccount().authority.toBase58()
+				user.getUserAccountOrThrow().authority.toBase58()
 			)
 		);
 		const userAuthKeys = Array.from(userAuths).map(
@@ -420,7 +433,7 @@ export class UserMap implements UserMapInterface {
 
 	/**
 	 * Syncs the UserMap using the default sync method (single getProgramAccounts call with filters).
-	 * This method may fail when drift has too many users. (nodejs response size limits)
+	 * This method may fail when velocity has too many users. (nodejs response size limits)
 	 * @returns
 	 */
 	private async defaultSync() {
@@ -517,7 +530,7 @@ export class UserMap implements UserMapInterface {
 
 	/**
 	 * Syncs the UserMap using the paginated sync method (multiple getMultipleAccounts calls with filters).
-	 * This method is more reliable when drift has many users.
+	 * This method is more reliable when velocity has many users.
 	 * @returns
 	 */
 	private async paginatedSync() {
@@ -541,9 +554,12 @@ export class UserMap implements UserMapInterface {
 				(account) => account.pubkey
 			);
 
-			const limitConcurrency = async (tasks, limit) => {
-				const executing = [];
-				const results = [];
+			const limitConcurrency = async (
+				tasks: Array<() => Promise<void>>,
+				limit: number
+			) => {
+				const executing: Promise<void>[] = [];
+				const results: Promise<void>[] = [];
 
 				for (let i = 0; i < tasks.length; i++) {
 					const executor = Promise.resolve().then(tasks[i]);
@@ -569,7 +585,7 @@ export class UserMap implements UserMapInterface {
 
 			// @ts-ignore
 			const chunkSize = this.syncConfig.chunkSize ?? 100;
-			const tasks = [];
+			const tasks: Array<() => Promise<void>> = [];
 			for (let i = 0; i < accountPublicKeys.length; i += chunkSize) {
 				const chunk = accountPublicKeys.slice(i, i + chunkSize);
 				tasks.push(async () => {
@@ -630,9 +646,7 @@ export class UserMap implements UserMapInterface {
 				throw err;
 			}
 		} finally {
-			if (this.syncPromiseResolver) {
-				this.syncPromiseResolver();
-			}
+			this.syncPromiseResolver();
 			this.syncPromise = undefined;
 		}
 	}

package/src/userMap/userMapConfig.ts

@@ -1,7 +1,6 @@
 import { Commitment, Connection, MemcmpFilter } from '@solana/web3.js';
 import { VelocityClient } from '../velocityClient';
 import { GrpcConfigs } from '../accounts/types';
-import { AtLeastOne } from '../util/deprecatedAlias';
 
 // passed into UserMap.getUniqueAuthorities to filter users
 export type UserAccountFilterCriteria = {
@@ -65,5 +64,6 @@ type UserMapConfigBase = {
 	additionalFilters?: MemcmpFilter[];
 };
 
-export type UserMapConfig = UserMapConfigBase &
-	AtLeastOne<'velocityClient', 'driftClient', VelocityClient>;
+export type UserMapConfig = UserMapConfigBase & {
+	velocityClient: VelocityClient;
+};

package/src/userMap/userStatsMap.ts

@@ -27,16 +27,12 @@ export class UserStatsMap {
 	 */
 	private userStatsMap = new Map<string, UserStats>();
 	private velocityClient: VelocityClient;
-	/** @deprecated Use `velocityClient` instead. `driftClient` will be removed in a future major. */
-	private get driftClient(): VelocityClient {
-		return this.velocityClient;
-	}
 	private bulkAccountLoader: BulkAccountLoader;
 	private decode;
 	private syncConfig: SyncConfig;
 
 	private syncPromise?: Promise<void>;
-	private syncPromiseResolver: () => void;
+	private syncPromiseResolver: () => void = () => {};
 
 	/**
 	 * Creates a new UserStatsMap instance.
@@ -53,7 +49,7 @@ export class UserStatsMap {
 		if (!bulkAccountLoader) {
 			bulkAccountLoader = new BulkAccountLoader(
 				velocityClient.connection,
-				velocityClient.opts.commitment,
+				velocityClient.opts?.commitment ?? 'confirmed',
 				0
 			);
 		}
@@ -114,8 +110,12 @@ export class UserStatsMap {
 
 	public async updateWithOrderRecord(record: OrderRecord, userMap: UserMap) {
 		const user = await userMap.mustGet(record.user.toString());
-		if (!this.has(user.getUserAccount().authority.toString())) {
-			await this.addUserStat(user.getUserAccount().authority, undefined, false);
+		if (!this.has(user.getUserAccountOrThrow().authority.toString())) {
+			await this.addUserStat(
+				user.getUserAccountOrThrow().authority,
+				undefined,
+				false
+			);
 		}
 	}
 
@@ -137,12 +137,14 @@ export class UserStatsMap {
 			const liqRecord = record as LiquidationRecord;
 
 			const user = await userMap.mustGet(liqRecord.user.toString());
-			await this.mustGet(user.getUserAccount().authority.toString());
+			await this.mustGet(user.getUserAccountOrThrow().authority.toString());
 
 			const liquidatorUser = await userMap.mustGet(
 				liqRecord.liquidator.toString()
 			);
-			await this.mustGet(liquidatorUser.getUserAccount().authority.toString());
+			await this.mustGet(
+				liquidatorUser.getUserAccountOrThrow().authority.toString()
+			);
 		} else if (record.eventType === 'OrderRecord') {
 			if (!userMap) {
 				return;
@@ -157,11 +159,11 @@ export class UserStatsMap {
 
 			if (actionRecord.taker) {
 				const taker = await userMap.mustGet(actionRecord.taker.toString());
-				await this.mustGet(taker.getUserAccount().authority.toString());
+				await this.mustGet(taker.getUserAccountOrThrow().authority.toString());
 			}
 			if (actionRecord.maker) {
 				const maker = await userMap.mustGet(actionRecord.maker.toString());
-				await this.mustGet(maker.getUserAccount().authority.toString());
+				await this.mustGet(maker.getUserAccountOrThrow().authority.toString());
 			}
 		} else if (record.eventType === 'SettlePnlRecord') {
 			if (!userMap) {
@@ -169,7 +171,7 @@ export class UserStatsMap {
 			}
 			const settlePnlRecord = record as SettlePnlRecord;
 			const user = await userMap.mustGet(settlePnlRecord.user.toString());
-			await this.mustGet(user.getUserAccount().authority.toString());
+			await this.mustGet(user.getUserAccountOrThrow().authority.toString());
 		} else if (record.eventType === 'NewUserRecord') {
 			const newUserRecord = record as NewUserRecord;
 			await this.mustGet(newUserRecord.userAuthority.toString());
@@ -183,7 +185,7 @@ export class UserStatsMap {
 		return this.userStatsMap.has(authorityPublicKey);
 	}
 
-	public get(authorityPublicKey: string): UserStats {
+	public get(authorityPublicKey: string): UserStats | undefined {
 		return this.userStatsMap.get(authorityPublicKey);
 	}
 
@@ -201,7 +203,13 @@ export class UserStatsMap {
 				false
 			);
 		}
-		return this.get(authorityPublicKey);
+		const userStats = this.get(authorityPublicKey);
+		if (!userStats) {
+			throw new Error(
+				`UserStatsMap: UserStats missing for authority ${authorityPublicKey} after addUserStat`
+			);
+		}
+		return userStats;
 	}
 
 	public values(): IterableIterator<UserStats> {
@@ -266,9 +274,12 @@ export class UserStatsMap {
 				accountsToLoad = accountsPrefetch.map((account) => account.pubkey);
 			}
 
-			const limitConcurrency = async (tasks, limit) => {
-				const executing = [];
-				const results = [];
+			const limitConcurrency = async (
+				tasks: Array<() => Promise<void>>,
+				limit: number
+			) => {
+				const executing: Array<Promise<void>> = [];
+				const results: Array<Promise<void>> = [];
 
 				for (let i = 0; i < tasks.length; i++) {
 					const executor = Promise.resolve().then(tasks[i]);
@@ -294,7 +305,7 @@ export class UserStatsMap {
 
 			// @ts-ignore
 			const chunkSize = this.syncConfig.chunkSize ?? 100;
-			const tasks = [];
+			const tasks: Array<() => Promise<void>> = [];
 			for (let i = 0; i < accountsToLoad.length; i += chunkSize) {
 				const chunk = accountsToLoad.slice(i, i + chunkSize);
 				tasks.push(async () => {
@@ -302,7 +313,7 @@ export class UserStatsMap {
 						await this.velocityClient.connection.getMultipleAccountsInfoAndContext(
 							chunk,
 							{
-								commitment: this.velocityClient.opts.commitment,
+								commitment: this.velocityClient.opts?.commitment,
 							}
 						);
 
@@ -346,9 +357,7 @@ export class UserStatsMap {
 		} catch (err) {
 			console.error(`Error in UserStatsMap.paginatedSync():`, err);
 		} finally {
-			if (this.syncPromiseResolver) {
-				this.syncPromiseResolver();
-			}
+			this.syncPromiseResolver();
 			this.syncPromise = undefined;
 		}
 	}

package/src/userStats.ts

@@ -18,17 +18,13 @@ import { grpcUserStatsAccountSubscriber } from './accounts/grpcUserStatsAccountS
 
 export class UserStats {
 	velocityClient: VelocityClient;
-	/** @deprecated Use `velocityClient` instead. `driftClient` will be removed in a future major. */
-	public get driftClient(): VelocityClient {
-		return this.velocityClient;
-	}
 	userStatsAccountPublicKey: PublicKey;
 	accountSubscriber: UserStatsAccountSubscriber;
-	isSubscribed: boolean;
+	isSubscribed = false;
 
 	public constructor(config: UserStatsConfig) {
 		// Type-system guarantees at least one of the two is supplied.
-		const velocityClient = (config.velocityClient ?? config.driftClient)!;
+		const velocityClient = config.velocityClient!;
 		this.velocityClient = velocityClient;
 		this.userStatsAccountPublicKey = config.userStatsAccountPublicKey;
 		if (config.accountSubscription?.type === 'polling') {
@@ -61,7 +57,7 @@ export class UserStats {
 			this.accountSubscriber =
 				config.accountSubscription.userStatsAccountSubscriber;
 		} else {
-			const exhaustiveCheck: never = config.accountSubscription;
+			const exhaustiveCheck: undefined = config.accountSubscription;
 
 			throw new Error(
 				`Unknown user stats account subscription type: ${exhaustiveCheck}`
@@ -87,27 +83,40 @@ export class UserStats {
 		this.isSubscribed = false;
 	}
 
-	public getAccountAndSlot(): DataAndSlot<UserStatsAccount> {
+	public getAccountAndSlot(): DataAndSlot<UserStatsAccount> | undefined {
 		return this.accountSubscriber.getUserStatsAccountAndSlot();
 	}
 
-	public getAccount(): UserStatsAccount {
-		return this.accountSubscriber.getUserStatsAccountAndSlot().data;
+	public getAccount(): UserStatsAccount | undefined {
+		return this.accountSubscriber.getUserStatsAccountAndSlot()?.data;
+	}
+
+	/**
+	 * Like {@link getAccount} but throws a named error instead of returning
+	 * `undefined` when the stats account has not been loaded yet.
+	 */
+	public getAccountOrThrow(): UserStatsAccount {
+		const account = this.getAccount();
+		if (!account) {
+			throw new Error('UserStats account not loaded');
+		}
+		return account;
 	}
 
 	public getReferrerInfo(): ReferrerInfo | undefined {
-		if (this.getAccount().referrer.equals(PublicKey.default)) {
+		const account = this.getAccount();
+		if (!account || account.referrer.equals(PublicKey.default)) {
 			return undefined;
 		} else {
 			return {
 				referrer: getUserAccountPublicKeySync(
 					this.velocityClient.program.programId,
-					this.getAccount().referrer,
+					account.referrer,
 					0
 				),
 				referrerStats: getUserStatsAccountPublicKey(
 					this.velocityClient.program.programId,
-					this.getAccount().referrer
+					account.referrer
 				),
 			};
 		}

package/src/userStatsConfig.ts

@@ -2,15 +2,15 @@ import { VelocityClient } from './velocityClient';
 import { Commitment, PublicKey } from '@solana/web3.js';
 import { BulkAccountLoader } from './accounts/bulkAccountLoader';
 import { GrpcConfigs, UserStatsAccountSubscriber } from './accounts/types';
-import { AtLeastOne } from './util/deprecatedAlias';
 
 type UserStatsConfigBase = {
 	accountSubscription?: UserStatsSubscriptionConfig;
 	userStatsAccountPublicKey: PublicKey;
 };
 
-export type UserStatsConfig = UserStatsConfigBase &
-	AtLeastOne<'velocityClient', 'driftClient', VelocityClient>;
+export type UserStatsConfig = UserStatsConfigBase & {
+	velocityClient: VelocityClient;
+};
 
 export type UserStatsSubscriptionConfig =
 	| {

package/src/util/TransactionConfirmationManager.ts

@@ -62,7 +62,7 @@ export class TransactionConfirmationManager {
 
 		let response: RpcResponseAndContext<SignatureResult> | null = null;
 
-		let subscriptionId: ClientSubscriptionId;
+		let subscriptionId: ClientSubscriptionId | undefined;
 
 		const confirmationPromise = new Promise((resolve, reject) => {
 			try {
@@ -262,13 +262,21 @@ export class TransactionConfirmationManager {
 
 			if (status.err) {
 				this.pendingConfirmations.delete(request.txSig);
+				const txError = await getTransactionErrorFromTxSig(
+					request.txSig,
+					this.connection
+				);
 				request.reject(
-					await getTransactionErrorFromTxSig(request.txSig, this.connection)
+					txError ??
+						new Error(
+							`Transaction failed but no error could be resolved for signature ${request.txSig}`
+						)
 				);
 				continue;
 			}
 
 			if (
+				status.confirmationStatus === undefined ||
 				confirmationStatusValues[status.confirmationStatus] === undefined ||
 				confirmationStatusValues[request.desiredConfirmationStatus] ===
 					undefined

package/src/util/chainClock.ts

@@ -35,7 +35,7 @@ export class ChainClock {
 		}
 	}
 
-	public getState(commitment: Commitment): ChainClockProgress {
+	public getState(commitment: Commitment): ChainClockProgress | undefined {
 		return this._state.get(commitment);
 	}
 }

package/src/util/computeUnits.ts

@@ -11,13 +11,17 @@ export async function findComputeUnitConsumption(
 	connection: Connection,
 	txSignature: string,
 	commitment: Finality = 'confirmed'
-): Promise<number[]> {
+): Promise<string[]> {
 	const tx = await connection.getTransaction(txSignature, { commitment });
-	const computeUnits = [];
+	const computeUnits: string[] = [];
+	const logMessages = tx?.meta?.logMessages;
+	if (!logMessages) {
+		return computeUnits;
+	}
 	const regex = new RegExp(
 		`Program ${programId.toString()} consumed ([0-9]{0,6}) of ([0-9]{0,7}) compute units`
 	);
-	tx.meta.logMessages.forEach((logMessage) => {
+	logMessages.forEach((logMessage) => {
 		const match = logMessage.match(regex);
 		if (match && match[1]) {
 			computeUnits.push(match[1]);

package/src/util/ed25519Utils.ts

@@ -37,7 +37,7 @@ const ED25519_INSTRUCTION_LAYOUT = BufferLayout.struct<
 
 /**
  * Constructs a minimal Ed25519 verification instruction that references the data
- * inside the drift custom instruction (e.g. postPythLazerOracleUpdate, placeSignedMsgTakerOrder).
+ * inside the velocity custom instruction (e.g. postPythLazerOracleUpdate, placeSignedMsgTakerOrder).
  *
  * @param customInstructionIndex The index of the custom instruction in the transaction (e.g. if tx contains compute budget limit, compute budget price, ed25519 verify, custom ix, this would be 3).
  * @param messageOffset The offset within the custom instruction data where the signed message begins.