-
Notifications
You must be signed in to change notification settings - Fork 95
Description
#127
Is nice step toward making this work but currently main issue is still that
codex-acp doesn't persist the changes to model and reasoning_effort.
this issue there describe it
zed-industries/codex-acp#172
session/set_config_option does set the options in memory.
But acpx calls codex-acp sets the options in memory ends. acpx get confirmation that codex-acp set the model/reasoning_effort and update the acpx session to reflect that.
Next real turn codex-acp session/load does not re-apply those set options and falls back to default set in config.
Result is acpx session state different model reasoning effort than reality.
Issue.2
Currently codex-acp is cached by npx so even if bugs are fixed upstream and behavior changes users may still use old version.
Maybe consider pinning codex-acp version or at least documenting it.