Skip to content

fix(ai): ignore intentional stream aborts#1992

Open
EfeDurmaz16 wants to merge 1 commit into
vercel:mainfrom
EfeDurmaz16:efe/suppress-abort-stream-errors
Open

fix(ai): ignore intentional stream aborts#1992
EfeDurmaz16 wants to merge 1 commit into
vercel:mainfrom
EfeDurmaz16:efe/suppress-abort-stream-errors

Conversation

@EfeDurmaz16
Copy link
Copy Markdown

Summary

  • treat AbortError stream closes as intentional in WorkflowChatTransport GET reconnect and POST send paths
  • avoid logging console.error or counting aborts toward reconnect failures
  • add regression tests for reconnect and initial-send abort streams

Closes #1971.

Testing

  • pnpm install --frozen-lockfile
  • pnpm --filter @workflow/ai exec vitest run src/workflow-chat-transport.test.ts
  • pnpm --filter @workflow/ai build
  • pnpm exec biome check packages/ai/src/workflow-chat-transport.ts packages/ai/src/workflow-chat-transport.test.ts .changeset/silent-abort-streams.md (passes with existing warnings in this file: non-null assertions in existing tests and complexity warnings)
  • git diff --check

Note: local Node is v26.0.0, so pnpm prints the repository engine warning for supported Node versions (^18 || ^20 || ^22 || ^24).

@EfeDurmaz16 EfeDurmaz16 requested a review from a team as a code owner May 14, 2026 21:34
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 14, 2026

🦋 Changeset detected

Latest commit: 45d6a68

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@workflow/ai Patch
tarballs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 14, 2026

@EfeDurmaz16 is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

Signed-off-by: EfeDurmaz16 <efebarandurmaz05@gmail.com>
@EfeDurmaz16 EfeDurmaz16 force-pushed the efe/suppress-abort-stream-errors branch from 3bc2423 to 45d6a68 Compare May 14, 2026 21:36
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.

WorkflowChatTransport.reconnectToStreamIterator logs console.error on intentional client abort

1 participant