Skip to content

fix: #3288 normalize RunState guardrail payloads#3289

Merged
seratch merged 1 commit intoopenai:mainfrom
Aphroq:fix/runstate-guardrail-json-payloads
May 8, 2026
Merged

fix: #3288 normalize RunState guardrail payloads#3289
seratch merged 1 commit intoopenai:mainfrom
Aphroq:fix/runstate-guardrail-json-payloads

Conversation

@Aphroq
Copy link
Copy Markdown
Contributor

@Aphroq Aphroq commented May 8, 2026

Summary

  • Normalize RunState guardrail result payloads so guardrail outputInfo, output guardrail agentOutput, and tool guardrail outputInfo remain JSON-compatible.
  • Add regression coverage for non-JSON guardrail payloads using datetime values in input/output guardrails and tool input/output guardrails.

Test plan

  • uv run pytest tests/test_run_state.py -k guardrail_results_to_string_normalizes_non_json
  • uv run pytest tests/test_run_state.py -k guardrail_results
  • bash .agents/skills/code-change-verification/scripts/run.sh

Issue number

Closes #3288

Checks

  • I've added new tests (if relevant)
  • I've added/updated the relevant documentation ()
  • I've run make lint and make format
  • I've made sure tests pass

@Aphroq Aphroq changed the title Normalize RunState guardrail payloads fix: #3288 Normalize RunState guardrail payloads May 8, 2026
@seratch seratch changed the title fix: #3288 Normalize RunState guardrail payloads fix: #3288 normalize RunState guardrail payloads May 8, 2026
@seratch seratch added this to the 0.17.x milestone May 8, 2026
@seratch seratch merged commit b4741e0 into openai:main May 8, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RunState guardrail payload serialization fails for non-JSON values

2 participants