Skip to content

Redact secrets from app-server command execution items - #36893

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/2ad056f21882bf5166182ad1a4ff0bf6471d9c0e
Aug 4, 2026
Merged

Redact secrets from app-server command execution items#36893
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/2ad056f21882bf5166182ad1a4ff0bf6471d9c0e

Conversation

@copyberry

@copyberry copyberry Bot commented Aug 4, 2026

Copy link
Copy Markdown

Redact secrets from app-server command execution items

Why

Client-facing command execution items can include recognizable secrets in the
rendered command or parsed command actions.

What changed

  • Redact secrets in commandExecution.command and commandActions for live
    items, completed items, and replayed thread history.
  • Keep command approval requests backed by the original executable command and
    parsed actions while using the redacted representation for display items.
  • Document that execution item commands are redacted display values rather than
    executable commands.

Testing

  • Cover redaction for command conversion, parsed search actions, legacy replay,
    approval flows, completion notifications, and rejected commands.

## Why

Client-facing command execution items can include recognizable secrets in the
rendered command or parsed command actions.

## What changed

- Redact secrets in `commandExecution.command` and `commandActions` for live
  items, completed items, and replayed thread history.
- Keep command approval requests backed by the original executable command and
  parsed actions while using the redacted representation for display items.
- Document that execution item commands are redacted display values rather than
  executable commands.

## Testing

- Cover redaction for command conversion, parsed search actions, legacy replay,
  approval flows, completion notifications, and rejected commands.

GitOrigin-RevId: 2ad056f21882bf5166182ad1a4ff0bf6471d9c0e
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/2ad056f21882bf5166182ad1a4ff0bf6471d9c0e branch from 5e16bfa to fcf636a Compare August 4, 2026 13:18
@copyberry
copyberry Bot merged commit fcf636a into main Aug 4, 2026
28 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/2ad056f21882bf5166182ad1a4ff0bf6471d9c0e branch August 4, 2026 13:19
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 4, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants