codex-rs: Make agent instructions modifiable #1338
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Enables agent instructions to be modifiable.
There are cases where I want to try out different instructions and their impact on the agents. Currently this involves re-compiling codex-rs when changes are made. This change enables adopters to (optionally) provide their own instructions during configuration, or through the
Op::ConfigureSession
.It might be more ideal that this data be
Option<Arc<str>>
instead ofOption<String>
as the data is not going to be modified once configured. I doubt there is much performance impact here other than copying a few KB of string data a few times.