@ngxtm/devkit 3.0.0 → 3.0.1

package/README.md

@@ -1,236 +1,96 @@
-# Devkit Agent Assistant
+# Devkit
 
-> Unified multi-agent system with auto-synced skills from multiple sources
+> Unified multi-agent skill system for AI coding assistants
 
 [![npm version](https://img.shields.io/npm/v/@ngxtm/devkit.svg)](https://www.npmjs.com/package/@ngxtm/devkit)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 
+Supercharge your AI coding assistant with **414+ skills**, **38 agents**, and **57 commands**. Works with Claude Code, Cursor, GitHub Copilot, and Gemini.
+
 ## Features
 
-- **414+ Skills** - Merged from antigravity, agent-assistant, and claudekit
-- **38 Agents** - 20 from agent-assistant + 18 from claudekit
-- **57 Commands** - Full workflow commands from both sources
-- **9 Hooks** - Session management, privacy, context awareness (claudekit)
-- **6 Output Styles** - Coding levels from ELI5 to God mode (claudekit)
-- **10 Rules** - Coding standards for TypeScript, React, NestJS, etc.
-- **Matrix Skill Discovery** - Intelligent skill injection based on context
+- **Smart Tech Detection** - Automatically detects your project stack and loads relevant skills
+- **Per-Project Installation** - Install only what you need, where you need it
+- **Context-Optimized** - Index-only mode reduces context usage by 99.95%
 - **Auto-Sync** - Daily updates from upstream sources via GitHub Actions
+- **Multi-Tool Support** - Works with Claude, Cursor, Copilot, and Gemini
 
-## Sources
-
-| Source | What's Included |
-|--------|-----------------|
-| [antigravity-awesome-skills](https://github.com/sickn33/antigravity-awesome-skills) | 256+ skills (primary, frequently updated) |
-| [agent-assistant](https://github.com/hainamchung/agent-assistant) | 310+ skills, Matrix system, 20 agents, 25 commands |
-| [claudekit](https://github.com/anthropics/claudekit) | 45 skills, 18 agents, 32 commands, hooks, output styles |
-| [skill-rule](https://github.com/ngxtm/skill-rule) | Coding standards for 10 frameworks |
-
-## Installation
+## Quick Start
 
 ```bash
 npm install -g @ngxtm/devkit
 devkit install
 ```
 
-### Install Modes
-
-| Mode | Command | Skills Size | Description |
-|------|---------|-------------|-------------|
-| **Index-only** (default) | `devkit install` | ~30KB | Only installs skills index, Claude loads skills on-demand |
-| Minimal | `devkit install --minimal` | ~2MB | Installs ~20 core skills |
-| Category | `devkit install -c=react` | varies | Installs specific categories |
-| Full | `devkit install --full` | ~59MB | Installs all 413+ skills (may cause context limit) |
-
-```bash
-# Recommended: Index-only (default)
-devkit install                      # Best for avoiding context limits
-devkit install claude               # Claude Code only
-
-# Alternative modes
-devkit install --minimal            # ~20 core skills
-devkit install --category=react,ts  # Specific categories
-devkit install --full               # All skills (large)
-devkit install --interactive        # Choose interactively
-```
-
-### How Index-Only Mode Works
-
-Instead of installing 3,500+ files (59MB), devkit installs:
-- `SKILLS_INDEX.md` - 30KB summary of all 411 skills
-- All commands (`/plan`, `/cook`, `/brainstorm`, etc.)
-- All agents (planner, debugger, reviewer, etc.)
-- Hooks, rules, output-styles
-
-When Claude needs a specific skill, it reads the index to find it, then loads the full skill on-demand. This reduces context usage by **99.95%**.
-
-### Available Categories
+## Installation Modes
 
-```bash
-devkit categories    # Show all categories
-```
+| Mode | Command | Size | Description |
+|------|---------|------|-------------|
+| **Index-only** | `devkit install` | ~30KB | On-demand skill loading (recommended) |
+| Minimal | `devkit install --minimal` | ~2MB | ~20 core skills |
+| Category | `devkit install -c=react,ts` | varies | Specific categories only |
+| Full | `devkit install --full` | ~59MB | All 414+ skills |
 
-| Category | Skills | Description |
-|----------|--------|-------------|
-| react | 9 | React, Next.js, Remix |
-| typescript | 4 | TypeScript patterns |
-| node | 6 | Node.js, NestJS, Express |
-| database | 5 | PostgreSQL, MongoDB, Redis |
-| devops | 7 | Docker, K8s, CI/CD, AWS |
-| testing | 6 | Jest, Playwright, Vitest |
-| security | 5 | OWASP, Auth, API security |
-| ai | 6 | AI agents, MCP, prompts |
-| mobile | 5 | React Native, Flutter |
-| frontend | 6 | CSS, Tailwind, a11y |
-| backend | 6 | APIs, microservices |
-| tools | 6 | Git, debugging, docs |
-
-## Uninstallation
+## Commands
 
 ```bash
-# Remove from all tools
-devkit uninstall
-
-# Remove from specific tool
-devkit uninstall claude
-devkit uninstall cursor
+# Planning & Building
+/plan           # Create implementation strategy
+/cook           # Build feature with full workflow
+/code           # Code with workflow
+
+# Development
+/fix            # Fix bugs
+/test           # Run tests
+/review         # Code review
+/scout          # Explore codebase
+
+# Setup
+/bootstrap      # Setup new project
+/coding-level   # Set output style (eli5 → god)
+/kanban         # Manage tasks
 ```
 
-## What Gets Installed
+## Coding Levels
 
-### For Claude Code (`~/.claude/`)
-
-```
-~/.claude/
-├── skills/                   # 414+ skills
-│   └── agent-assistant/      # Core framework
-│       ├── agents/           # 20 agents
-│       ├── commands/         # 25 commands
-│       ├── matrix-skills/    # Skill discovery system
-│       └── claudekit/        # Claudekit components
-│           ├── agents/       # 18 agents
-│           └── commands/     # 32 commands
-├── rules/                    # Coding standards
-├── hooks/                    # 9 hooks
-├── output-styles/            # 6 coding levels
-├── workflows/                # Workflow definitions
-└── settings.json             # Claude settings
-```
+| Level | Command | Style |
+|-------|---------|-------|
+| 0 | `/coding-level 0` | ELI5 - Learning friendly |
+| 1 | `/coding-level 1` | Junior - Detailed comments |
+| 2 | `/coding-level 2` | Mid - Balanced |
+| 3 | `/coding-level 3` | Senior - Concise |
+| 4 | `/coding-level 4` | Lead - Architecture focus |
+| 5 | `/coding-level 5` | God - Maximum efficiency |
 
-### For Other Tools
-
-- **Cursor**: Skills, agents, commands, rules
-- **GitHub Copilot**: Skills, rules
-- **Gemini/Antigravity**: Skills, rules
-
-## Quick Start Commands
-
-After installation, use these commands in your AI coding tool:
-
-### From agent-assistant
-```
-/cook           - Build a feature with full workflow
-/plan           - Plan implementation strategy
-/review         - Code review
-/test           - Run tests
-/fix            - Fix bugs
-```
+## Categories
 
-### From claudekit
-```
-/bootstrap      - Setup new project
-/coding-level   - Set output style (eli5/junior/mid/senior/lead/god)
-/code           - Code with workflow
-/scout          - Explore codebase
-/kanban         - Manage tasks
+```bash
+devkit categories    # List all categories
 ```
 
-## Output Styles (claudekit)
-
-Set your preferred coding level:
-
-| Level | Style | Best For |
-|-------|-------|----------|
-| `/coding-level 0` | ELI5 | Learning, explanations |
-| `/coding-level 1` | Junior | Detailed comments, safe patterns |
-| `/coding-level 2` | Mid | Balanced, best practices |
-| `/coding-level 3` | Senior | Concise, optimized |
-| `/coding-level 4` | Lead | Architecture focus |
-| `/coding-level 5` | God | Minimal, maximum efficiency |
-
-## Hooks (Claude Code only)
-
-Claudekit hooks provide:
-- **session-init** - Initialize session context
-- **privacy-block** - Protect sensitive files
-- **scout-block** - Control exploration scope
-- **dev-rules-reminder** - Enforce coding standards
-- **usage-context-awareness** - Smart context management
-- And more...
-
-## Auto-Sync
-
-Skills and rules are automatically updated via GitHub Actions:
-
-- **Skills**: Daily sync from antigravity + agent-assistant
-- **Rules**: Weekly sync from skill-rule
-- **Hooks/Claudekit**: Manual sync (local source)
-
-## Configuration
+| Category | Skills | Category | Skills |
+|----------|--------|----------|--------|
+| react | 9 | database | 5 |
+| typescript | 4 | devops | 7 |
+| node | 6 | testing | 6 |
+| security | 5 | ai | 6 |
+| mobile | 5 | frontend | 6 |
+| backend | 6 | tools | 6 |
 
-Edit `SYNC_CONFIG.yaml` to customize sources:
-
-```yaml
-skill_sources:
-  - name: antigravity-awesome-skills
-    repo: sickn33/antigravity-awesome-skills
-    enabled: true
-    priority: 1
-
-  - name: agent-assistant
-    repo: hainamchung/agent-assistant
-    enabled: true
-    priority: 2
-
-sync:
-  merge_strategy: prefer-primary
-  schedule: "0 0 * * *"
-```
-
-## Development
+## Uninstall
 
 ```bash
-# Clone the repo
-git clone https://github.com/YOUR_USERNAME/devkit-assistant.git
-cd devkit-assistant
-
-# Install dependencies
-npm install
-
-# Run initial sync (first time only)
-# Windows:
-initial-sync.bat
-
-# Linux/Mac:
-./initial-sync.sh
-
-# Test install locally
-node cli/install.js
+devkit uninstall          # Remove from all tools
+devkit uninstall claude   # Remove from specific tool
 ```
 
 ## Contributing
 
 1. Fork the repository
-2. Add your skills to `skills/your-skill-name/SKILL.md`
+2. Add skills to `skills/your-skill-name/SKILL.md`
 3. Run `python scripts/update_matrix.py`
 4. Submit a pull request
 
 ## License
 
-MIT License - see [LICENSE](LICENSE) for details.
-
-## Credits
-
-- [antigravity-awesome-skills](https://github.com/sickn33/antigravity-awesome-skills) by sickn33
-- [agent-assistant](https://github.com/hainamchung/agent-assistant) by hainamchung
-- [claudekit](https://github.com/anthropics/claudekit) by Anthropic
-- [skill-rule](https://github.com/ngxtm/skill-rule) by ngxtm
+MIT

package/hooks/lib/__tests__/ck-config-utils.test.cjs

@@ -505,6 +505,11 @@ test('getGitRoot from nested subdir returns correct root', () => {
 console.log('\n=== Symlinked directory tests ===\n');
 
 test('getGitRoot resolves through symlink to git repo', () => {
+  // Skip on Windows - symlinks require elevated privileges
+  if (process.platform === 'win32') {
+    console.log('  → Skipped: Symlinks require elevated privileges on Windows');
+    return;
+  }
   const realDir = path.join(os.tmpdir(), 'ck-test-real-' + Date.now());
   const linkDir = path.join(os.tmpdir(), 'ck-test-link-' + Date.now());
   fs.mkdirSync(realDir, { recursive: true });
@@ -529,6 +534,11 @@ test('getGitRoot resolves through symlink to git repo', () => {
 });
 
 test('getGitRoot with symlinked subdirectory', () => {
+  // Skip on Windows - symlinks require elevated privileges
+  if (process.platform === 'win32') {
+    console.log('  → Skipped: Symlinks require elevated privileges on Windows');
+    return;
+  }
   const realDir = path.join(os.tmpdir(), 'ck-test-real-sub-' + Date.now());
   const subDir = path.join(realDir, 'subdir');
   const linkToSub = path.join(os.tmpdir(), 'ck-test-link-sub-' + Date.now());

package/hooks/lib/__tests__/statusline-integration.test.cjs

@@ -4,7 +4,7 @@
 /**
  * Integration Tests for Statusline Main Script
  * Tests the complete statusline.cjs with sample JSON input
- * Run: node .claude/hooks/lib/__tests__/statusline-integration.test.cjs
+ * Run: node hooks/lib/__tests__/statusline-integration.test.cjs
  */
 
 const fs = require('fs');
@@ -12,6 +12,12 @@ const path = require('path');
 const os = require('os');
 const { execSync } = require('child_process');
 
+// Determine the correct statusline.cjs path (at project root)
+const statuslinePath = path.resolve(__dirname, '..', '..', '..', 'statusline.cjs');
+
+// Check if running on Windows
+const isWindows = process.platform === 'win32';
+
 let passed = 0;
 let failed = 0;
 const failures = [];
@@ -47,6 +53,29 @@ function assertContains(actual, search, msg = '') {
   }
 }
 
+/**
+ * Cross-platform helper to run statusline with JSON input
+ * Uses temp file approach to avoid shell quoting issues on Windows
+ */
+function runStatusline(jsonInput, extraEnv = {}, options = {}) {
+  const tmpFile = path.join(os.tmpdir(), `statusline-test-${Date.now()}-${Math.random().toString(36).substring(7)}.json`);
+  try {
+    fs.writeFileSync(tmpFile, jsonInput);
+    const cmd = isWindows
+      ? `type "${tmpFile}" | node "${statuslinePath}"`
+      : `cat "${tmpFile}" | node "${statuslinePath}"`;
+    return execSync(cmd, {
+      encoding: 'utf8',
+      stdio: ['pipe', 'pipe', 'pipe'],
+      env: { ...process.env, ...extraEnv },
+      shell: true,
+      ...options
+    });
+  } finally {
+    try { fs.unlinkSync(tmpFile); } catch {}
+  }
+}
+
 console.log('\n═══════════════════════════════════════════════════════');
 console.log('STATUSLINE INTEGRATION TESTS');
 console.log('═══════════════════════════════════════════════════════\n');
@@ -64,10 +93,7 @@ const minimalInput = JSON.stringify({
 });
 
 try {
-  const result = execSync(`echo '${minimalInput.replace(/'/g, "'\\''")}'  | node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe']
-  });
+  const result = runStatusline(minimalInput);
 
   test('Minimal input produces output', () => {
     assertTrue(result.length > 0, 'Should produce some output');
@@ -109,11 +135,7 @@ try {
     context_window: { context_window_size: 200000 }
   });
 
-  const gitResult = execSync(`echo '${gitInput.replace(/'/g, "'\\''")}'  | node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    cwd: tmpDir,
-    stdio: ['pipe', 'pipe', 'pipe']
-  });
+  const gitResult = runStatusline(gitInput);
 
   test('Git input processed without error', () => {
     assertTrue(gitResult.length > 0, 'Should produce output for git repo');
@@ -153,10 +175,7 @@ const contextInput = JSON.stringify({
 });
 
 try {
-  const contextResult = execSync(`echo '${contextInput.replace(/'/g, "'\\''")}'  | node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe']
-  });
+  const contextResult = runStatusline(contextInput);
 
   test('Context window data processed', () => {
     assertTrue(contextResult.length > 0, 'Should process context window data');
@@ -180,8 +199,6 @@ try {
 
 console.log('\nTEST 4: JSON with Cost Info\n');
 
-process.env.CLAUDE_BILLING_MODE = 'api';
-
 const costInput = JSON.stringify({
   model: { display_name: 'Claude' },
   workspace: { current_dir: '/home/user' },
@@ -194,11 +211,7 @@ const costInput = JSON.stringify({
 });
 
 try {
-  const costResult = execSync(`echo '${costInput.replace(/'/g, "'\\''")}'  | CLAUDE_BILLING_MODE=api node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe'],
-    env: { ...process.env, CLAUDE_BILLING_MODE: 'api' }
-  });
+  const costResult = runStatusline(costInput, { CLAUDE_BILLING_MODE: 'api' });
 
   test('Cost info displayed in API mode', () => {
     assertTrue(costResult.length > 0, 'Should display cost info');
@@ -222,10 +235,7 @@ try {
 console.log('\nTEST 5: Invalid JSON Handling\n');
 
 try {
-  const invalidResult = execSync(`echo 'not valid json'  | node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe']
-  });
+  const invalidResult = runStatusline('not valid json');
 
   test('Invalid JSON produces fallback output', () => {
     assertTrue(invalidResult.length > 0, 'Should produce fallback output');
@@ -247,10 +257,7 @@ try {
 console.log('\nTEST 6: Empty Input Handling\n');
 
 try {
-  const emptyResult = execSync(`echo '' | node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe']
-  });
+  const emptyResult = runStatusline('');
 
   test('Empty input handled', () => {
     // Should either error gracefully or produce fallback
@@ -281,10 +288,7 @@ const multilineInput = JSON.stringify({
 });
 
 try {
-  const multilineResult = execSync(`echo '${multilineInput.replace(/'/g, "'\\''")}'  | node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe']
-  });
+  const multilineResult = runStatusline(multilineInput);
 
   test('Multi-line output generates content', () => {
     assertTrue(multilineResult.length > 0, 'Should generate output');
@@ -322,10 +326,7 @@ const expandInput = JSON.stringify({
 });
 
 try {
-  const expandResult = execSync(`echo '${expandInput.replace(/'/g, "'\\''")}'  | node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe']
-  });
+  const expandResult = runStatusline(expandInput);
 
   test('Home directory expanded to tilde', () => {
     assertTrue(expandResult.includes('~') || expandResult.includes('projects'), 'Should expand or contain path');
@@ -352,11 +353,7 @@ const colorInput = JSON.stringify({
 
 try {
   // Test with NO_COLOR=1
-  const noColorResult = execSync(`echo '${colorInput.replace(/'/g, "'\\''")}'  | NO_COLOR=1 node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe'],
-    env: { ...process.env, NO_COLOR: '1' }
-  });
+  const noColorResult = runStatusline(colorInput, { NO_COLOR: '1' });
 
   test('NO_COLOR=1 produces output', () => {
     assertTrue(noColorResult.length > 0, 'Should produce output with NO_COLOR=1');
@@ -390,11 +387,7 @@ const wideInput = JSON.stringify({
 
 let wideLines = 0;
 try {
-  const wideResult = execSync(`echo '${wideInput.replace(/'/g, "'\\''")}'  | COLUMNS=160 node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe'],
-    env: { ...process.env, COLUMNS: '160' }
-  });
+  const wideResult = runStatusline(wideInput, { COLUMNS: '160' });
   wideLines = wideResult.trim().split('\n').length;
 
   test('Wide terminal (160 cols) produces output', () => {
@@ -425,11 +418,7 @@ const narrowInput = JSON.stringify({
 });
 
 try {
-  const narrowResult = execSync(`echo '${narrowInput.replace(/'/g, "'\\''")}'  | COLUMNS=80 node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe'],
-    env: { ...process.env, COLUMNS: '80' }
-  });
+  const narrowResult = runStatusline(narrowInput, { COLUMNS: '80' });
   const narrowLines = narrowResult.trim().split('\n').length;
 
   test('Narrow terminal (80 cols) produces output', () => {
@@ -463,11 +452,7 @@ const longPathInput = JSON.stringify({
 });
 
 try {
-  const longPathResult = execSync(`echo '${longPathInput.replace(/'/g, "'\\''")}'  | COLUMNS=100 node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe'],
-    env: { ...process.env, COLUMNS: '100' }
-  });
+  const longPathResult = runStatusline(longPathInput, { COLUMNS: '100' });
 
   test('Long path produces output without crash', () => {
     assertTrue(longPathResult.length > 0, 'Should produce output');
@@ -497,11 +482,7 @@ const longModelInput = JSON.stringify({
 });
 
 try {
-  const longModelResult = execSync(`echo '${longModelInput.replace(/'/g, "'\\''")}'  | COLUMNS=100 node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe'],
-    env: { ...process.env, COLUMNS: '100' }
-  });
+  const longModelResult = runStatusline(longModelInput, { COLUMNS: '100' });
 
   test('Long model name produces output', () => {
     assertTrue(longModelResult.length > 0, 'Should produce output');
@@ -580,10 +561,7 @@ const agentTodoInput = JSON.stringify({
 });
 
 try {
-  const agentTodoResult = execSync(`echo '${agentTodoInput.replace(/'/g, "'\\''")}'  | node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe']
-  });
+  const agentTodoResult = runStatusline(agentTodoInput);
 
   test('Agent/Todo tracking produces output', () => {
     assertTrue(agentTodoResult.length > 0, 'Should produce output');
@@ -624,10 +602,7 @@ const zeroContextInput = JSON.stringify({
 });
 
 try {
-  const zeroResult = execSync(`echo '${zeroContextInput.replace(/'/g, "'\\''")}'  | node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe']
-  });
+  const zeroResult = runStatusline(zeroContextInput);
 
   test('Zero context produces output', () => {
     assertTrue(zeroResult.length > 0, 'Should produce output');
@@ -638,11 +613,7 @@ try {
 
 // Test with very small terminal
 try {
-  const tinyResult = execSync(`echo '${wideInput.replace(/'/g, "'\\''")}'  | COLUMNS=40 node .claude/statusline.cjs`, {
-    encoding: 'utf8',
-    stdio: ['pipe', 'pipe', 'pipe'],
-    env: { ...process.env, COLUMNS: '40' }
-  });
+  const tinyResult = runStatusline(wideInput, { COLUMNS: '40' });
 
   test('Very narrow terminal (40 cols) handles gracefully', () => {
     assertTrue(tinyResult.length > 0, 'Should produce output even at 40 cols');

package/hooks/scout-block/tests/test-monorepo-scenarios.cjs

@@ -138,9 +138,9 @@ const scenarios = [
     desc: 'Grep in src'
   },
   {
-    input: { tool_name: 'Glob', tool_input: { pattern: '**/*.ts' } },
+    input: { tool_name: 'Glob', tool_input: { pattern: 'src/**/*.ts' } },
     expected: 'ALLOWED',
-    desc: 'Glob all .ts files'
+    desc: 'Glob scoped .ts files in src'
   },
   {
     input: { tool_name: 'Bash', tool_input: { command: 'find packages -name "*.json" | head' } },

package/hooks/tests/test-ckignore.cjs

@@ -9,7 +9,7 @@ const { execSync } = require('child_process');
 const fs = require('fs');
 const path = require('path');
 
-const scriptPath = path.join(__dirname, '..', 'scout-block', 'scout-block.sh');
+const scriptPath = path.join(__dirname, '..', 'scout-block.cjs');
 const ckignorePath = path.join(__dirname, '..', '..', '.ckignore');
 const ckignoreBackupPath = ckignorePath + '.backup';
 
@@ -23,7 +23,7 @@ if (fs.existsSync(ckignorePath)) {
 function runTest(name, input, expected) {
   try {
     const inputJson = JSON.stringify(input);
-    execSync(`bash "${scriptPath}"`, {
+    execSync(`node "${scriptPath}"`, {
       input: inputJson,
       encoding: 'utf-8',
       stdio: ['pipe', 'pipe', 'pipe']
@@ -48,6 +48,11 @@ function restoreCkignore() {
     if (fs.existsSync(ckignoreBackupPath)) {
       fs.unlinkSync(ckignoreBackupPath);
     }
+  } else {
+    // If there was no original .ckignore, remove the test-created one
+    if (fs.existsSync(ckignorePath)) {
+      fs.unlinkSync(ckignorePath);
+    }
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngxtm/devkit",
-  "version": "3.0.0",
+  "version": "3.0.1",
   "description": "Per-project AI skills with smart tech detection - lightweight and context-optimized",
   "main": "cli/index.js",
   "bin": {

package/hooks/tests/test-modularization-hook.cjs

@@ -1,126 +0,0 @@
-#!/usr/bin/env node
-
-/**
- * Test script for modularization-hook.js (PostToolUse hook)
- * Tests if the hook correctly identifies files exceeding LOC threshold
- */
-
-const { execSync } = require('child_process');
-const path = require('path');
-const fs = require('fs');
-const os = require('os');
-
-// Create a temporary test file
-const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'modularization-test-'));
-const testFilePath = path.join(tempDir, 'test-file.js');
-
-// Create file with 250 lines (exceeds 200 LOC threshold)
-const longContent = Array(250).fill('// Test line').join('\n');
-fs.writeFileSync(testFilePath, longContent);
-
-// Create file with 50 lines (under threshold)
-const shortFilePath = path.join(tempDir, 'short-file.js');
-const shortContent = Array(50).fill('// Test line').join('\n');
-fs.writeFileSync(shortFilePath, shortContent);
-
-const testCases = [
-  {
-    name: 'Write tool with file exceeding LOC threshold',
-    input: {
-      tool_name: 'Write',
-      tool_input: {
-        file_path: testFilePath,
-        content: longContent
-      }
-    },
-    expectSuggestion: true
-  },
-  {
-    name: 'Edit tool with file exceeding LOC threshold',
-    input: {
-      tool_name: 'Edit',
-      tool_input: {
-        file_path: testFilePath,
-        old_string: '// Test line',
-        new_string: '// Modified line'
-      }
-    },
-    expectSuggestion: true
-  },
-  {
-    name: 'Write tool with short file (under threshold)',
-    input: {
-      tool_name: 'Write',
-      tool_input: {
-        file_path: shortFilePath,
-        content: shortContent
-      }
-    },
-    expectSuggestion: false
-  },
-  {
-    name: 'Write tool with non-existent file',
-    input: {
-      tool_name: 'Write',
-      tool_input: {
-        file_path: '/tmp/non-existent-file.js',
-        content: 'test'
-      }
-    },
-    expectSuggestion: false
-  }
-];
-
-console.log('Testing modularization-hook.js...\n');
-
-const scriptPath = path.join(__dirname, '..', 'modularization-hook.js');
-let passed = 0;
-let failed = 0;
-
-for (const test of testCases) {
-  try {
-    const input = JSON.stringify(test.input);
-    const result = execSync(`node "${scriptPath}"`, {
-      input,
-      encoding: 'utf-8',
-      stdio: ['pipe', 'pipe', 'pipe'],
-      env: { ...process.env, MODULARIZATION_HOOK_DEBUG: 'false' }
-    });
-
-    let hasSuggestion = false;
-    if (result && result.trim()) {
-      try {
-        const output = JSON.parse(result.trim());
-        hasSuggestion = output.hookSpecificOutput?.additionalContext?.includes('LOC');
-      } catch (e) {
-        // Not valid JSON or no output
-      }
-    }
-
-    const success = hasSuggestion === test.expectSuggestion;
-
-    if (success) {
-      console.log(`✓ ${test.name}: ${hasSuggestion ? 'suggestion shown' : 'no suggestion'}`);
-      passed++;
-    } else {
-      console.log(`✗ ${test.name}: expected ${test.expectSuggestion ? 'suggestion' : 'no suggestion'}, got ${hasSuggestion ? 'suggestion' : 'no suggestion'}`);
-      if (result) {
-        console.log(`  Output: ${result.trim()}`);
-      }
-      failed++;
-    }
-  } catch (error) {
-    console.log(`✗ ${test.name}: error executing hook`);
-    console.log(`  Error: ${error.message}`);
-    if (error.stderr) {
-      console.log(`  Stderr: ${error.stderr.toString()}`);
-    }
-    failed++;
-  }
-}
-
-// Cleanup
-fs.rmSync(tempDir, { recursive: true, force: true });
-
-console.log(`\nResults: ${passed} passed, ${failed} failed`);
-process.exit(failed > 0 ? 1 : 0);