@hatchet-dev/typescript-sdk 1.29.1 → 1.29.2

package/clients/dispatcher/heartbeat/heartbeat-severity.d.ts

@@ -1,3 +1,3 @@
 import { FailureSeverity } from '../../../util/failure-severity';
 export declare const MAX_MISSED_HEARTBEATS = 3;
-export declare function classifyHeartbeatFailure(code: number | undefined, missedHeartbeats: number): FailureSeverity;
+export declare function classifyHeartbeatFailure(e: unknown, missedHeartbeats: number): FailureSeverity;

package/clients/dispatcher/heartbeat/heartbeat-severity.js

@@ -6,6 +6,8 @@ const grpc_error_1 = require("../../../util/grpc-error");
 const failure_severity_1 = require("../../../util/failure-severity");
 exports.MAX_MISSED_HEARTBEATS = 3;
 // determines whether to immediately error log or wait for additional errors
-function classifyHeartbeatFailure(code, missedHeartbeats) {
-    return (0, failure_severity_1.classifyRepeatedFailure)((0, grpc_error_1.isConnectionError)(code), missedHeartbeats, exports.MAX_MISSED_HEARTBEATS);
+function classifyHeartbeatFailure(e, missedHeartbeats) {
+    const code = (0, grpc_error_1.getGrpcErrorCode)(e);
+    const isTransient = (0, grpc_error_1.isConnectionError)(code) || (0, grpc_error_1.isTimeoutOrAbortError)(e);
+    return (0, failure_severity_1.classifyRepeatedFailure)(isTransient, missedHeartbeats, exports.MAX_MISSED_HEARTBEATS);
 }

package/clients/dispatcher/heartbeat/heartbeat-worker.js

@@ -20,6 +20,10 @@ const dispatcher_client_1 = require("../dispatcher-client");
 const heartbeat_controller_1 = require("./heartbeat-controller");
 const heartbeat_severity_1 = require("./heartbeat-severity");
 const HEARTBEAT_INTERVAL = 4000;
+// Fail fast on a stalled connection: without a deadline, a heartbeat RPC on a dead
+// TCP connection can hang for 50+ seconds, well past the engine's 30s reassignment
+// window, while setInterval stacks overlapping in-flight requests behind it.
+const HEARTBEAT_TIMEOUT = 5000;
 const postMessage = (message) => {
     worker_threads_1.parentPort === null || worker_threads_1.parentPort === void 0 ? void 0 : worker_threads_1.parentPort.postMessage(message);
 };
@@ -54,7 +58,7 @@ class HeartbeatWorker {
                     yield this.client.heartbeat({
                         workerId: this.workerId,
                         heartbeatAt: new Date(),
-                    });
+                    }, { signal: AbortSignal.timeout(HEARTBEAT_TIMEOUT) });
                     const now = new Date().getTime();
                     const actualInterval = now - this.timeLastHeartbeat;
                     if (actualInterval > HEARTBEAT_INTERVAL * 1.2) {
@@ -88,7 +92,7 @@ class HeartbeatWorker {
                     this.missedHeartbeats += 1;
                     const message = `Failed to send heartbeat: ${(0, hatchet_error_1.getErrorMessage)(e)}`;
                     this.logger.debug(message);
-                    const severity = (0, heartbeat_severity_1.classifyHeartbeatFailure)((0, grpc_error_1.getGrpcErrorCode)(e), this.missedHeartbeats);
+                    const severity = (0, heartbeat_severity_1.classifyHeartbeatFailure)(e, this.missedHeartbeats);
                     if (severity !== 'silent') {
                         postMessage({
                             type: severity,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hatchet-dev/typescript-sdk",
-  "version": "1.29.1",
+  "version": "1.29.2",
   "description": "Background task orchestration & visibility for developers",
   "types": "dist/index.d.ts",
   "files": [

package/util/grpc-error.d.ts

@@ -4,6 +4,13 @@
  */
 export declare const CONNECTION_ERROR_CODES: number[];
 export declare function isConnectionError(code: number | undefined): boolean;
+/**
+ * Returns true if `e` is the DOMException thrown when an AbortSignal (e.g. from
+ * AbortSignal.timeout()) fires. This rejects the underlying call directly rather than
+ * surfacing as a gRPC status code, so it needs to be classified as transient separately
+ * from isConnectionError.
+ */
+export declare function isTimeoutOrAbortError(e: unknown): boolean;
 /**
  * Returns the gRPC status code from an unknown value (e.g. from a catch block).
  * Used for checking Status.CANCELLED, Status.UNAVAILABLE, etc.

package/util/grpc-error.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.CONNECTION_ERROR_CODES = void 0;
 exports.isConnectionError = isConnectionError;
+exports.isTimeoutOrAbortError = isTimeoutOrAbortError;
 exports.getGrpcErrorCode = getGrpcErrorCode;
 exports.getGrpcErrorDetails = getGrpcErrorDetails;
 const nice_grpc_1 = require("nice-grpc");
@@ -13,6 +14,15 @@ exports.CONNECTION_ERROR_CODES = [nice_grpc_1.Status.UNAVAILABLE, nice_grpc_1.St
 function isConnectionError(code) {
     return code !== undefined && exports.CONNECTION_ERROR_CODES.includes(code);
 }
+/**
+ * Returns true if `e` is the DOMException thrown when an AbortSignal (e.g. from
+ * AbortSignal.timeout()) fires. This rejects the underlying call directly rather than
+ * surfacing as a gRPC status code, so it needs to be classified as transient separately
+ * from isConnectionError.
+ */
+function isTimeoutOrAbortError(e) {
+    return e instanceof DOMException && (e.name === 'TimeoutError' || e.name === 'AbortError');
+}
 /**
  * Returns the gRPC status code from an unknown value (e.g. from a catch block).
  * Used for checking Status.CANCELLED, Status.UNAVAILABLE, etc.

package/version.d.ts

@@ -1 +1 @@
-export declare const HATCHET_VERSION = "1.29.1";
+export declare const HATCHET_VERSION = "1.29.2";

package/version.js

@@ -1,4 +1,4 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.HATCHET_VERSION = void 0;
-exports.HATCHET_VERSION = '1.29.1';
+exports.HATCHET_VERSION = '1.29.2';