Skip to content

refactor(core): move embedded system skills into codex-skills crate#12435

Merged
bolinfest merged 1 commit intomainfrom
pr12435
Feb 21, 2026
Merged

refactor(core): move embedded system skills into codex-skills crate#12435
bolinfest merged 1 commit intomainfrom
pr12435

Conversation

@bolinfest
Copy link
Collaborator

@bolinfest bolinfest commented Feb 21, 2026

Why

codex-core was carrying the embedded system-skill sample assets (and a build.rs that walks those files to register rerun triggers). Those assets change infrequently, but any change under codex-core still ties them to codex-core's build/cache lifecycle.

This change moves the embedded system-skills packaging into a dedicated codex-skills crate so it can be cached independently. That reduces unnecessary invalidation/rebuild pressure on codex-core when the skills bundle is the only thing that changes.

What Changed

  • Added a new codex-rs/skills crate (codex-skills) with:
    • Cargo.toml
    • BUILD.bazel
    • build.rs to track skill asset file changes for Cargo rebuilds
    • src/lib.rs containing the embedded system-skills install/cache logic previously in codex-core
  • Moved the embedded sample skill assets from codex-rs/core/src/skills/assets/samples to codex-rs/skills/src/assets/samples.
  • Updated codex-rs/core/Cargo.toml to depend on codex-skills and removed codex-core's direct include_dir dependency.
  • Removed codex-core's build.rs.
  • Replaced codex-rs/core/src/skills/system.rs implementation with a thin re-export wrapper to keep existing codex-core call sites unchanged.
  • Updated workspace manifests/lockfile (codex-rs/Cargo.toml, codex-rs/Cargo.lock) for the new crate.

@bolinfest bolinfest changed the title fix: introduce codex-skills crate refactor(core): move embedded system skills into codex-skills crate Feb 21, 2026
@bolinfest bolinfest requested a review from xl-openai February 21, 2026 08:10
@bolinfest bolinfest enabled auto-merge (squash) February 21, 2026 08:30
@bolinfest bolinfest merged commit 85ce91a into main Feb 21, 2026
33 checks passed
@bolinfest bolinfest deleted the pr12435 branch February 21, 2026 08:34
@github-actions github-actions bot locked and limited conversation to collaborators Feb 21, 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