Skip to content

fix: omit null computer safety checks#2750

Closed
sztoplover-bit wants to merge 1 commit intoopenai:mainfrom
sztoplover-bit:fix/omit-null-computer-safety-checks-ga
Closed

fix: omit null computer safety checks#2750
sztoplover-bit wants to merge 1 commit intoopenai:mainfrom
sztoplover-bit:fix/omit-null-computer-safety-checks-ga

Conversation

@sztoplover-bit
Copy link

Summary

  • omit acknowledged_safety_checks from computer_call_output when it is None
  • keep acknowledged safety checks unchanged when they are present
  • add a regression assertion for the no-safety-check path

Testing

  • .venv/bin/pytest tests/test_computer_action.py

Fixes #2741

@github-actions github-actions bot added bug Something isn't working feature:core labels Mar 21, 2026
@seratch
Copy link
Member

seratch commented Mar 21, 2026

There is already #2742; (the server side will eventually resolve this issue)

@seratch seratch closed this Mar 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working feature:core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GA computer tool rejects acknowledged_safety_checks: null from ComputerCallOutput

2 participants