Vouch Request: yashs33244 #3945

Filing this Vouch Request per the repo's contributor-vouching process (unvouched PRs are auto-closed). Please apply the vouch-request label if needed.

Why do you want to contribute?

Hi! I use Trigger.dev and, while reading through the realtime client, I found a bug I'd like to fix: booleanHelpersFromRunStatus in packages/core/src/v3/apiClient/runStream.ts computes isCompleted as successful || failed, omitting the terminal CANCELED status. So a canceled run on the realtime path (runs.subscribeToRun() / useRealtimeRun()) reports isCompleted: false, while REST (runs.retrieve()) and the docs treat it as completed — and react-hooks useRun relies on isCompleted to stop refetching, so canceled runs never settle on the realtime path. I have a focused fix ready: a per-status completedStatuses set, a load-bearing regression test driving a CANCELED run through RunSubscription, and a changeset. (I also spotted a separate, pre-existing REST isFailed inconsistency for canceled runs, which I'd file as its own issue rather than mix in.)

Prior contributions or relevant experience

I keep PRs small, tested, and conventional. Some recent open-source fixes:

• fix(hf): don't print huggingface_hub progress bars by default dlt-hub/dlt#4058, fix(cli): route dlthub handoff NOTE to stderr so piping isn't broken dlt-hub/dlt#4059
• fix(server): raise OAUTH2_CC token length cap to 4096 NangoHQ/nango#6494, fix(cli): validate trackDeletes ordering per-model, not globally NangoHQ/nango#6495
• fix(header-analysis): correct x-vercel-cache REVALIDATED state typo openstatusHQ/openstatus#2283
• fix(editor): use Selection.atStart when clearing selection on blur resend/react-email#3568, fix(ui): count CRLF as one line break in getLineAndColumnFromOffset resend/react-email#3569
• fix: only send S3 checksums when required to support S3-compatible storage documenso/documenso#2984 · fix(rest): type update.buildHeaders params with UpdateParams<any> refinedev/refine#7456

Happy to share more context on any of these.