@redflow/client 0.0.4 → 0.0.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@redflow/client",
-  "version": "0.0.4",
+  "version": "0.0.5",
   "type": "module",
   "main": "src/index.ts",
   "module": "src/index.ts",

package/src/client.ts

@@ -16,16 +16,17 @@ import { keys } from "./internal/keys";
 import { safeJsonParse, safeJsonStringify, safeJsonTryParse } from "./internal/json";
 import { nowMs } from "./internal/time";
 import { sleep } from "./internal/sleep";
-import type {
-  EmitWorkflowOptions,
-  ListedRun,
-  ListRunsParams,
-  RunHandle,
-  RunOptions,
-  RunState,
-  RunStatus,
-  StepState,
-  WorkflowMeta,
+import {
+  DEFAULT_MAX_ATTEMPTS,
+  type EmitWorkflowOptions,
+  type ListedRun,
+  type ListRunsParams,
+  type RunHandle,
+  type RunOptions,
+  type RunState,
+  type RunStatus,
+  type StepState,
+  type WorkflowMeta,
 } from "./types";
 import type { WorkflowRegistry } from "./registry";
 
@@ -392,7 +393,7 @@ export class RedflowClient {
         ? Math.floor(options.__maxAttemptsOverride)
         : null;
 
-    const maxAttempts = maxAttemptsOverride ?? (await this.getMaxAttemptsForWorkflow(workflowName)) ?? 1;
+    const maxAttempts = maxAttemptsOverride ?? (await this.getMaxAttemptsForWorkflow(workflowName)) ?? DEFAULT_MAX_ATTEMPTS;
 
     return await this.enqueueRun<TOutput>({
       workflowName,
@@ -468,7 +469,7 @@ export class RedflowClient {
       output,
       error,
       attempt: Number(data.attempt ?? "0"),
-      maxAttempts: Number(data.maxAttempts ?? "1"),
+      maxAttempts: Number(data.maxAttempts ?? String(DEFAULT_MAX_ATTEMPTS)),
       createdAt: Number(data.createdAt ?? "0"),
       availableAt: data.availableAt ? Number(data.availableAt) : undefined,
       startedAt: data.startedAt ? Number(data.startedAt) : undefined,

package/src/types.ts

@@ -12,8 +12,10 @@ export type CronTrigger = {
   id?: string;
 };
 
+export const DEFAULT_MAX_ATTEMPTS = 3;
+
 export type WorkflowRetries = {
-  /** Total attempts including the first one. Default: 1 (no retries). */
+  /** Total attempts including the first one. Default: 3. */
   maxAttempts?: number;
 };
 

package/src/worker.ts

@@ -16,7 +16,7 @@ import { safeJsonParse, safeJsonStringify, safeJsonTryParse } from "./internal/j
 import { sleep } from "./internal/sleep";
 import { nowMs } from "./internal/time";
 import { getDefaultRegistry, type WorkflowRegistry } from "./registry";
-import type { OnFailureContext, RunStatus, StepApi, StepStatus } from "./types";
+import { DEFAULT_MAX_ATTEMPTS, type OnFailureContext, type RunStatus, type StepApi, type StepStatus } from "./types";
 
 export type StartWorkerOptions = {
   /** Stable application id used for registry sync stale-cleanup scoping. */
@@ -419,7 +419,7 @@ async function processRun(args: {
     const workflowName = run.workflow ?? "";
     const def = workflowName ? registry.get(workflowName) : undefined;
     const maxConcurrency = normalizeMaxConcurrency(def?.options.maxConcurrency);
-    const maxAttempts = Number(run.maxAttempts ?? "1");
+    const maxAttempts = Number(run.maxAttempts ?? String(DEFAULT_MAX_ATTEMPTS));
     const cancelRequestedAt = run.cancelRequestedAt ? Number(run.cancelRequestedAt) : 0;
     if (cancelRequestedAt > 0) {
       await client.finalizeRun(runId, { status: "canceled", finishedAt: nowMs() });