Skip to content

Update Codex profile config layout#93

Merged
rohita5l merged 1 commit into
mainfrom
fix-codex-profile-config
May 27, 2026
Merged

Update Codex profile config layout#93
rohita5l merged 1 commit into
mainfrom
fix-codex-profile-config

Conversation

@AarushiShah-db

@AarushiShah-db AarushiShah-db commented May 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Write ucode Codex settings to ~/.codex/ucode.config.toml for the new Codex profile loader.
  • Remove ucode's legacy profile selector/table from shared ~/.codex/config.toml during configure.
  • Require Codex CLI 0.134.0+ for the new profile-file layout, with a clear blocker for older installed versions.
  • When ucode configure detects an older Codex CLI, warn that the update is required and run the Codex update path instead of treating it as optional.
  • Update Codex unit tests and e2e harness paths for the profile-specific config file.

Tests

  • uv run pytest --ignore=tests/test_e2e.py
  • uv run pytest tests/test_agent_codex.py tests/test_agents_init.py
  • uv run ty check src/
  • uv run ruff check .
  • uv run ruff format --check src/ tests/

Note

This intentionally requires updating older Codex CLIs instead of maintaining two config layouts. Codex CLI 0.134.0+ rejects legacy same-name profile config when launched with --profile, so
ucode now uses the profile-specific config file layout and blocks older Codex versions with: npm install -g @openai/codex or ucode configure.

@AarushiShah-db AarushiShah-db force-pushed the fix-codex-profile-config branch from 8c6e457 to 4a21960 Compare May 26, 2026 23:05
@AarushiShah-db AarushiShah-db force-pushed the fix-codex-profile-config branch from 4a21960 to 2baf209 Compare May 26, 2026 23:15
@AarushiShah-db AarushiShah-db requested a review from rohita5l May 26, 2026 23:18
@rohita5l rohita5l merged commit 32c7163 into main May 27, 2026
2 checks passed
@rohita5l rohita5l deleted the fix-codex-profile-config branch May 27, 2026 13:15
Sign up for free to 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.

2 participants