thanh-kit 2.5.0 → 2.5.2

package/templates/scripts/worktree.test.cjs

@@ -1,334 +1,9 @@
 #!/usr/bin/env node
 /**
- * Test suite for worktree.cjs
- * Run: node .claude/scripts/worktree.test.cjs
+ * Backward-compatible wrapper for worktree tests.
+ *
+ * Canonical test suite:
+ *   .claude/skills/worktree/scripts/worktree.test.cjs
  */
 
-const { execSync } = require('child_process');
-const path = require('path');
-const fs = require('fs');
-
-const SCRIPT_PATH = path.join(__dirname, 'worktree.cjs');
-const STANDALONE_DIR = path.dirname(path.dirname(__dirname)); // worktree dir
-const MONOREPO_DIR = '/home/kai/claudekit';
-
-let passed = 0;
-let failed = 0;
-const results = [];
-
-// Test helper
-function run(args, options = {}) {
-  const cwd = options.cwd || STANDALONE_DIR;
-  try {
-    const output = execSync(`node "${SCRIPT_PATH}" ${args}`, {
-      encoding: 'utf-8',
-      cwd,
-      stdio: ['pipe', 'pipe', 'pipe']
-    });
-    return { success: true, output: output.trim(), exitCode: 0 };
-  } catch (error) {
-    return {
-      success: false,
-      output: error.stdout?.toString().trim() || '',
-      stderr: error.stderr?.toString().trim() || '',
-      exitCode: error.status || 1
-    };
-  }
-}
-
-function test(name, fn) {
-  try {
-    fn();
-    passed++;
-    results.push({ name, status: 'PASS' });
-    console.log(`  ✓ ${name}`);
-  } catch (error) {
-    failed++;
-    results.push({ name, status: 'FAIL', error: error.message });
-    console.log(`  ✗ ${name}`);
-    console.log(`    Error: ${error.message}`);
-  }
-}
-
-function assert(condition, message) {
-  if (!condition) throw new Error(message || 'Assertion failed');
-}
-
-function assertJSON(str) {
-  try {
-    return JSON.parse(str);
-  } catch {
-    throw new Error(`Invalid JSON: ${str.slice(0, 100)}...`);
-  }
-}
-
-// ============================================
-// INFO COMMAND TESTS
-// ============================================
-console.log('\n📋 INFO Command Tests');
-
-test('info returns valid JSON', () => {
-  const result = run('info --json');
-  assert(result.success, 'Command should succeed');
-  const json = assertJSON(result.output);
-  assert(json.info === true, 'Should have info: true');
-});
-
-test('info detects repo type', () => {
-  const result = run('info --json');
-  const json = assertJSON(result.output);
-  assert(['standalone', 'monorepo'].includes(json.repoType), 'Should detect repo type');
-});
-
-test('info detects base branch', () => {
-  const result = run('info --json');
-  const json = assertJSON(result.output);
-  assert(json.baseBranch, 'Should detect base branch');
-  assert(['dev', 'develop', 'main', 'master'].includes(json.baseBranch), 'Should be valid branch');
-});
-
-test('info finds env files', () => {
-  const result = run('info --json');
-  const json = assertJSON(result.output);
-  assert(Array.isArray(json.envFiles), 'Should have envFiles array');
-});
-
-test('info detects dirty state', () => {
-  const result = run('info --json');
-  const json = assertJSON(result.output);
-  assert(typeof json.dirtyState === 'boolean', 'Should have dirtyState boolean');
-});
-
-test('info detects monorepo from monorepo root', () => {
-  if (!fs.existsSync(MONOREPO_DIR)) return; // Skip if not available
-  const result = run('info --json', { cwd: MONOREPO_DIR });
-  const json = assertJSON(result.output);
-  assert(json.repoType === 'monorepo', 'Should detect monorepo');
-  assert(json.projects.length > 0, 'Should have projects');
-});
-
-test('info returns text output without --json', () => {
-  const result = run('info');
-  assert(result.success, 'Command should succeed');
-  assert(result.output.includes('Repository Info'), 'Should have text output');
-});
-
-// ============================================
-// LIST COMMAND TESTS
-// ============================================
-console.log('\n📂 LIST Command Tests');
-
-test('list returns valid JSON', () => {
-  const result = run('list --json');
-  assert(result.success, 'Command should succeed');
-  const json = assertJSON(result.output);
-  assert(json.success === true, 'Should have success: true');
-  assert(Array.isArray(json.worktrees), 'Should have worktrees array');
-});
-
-test('list worktrees have required fields', () => {
-  const result = run('list --json');
-  const json = assertJSON(result.output);
-  if (json.worktrees.length > 0) {
-    const wt = json.worktrees[0];
-    assert(wt.path, 'Worktree should have path');
-    assert(wt.commit, 'Worktree should have commit');
-    assert(wt.branch, 'Worktree should have branch');
-  }
-});
-
-test('list returns text output without --json', () => {
-  const result = run('list');
-  assert(result.success, 'Command should succeed');
-  assert(result.output.includes('worktrees'), 'Should have text output');
-});
-
-// ============================================
-// CREATE COMMAND TESTS
-// ============================================
-console.log('\n🆕 CREATE Command Tests');
-
-test('create requires feature name', () => {
-  const result = run('create --json');
-  assert(!result.success, 'Should fail without feature');
-  const json = assertJSON(result.output);
-  assert(json.error.code === 'MISSING_FEATURE', 'Should have MISSING_FEATURE error');
-});
-
-test('create dry-run does not create worktree', () => {
-  const result = run('create test-dry-run --prefix feat --dry-run --json');
-  assert(result.success, 'Dry-run should succeed');
-  const json = assertJSON(result.output);
-  assert(json.dryRun === true, 'Should have dryRun: true');
-  assert(json.wouldCreate, 'Should have wouldCreate object');
-});
-
-test('create dry-run shows correct branch name', () => {
-  const result = run('create my-feature --prefix fix --dry-run --json');
-  const json = assertJSON(result.output);
-  assert(json.wouldCreate.branch === 'fix/my-feature', 'Branch should be fix/my-feature');
-});
-
-test('create sanitizes feature name - spaces', () => {
-  const result = run('create "my cool feature" --dry-run --json');
-  const json = assertJSON(result.output);
-  assert(json.wouldCreate.branch.includes('my-cool-feature'), 'Should sanitize spaces');
-});
-
-test('create sanitizes feature name - uppercase', () => {
-  const result = run('create "MyFeature" --dry-run --json');
-  const json = assertJSON(result.output);
-  assert(json.wouldCreate.branch.includes('myfeature'), 'Should lowercase');
-});
-
-test('create sanitizes feature name - special chars', () => {
-  const result = run('create "feat@#$test" --dry-run --json');
-  const json = assertJSON(result.output);
-  assert(!json.wouldCreate.branch.includes('@'), 'Should remove special chars');
-});
-
-test('create respects --prefix flag', () => {
-  const prefixes = ['feat', 'fix', 'docs', 'refactor', 'test', 'chore', 'perf'];
-  for (const prefix of prefixes) {
-    const result = run(`create test-${prefix} --prefix ${prefix} --dry-run --json`);
-    const json = assertJSON(result.output);
-    assert(json.wouldCreate.branch.startsWith(`${prefix}/`), `Should use ${prefix} prefix`);
-  }
-});
-
-test('create shows base branch', () => {
-  const result = run('create test-base --dry-run --json');
-  const json = assertJSON(result.output);
-  assert(json.wouldCreate.baseBranch, 'Should show base branch');
-});
-
-test('create shows worktree path', () => {
-  const result = run('create test-path --dry-run --json');
-  const json = assertJSON(result.output);
-  assert(json.wouldCreate.worktreePath, 'Should show worktree path');
-  assert(json.wouldCreate.worktreePath.includes('worktrees'), 'Path should include worktrees dir');
-});
-
-test('create in monorepo requires project', () => {
-  if (!fs.existsSync(MONOREPO_DIR)) return;
-  const result = run('create --json', { cwd: MONOREPO_DIR });
-  assert(!result.success, 'Should fail without project in monorepo');
-  const json = assertJSON(result.output);
-  assert(json.error.code === 'MISSING_ARGS', 'Should have MISSING_ARGS error');
-});
-
-test('create in monorepo with project works', () => {
-  if (!fs.existsSync(MONOREPO_DIR)) return;
-  const result = run('create engineer test-mono --prefix feat --dry-run --json', { cwd: MONOREPO_DIR });
-  assert(result.success, 'Should succeed with project');
-  const json = assertJSON(result.output);
-  assert(json.wouldCreate.project === 'claudekit-engineer', 'Should detect project');
-});
-
-test('create detects invalid project', () => {
-  if (!fs.existsSync(MONOREPO_DIR)) return;
-  const result = run('create nonexistent test-invalid --json', { cwd: MONOREPO_DIR });
-  assert(!result.success, 'Should fail with invalid project');
-  const json = assertJSON(result.output);
-  assert(json.error.code === 'PROJECT_NOT_FOUND', 'Should have PROJECT_NOT_FOUND error');
-});
-
-// ============================================
-// REMOVE COMMAND TESTS
-// ============================================
-console.log('\n🗑️  REMOVE Command Tests');
-
-test('remove requires worktree name', () => {
-  const result = run('remove --json');
-  assert(!result.success, 'Should fail without name');
-  const json = assertJSON(result.output);
-  assert(json.error.code === 'MISSING_WORKTREE', 'Should have MISSING_WORKTREE error');
-});
-
-test('remove dry-run does not remove worktree', () => {
-  // First get a worktree name from list
-  const listResult = run('list --json');
-  const listJson = assertJSON(listResult.output);
-  const removable = listJson.worktrees.find(w => !w.path.includes('.git/'));
-
-  if (removable) {
-    const name = path.basename(removable.path);
-    const result = run(`remove "${name}" --dry-run --json`);
-    assert(result.success, 'Dry-run should succeed');
-    const json = assertJSON(result.output);
-    assert(json.dryRun === true, 'Should have dryRun: true');
-    assert(json.wouldRemove, 'Should have wouldRemove object');
-  }
-});
-
-test('remove handles not found', () => {
-  const result = run('remove nonexistent-worktree-xyz --json');
-  assert(!result.success, 'Should fail for nonexistent');
-  const json = assertJSON(result.output);
-  assert(json.error.code === 'WORKTREE_NOT_FOUND', 'Should have WORKTREE_NOT_FOUND error');
-});
-
-test('remove error includes available worktrees', () => {
-  const result = run('remove nonexistent-worktree-xyz --json');
-  const json = assertJSON(result.output);
-  assert(Array.isArray(json.error.availableWorktrees), 'Should list available worktrees');
-});
-
-// ============================================
-// ERROR HANDLING TESTS
-// ============================================
-console.log('\n⚠️  Error Handling Tests');
-
-test('unknown command returns error', () => {
-  const result = run('unknowncommand --json');
-  assert(!result.success, 'Should fail');
-  const json = assertJSON(result.output);
-  assert(json.error.code === 'UNKNOWN_COMMAND', 'Should have UNKNOWN_COMMAND error');
-});
-
-test('no command returns error', () => {
-  const result = run('--json');
-  assert(!result.success, 'Should fail');
-  const json = assertJSON(result.output);
-  assert(json.error.code === 'UNKNOWN_COMMAND', 'Should have UNKNOWN_COMMAND error');
-});
-
-test('errors have suggestion field', () => {
-  const result = run('create --json');
-  const json = assertJSON(result.output);
-  assert(json.error.suggestion, 'Error should have suggestion');
-});
-
-test('success commands return exit code 0', () => {
-  const result = run('info --json');
-  assert(result.exitCode === 0, 'Exit code should be 0');
-});
-
-test('error commands return exit code 1', () => {
-  const result = run('create --json');
-  assert(result.exitCode === 1, 'Exit code should be 1');
-});
-
-test('non-git directory returns error', () => {
-  const result = run('info --json', { cwd: '/tmp' });
-  assert(!result.success, 'Should fail in non-git dir');
-  const json = assertJSON(result.output);
-  assert(json.error.code === 'NOT_GIT_REPO', 'Should have NOT_GIT_REPO error');
-});
-
-// ============================================
-// SUMMARY
-// ============================================
-console.log('\n' + '='.repeat(50));
-console.log(`\n📊 Test Results: ${passed} passed, ${failed} failed\n`);
-
-if (failed > 0) {
-  console.log('Failed tests:');
-  results.filter(r => r.status === 'FAIL').forEach(r => {
-    console.log(`  - ${r.name}: ${r.error}`);
-  });
-  process.exit(1);
-} else {
-  console.log('✅ All tests passed!\n');
-  process.exit(0);
-}
+require('../skills/worktree/scripts/worktree.test.cjs');

