Add Pi progress display and tool use formatting#833
Draft
dersam wants to merge 1 commit intodersam/pi-provider-invocation-executionfrom
Draft
Add Pi progress display and tool use formatting#833dersam wants to merge 1 commit intodersam/pi-provider-invocation-executionfrom
dersam wants to merge 1 commit intodersam/pi-provider-invocation-executionfrom
Conversation
This was referenced Mar 2, 2026
Contributor
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
Add ToolUse and ToolResult formatter classes for human-readable progress display during Pi agent execution, mirroring Claude's pattern. ToolUse provides specialized formatting for known Pi tools: - bash: shows command - read/write/edit: shows file path - grep: shows pattern and optional path - find: shows pattern and optional directory - ls: shows path - Unknown tools: shows name and arguments hash ToolResult formats tool execution results with: - OK/ERROR status indicator - Truncated content preview (max 200 chars, whitespace collapsed) Progress display enhancements: - MessageUpdateMessage#format now delegates toolcall_end events to ToolUse - TurnStartMessage#format shows '--- turn start ---' marker - TurnEndMessage#format shows model, token counts, and cost per turn - ToolExecutionStartMessage#format shows '⚙ executing tool...' - ToolExecutionEndMessage#format shows '⚙ tool execution complete' - PiInvocation#handle_message now calls format on all message types (not just MessageUpdateMessage) when show_progress is enabled 37 new tests (162 total Pi tests), full suite: 1037 tests.
dersam
force-pushed
the
dersam/pi-provider-progress-display
branch
from
87cf99b to
7bf6877
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add ToolUse and ToolResult formatter classes for human-readable progress
display during Pi agent execution, mirroring Claude's pattern.
ToolUse provides specialized formatting for known Pi tools:
ToolResult formats tool execution results with:
Progress display enhancements:
(not just MessageUpdateMessage) when show_progress is enabled
37 new tests (162 total Pi tests), full suite: 1037 tests.