feat: install UI/UX Pro Max for all AI assistants via uipro-cli - #213
feat: install UI/UX Pro Max for all AI assistants via uipro-cli#213pcbermudez79 wants to merge 1 commit into
Conversation
Ran `npx uipro-cli@latest init` to install UI/UX Pro Max skill across all supported AI coding assistants (Cursor, Windsurf, RooCode, Kiro, Codex, Qoder, Gemini CLI, Trae, OpenCode, Continue, CodeBuddy, GitHub Copilot) and updated the Claude skill SKILL.md. https://claude.ai/code/session_01HH9D5Mi6c3nXmH4QzaMGRi
|
Ok |
|
Recommend closing, @mrgoonie. This is the committed OUTPUT of running uipro init locally (~41k lines of generated per-assistant copies in 12 new top-level dirs) -- build artifacts, not source. The CLI generates these dynamically. |
mrgoonie
left a comment
mrgoonie
left a comment
There was a problem hiding this comment.
PR #213 review
Findings: oversized generated assistant-install output. Metadata: 365 files, +41478/-414, clean merge state, no checks. Body says it ran npx uipro-cli@latest init and installed output for many AI assistants. Comment by contributor notes this is generated output/build artifacts and CLI generates dynamically.
Verdict: request changes/close, not merge. Reason: commits generated per-assistant copies rather than source-of-truth changes. Project source of truth is src/ui-ux-pro-max + cli templates/assets; generated local install folders should not be committed. Needs narrow source change or documentation/CLI improvement only.
4 participants
Ran
npx uipro-cli@latest initto install UI/UX Pro Max skill across all supported AI coding assistants (Cursor, Windsurf, RooCode, Kiro, Codex, Qoder, Gemini CLI, Trae, OpenCode, Continue, CodeBuddy, GitHub Copilot) and updated the Claude skill SKILL.md.https://claude.ai/code/session_01HH9D5Mi6c3nXmH4QzaMGRi