Format Python SDK dangerous bypass updates

This commit is contained in:
Ahmed Ibrahim
2026-05-15 20:07:49 -07:00
parent d8ea048b5e
commit a1ef35b7b0
4 changed files with 10 additions and 30 deletions

View File

@@ -882,9 +882,7 @@ def _approval_mode_turn_sandbox_policy_lines(*, indent: str = " ") -> lis
]
def _model_arg_lines(
fields: list[PublicFieldSpec], *, indent: str = " "
) -> list[str]:
def _model_arg_lines(fields: list[PublicFieldSpec], *, indent: str = " ") -> list[str]:
return [f"{indent}{field.wire_name}={field.py_name}," for field in fields]

View File

@@ -15,9 +15,7 @@ class ApprovalMode(str, Enum):
deny_all = "deny_all"
auto_review = "auto_review"
dangerously_bypass_approvals_and_sandbox = (
"dangerously_bypass_approvals_and_sandbox"
)
dangerously_bypass_approvals_and_sandbox = "dangerously_bypass_approvals_and_sandbox"
def _approval_mode_settings(

View File

@@ -194,9 +194,7 @@ class Codex:
sandbox: SandboxMode | None = None,
service_tier: str | None = None,
) -> Thread:
approval_policy, approvals_reviewer = _approval_mode_override_settings(
approval_mode
)
approval_policy, approvals_reviewer = _approval_mode_override_settings(approval_mode)
sandbox = _thread_sandbox_for_approval_mode(approval_mode, sandbox)
params = ThreadResumeParams(
thread_id=thread_id,
@@ -231,9 +229,7 @@ class Codex:
service_tier: str | None = None,
thread_source: ThreadSource | None = None,
) -> Thread:
approval_policy, approvals_reviewer = _approval_mode_override_settings(
approval_mode
)
approval_policy, approvals_reviewer = _approval_mode_override_settings(approval_mode)
sandbox = _thread_sandbox_for_approval_mode(approval_mode, sandbox)
params = ThreadForkParams(
thread_id=thread_id,
@@ -405,9 +401,7 @@ class AsyncCodex:
service_tier: str | None = None,
) -> AsyncThread:
await self._ensure_initialized()
approval_policy, approvals_reviewer = _approval_mode_override_settings(
approval_mode
)
approval_policy, approvals_reviewer = _approval_mode_override_settings(approval_mode)
sandbox = _thread_sandbox_for_approval_mode(approval_mode, sandbox)
params = ThreadResumeParams(
thread_id=thread_id,
@@ -443,9 +437,7 @@ class AsyncCodex:
thread_source: ThreadSource | None = None,
) -> AsyncThread:
await self._ensure_initialized()
approval_policy, approvals_reviewer = _approval_mode_override_settings(
approval_mode
)
approval_policy, approvals_reviewer = _approval_mode_override_settings(approval_mode)
sandbox = _thread_sandbox_for_approval_mode(approval_mode, sandbox)
params = ThreadForkParams(
thread_id=thread_id,
@@ -534,9 +526,7 @@ class Thread:
summary: ReasoningSummary | None = None,
) -> TurnHandle:
wire_input = _to_wire_input(input)
approval_policy, approvals_reviewer = _approval_mode_override_settings(
approval_mode
)
approval_policy, approvals_reviewer = _approval_mode_override_settings(approval_mode)
sandbox_policy = _turn_sandbox_policy_for_approval_mode(
approval_mode,
sandbox_policy,
@@ -624,9 +614,7 @@ class AsyncThread:
) -> AsyncTurnHandle:
await self._codex._ensure_initialized()
wire_input = _to_wire_input(input)
approval_policy, approvals_reviewer = _approval_mode_override_settings(
approval_mode
)
approval_policy, approvals_reviewer = _approval_mode_override_settings(approval_mode)
sandbox_policy = _turn_sandbox_policy_for_approval_mode(
approval_mode,
sandbox_policy,

View File

@@ -255,18 +255,14 @@ def test_outside_workspace_write_rejected_for_deny_all_and_allowed_for_bypass(
with AppServerHarness(tmp_path) as harness:
rejected_args = json.dumps(
{
"command": (
f"printf %s rejected > {shlex.quote(str(rejected_path))}"
),
"command": (f"printf %s rejected > {shlex.quote(str(rejected_path))}"),
"login": False,
"timeout_ms": 1_000,
}
)
dangerous_args = json.dumps(
{
"command": (
f"printf %s dangerous > {shlex.quote(str(allowed_path))}"
),
"command": (f"printf %s dangerous > {shlex.quote(str(allowed_path))}"),
"login": False,
"timeout_ms": 1_000,
}