Skip to content

feat(ux): announce model and effort before Codex run - #24

Merged
axisrow merged 1 commit into
mainfrom
feat/announce-model-effort
Jul 20, 2026
Merged

axisrow merged 1 commit into
mainfrom
feat/announce-model-effort

Conversation

@axisrow

@axisrow axisrow commented Jul 20, 2026

Copy link
Copy Markdown
Owner

All /codex:* commands now print 'Codex: model= effort=' to stderr before starting. Shows 'default' when not passed explicitly. Resolved model shows actual id (gpt-5.6-sol for 'sol'). User transparency. 162 pass / 4 pre-existing.

All /codex:* commands (review, adversarial-review, task, rescue) now print
'Codex: model=<resolved> effort=<resolved>' to stderr before starting the
Codex run. Shows 'default' when the user didn't pass --model/--effort
(Codex CLI decides internally). Resolved model shows the actual id (e.g.
gpt-5.6-sol for alias 'sol'). Gives the user transparency into what Codex
is actually running with.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
@axisrow
axisrow merged commit b7ef4a2 into main Jul 20, 2026
@axisrow
axisrow deleted the feat/announce-model-effort branch July 20, 2026 11:55
axisrow added a commit that referenced this pull request Jul 21, 2026
…closes #34) (#35)

The launch line for the codex:codex-rescue subagent showed only the topic
(codex:codex-rescue(<topic>)) and never the model/effort Codex actually uses —
painful in background runs where the companion's stderr announce is swallowed.

The Agent-tool `description` is free-form text composed by the caller before
the subagent selects the model, so instruct the caller (commands/rescue.md) to
include the resolved model/effort there: `<topic> model=<…> effort=<…>`.
Aliases resolve to full ids (spark→gpt-5.3-codex-spark, sol→gpt-5.6-sol,
terra→gpt-5.6-terra, luna→gpt-5.6-luna); unset → `default`. No desync risk
because codex-rescue selects the model only from explicit user flags.

Companion unchanged — announceRun (#24) stays for foreground-stderr; the
launch line is covered by the skill layer.

Co-authored-by: axisrow <[email protected]>
Co-authored-by: Claude <[email protected]>
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