@wichayutdew/pi-workflows 0.2.2 → 0.2.3

package/examples/mr-comments.workflow.yaml

@@ -12,7 +12,7 @@ steps:
     prompt:
       file: prompts/mr-comments/inspect.md
     subagent:
-      agent: pi-workflows.step
+      agent: scout
       context: fresh
       timeoutMs: 600000
       turnBudget:
@@ -33,7 +33,7 @@ steps:
     prompt:
       file: prompts/mr-comments/plan.md
     subagent:
-      agent: pi-workflows.step
+      agent: planner
       context: fresh
       timeoutMs: 600000
       turnBudget:
@@ -65,7 +65,7 @@ steps:
     prompt:
       file: prompts/mr-comments/implement.md
     subagent:
-      agent: pi-workflows.step
+      agent: worker
       context: fresh
       timeoutMs: 1200000
       artifacts: true
@@ -91,7 +91,7 @@ steps:
     prompt:
       file: prompts/mr-comments/verify.md
     subagent:
-      agent: pi-workflows.step
+      agent: reviewer
       context: fresh
       timeoutMs: 600000
       turnBudget:

package/examples/prompts/mr-comments/implement.md

@@ -1,8 +1,13 @@
 Implement the approved plan with a small, coherent diff.
 
 Use repository conventions and the allowed skill. Add or update focused tests
-where practical. Run only exact reviewed worker commands plus the static
-allow-list. Do not perform remote actions. Complete with `ready` when
-implementation and focused checks are complete; include the exact fenced JSON
-contract in the summary for verification. Complete with `blocked` if the
-harness, configuration, or repository must be fixed first.
+where practical. Treat the approved plan as the final contract: do not ask a
+terminal question or block solely because a profile or skill prefers an
+unapproved ceremony or command. Apply TDD with the authorized commands when the
+reviewed contract supports it; otherwise implement and record that constraint.
+Run each exact reviewed worker command with its enclosing
+`repositories[].cwd`, plus the static allow-list. Do not perform remote actions.
+Complete with `ready` when implementation and focused checks are complete;
+include the exact fenced JSON contract in the summary for verification.
+Complete with `blocked` only if the harness, configuration, repository, or
+approved contract must be repaired first.

package/examples/prompts/mr-comments/plan.md

@@ -5,9 +5,14 @@ Include scope, exact files or symbols, risks, and verification. Account for prio
 {{gate.feedback}}
 
 Include one fenced `json` verification contract with a top-level
-`repositories` array. Put exact standalone implementation commands under
-`worker[].command` and independent commands under `reviewer[].command`.
+`repositories` array. Every repository entry must contain its exact absolute
+`cwd`; this workflow requires one distinct repository directory. Put exact
+standalone implementation commands under `worker[].command` and independent
+commands under `reviewer[].command`. Resolve any TDD setup, test selection, or
+command limitation in this reviewed contract; later steps must not invent
+another prerequisite or ask for clarification.
 
-When the plan is ready, call `workflow_complete_step` with outcome `submit` and
-place the full Markdown plan in `artifact`. The Plannotator gate decides the
-next transition; its reviewed artifact becomes the next step's handoff.
+When the plan is ready, call `structured_output` with a `value` whose outcome
+is `submit` and whose `artifact` is the full Markdown plan. The Plannotator gate
+decides the next transition; its reviewed artifact becomes the next step's
+handoff.

package/examples/prompts/mr-comments/verify.md

@@ -1,7 +1,8 @@
 Review the implementation against the requested merge-request feedback and approved plan.
 
-Inspect the diff and run the exact reviewed reviewer commands plus the static
-allow-list. Complete with `passed` only when evidence supports completion.
+Inspect the diff and run each exact reviewed reviewer command with its enclosing
+`repositories[].cwd`, plus the static allow-list. Do not ask a terminal
+question. Complete with `passed` only when evidence supports completion.
 Complete with `failed` to return to implementation and preserve the exact
-fenced JSON contract in the summary, or `blocked` when a human must repair the
-workflow or environment.
+fenced JSON contract in the summary, or `blocked` when the workflow or
+environment must be repaired.

package/examples/settings.yaml

@@ -1,6 +1,7 @@
 # yaml-language-server: $schema=../schemas/settings.schema.json
 version: 1
 allowProjectWorkflows: false
