Skip to content

feat(codex): make terminal animations configurable - #181

Merged
Ark0N merged 2 commits into
Ark0N:masterfrom
Lint111:agent/split-codex-animations
Aug 4, 2026
Merged

feat(codex): make terminal animations configurable#181
Ark0N merged 2 commits into
Ark0N:masterfrom
Lint111:agent/split-codex-animations

Conversation

@Lint111

Copy link
Copy Markdown
Contributor

Summary

  • Add an opt-in “Animated Status Effects” switch to Codex CLI settings.
  • Persist the preference through the strict settings schema.
  • Carry the value in CodexConfig for new local sessions.
  • Launch Codex with --config tui.animations=true|false.

Motivation

Codex's decorative TUI animations create frequent redraws that are expensive and visually noisy over mobile or remote terminal streaming. Users can now choose the native Codex behavior, with animations disabled by default for newly launched Codex sessions.

Scope

This is the Codex animation-preference slice extracted from #173. It includes the UI, persistence and request schemas, session type, command builder, and focused tests. It does not change terminal rendering or suppress output in Codeman.

Validation

  • npx vitest run --config config/vitest.config.ts test/run-mode-ui.test.ts test/tmux-manager.test.ts: 61 passed
  • Focused Playwright probe: control defaulted off, saved on, and remained on after page reload
  • npm run build: passed
  • Commit formatting hook: passed
  • git diff --check: passed

Existing browser-suite debt

test/mobile/settings.test.ts reported 17 passes and two failures in unrelated subagentTrackingEnabled expectations. The identical two failures reproduce on untouched origin/master; this branch does not change those settings.

@Lint111
Lint111 marked this pull request as ready for review July 29, 2026 17:27
CopilotAI review requested due to automatic review settings July 29, 2026 17:27

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Ark0N
Ark0N merged commit ddc267c into Ark0N:masterAug 4, 2026
2 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Lint111@Ark0N@claude