mirror of
https://github.com/nextlevelbuilder/ui-ux-pro-max-skill.git
synced 2026-08-01 19:06:14 +00:00
ef5f5ba0e6
* fix(cli): install all 7 skills via uipro init, not just the orchestrator `uipro init` rendered only the orchestrator (ui-ux-pro-max) and never delivered the 6 sibling skills (banner-design, brand, design, design-system, slides, ui-styling), so users got 1 of 7 skills (#362). - sync-assets.mjs: bundle the 6 sub-skills into cli/assets/skills/ as static copies (source of truth: .claude/skills/), with sync + check coverage. Excludes ui-styling/canvas-fonts (~5.8MB of TTF) and __pycache__/.pyc cruft — a skill registers from its SKILL.md, not its fonts — so the bundle adds ~0.9MB, not ~6.6MB. - template.ts: after rendering the orchestrator, install each bundled sub-skill as a sibling. The skills parent is derived from the platform's skillPath (skills/ for most, prompts/ for copilot, steering/ for kiro) rather than hardcoded. - uninstall.ts: remove the sub-skills too. Verified: check:assets in sync, tsc passes, and a per-platform install harness delivers all 7 skills to the correct parent dir with no fonts. Closes #362 * fix(cli): filter excluded files from target side of check:assets check:assets filtered sourceFiles with isExcludedAssetFile but not targetFiles, so a stray cli/assets/scripts/__pycache__/*.pyc (generated by a local Python run) was reported as an "extra asset file" and failed the gate. Apply the same predicate to targetFiles in both the dirsToSync and sub-skill loops. Verified: check:assets now passes with __pycache__/*.pyc present in the target tree; typecheck passes. * fix(cli): uninstall from each platform's real skills dir, not hardcoded skills/ removeSkillDir() hardcoded <folder>/skills/<name>, but the installer places skills under each platform config's skillPath parent — copilot in .github/prompts/, kiro in .kiro/steering/. So uninstall left those platforms' skills (orchestrator + sub-skills) behind. Derive the install parent from loadPlatformConfig(aiType).folderStructure (same source the installer uses), and keep the legacy <folder>/skills/ cleanup (incl. .shared/) for older installs. Deduped via a Set. Verified: typecheck passes; an install+uninstall harness removes all 7 skills with zero leftovers for claude (.claude/skills), copilot (.github/prompts) and kiro (.kiro/steering). * fix(cli): re-sync bundled sub-skills after #385 stripped ckm- names #385 merged to main and removed the ckm- prefix from the six .claude/skills/*/SKILL.md name fields. This branch's bundled copies under cli/assets/skills/ still carried the old ckm- names, so after the PR merges with main the source no longer matched the bundle and the check-asset-sync CI gate failed (stale asset file: skills/*/SKILL.md). Merge main and regenerate the bundle so cli/assets/skills matches the current .claude/skills source of truth. check:assets and typecheck pass.
735 B
735 B
| 1 | content_type | primary_size | secondary_size | accent_size | weight_contrast | letter_spacing | line_height |
|---|---|---|---|---|---|---|---|
| 2 | hero-statement | 120px | 32px | 14px | 700-400 | tight | 1.0 |
| 3 | metric-callout | 96px | 18px | 12px | 700-500 | normal | 1.1 |
| 4 | feature-grid | 28px | 16px | 12px | 600-400 | normal | 1.4 |
| 5 | quote-block | 36px | 18px | 14px | 400-italic | loose | 1.5 |
| 6 | data-insight | 48px | 20px | 14px | 700-400 | normal | 1.2 |
| 7 | cta-action | 64px | 24px | 16px | 700-500 | tight | 1.1 |
| 8 | title-only | 80px | 24px | 14px | 700-400 | tight | 1.0 |
| 9 | subtitle-heavy | 56px | 28px | 16px | 600-400 | normal | 1.2 |
| 10 | body-focus | 24px | 18px | 14px | 500-400 | normal | 1.6 |
| 11 | comparison | 32px | 16px | 12px | 600-400 | normal | 1.3 |
| 12 | timeline | 28px | 16px | 12px | 500-400 | normal | 1.4 |
| 13 | pricing | 48px | 20px | 14px | 700-500 | normal | 1.2 |
| 14 | team | 24px | 16px | 14px | 600-400 | normal | 1.4 |
| 15 | testimonial | 32px | 20px | 14px | 400-italic | loose | 1.5 |