Skip to content

Fix dangling tool terminalization in stream normalizer#163

Draft
vutuanlinh2k2 wants to merge 1 commit into
mainfrom
fix/dangling-tool-terminalization
Draft

Fix dangling tool terminalization in stream normalizer#163
vutuanlinh2k2 wants to merge 1 commit into
mainfrom
fix/dangling-tool-terminalization

Conversation

@vutuanlinh2k2

Copy link
Copy Markdown
Contributor

Summary

  • prefer stable tool call IDs before transient part IDs in the shared stream normalizer
  • terminalize dangling running tool parts during assistant finalization
  • preserve prior tool state fields when terminal updates omit them

Tests

  • pnpm exec vitest run tests/stream-normalizer.test.ts
  • pnpm typecheck
  • NODE_OPTIONS=--max-old-space-size=6144 pnpm build

Notes

  • Full pnpm test hits existing local sandbox deferred-file failures on macOS shell/base64 behavior; the new stream-normalizer test passes both standalone and inside the full run.

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