Skip to content

Add safer TUI defaults for cyber models - #37055

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/8f25b0082d3051d9ff91f7355921236d0f82f758
Aug 5, 2026
Merged

Add safer TUI defaults for cyber models#37055
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/8f25b0082d3051d9ff91f7355921236d0f82f758

Conversation

@copyberry

@copyberry copyberry Bot commented Aug 5, 2026

Copy link
Copy Markdown

Add safer TUI defaults for cyber models

What changed

  • Propagate the optional modelSpecialty field through the model catalog and
    app-server model/list response.
  • When a newly selected model has the cyber specialty, default an active TUI
    thread to workspace-write permissions with on-request approval. Use auto
    review when available, otherwise keep the user as reviewer, while respecting
    configured permission requirements.
  • Show a notice when auto review is applied and strengthen the full-access
    warning for cyber models.
  • Preserve explicitly selected permissions when only reasoning settings change.

Testing

  • Cover auto-review defaults, requirement-aware fallback behavior, permission
    preservation across reasoning changes, and the new warning notice.

## What changed

- Propagate the optional `modelSpecialty` field through the model catalog and
  app-server `model/list` response.
- When a newly selected model has the `cyber` specialty, default an active TUI
  thread to workspace-write permissions with on-request approval. Use auto
  review when available, otherwise keep the user as reviewer, while respecting
  configured permission requirements.
- Show a notice when auto review is applied and strengthen the full-access
  warning for cyber models.
- Preserve explicitly selected permissions when only reasoning settings change.

## Testing

- Cover auto-review defaults, requirement-aware fallback behavior, permission
  preservation across reasoning changes, and the new warning notice.

GitOrigin-RevId: 8f25b0082d3051d9ff91f7355921236d0f82f758
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/8f25b0082d3051d9ff91f7355921236d0f82f758 branch from 05ea41a to 757c151 Compare August 5, 2026 08:05
@copyberry
copyberry Bot merged commit 757c151 into main Aug 5, 2026
30 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/8f25b0082d3051d9ff91f7355921236d0f82f758 branch August 5, 2026 08:06
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 5, 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.

10 participants