Просмотр исходного кода

fix: update symlink verification instructions and emphasize mandatory steps in SKILL.md

Yuki Takei 2 недель назад
Родитель
Сommit
d4c70a9e09

+ 4 - 0
apps/app/.claude/skills/learned/fix-broken-next-symlinks/SKILL.md

@@ -3,6 +3,10 @@ name: fix-broken-next-symlinks
 description: Fix broken symlinks in .next/node_modules/ — diagnose, decide allowlist vs dependencies, and verify
 ---
 
+## IMPORTANT
+
+This document is a **mandatory step-by-step procedure**. When fixing broken symlinks, execute every step in order. In particular, verification **always** requires the full 3-command sequence: `build` → `assemble-prod.sh` → `check-next-symlinks.sh`. Never skip `assemble-prod.sh` — the symlink check is only meaningful after production assembly.
+
 ## Problem
 
 Turbopack externalizes packages into `.next/node_modules/` as symlinks, even for packages imported only via dynamic `import()` inside `useEffect`. After `assemble-prod.sh` runs `pnpm deploy --prod`, `devDependencies` are excluded, breaking those symlinks. `check-next-symlinks.sh` detects these and fails the build.

+ 3 - 2
apps/app/bin/check-next-symlinks.sh

@@ -35,8 +35,9 @@ if [ -n "$broken" ]; then
   echo "  1. Moved from devDependencies to dependencies in apps/app/package.json"
   echo "  2. Added to ALLOWED_BROKEN in this script (if only used via useEffect + dynamic import)"
   echo ""
-  echo "See: apps/app/.claude/skills/learned/fix-broken-next-symlinks/SKILL.md"
-  echo "See also: apps/app/.claude/rules/package-dependencies.md"
+  echo "Fix: Follow the step-by-step procedure in apps/app/.claude/skills/learned/fix-broken-next-symlinks/SKILL.md"
+  echo "     You MUST execute every step in order — do NOT skip assemble-prod.sh when verifying."
+  echo "Ref: apps/app/.claude/rules/package-dependencies.md"
   exit 1
 fi