Skip to content

[codex] Add direct Codex skill package#302

Draft
Sabotage1 wants to merge 1 commit into
nextlevelbuilder:mainfrom
Sabotage1:codex/add-direct-skill-package
Draft

[codex] Add direct Codex skill package#302
Sabotage1 wants to merge 1 commit into
nextlevelbuilder:mainfrom
Sabotage1:codex/add-direct-skill-package

Conversation

@Sabotage1

Copy link
Copy Markdown

Summary

Adds a self-contained Codex skill package at .codex/skills/ui-ux-pro-max.

The existing .claude/skills/ui-ux-pro-max package uses repository symlinks for data and scripts. Codex GitHub skill installation can download those symlinks as plain pointer files, which leaves the installed search script unusable. This package copies SKILL.md, data/, and scripts/ directly so Codex users can install from a GitHub tree path without resolving symlinks.

Also documents the direct Codex install URL in the README.

Validation

  • Ran python3 .codex/skills/ui-ux-pro-max/scripts/search.py "fintech crypto" --design-system -f markdown

@mrgoonie

Copy link
Copy Markdown
Contributor

Summary: I’m deferring this PR from the cron-safe maintainer lane because it is far too large for a bounded metadata-first review pass.

Decision: deferred / needs focused maintainer review

Evidence:

  • Scope is 36 changed files with +9,933 lines.
  • The title adds a direct Codex skill package, which is a packaging/distribution capability and needs source-of-truth, install-path, generated-asset, and platform compatibility review before code-level approval.
  • Current cron policy defers PRs over 20 changed files or 800 additions to avoid shallow approvals of broad changes.

Next step: please narrow this PR or add a maintainer-ready design note explaining the intended canonical source files, generated/package artifacts, install target paths, verification commands, and why this Codex package should be separate from or integrated with the existing platform template system. After that, a focused maintainer review can check the implementation properly.

@mrgoonie mrgoonie added agent:github-maintain Processed by github-maintain automation maintain:deferred Deferred by maintain workflow pr:oversized Too large for cron-safe review budget labels Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent:github-maintain Processed by github-maintain automation maintain:deferred Deferred by maintain workflow pr:oversized Too large for cron-safe review budget

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants