chore(skills): remove openai sidecars#60
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the per-skill skills/*/agents/openai.yaml sidecar files that previously provided OpenAI Codex UI metadata and invocation policy hints, intending to rely on the canonical SKILL.md artifacts and existing plugin packaging instead.
Changes:
- Deleted
agents/openai.yamlsidecars from shipped skills to reduce per-skill metadata maintenance. - Left canonical
SKILL.mdskill definitions and plugin packaging structure unchanged.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| skills/validate-usercontext-instructions/agents/openai.yaml | Remove OpenAI Codex sidecar metadata file for this skill |
| skills/validate-skill/agents/openai.yaml | Remove OpenAI Codex sidecar metadata file for this skill |
| skills/validate-project-instructions/agents/openai.yaml | Remove OpenAI Codex sidecar metadata file for this skill |
| skills/validate-agents-md/agents/openai.yaml | Remove OpenAI Codex sidecar metadata file for this skill |
| skills/restore-checkpoint/agents/openai.yaml | Remove OpenAI Codex sidecar metadata file for this skill |
| skills/repository-drift-control/agents/openai.yaml | Remove OpenAI Codex sidecar metadata file for this skill |
| skills/create-usercontext-instructions/agents/openai.yaml | Remove OpenAI Codex sidecar metadata file for this skill |
| skills/create-skill/agents/openai.yaml | Remove OpenAI Codex sidecar metadata file for this skill |
| skills/create-project-instructions/agents/openai.yaml | Remove OpenAI Codex sidecar metadata file for this skill |
| skills/create-checkpoint/agents/openai.yaml | Remove OpenAI Codex sidecar metadata file for this skill |
| skills/create-agents-md/agents/openai.yaml | Remove OpenAI Codex sidecar metadata file for this skill |
Comments suppressed due to low confidence (2)
skills/repository-drift-control/agents/openai.yaml:1
- PR description says the OpenAI sidecars only carried an
allow_implicit_invocation: truesetting that matches the default, but this sidecar explicitly setsallow_implicit_invocation: false. If Codex falls back to a default oftruewhen the sidecar is absent, removing this file changes invocation behavior for this governance skill. Please either (a) confirm Codex’s fallback default keeps this non-implicit behavior, (b) encode the explicit-invocation policy in the canonical metadata path used without sidecars, or (c) keep sidecars for the small set of skills that requireallow_implicit_invocation: false. Also consider correcting the PR description to reflect that not all sidecars hadallow_implicit_invocation: true.
skills/validate-agents-md/agents/openai.yaml:1 - After removing these
agents/openai.yamlfiles, repository docs currently become stale: README.md explicitly states each skill includes anagents/openai.yamlsidecar (README.md:396-401) and CHANGELOG.md entries referenceagents/openai.yamlsidecars (e.g., CHANGELOG.md:21-22,48-49,56). Please update those docs to reflect the new Codex metadata fallback behavior so users don’t look for files that no longer exist.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
skills/*/agents/openai.yamlsidecars from the shipped skillsSKILL.mdfiles and Codex plugin packaging unchangedWhy
These sidecars were only carrying lightweight Codex UI metadata and an explicit
allow_implicit_invocation: truesetting, which matches the documented default behavior. They were not required for plugin installation or core skill discovery, so removing them reduces maintenance overhead without changing the main workflow.User impact
SKILL.mdfiles and plugin packagingValidation
skills/*/agents/openai.yamlfiles remain in the worktree