+statusShortcut: ctrl+alt+w
 permissionCeiling:
   tools: [read, grep, bash]
   mcp: []
@@ -9,7 +10,7 @@ permissionCeiling:
   bash:
     mode: read-only
   subagent:
-    agents: [pi-workflows.step, scout, worker, reviewer]
+    agents: [scout, planner, worker, reviewer]
     contexts: [fresh]
     models: []
     maxTimeoutMs: 900000
@@ -17,3 +18,4 @@ permissionCeiling:
     maxGraceTurns: 3
     maxToolCalls: 100
     artifacts: false
+    retryToolFailures: false

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@wichayutdew/pi-workflows",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "description": "A declarative, pauseable workflow harness for Pi",
-  "license": "Apache-2.0",
+  "license": "MIT",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/wichayutdew/pi-workflows.git"
@@ -43,9 +43,12 @@
     "format:check": "bunx --bun prettier . --check",
     "lint": "bun --bun eslint .",
     "lint:fix": "bun --bun eslint . --fix",
-    "test": "bun test",
+    "test": "bun test --no-orphans --reporter=dots",
+    "test:coverage": "bun run test && bun run coverage:check",
+    "coverage:check": "bun .github/scripts/check-coverage.mjs coverage/lcov.info",
+    "test:e2e": "bun --config=bunfig.e2e.toml test --no-orphans test/e2e/workflow-runtime.test.ts --reporter=dots",
     "typecheck": "tsc --noEmit",
-    "check": "bun run lint && bun run format:check && bun run typecheck && bun test && bun run build"
+    "check": "bun run lint && bun run format:check && bun run typecheck && bun run test:coverage && bun run test:e2e && bun run build"
   },
   "dependencies": {
     "yaml": "^2.8.3"
@@ -53,7 +56,7 @@
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*",
     "@earendil-works/pi-tui": "*",
-    "pi-subagents": ">=0.35.1",
+    "pi-subagents": ">=0.36.0",
     "typebox": "*"
   },
   "peerDependenciesMeta": {
@@ -65,10 +68,11 @@
     "@earendil-works/pi-coding-agent": "^0.81.1",
     "@earendil-works/pi-tui": "^0.81.1",
     "@eslint/js": "^10.0.1",
+    "@types/bun": "^1.3.14",
     "@types/node": "^24.0.0",
     "eslint": "^10.7.0",
     "globals": "^17.7.0",
-    "pi-subagents": "^0.35.1",
+    "pi-subagents": "0.36.0",
     "prettier": "^3.9.5",
     "typebox": "^1.1.38",
     "typescript": "^5.9.3",

package/schemas/settings.schema.json

@@ -15,6 +15,14 @@
       "type": "boolean",
       "default": false
     },
+    "statusShortcut": {
+      "type": "string",
+      "default": "ctrl+alt+w",
+      "minLength": 1,
+      "maxLength": 64,
+      "pattern": "^\\S+$",
+      "description": "Pi key identifier used to toggle workflow status. Restart extensions with Pi /reload after changing it."
+    },
     "permissionCeiling": {
       "$ref": "workflow.schema.json#/$defs/permissionCeiling"
     }

package/schemas/workflow.schema.json

@@ -360,7 +360,7 @@
               "default": "pi-workflows.step"
             },
             "context": {
-              "enum": ["fresh", "fork"],
+              "enum": ["fresh"],
               "default": "fresh"
             },
             "model": {
@@ -381,6 +381,10 @@
             "artifacts": {
               "type": "boolean",
               "default": false
+            },
+            "retryToolFailures": {
+              "type": "boolean",
+              "default": false
             }
           }
         }
@@ -413,7 +417,7 @@
           "minItems": 1,
           "uniqueItems": true,
           "items": {
-            "enum": ["fresh", "fork"]
+            "enum": ["fresh"]
           }
         },
         "models": {
@@ -445,6 +449,10 @@
         },
         "artifacts": {
           "type": "boolean"
+        },
+        "retryToolFailures": {
+          "type": "boolean",
+          "default": false
         }
       }
     },

package/src/command-names.ts

