refactor(cli)!: rename all skills and agents to consistent ce- prefix#503
refactor(cli)!: rename all skills and agents to consistent ce- prefix#503
Conversation
![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: 79b5508dbc
ℹ️ 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".
plugins/compound-engineering/skills/ce-code-review/references/persona-catalog.md
Show resolved
Hide resolved
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8cab52e383
ℹ️ 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".
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
16e4238 to
3f8a92a
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 00b3c26e06
ℹ️ 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".
tmchow
changed the title
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
00b3c26 to
b27f0c0
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b27f0c0c60
ℹ️ 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".
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
b27f0c0 to
79c262b
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 79c262bf35
ℹ️ 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".
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
79c262b to
8b7a83f
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8b7a83fbdf
ℹ️ 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".
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
8b7a83f to
e08be12
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e08be12b45
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fd32cb77bc
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 84975ffc05
ℹ️ 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".
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
2 times, most recently
from
aafcdfc to
ec64d8f
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ec64d8f8c0
ℹ️ 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".
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
ec64d8f to
fe94616
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fe94616987
ℹ️ 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".
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
fe94616 to
05aa0b0
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 05aa0b03e6
ℹ️ 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".
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
05aa0b0 to
b8d7eb8
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b8d7eb8412
ℹ️ 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".
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
3 times, most recently
from
a0eb184 to
3834842
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3834842e12
ℹ️ 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".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 12a79e7c6b
ℹ️ 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".
- scope Kiro stale agent cleanup to compound fingerprints - clean legacy Codex workflow wrappers across pre- and post-rename formats - remove stale OpenClaw agent-* directories during upgrades - cover session-historian and slack-researcher cleanup regressions
There was a problem hiding this comment.
💡 Codex Review
Write review artifacts under ce-code-review path
This block still creates and references .context/compound-engineering/ce-review/..., but later sections of the renamed ce-code-review skill state that per-run artifacts live under .context/compound-engineering/ce-code-review/... (including assumptions that reviewer JSON files are already there). In mode:headless and downstream artifact synthesis, this split can leave reviewer artifacts in a different directory than the run artifact, causing missing-detail reads and orphaned run data after the rename.
ℹ️ 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".
- scope Kiro stale agent cleanup to compound fingerprints - clean legacy Codex workflow wrappers across pre- and post-rename formats - remove stale OpenClaw agent-* directories during upgrades - cover session-historian and slack-researcher cleanup regressions
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
abb15c1 to
30f1b56
Compare
- scope Kiro stale agent cleanup to compound fingerprints - clean legacy Codex workflow wrappers across pre- and post-rename formats - remove stale OpenClaw agent-* directories during upgrades - cover session-historian and slack-researcher cleanup regressions
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
30f1b56 to
7876130
Compare
There was a problem hiding this comment.
💡 Codex Review
Keep ce-code-review artifacts under a single directory
This block still creates and uses .context/compound-engineering/ce-review/... for reviewer outputs even though the renamed skill now documents and emits artifacts under .context/compound-engineering/ce-code-review/<run-id>/. Splitting writes across ce-review and ce-code-review breaks the artifact contract for downstream callers that read the reported ce-code-review path and expect per-reviewer JSON files to be there.
ℹ️ 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".
- scope Kiro stale agent cleanup to compound fingerprints - clean legacy Codex workflow wrappers across pre- and post-rename formats - remove stale OpenClaw agent-* directories during upgrades - cover session-historian and slack-researcher cleanup regressions
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
7876130 to
4e0fff6
Compare
Standardize all 37 compound-engineering skills and 49 agents on a `ce-` hyphen prefix, eliminating namespace collisions with other plugins and removing the colon character that required filesystem sanitization. Key changes: - `ce:plan` → `ce-plan`, `git-commit` → `ce-commit`, etc. (29 skill directory renames + 8 frontmatter-only updates) - `adversarial-reviewer` → `ce-adversarial-reviewer`, etc. (49 agent file renames + frontmatter updates) - `ce-review` → `ce-code-review` (clearer purpose) - `ce-document-review` → `ce-doc-review` (more concise) - Agent references simplified: `compound-engineering:review:X` → `review:ce-X` - All bare agent names in prose now carry `ce-` prefix - Codex converter: remove workflow prompt wrappers (skills are directly invocable), replace `isCanonicalCodexWorkflowSkill` prefix check with direct skill copying, map deprecated `workflows:*` aliases to skill targets instead of dead prompt targets - Remove `codexPrompt` field from types/parser (no longer needed) - Path sanitization tests use non-CE examples to preserve colon coverage - New invariant test: no CE skill name contains a colon BREAKING CHANGE: All skill and agent names changed. `/ce:plan` is now `/ce-plan`, `/ce:review` is now `/ce-code-review`, agent references use `<category>:ce-<name>` format instead of `compound-engineering:<category>:<name>`. Closes #337 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…mizer skill The extract-commands-normalize test imported from the old `claude-permissions-optimizer` directory path. Update to `ce-claude-permissions-optimizer` to match the rename. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
… mapping, OpenCode 2-segment rewrite - Update ce-work and ce-work-beta to reference `ce-commit-push-pr` and `ce-commit` instead of old `git-commit-push-pr`/`git-commit` names - Add WORKFLOW_ALIAS_OVERRIDES map so `workflows:review` correctly resolves to `ce-code-review` instead of non-existent `ce-review` - Add 2-segment `category:ce-agent` rewrite to OpenCode transform so refs like `review:ce-correctness-reviewer` flatten to `ce-correctness-reviewer` Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…dled plugin uses ce- prefix The fixture's agent-one.md has `name: repo-research-analyst` (no ce- prefix), so fixture-based tests check for `repo-research-analyst.md`. The bundled plugin test checks for `ce-repo-research-analyst.md`. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Users who previously ran `install --to codex` (or any target) have old skill dirs (git-commit/, setup/, document-review/) and agent files (adversarial-reviewer.md, etc.) that would coexist with the new ce-* names after re-installing. This causes duplicate/conflicting entries at runtime. Add a one-time legacy cleanup module that removes known stale paths before writing new content. Wired into all 10 target writers. Handles: - 31 stale skill dirs (29 renamed + ce-review, ce-document-review) - 49 stale agent files/dirs (all agents gained ce- prefix) - 8 stale Codex prompt wrappers (removed in this release) Cleanup is idempotent and marked with TODO(cleanup) for removal after the v3 transition window. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Covers skill dir removal, agent file removal (flat .md, .agent.md, and directory formats), prompt wrapper removal, preservation of non-stale files, and idempotency. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Make stale skill and agent removal fingerprint-aware so shared install roots keep user-owned files with overlapping names. Also move Qwen cleanup ahead of agent writes and clean Pi legacy agent directories from skills/ instead of prompts, with regression tests for both cases.
Remove pre-rename agent skill directories from Gemini installs before writing generated skills so stale unprefixed agents do not remain discoverable after the ce- rename.
Teach stale cleanup about the historical setup skill description and Kiro's legacy JSON/prompt agent formats so rename cleanup still works after later wording changes and on Kiro installs.
Align the rebased session history agent with the plugin's ce- naming convention and update the README plus skill dispatch references to match.
- scope Kiro stale agent cleanup to compound fingerprints - clean legacy Codex workflow wrappers across pre- and post-rename formats - remove stale OpenClaw agent-* directories during upgrades - cover session-historian and slack-researcher cleanup regressions
Update the live skill dispatch to use research:ce-slack-researcher so the command resolves the renamed agent at runtime.
Update the pipeline contract assertions to match the current hyphenated ce-work-beta and ce-frontend-design skill references used by the skill docs.
tmchow
force-pushed
the
feat/ce-skill-prefix-rename
branch
from
4e0fff6 to
859a241
Compare
There was a problem hiding this comment.
💡 Codex Review
Align ce-code-review artifact directory name
This step still creates the run directory under .context/compound-engineering/ce-review/... even though the renamed workflow now documents and emits artifacts under .context/compound-engineering/ce-code-review/... elsewhere in the same skill. In autofix/headless runs this splits artifacts across two directory names, so callers or follow-on tooling that read only the new ce-code-review path can miss the per-reviewer JSON outputs and degrade post-processing.
ℹ️ 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".
1 participant
Why
Generic skill names (
setup,plan,review) collide when users install multiple Claude Code plugins. Our naming was inconsistent: 8 workflow skills usedce:colon prefix, 33 others had no prefix, and agent references used verbose 3-segment format (compound-engineering:review:adversarial-reviewer). The colon also required filesystem sanitization on Windows.This PR standardizes everything on
ce-hyphen prefix — skills, agents, and cross-references.What changed
Skill renames (37 skills):
ce:plan→ce-plan(colon to hyphen, 8 skills)git-commit→ce-commit,setup→ce-setup, etc. (prefix added, 24 skills)git-worktree→ce-worktree,git-commit-push-pr→ce-commit-push-pr(git- prefix replaced, 4 skills)report-bug-ce→ce-report-bug(normalized, 1 skill)ce-review→ce-code-review,ce-document-review→ce-doc-reviewagent-browser,rclone(upstream),lfg,slfg(memorable names)Agent renames (49 agents):
ce-prefix within their category dirs:adversarial-reviewer.md→ce-adversarial-reviewer.mdcompound-engineering:review:adversarial-reviewer→review:ce-adversarial-reviewerlearnings-researcher) →ce-learnings-researcherCodex converter simplification:
workflows:*aliases now map to skill targets instead of dead prompt targetscodexPromptfield from types/parser (no longer needed)Tests:
ce:brainstormtoother:skillto preserve colon coverageHow to review
This is a large but mechanical change (180 files). Suggested approach:
src/converters/claude-to-codex.ts) — the only real logic change. The prompt wrapper removal and alias-to-skill-target mapping are the interesting bits.lfg/SKILL.mdandslfg/SKILL.md— these orchestrate the full workflow chain and are where broken refs would be most visible.git mvoperations and frontmatter updates are mechanical. The test suite andrelease:validateconfirm consistency.Test plan
bun test— 572 pass (2 pre-existing network failures unrelated to this change)bun run release:validate— 49 agents, 41 skills, 0 MCP servers/ce:,compound-engineering:category:agent, or bare agent names in active codeagent-browser,rclone,lfg,slfgcorrectly excluded from renamingPost-Deploy Monitoring & Validation
No runtime deployment — plugin content and CLI converter change. After marketplace publish:
/ce-plan,/ce-code-review,/ce-workresolve in a fresh Claude Code sessionce-names and no prompt wrappersBREAKING CHANGE: All skill and agent names changed.
/ce:plan→/ce-plan,/ce:review→/ce-code-review, agent refs use<category>:ce-<name>format.Closes #337
🤖 Generated with Claude Code · Compound Engineered · Model:
claude-opus-4-6· Harness:claude-code