Skip to content

Add taskoutput tool result formatter#992

Merged
dersam merged 1 commit into
mainfrom
06-26/tool-result-format-taskoutput-regex
Jun 26, 2026
Merged

Add taskoutput tool result formatter#992
dersam merged 1 commit into
mainfrom
06-26/tool-result-format-taskoutput-regex

Conversation

@LasmarKhalifa

@LasmarKhalifa LasmarKhalifa commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Closes #978

Taskoutput's content is a String that resembles XML content, something like:
content: "<status>completed</status><output>Some output</output>"
However it is not proper XML. It contains no shared root and special characters are not escaped.

This implementation uses regex to extact the information we need from the content.

Prototypes [1] [2] [3] explored various options, but regex was ultimately the best suited for our current needs.

LasmarKhalifa commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

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

@LasmarKhalifa LasmarKhalifa marked this pull request as ready for review June 26, 2026 17:18

dersam commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Merge activity

  • Jun 26, 5:23 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jun 26, 5:24 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jun 26, 5:24 PM UTC: @dersam merged this pull request with Graphite.

@dersam dersam force-pushed the 06-26/tool-result-format-taskoutput-regex branch from 9467bf0 to 5303a93 Compare June 26, 2026 17:23
@dersam dersam merged commit a7c948f into main Jun 26, 2026
3 checks 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.

ToolResult#format_taskoutput

2 participants