@spacemolt/lib 14.0.0 → 14.0.1

package/dist/account.d.ts

@@ -380,27 +380,36 @@ export declare class Account {
      */
     private checkSubscriptionsAgainstLocation;
     /**
-     * Fleet followers do not submit the movement mutation themselves, so their
-     * arrival is delivered as a plain `ok` push rather than an `action_result`
-     * carrying a state delta. Bridge the location data present on those pushes
-     * into the cache. Fleet docking is the one exception: its push only contains
-     * the station's display name, not the canonical base ID used by `docked_at`,
-     * so refresh the canonical snapshot instead of caching the wrong identifier.
+     * Movement that the state delta does not cover arrives as a plain `ok` push.
+     * Bridge its location data into the cache.
+     *
+     * Two kinds of movement land here. **Transit start** (`travel`/`jump`, and
+     * their `fleet_` variants) is never delivered as a delta by design: the
+     * server holds one `action_result` back for the arrival so it can settle the
+     * mutation on the same `request_id`, and sends this push in the meantime.
+     * Everything the departure changed is either on the push or implied by it,
+     * so these patch and stop — the one field they cannot carry is `ship.fuel`,
+     * which the arrival delta corrects a transit later. **Fleet follower
+     * arrival** is the other kind: a follower never submitted the mutation, so
+     * on a server that cannot tell their connection speaks v2 there is no delta
+     * at all, and only then is a reconcile worth its round trip.
      *
      * These pushes carry only the few fields the movement itself changed, so
      * they go through `patchSection` — unlike a server delta, which always
-     * carries a complete `location` and is applied with `applyDelta`. Patching
-     * keeps the rest of the section (system, connections, POI details) intact
-     * until `reconcileFleetState` refreshes the authoritative snapshot; the
-     * per-POI fields that the move invalidates are only as stale as that
-     * round trip.
+     * carries a complete `location` and is applied with `applyDelta`.
      */
-    private applyFleetMovementPush;
+    private applyMovementPush;
     /**
-     * Fleet movement pushes omit follower fuel and skill deltas, while docking
-     * omits the canonical base ID. Reconcile those fields from one authoritative
-     * snapshot. A newer movement push invalidates an older in-flight query so a
-     * delayed dock/arrival snapshot cannot move the cache backwards.
+     * Refill a fleet follower's location from one authoritative snapshot.
+     *
+     * An arrival push carries the destination's name and little else, so the
+     * system-, POI- and presence-scoped fields it invalidates have nothing to
+     * refill them and no formula to derive them. This is the one movement gap a
+     * round trip is the only answer to, and only on a server that does not send
+     * followers a delta — a current one does, and these branches never run.
+     *
+     * A newer movement push invalidates an older in-flight query so a delayed
+     * snapshot cannot move the cache backwards.
      */
     private reconcileFleetState;
     /** Close the connection deliberately. Suppresses auto-reconnect. */

package/dist/index.js

@@ -4189,6 +4189,8 @@ var ARRIVED_IN_NEW_SYSTEM = {
   empire: undefined,
   security_status: undefined
 };
+var DEPARTED = { ...ARRIVED, poi_id: "", poi_name: "", poi_type: "" };
+var DEPARTED_SYSTEM = { ...ARRIVED_IN_NEW_SYSTEM, ...DEPARTED, system_id: "", system_name: "" };
 var DEFAULT_URL = "wss://game.spacemolt.com/ws/v2";
 var DEFAULT_CONNECT_TIMEOUT_MS = 15000;
 var DEFAULT_MUTATION_TIMEOUT_MS = 600000;
@@ -4601,25 +4603,26 @@ class Account {
       this.subscribedObservationPoiId = undefined;
     }
   }
-  applyFleetMovementPush(frame) {
+  applyMovementPush(frame) {
     if (frame.type !== "ok" || !isRecord(frame.payload) || typeof frame.payload.action !== "string")
       return;
     const payload = frame.payload;
     let changed = [];
     let reconcile = false;
     switch (payload.action) {
+      case "jump":
       case "fleet_jump": {
         if (typeof payload.destination !== "string" || typeof payload.arrival_tick !== "number") {
           this.warnMalformedFrame(frame);
           return;
         }
         changed = this.cache.patchSection("location", {
+          ...DEPARTED_SYSTEM,
           in_transit: true,
           transit_arrival_tick: payload.arrival_tick,
           transit_dest_system_id: payload.destination,
           transit_type: "jump"
         });
-        reconcile = true;
         break;
       }
       case "jumped":
@@ -4637,18 +4640,19 @@ class Account {
         reconcile = true;
         break;
       }
+      case "travel":
       case "fleet_travel": {
         if (typeof payload.destination !== "string" || typeof payload.arrival_tick !== "number") {
           this.warnMalformedFrame(frame);
           return;
         }
         changed = this.cache.patchSection("location", {
+          ...DEPARTED,
           in_transit: true,
           transit_arrival_tick: payload.arrival_tick,
           transit_dest_poi_id: payload.destination,
           transit_type: "travel"
         });
-        reconcile = true;
         break;
       }
       case "arrived": {
@@ -4668,7 +4672,11 @@ class Account {
         changed = this.cache.patchSection("location", { docked_at: null });
         break;
       case "fleet_dock":
-        reconcile = true;
+        if (typeof payload.base_id === "string") {
+          changed = this.cache.patchSection("location", { docked_at: payload.base_id });
+        } else {
+          reconcile = true;
+        }
         break;
       default:
         return;
@@ -4824,7 +4832,9 @@ class Account {
             this.emitStateChange(changed);
         }
         if (!this.correlator.handle(frame)) {
-          console.warn(`[spacemolt] action_result for ${frame.request_id ?? "?"} arrived with no matching pending mutation (already timed out?)`);
+          if (frame.request_id) {
+            console.warn(`[spacemolt] action_result for ${frame.request_id} arrived with no matching pending mutation (already timed out?)`);
+          }
           this.emitter.emit(frame);
         }
         return;
@@ -4852,7 +4862,7 @@ class Account {
         return;
       }
       default:
-        this.applyFleetMovementPush(frame);
+        this.applyMovementPush(frame);
         this.emitter.emit(frame);
     }
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spacemolt/lib",
-  "version": "14.0.0",
+  "version": "14.0.1",
   "description": "The definitive TypeScript library for SpaceMolt. WebSocket-v2-first, multi-account, with local state caches. Internals are regenerated from the server's OpenAPI spec.",
   "type": "module",
   "module": "./dist/index.js",