@temporalio/workflow 1.17.1 → 1.17.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@temporalio/workflow",
-  "version": "1.17.1",
+  "version": "1.17.3",
   "description": "Temporal.io SDK Workflow sub-package",
   "keywords": [
     "temporal",
@@ -20,12 +20,18 @@
   "author": "Temporal Technologies Inc. <sdk@temporal.io>",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
+  "ava": {
+    "timeout": "60s",
+    "concurrency": 1,
+    "workerThreads": false
+  },
   "dependencies": {
     "nexus-rpc": "^0.0.2",
-    "@temporalio/proto": "1.17.1",
-    "@temporalio/common": "1.17.1"
+    "@temporalio/common": "1.17.3",
+    "@temporalio/proto": "1.17.3"
   },
   "devDependencies": {
+    "ava": "^5.3.1",
     "source-map": "^0.7.4"
   },
   "publishConfig": {
@@ -36,7 +42,12 @@
   },
   "files": [
     "src",
-    "lib"
+    "lib",
+    "!src/__tests__",
+    "!lib/__tests__"
   ],
-  "scripts": {}
+  "scripts": {
+    "build": "tsc --build",
+    "test": "ava ./lib/__tests__/test-*.js"
+  }
 }

package/src/current-random.ts

@@ -0,0 +1,5 @@
+import { assertInWorkflowContext } from './global-attributes';
+
+export function currentRandom(): number {
+  return assertInWorkflowContext('Workflow random APIs may only be used from workflow context.').currentRandom();
+}

package/src/flags.ts

@@ -114,7 +114,7 @@ type AltConditionFn = (ctx: { info: WorkflowInfo; sdkVersion?: string }) => bool
 
 function buildIdSdkVersionMatches(version: RegExp): AltConditionFn {
   const regex = new RegExp(`^@temporalio/worker@(${version.source})[+]`);
-  return ({ info }) => info.currentBuildId != null && regex.test(info.currentBuildId); // eslint-disable-line @typescript-eslint/no-deprecated
+  return ({ info }) => info.currentBuildId != null && regex.test(info.currentBuildId);
 }
 
 type SemVer = {

package/src/global-overrides.ts

@@ -5,6 +5,7 @@
  */
 import { msToTs } from '@temporalio/common/lib/time';
 import { CancellationScope } from './cancellation-scope';
+import { currentRandom } from './current-random';
 import { DeterminismViolationError } from './errors';
 import { getActivator } from './global-attributes';
 import { SdkFlags } from './flags';
@@ -103,6 +104,6 @@ export function overrideGlobals(): void {
     }
   };
 
-  // activator.random is mutable, don't hardcode its reference
-  Math.random = () => getActivator().random();
+  // currentRandom() dispatches to a scoped stream when WorkflowRandomStream.with(...) is active.
+  Math.random = currentRandom;
 }

package/src/index.ts

