@hatchet-dev/typescript-sdk 1.19.2-alpha.0 → 1.19.2-alpha.2

package/package.json

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

package/v1/client/worker/context.d.ts

@@ -81,8 +81,10 @@ export declare class Context<T, K = {}> {
     worker: ContextWorker;
     overridesData: Record<string, any>;
     _logger: Logger;
+    /** @deprecated — kept for backward compat; prefer {@link nextChildIndex}. */
     spawnIndex: number;
     streamIndex: number;
+    protected nextChildIndex(n?: number): number;
     constructor(action: Action, v1: HatchetClient, worker: InternalWorker);
     get abortController(): AbortController;
     get cancelled(): boolean;

package/v1/client/worker/context.js

@@ -30,6 +30,7 @@ const transformer_1 = require("../../conditions/transformer");
 const condition_1 = require("../../../protoc/v1/shared/condition");
 const apply_namespace_1 = require("../../../util/apply-namespace");
 const abort_error_1 = require("../../../util/abort-error");
+const parent_run_context_vars_1 = require("../../parent-run-context-vars");
 const crypto_1 = require("crypto");
 const duration_1 = require("../duration");
 const engine_version_1 = require("./engine-version");
@@ -74,10 +75,19 @@ class ContextWorker {
 }
 exports.ContextWorker = ContextWorker;
 class Context {
+    nextChildIndex(n = 1) {
+        var _a;
+        const ctx = parent_run_context_vars_1.parentRunContextManager.getContext();
+        const idx = (_a = ctx === null || ctx === void 0 ? void 0 : ctx.childIndex) !== null && _a !== void 0 ? _a : this.spawnIndex;
+        parent_run_context_vars_1.parentRunContextManager.incrementChildIndex(n);
+        this.spawnIndex = idx + n;
+        return idx;
+    }
     constructor(action, v1, worker) {
         // @deprecated use ctx.abortController instead
         this.controller = new AbortController();
         this.overridesData = {};
+        /** @deprecated — kept for backward compat; prefer {@link nextChildIndex}. */
         this.spawnIndex = 0;
         this.streamIndex = 0;
         try {
@@ -379,8 +389,8 @@ class Context {
             throw new hatchet_error_1.default(`Cannot run with sticky: workflow ${workflowName} is not registered on the worker`);
         }
         const { workflowRunId, taskRunExternalId } = this.action;
-        const finalOpts = Object.assign(Object.assign({}, opts), { parentId: workflowRunId, parentTaskRunExternalId: taskRunExternalId, childIndex: this.spawnIndex, childKey: options === null || options === void 0 ? void 0 : options.key, desiredWorkerId: sticky ? this.worker.id() : undefined, _standaloneTaskName: workflow instanceof declaration_1.TaskWorkflowDeclaration ? workflow._standalone_task_name : undefined });
-        this.spawnIndex += 1;
+        const childIndex = this.nextChildIndex();
+        const finalOpts = Object.assign(Object.assign({}, opts), { parentId: workflowRunId, parentTaskRunExternalId: taskRunExternalId, childIndex, childKey: options === null || options === void 0 ? void 0 : options.key, desiredWorkerId: sticky ? this.worker.id() : undefined, _standaloneTaskName: workflow instanceof declaration_1.TaskWorkflowDeclaration ? workflow._standalone_task_name : undefined });
         return { workflowName, opts: finalOpts };
     }
     spawn(workflow, input, options) {
@@ -561,12 +571,12 @@ class Context {
                 // `signal` must never be sent over the wire.
                 const optsWithoutSignal = Object.assign({}, opts);
                 delete optsWithoutSignal.signal;
+                const childIndex = this.nextChildIndex();
                 const resp = {
                     workflowName: name,
                     input,
-                    options: Object.assign(Object.assign({}, optsWithoutSignal), { parentId: workflowRunId, parentTaskRunExternalId: taskRunExternalId, childIndex: this.spawnIndex, desiredWorkerId: sticky ? this.worker.id() : undefined }),
+                    options: Object.assign(Object.assign({}, optsWithoutSignal), { parentId: workflowRunId, parentTaskRunExternalId: taskRunExternalId, childIndex, desiredWorkerId: sticky ? this.worker.id() : undefined }),
                 };
-                this.spawnIndex += 1;
                 return resp;
             });
             try {
@@ -614,8 +624,8 @@ class Context {
                 throw new hatchet_error_1.default(`cannot run with sticky: workflow ${name} is not registered on the worker`);
             }
             try {
-                const resp = yield this.v1.admin.runWorkflow(name, input, Object.assign({ parentId: workflowRunId, parentTaskRunExternalId: taskRunExternalId, childIndex: this.spawnIndex, desiredWorkerId: sticky ? this.worker.id() : undefined }, opts));
-                this.spawnIndex += 1;
+                const childIndex = this.nextChildIndex();
+                const resp = yield this.v1.admin.runWorkflow(name, input, Object.assign({ parentId: workflowRunId, parentTaskRunExternalId: taskRunExternalId, childIndex, desiredWorkerId: sticky ? this.worker.id() : undefined }, opts));
                 if (workflow instanceof declaration_1.TaskWorkflowDeclaration) {
                     resp._standaloneTaskName = workflow._standalone_task_name;
                 }
@@ -796,12 +806,13 @@ class DurableContext extends Context {
             workflowName = workflow.name;
         }
         workflowName = (0, apply_namespace_1.applyNamespace)(workflowName, this.v1.config.namespace).toLowerCase();
+        const childIndex = this.nextChildIndex();
         const triggerOpts = {
             name: workflowName,
             input: JSON.stringify(input || {}),
             parentId: this.action.workflowRunId,
             parentTaskRunExternalId: this.action.taskRunExternalId,
-            childIndex: this.spawnIndex,
+            childIndex,
             childKey: options === null || options === void 0 ? void 0 : options.key,
             additionalMetadata: (options === null || options === void 0 ? void 0 : options.additionalMetadata)
                 ? JSON.stringify(options.additionalMetadata)
@@ -810,7 +821,6 @@ class DurableContext extends Context {
             priority: options === null || options === void 0 ? void 0 : options.priority,
             desiredWorkerLabels: {},
         };
-        this.spawnIndex += 1;
         return { workflowName, triggerOpts };
     }
     /**

package/v1/declaration.js

@@ -64,11 +64,6 @@ class BaseWorkflowDeclaration {
                 throw UNBOUND_ERR;
             }
             const parentRunContext = parent_run_context_vars_1.parentRunContextManager.getContext();
-            // Snapshot the child index BEFORE incrementing — getContext() returns a mutable
-            // reference, so any later increment would corrupt this value.
-            const childIndex = parentRunContext === null || parentRunContext === void 0 ? void 0 : parentRunContext.childIndex;
-            const inputCount = Array.isArray(input) ? input.length : 1;
-            parent_run_context_vars_1.parentRunContextManager.incrementChildIndex(inputCount);
             if (!parentRunContext && ((options === null || options === void 0 ? void 0 : options.childKey) || (options === null || options === void 0 ? void 0 : options.sticky))) {
                 this.client.admin.logger.warn('ignoring childKey or sticky because run is not being spawned from a parent task');
             }
@@ -82,19 +77,21 @@ class BaseWorkflowDeclaration {
                     : undefined,
                 warn: (message) => this.client.admin.logger.warn(message),
             });
-            const runOpts = Object.assign(Object.assign({}, (options !== null && options !== void 0 ? options : {})), { parentId: parentRunContext === null || parentRunContext === void 0 ? void 0 : parentRunContext.parentId, parentTaskRunExternalId: parentRunContext === null || parentRunContext === void 0 ? void 0 : parentRunContext.parentTaskRunExternalId, childIndex, sticky: (options === null || options === void 0 ? void 0 : options.sticky) ? parentRunContext === null || parentRunContext === void 0 ? void 0 : parentRunContext.desiredWorkerId : undefined, childKey: options === null || options === void 0 ? void 0 : options.childKey });
+            // Snapshot childIndex before incrementing — shared with Context.spawnIndex
+            // via parentRunContextManager so both spawning APIs produce unique indices.
+            const baseChildIndex = parentRunContext === null || parentRunContext === void 0 ? void 0 : parentRunContext.childIndex;
+            const inputCount = Array.isArray(input) ? input.length : 1;
+            parent_run_context_vars_1.parentRunContextManager.incrementChildIndex(inputCount);
+            const runOpts = Object.assign(Object.assign({}, (options !== null && options !== void 0 ? options : {})), { parentId: parentRunContext === null || parentRunContext === void 0 ? void 0 : parentRunContext.parentId, parentTaskRunExternalId: parentRunContext === null || parentRunContext === void 0 ? void 0 : parentRunContext.parentTaskRunExternalId, childIndex: baseChildIndex, sticky: (options === null || options === void 0 ? void 0 : options.sticky) ? parentRunContext === null || parentRunContext === void 0 ? void 0 : parentRunContext.desiredWorkerId : undefined, childKey: options === null || options === void 0 ? void 0 : options.childKey });
             if (Array.isArray(input)) {
                 let resp = [];
                 for (let i = 0; i < input.length; i += 500) {
                     const batch = input.slice(i, i + 500);
-                    const batchResp = yield this.client.admin.runWorkflows(batch.map((inp, batchIdx) => {
-                        var _a;
-                        return ({
-                            workflowName: this.definition.name,
-                            input: inp,
-                            options: Object.assign(Object.assign({}, runOpts), { childIndex: ((_a = runOpts.childIndex) !== null && _a !== void 0 ? _a : 0) + i + batchIdx }),
-                        });
-                    }));
+                    const batchResp = yield this.client.admin.runWorkflows(batch.map((inp, batchIdx) => ({
+                        workflowName: this.definition.name,
+                        input: inp,
+                        options: Object.assign(Object.assign({}, runOpts), { childIndex: (baseChildIndex !== null && baseChildIndex !== void 0 ? baseChildIndex : 0) + i + batchIdx }),
+                    })));
                     resp = resp.concat(batchResp);
                 }
                 const res = [];

package/v1/parent-run-context-vars.d.ts

@@ -1,4 +1,4 @@
-import { DurableContext } from './client/worker/context';
+import type { DurableContext } from './client/worker/context';
 export interface ParentRunContext {
     parentId: string;
     /**

package/v1/parent-run-context-vars.js

@@ -16,15 +16,16 @@ class ParentRunContextManager {
         var _a;
         const parentRunContext = this.getContext();
         if (parentRunContext) {
-            this.setContext(Object.assign(Object.assign({}, parentRunContext), { parentId: opts.parentId, parentTaskRunExternalId: opts.parentTaskRunExternalId, childIndex: ((_a = parentRunContext.childIndex) !== null && _a !== void 0 ? _a : 0) + 1 }));
+            parentRunContext.parentId = opts.parentId;
+            parentRunContext.parentTaskRunExternalId = opts.parentTaskRunExternalId;
+            parentRunContext.childIndex = ((_a = parentRunContext.childIndex) !== null && _a !== void 0 ? _a : 0) + 1;
+            this.setContext(parentRunContext);
         }
     }
     incrementChildIndex(n) {
         var _a;
         const parentRunContext = this.getContext();
         if (parentRunContext) {
-            // Build a fresh object to avoid mutating the live store reference, which
-            // would corrupt any values snapshotted before this call.
             this.setContext(Object.assign(Object.assign({}, parentRunContext), { childIndex: ((_a = parentRunContext.childIndex) !== null && _a !== void 0 ? _a : 0) + n }));
         }
     }

package/version.d.ts

@@ -1 +1 @@
-export declare const HATCHET_VERSION = "1.19.2-alpha.0";
+export declare const HATCHET_VERSION = "1.19.2-alpha.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.19.2-alpha.0';
+exports.HATCHET_VERSION = '1.19.2-alpha.2';