Skip to content

Fix resumed step message projection#452

Open
CooperSheroy wants to merge 1 commit into
vercel:mainfrom
CooperSheroy:catalina/preserve-resumed-step-messages-20260701
Open

Fix resumed step message projection#452
CooperSheroy wants to merge 1 commit into
vercel:mainfrom
CooperSheroy:catalina/preserve-resumed-step-messages-20260701

Conversation

@CooperSheroy

Copy link
Copy Markdown

Summary

  • Preserve completed assistant text and reasoning parts when the runtime resumes a turn with a reused step index, such as after OAuth authorization.
  • Keep active streaming text/reasoning updates in place while appending a new part once the previous part is done.
  • Add a regression test for the OAuth resume event shape reported in Default message reducer overwrites text when OAuth resumes in same step #436.

Closes #436.

Validation

  • corepack pnpm exec oxfmt packages/eve/src/client/message-reducer.ts packages/eve/src/client/message-reducer.test.ts .changeset/resumed-step-message-parts.md
  • corepack pnpm exec oxlint packages/eve/src/client/message-reducer.ts packages/eve/src/client/message-reducer.test.ts
  • corepack pnpm --filter eve exec vitest run --config vitest.unit.config.ts src/client/message-reducer.test.ts
  • pnpm --filter eve run typecheck
  • git diff --check

Note: local Node is v22.22.2 while the repo declares Node >=24, so pnpm emitted engine warnings during validation.

Signed-off-by: CooperSheroy <sheroycoops@gmail.com>
@vercel

vercel Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

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

A member of the Team first needs to authorize it.

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.

Default message reducer overwrites text when OAuth resumes in same step

1 participant