fix(project create): don't stream the per-project skill install's clack UI (#52)

insta project create shells npx skills add for the stack skills (insta,
neon-postgres, tigris, better-auth) with stdio='inherit' — dumping the same
clone-spinner/banner noise we already removed from insta setup agent. Run it
silent instead; the per-skill ✓/failed line is the clean output and still
appears as each skill finishes (live progress). Also stops the child inheriting
a piped stdin.

Before: 'installing … via npx skills add …' + spinner spam per skill
After:
  installing related agent skills (insta, neon-postgres, tigris, better-auth) …
  insta ✓
  neon-postgres ✓
  tigris ✓
  better-auth ✓

77/77 green.


Claude-Session: https://claude.ai/code/session_01GMbAe5K1RfwaAcge5inX7P

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

Author: tonychang04

src/ensure-skills.ts

@@ -56,9 +56,13 @@ export async function installSkills(deps: Deps): Promise<void> {
   const run = deps.run ?? defaultRunner
   const print = deps.print ?? ((s: string) => process.stdout.write(s + '\n'))
   try {
-    print('  installing related agent skills (insta, neon-postgres, tigris, better-auth) via `npx skills add` …')
+    print('  installing related agent skills (insta, neon-postgres, tigris, better-auth) …')
     for (const s of SKILLS) {
-      const r = await run('npx', s.args, true) // streamed so the user sees download progress
+      // Don't stream: the `skills` tool's clack UI (clone spinner, banners) is noise. Run it
+      // silent (stdio 'ignore') and let the per-skill ✓/failed line below be the clean output —
+      // it appears as each skill finishes, so there's still live progress. (Also avoids the
+      // child inheriting a piped stdin.)
+      const r = await run('npx', s.args)
       print(r.ok ? `  ${s.label} ✓` : `  ${s.label} failed — add manually: npx ${s.args.join(' ')}`)
     }
     const added = ensureGitignore(deps.cwd, SKILL_DIRS)