Skip to content

Support explicit harness field in agent traces#8127

Open
jedisct1 wants to merge 1 commit intohuggingface:parse_agent_tracesfrom
Swival:add-swival-harness
Open

Support explicit harness field in agent traces#8127
jedisct1 wants to merge 1 commit intohuggingface:parse_agent_tracesfrom
Swival:add-swival-harness

Conversation

@jedisct1
Copy link
Copy Markdown

@jedisct1 jedisct1 commented Apr 8, 2026

Hi!

This small PR allows an optional harness field to be parsed.

When present, the harness field takes priority over type-value heuristics.

This allows agents to self-identify even when they contain property names that collide with an existing harness ("session", "system", etc.)

When present, the harness field takes priority over type-value heuristics.
This allows agents to self-identify even when their trace structure and
type values collide with an existing harness like claude_code.
Copy link
Copy Markdown
Member

@lhoestq lhoestq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm ! cc @cfahlgren1 it should be ok ?

@jedisct1
Copy link
Copy Markdown
Author

For a concrete example, https://huggingface.co/datasets/jedisct1/agent-traces-swival currently shows claude_code because it shares field names with Claude Code, despite an explicit harness field identifying the harness as not Claude Code.

@cfahlgren1
Copy link
Copy Markdown

cfahlgren1 commented Apr 10, 2026

For a concrete example, https://huggingface.co/datasets/jedisct1/agent-traces-swival currently shows claude_code because it shares field names with Claude Code, despite an explicit harness field identifying the harness as not Claude Code.

is there any parsing issues from the actual traces, or more about just the name of the harness?

@jedisct1
Copy link
Copy Markdown
Author

@cfahlgren1 No parsing issues. This is just about showing the correct harness name.

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.

3 participants