@@ -5,7 +5,6 @@ export const HARNESS_COMMAND_NAMES = [
   'workflow-reload',
   'workflow-resume',
   'workflow-start',
-  'workflow-status',
 ] as const;
 
 /**

package/src/commands.ts

@@ -15,7 +15,6 @@ export interface WorkflowCommandController {
   resume(ctx: ExtensionCommandContext): Promise<void>;
   abort(reason: string, ctx: ExtensionCommandContext): Promise<void>;
   reload(ctx: ExtensionCommandContext): Promise<void>;
-  status(ctx: ExtensionCommandContext): Promise<void>;
 }
 
 function splitFirst(value: string): [string, string] {
@@ -72,9 +71,4 @@ export function registerHarnessCommands(
     description: 'Reload workflow files while no workflow is running',
     handler: async (_args, ctx) => controller.reload(ctx),
   });
-
-  pi.registerCommand('workflow-status', {
-    description: 'Open the active workflow status board',
-    handler: async (_args, ctx) => controller.status(ctx),
-  });
 }

package/src/config/ceiling.ts

@@ -113,6 +113,14 @@ export function checkWorkflowAgainstCeiling(
         `${subagentPath}.artifacts: exceeds the user permission ceiling`,
       );
     }
+    if (
+      step.subagent.retryToolFailures &&
+      !ceiling.subagent.retryToolFailures
+    ) {
+      errors.push(
+        `${subagentPath}.retryToolFailures: exceeds the user permission ceiling`,
+      );
+    }
     if (!step.subagent.turnBudget) {
       errors.push(
         `${subagentPath}.turnBudget: required for a project workflow`,

package/src/config/load.ts

@@ -184,7 +184,7 @@ async function loadWorkflowDirectory(
   return { workflows, diagnostics };
 }
 
-async function loadSettings(userDirectory: string): Promise<{
+export async function loadSettings(userDirectory: string): Promise<{
   settings: WorkflowSettings;
   diagnostics: ConfigDiagnostic[];
 }> {
@@ -284,14 +284,8 @@ export async function loadCatalog(
           'warning',
         ),
       );
-    } else if (!settingsResult.settings.permissionCeiling) {
-      diagnostics.push(
-        diagnostic(
-          projectDirectory,
-          'project workflows were skipped because no user permission ceiling is configured',
-        ),
-      );
     } else {
+      const permissionCeiling = settingsResult.settings.permissionCeiling!;
       const projectResult = await loadWorkflowDirectory(
         projectDirectory,
         'project',
@@ -300,7 +294,7 @@ export async function loadCatalog(
       for (const workflow of projectResult.workflows) {
         const ceilingErrors = checkWorkflowAgainstCeiling(
           workflow.definition,
-          settingsResult.settings.permissionCeiling,
+          permissionCeiling,
         );
         if (ceilingErrors.length > 0) {
           diagnostics.push(

package/src/config/types.ts

@@ -1,4 +1,7 @@
+import type { KeyId } from '@earendil-works/pi-tui';
+
 export const WORKFLOW_SCHEMA_VERSION = 1 as const;
+export const DEFAULT_STATUS_SHORTCUT = 'ctrl+alt+w' as const satisfies KeyId;
 export const SUBAGENT_RUNTIME_NAME_PATTERN =
   /^[a-z0-9][a-z0-9-]*(?:\.[a-z0-9][a-z0-9-]*)*$/;
 
@@ -43,7 +46,7 @@ export interface StepRequirements {
   skills: string[];
 }
 
-export type SubagentContext = 'fresh' | 'fork';
+export type SubagentContext = 'fresh';
 
 export interface SubagentTurnBudget {
   maxTurns: number;
@@ -57,15 +60,20 @@ export interface SubagentToolBudget {
 }
 
 export interface StepSubagent {
-  /** Configured pi-subagents agent name. */
+  /** Pi Subagents agent profile launched for this isolated step. */
   agent: string;
-  /** Fresh keeps workflow work out of the parent transcript. */
+  /** Workflow steps always use a fresh context. */
   context: SubagentContext;
   model?: string;
   timeoutMs: number;
   turnBudget?: SubagentTurnBudget;
   toolBudget?: SubagentToolBudget;
   artifacts: boolean;
+  /**
+   * Authorizes one fresh-context continuation after a tool failure when the
+   * trusted complete transcript proves every recorded call was replay-safe.
+   */
+  retryToolFailures: boolean;
 }
 
 export type PromptSpec = { inline: string } | { file: string };
