Skip to content

[PROTOTYPE] Regex for parsing taskoutput results#984

Closed
LasmarKhalifa wants to merge 1 commit into
06-23/tool-result-format-taskoutput-rexml-prototypefrom
06-23/tool-result-foramt-taskoutput-regex-prototype
Closed

[PROTOTYPE] Regex for parsing taskoutput results#984
LasmarKhalifa wants to merge 1 commit into
06-23/tool-result-format-taskoutput-rexml-prototypefrom
06-23/tool-result-foramt-taskoutput-regex-prototype

Conversation

@LasmarKhalifa

@LasmarKhalifa LasmarKhalifa commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Regex-based parser prototype for taskoutput's tool result message.

This introduces a helper method that looks for any tag and returns its contents. It's regex-based, which means there are no issues related to unescaped characters. This also means it can be used for error_lines(as shown in this prototype).

The drawback is that in some cases, this can fail (cases such as <status>a</status>x</status>). However, we don't expect this to ever happen in our use cases.

@LasmarKhalifa LasmarKhalifa force-pushed the 06-23/tool-result-format-taskoutput-rexml-prototype branch from a03135a to 8f609d2 Compare June 25, 2026 18:57
@LasmarKhalifa LasmarKhalifa force-pushed the 06-23/tool-result-foramt-taskoutput-regex-prototype branch from 1c6d59b to 555c5c1 Compare June 25, 2026 18:57
@LasmarKhalifa LasmarKhalifa changed the title [PROTOTYPE] [WIP] Regex for parsing taskoutput results [PROTOTYPE] Regex for parsing taskoutput results Jun 26, 2026
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