Skip to content

fix: reject TS workflow result failures as Error - #4529

Open
jatinder14 wants to merge 1 commit into
hatchet-dev:mainfrom
jatinder14:fix/workflow-run-ref-error-rejection
Open

fix: reject TS workflow result failures as Error#4529
jatinder14 wants to merge 1 commit into
hatchet-dev:mainfrom
jatinder14:fix/workflow-run-ref-error-rejection

Conversation

@jatinder14

Copy link
Copy Markdown

Summary

  • reject failed TypeScript SDK workflow results with a standard Error
  • retain only failed-step messages in the error text
  • add regression coverage for mixed successful and failed steps

Closes #4487

Test plan

  • pnpm test:unit -- src/util/workflow-run-ref.test.ts
  • pnpm exec prettier --check src/util/workflow-run-ref.ts src/util/workflow-run-ref.test.ts
  • pnpm exec tsc --noEmit

Ensure TS SDK callers receive a standard Error with only failed step messages.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Jul 27, 2026

Copy link
Copy Markdown

@jatinder-mahajan-certifyos is attempting to deploy a commit to the Hatchet Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the sdk-ts Related to the Typescript SDK label Jul 27, 2026
@jatinder14
jatinder14 force-pushed the fix/workflow-run-ref-error-rejection branch from 9b0d59b to a4f81fe Compare July 28, 2026 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sdk-ts Related to the Typescript SDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TS SDK: workflow-run-ref rejects with mixed string|undefined[] when any step fails

1 participant