@velocity-exchange/sdk 0.1.0 → 0.2.0

package/lib/node/velocityClient.js

@@ -90,7 +90,7 @@ class VelocityClient {
         this._isSubscribed = val;
     }
     constructor(config) {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s, _t, _u, _v, _w, _x, _y, _z, _0, _1, _2, _3, _4, _5, _6, _7, _8, _9, _10, _11, _12, _13, _14, _15, _16, _17, _18, _19, _20, _21, _22, _23, _24, _25;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r, _s, _t, _u, _v, _w, _x, _y, _z, _0, _1, _2, _3, _4, _5, _6, _7, _8, _9, _10, _11, _12, _13, _14, _15, _16, _17, _18, _19, _20, _21, _22, _23, _24, _25, _26;
         this.users = new Map();
         this._isSubscribed = false;
         this.perpMarketLastSlotCache = new Map();
@@ -197,34 +197,28 @@ class VelocityClient {
                 accountSubscription: this.userAccountSubscriptionConfig,
             });
         }
-        this.marketLookupTable = config.marketLookupTable;
-        if (!this.marketLookupTable) {
-            this.marketLookupTable = new web3_js_1.PublicKey(config_1.configs[this.env].MARKET_LOOKUP_TABLE);
-        }
-        this.marketLookupTables = config.marketLookupTables;
-        if (!this.marketLookupTables) {
-            this.marketLookupTables = config_1.configs[this.env].MARKET_LOOKUP_TABLES.map((tableAddr) => new web3_js_1.PublicKey(tableAddr));
-        }
+        this.marketLookupTables =
+            (_5 = config.marketLookupTables) !== null && _5 !== void 0 ? _5 : config_1.configs[this.env].MARKET_LOOKUP_TABLES.map((tableAddr) => new web3_js_1.PublicKey(tableAddr));
         const delistedMarketSetting = config.delistedMarketSetting || types_2.DelistedMarketSetting.Unsubscribe;
         const noMarketsAndOraclesSpecified = config.perpMarketIndexes === undefined &&
             config.spotMarketIndexes === undefined &&
             config.oracleInfos === undefined;
