Skip to content

fix: install Codex-native cc skill wrappers#14

Closed
inbarajaldrin wants to merge 1 commit intosendbird:mainfrom
inbarajaldrin:fix/codex-skill-discovery
Closed

fix: install Codex-native cc skill wrappers#14
inbarajaldrin wants to merge 1 commit intosendbird:mainfrom
inbarajaldrin:fix/codex-skill-discovery

Conversation

@inbarajaldrin
Copy link
Copy Markdown

Summary

In my Codex environment, the plugin-local skills/ directory was not surfacing as discoverable $cc:* skills. The plugin runtime, hooks, and cc-rescue agent were installed correctly, but the skills only became discoverable when installed through Codex’s native ~/.codex/skills path.

Changes

  • Installer now writes namespaced cc-* skill wrappers into ~/.codex/skills
  • Installer now writes matching cc-* prompt wrappers into ~/.codex/prompts
  • Generated wrappers point directly at the installed companion entrypoint under ~/.codex/plugins/cc
  • Uninstall removes the generated cc-* skill and prompt wrappers
  • Added installer coverage for generated wrapper installation and cleanup

Verification

  • node --test tests/installer-cli.test.mjs
  • Fresh reinstall generates:
    • ~/.codex/skills/cc-review/SKILL.md
    • ~/.codex/skills/cc-rescue/SKILL.md
    • matching ~/.codex/prompts/cc-*.md
  • $cc:setup is discoverable and reports ready
  • $cc:status is discoverable and returns tracked-job status
  • $cc:rescue is discoverable and successfully executes a write/read/delete test task

Copy link
Copy Markdown
Contributor

Thanks for putting this together and for calling out the Codex skill-discovery gap. Since the plugin moved to the 1.0.1 installer flow with the official Codex app-server plugin/install path, we took a narrower follow-up in #17 instead of carrying wrapper installs all the time.

The new PR keeps official install as the primary path on current Codex builds, and limits cc-* skill/prompt wrappers to the fallback path only when plugin/install is unavailable. It also adds coverage for that compatibility path and keeps $cc:setup as the one-shot repair path for missing hooks on marketplace-style installs.

Appreciate the report and the concrete proposal here. I’m going to close this PR in favor of #17 so the discussion stays on the version that matches the current installer architecture.

@sf-jin-ku
Copy link
Copy Markdown
Contributor

Closing this in favor of #17, which keeps official app-server install as the primary path on current Codex builds and narrows cc wrapper generation to the compatibility fallback only.

@sf-jin-ku sf-jin-ku closed this Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants