Skip to content

Keep plugin and skill snapshots scoped to each step - #50050

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/37d3cbe8c7e062fd06290aa985ada4d3b3462d43
Oct 1, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/37d3cbe8c7e062fd06290aa985ada4d3b3462d43

Conversation

@copyberry

@copyberry copyberry Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Keep plugin and skill snapshots scoped to each step

Why

Environment changes and overlapping catalog refreshes can cause a step to use another step's selected plugins or executor skills. Model context and skill tools need to agree on the capabilities captured for that step.

What changed

  • Resolve selected plugins from each step's ready capability roots, honoring disabled plugins, and retain the snapshot with the step's extension data.
  • Use that snapshot for skill attribution in World State and the first step's active-plugin telemetry.
  • Return executor skill catalogs directly from refresh so overlapping requests render their own results while retaining thread caches.

Testing

Add regression coverage for overlapping executor catalogs and environment changes during step preparation. Verify that skills.list retains the original step's skills, the next step advertises the new skills, and disabled executor and plugin skills remain absent from model context.

## Why

Environment changes and overlapping catalog refreshes can cause a step to use another step's selected plugins or executor skills. Model context and skill tools need to agree on the capabilities captured for that step.

## What changed

- Resolve selected plugins from each step's ready capability roots, honoring disabled plugins, and retain the snapshot with the step's extension data.
- Use that snapshot for skill attribution in World State and the first step's active-plugin telemetry.
- Return executor skill catalogs directly from refresh so overlapping requests render their own results while retaining thread caches.

## Testing

Add regression coverage for overlapping executor catalogs and environment changes during step preparation. Verify that `skills.list` retains the original step's skills, the next step advertises the new skills, and disabled executor and plugin skills remain absent from model context.

GitOrigin-RevId: 37d3cbe8c7e062fd06290aa985ada4d3b3462d43
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/37d3cbe8c7e062fd06290aa985ada4d3b3462d43 branch from 5059991 to 9561a34 Compare October 1, 2026 17:45
@copyberry
copyberry Bot merged commit 9561a34 into main Oct 1, 2026
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/37d3cbe8c7e062fd06290aa985ada4d3b3462d43 branch October 1, 2026 17:45
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 1, 2026
@borekb
borekb deployed to issue-triage October 1, 2026 17:49 — with GitHub Actions Active
@borekb
borekb deployed to issue-triage October 1, 2026 17:49 — with GitHub Actions Active
@borekb
borekb deployed to issue-triage October 1, 2026 17:49 — with GitHub Actions Active

This branch was successfully deployed

1 active deployment
issue-triage — 9561a345 Deployed Oct 1, 2026 by borekb via Generate label suggestions #50623
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