Skip to content

Add skills.roots config for explicit skill roots#15394

Draft
prabhata-oai wants to merge 1 commit intomainfrom
prabhata/skills-roots-override
Draft

Add skills.roots config for explicit skill roots#15394
prabhata-oai wants to merge 1 commit intomainfrom
prabhata/skills-roots-override

Conversation

@prabhata-oai
Copy link
Copy Markdown

Summary

  • Add skills.roots to skills config so users can specify absolute local skill folders.
  • When skills.roots is non-empty, use configured roots instead of default local roots.
  • Preserve plugin-provided roots.
  • Add tests for roots override behavior and config parsing.
  • Regenerate core/config.schema.json to include the new config field.

Validation

  • cargo test -p codex-core config_schema_matches_fixture -- --nocapture
  • cargo test -p codex-core skill_roots_from_layer_stack_can_override_defaults_with_explicit_roots -- --nocapture
  • cargo test -p codex-core parses_bundled_skills_config -- --nocapture

Tickets

  • None

Add skills.roots to config so users can specify absolute skill folders.
Use configured roots as the local root set when provided; otherwise keep default local roots.
Add loader/config tests and regenerate the committed config schema fixture.

Co-authored-by: Codex <noreply@openai.com>
@github-actions
Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


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


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@etraut-openai etraut-openai added the oai PRs contributed by OpenAI employees label Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

oai PRs contributed by OpenAI employees

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants