Skip to content

[codex] Fix plugin packaging layout#4

Merged
rob163 merged 1 commit into
mainfrom
codex/fix-plugin-packaging
Jun 8, 2026
Merged

[codex] Fix plugin packaging layout#4
rob163 merged 1 commit into
mainfrom
codex/fix-plugin-packaging

Conversation

@rob163

@rob163 rob163 commented Jun 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • Move the Codex and Claude plugin manifests into a self-contained plugin package at plugins/html-as-cli-panel-skill.
  • Make the root SKILL.md, root assets/cli-panel.html, and skills/cli-panel entries symlink into that package instead of duplicating the template.
  • Point Codex and Claude marketplace entries at the packaged plugin directory.
  • Update validation to check symlink entrypoints, plugin metadata, and installed skill/template paths.

Why

PR #3 made the install commands appear to succeed, but the Codex plugin cache only received the codex subdirectory. The codex/skills -> ../skills symlink pointed outside the cached plugin root, so the actual skill package was not installed. This layout keeps the plugin self-contained while preserving direct git clone and skills CLI entrypoints.

Validation

  • npm run validate
  • git diff --cached --check
  • CODEX_HOME=$(mktemp -d) codex plugin marketplace add .
  • CODEX_HOME=$(mktemp -d) codex plugin add html-as-cli-panel-skill@html-as-cli-panel-skill
  • Verified the installed Codex plugin cache contains skills/cli-panel/SKILL.md and skills/cli-panel/assets/cli-panel.html.

@rob163 rob163 merged commit c2912bb into main Jun 8, 2026
1 check passed
@rob163 rob163 deleted the codex/fix-plugin-packaging branch June 8, 2026 13:20
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.

1 participant