Skip to content

fix(vendor): an empty inbound Subject no longer fails a parked step (CL-8590) - #936

Merged
TheGreatAxios merged 3 commits into
mainfrom
cl-8590-vendored-empty-subject
Sep 18, 2026
Merged

TheGreatAxios merged 3 commits into
mainfrom
cl-8590-vendored-empty-subject

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • A mail with a present-but-textless Subject: header decoded to subject: "", which createInboundMessage rejects, failing the step and ending the run.
  • This is an owner-approved temporary vendored delta, recorded in VENDORED.md with its kill condition.

Changes

  • vendor/intx/workflow-host/src/adapters/step-invoker.ts: buildInboundMessageFromMail now omits subject when the trimmed header is empty, matching the existing content guard.
  • Exported buildInboundMessageFromMail for direct unit testing.
  • VENDORED.md / vendor/intx/workflow-host/VENDORED-FROM: recorded the delta and its kill condition (upstream tolerates an empty Subject, INTR-577).

Testing

  • Added a co-located unit test proving a mail with subject: "" produces an inbound message with no subject.
  • bun run check passes.

@linear-code

linear-code Bot commented Sep 18, 2026

Copy link
Copy Markdown

CL-8590

@TheGreatAxios
TheGreatAxios merged commit cd14491 into main Sep 18, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant