socketduct 0.1.27 → 0.1.28

package/changelog.d/20261025162000-gateway-unavailability-report-is-not-a-timer.md

@@ -0,0 +1 @@
+A hub route-class unavailability report now lasts for the life of the connection instead of five seconds, so a transport set stops re-selecting a route class the hub already declared unserviceable and no longer fails the caller's first open on it.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "socketduct",
-  "version": "0.1.27",
+  "version": "0.1.28",
   "description": "Resilient authenticated tunnel/proxy for keeping socket streams alive across flaky links.",
   "type": "module",
   "main": "./src/index.js",

package/src/client/multiplexed-transport-set.js

@@ -44,7 +44,7 @@ const GATEWAY_ID = /^[A-Za-z0-9][A-Za-z0-9_-]{1,63}$/
  * @property {import('../logging/structured-logger.js').DiagnosticLogger} [logger] - Optional privacy-safe structured logger shared by the set and its members.
  * @typedef {object} MemberConnectionStatus
  * @property {string} physicalState - Member lifecycle.
- * @property {boolean} [gatewayUnavailable] - Hub recently reported no selectable gateway for this member's route class.
+ * @property {boolean} [gatewayUnavailable] - Hub reported no selectable gateway for this member's route class on the current connection.
  * @property {number} connectionGeneration - Current authenticated generation.
  * @property {number} physicalConnects - Authenticated physical generations.
  * @property {number} reconnectAttempts - Physical reconnect attempts.

package/src/reverse/client.js

@@ -32,7 +32,6 @@ import {attachDisconnectAttribution, createDisconnectAttribution, parseDisconnec
 const DEFAULT_STREAM_BUFFER_BYTES = 4 * 1024 * 1024
 const DEFAULT_CONNECTION_BUFFER_BYTES = 32 * 1024 * 1024
 const MANIFEST_PAGE_STREAMS = 64
-const GATEWAY_UNAVAILABLE_TTL_MS = 5_000
 
 /**
  * @typedef {object} ReverseGatewayConnectionOptions
@@ -373,13 +372,18 @@ export class ReverseGatewayConnection extends EventEmitter {
   }
 
   /**
-   * Whether the last hub-reported gateway state for this member was unavailable
-   * and the report is still fresh. A stale report never pins the member out.
-   * @returns {boolean} True while a fresh hub unavailability report is active.
+   * Whether the hub last reported this member's route class as having no selectable
+   * gateway. The report is a property of the live connection, not a short-lived cache:
+   * it arrives once per connection at selection time and only changes when the hub says
+   * otherwise or an open succeeds. Expiring it on a timer made the transport set pick a
+   * route class the hub had already declared unserviceable, and the reset that followed
+   * failed the caller's operation — a build lost its first transport open this way.
+   * The transport set still falls back to marked members when every member is marked, so
+   * a mark can never wedge the set.
+   * @returns {boolean} True while the hub reports no selectable gateway for this member.
    */
   #gatewayUnavailable() {
     return this.#gatewayUnavailableSince !== undefined
-      && Date.now() - this.#gatewayUnavailableSince < GATEWAY_UNAVAILABLE_TTL_MS
   }
 
   status() {

package/test/integration/reverse-route-class.test.js

@@ -436,6 +436,12 @@ describe("reverse gateway route-class", () => {
     expect(client.status().physicalState).toBe("ready")
     expect(client.status().gatewayUnavailable).toBe(true)
 
+    // The report belongs to the live connection: it must survive any short window a caller
+    // might wait between the announce and its first open. A timer-bound mark made the transport
+    // set pick this route class again and the resulting reset failed the caller's operation.
+    await new Promise((resolve) => setTimeout(resolve, 5200))
+    expect({value: client.status().gatewayUnavailable, context: "a hub unavailability report must not expire on a timer"}).toMatchObject({value: true})
+
     // Once the gateway registers, a replacement selection announces selectable=true and clears the mark.
     const gatewayA = new OutboundReverseGateway({
       gatewayId: "server-a-starlink",