Skip to content

Remove the legacy core skill loader - #37457

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/214d06d59bf3033ee0f220ea5959ee4feff66782
Aug 7, 2026
Merged

Remove the legacy core skill loader#37457
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/214d06d59bf3033ee0f220ea5959ee4feff66782

Conversation

@copyberry

@copyberry copyberry Bot commented Aug 7, 2026 •
edited
Loading

Copy link
Copy Markdown

Remove the legacy core skill loader

What changed

  • Use HostSkillRoot throughout host root resolution and loading, including plugin roots.
  • Remove the duplicate loader, root snapshot, and product-filtering implementation from core-skills now that loading is owned by the skills extension.
  • Move and expand loader coverage for discovery, namespaces, filesystem routing, root merging, symlinks, and frontmatter parsing.

Testing

  • Added focused unit and integration tests under ext/skills and skills for the consolidated loader behavior.

@felixxia-oai copyberry
Remove the legacy core skill loader (#37457)
33e365b 
## What changed

- Use `HostSkillRoot` throughout host root resolution and loading, including plugin roots.
- Remove the duplicate loader, root snapshot, and product-filtering implementation from `core-skills` now that loading is owned by the skills extension.
- Move and expand loader coverage for discovery, namespaces, filesystem routing, root merging, symlinks, and frontmatter parsing.

## Testing

- Added focused unit and integration tests under `ext/skills` and `skills` for the consolidated loader behavior.

GitOrigin-RevId: 214d06d59bf3033ee0f220ea5959ee4feff66782
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/214d06d59bf3033ee0f220ea5959ee4feff66782 branch from 50650da to 33e365b Compare August 7, 2026 15:17
@copyberry
copyberry Bot merged commit 33e365b into main Aug 7, 2026
29 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/214d06d59bf3033ee0f220ea5959ee4feff66782 branch August 7, 2026 15:18
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@yunhungo @xwm1782379277 @YeNai-ShaoXianChao @felixxia-oai