Alexander ef5f5ba0e6
fix(cli): install all 7 skills via uipro init, not just the orchestrator (#362) (#387)
* 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.
2026-06-25 17:33:23 +07:00

115 lines
4.6 KiB
Python

#!/usr/bin/env python3
# -*- coding: utf-8 -*-
"""
Logo Design Search - CLI for searching logo design guidelines
Usage: python search.py "<query>" [--domain <domain>] [--max-results 3]
python search.py "<query>" --design-brief [-p "Brand Name"]
Domains: style, color, industry
"""
import argparse
from core import CSV_CONFIG, MAX_RESULTS, search, search_all
def format_output(result):
"""Format results for Claude consumption (token-optimized)"""
if "error" in result:
return f"Error: {result['error']}"
output = []
output.append(f"## Logo Design Search Results")
output.append(f"**Domain:** {result['domain']} | **Query:** {result['query']}")
output.append(f"**Source:** {result['file']} | **Found:** {result['count']} results\n")
for i, row in enumerate(result['results'], 1):
output.append(f"### Result {i}")
for key, value in row.items():
value_str = str(value)
if len(value_str) > 300:
value_str = value_str[:300] + "..."
output.append(f"- **{key}:** {value_str}")
output.append("")
return "\n".join(output)
def generate_design_brief(query, brand_name=None):
"""Generate a comprehensive logo design brief based on query"""
results = search_all(query, max_results=2)
output = []
output.append("=" * 60)
if brand_name:
output.append(f" LOGO DESIGN BRIEF: {brand_name.upper()}")
else:
output.append(" LOGO DESIGN BRIEF")
output.append("=" * 60)
output.append(f" Query: {query}")
output.append("=" * 60)
output.append("")
# Industry recommendations
if "industry" in results:
output.append("## INDUSTRY ANALYSIS")
for r in results["industry"]:
output.append(f"**Industry:** {r.get('Industry', 'N/A')}")
output.append(f"- Recommended Styles: {r.get('Recommended Styles', 'N/A')}")
output.append(f"- Colors: {r.get('Primary Colors', 'N/A')}")
output.append(f"- Typography: {r.get('Typography', 'N/A')}")
output.append(f"- Symbols: {r.get('Common Symbols', 'N/A')}")
output.append(f"- Mood: {r.get('Mood', 'N/A')}")
output.append(f"- Best Practices: {r.get('Best Practices', 'N/A')}")
output.append(f"- Avoid: {r.get('Avoid', 'N/A')}")
output.append("")
# Style recommendations
if "style" in results:
output.append("## STYLE RECOMMENDATIONS")
for r in results["style"]:
output.append(f"**{r.get('Style Name', 'N/A')}** ({r.get('Category', 'N/A')})")
output.append(f"- Colors: {r.get('Primary Colors', 'N/A')} | {r.get('Secondary Colors', 'N/A')}")
output.append(f"- Typography: {r.get('Typography', 'N/A')}")
output.append(f"- Effects: {r.get('Effects', 'N/A')}")
output.append(f"- Best For: {r.get('Best For', 'N/A')}")
output.append(f"- Complexity: {r.get('Complexity', 'N/A')}")
output.append("")
# Color recommendations
if "color" in results:
output.append("## COLOR PALETTE OPTIONS")
for r in results["color"]:
output.append(f"**{r.get('Palette Name', 'N/A')}**")
output.append(f"- Primary: {r.get('Primary Hex', 'N/A')}")
output.append(f"- Secondary: {r.get('Secondary Hex', 'N/A')}")
output.append(f"- Accent: {r.get('Accent Hex', 'N/A')}")
output.append(f"- Background: {r.get('Background Hex', 'N/A')}")
output.append(f"- Psychology: {r.get('Psychology', 'N/A')}")
output.append("")
output.append("=" * 60)
return "\n".join(output)
if __name__ == "__main__":
parser = argparse.ArgumentParser(description="Logo Design Search")
parser.add_argument("query", help="Search query")
parser.add_argument("--domain", "-d", choices=list(CSV_CONFIG.keys()), help="Search domain")
parser.add_argument("--max-results", "-n", type=int, default=MAX_RESULTS, help="Max results (default: 3)")
parser.add_argument("--json", action="store_true", help="Output as JSON")
parser.add_argument("--design-brief", "-db", action="store_true", help="Generate comprehensive design brief")
parser.add_argument("--brand-name", "-p", type=str, default=None, help="Brand name for design brief")
args = parser.parse_args()
if args.design_brief:
result = generate_design_brief(args.query, args.brand_name)
print(result)
else:
result = search(args.query, args.domain, args.max_results)
if args.json:
import json
print(json.dumps(result, indent=2, ensure_ascii=False))
else:
print(format_output(result))