package/templates/settings.json

@@ -1,242 +1,107 @@
 {
-    "alwaysThinkingEnabled": true,
-    "attribution": {
-        "commit": "Generated with [Claude Code](https://claude.com/claude-code)",
-        "pr": "Generated with [Claude Code](https://claude.com/claude-code)"
-    },
-    "cleanupPeriodDays": 30,
-    "enabledPlugins": {
-        "code-review@claude-plugins-official": true,
-        "commit-commands@claude-plugins-official": true,
-        "csharp-lsp@claude-plugins-official": true,
-        "frontend-design@claude-plugins-official": true,
-        "github@claude-plugins-official": true,
-        "playwright@claude-plugins-official": false,
-        "typescript-lsp@claude-plugins-official": true
-    },
-    "env": {
-        "CLAUDE_BASH_MAINTAIN_PROJECT_WORKING_DIR": "1"
-    },
-    "hooks": {
-        "Notification": [
-            {
-                "hooks": [
-                    {
-                        "command": "node .claude/hooks/notify-waiting.js",
-                        "type": "command"
-                    }
-                ]
-            }
-        ],
-        "PostToolUse": [
-            {
-                "hooks": [
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/post-edit-prettier.cjs",
-                        "type": "command"
-                    }
-                ],
-                "matcher": "Edit|Write"
-            }
-        ],
-        "PreCompact": [
-            {
-                "hooks": [
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/write-compact-marker.cjs",
-                        "type": "command"
-                    }
-                ],
-                "matcher": "manual|auto"
-            }
-        ],
-        "PreToolUse": [
-            {
-                "hooks": [
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/scout-block.cjs",
-                        "type": "command"
-                    },
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/privacy-block.cjs",
-                        "type": "command"
-                    }
-                ],
-                "matcher": "Bash|Glob|Grep|Read|Edit|Write"
-            },
-            {
-                "hooks": [
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/design-system-context.cjs",
-                        "type": "command"
-                    },
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/backend-csharp-context.cjs",
-                        "type": "command"
-                    },
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/frontend-typescript-context.cjs",
-                        "type": "command"
-                    },
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/scss-styling-context.cjs",
-                        "type": "command"
-                    }
-                ],
-                "matcher": "Edit|Write|MultiEdit"
-            }
-        ],
-        "SessionEnd": [
-            {
-                "hooks": [
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/session-end.cjs",
-                        "type": "command"
-                    }
-                ],
-                "matcher": "clear"
-            }
-        ],
-        "SessionStart": [
-            {
-                "hooks": [
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/session-init.cjs",
-                        "type": "command"
-                    }
-                ],
-                "matcher": "startup|resume|clear|compact"
-            }
-        ],
-        "SubagentStart": [
-            {
-                "hooks": [
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/subagent-init.cjs",
-                        "type": "command"
-                    }
-                ],
-                "matcher": "*"
-            }
-        ],
-        "UserPromptSubmit": [
-            {
-                "hooks": [
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/workflow-router.cjs",
-                        "type": "command"
-                    },
-                    {
-                        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/hooks/dev-rules-reminder.cjs",
-                        "type": "command"
-                    }
-                ]
-            }
+  "includeCoAuthoredBy": false,
+  "statusLine": {
+    "type": "command",
+    "command": "node .claude/statusline.cjs",
+    "padding": 0
+  },
+  "hooks": {
+    "SessionStart": [
+      {
+        "matcher": "startup|resume|clear|compact",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node .claude/hooks/session-init.cjs"
+          }
         ]
-    },
-    "includeCoAuthoredBy": false,
-    "permissions": {
-        "allow": [
-            "Bash(git:*)",
-            "Bash(npm:*)",
-            "Bash(npx:*)",
-            "Bash(dotnet:*)",
-            "Bash(nx:*)",
-            "Bash(node:*)",
-            "Bash(find:*)",
-            "Bash(ls:*)",
-            "Bash(cat:*)",
-            "Bash(head:*)",
-            "Bash(tail:*)",
-            "Bash(wc:*)",
-            "Bash(pwd)",
-            "Bash(echo:*)",
-            "Bash(mkdir:*)",
-            "Bash(cp:*)",
-            "Bash(mv:*)",
-            "Bash(touch:*)",
-            "Bash(grep:*)",
-            "Bash(awk:*)",
-            "Bash(sed:*)",
-            "Bash(sort:*)",
-            "Bash(uniq:*)",
-            "Bash(diff:*)",
-            "Bash(curl:*)",
-            "Bash(wget:*)",
-            "Bash(tar:*)",
-            "Bash(zip:*)",
-            "Bash(unzip:*)",
-            "Bash(cd:*)",
-            "Bash(which:*)",
-            "Bash(whoami)",
-            "Bash(date)",
-            "Bash(env)",
-            "Bash(printenv:*)",
-            "Bash(dir:*)",
-            "Bash(type:*)",
-            "Edit",
-            "ExitPlanMode",
-            "NotebookEdit",
-            "Skill",
-            "SlashCommand",
-            "WebFetch",
-            "WebSearch",
-            "Write",
-            "Read",
-            "Glob",
-            "Grep",
-            "Bash(npm run:*)",
-            "Bash(dotnet:*)",
-            "Bash(nx:*)",
-            "Bash(git:*)",
-            "mcp__filesystem__list_directory",
-            "mcp__filesystem__directory_tree",
-            "mcp__filesystem__read_text_file",
-            "mcp__filesystem__read_multiple_files"
-        ],
-        "ask": ["Bash(git push:*)", "Bash(npm publish:*)", "Bash(docker push:*)"],
-        "defaultMode": "bypassPermissions",
-        "deny": [
-            "Bash(rm -rf /*)",
-            "Bash(rm -rf ~/*)",
-            "Bash(rm -rf .)",
-            "Bash(git push --force:*)",
-            "Bash(git push -f:*)",
-            "Bash(git reset --hard:*)",
-            "Bash(git clean -fd:*)",
-            "Bash(git branch -D:*)",
-            "Bash(git stash drop:*)",
-            "Edit(**/.env*)",
-            "Edit(.env*)",
-            "Edit(**/secrets/**)",
-            "Edit(**/credentials*)",
-            "Edit(**/generated/**)",
-            "Edit(**/obj/**)",
-            "Edit(**/bin/**)",
-            "Edit(**/dist/**)",
-            "Edit(**/node_modules/**)",
-            "Edit(.github/workflows/**)",
-            "Read(**/.env*)",
-            "Read(.env*)",
-            "Read(**/secrets/**)",
-            "Read(**/credentials*)",
-            "Read(~/.ssh/**)",
-            "Read(~/.aws/**)",
-            "Read(~/.config/gcloud/**)",
-            "Write(**/.env*)",
-            "Write(.env*)",
-            "Write(**/secrets/**)",
-            "Write(**/credentials*)",
-            "Write(**/generated/**)",
-            "Write(**/obj/**)",
-            "Write(**/bin/**)",
-            "Write(**/dist/**)",
-            "Write(**/node_modules/**)"
+      }
+    ],
+    "SubagentStart": [
+      {
+        "matcher": "*",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node .claude/hooks/subagent-init.cjs"
+          },
+          {
+            "type": "command",
+            "command": "node .claude/hooks/team-context-inject.cjs"
+          }
         ]
