Repository navigation
feat: add process-skill Codex openai.yaml metadata - #47
Merged
Merged
Conversation
Commit minimal agents/openai.yaml for the 14 process skills that lacked them, so package-codex-plugin.sh can build from this checkout without a prior official pack. Keep destination-side yaml preservation on sync. Co-authored-by: 悟空非空也 <1390128154@qq.com>
3 of 5 tasks
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.
Who is submitting this PR? (required)
cursor-grok-4.6-xhigh-fast)bc-7dcec1b4-f2a4-501e-a6d5-52a95659918f; Project coordinatorbc-00bc4604-2af3-4dc7-9e8c-350a7f19ede4)7cc09ade441962074ac64bed018ef0fc37734946, 16 files, +150/−15) in the Project chat on 2026-10-04 and then said to open this PR.What problem are you trying to solve?
14 process skills had no
agents/openai.yaml.scripts/package-codex-plugin.shfilled those files from a prior official pack (--metadata-source) and died if a skill was missing there. CI stayed green by writing those 14 yaml files intests/codex/test-package-codex-plugin.sh(write_metadata_fixture()). This checkout alone could not cut a complete Codex zip.This is the leftover architecture item 3, reconfirmed on
origin/mainbcadfde(v6.4.0) in the 2026-10-04 optimization review. Not a live Codex portal upload failure. The user experience: a new process skill, or a clean checkout with no_tmp/sup-codex-packaging/, cannot ship.The 14 skills:
brainstorming,dispatching-parallel-agents,executing-plans,finishing-a-development-branch,receiving-code-review,requesting-code-review,subagent-driven-development,systematic-debugging,test-driven-development,using-git-worktrees,using-wukong-code,verification-before-completion,writing-plans,writing-skills.What does this PR change?
Adds one minimal
agents/openai.yamlper missing process skill (interface.display_name/short_description/default_prompt+policy.allow_implicit_invocation: true), matching the existing 12 files.package-codex-plugin.shno longer requires--metadata-sourcewhen the source tree is complete. Sync still preserves destination-side yaml. NoSKILL.mdedits.Is this change appropriate for the core library?
Yes. This is core Codex publish metadata, not a new skill.
package-codex-plugin.shalready live in core. Missing yaml is why the packer needs an external tree.grilling,language-guidance, 10product-design*). Official Superpowers yaml inopenai/pluginsis still two fields (display_name+short_description); this checkout’s existing files already adddefault_promptand implicit policy.tests/codex/test-package-codex-plugin.sh(source yaml wins over fixture; no-metadata-source archive matches the fixture run; incomplete fallback still rejected ifwriting-plansyaml is omitted) andtests/codex-plugin-sync/test-sync-to-codex-plugin.shpassed. After commit,package-codex-plugin.sh --output …with no--metadata-sourceproduced 26 skills / 26 yaml. No Codex portal upload.What alternatives did you consider?
--metadata-sourceand only document it. Rejected: that is the current failure mode; CI hides it with fixtures.SKILL.mdand do not commit the files. Rejected: OpenAI metadata is their contract; the existing 12 files are committed, and pack/sync already treat committed yaml as source of truth.codex-package.manifest.json. Out of scope (optimization item 3 in the 2026-10-04 list).Does this PR contain multiple unrelated changes?
No. The 14 yaml files, the packer change that makes a complete tree self-sufficient, and the package tests that lock “no
--metadata-sourcewhen source is complete” are one metadata-completeness fix. No skill prose, PD bootstrap, or short-description work.Existing PRs
No open PR on
cursor/process-openai-yaml-918f. No closed PR commits these 14 files. #36 only unified PD transport paths. #13 addedgrillingyaml only. #11 / #12 addedlanguage-guidanceand the “source yaml wins over fixture” packer behavior.Environment tested
tests/codex/test-package-codex-plugin.sh,tests/codex-plugin-sync/test-sync-to-codex-plugin.sh; post-commitpackage-codex-plugin.shzip with no--metadata-source)cursor-grok-4.6-xhigh-fastscripts/lint-shell.shwas not run (shellchecknot on PATH).bash -non touched shell.New harness support (required if this PR adds a new harness)
Not applicable. This PR does not add a new harness.
Clean-session transcript for "Let's make a react todo list"
Evaluation
the session that led to this change?
After a 2026-10-04 optimization review of
origin/main, the human partner accepted “先做第 1 条”: commit minimalagents/openai.yamlfor the 14 process skills so this checkout can package Codex without an external pack.Zero live harness or portal sessions. After the change we ran the Codex package and sync suites above, then packed a zip from the committed tree.
Before: 14 skills missing yaml; packer required a prior official pack; CI invented those files. After: 26/26 yaml in tree; packer succeeds with no
--metadata-source; tests lock source-complete and incomplete-fallback paths. Sync still keeps destination-owned yaml. We did not upload to the Codex portal.using-wukong-codeyaml uses this repo’s name;using-git-worktreesfollows this fork’s “native isolation or worktree” wording.Language-pack evidence
Rigor
wukong-code:writing-skillsandcompleted adversarial pressure testing (paste results below)
rationalizations, "human partner" language) without extensive evals
showing the change is an improvement
This is Codex skill metadata and packer fallback, not a skill-prose rewrite.
writing-skillswas not used. No SKILL.md / Red Flags / “human partner” edits. Coverage is the package/sync suites plus a real no-metadata-source zip. We did not run live multi-session pressure tests.Human review