chore(ce-work-beta): remove beta Codex-delegation skill#1013
Merged
Conversation
The experimental Codex external-delegation skill (beta, disable-model-invocation, manually invoked, no downstream handoffs) is retired -- ongoing maintenance of the shell-out path outweighed its value, so the experiment ends rather than graduating to stable. - Delete skills/ce-work-beta/ and the now-moot promotion checklist - Add ce-work-beta to STALE_SKILL_DIRS and the legacy-artifacts skills list so upgrades sweep stale flat installs - Add a retired-skill prompt-fingerprint fallback in loadLegacyFingerprints so the orphaned Codex prompt wrapper is still swept once the skill stops shipping (description-gated; the prompts dir is cross-plugin) - Update README inventory (27 -> 26 skills) and the release-metadata test count; drop/relocate ce-work-beta test coverage - Decouple codex-delegation-best-practices.md into a standalone retrospective, keeping the token-economics learnings
This was referenced Jun 26, 2026
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d11af3b3e9
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
- Add ce-work-beta to LEGACY_ONLY_SKILL_DESCRIPTIONS so the retired skill dir is still swept on upgrade -- without a fingerprint, skills.get was undefined and isLegacyPluginOwned returned false before deleting - Add the last-shipped ce-work-beta description to LEGACY_PROMPT_DESCRIPTION_ALIASES so a prompt wrapper generated from the final live skill isn't misclassified as foreign and left behind - Add regression tests for both retired-artifact sweep paths
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1b62cab7af
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Add ce:work-beta to EXTRA_LEGACY_ARTIFACTS_BY_PLUGIN.commands so the retired Codex slash-prompt wrapper (~/.codex/prompts/ce-work-beta.md) is enumerated by getLegacyCodexArtifacts. Flat installs without an install manifest use that enumeration, not the test-only cleanupStalePrompts helper, so the prior fingerprint/alias fixes (which gate ownership) were never consulted for the prompt wrapper without this entry. - Regression test: assert ce-work-beta skill and ce-work-beta.md prompt both appear in the Codex legacy-artifacts enumeration
tmchow
added a commit
that referenced
this pull request
Jun 26, 2026
#1013 deleted the ce-work-beta skill entirely. During the rebase its modify/delete conflicts were resolved by accepting the deletion, and the pure ce-work-beta port commit was dropped. Remove the now-dangling ce-work-beta read + assertions from the unified-plan contract test (renamed to 'ce-work is readiness-aware').
tmchow
added a commit
that referenced
this pull request
Jun 26, 2026
#1013 deleted the ce-work-beta skill entirely. During the rebase its modify/delete conflicts were resolved by accepting the deletion, and the pure ce-work-beta port commit was dropped. Remove the now-dangling ce-work-beta read + assertions from the unified-plan contract test (renamed to 'ce-work is readiness-aware').
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
1 participant
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
Retires the experimental
ce-work-betaskill — a beta, manually-invoked Codex external-delegation mode. It never left beta, carrieddisable-model-invocation, and had no downstream handoffs; keeping its shell-out orchestration correct asce-workevolved cost more than the experiment returned. This ends the experiment rather than promoting it to stable.What changes
ce-work-betaartifacts: the name is added toSTALE_SKILL_DIRSand the legacy-artifacts skills list, andloadLegacyFingerprintsnow seeds a prompt fingerprint from the historical-alias map when a prompt's mapped skill no longer ships.codex-delegation-best-practices.mdis reframed as a standalone retrospective. The skill-specific promotion checklist is removed.Why the cleanup fallback
Codex prompt-wrapper cleanup is description-gated — the prompts directory is shared across plugins, so a name-only match would risk deleting another plugin's file. That fingerprint was sourced from the live skill's description, so deleting the skill silently stopped the orphaned wrapper from ever matching. The new fallback mirrors the existing
LEGACY_ONLY_SKILL_DESCRIPTIONS/LEGACY_ONLY_AGENT_DESCRIPTIONSpattern for retired skills and agents, which had no prompt equivalent until now. Skill directory cleanup is unaffected — it matches by name, which is safe inside the plugin-owned skills tree.Scope
Historical
docs/plans/,docs/brainstorms/, andCHANGELOG.mdentries that mentionce-work-betaare intentionally left untouched — version control holds that history.Test plan
bun test: 1564 pass, 0 fail. The prompt-fingerprint fallback is covered by the existing legacy-cleanup regression that asserts a stalece-work-beta.mdwrapper is swept.bun run release:validate: in sync at 0 agents, 26 skills, 0 MCP servers.Post-Deploy Monitoring & Validation
No additional operational monitoring required — this is plugin/CLI source with no runtime service. Behavior is proven by the test suite and
release:validate, and the cleanup-registry behavior is exercised by the legacy-cleanup tests. On the next plugin release, upgrading installs sweep stalece-work-betaskill dirs and Codex prompt wrappers via the registries.