Skip to content

fix(multishot): avoid empty driver messages#275

Merged
drewstone merged 1 commit into
mainfrom
fix/multishot-driver-nonempty
Jun 22, 2026
Merged

fix(multishot): avoid empty driver messages#275
drewstone merged 1 commit into
mainfrom
fix/multishot-driver-nonempty

Conversation

@drewstone

Copy link
Copy Markdown
Contributor

Summary

  • represent tool-only assistant turns as concise non-empty tool-call summaries in the multishot driver transcript
  • skip blank transcript rows before calling the persona driver
  • bump agent-eval / Python client fallback version to 0.96.3

Verification

  • pnpm vitest run tests/multishot/multishot.test.ts — 8/8 passed
  • pnpm typecheck — passed
  • pnpm test — 2529 passed, 2 skipped
  • python3 -m py_compile clients/python/src/agent_eval_rpc/init.py — passed
  • NODE_OPTIONS=--max-old-space-size=8192 pnpm build — passed
  • pnpm verify:package — passed
  • git diff --check — passed
  • git merge-tree --write-tree origin/main HEAD — passed

Why

Tax Agent state multishot runs were failing with router 400 Invalid request: text content is empty after tool-only assistant turns. This belongs in the shared multishot runner, not as a Tax-local workaround.

@tangletools tangletools left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Auto-approved PR — 6ce38b09

Blanket team auto-approval is enabled for this reviewer service.
The full PR reviewer audit still runs separately and will publish findings if it detects issues.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-06-22T17:36:39Z

@drewstone drewstone merged commit 0cad442 into main Jun 22, 2026
1 check 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.

2 participants