-    },
-    "statusLine": {
-        "command": "node \"%CLAUDE_PROJECT_DIR%\"/.claude/statusline.cjs",
-        "padding": 0,
-        "type": "command"
-    }
-}
+      }
+    ],
+    "SubagentStop": [
+      {
+        "matcher": "Plan",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node .claude/hooks/cook-after-plan-reminder.cjs"
+          }
+        ]
+      }
+    ],
+    "UserPromptSubmit": [
+      {
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node .claude/hooks/dev-rules-reminder.cjs"
+          },
+          {
+            "type": "command",
+            "command": "node .claude/hooks/usage-context-awareness.cjs",
+            "timeout": 30
+          }
+        ]
+      }
+    ],
+    "PreToolUse": [
+      {
+        "matcher": "Write",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node .claude/hooks/descriptive-name.cjs"
+          }
+        ]
+      },
+      {
+        "matcher": "Bash|Glob|Grep|Read|Edit|Write",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node .claude/hooks/scout-block.cjs"
+          },
+          {
+            "type": "command",
+            "command": "node .claude/hooks/privacy-block.cjs"
+          }
+        ]
+      }
+    ],
+    "PostToolUse": [
+      {
+        "matcher": "Edit|Write|MultiEdit",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node .claude/hooks/post-edit-simplify-reminder.cjs"
+          }
+        ]
+      },
+      {
+        "matcher": "Bash|Edit|Write|MultiEdit|NotebookEdit",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "node .claude/hooks/usage-context-awareness.cjs",
+            "timeout": 10
+          }
+        ]
+      }
+    ]
+  }
+}