@@ -93,8 +93,8 @@ export {
   PayloadConverter,
   RetryPolicy,
   rootCause,
-  SearchAttributes, // eslint-disable-line @typescript-eslint/no-deprecated
-  SearchAttributeValue, // eslint-disable-line @typescript-eslint/no-deprecated
+  SearchAttributes,
+  SearchAttributeValue,
   ServerFailure,
   TemporalFailure,
   TerminatedFailure,
@@ -136,6 +136,8 @@ export {
   WorkflowInfo,
 } from './interfaces';
 export { proxySinks, Sink, SinkCall, SinkFunction, Sinks } from './sinks';
+export type { WorkflowRandomStream } from './random-streams';
+export { getRandomStream, workflowRandom } from './random-streams';
 export { log } from './logs';
 export { Trigger } from './trigger';
 export * from './workflow';

package/src/interceptor-composition.ts

@@ -0,0 +1,31 @@
+import { composeInterceptorsWith, type Next } from '@temporalio/common/lib/interceptors';
+import { getActivator } from './global-attributes';
+
+/**
+ * Compose workflow interceptors while making every `next(...)` continuation re-enter
+ * the workflow-random scope that was current when that continuation was created.
+ *
+ * This is intentionally a thin wrapper over the shared interceptor composition helper
+ * in `packages/common`. The shared helper owns the actual chain-construction algorithm,
+ * ordering, and `next(...)` wiring so workflow code cannot silently drift away from the
+ * semantics used by the rest of the SDK.
+ *
+ * The workflow-specific behavior lives entirely in the `wrapNext` hook we pass into that
+ * shared helper. Each `next(...)` continuation handed to a workflow interceptor is wrapped
+ * with `Activator.bindCurrentRandom(...)`, which captures the currently active workflow
+ * random scope, including the absence of any scoped override. When the interceptor later
+ * calls `next(...)`, the wrapper restores that captured scope before entering the rest of
+ * the interceptor chain or the base workflow/runtime handler.
+ *
+ * That is the behavior we need for `WorkflowRandomStream.with(...)`: a temporary
+ * plugin/interceptor scope should apply to the plugin's own code, but it must not
+ * leak through `next(...)` into downstream workflow code unless that downstream code
+ * explicitly establishes its own scope.
+ */
+export function composeInterceptors<I, M extends keyof I>(interceptors: I[], method: M, next: Next<I, M>): Next<I, M> {
+  const activator = getActivator();
+  return composeInterceptorsWith(interceptors, method, next, ((wrappedNext) =>
+    activator.bindCurrentRandom(wrappedNext as any)) as <F extends (...args: any[]) => any>(
+    next: F
+  ) => F) as unknown as Next<I, M>;
+}

package/src/interfaces.ts

@@ -47,7 +47,7 @@ export interface WorkflowInfo {
    * This value may change during the lifetime of an Execution.
    * @deprecated Use {@link typedSearchAttributes} instead.
    */
-  readonly searchAttributes: SearchAttributes; // eslint-disable-line @typescript-eslint/no-deprecated
+  readonly searchAttributes: SearchAttributes;
 
   /**
    * Indexed information attached to the Workflow Execution, exposed through an interface.
@@ -337,7 +337,7 @@ export interface ContinueAsNewOptions {
    * Searchable attributes to attach to next Workflow run
    * @deprecated Use {@link typedSearchAttributes} instead.
    */
-  searchAttributes?: SearchAttributes; // eslint-disable-line @typescript-eslint/no-deprecated
+  searchAttributes?: SearchAttributes;
   /**
    * Specifies additional indexed information to attach to the Workflow Execution. More info:
    * https://docs.temporal.io/docs/typescript/search-attributes
@@ -357,7 +357,7 @@ export interface ContinueAsNewOptions {
    *
    * @deprecated Worker Versioning is now deprecated. Please use the Worker Deployment API instead: https://docs.temporal.io/worker-deployments
    */
-  versioningIntent?: VersioningIntent; // eslint-disable-line @typescript-eslint/no-deprecated
+  versioningIntent?: VersioningIntent;
   /**
    * Defines the versioning behavior to be used by the first task of a new workflow run in a continue-as-new chain.
    *
@@ -567,7 +567,7 @@ export interface ChildWorkflowOptions extends Omit<CommonWorkflowOptions, 'workf
    *
    * @deprecated Worker Versioning is now deprecated. Please use the Worker Deployment API instead: https://docs.temporal.io/worker-deployments
    */
-  versioningIntent?: VersioningIntent; // eslint-disable-line @typescript-eslint/no-deprecated
+  versioningIntent?: VersioningIntent;
 }
 
 export type RequiredChildWorkflowOptions = Required<Pick<ChildWorkflowOptions, 'workflowId' | 'cancellationType'>> & {
@@ -646,6 +646,7 @@ export interface WorkflowCreateOptions {
   randomnessSeed: number[];
   now: number;
   showStackTraceSources: boolean;
+  failureExceptionTypeNames?: string[];
 }
 
 export interface WorkflowCreateOptionsInternal extends WorkflowCreateOptions {