@metamask-previews/perps-controller 16.0.0-preview-2d576f0c9 → 16.1.0-preview-7dd72ded9

package/CHANGELOG.md

@@ -7,6 +7,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [16.1.0]
+
+### Fixed
+
+- Revalidate HyperLiquid positions over HTTP before reporting a WebSocket cache miss during a TP/SL update. ([#10101](https://github.com/MetaMask/core/pull/10101))
+  - A current DEX slice is stamped with the connection epoch, which proves it belongs to the live subscription but not that it has caught up with the most recent fill. `updatePositionTPSL` therefore treated the post-fill window, where the position exists on the venue but has not yet been published, as identical to a position that was closed, and returned `POSITION_NOT_FOUND` without making a request.
+  - Only `updatePositionTPSL` opts in. `closePosition`, `closePositions` and the margin operations keep failing closed on a cache miss, because they act on a position the user is looking at: an unpublished position is not rendered, so those flows cannot reach the post-fill window. TP/SL attachment is the one path a client fires automatically within milliseconds of placing an order, with no human in the loop. Keeping the opt-in narrow also keeps the added REST traffic off the batch paths.
+  - A cache hit still returns the WebSocket slice untouched, so a stale size on a symbol that is present is resolved exactly as before, and the reduce-only sizing guarantees added in [#10037](https://github.com/MetaMask/core/pull/10037) are unaffected.
+
 ## [16.0.0]
 
 ### Added
@@ -866,7 +875,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Bump `@metamask/controller-utils` from `^11.18.0` to `^11.19.0` ([#7995](https://github.com/MetaMask/core/pull/7995))
 
-[Unreleased]: https://github.com/MetaMask/core/compare/@metamask/perps-controller@16.0.0...HEAD
+[Unreleased]: https://github.com/MetaMask/core/compare/@metamask/perps-controller@16.1.0...HEAD
+[16.1.0]: https://github.com/MetaMask/core/compare/@metamask/perps-controller@16.0.0...@metamask/perps-controller@16.1.0
 [16.0.0]: https://github.com/MetaMask/core/compare/@metamask/perps-controller@15.1.0...@metamask/perps-controller@16.0.0
 [15.1.0]: https://github.com/MetaMask/core/compare/@metamask/perps-controller@15.0.0...@metamask/perps-controller@15.1.0
 [15.0.0]: https://github.com/MetaMask/core/compare/@metamask/perps-controller@14.0.0...@metamask/perps-controller@15.0.0

package/dist/providers/HyperLiquidProvider.cjs

@@ -2188,7 +2188,7 @@ class HyperLiquidProvider {
             // when available, otherwise query that DEX over HTTP. If neither source
             // answers, fail closed: trusting the snapshot would recreate the stale
             // side/size reduce-only rejection this method is meant to prevent.
-            const currentPositions = await __classPrivateFieldGet(this, _HyperLiquidProvider_instances, "m", _HyperLiquidProvider_getPositionsForOperation).call(this, (0, hyperLiquidAdapter_js_1.parseAssetName)(symbol).dex ?? '');
+            const currentPositions = await __classPrivateFieldGet(this, _HyperLiquidProvider_instances, "m", _HyperLiquidProvider_getPositionsForOperation).call(this, (0, hyperLiquidAdapter_js_1.parseAssetName)(symbol).dex ?? '', { revalidateMissingSymbol: symbol });
             const position = currentPositions.find((pos) => pos.symbol === symbol);
             if (livePosition &&
                 position &&
@@ -10246,8 +10246,11 @@ async function _HyperLiquidProvider_getStandaloneValidatedDexs() {
  *
  * A symbol-specific caller can use its current-connection DEX slice directly.
  * The current-epoch slice wins before any REST request because it is the
- * provider's live subscription source for that exact DEX. If the target DEX
- * has not published, the caller uses REST instead of a stale aggregate.
+ * provider's live subscription source for that exact DEX. A caller may opt
+ * into one targeted REST revalidation when a required symbol is absent, which
+ * covers the post-fill window before the next WebSocket position update. If
+ * the target DEX has not published, the caller uses REST instead of a stale
+ * aggregate.
  *
  * A whole-list caller uses one shallow-copied WebSocket snapshot only when
  * every configured DEX is current. Otherwise it falls back to REST. No path
@@ -10256,15 +10259,31 @@ async function _HyperLiquidProvider_getStandaloneValidatedDexs() {
  *
  * @param targetDex - DEX for a single-symbol lookup, or undefined for a full
  * position list.
+ * @param options - Optional lookup behavior for a symbol-specific operation.
+ * @param options.revalidateMissingSymbol - Query the target DEX over REST when
+ * its current WebSocket slice does not contain this symbol.
  * @returns Positions from one provenance-safe cache/REST path.
  */
-async function _HyperLiquidProvider_getPositionsForOperation(targetDex) {
+async function _HyperLiquidProvider_getPositionsForOperation(targetDex, options = {}) {
     if (targetDex !== undefined) {
         const targetPositions = __classPrivateFieldGet(this, _HyperLiquidProvider_subscriptionService, "f").getCachedPositionsForDex(targetDex);
-        if (targetPositions !== null) {
-            return [...targetPositions];
+        if (targetPositions === null) {
+            __classPrivateFieldGet(this, _HyperLiquidProvider_deps, "f").debugLogger.log('Target DEX position cache unavailable: fetching REST positions', { dex: targetDex || 'main' });
+        }
+        else {
+            const { revalidateMissingSymbol } = options;
+            const shouldRevalidate = revalidateMissingSymbol !== undefined &&
+                targetPositions.find((position) => position.symbol === revalidateMissingSymbol) === undefined;
+            if (shouldRevalidate) {
+                __classPrivateFieldGet(this, _HyperLiquidProvider_deps, "f").debugLogger.log('Target symbol missing from position cache: revalidating REST positions', {
+                    dex: targetDex || 'main',
+                    symbol: revalidateMissingSymbol,
+                });
+            }
+            else {
+                return [...targetPositions];
+            }
         }
-        __classPrivateFieldGet(this, _HyperLiquidProvider_deps, "f").debugLogger.log('Target DEX position cache unavailable: fetching REST positions', { dex: targetDex || 'main' });
         const { answered, positions } = await __classPrivateFieldGet(this, _HyperLiquidProvider_instances, "m", _HyperLiquidProvider_queryDexPositions).call(this, targetDex || null);
         if (!answered) {
             throw new Error(perpsErrorCodes_js_1.PERPS_ERROR_CODES.PROVIDER_NOT_AVAILABLE);