@@ -138,11 +146,13 @@ export interface SubagentPermissionCeiling {
   maxGraceTurns: number;
   maxToolCalls: number;
   artifacts: boolean;
+  retryToolFailures: boolean;
 }
 
 export interface WorkflowSettings {
   version: typeof WORKFLOW_SCHEMA_VERSION;
   allowProjectWorkflows: boolean;
+  statusShortcut: KeyId;
   permissionCeiling?: PermissionCeiling;
 }
 
@@ -179,9 +189,11 @@ export const DEFAULT_STEP_SUBAGENT: StepSubagent = {
   context: 'fresh',
   timeoutMs: 900_000,
   artifacts: false,
+  retryToolFailures: false,
 };
 
 export const DEFAULT_SETTINGS: WorkflowSettings = {
   version: WORKFLOW_SCHEMA_VERSION,
   allowProjectWorkflows: false,
+  statusShortcut: DEFAULT_STATUS_SHORTCUT,
 };

package/src/config/validate.ts

@@ -1,12 +1,13 @@
 import {
   DEFAULT_SETTINGS,
+  DEFAULT_STATUS_SHORTCUT,
   DEFAULT_STEP_SUBAGENT,
   EMPTY_PERMISSIONS,
   EMPTY_REQUIREMENTS,
   SUBAGENT_RUNTIME_NAME_PATTERN,
   WORKFLOW_SCHEMA_VERSION,
-  type BashMode,
   type BashApprovalSource,
+  type BashMode,
   type BashPermission,
   type BashRule,
   type PermissionCeiling,
@@ -38,6 +39,60 @@ const MCP_SELECTOR_PATTERN = /^[A-Za-z0-9_.:-]+(?:\/[A-Za-z0-9_.:-]+)?$/;
 const EXECUTABLE_PATTERN = /^[A-Za-z0-9_./+-]+$/;
 const BASH_APPROVAL_SOURCE_PATTERN =
   /^(verification-worker|verification-reviewer|remote-actions)$/;
+const SHORTCUT_MODIFIERS = new Set(['ctrl', 'shift', 'alt', 'super']);
+const SHORTCUT_NAMED_KEYS = new Map<string, string>([
+  ['escape', 'escape'],
+  ['esc', 'esc'],
+  ['enter', 'enter'],
+  ['return', 'return'],
+  ['tab', 'tab'],
+  ['space', 'space'],
+  ['backspace', 'backspace'],
+  ['delete', 'delete'],
+  ['insert', 'insert'],
+  ['clear', 'clear'],
+  ['home', 'home'],
+  ['end', 'end'],
+  ['pageup', 'pageUp'],
+  ['pagedown', 'pageDown'],
+  ['up', 'up'],
+  ['down', 'down'],
+  ['left', 'left'],
+  ['right', 'right'],
+]);
+const SHORTCUT_SYMBOL_KEYS = new Set([
+  '`',
+  '-',
+  '=',
+  '[',
+  ']',
+  '\\',
+  ';',
+  "'",
+  ',',
+  '.',
+  '/',
+  '!',
+  '@',
+  '#',
+  '$',
+  '%',
+  '^',
+  '&',
+  '*',
+  '(',
+  ')',
+  '_',
+  '|',
+  '~',
+  '{',
+  '}',
+  ':',
+  '<',
+  '>',
+  '?',
+]);
+const MAX_SHORTCUT_CHARS = 64;
 const PROMPT_VARIABLES = new Set([
   'workflow.input',
   'workflow.id',
@@ -91,6 +146,54 @@ function readString(
   return result;
 }
 
+function canonicalShortcutKey(value: string): string | undefined {
+  if (/^[a-z0-9]$/.test(value) || SHORTCUT_SYMBOL_KEYS.has(value)) {
+    return value;
+  }
+  if (/^f(?:[1-9]|1[0-2])$/.test(value)) return value;
+  return SHORTCUT_NAMED_KEYS.get(value);
+}
+
+function readStatusShortcut(
+  value: unknown,
+  path: string,
+  errors: string[],
+): WorkflowSettings['statusShortcut'] {
+  if (value === undefined) return DEFAULT_STATUS_SHORTCUT;
+  const shortcut = readString(value, path, errors);
+  if (!shortcut) return DEFAULT_STATUS_SHORTCUT;
+  if (shortcut.length > MAX_SHORTCUT_CHARS) {
+    errors.push(`${path}: must be at most ${MAX_SHORTCUT_CHARS} characters`);
+    return DEFAULT_STATUS_SHORTCUT;
+  }
+
+  const parts = shortcut.toLowerCase().split('+');
+  const key = parts.pop() ?? '';
+  const modifiers = parts;
+  const uniqueModifiers = new Set(modifiers);
+  const canonicalKey = canonicalShortcutKey(key);
+  const isPlainTypingKey =
+    modifiers.length === 0 && (key.length === 1 || key === 'space');
+  const isUnmatchableModifiedKey =
+    modifiers.length > 0 &&
+    (key === 'escape' || key === 'esc' || /^f(?:[1-9]|1[0-2])$/.test(key));
+  if (
+    !canonicalKey ||
+    modifiers.length > SHORTCUT_MODIFIERS.size ||
+    uniqueModifiers.size !== modifiers.length ||
+    modifiers.some((modifier) => !SHORTCUT_MODIFIERS.has(modifier)) ||
+    isPlainTypingKey ||
+    isUnmatchableModifiedKey
+  ) {
+    errors.push(`${path}: expected a supported Pi key id such as "ctrl+alt+w"`);
+    return DEFAULT_STATUS_SHORTCUT;
+  }
+
+  return [...modifiers, canonicalKey].join(
+    '+',
+  ) as WorkflowSettings['statusShortcut'];
+}
+
 function readInteger(
   value: unknown,
   fallback: number,
@@ -268,18 +371,7 @@ function parseBashPermission(
           errors,
           BASH_APPROVAL_SOURCE_PATTERN,
         )
-  ).filter((source): source is BashApprovalSource => {
-    const valid =
-      source === 'verification-worker' ||
-      source === 'verification-reviewer' ||
-      source === 'remote-actions';
-    if (!valid) {
-      errors.push(
-        `${path}.approvedSources: expected verification-worker, verification-reviewer, or remote-actions`,
-      );
-    }
-    return valid;
-  });
+  ) as BashApprovalSource[];
 
   const normalizedMode = validMode ? mode : 'deny';
   if (normalizedMode !== 'allow-list' && allow.length > 0) {
@@ -511,7 +603,7 @@ function parseStepSubagent(
     return { ...DEFAULT_STEP_SUBAGENT, agent };
   }
   if (!isObject(value)) {
-    errors.push(`${path}: expected a workflow subagent name or object`);
+    errors.push(`${path}: expected an agent profile name or object`);
     return undefined;
   }
   rejectUnknownKeys(
@@ -524,6 +616,7 @@ function parseStepSubagent(
       'turnBudget',
       'toolBudget',
       'artifacts',
+      'retryToolFailures',
     ],
     path,
     errors,
@@ -539,12 +632,9 @@ function parseStepSubagent(
     value.context === undefined
       ? DEFAULT_STEP_SUBAGENT.context
       : readString(value.context, `${path}.context`, errors);
-  const context: SubagentContext =
-    contextValue === 'fork' || contextValue === 'fresh'
-      ? contextValue
-      : DEFAULT_STEP_SUBAGENT.context;
-  if (contextValue !== 'fork' && contextValue !== 'fresh') {
-    errors.push(`${path}.context: expected fresh or fork`);
+  const context: SubagentContext = DEFAULT_STEP_SUBAGENT.context;
+  if (contextValue !== 'fresh') {
+    errors.push(`${path}.context: expected fresh`);
   }
   const model =
     value.model === undefined
@@ -575,6 +665,12 @@ function parseStepSubagent(
     `${path}.artifacts`,
     errors,
   );
+  const retryToolFailures = readBoolean(
+    value.retryToolFailures,
+    DEFAULT_STEP_SUBAGENT.retryToolFailures,
+    `${path}.retryToolFailures`,
+    errors,
+  );
   return {
     agent,
     context,
@@ -583,6 +679,7 @@ function parseStepSubagent(
     ...(turnBudget ? { turnBudget } : {}),
     ...(toolBudget ? { toolBudget } : {}),
     artifacts,
+    retryToolFailures,
   };
 }
 
@@ -766,6 +863,14 @@ function parseStep(
     `${path}.permissions`,
     errors,
   );
+  if (
+    subagent?.retryToolFailures &&
+    permissions.tools.some((tool) => tool === 'edit' || tool === 'write')
+  ) {
+    errors.push(
+      `${path}.subagent.retryToolFailures: requires a step without edit or write tools`,
+    );
+  }
   const requires = parseRequirements(
     value.requires,
     permissions,
@@ -1008,6 +1113,7 @@ function parseSubagentPermissionCeiling(
       'maxGraceTurns',
       'maxToolCalls',
       'artifacts',
+      'retryToolFailures',
     ],
     path,
     errors,
@@ -1022,7 +1128,7 @@ function parseSubagentPermissionCeiling(
     value.contexts,
     `${path}.contexts`,
     errors,
-    /^(?:fresh|fork)$/,
+    /^fresh$/,
   ) as SubagentContext[];
   const models = readStringList(
     value.models,
@@ -1078,6 +1184,12 @@ function parseSubagentPermissionCeiling(
     `${path}.artifacts`,
     errors,
   );
+  const retryToolFailures = readBoolean(
+    value.retryToolFailures,
+    false,
+    `${path}.retryToolFailures`,
+    errors,
+  );
   return {
     agents,
     contexts,
@@ -1087,6 +1199,7 @@ function parseSubagentPermissionCeiling(
     maxGraceTurns,
     maxToolCalls,
     artifacts,
+    retryToolFailures,
   };
 }
 
@@ -1099,7 +1212,13 @@ export function validateSettings(
   }
   rejectUnknownKeys(
     value,
-    ['$schema', 'version', 'allowProjectWorkflows', 'permissionCeiling'],
+    [
+      '$schema',
+      'version',
+      'allowProjectWorkflows',
+      'statusShortcut',
+      'permissionCeiling',
+    ],
     'settings',
     errors,
   );
@@ -1115,6 +1234,11 @@ export function validateSettings(
   } else if (value.allowProjectWorkflows !== undefined) {
     errors.push('settings.allowProjectWorkflows: expected a boolean');
   }
+  const statusShortcut = readStatusShortcut(
+    value.statusShortcut,
+    'settings.statusShortcut',
+    errors,
+  );
   const permissionCeiling = parsePermissionCeiling(
     value.permissionCeiling,
     'settings.permissionCeiling',
@@ -1130,6 +1254,7 @@ export function validateSettings(
     value: {
       ...DEFAULT_SETTINGS,
       allowProjectWorkflows,
+      statusShortcut,
       ...(permissionCeiling ? { permissionCeiling } : {}),
     },
     errors,

package/src/engine/state.ts

@@ -60,6 +60,8 @@ export interface WorkflowRun {
   gateFeedback: string;
   pauseReason?: string | undefined;
   pausedFrom?: 'running' | 'awaiting-gate' | undefined;
+  /** Current step when execution failed and was paused for a resumable retry. */
+  failedStepId?: string | undefined;
   pendingGate?: PendingGate | undefined;
 }
 
@@ -141,6 +143,7 @@ export function isWorkflowRun(value: unknown): value is WorkflowRun {
       typeof run.reviewedArtifact === 'string') &&
     (run.stepHandoff === undefined || typeof run.stepHandoff === 'string') &&
     (run.pauseReason === undefined || typeof run.pauseReason === 'string') &&
+    (run.failedStepId === undefined || typeof run.failedStepId === 'string') &&
     (run.pausedFrom === undefined ||
       run.pausedFrom === 'running' ||
       run.pausedFrom === 'awaiting-gate');
@@ -154,6 +157,9 @@ export function isWorkflowRun(value: unknown): value is WorkflowRun {
     run.status === 'paused'
       ? run.pausedFrom === 'running' || run.pausedFrom === 'awaiting-gate'
       : run.pausedFrom === undefined;
+  const failureStateIsValid =
+    run.failedStepId === undefined ||
+    (run.status === 'paused' && run.failedStepId === run.currentStepId);
   const gateStateIsValid = !gateIsValid
     ? false
     : run.pendingGate === undefined
@@ -177,6 +183,7 @@ export function isWorkflowRun(value: unknown): value is WorkflowRun {
     optionalsAreValid &&
     statusIsValid &&
     pauseStateIsValid &&
+    failureStateIsValid &&
     gateStateIsValid &&
     typeof run.startedAt === 'number' &&
     typeof run.updatedAt === 'number' &&