Skip to content

codex-rs: Make agent instructions modifiable #1338

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

akuseru
Copy link

@akuseru akuseru commented Jun 15, 2025

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 of Option<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.

akuseru added 2 commits June 15, 2025 16:32
Enables agent instructions to be modified at the agent level. (ideal for multiple agents, or testing & expirementing with agents)

This could be expanded by the CLI to enable clients to customize the instructions.
Make agent instructions modifiable through an OP.
Copy link

github-actions bot commented Jun 15, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@akuseru
Copy link
Author

akuseru commented Jun 15, 2025

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Jun 15, 2025
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