Skip to content

fix nested config items in codex app - #493

Closed
lilly-luo wants to merge 2 commits into
mainfrom
lilly/fix-codex-app-again
Closed

fix nested config items in codex app#493
lilly-luo wants to merge 2 commits into
mainfrom
lilly/fix-codex-app-again

Conversation

@lilly-luo

@lilly-luo lilly-luo commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

#473 was a previous attempt to fix codex app in ucode. i tested by running uv run ucode codex app and it worked. however, when i installed a new ucode and ran ucode codex app, it failed with the error below.

this is because when i tested previously, uv run uses the uv.lock where as a freshly installed ucode does not respect uv.lock. we specify tomlkit>=0.13.0. the uv.lock has 0.14 pinned which works but a freshly installed ucode has tomlkit 0.15.1 which is not compatible with this change.

i've since updated the uv.lock #496 so that uv run will be consistent with future installs so we don't have this discrepancy in the future. this PR makes a fix such that ucode codex app will actually work

before:

➜  ucode git:(lilly/fix-codex-app-again) ucode codex app

╭────────────────────────────────────╮
│ Launching Codex with Unity Gateway │
╰────────────────────────────────────╯
✔ Starting Codex
╭────────────────────────────────────────────────────── Traceback (most recent call last) ───────────────────────────────────────────────────────╮
│ /Users/lilly.luo/.local/share/uv/tools/ucode/lib/python3.12/site-packages/ucode/cli.py:2402 in codex_cmd                                       │
│                                                                                                                                                │
│   2399 │   │   print_success("Codex smart routing disabled; ucode routing hooks removed")                                                      │
.......
ValueError: Inline tables cannot contain a table

after (with the updated uv.lock):

➜  ucode git:(lilly/fix-codex-app-again) uv run codex app 
Opening Codex Desktop at /Applications/Codex.app...
Opening workspace /Users/lilly.luo/ucode...
Screenshot 2026-09-04 at 12 48 52 PM

@lilly-luo
lilly-luo force-pushed the lilly/fix-codex-app-again branch from d938d39 to 8d6d2c2 Compare September 4, 2026 22:01
@lilly-luo

Copy link
Copy Markdown
Collaborator Author

this will be fixed here: #502

@lilly-luo lilly-luo closed this Sep 5, 2026
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.

1 participant