Skip to content

[codex] Remove experimental instructions file config#22724

Merged
dylan-hurd-oai merged 1 commit into
mainfrom
dh--config-rm-exp-instructions-file
May 15, 2026
Merged

[codex] Remove experimental instructions file config#22724
dylan-hurd-oai merged 1 commit into
mainfrom
dh--config-rm-exp-instructions-file

Conversation

@dylan-hurd-oai
Copy link
Copy Markdown
Collaborator

Summary

Remove the deprecated experimental_instructions_file config setting from the typed config surface and the remaining deprecation-notice plumbing. model_instructions_file remains the supported setting and its loading path is unchanged.

The setting was deprecated when it was renamed to model_instructions_file on January 20, 2026 in #9555.

Changes

  • Remove experimental_instructions_file from ConfigToml and ConfigProfile.
  • Delete the custom config-layer scan and session deprecation notice for the removed setting.
  • Stop clearing the removed field from generated session config locks.
  • Remove the obsolete deprecation-notice test case while keeping model_instructions_file coverage intact.

Validation

  • just write-config-schema
  • just fmt
  • cargo test -p codex-config
  • cargo test -p codex-core model_instructions_file
  • just fix -p codex-core
  • git diff --check

Drop the deprecated experimental_instructions_file config field and its deprecation notice path now that model_instructions_file is the only supported setting.

Co-authored-by: Codex <noreply@openai.com>
@dylan-hurd-oai dylan-hurd-oai marked this pull request as ready for review May 15, 2026 01:04
@dylan-hurd-oai dylan-hurd-oai requested a review from a team as a code owner May 15, 2026 01:04
@dylan-hurd-oai dylan-hurd-oai merged commit 7dbe1c9 into main May 15, 2026
31 checks passed
@dylan-hurd-oai dylan-hurd-oai deleted the dh--config-rm-exp-instructions-file branch May 15, 2026 01:04
@github-actions github-actions Bot locked and limited conversation to collaborators May 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants