Skip to content

fix(pi): preserve tool call arguments - #52

Merged
drewstone merged 1 commit into
mainfrom
fix/pi-tool-arguments
Jul 30, 2026
Merged

fix(pi): preserve tool call arguments#52
drewstone merged 1 commit into
mainfrom
fix/pi-tool-arguments

Conversation

@drewstone

Copy link
Copy Markdown
Contributor

Pi raw traces store tool inputs under the arguments field. The adapter only read input and args, so every Pi tool call appeared to have no input.

This maps the existing field through the existing canonical serializer and adds a realistic regression case.

Proof:

  • Real corpus: 1,420 sessions parsed; 79,243/79,243 tool calls captured after versus 0/79,243 before
  • Full tests: 369/369 pass
  • Typecheck passes
  • Clean merge against current origin/main

A separate pre-existing Pi tool-result association defect is intentionally excluded from this focused change.

@drewstone
drewstone merged commit 671ba3b into main Jul 30, 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.

1 participant