-        if (((_5 = config.accountSubscription) === null || _5 === void 0 ? void 0 : _5.type) === 'polling') {
-            this.accountSubscriber = new pollingVelocityClientAccountSubscriber_1.PollingVelocityClientAccountSubscriber(this.program, config.accountSubscription.accountLoader, (_6 = config.perpMarketIndexes) !== null && _6 !== void 0 ? _6 : [], (_7 = config.spotMarketIndexes) !== null && _7 !== void 0 ? _7 : [], (_8 = config.oracleInfos) !== null && _8 !== void 0 ? _8 : [], noMarketsAndOraclesSpecified, delistedMarketSetting);
-        }
-        else if (((_9 = config.accountSubscription) === null || _9 === void 0 ? void 0 : _9.type) === 'grpc') {
-            const accountSubscriberClass = (_11 = (_10 = config.accountSubscription) === null || _10 === void 0 ? void 0 : _10.velocityClientAccountSubscriber) !== null && _11 !== void 0 ? _11 : grpcVelocityClientAccountSubscriber_1.grpcVelocityClientAccountSubscriber;
-            this.accountSubscriber = new accountSubscriberClass(config.accountSubscription.grpcConfigs, this.program, (_12 = config.perpMarketIndexes) !== null && _12 !== void 0 ? _12 : [], (_13 = config.spotMarketIndexes) !== null && _13 !== void 0 ? _13 : [], (_14 = config.oracleInfos) !== null && _14 !== void 0 ? _14 : [], noMarketsAndOraclesSpecified, delistedMarketSetting, {
-                resubTimeoutMs: (_15 = config.accountSubscription) === null || _15 === void 0 ? void 0 : _15.resubTimeoutMs,
-                logResubMessages: (_16 = config.accountSubscription) === null || _16 === void 0 ? void 0 : _16.logResubMessages,
+        if (((_6 = config.accountSubscription) === null || _6 === void 0 ? void 0 : _6.type) === 'polling') {
+            this.accountSubscriber = new pollingVelocityClientAccountSubscriber_1.PollingVelocityClientAccountSubscriber(this.program, config.accountSubscription.accountLoader, (_7 = config.perpMarketIndexes) !== null && _7 !== void 0 ? _7 : [], (_8 = config.spotMarketIndexes) !== null && _8 !== void 0 ? _8 : [], (_9 = config.oracleInfos) !== null && _9 !== void 0 ? _9 : [], noMarketsAndOraclesSpecified, delistedMarketSetting);
+        }
+        else if (((_10 = config.accountSubscription) === null || _10 === void 0 ? void 0 : _10.type) === 'grpc') {
+            const accountSubscriberClass = (_12 = (_11 = config.accountSubscription) === null || _11 === void 0 ? void 0 : _11.velocityClientAccountSubscriber) !== null && _12 !== void 0 ? _12 : grpcVelocityClientAccountSubscriber_1.grpcVelocityClientAccountSubscriber;
+            this.accountSubscriber = new accountSubscriberClass(config.accountSubscription.grpcConfigs, this.program, (_13 = config.perpMarketIndexes) !== null && _13 !== void 0 ? _13 : [], (_14 = config.spotMarketIndexes) !== null && _14 !== void 0 ? _14 : [], (_15 = config.oracleInfos) !== null && _15 !== void 0 ? _15 : [], noMarketsAndOraclesSpecified, delistedMarketSetting, {
+                resubTimeoutMs: (_16 = config.accountSubscription) === null || _16 === void 0 ? void 0 : _16.resubTimeoutMs,
+                logResubMessages: (_17 = config.accountSubscription) === null || _17 === void 0 ? void 0 : _17.logResubMessages,
             });
         }
         else {
-            const accountSubscriberClass = (_18 = (_17 = config.accountSubscription) === null || _17 === void 0 ? void 0 : _17.velocityClientAccountSubscriber) !== null && _18 !== void 0 ? _18 : webSocketVelocityClientAccountSubscriber_1.WebSocketVelocityClientAccountSubscriber;
-            this.accountSubscriber = new accountSubscriberClass(this.program, (_19 = config.perpMarketIndexes) !== null && _19 !== void 0 ? _19 : [], (_20 = config.spotMarketIndexes) !== null && _20 !== void 0 ? _20 : [], (_21 = config.oracleInfos) !== null && _21 !== void 0 ? _21 : [], noMarketsAndOraclesSpecified, delistedMarketSetting, {
-                resubTimeoutMs: (_22 = config.accountSubscription) === null || _22 === void 0 ? void 0 : _22.resubTimeoutMs,
-                logResubMessages: (_23 = config.accountSubscription) === null || _23 === void 0 ? void 0 : _23.logResubMessages,
-            }, (_24 = config.accountSubscription) === null || _24 === void 0 ? void 0 : _24.commitment);
+            const accountSubscriberClass = (_19 = (_18 = config.accountSubscription) === null || _18 === void 0 ? void 0 : _18.velocityClientAccountSubscriber) !== null && _19 !== void 0 ? _19 : webSocketVelocityClientAccountSubscriber_1.WebSocketVelocityClientAccountSubscriber;
+            this.accountSubscriber = new accountSubscriberClass(this.program, (_20 = config.perpMarketIndexes) !== null && _20 !== void 0 ? _20 : [], (_21 = config.spotMarketIndexes) !== null && _21 !== void 0 ? _21 : [], (_22 = config.oracleInfos) !== null && _22 !== void 0 ? _22 : [], noMarketsAndOraclesSpecified, delistedMarketSetting, {
+                resubTimeoutMs: (_23 = config.accountSubscription) === null || _23 === void 0 ? void 0 : _23.resubTimeoutMs,
+                logResubMessages: (_24 = config.accountSubscription) === null || _24 === void 0 ? void 0 : _24.logResubMessages,
+            }, (_25 = config.accountSubscription) === null || _25 === void 0 ? void 0 : _25.commitment);
         }
         this.eventEmitter = this.accountSubscriber.eventEmitter;
         this.metricsEventEmitter = new events_1.EventEmitter();
@@ -232,7 +226,7 @@ class VelocityClient {
             this.enableMetricsEvents = true;
         }
         this.txSender =
-            (_25 = config.txSender) !== null && _25 !== void 0 ? _25 : new retryTxSender_1.RetryTxSender({
+            (_26 = config.txSender) !== null && _26 !== void 0 ? _26 : new retryTxSender_1.RetryTxSender({
                 connection: this.connection,
                 wallet: this.wallet,
                 opts: this.opts,
@@ -312,6 +306,17 @@ class VelocityClient {
         var _a;
         return (_a = this.accountSubscriber.getMarketAccountAndSlot(marketIndex)) === null || _a === void 0 ? void 0 : _a.data;
     }
+    /**
+     * Like {@link getPerpMarketAccount} but throws if the market is not loaded,
+     * for call sites that require a guaranteed account.
+     */
+    getPerpMarketAccountOrThrow(marketIndex) {
+        const perpMarketAccount = this.getPerpMarketAccount(marketIndex);
+        if (!perpMarketAccount) {
+            throw new Error(`Perp market ${marketIndex} not found`);
+        }
+        return perpMarketAccount;
+    }
     /**
      * Forces a fetch to rpc before returning accounts. Useful for anchor tests.
      * @param marketIndex
@@ -335,15 +340,28 @@ class VelocityClient {
             .map((value) => value.data);
     }
     getSpotMarketAccount(marketIndex) {
-        return this.accountSubscriber.getSpotMarketAccountAndSlot(marketIndex).data;
+        var _a;
+        return (_a = this.accountSubscriber.getSpotMarketAccountAndSlot(marketIndex)) === null || _a === void 0 ? void 0 : _a.data;
+    }
+    /**
+     * Like {@link getSpotMarketAccount} but throws if the market is not loaded,
+     * for call sites that require a guaranteed account.
+     */
+    getSpotMarketAccountOrThrow(marketIndex) {
+        const spotMarketAccount = this.getSpotMarketAccount(marketIndex);
+        if (!spotMarketAccount) {
+            throw new Error(`Spot market ${marketIndex} not found`);
+        }
+        return spotMarketAccount;
     }
     /**
      * Forces a fetch to rpc before returning accounts. Useful for anchor tests.
      * @param marketIndex
      */
     async forceGetSpotMarketAccount(marketIndex) {
+        var _a;
         await this.accountSubscriber.fetch();
-        return this.accountSubscriber.getSpotMarketAccountAndSlot(marketIndex).data;
+        return (_a = this.accountSubscriber.getSpotMarketAccountAndSlot(marketIndex)) === null || _a === void 0 ? void 0 : _a.data;
     }
     getSpotMarketAccounts() {
         return this.accountSubscriber
@@ -352,23 +370,11 @@ class VelocityClient {
             .map((value) => value.data);
     }
     getQuoteSpotMarketAccount() {
-        return this.accountSubscriber.getSpotMarketAccountAndSlot(numericConstants_1.QUOTE_SPOT_MARKET_INDEX).data;
+        return this.getSpotMarketAccountOrThrow(numericConstants_1.QUOTE_SPOT_MARKET_INDEX);
     }
     getOraclePriceDataAndSlot(oraclePublicKey, oracleSource) {
         return this.accountSubscriber.getOraclePriceDataAndSlot((0, oracleId_1.getOracleId)(oraclePublicKey, oracleSource));
     }
-    /** @deprecated use fetchAllLookupTableAccounts() */
-    async fetchMarketLookupTableAccount() {
-        if (this.lookupTableAccount)
-            return this.lookupTableAccount;
-        if (!this.marketLookupTable) {
-            console.log('Market lookup table address not set');
-            return;
-        }
-        const lookupTableAccount = (await this.connection.getAddressLookupTable(this.marketLookupTable)).value;
-        this.lookupTableAccount = lookupTableAccount;
-        return lookupTableAccount;
-    }
     async fetchAllLookupTableAccounts() {
         if (this.lookupTableAccounts)
             return this.lookupTableAccounts;
@@ -493,7 +499,8 @@ class VelocityClient {
         const authorityChanged = authority && !((_a = this.authority) === null || _a === void 0 ? void 0 : _a.equals(authority));
         this.activeSubAccountId = subAccountId;
         this.authority = authority !== null && authority !== void 0 ? authority : this.authority;
-        this.userStatsAccountPublicKey = (0, pda_1.getUserStatsAccountPublicKey)(this.program.programId, this.authority);
+        const userStatsAccountPublicKey = (0, pda_1.getUserStatsAccountPublicKey)(this.program.programId, this.authority);
+        this.userStatsAccountPublicKey = userStatsAccountPublicKey;
         /* If changing the user authority ie switching from delegate to non-delegate account, need to re-subscribe to the user stats account */
         if (authorityChanged && this.userStats) {
             if (this.userStats.isSubscribed) {
@@ -501,16 +508,17 @@ class VelocityClient {
             }
             this.userStats = new userStats_1.UserStats({
                 velocityClient: this,
-                userStatsAccountPublicKey: this.userStatsAccountPublicKey,
+                userStatsAccountPublicKey: userStatsAccountPublicKey,
                 accountSubscription: this.userStatsAccountSubscriptionConfig,
             });
             this.userStats.subscribe();
         }
     }
     async addUser(subAccountId, authority, userAccount) {
+        var _a;
         authority = authority !== null && authority !== void 0 ? authority : this.authority;
         const userKey = this.getUserMapKey(subAccountId, authority);
-        if (this.users.has(userKey) && this.users.get(userKey).isSubscribed) {
+        if (this.users.has(userKey) && ((_a = this.users.get(userKey)) === null || _a === void 0 ? void 0 : _a.isSubscribed)) {
             return true;
         }
         const user = this.createUser(subAccountId, this.userAccountSubscriptionConfig, authority);
@@ -887,7 +895,16 @@ class VelocityClient {
             userStatsAccount = await (0, fetch_1.fetchUserStatsAccount)(this.connection, this.program, this.wallet.publicKey);
         }
         else {
-            userStatsAccount = userStats.getAccount();
+            const account = userStats.getAccount();
+            if (!account) {
+                userStatsAccount = await (0, fetch_1.fetchUserStatsAccount)(this.connection, this.program, this.wallet.publicKey);
+            }
+            else {
+                userStatsAccount = account;
+            }
+        }
+        if (!userStatsAccount) {
+            throw new Error('UserStats account does not exist');
         }
         return userStatsAccount.numberOfSubAccountsCreated;
     }
@@ -969,7 +986,7 @@ class VelocityClient {
         let remainingAccounts;
         try {
             remainingAccounts = this.getRemainingAccounts({
-                userAccounts: [this.getUserAccount(subAccountId)],
+                userAccounts: [this.getUserAccountOrThrow(subAccountId)],
             });
         }
         catch (err) {
@@ -1201,7 +1218,7 @@ class VelocityClient {
             if ((0, spotPosition_1.isSpotPositionAvailable)(spotPosition)) {
                 continue;
             }
-            const spotMarket = this.getSpotMarketAccount(spotPosition.marketIndex);
+            const spotMarket = this.getSpotMarketAccountOrThrow(spotPosition.marketIndex);
             remainingAccounts.push({
                 isSigner: false,
                 isWritable: true,
@@ -1287,27 +1304,44 @@ class VelocityClient {
     getUser(subAccountId, authority) {
         subAccountId = subAccountId !== null && subAccountId !== void 0 ? subAccountId : this.activeSubAccountId;
         authority = authority !== null && authority !== void 0 ? authority : this.authority;
+        if (subAccountId === undefined || authority === undefined) {
+            throw new Error('Subaccount ID and authority are required');
+        }
         const userMapKey = this.getUserMapKey(subAccountId, authority);
-        if (!this.users.has(userMapKey)) {
+        const user = this.users.get(userMapKey);
+        if (!user) {
             throw new Error(`VelocityClient has no user for user id ${userMapKey}`);
         }
-        return this.users.get(userMapKey);
+        return user;
     }
     hasUser(subAccountId, authority) {
         subAccountId = subAccountId !== null && subAccountId !== void 0 ? subAccountId : this.activeSubAccountId;
         authority = authority !== null && authority !== void 0 ? authority : this.authority;
+        if (subAccountId === undefined || authority === undefined) {
+            throw new Error('Subaccount ID and authority are required');
+        }
         const userMapKey = this.getUserMapKey(subAccountId, authority);
         return this.users.has(userMapKey);
     }
     getUsers() {
         // delegate users get added to the end
         return [...this.users.values()]
-            .filter((acct) => acct.getUserAccount().authority.equals(this.wallet.publicKey))
-            .concat([...this.users.values()].filter((acct) => !acct.getUserAccount().authority.equals(this.wallet.publicKey)));
+            .filter((acct) => { var _a; return (_a = acct.getUserAccount()) === null || _a === void 0 ? void 0 : _a.authority.equals(this.wallet.publicKey); })
+            .concat([...this.users.values()].filter((acct) => { var _a; return !((_a = acct.getUserAccount()) === null || _a === void 0 ? void 0 : _a.authority.equals(this.wallet.publicKey)); }));
     }
     getUserStats() {
         return this.userStats;
     }
+    /**
+     * Like {@link getUserStats} but throws if there is no UserStats
+     * subscription, for call sites that require a guaranteed account.
+     */
+    getUserStatsOrThrow() {
+        if (!this.userStats) {
+            throw new Error('VelocityClient has no UserStats subscription');
+        }
+        return this.userStats;
+    }
     async fetchReferrerNameAccount(name) {
         const nameBuffer = (0, userName_1.encodeName)(name);
         const referrerNameAccountPublicKey = (0, pda_1.getReferrerNamePublicKeySync)(this.program.programId, nameBuffer);
@@ -1326,6 +1360,14 @@ class VelocityClient {
     getUserAccount(subAccountId, authority) {
         return this.getUser(subAccountId, authority).getUserAccount();
     }
+    /**
+     * Like {@link getUserAccount} but throws a named error instead of returning
+     * `undefined` when the account has not been loaded yet. Use at call sites
+     * that structurally require a loaded account.
+     */
+    getUserAccountOrThrow(subAccountId, authority) {
+        return this.getUser(subAccountId, authority).getUserAccountOrThrow();
+    }
     /**
      * Forces a fetch to rpc before returning accounts. Useful for anchor tests.
      * @param subAccountId
@@ -1338,7 +1380,7 @@ class VelocityClient {
         return this.getUser(subAccountId, authority).getUserAccountAndSlot();
     }
     getSpotPosition(marketIndex, subAccountId) {
-        return this.getUserAccount(subAccountId).spotPositions.find((spotPosition) => spotPosition.marketIndex === marketIndex);
+        return this.getUserAccountOrThrow(subAccountId).spotPositions.find((spotPosition) => spotPosition.marketIndex === marketIndex);
     }
     getQuoteAssetTokenAmount() {
         return this.getTokenAmount(numericConstants_1.QUOTE_SPOT_MARKET_INDEX);
@@ -1356,7 +1398,7 @@ class VelocityClient {
         if (spotPosition === undefined) {
             return numericConstants_1.ZERO;
         }
-        const spotMarket = this.getSpotMarketAccount(marketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(marketIndex);
         return (0, spotBalance_1.getSignedTokenAmount)((0, spotBalance_1.getTokenAmount)(spotPosition.scaledBalance, spotMarket, spotPosition.balanceType), spotPosition.balanceType);
     }
     /**
@@ -1365,7 +1407,7 @@ class VelocityClient {
      * @param amount
      */
     convertToSpotPrecision(marketIndex, amount) {
-        const spotMarket = this.getSpotMarketAccount(marketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(marketIndex);
         return (0, spotMarket_1.castNumberToSpotPrecision)(amount, spotMarket);
     }
     /**
@@ -1407,11 +1449,41 @@ class VelocityClient {
             this.mustIncludeSpotMarketIndexes.add(spotMarketIndex);
         });
     }
+    cachePerpMarketSlot(slot, ...marketIndexes) {
+        for (const marketIndex of marketIndexes) {
+            if (slot !== undefined) {
+                this.perpMarketLastSlotCache.set(marketIndex, slot);
+            }
+            else {
+                this.perpMarketLastSlotCache.delete(marketIndex);
+            }
+        }
+    }
+    cacheSpotMarketSlot(slot, ...marketIndexes) {
+        for (const marketIndex of marketIndexes) {
+            if (slot !== undefined) {
+                this.spotMarketLastSlotCache.set(marketIndex, slot);
+            }
+            else {
+                this.spotMarketLastSlotCache.delete(marketIndex);
+            }
+        }
+    }
     getRemainingAccounts(params) {
-        return VelocityCore_1.VelocityCore.remainingAccounts.getRemainingAccounts(this, params);
+        return VelocityCore_1.VelocityCore.remainingAccounts.getRemainingAccounts({
+            getPerpMarketAccount: (marketIndex) => this.getPerpMarketAccountOrThrow(marketIndex),
+            getSpotMarketAccount: (marketIndex) => this.getSpotMarketAccountOrThrow(marketIndex),
+            getUserAccountAndSlot: (subAccountId, authority) => this.getUserAccountAndSlot(subAccountId, authority),
+            activeSubAccountId: this.activeSubAccountId,
+            authority: this.authority,
+            perpMarketLastSlotCache: this.perpMarketLastSlotCache,
+            spotMarketLastSlotCache: this.spotMarketLastSlotCache,
+            mustIncludePerpMarketIndexes: this.mustIncludePerpMarketIndexes,
+            mustIncludeSpotMarketIndexes: this.mustIncludeSpotMarketIndexes,
+        }, params);
     }
     addPerpMarketToRemainingAccountMaps(marketIndex, writable, oracleAccountMap, spotMarketAccountMap, perpMarketAccountMap) {
-        const perpMarketAccount = this.getPerpMarketAccount(marketIndex);
+        const perpMarketAccount = this.getPerpMarketAccountOrThrow(marketIndex);
         perpMarketAccountMap.set(marketIndex, {
             pubkey: perpMarketAccount.pubkey,
             isSigner: false,
@@ -1426,7 +1498,7 @@ class VelocityClient {
         this.addSpotMarketToRemainingAccountMaps(perpMarketAccount.quoteSpotMarketIndex, false, oracleAccountMap, spotMarketAccountMap);
     }
     addSpotMarketToRemainingAccountMaps(marketIndex, writable, oracleAccountMap, spotMarketAccountMap) {
-        const spotMarketAccount = this.getSpotMarketAccount(marketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(marketIndex);
         spotMarketAccountMap.set(spotMarketAccount.marketIndex, {
             pubkey: spotMarketAccount.pubkey,
             isSigner: false,
@@ -1491,10 +1563,32 @@ class VelocityClient {
             perpMarketAccountMap,
         };
     }
+    /**
+     * Look up an open order by its program-assigned order ID from the cached user account.
+     *
+     * `orderId` is the monotonically incrementing u32 counter that the program assigns at
+     * placement time — it is not known until the place instruction executes on-chain. Use
+     * {@link getOrderByUserId} when you need to look up an order by the caller-supplied
+     * `userOrderId` instead.
+     *
+     * Returns `undefined` when the order is not found (already filled, cancelled, or the
+     * account cache is stale).
+     */
     getOrder(orderId, subAccountId) {
         var _a;
         return (_a = this.getUserAccount(subAccountId)) === null || _a === void 0 ? void 0 : _a.orders.find((order) => order.orderId === orderId);
     }
+    /**
+     * Look up an open order by the caller-supplied `userOrderId` from the cached user account.
+     *
+     * `userOrderId` is a 1-255 slot chosen by the caller in {@link OrderParams} and is stable
+     * across the life of the order — useful when you need to reference an order before the
+     * program-assigned {@link Order.orderId} is known (e.g. immediately after placing without
+     * waiting for confirmation). Use {@link getOrder} when you have the program-assigned ID.
+     *
+     * Returns `undefined` when the order is not found (already filled, cancelled, or the
+     * account cache is stale).
+     */
     getOrderByUserId(userOrderId, subAccountId) {
         var _a;
         return (_a = this.getUserAccount(subAccountId)) === null || _a === void 0 ? void 0 : _a.orders.find((order) => order.userOrderId === userOrderId);
@@ -1506,7 +1600,7 @@ class VelocityClient {
      * @param tokenProgram
      */
     async getAssociatedTokenAccount(marketIndex, useNative = true, tokenProgram = spl_token_1.TOKEN_PROGRAM_ID, authority = this.wallet.publicKey, allowOwnerOffCurve = false) {
-        const spotMarket = this.getSpotMarketAccount(marketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(marketIndex);
         if (useNative && spotMarket.mint.equals(spotMarkets_1.WRAPPED_SOL_MINT)) {
             return authority;
         }
@@ -1533,7 +1627,7 @@ class VelocityClient {
     }
     async getDepositTxnIx(amount, marketIndex, associatedTokenAccount, subAccountId, reduceOnly = false, overrides) {
         var _a;
-        const spotMarketAccount = this.getSpotMarketAccount(marketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(marketIndex);
         const isSolMarket = spotMarketAccount.mint.equals(spotMarkets_1.WRAPPED_SOL_MINT);
         const signer = (_a = overrides === null || overrides === void 0 ? void 0 : overrides.authority) !== null && _a !== void 0 ? _a : this.wallet.publicKey;
         const createWSOLTokenAccount = isSolMarket && associatedTokenAccount.equals(signer);
@@ -1594,7 +1688,7 @@ class VelocityClient {
     async deposit(amount, marketIndex, associatedTokenAccount, subAccountId, reduceOnly = false, txParams, initSwiftAccount = false, overrides) {
         const tx = await this.createDepositTxn(amount, marketIndex, associatedTokenAccount, subAccountId, reduceOnly, txParams, initSwiftAccount, overrides);
         const { txSig, slot } = await this.sendTransaction(tx, [], this.opts);
-        this.spotMarketLastSlotCache.set(marketIndex, slot);
+        this.cacheSpotMarketSlot(slot, marketIndex);
         return txSig;
     }
     async getDepositInstruction(amount, marketIndex, userTokenAccount, subAccountId, reduceOnly = false, userInitialized = true, overrides) {
@@ -1602,8 +1696,12 @@ class VelocityClient {
         const userAccountPublicKey = await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.authority, subAccountId !== null && subAccountId !== void 0 ? subAccountId : this.activeSubAccountId);
         let remainingAccounts = [];
         if (userInitialized) {
+            const userAccount = await this.forceGetUserAccount(subAccountId);
+            if (!userAccount) {
+                throw new Error('User account not loaded after force fetch');
+            }
             remainingAccounts = this.getRemainingAccounts({
-                userAccounts: [await this.forceGetUserAccount(subAccountId)],
+                userAccounts: [userAccount],
                 useMarketLastSlotCache: true,
                 writableSpotMarketIndexes: [marketIndex],
             });
@@ -1614,7 +1712,7 @@ class VelocityClient {
                 writableSpotMarketIndexes: [marketIndex],
             });
         }
-        const spotMarketAccount = this.getSpotMarketAccount(marketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(marketIndex);
         this.addTokenMintToRemainingAccounts(spotMarketAccount, remainingAccounts);
         if (this.isTransferHook(spotMarketAccount)) {
             await this.addExtraAccountMetasToRemainingAccounts(spotMarketAccount.mint, remainingAccounts);
@@ -1703,7 +1801,11 @@ class VelocityClient {
             return;
         }
         const extraAccountMetasAddress = (0, spl_token_1.getExtraAccountMetaAddress)(mint, hookAccount.programId);
-        const extraAccountMetas = (0, spl_token_1.getExtraAccountMetas)(await this.connection.getAccountInfo(extraAccountMetasAddress));
+        const extraAccountMetasAccount = await this.connection.getAccountInfo(extraAccountMetasAddress);
+        if (!extraAccountMetasAccount) {
+            throw new Error('Extra account metas account not found for transfer hook');
+        }
+        const extraAccountMetas = (0, spl_token_1.getExtraAccountMetas)(extraAccountMetasAccount);
         for (const acc of extraAccountMetas) {
             // assuming it's an extra account meta that does not rely on ix data
             const resolvedAcc = await (0, spl_token_1.resolveExtraAccountMeta)(this.connection, acc, remainingAccounts, Buffer.from([]), hookAccount.programId);
@@ -1733,7 +1835,7 @@ class VelocityClient {
             const [, initializeSignedMsgUserOrdersAccountIx] = await this.getInitializeSignedMsgUserOrdersAccountIx(this.authority, 8, overrides);
             ixs.push(initializeSignedMsgUserOrdersAccountIx);
         }
-        const spotMarket = this.getSpotMarketAccount(marketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(marketIndex);
         const isSolMarket = spotMarket.mint.equals(spotMarkets_1.WRAPPED_SOL_MINT);
         // Use external wallet for deposit source if provided, otherwise use the wallet
         const depositSource = (_a = overrides === null || overrides === void 0 ? void 0 : overrides.externalWallet) !== null && _a !== void 0 ? _a : this.wallet.publicKey;
@@ -1784,6 +1886,9 @@ class VelocityClient {
         }
         ixs.push(depositCollateralIx);
         if (!donateAmount.eq(numericConstants_1.ZERO)) {
+            if (!wsolTokenAccount) {
+                throw new Error('wsolTokenAccount is required to donate to rev pool');
+            }
             const donateIx = await this.getDepositIntoSpotMarketRevenuePoolIx(1, donateAmount, wsolTokenAccount);
             ixs.push(donateIx);
         }
@@ -1795,6 +1900,9 @@ class VelocityClient {
         // Close the wrapped sol account at the end of the transaction
         // Return funds to the deposit source (external wallet if provided)
         if (createWSOLTokenAccount) {
+            if (!wsolTokenAccount) {
+                throw new Error('wsolTokenAccount was not created');
+            }
             ixs.push((0, spl_token_1.createCloseAccountInstruction)(wsolTokenAccount, depositSource, depositSource, []));
         }
         return {
@@ -1827,7 +1935,7 @@ class VelocityClient {
         const [tx, userAccountPublicKey] = await this.createInitializeUserAccountAndDepositCollateral(amount, userTokenAccount, marketIndex, subAccountId, name, fromSubAccountId, referrerInfo, donateAmount, txParams, customMaxMarginRatio, poolId, overrides);
         const additionalSigners = [];
         const { txSig, slot } = await this.sendTransaction(tx, additionalSigners, this.opts);
-        this.spotMarketLastSlotCache.set(marketIndex, slot);
+        this.cacheSpotMarketSlot(slot, marketIndex);
         await this.addUser(subAccountId);
         return [txSig, userAccountPublicKey];
     }
@@ -1850,7 +1958,7 @@ class VelocityClient {
     }
     async getWithdrawalIxs(amount, marketIndex, associatedTokenAddress, reduceOnly = false, subAccountId) {
         const withdrawIxs = [];
-        const spotMarketAccount = this.getSpotMarketAccount(marketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(marketIndex);
         const isSolMarket = spotMarketAccount.mint.equals(spotMarkets_1.WRAPPED_SOL_MINT);
         const authority = this.wallet.publicKey;
         const createWSOLTokenAccount = isSolMarket && associatedTokenAddress.equals(authority);
@@ -1886,7 +1994,7 @@ class VelocityClient {
         const withdrawIxs = await this.getWithdrawalIxs(amount, marketIndex, associatedTokenAddress, reduceOnly, subAccountId);
         const tx = await this.buildTransaction(withdrawIxs, txParams !== null && txParams !== void 0 ? txParams : this.txParams);
         const { txSig, slot } = await this.sendTransaction(tx, additionalSigners, this.opts);
-        this.spotMarketLastSlotCache.set(marketIndex, slot);
+        this.cacheSpotMarketSlot(slot, marketIndex);
         return txSig;
     }
     async withdrawAllDustPositions(subAccountId, txParams, opts) {
@@ -1915,12 +2023,12 @@ class VelocityClient {
     async getWithdrawIx(amount, marketIndex, userTokenAccount, reduceOnly = false, subAccountId) {
         const user = await this.getUserAccountPublicKey(subAccountId);
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(subAccountId)],
+            userAccounts: [this.getUserAccountOrThrow(subAccountId)],
             useMarketLastSlotCache: true,
             writableSpotMarketIndexes: [marketIndex],
             readableSpotMarketIndexes: [numericConstants_1.QUOTE_SPOT_MARKET_INDEX],
         });
-        const spotMarketAccount = this.getSpotMarketAccount(marketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(marketIndex);
         this.addTokenMintToRemainingAccounts(spotMarketAccount, remainingAccounts);
         if (this.isTransferHook(spotMarketAccount)) {
             await this.addExtraAccountMetasToRemainingAccounts(spotMarketAccount.mint, remainingAccounts);
@@ -1955,7 +2063,7 @@ class VelocityClient {
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(await this.getTransferDepositIx(amount, marketIndex, fromSubAccountId, toSubAccountId), txParams), [], this.opts);
         if (fromSubAccountId === this.activeSubAccountId ||
             toSubAccountId === this.activeSubAccountId) {
-            this.spotMarketLastSlotCache.set(marketIndex, slot);
+            this.cacheSpotMarketSlot(slot, marketIndex);
         }
         return txSig;
     }
@@ -1964,9 +2072,10 @@ class VelocityClient {
         const toUser = await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.wallet.publicKey, toSubAccountId);
         let remainingAccounts;
         const userMapKey = this.getUserMapKey(fromSubAccountId, this.wallet.publicKey);
-        if (this.users.has(userMapKey)) {
+        const mapUser = this.users.get(userMapKey);
+        if (mapUser) {
             remainingAccounts = this.getRemainingAccounts({
-                userAccounts: [this.users.get(userMapKey).getUserAccount()],
+                userAccounts: [mapUser.getUserAccountOrThrow()],
                 useMarketLastSlotCache: true,
                 writableSpotMarketIndexes: [marketIndex],
             });
@@ -1987,7 +2096,7 @@ class VelocityClient {
                 toUser,
                 userStats: this.getUserStatsAccountPublicKey(),
                 state: await this.getStatePublicKey(),
-                spotMarketVault: this.getSpotMarketAccount(marketIndex).vault,
+                spotMarketVault: this.getSpotMarketAccountOrThrow(marketIndex).vault,
             },
             remainingAccounts,
         });
@@ -1996,7 +2105,7 @@ class VelocityClient {
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(await this.getTransferDepositByDelegateIx(amount, marketIndex, fromSubAccountId, toSubAccountId), txParams), [], this.opts);
         if (fromSubAccountId === this.activeSubAccountId ||
             toSubAccountId === this.activeSubAccountId) {
-            this.spotMarketLastSlotCache.set(marketIndex, slot);
+            this.cacheSpotMarketSlot(slot, marketIndex);
         }
         return txSig;
     }
@@ -2005,9 +2114,10 @@ class VelocityClient {
         const toUser = await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.authority, toSubAccountId);
         let remainingAccounts;
         const userMapKey = this.getUserMapKey(fromSubAccountId, this.authority);
-        if (this.users.has(userMapKey)) {
+        const mapUser = this.users.get(userMapKey);
+        if (mapUser) {
             remainingAccounts = this.getRemainingAccounts({
-                userAccounts: [this.users.get(userMapKey).getUserAccount()],
+                userAccounts: [mapUser.getUserAccountOrThrow()],
                 useMarketLastSlotCache: true,
                 writableSpotMarketIndexes: [marketIndex],
             });
@@ -2027,7 +2137,7 @@ class VelocityClient {
                 toUser,
                 userStats: this.getUserStatsAccountPublicKey(),
                 state: await this.getStatePublicKey(),
-                spotMarketVault: this.getSpotMarketAccount(marketIndex).vault,
+                spotMarketVault: this.getSpotMarketAccountOrThrow(marketIndex).vault,
             },
             remainingAccounts,
         });
@@ -2036,19 +2146,16 @@ class VelocityClient {
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(await this.getTransferPoolsIx(depositFromMarketIndex, depositToMarketIndex, borrowFromMarketIndex, borrowToMarketIndex, depositAmount, borrowAmount, fromSubAccountId, toSubAccountId), txParams), [], this.opts);
         if (fromSubAccountId === this.activeSubAccountId ||
             toSubAccountId === this.activeSubAccountId) {
-            this.spotMarketLastSlotCache.set(depositFromMarketIndex, slot);
-            this.spotMarketLastSlotCache.set(depositToMarketIndex, slot);
-            this.spotMarketLastSlotCache.set(borrowFromMarketIndex, slot);
-            this.spotMarketLastSlotCache.set(borrowToMarketIndex, slot);
+            this.cacheSpotMarketSlot(slot, depositFromMarketIndex, depositToMarketIndex, borrowFromMarketIndex, borrowToMarketIndex);
         }
         return txSig;
     }
     async getTransferPoolsIx(depositFromMarketIndex, depositToMarketIndex, borrowFromMarketIndex, borrowToMarketIndex, depositAmount, borrowAmount, fromSubAccountId, toSubAccountId, isToNewSubAccount) {
         const fromUser = await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.wallet.publicKey, fromSubAccountId);
         const toUser = await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.wallet.publicKey, toSubAccountId);
-        const userAccounts = [this.getUserAccount(fromSubAccountId)];
+        const userAccounts = [this.getUserAccountOrThrow(fromSubAccountId)];
         if (!isToNewSubAccount) {
-            userAccounts.push(this.getUserAccount(toSubAccountId));
+            userAccounts.push(this.getUserAccountOrThrow(toSubAccountId));
         }
         const remainingAccounts = this.getRemainingAccounts({
             userAccounts,
@@ -2061,8 +2168,8 @@ class VelocityClient {
             ],
         });
         const tokenPrograms = new Set();
-        const depositFromSpotMarket = this.getSpotMarketAccount(depositFromMarketIndex);
-        const borrowFromSpotMarket = this.getSpotMarketAccount(borrowFromMarketIndex);
+        const depositFromSpotMarket = this.getSpotMarketAccountOrThrow(depositFromMarketIndex);
+        const borrowFromSpotMarket = this.getSpotMarketAccountOrThrow(borrowFromMarketIndex);
         tokenPrograms.add(this.getTokenProgramForSpotMarket(depositFromSpotMarket).toBase58());
         tokenPrograms.add(this.getTokenProgramForSpotMarket(borrowFromSpotMarket).toBase58());
         for (const tokenProgram of tokenPrograms) {
@@ -2079,10 +2186,10 @@ class VelocityClient {
                 toUser,
                 userStats: this.getUserStatsAccountPublicKey(),
                 state: await this.getStatePublicKey(),
-                depositFromSpotMarketVault: this.getSpotMarketAccount(depositFromMarketIndex).vault,
-                depositToSpotMarketVault: this.getSpotMarketAccount(depositToMarketIndex).vault,
-                borrowFromSpotMarketVault: this.getSpotMarketAccount(borrowFromMarketIndex).vault,
-                borrowToSpotMarketVault: this.getSpotMarketAccount(borrowToMarketIndex).vault,
+                depositFromSpotMarketVault: this.getSpotMarketAccountOrThrow(depositFromMarketIndex).vault,
+                depositToSpotMarketVault: this.getSpotMarketAccountOrThrow(depositToMarketIndex).vault,
+                borrowFromSpotMarketVault: this.getSpotMarketAccountOrThrow(borrowFromMarketIndex).vault,
+                borrowToSpotMarketVault: this.getSpotMarketAccountOrThrow(borrowToMarketIndex).vault,
                 velocitySigner: this.getSignerPublicKey(),
             },
             remainingAccounts,
@@ -2097,8 +2204,8 @@ class VelocityClient {
         const toUser = await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.authority, toSubAccountId);
         const remainingAccounts = this.getRemainingAccounts({
             userAccounts: [
-                this.getUserAccount(fromSubAccountId),
-                this.getUserAccount(toSubAccountId),
+                this.getUserAccountOrThrow(fromSubAccountId),
+                this.getUserAccountOrThrow(toSubAccountId),
             ],
             useMarketLastSlotCache: true,
             writablePerpMarketIndexes: [marketIndex],
@@ -2140,9 +2247,9 @@ class VelocityClient {
     }
     async getDepositIntoIsolatedPerpPositionIx(amount, perpMarketIndex, userTokenAccount, subAccountId) {
         const userAccountPublicKey = await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.authority, subAccountId !== null && subAccountId !== void 0 ? subAccountId : this.activeSubAccountId);
-        const perpMarketAccount = this.getPerpMarketAccount(perpMarketIndex);
+        const perpMarketAccount = this.getPerpMarketAccountOrThrow(perpMarketIndex);
         const spotMarketIndex = perpMarketAccount.quoteSpotMarketIndex;
-        const spotMarketAccount = this.getSpotMarketAccount(spotMarketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(spotMarketIndex);
         const remainingAccounts = this.getRemainingAccounts({
             userAccounts: [],
             writableSpotMarketIndexes: [spotMarketIndex],
@@ -2168,7 +2275,7 @@ class VelocityClient {
         const transferIx = await this.getTransferIsolatedPerpPositionDepositIx(amount, perpMarketIndex, subAccountId, noBuffer);
         const needsToSettle = amount.lt(tokenAmountDeposited.neg()) || amount.eq(numericConstants_1.MIN_I64) || trySettle;
         if (needsToSettle) {
-            const settleIx = await this.settleMultiplePNLsIx(await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.authority, subAccountId !== null && subAccountId !== void 0 ? subAccountId : this.activeSubAccountId), this.getUserAccount(subAccountId), [perpMarketIndex], types_1.SettlePnlMode.TRY_SETTLE);
+            const settleIx = await this.settleMultiplePNLsIx(await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.authority, subAccountId !== null && subAccountId !== void 0 ? subAccountId : this.activeSubAccountId), this.getUserAccountOrThrow(subAccountId), [perpMarketIndex], types_1.SettlePnlMode.TRY_SETTLE);
             ixs.push(settleIx);
         }
         ixs.push(transferIx);
@@ -2181,10 +2288,10 @@ class VelocityClient {
     }
     async getTransferIsolatedPerpPositionDepositIx(amount, perpMarketIndex, subAccountId, noAmountBuffer, signingAuthority) {
         const userAccountPublicKey = await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.authority, subAccountId !== null && subAccountId !== void 0 ? subAccountId : this.activeSubAccountId);
-        const perpMarketAccount = this.getPerpMarketAccount(perpMarketIndex);
+        const perpMarketAccount = this.getPerpMarketAccountOrThrow(perpMarketIndex);
         const spotMarketIndex = perpMarketAccount.quoteSpotMarketIndex;
-        const spotMarketAccount = this.getSpotMarketAccount(spotMarketIndex);
-        const user = await this.getUserAccount(subAccountId);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(spotMarketIndex);
+        const user = await this.getUserAccountOrThrow(subAccountId);
         const remainingAccounts = this.getRemainingAccounts({
             userAccounts: [user],
             writableSpotMarketIndexes: [spotMarketIndex],
@@ -2211,16 +2318,20 @@ class VelocityClient {
     }
     async getWithdrawFromIsolatedPerpPositionIxsBundle(amount, perpMarketIndex, subAccountId, userTokenAccount) {
         const userAccountPublicKey = await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.authority, subAccountId !== null && subAccountId !== void 0 ? subAccountId : this.activeSubAccountId);
-        const userAccount = this.getUserAccount(subAccountId);
+        const userAccount = this.getUserAccountOrThrow(subAccountId);
         const tokenAmountDeposited = this.getIsolatedPerpPositionTokenAmount(perpMarketIndex);
-        const isolatedPositionUnrealizedPnl = (0, position_1.calculateClaimablePnl)(this.getPerpMarketAccount(perpMarketIndex), this.getSpotMarketAccount(this.getPerpMarketAccount(perpMarketIndex).quoteSpotMarketIndex), userAccount.perpPositions.find((p) => p.marketIndex === perpMarketIndex), this.getOracleDataForSpotMarket(this.getPerpMarketAccount(perpMarketIndex).quoteSpotMarketIndex));
+        const isolatedPerpPosition = userAccount.perpPositions.find((p) => p.marketIndex === perpMarketIndex);
+        if (!isolatedPerpPosition) {
+            throw new Error(`No perp position found for market index ${perpMarketIndex}`);
+        }
+        const isolatedPositionUnrealizedPnl = (0, position_1.calculateClaimablePnl)(this.getPerpMarketAccountOrThrow(perpMarketIndex), this.getSpotMarketAccountOrThrow(this.getPerpMarketAccountOrThrow(perpMarketIndex).quoteSpotMarketIndex), isolatedPerpPosition, this.getOracleDataForSpotMarket(this.getPerpMarketAccountOrThrow(perpMarketIndex).quoteSpotMarketIndex));
         const depositAmountPlusUnrealizedPnl = tokenAmountDeposited.add(isolatedPositionUnrealizedPnl);
         const amountToWithdraw = amount.gt(depositAmountPlusUnrealizedPnl)
             ? numericConstants_1.MIN_I64 // min i64
             : amount;
         let associatedTokenAccount = userTokenAccount;
         if (!associatedTokenAccount) {
-            const perpMarketAccount = this.getPerpMarketAccount(perpMarketIndex);
+            const perpMarketAccount = this.getPerpMarketAccountOrThrow(perpMarketIndex);
             const quoteSpotMarketIndex = perpMarketAccount.quoteSpotMarketIndex;
             associatedTokenAccount = await this.getAssociatedTokenAccount(quoteSpotMarketIndex);
         }
@@ -2236,11 +2347,11 @@ class VelocityClient {
     }
     async getWithdrawFromIsolatedPerpPositionIx(amount, perpMarketIndex, userTokenAccount, subAccountId) {
         const userAccountPublicKey = await (0, pda_1.getUserAccountPublicKey)(this.program.programId, this.authority, subAccountId !== null && subAccountId !== void 0 ? subAccountId : this.activeSubAccountId);
-        const perpMarketAccount = this.getPerpMarketAccount(perpMarketIndex);
+        const perpMarketAccount = this.getPerpMarketAccountOrThrow(perpMarketIndex);
         const spotMarketIndex = perpMarketAccount.quoteSpotMarketIndex;
-        const spotMarketAccount = this.getSpotMarketAccount(spotMarketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(spotMarketIndex);
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(subAccountId)],
+            userAccounts: [this.getUserAccountOrThrow(subAccountId)],
             writableSpotMarketIndexes: [spotMarketIndex],
             readablePerpMarketIndex: [perpMarketIndex],
         });
@@ -2263,7 +2374,7 @@ class VelocityClient {
         return txSig;
     }
     async updateSpotMarketCumulativeInterestIx(marketIndex) {
-        const spotMarket = this.getSpotMarketAccount(marketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(marketIndex);
         return await this.program.instruction.updateSpotMarketCumulativeInterest({
             accounts: {
                 state: await this.getStatePublicKey(),
@@ -2348,7 +2459,7 @@ class VelocityClient {
         const preppedTxs = await this.prepareMarketOrderTxs(orderParams, userAccountPublicKey, userAccount, makerInfo, txParams, bracketOrdersParams, cancelExistingOrders, settlePnl);
         const signedTxs = (await this.txHandler.getSignedTransactionMap(preppedTxs, this.wallet)).signedTxMap;
         const { txSig, slot } = await this.sendTransaction(signedTxs.marketOrderTx, [], this.opts, true);
-        this.perpMarketLastSlotCache.set(orderParams.marketIndex, slot);
+        this.cachePerpMarketSlot(slot, orderParams.marketIndex);
         return {
             txSig,
             signedFillTx: signedTxs.fillTx,
@@ -2364,7 +2475,7 @@ class VelocityClient {
             preIxs.push(await this.getTransferIsolatedPerpPositionDepositIx(isolatedPositionDepositAmount, orderParams.marketIndex, subAccountId));
         }
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(await this.getPlacePerpOrderIx(orderParams, subAccountId), txParams, undefined, undefined, undefined, undefined, preIxs), [], this.opts);
-        this.perpMarketLastSlotCache.set(orderParams.marketIndex, slot);
+        this.cachePerpMarketSlot(slot, orderParams.marketIndex);
         return txSig;
     }
     /**
@@ -2418,7 +2529,7 @@ class VelocityClient {
         const remainingAccounts = this.getRemainingAccounts({
             userAccounts: (depositToTradeArgs === null || depositToTradeArgs === void 0 ? void 0 : depositToTradeArgs.isMakingNewAccount)
                 ? []
-                : [this.getUserAccount(subAccountId)],
+                : [this.getUserAccountOrThrow(subAccountId)],
             useMarketLastSlotCache: false,
             readablePerpMarketIndex: orderParams.marketIndex,
             readableSpotMarketIndexes: isDepositToTradeTx
@@ -2447,7 +2558,7 @@ class VelocityClient {
         const marketAccountInfos = [];
         const oracleAccountInfos = [];
         for (const marketIndex of marketIndexes) {
-            const market = this.getPerpMarketAccount(marketIndex);
+            const market = this.getPerpMarketAccountOrThrow(marketIndex);
             marketAccountInfos.push({
                 pubkey: market.pubkey,
                 isWritable: true,
@@ -2508,6 +2619,20 @@ class VelocityClient {
             },
         });
     }
+    /**
+     * Cancel an open order and broadcast the transaction.
+     *
+     * When `orderId` is `undefined` (omitted or passed explicitly), the instruction is
+     * sent with a `null` order ID and the program cancels the most recently placed order
+     * on-chain via `get_last_order_id`. This is safe to use in a composed transaction
+     * where a place instruction runs first and the assigned order ID is not yet known.
+     *
+     * Note: when `orderId` is `undefined` and `overrides.withdrawIsolatedDepositAmount`
+     * is also provided, `getOrder` will return `undefined` (the ID is unknown client-side),
+     * causing the withdraw path to throw — supply an explicit `orderId` in that case.
+     *
+     * @see {@link getCancelOrderIx} to obtain the instruction without sending.
+     */
     async cancelOrder(orderId, txParams, subAccountId, overrides) {
         const cancelIx = await this.getCancelOrderIx(orderId, subAccountId);
         const instructions = [cancelIx];
@@ -2516,6 +2641,9 @@ class VelocityClient {
             const perpMarketIndex = order === null || order === void 0 ? void 0 : order.marketIndex;
             const withdrawAmount = overrides.withdrawIsolatedDepositAmount;
             if (withdrawAmount.gt(numericConstants_1.ZERO)) {
+                if (perpMarketIndex === undefined) {
+                    throw new Error(`Order ${orderId} not found when withdrawing isolated deposit`);
+                }
                 const withdrawIxs = await this.getWithdrawFromIsolatedPerpPositionIxsBundle(withdrawAmount, perpMarketIndex, subAccountId);
                 instructions.push(...withdrawIxs);
             }
@@ -2523,10 +2651,25 @@ class VelocityClient {
         const { txSig } = await this.sendTransaction(await this.buildTransaction(instructions, txParams), [], this.opts);
         return txSig;
     }
+    /**
+     * Build a `cancelOrder` instruction for the given order.
+     *
+     * When `orderId` is `undefined` (omitted or passed explicitly), the instruction is
+     * built with a `null` order ID (`orderId ?? null`). The program interprets a `null`
+     * ID as "cancel the user's most recently placed order" (via `get_last_order_id`
+     * on-chain). This is useful when composing a multi-instruction transaction where a
+     * place instruction precedes the cancel and the program-assigned order ID is not yet
+     * known at build time.
+     *
+     * When `orderId` is supplied, only that specific order is cancelled.
+     *
+     * @see {@link cancelOrder} to send the transaction directly.
+     * @see {@link getCancelOrderByUserIdIx} to cancel by the caller-supplied `userOrderId`.
+     */
     async getCancelOrderIx(orderId, subAccountId) {
         const user = await this.getUserAccountPublicKey(subAccountId);
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(subAccountId)],
+            userAccounts: [this.getUserAccountOrThrow(subAccountId)],
             useMarketLastSlotCache: true,
         });
         return await VelocityCore_1.VelocityCore.buildCancelOrderInstruction({
@@ -2544,10 +2687,13 @@ class VelocityClient {
     }
     async getCancelOrderByUserIdIx(userOrderId, subAccountId) {
         const user = await this.getUserAccountPublicKey(subAccountId);
-        const order = this.getOrderByUserId(userOrderId);
-        const oracle = this.getPerpMarketAccount(order.marketIndex).oracle;
+        const order = this.getOrderByUserId(userOrderId, subAccountId);
+        if (!order) {
+            throw new Error(`Order with user order id ${userOrderId} not found`);
+        }
+        const oracle = this.getPerpMarketAccountOrThrow(order.marketIndex).oracle;
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(subAccountId)],
+            userAccounts: [this.getUserAccountOrThrow(subAccountId)],
             useMarketLastSlotCache: true,
         });
         return await VelocityCore_1.VelocityCore.buildCancelOrderByUserIdInstruction({
@@ -2584,7 +2730,7 @@ class VelocityClient {
     async getCancelOrdersByIdsIx(orderIds, subAccountId, user, overrides) {
         var _a, _b, _c;
         const userAccountPubKey = (_a = user === null || user === void 0 ? void 0 : user.userAccountPublicKey) !== null && _a !== void 0 ? _a : (await this.getUserAccountPublicKey(subAccountId));
-        const userAccount = (_b = user === null || user === void 0 ? void 0 : user.getUserAccount()) !== null && _b !== void 0 ? _b : this.getUserAccount(subAccountId);
+        const userAccount = (_b = user === null || user === void 0 ? void 0 : user.getUserAccount()) !== null && _b !== void 0 ? _b : this.getUserAccountOrThrow(subAccountId);
         const remainingAccounts = this.getRemainingAccounts({
             userAccounts: [userAccount],
             useMarketLastSlotCache: true,
@@ -2616,7 +2762,7 @@ class VelocityClient {
             }
         }
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(subAccountId)],
+            userAccounts: [this.getUserAccountOrThrow(subAccountId)],
             readablePerpMarketIndex,
             readableSpotMarketIndexes,
             useMarketLastSlotCache: true,
@@ -2680,7 +2826,7 @@ class VelocityClient {
             }
         }
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(subAccountId)],
+            userAccounts: [this.getUserAccountOrThrow(subAccountId)],
             readablePerpMarketIndex,
             readableSpotMarketIndexes,
             useMarketLastSlotCache: true,
@@ -2722,7 +2868,7 @@ class VelocityClient {
             }
         }
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(subAccountId)],
+            userAccounts: [this.getUserAccountOrThrow(subAccountId)],
             readablePerpMarketIndex,
             readableSpotMarketIndexes,
             useMarketLastSlotCache: true,
@@ -2765,7 +2911,7 @@ class VelocityClient {
         const user = await this.getUserAccountPublicKey(subAccountId);
         const isPerp = (0, types_1.isVariant)(params.marketType, 'perp');
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(subAccountId)],
+            userAccounts: [this.getUserAccountOrThrow(subAccountId)],
             readablePerpMarketIndex: isPerp ? [params.marketIndex] : [],
             readableSpotMarketIndexes: isPerp ? [] : [params.marketIndex],
             useMarketLastSlotCache: true,
@@ -2804,6 +2950,7 @@ class VelocityClient {
     // is detected from the order bitflags; pass the taker's decoded escrow (e.g.
     // from a RevenueShareEscrowMap) so referred takers also get it attached.
     takerEscrow) {
+        var _a;
         const userStatsPublicKey = (0, pda_1.getUserStatsAccountPublicKey)(this.program.programId, userAccount.authority);
         let filler;
         if (fillerAuthority) {
@@ -2821,7 +2968,10 @@ class VelocityClient {
         }
         const marketIndex = order
             ? order.marketIndex
-            : userAccount.orders.find((order) => order.orderId === userAccount.nextOrderId - 1).marketIndex;
+            : (_a = userAccount.orders.find((order) => order.orderId === userAccount.nextOrderId - 1)) === null || _a === void 0 ? void 0 : _a.marketIndex;
+        if (marketIndex === undefined) {
+            throw new Error('No order found to fill');
+        }
         makerInfo = Array.isArray(makerInfo)
             ? makerInfo
             : makerInfo
@@ -2869,7 +3019,13 @@ class VelocityClient {
         if (takerEscrowMeta) {
             remainingAccounts.push(takerEscrowMeta);
         }
-        const orderId = isSignedMsg ? null : order.orderId;
+        let orderId = null;
+        if (!isSignedMsg) {
+            if (!order) {
+                throw new Error('order is required to fill a non-signedMsg order');
+            }
+            orderId = order.orderId;
+        }
         return await VelocityCore_1.VelocityCore.buildFillPerpOrderInstruction({
             program: this.program,
             orderId,
@@ -2987,13 +3143,12 @@ class VelocityClient {
         const lookupTables = res.lookupTables;
         const tx = (await this.buildTransaction(ixs, txParams, 0, lookupTables));
         const { txSig, slot } = await this.sendTransaction(tx);
-        this.spotMarketLastSlotCache.set(outMarketIndex, slot);
-        this.spotMarketLastSlotCache.set(inMarketIndex, slot);
+        this.cacheSpotMarketSlot(slot, outMarketIndex, inMarketIndex);
         return txSig;
     }
     async getTitanSwapIx({ titanClient, outMarketIndex, inMarketIndex, outAssociatedTokenAccount, inAssociatedTokenAccount, amount, slippageBps, swapMode, onlyDirectRoutes, reduceOnly, userAccountPublicKey, }) {
-        const outMarket = this.getSpotMarketAccount(outMarketIndex);
-        const inMarket = this.getSpotMarketAccount(inMarketIndex);
+        const outMarket = this.getSpotMarketAccountOrThrow(outMarketIndex);
+        const inMarket = this.getSpotMarketAccountOrThrow(inMarketIndex);
         const isExactOut = swapMode === 'ExactOut';
         const exactOutBufferedAmountIn = amount.muln(1001).divn(1000); // Add 10bp buffer
         const preInstructions = [];
@@ -3046,8 +3201,8 @@ class VelocityClient {
         return { ixs, lookupTables };
     }
     async getJupiterSwapIxV6({ jupiterClient, outMarketIndex, inMarketIndex, outAssociatedTokenAccount, inAssociatedTokenAccount, amount, slippageBps, swapMode, onlyDirectRoutes, quote, reduceOnly, userAccountPublicKey, }) {
-        const outMarket = this.getSpotMarketAccount(outMarketIndex);
-        const inMarket = this.getSpotMarketAccount(inMarketIndex);
+        const outMarket = this.getSpotMarketAccountOrThrow(outMarketIndex);
+        const inMarket = this.getSpotMarketAccountOrThrow(inMarketIndex);
         if (!quote) {
             const fetchedQuote = await jupiterClient.getQuote({
                 inputMint: inMarket.mint,
@@ -3128,8 +3283,11 @@ class VelocityClient {
         const userAccountPublicKeyToUse = userAccountPublicKey || (await this.getUserAccountPublicKey());
         const userAccounts = [];
         try {
-            if (this.hasUser() && this.getUser().getUserAccountAndSlot()) {
-                userAccounts.push(this.getUser().getUserAccountAndSlot().data);
+            const userAccount = this.hasUser()
+                ? this.getUser().getUserAccount()
+                : undefined;
+            if (userAccount) {
+                userAccounts.push(userAccount);
             }
         }
         catch (err) {
@@ -3140,8 +3298,8 @@ class VelocityClient {
             writableSpotMarketIndexes: [outMarketIndex, inMarketIndex],
             readableSpotMarketIndexes: [numericConstants_1.QUOTE_SPOT_MARKET_INDEX],
         });
-        const outSpotMarket = this.getSpotMarketAccount(outMarketIndex);
-        const inSpotMarket = this.getSpotMarketAccount(inMarketIndex);
+        const outSpotMarket = this.getSpotMarketAccountOrThrow(outMarketIndex);
+        const inSpotMarket = this.getSpotMarketAccountOrThrow(inMarketIndex);
         const outTokenProgram = this.getTokenProgramForSpotMarket(outSpotMarket);
         const inTokenProgram = this.getTokenProgramForSpotMarket(inSpotMarket);
         if (!outTokenProgram.equals(inTokenProgram)) {
@@ -3205,8 +3363,8 @@ class VelocityClient {
     }
     async getSwapIxV2({ swapClient, outMarketIndex, inMarketIndex, outAssociatedTokenAccount, inAssociatedTokenAccount, amount, slippageBps, swapMode, onlyDirectRoutes, reduceOnly, quote, v6, userAccountPublicKey, }) {
         // Get market accounts to determine mints
-        const outMarket = this.getSpotMarketAccount(outMarketIndex);
-        const inMarket = this.getSpotMarketAccount(inMarketIndex);
+        const outMarket = this.getSpotMarketAccountOrThrow(outMarketIndex);
+        const inMarket = this.getSpotMarketAccountOrThrow(inMarketIndex);
         const isExactOut = swapMode === 'ExactOut';
         const preInstructions = [];
         // Handle token accounts if not provided
@@ -3280,7 +3438,7 @@ class VelocityClient {
         return this.sendTransaction(tx);
     }
     async getStakeForMSOLIx({ amount, userAccountPublicKey, }) {
-        const wSOLMint = this.getSpotMarketAccount(1).mint;
+        const wSOLMint = this.getSpotMarketAccountOrThrow(1).mint;
         const mSOLAccount = await this.getAssociatedTokenAccount(2);
         const wSOLAccount = await this.getAssociatedTokenAccount(1, false);
         const wSOLAccountExists = await this.checkIfAccountExists(wSOLAccount);
@@ -3431,7 +3589,7 @@ class VelocityClient {
     }
     async placeAndTakePerpOrder(orderParams, makerInfo, successCondition, auctionDurationPercentage, txParams, subAccountId, takerEscrow) {
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(await this.getPlaceAndTakePerpOrderIx(orderParams, makerInfo, successCondition, auctionDurationPercentage, subAccountId, undefined, takerEscrow), txParams), [], this.opts);
-        this.perpMarketLastSlotCache.set(orderParams.marketIndex, slot);
+        this.cachePerpMarketSlot(slot, orderParams.marketIndex);
         return txSig;
     }
     async preparePlaceAndTakePerpOrderWithAdditionalOrders(orderParams, makerInfo, bracketOrdersParams = new Array(), txParams, subAccountId, cancelExistingOrders, settlePnl, exitEarlyIfSimFails, auctionDurationPercentage, optionalIxs, isolatedPositionDepositAmount) {
@@ -3469,6 +3627,9 @@ class VelocityClient {
                 useSimulatedComputeUnits: false,
             };
             if (shouldUseSimulationComputeUnits || shouldExitIfSimulationFails) {
+                if (!txParams) {
+                    throw new Error('txParams is required when simulating compute units or exiting early on failed simulation');
+                }
                 const placeAndTakeTxToSim = (await this.buildTransaction(placeAndTakeIxs, txParams, undefined, lookupTableAccounts, true, recentBlockHash, optionalIxs));
                 const simulationResult = await txParamProcessor_1.TransactionParamProcessor.getTxSimComputeUnits(placeAndTakeTxToSim, this.connection, (_b = txParams.computeUnitsBufferMultiplier) !== null && _b !== void 0 ? _b : 1.2, txParams.lowerBoundCu);
                 if (shouldExitIfSimulationFails && !simulationResult.success) {
@@ -3495,7 +3656,7 @@ class VelocityClient {
         const prepSettlePnlTx = async () => {
             if (settlePnl && (0, types_1.isVariant)(orderParams.marketType, 'perp')) {
                 const userAccountPublicKey = await this.getUserAccountPublicKey(subAccountId);
-                const settlePnlIx = await this.settlePNLIx(userAccountPublicKey, this.getUserAccount(subAccountId), orderParams.marketIndex);
+                const settlePnlIx = await this.settlePNLIx(userAccountPublicKey, this.getUserAccountOrThrow(subAccountId), orderParams.marketIndex);
                 txsToSign.settlePnlTx = await this.buildTransaction([settlePnlIx], txParams, this.txVersion, lookupTableAccounts, undefined, recentBlockHash, optionalIxs);
             }
             return;
@@ -3518,8 +3679,11 @@ class VelocityClient {
         const signedTxs = (await this.txHandler.getSignedTransactionMap(txsToSign, 
         // @ts-ignore
         this.provider.wallet)).signedTxMap;
+        if (!signedTxs.placeAndTakeTx) {
+            throw new Error('placeAndTakeTx was not built');
+        }
         const { txSig, slot } = await this.sendTransaction(signedTxs.placeAndTakeTx, [], this.opts, true);
-        this.perpMarketLastSlotCache.set(orderParams.marketIndex, slot);
+        this.cachePerpMarketSlot(slot, orderParams.marketIndex);
         return {
             txSig,
             signedCancelExistingOrdersTx: signedTxs.cancelExistingOrdersTx,
@@ -3541,7 +3705,7 @@ class VelocityClient {
             : makerInfo
                 ? [makerInfo]
                 : [];
-        const userAccounts = [this.getUserAccount(subAccountId)];
+        const userAccounts = [this.getUserAccountOrThrow(subAccountId)];
         for (const maker of makerInfo) {
             userAccounts.push(maker.makerUserAccount);
         }
@@ -3585,7 +3749,7 @@ class VelocityClient {
     }
     async placeAndMakePerpOrder(orderParams, takerInfo, txParams, subAccountId, takerEscrow) {
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(await this.getPlaceAndMakePerpOrderIx(orderParams, takerInfo, subAccountId, takerEscrow), txParams), [], this.opts);
-        this.perpMarketLastSlotCache.set(orderParams.marketIndex, slot);
+        this.cachePerpMarketSlot(slot, orderParams.marketIndex);
         return txSig;
     }
     async getPlaceAndMakePerpOrderIx(orderParams, takerInfo, subAccountId, 
@@ -3599,7 +3763,7 @@ class VelocityClient {
         const user = await this.getUserAccountPublicKey(subAccountId);
         const remainingAccounts = this.getRemainingAccounts({
             userAccounts: [
-                this.getUserAccount(subAccountId),
+                this.getUserAccountOrThrow(subAccountId),
                 takerInfo.takerUserAccount,
             ],
             useMarketLastSlotCache: true,
@@ -3674,6 +3838,9 @@ class VelocityClient {
         });
     }
     signMessage(message, keypair = this.wallet.payer) {
+        if (!keypair) {
+            throw new Error('No keypair available to sign message');
+        }
         return Buffer.from(tweetnacl_1.default.sign.detached(message, keypair.secretKey));
     }
     async placeSignedMsgTakerOrder(signedSignedMsgOrderParams, marketIndex, takerInfo, precedingIxs = [], overrideCustomIxIndex, txParams) {
@@ -3727,7 +3894,7 @@ class VelocityClient {
     async placeAndMakeSignedMsgPerpOrder(signedSignedMsgOrderParams, signedMsgOrderUuid, takerInfo, orderParams, txParams, subAccountId, precedingIxs = [], overrideCustomIxIndex, takerEscrow) {
         const ixs = await this.getPlaceAndMakeSignedMsgPerpOrderIxs(signedSignedMsgOrderParams, signedMsgOrderUuid, takerInfo, orderParams, subAccountId, precedingIxs, overrideCustomIxIndex, takerEscrow);
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(ixs, txParams), [], this.opts);
-        this.perpMarketLastSlotCache.set(orderParams.marketIndex, slot);
+        this.cachePerpMarketSlot(slot, orderParams.marketIndex);
         return txSig;
     }
     async getPlaceAndMakeSignedMsgPerpOrderIxs(signedSignedMsgOrderParams, signedMsgOrderUuid, takerInfo, orderParams, subAccountId, precedingIxs = [], overrideCustomIxIndex, 
@@ -3741,7 +3908,7 @@ class VelocityClient {
         const user = await this.getUserAccountPublicKey(subAccountId);
         const remainingAccounts = this.getRemainingAccounts({
             userAccounts: [
-                this.getUserAccount(subAccountId),
+                this.getUserAccountOrThrow(subAccountId),
                 takerInfo.takerUserAccount,
             ],
             useMarketLastSlotCache: false,
@@ -3867,9 +4034,13 @@ class VelocityClient {
      * @returns
      */
     async getModifyOrderIx({ orderId, newDirection, newBaseAmount, newLimitPrice, newOraclePriceOffset, newTriggerPrice, newTriggerCondition, auctionDuration, auctionStartPrice, auctionEndPrice, reduceOnly, postOnly, bitFlags, maxTs, policy, }, subAccountId, overrides) {
-        var _a, _b, _c, _d, _e, _f, _g;
+        var _a, _b, _c, _d, _e;
         const userPubKey = (_b = (_a = overrides === null || overrides === void 0 ? void 0 : overrides.user) === null || _a === void 0 ? void 0 : _a.getUserAccountPublicKey()) !== null && _b !== void 0 ? _b : (await this.getUserAccountPublicKey(subAccountId));
-        const userAccount = (_d = (_c = overrides === null || overrides === void 0 ? void 0 : overrides.user) === null || _c === void 0 ? void 0 : _c.getUserAccount()) !== null && _d !== void 0 ? _d : this.getUserAccount(subAccountId);
+        const overrideUserAccount = (_c = overrides === null || overrides === void 0 ? void 0 : overrides.user) === null || _c === void 0 ? void 0 : _c.getUserAccount();
+        if ((overrides === null || overrides === void 0 ? void 0 : overrides.user) && !overrideUserAccount) {
+            throw new Error('modifyOrder: override user account is not loaded');
+        }
+        const userAccount = overrideUserAccount !== null && overrideUserAccount !== void 0 ? overrideUserAccount : this.getUserAccountOrThrow(subAccountId);
         const remainingAccounts = this.getRemainingAccounts({
             userAccounts: [userAccount],
             useMarketLastSlotCache: true,
@@ -3890,7 +4061,7 @@ class VelocityClient {
             policy: policy || null,
             maxTs: maxTs || null,
         };
-        const authority = (_g = (_e = overrides === null || overrides === void 0 ? void 0 : overrides.authority) !== null && _e !== void 0 ? _e : (_f = overrides === null || overrides === void 0 ? void 0 : overrides.user) === null || _f === void 0 ? void 0 : _f.getUserAccount().authority) !== null && _g !== void 0 ? _g : this.wallet.publicKey;
+        const authority = (_e = (_d = overrides === null || overrides === void 0 ? void 0 : overrides.authority) !== null && _d !== void 0 ? _d : overrideUserAccount === null || overrideUserAccount === void 0 ? void 0 : overrideUserAccount.authority) !== null && _e !== void 0 ? _e : this.wallet.publicKey;
         return await VelocityCore_1.VelocityCore.buildModifyOrderInstruction({
             program: this.program,
             orderId,
@@ -3927,7 +4098,7 @@ class VelocityClient {
     async getModifyOrderByUserIdIx({ userOrderId, newDirection, newBaseAmount, newLimitPrice, newOraclePriceOffset, newTriggerPrice, newTriggerCondition, auctionDuration, auctionStartPrice, auctionEndPrice, reduceOnly, postOnly, bitFlags, maxTs, policy, }, subAccountId) {
         const user = await this.getUserAccountPublicKey(subAccountId);
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(subAccountId)],
+            userAccounts: [this.getUserAccountOrThrow(subAccountId)],
             useMarketLastSlotCache: true,
         });
         const orderParams = {
@@ -3965,7 +4136,7 @@ class VelocityClient {
             : marketIndexes;
         if (filterInvalidMarkets) {
             for (const marketIndex of marketIndexes) {
-                const perpMarketAccount = this.getPerpMarketAccount(marketIndex);
+                const perpMarketAccount = this.getPerpMarketAccountOrThrow(marketIndex);
                 const oraclePriceData = this.getOracleDataForPerpMarket(marketIndex);
                 const stateAccountAndSlot = this.accountSubscriber.getStateAccountAndSlot();
                 const oracleGuardRails = stateAccountAndSlot.data.oracleGuardRails;
@@ -4205,7 +4376,7 @@ class VelocityClient {
     }
     async liquidatePerp(userAccountPublicKey, userAccount, marketIndex, maxBaseAssetAmount, limitPrice, txParams, liquidatorSubAccountId) {
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(await this.getLiquidatePerpIx(userAccountPublicKey, userAccount, marketIndex, maxBaseAssetAmount, limitPrice, liquidatorSubAccountId), txParams), [], this.opts);
-        this.perpMarketLastSlotCache.set(marketIndex, slot);
+        this.cachePerpMarketSlot(slot, marketIndex);
         return txSig;
     }
     async getLiquidatePerpIx(userAccountPublicKey, userAccount, marketIndex, maxBaseAssetAmount, limitPrice, liquidatorSubAccountId) {
@@ -4213,7 +4384,10 @@ class VelocityClient {
         const liquidator = await this.getUserAccountPublicKey(liquidatorSubAccountId);
         const liquidatorStatsPublicKey = this.getUserStatsAccountPublicKey();
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(liquidatorSubAccountId), userAccount],
+            userAccounts: [
+                this.getUserAccountOrThrow(liquidatorSubAccountId),
+                userAccount,
+            ],
             useMarketLastSlotCache: true,
             writablePerpMarketIndexes: [marketIndex],
         });
@@ -4233,7 +4407,7 @@ class VelocityClient {
     }
     async liquidatePerpWithFill(userAccountPublicKey, userAccount, marketIndex, makerInfos, txParams, liquidatorSubAccountId) {
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(await this.getLiquidatePerpWithFillIx(userAccountPublicKey, userAccount, marketIndex, makerInfos, liquidatorSubAccountId), txParams), [], this.opts);
-        this.perpMarketLastSlotCache.set(marketIndex, slot);
+        this.cachePerpMarketSlot(slot, marketIndex);
         return txSig;
     }
     async getLiquidatePerpWithFillIx(userAccountPublicKey, userAccount, marketIndex, makerInfos, liquidatorSubAccountId) {
@@ -4273,8 +4447,7 @@ class VelocityClient {
     }
     async liquidateSpot(userAccountPublicKey, userAccount, assetMarketIndex, liabilityMarketIndex, maxLiabilityTransfer, limitPrice, txParams, liquidatorSubAccountId) {
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(await this.getLiquidateSpotIx(userAccountPublicKey, userAccount, assetMarketIndex, liabilityMarketIndex, maxLiabilityTransfer, limitPrice, liquidatorSubAccountId), txParams), [], this.opts);
-        this.spotMarketLastSlotCache.set(assetMarketIndex, slot);
-        this.spotMarketLastSlotCache.set(liabilityMarketIndex, slot);
+        this.cacheSpotMarketSlot(slot, assetMarketIndex, liabilityMarketIndex);
         return txSig;
     }
     async getLiquidateSpotIx(userAccountPublicKey, userAccount, assetMarketIndex, liabilityMarketIndex, maxLiabilityTransfer, limitPrice, liquidatorSubAccountId) {
@@ -4282,7 +4455,10 @@ class VelocityClient {
         const liquidator = await this.getUserAccountPublicKey(liquidatorSubAccountId);
         const liquidatorStatsPublicKey = this.getUserStatsAccountPublicKey();
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(liquidatorSubAccountId), userAccount],
+            userAccounts: [
+                this.getUserAccountOrThrow(liquidatorSubAccountId),
+                userAccount,
+            ],
             useMarketLastSlotCache: true,
             writableSpotMarketIndexes: [liabilityMarketIndex, assetMarketIndex],
         });
@@ -4299,8 +4475,8 @@ class VelocityClient {
         });
     }
     async getJupiterLiquidateSpotWithSwapIxV6({ jupiterClient, liabilityMarketIndex, assetMarketIndex, swapAmount, assetTokenAccount, liabilityTokenAccount, slippageBps, swapMode, onlyDirectRoutes, quote, userAccount, userAccountPublicKey, liquidatorSubAccountId, maxAccounts, }) {
-        const liabilityMarket = this.getSpotMarketAccount(liabilityMarketIndex);
-        const assetMarket = this.getSpotMarketAccount(assetMarketIndex);
+        const liabilityMarket = this.getSpotMarketAccountOrThrow(liabilityMarketIndex);
+        const assetMarket = this.getSpotMarketAccountOrThrow(assetMarketIndex);
         if (!quote) {
             const fetchedQuote = await jupiterClient.getQuote({
                 inputMint: assetMarket.mint,
@@ -4378,8 +4554,8 @@ class VelocityClient {
             writableSpotMarketIndexes: [liabilityMarketIndex, assetMarketIndex],
             readableSpotMarketIndexes: [numericConstants_1.QUOTE_SPOT_MARKET_INDEX],
         });
-        const liabilitySpotMarket = this.getSpotMarketAccount(liabilityMarketIndex);
-        const assetSpotMarket = this.getSpotMarketAccount(assetMarketIndex);
+        const liabilitySpotMarket = this.getSpotMarketAccountOrThrow(liabilityMarketIndex);
+        const assetSpotMarket = this.getSpotMarketAccountOrThrow(assetMarketIndex);
         const liabilityTokenProgram = this.getTokenProgramForSpotMarket(liabilitySpotMarket);
         const assetTokenProgram = this.getTokenProgramForSpotMarket(assetSpotMarket);
         if (!liabilityTokenProgram.equals(assetTokenProgram)) {
@@ -4442,68 +4618,10 @@ class VelocityClient {
         });
         return { beginSwapIx, endSwapIx };
     }
-    async getInsuranceFundSwapIx({ inMarketIndex, outMarketIndex, amountIn, inTokenAccount, outTokenAccount, }) {
-        const remainingAccounts = await this.getRemainingAccounts({
-            userAccounts: [],
-            writableSpotMarketIndexes: [inMarketIndex, outMarketIndex],
-        });
-        const inSpotMarket = this.getSpotMarketAccount(inMarketIndex);
-        const outSpotMarket = this.getSpotMarketAccount(outMarketIndex);
-        if (this.isToken2022(inSpotMarket) || this.isToken2022(outSpotMarket)) {
-            remainingAccounts.push({
-                pubkey: inSpotMarket.mint,
-                isWritable: false,
-                isSigner: false,
-            });
-            remainingAccounts.push({
-                pubkey: outSpotMarket.mint,
-                isWritable: false,
-                isSigner: false,
-            });
-            if (this.isTransferHook(inSpotMarket)) {
-                this.addExtraAccountMetasToRemainingAccounts(inSpotMarket.mint, remainingAccounts);
-            }
-            if (this.isTransferHook(outSpotMarket)) {
-                this.addExtraAccountMetasToRemainingAccounts(outSpotMarket.mint, remainingAccounts);
-            }
-        }
-        const ifRebalanceConfig = (0, pda_1.getIfRebalanceConfigPublicKey)(this.program.programId, inMarketIndex, outMarketIndex);
-        const beginSwapIx = await this.program.instruction.beginInsuranceFundSwap(inMarketIndex, outMarketIndex, amountIn, {
-            accounts: {
-                state: await this.getStatePublicKey(),
-                authority: this.wallet.publicKey,
-                outInsuranceFundVault: outSpotMarket.insuranceFund.vault,
-                inInsuranceFundVault: inSpotMarket.insuranceFund.vault,
-                outTokenAccount,
-                inTokenAccount,
-                ifRebalanceConfig: ifRebalanceConfig,
-                tokenProgram: spl_token_1.TOKEN_PROGRAM_ID,
-                velocitySigner: this.getStateAccount().signer,
-                instructions: web3_js_1.SYSVAR_INSTRUCTIONS_PUBKEY,
-            },
-            remainingAccounts,
-        });
-        const endSwapIx = await this.program.instruction.endInsuranceFundSwap(inMarketIndex, outMarketIndex, {
-            accounts: {
-                state: await this.getStatePublicKey(),
-                authority: this.wallet.publicKey,
-                outInsuranceFundVault: outSpotMarket.insuranceFund.vault,
-                inInsuranceFundVault: inSpotMarket.insuranceFund.vault,
-                outTokenAccount,
-                inTokenAccount,
-                ifRebalanceConfig: ifRebalanceConfig,
-                tokenProgram: spl_token_1.TOKEN_PROGRAM_ID,
-                velocitySigner: this.getStateAccount().signer,
-                instructions: web3_js_1.SYSVAR_INSTRUCTIONS_PUBKEY,
-            },
-            remainingAccounts,
-        });
-        return { beginSwapIx, endSwapIx };
-    }
     async liquidateBorrowForPerpPnl(userAccountPublicKey, userAccount, perpMarketIndex, liabilityMarketIndex, maxLiabilityTransfer, limitPrice, txParams, liquidatorSubAccountId) {
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(await this.getLiquidateBorrowForPerpPnlIx(userAccountPublicKey, userAccount, perpMarketIndex, liabilityMarketIndex, maxLiabilityTransfer, limitPrice, liquidatorSubAccountId), txParams), [], this.opts);
-        this.perpMarketLastSlotCache.set(perpMarketIndex, slot);
-        this.spotMarketLastSlotCache.set(liabilityMarketIndex, slot);
+        this.cachePerpMarketSlot(slot, perpMarketIndex);
+        this.cacheSpotMarketSlot(slot, liabilityMarketIndex);
         return txSig;
     }
     async getLiquidateBorrowForPerpPnlIx(userAccountPublicKey, userAccount, perpMarketIndex, liabilityMarketIndex, maxLiabilityTransfer, limitPrice, liquidatorSubAccountId) {
@@ -4511,7 +4629,10 @@ class VelocityClient {
         const liquidator = await this.getUserAccountPublicKey(liquidatorSubAccountId);
         const liquidatorStatsPublicKey = this.getUserStatsAccountPublicKey();
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(liquidatorSubAccountId), userAccount],
+            userAccounts: [
+                this.getUserAccountOrThrow(liquidatorSubAccountId),
+                userAccount,
+            ],
             writablePerpMarketIndexes: [perpMarketIndex],
             writableSpotMarketIndexes: [liabilityMarketIndex],
         });
@@ -4529,8 +4650,8 @@ class VelocityClient {
     }
     async liquidatePerpPnlForDeposit(userAccountPublicKey, userAccount, perpMarketIndex, assetMarketIndex, maxPnlTransfer, limitPrice, txParams, liquidatorSubAccountId) {
         const { txSig, slot } = await this.sendTransaction(await this.buildTransaction(await this.getLiquidatePerpPnlForDepositIx(userAccountPublicKey, userAccount, perpMarketIndex, assetMarketIndex, maxPnlTransfer, limitPrice, liquidatorSubAccountId), txParams), [], this.opts);
-        this.perpMarketLastSlotCache.set(perpMarketIndex, slot);
-        this.spotMarketLastSlotCache.set(assetMarketIndex, slot);
+        this.cachePerpMarketSlot(slot, perpMarketIndex);
+        this.cacheSpotMarketSlot(slot, assetMarketIndex);
         return txSig;
     }
     async getLiquidatePerpPnlForDepositIx(userAccountPublicKey, userAccount, perpMarketIndex, assetMarketIndex, maxPnlTransfer, limitPrice, liquidatorSubAccountId) {
@@ -4538,7 +4659,10 @@ class VelocityClient {
         const liquidator = await this.getUserAccountPublicKey(liquidatorSubAccountId);
         const liquidatorStatsPublicKey = this.getUserStatsAccountPublicKey();
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(liquidatorSubAccountId), userAccount],
+            userAccounts: [
+                this.getUserAccountOrThrow(liquidatorSubAccountId),
+                userAccount,
+            ],
             writablePerpMarketIndexes: [perpMarketIndex],
             writableSpotMarketIndexes: [assetMarketIndex],
         });
@@ -4563,7 +4687,10 @@ class VelocityClient {
         const liquidator = await this.getUserAccountPublicKey(liquidatorSubAccountId);
         const liquidatorStatsPublicKey = this.getUserStatsAccountPublicKey();
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(liquidatorSubAccountId), userAccount],
+            userAccounts: [
+                this.getUserAccountOrThrow(liquidatorSubAccountId),
+                userAccount,
+            ],
             writablePerpMarketIndexes: [marketIndex],
             writableSpotMarketIndexes: [numericConstants_1.QUOTE_SPOT_MARKET_INDEX],
         });
@@ -4593,10 +4720,13 @@ class VelocityClient {
         const liquidator = await this.getUserAccountPublicKey(liquidatorSubAccountId);
         const liquidatorStatsPublicKey = this.getUserStatsAccountPublicKey();
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount(liquidatorSubAccountId), userAccount],
+            userAccounts: [
+                this.getUserAccountOrThrow(liquidatorSubAccountId),
+                userAccount,
+            ],
             writableSpotMarketIndexes: [marketIndex],
         });
-        const spotMarket = this.getSpotMarketAccount(marketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(marketIndex);
         const tokenProgramId = this.getTokenProgramForSpotMarket(spotMarket);
         this.addTokenMintToRemainingAccounts(spotMarket, remainingAccounts);
         if (this.isTransferHook(spotMarket)) {
@@ -4637,7 +4767,7 @@ class VelocityClient {
         return txSig;
     }
     async getUpdatePrelaunchOracleIx(perpMarketIndex) {
-        const perpMarket = this.getPerpMarketAccount(perpMarketIndex);
+        const perpMarket = this.getPerpMarketAccountOrThrow(perpMarketIndex);
         if (!(0, types_1.isVariant)(perpMarket.oracleSource, 'prelaunch')) {
             throw new Error(`Wrong oracle source ${perpMarket.oracleSource}`);
         }
@@ -4654,7 +4784,7 @@ class VelocityClient {
         return txSig;
     }
     async getUpdatePerpBidAskTwapIx(perpMarketIndex, makers) {
-        const perpMarket = this.getPerpMarketAccount(perpMarketIndex);
+        const perpMarket = this.getPerpMarketAccountOrThrow(perpMarketIndex);
         const remainingAccounts = [];
         for (const [maker, makerStats] of makers) {
             remainingAccounts.push({
@@ -4707,10 +4837,14 @@ class VelocityClient {
         this.eventEmitter.emit(eventName, data);
     }
     getOracleDataForPerpMarket(marketIndex) {
-        return this.accountSubscriber.getOraclePriceDataAndSlotForPerpMarket(marketIndex).data;
+        const oraclePriceDataAndSlot = this.accountSubscriber.getOraclePriceDataAndSlotForPerpMarket(marketIndex);
+        if (!oraclePriceDataAndSlot) {
+            throw new Error(`No oracle price data for perp market ${marketIndex}`);
+        }
+        return oraclePriceDataAndSlot.data;
     }
     getMMOracleDataForPerpMarket(marketIndex) {
-        const perpMarket = this.getPerpMarketAccount(marketIndex);
+        const perpMarket = this.getPerpMarketAccountOrThrow(marketIndex);
         const oracleData = this.getOracleDataForPerpMarket(marketIndex);
         const stateAccountAndSlot = this.accountSubscriber.getStateAccountAndSlot();
         const isMMOracleActive = !perpMarket.marketStats.mmOracleSlot.eq(numericConstants_1.ZERO);
@@ -4730,11 +4864,12 @@ class VelocityClient {
                 .gt(oracleData.sequenceId.div(new anchor_1.BN(10000)));
         let isExchangeOracleMoreRecent = true;
         if (doSlotCheckForRecency &&
-            oracleData.slot <= perpMarket.marketStats.mmOracleSlot) {
+            oracleData.slot.lte(perpMarket.marketStats.mmOracleSlot)) {
             isExchangeOracleMoreRecent = false;
         }
         else if (!doSlotCheckForRecency &&
-            oracleData.sequenceId < mmOracleSequenceId) {
+            oracleData.sequenceId != null &&
+            oracleData.sequenceId.lt(mmOracleSequenceId)) {
             isExchangeOracleMoreRecent = false;
         }
         const conf = (0, utils_3.getOracleConfidenceFromMMOracleData)(perpMarket.marketStats.mmOraclePrice, oracleData);
@@ -4761,7 +4896,11 @@ class VelocityClient {
         }
     }
     getOracleDataForSpotMarket(marketIndex) {
-        return this.accountSubscriber.getOraclePriceDataAndSlotForSpotMarket(marketIndex).data;
+        const oraclePriceDataAndSlot = this.accountSubscriber.getOraclePriceDataAndSlotForSpotMarket(marketIndex);
+        if (!oraclePriceDataAndSlot) {
+            throw new Error(`No oracle price data for spot market ${marketIndex}`);
+        }
+        return oraclePriceDataAndSlot.data;
     }
     async initializeInsuranceFundStake(marketIndex, txParams) {
         const { txSig } = await this.sendTransaction(await this.buildTransaction(await this.getInitializeInsuranceFundStakeIx(marketIndex), txParams), [], this.opts);
@@ -4771,7 +4910,7 @@ class VelocityClient {
         const ifStakeAccountPublicKey = (0, pda_1.getInsuranceFundStakeAccountPublicKey)(this.program.programId, this.wallet.publicKey, marketIndex);
         const accounts = {
             insuranceFundStake: ifStakeAccountPublicKey,
-            spotMarket: this.getSpotMarketAccount(marketIndex).pubkey,
+            spotMarket: this.getSpotMarketAccountOrThrow(marketIndex).pubkey,
             userStats: (0, pda_1.getUserStatsAccountPublicKey)(this.program.programId, this.wallet.publicKey // only allow payer to initialize own insurance fund stake account
             ),
             authority: this.wallet.publicKey,
@@ -4785,7 +4924,7 @@ class VelocityClient {
         });
     }
     async getAddInsuranceFundStakeIx(marketIndex, amount, collateralAccountPublicKey) {
-        const spotMarket = this.getSpotMarketAccount(marketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(marketIndex);
         const ifStakeAccountPublicKey = (0, pda_1.getInsuranceFundStakeAccountPublicKey)(this.program.programId, this.wallet.publicKey, marketIndex);
         const remainingAccounts = [];
         this.addTokenMintToRemainingAccounts(spotMarket, remainingAccounts);
@@ -4832,7 +4971,7 @@ class VelocityClient {
      */
     async getAddInsuranceFundStakeIxs({ marketIndex, amount, collateralAccountPublicKey, initializeStakeAccount, fromSubaccount, }) {
         const addIfStakeIxs = [];
-        const spotMarketAccount = this.getSpotMarketAccount(marketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(marketIndex);
         const isSolMarket = spotMarketAccount.mint.equals(spotMarkets_1.WRAPPED_SOL_MINT);
         const createWSOLTokenAccount = isSolMarket && collateralAccountPublicKey.equals(this.wallet.publicKey);
         const tokenProgramId = this.getTokenProgramForSpotMarket(spotMarketAccount);
@@ -4870,7 +5009,7 @@ class VelocityClient {
         return addIfStakeIxs;
     }
     async requestRemoveInsuranceFundStake(marketIndex, amount, txParams) {
-        const spotMarketAccount = this.getSpotMarketAccount(marketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(marketIndex);
         const ifStakeAccountPublicKey = (0, pda_1.getInsuranceFundStakeAccountPublicKey)(this.program.programId, this.wallet.publicKey, marketIndex);
         const ix = await this.program.instruction.requestRemoveInsuranceFundStake(marketIndex, amount, {
             accounts: {
@@ -4888,7 +5027,7 @@ class VelocityClient {
         return txSig;
     }
     async cancelRequestRemoveInsuranceFundStake(marketIndex, txParams) {
-        const spotMarketAccount = this.getSpotMarketAccount(marketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(marketIndex);
         const ifStakeAccountPublicKey = (0, pda_1.getInsuranceFundStakeAccountPublicKey)(this.program.programId, this.wallet.publicKey, marketIndex);
         const ix = await this.program.instruction.cancelRequestRemoveInsuranceFundStake(marketIndex, {
             accounts: {
@@ -4907,7 +5046,7 @@ class VelocityClient {
     }
     async removeInsuranceFundStake(marketIndex, collateralAccountPublicKey, txParams) {
         const removeIfStakeIxs = [];
-        const spotMarketAccount = this.getSpotMarketAccount(marketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(marketIndex);
         const ifStakeAccountPublicKey = (0, pda_1.getInsuranceFundStakeAccountPublicKey)(this.program.programId, this.wallet.publicKey, marketIndex);
         const additionalSigners = [];
         const isSolMarket = spotMarketAccount.mint.equals(spotMarkets_1.WRAPPED_SOL_MINT);
@@ -4966,7 +5105,7 @@ class VelocityClient {
     }
     async getUpdateUserQuoteAssetInsuranceStakeIx(authority) {
         const marketIndex = numericConstants_1.QUOTE_SPOT_MARKET_INDEX;
-        const spotMarket = this.getSpotMarketAccount(marketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(marketIndex);
         const ifStakeAccountPublicKey = (0, pda_1.getInsuranceFundStakeAccountPublicKey)(this.program.programId, authority, marketIndex);
         const userStatsPublicKey = (0, pda_1.getUserStatsAccountPublicKey)(this.program.programId, authority);
         const ix = this.program.instruction.updateUserQuoteAssetInsuranceStake({
@@ -4981,36 +5120,13 @@ class VelocityClient {
         });
         return ix;
     }
-    async updateUserGovTokenInsuranceStake(authority, txParams) {
-        const ix = await this.getUpdateUserGovTokenInsuranceStakeIx(authority);
-        const tx = await this.buildTransaction(ix, txParams);
-        const { txSig } = await this.sendTransaction(tx, [], this.opts);
-        return txSig;
-    }
-    async getUpdateUserGovTokenInsuranceStakeIx(authority) {
-        const marketIndex = numericConstants_1.GOV_SPOT_MARKET_INDEX;
-        const spotMarket = this.getSpotMarketAccount(marketIndex);
-        const ifStakeAccountPublicKey = (0, pda_1.getInsuranceFundStakeAccountPublicKey)(this.program.programId, authority, marketIndex);
-        const userStatsPublicKey = (0, pda_1.getUserStatsAccountPublicKey)(this.program.programId, authority);
-        const ix = this.program.instruction.updateUserGovTokenInsuranceStake({
-            accounts: {
-                state: await this.getStatePublicKey(),
-                spotMarket: spotMarket.pubkey,
-                insuranceFundStake: ifStakeAccountPublicKey,
-                userStats: userStatsPublicKey,
-                signer: this.wallet.publicKey,
-                insuranceFundVault: spotMarket.insuranceFund.vault,
-            },
-        });
-        return ix;
-    }
     async settleRevenueToInsuranceFund(spotMarketIndex, txParams) {
         const tx = await this.buildTransaction(await this.getSettleRevenueToInsuranceFundIx(spotMarketIndex), txParams);
         const { txSig } = await this.sendTransaction(tx, [], this.opts);
         return txSig;
     }
     async getSettleRevenueToInsuranceFundIx(spotMarketIndex) {
-        const spotMarketAccount = this.getSpotMarketAccount(spotMarketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(spotMarketIndex);
         const tokenProgramId = this.getTokenProgramForSpotMarket(spotMarketAccount);
         const remainingAccounts = [];
         this.addTokenMintToRemainingAccounts(spotMarketAccount, remainingAccounts);
@@ -5030,17 +5146,35 @@ class VelocityClient {
         });
         return ix;
     }
+    async sweepPerpMarketFees(perpMarketIndex, txParams) {
+        const tx = await this.buildTransaction(await this.getSweepPerpMarketFeesIx(perpMarketIndex), txParams);
+        const { txSig } = await this.sendTransaction(tx, [], this.opts);
+        return txSig;
+    }
+    async getSweepPerpMarketFeesIx(perpMarketIndex) {
+        const perpMarketAccount = this.getPerpMarketAccountOrThrow(perpMarketIndex);
+        const spotMarketAccount = this.getSpotMarketAccountOrThrow(perpMarketAccount.quoteSpotMarketIndex);
+        const ix = await this.program.instruction.sweepPerpMarketFees(perpMarketIndex, {
+            accounts: {
+                state: await this.getStatePublicKey(),
+                perpMarket: perpMarketAccount.pubkey,
+                spotMarket: spotMarketAccount.pubkey,
+                oracle: perpMarketAccount.oracle,
+            },
+        });
+        return ix;
+    }
     async resolvePerpPnlDeficit(spotMarketIndex, perpMarketIndex, txParams) {
         const { txSig } = await this.sendTransaction(await this.buildTransaction(await this.getResolvePerpPnlDeficitIx(spotMarketIndex, perpMarketIndex), txParams), [], this.opts);
         return txSig;
     }
     async getResolvePerpPnlDeficitIx(spotMarketIndex, perpMarketIndex) {
         const remainingAccounts = this.getRemainingAccounts({
-            userAccounts: [this.getUserAccount()],
+            userAccounts: [],
             writablePerpMarketIndexes: [perpMarketIndex],
             writableSpotMarketIndexes: [spotMarketIndex],
         });
-        const spotMarket = this.getSpotMarketAccount(spotMarketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(spotMarketIndex);
         const tokenProgramId = this.getTokenProgramForSpotMarket(spotMarket);
         return await this.program.instruction.resolvePerpPnlDeficit(spotMarketIndex, perpMarketIndex, {
             accounts: {
@@ -5055,7 +5189,7 @@ class VelocityClient {
         });
     }
     async getDepositIntoSpotMarketRevenuePoolIx(marketIndex, amount, userTokenAccountPublicKey) {
-        const spotMarket = await this.getSpotMarketAccount(marketIndex);
+        const spotMarket = await this.getSpotMarketAccountOrThrow(marketIndex);
         const remainingAccounts = [];
         this.addTokenMintToRemainingAccounts(spotMarket, remainingAccounts);
         if (this.isTransferHook(spotMarket)) {
@@ -5089,8 +5223,8 @@ class VelocityClient {
     }
     getPerpMarketExtendedInfo(marketIndex) {
         var _a, _b;
-        const marketAccount = this.getPerpMarketAccount(marketIndex);
-        const quoteAccount = this.getSpotMarketAccount(numericConstants_1.QUOTE_SPOT_MARKET_INDEX);
+        const marketAccount = this.getPerpMarketAccountOrThrow(marketIndex);
+        const quoteAccount = this.getSpotMarketAccountOrThrow(numericConstants_1.QUOTE_SPOT_MARKET_INDEX);
         const extendedInfo = {
             marketIndex,
             minOrderSize: (_a = marketAccount.marketStats) === null || _a === void 0 ? void 0 : _a.minOrderSize,
@@ -5121,12 +5255,12 @@ class VelocityClient {
         let takerFee = feeTier.feeNumerator / feeTier.feeDenominator;
         let makerFee = feeTier.makerRebateNumerator / feeTier.makerRebateDenominator;
         if (marketIndex !== undefined) {
-            let marketAccount = null;
+            let marketAccount;
             if ((0, types_1.isVariant)(marketType, 'perp')) {
-                marketAccount = this.getPerpMarketAccount(marketIndex);
+                marketAccount = this.getPerpMarketAccountOrThrow(marketIndex);
             }
             else {
-                marketAccount = this.getSpotMarketAccount(marketIndex);
+                marketAccount = this.getSpotMarketAccountOrThrow(marketIndex);
             }
             takerFee += (takerFee * marketAccount.feeAdjustment) / 100;
             makerFee += (makerFee * marketAccount.feeAdjustment) / 100;
@@ -5189,7 +5323,7 @@ class VelocityClient {
         return [verifyIx, ix];
     }
     async getPauseSpotMarketDepositWithdrawIx(spotMarketIndex) {
-        const spotMarket = await this.getSpotMarketAccount(spotMarketIndex);
+        const spotMarket = await this.getSpotMarketAccountOrThrow(spotMarketIndex);
         return this.program.instruction.pauseSpotMarketDepositWithdraw({
             accounts: {
                 state: await this.getStatePublicKey(),
@@ -5223,7 +5357,7 @@ class VelocityClient {
             programId: this.program.programId,
             keys: [
                 {
-                    pubkey: this.getPerpMarketAccount(marketIndex).pubkey,
+                    pubkey: this.getPerpMarketAccountOrThrow(marketIndex).pubkey,
                     isWritable: true,
                     isSigner: false,
                 },
@@ -5266,7 +5400,7 @@ class VelocityClient {
             programId: this.program.programId,
             keys: [
                 {
-                    pubkey: this.getPerpMarketAccount(marketIndex).pubkey,
+                    pubkey: this.getPerpMarketAccountOrThrow(marketIndex).pubkey,
                     isWritable: true,
                     isSigner: false,
                 },
@@ -5360,7 +5494,7 @@ class VelocityClient {
                 state: await this.getStatePublicKey(),
                 keeper: this.wallet.publicKey,
                 ammCache: (0, pda_1.getAmmCachePublicKey)(this.program.programId),
-                quoteMarket: this.getSpotMarketAccount(0).pubkey,
+                quoteMarket: this.getSpotMarketAccountOrThrow(0).pubkey,
             },
             remainingAccounts,
         });
@@ -5370,7 +5504,7 @@ class VelocityClient {
         return txSig;
     }
     async getUpdateConstituentOracleInfoIx(constituent) {
-        const spotMarket = this.getSpotMarketAccount(constituent.spotMarketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(constituent.spotMarketIndex);
         return this.program.instruction.updateConstituentOracleInfo({
             accounts: {
                 keeper: this.wallet.publicKey,
@@ -5392,12 +5526,12 @@ class VelocityClient {
         });
         const constituentInTokenAccount = (0, pda_1.getConstituentVaultPublicKey)(this.program.programId, lpPool, inMarketIndex);
         const constituentOutTokenAccount = (0, pda_1.getConstituentVaultPublicKey)(this.program.programId, lpPool, outMarketIndex);
-        const userInTokenAccount = await (0, spl_token_1.getAssociatedTokenAddress)(this.getSpotMarketAccount(inMarketIndex).mint, userAuthority);
-        const userOutTokenAccount = await (0, spl_token_1.getAssociatedTokenAddress)(this.getSpotMarketAccount(outMarketIndex).mint, userAuthority);
+        const userInTokenAccount = await (0, spl_token_1.getAssociatedTokenAddress)(this.getSpotMarketAccountOrThrow(inMarketIndex).mint, userAuthority);
+        const userOutTokenAccount = await (0, spl_token_1.getAssociatedTokenAddress)(this.getSpotMarketAccountOrThrow(outMarketIndex).mint, userAuthority);
         const inConstituent = (0, pda_1.getConstituentPublicKey)(this.program.programId, lpPool, inMarketIndex);
         const outConstituent = (0, pda_1.getConstituentPublicKey)(this.program.programId, lpPool, outMarketIndex);
-        const inMarketMint = this.getSpotMarketAccount(inMarketIndex).mint;
-        const outMarketMint = this.getSpotMarketAccount(outMarketIndex).mint;
+        const inMarketMint = this.getSpotMarketAccountOrThrow(inMarketIndex).mint;
+        const outMarketMint = this.getSpotMarketAccountOrThrow(outMarketIndex).mint;
         const constituentTargetBase = (0, pda_1.getConstituentTargetBasePublicKey)(this.program.programId, lpPool);
         return this.program.instruction.lpPoolSwap(inMarketIndex, outMarketIndex, inAmount, minOutAmount, {
             remainingAccounts,
@@ -5469,7 +5603,7 @@ class VelocityClient {
             userAccounts: [],
             writableSpotMarketIndexes: [inMarketIndex],
         });
-        const spotMarket = this.getSpotMarketAccount(inMarketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(inMarketIndex);
         const inMarketMint = spotMarket.mint;
         const isSolMarket = inMarketMint.equals(spotMarkets_1.WRAPPED_SOL_MINT);
         let wSolTokenAccount;
@@ -5531,7 +5665,7 @@ class VelocityClient {
             userAccounts: [],
             readableSpotMarketIndexes: [inMarketIndex],
         });
-        const spotMarket = this.getSpotMarketAccount(inMarketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(inMarketIndex);
         const inMarketMint = spotMarket.mint;
         const inConstituent = (0, pda_1.getConstituentPublicKey)(this.program.programId, lpPool.pubkey, inMarketIndex);
         const lpMint = lpPool.mint;
@@ -5564,7 +5698,7 @@ class VelocityClient {
             userAccounts: [],
             writableSpotMarketIndexes: [outMarketIndex],
         });
-        const spotMarket = this.getSpotMarketAccount(outMarketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(outMarketIndex);
         const outMarketMint = spotMarket.mint;
         const outConstituent = (0, pda_1.getConstituentPublicKey)(this.program.programId, lpPool.pubkey, outMarketIndex);
         if (outMarketMint.equals(spotMarkets_1.WRAPPED_SOL_MINT)) {
@@ -5610,7 +5744,7 @@ class VelocityClient {
             userAccounts: [],
             writableSpotMarketIndexes: [outMarketIndex],
         });
-        const spotMarket = this.getSpotMarketAccount(outMarketIndex);
+        const spotMarket = this.getSpotMarketAccountOrThrow(outMarketIndex);
         const outMarketMint = spotMarket.mint;
         const outConstituent = (0, pda_1.getConstituentPublicKey)(this.program.programId, lpPool.pubkey, outMarketIndex);
         const lpMint = lpPool.mint;
@@ -5710,7 +5844,7 @@ class VelocityClient {
         const remainingAccounts = [];
         remainingAccounts.push(...perpMarketIndexes.map((index) => {
             return {
-                pubkey: this.getPerpMarketAccount(index).pubkey,
+                pubkey: this.getPerpMarketAccountOrThrow(index).pubkey,
                 isSigner: false,
                 isWritable: true,
             };
@@ -5824,7 +5958,7 @@ class VelocityClient {
         });
     }
     isOrderIncreasingPosition(orderParams, subAccountId) {
-        const userAccount = this.getUserAccount(subAccountId);
+        const userAccount = this.getUserAccountOrThrow(subAccountId);
         const perpPosition = userAccount.perpPositions.find((p) => p.marketIndex === orderParams.marketIndex);
         if (!perpPosition)
             return true;