add bash tool result formatter

[LasmarKhalifa]

No description provided.

[LasmarKhalifa]

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.
Learn more

• add task tool result formatter #963
• add agent tool result formatter #962
• add taskoutput tool result formatter backed by nokogiri #960
• add taskcreate tool result formatter #959
• add taskupdate tool result formatter #958
• add todowrite tool result formatter #957
• add skill tool result formatter #956
• add edit tool result formatter #955
• add write tool result formatter #954
• add grep tool result formatter #953
• add glob tool result formatter #952
• add read tool result formatter #951
• add bash tool result formatter #949 👈 (View in Graphite)
• Centralize tool result error handling in format with error_line helper #948
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[juniper-shopify]

Really nice abstraction — the doc comment calling this "the success-side twin of #error_line" is a great conceptual frame. Future readers will immediately understand how the error/success paths mirror each other.

It does sound like it was written by an AI :p but such is life these days, and it is clear

[juniper-shopify]

Not blocking, but something to think about for the future: TRUNCATE_LIMIT and truncate are also defined in tool_use.rb (in the parallel stack). When both stacks land, that's duplicated constants and logic.

It does feel like premature optimization to put this constant in some shared abstraction before these stacks are even shipped, but at least think about where this might live, and either put it there up front or circle back after all of these merge to consolidate.