@agentic15.com/agentic15-claude-zen 4.2.3 → 5.0.4

package/{templates/.claude → framework}/settings.json

@@ -9,6 +9,7 @@
       "Bash(git log:*)",
       "Bash(git diff:*)",
       "Bash(git branch:*)",
+      "Bash(npx agentic15 status:*)",
       "Edit(./Agent/**)",
       "Write(./Agent/**)",
       "Edit(./scripts/**)",
@@ -31,8 +32,6 @@
       "Write(./.env*)",
       "Edit(./.claude/PROJECT-PLAN.json)",
       "Edit(./.claude/settings.json)",
-      "Edit(./.claude/hooks/**)",
-      "Write(./.claude/hooks/**)",
       "Edit(./.claude/TASK-TRACKER.json)",
       "Bash(agentic15:*)",
       "Bash(npx agentic15:*)",
@@ -84,7 +83,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "node .claude/hooks/session-start-context.js"
+            "command": "node node_modules/@agentic15.com/agentic15-claude-zen/framework/hooks/session-start-context.js"
           }
         ]
       }
@@ -95,7 +94,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "node .claude/hooks/require-active-task.js"
+            "command": "node node_modules/@agentic15.com/agentic15-claude-zen/framework/hooks/require-active-task.js"
           }
         ]
       },
@@ -104,7 +103,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "node .claude/hooks/enforce-plan-template.js"
+            "command": "node node_modules/@agentic15.com/agentic15-claude-zen/framework/hooks/enforce-plan-template.js"
           }
         ]
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentic15.com/agentic15-claude-zen",
-  "version": "4.2.3",
+  "version": "5.0.4",
   "description": "Structured AI-assisted development framework for Claude Code with enforced quality standards",
   "main": "src/index.js",
   "bin": {
@@ -44,6 +44,7 @@
   "files": [
     "bin/",
     "src/",
+    "framework/",
     "templates/",
     "templates/.gitignore",
     "README.md",
@@ -54,7 +55,6 @@
     "node": ">=18.0.0"
   },
   "dependencies": {
-    "@octokit/rest": "^20.0.2",
     "commander": "^12.1.0"
   },
   "devDependencies": {

package/src/cli/AuthCommand.js

@@ -1,195 +1,201 @@
-import { execSync } from 'child_process';
-import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'fs';
-import { join, dirname } from 'path';
-import { fileURLToPath } from 'url';
-import readline from 'readline';
-import { Octokit } from '@octokit/rest';
-
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
-
-export class AuthCommand {
-  static async setup() {
-    console.log('\n🔐 GitHub Authentication Setup\n');
-
-    // Step 1: Display current git configuration
-    this.displayGitConfig();
-
-    // Step 2: Confirm configuration
-    const configOk = await this.confirmConfig();
-    if (!configOk) {
-      this.showConfigCommands();
-      console.log('\n❌ Setup cancelled. Please configure git first.\n');
-      process.exit(1);
-    }
-
-    // Step 3: Get GitHub token
-    const token = await this.promptToken();
-
-    // Step 4: Validate token
-    const isValid = await this.validateToken(token);
-    if (!isValid) {
-      console.log('\n❌ Invalid GitHub token. Please try again.\n');
-      process.exit(1);
-    }
-
-    // Step 5: Auto-detect owner/repo
-    const { owner, repo } = this.detectRepo();
-
-    // Step 6: Save configuration
-    this.saveConfig(token, owner, repo);
-
-    console.log('\n✅ GitHub authentication configured successfully!\n');
-    console.log(`   Owner: ${owner}`);
-    console.log(`   Repo: ${repo}`);
-    console.log(`   Config: .claude/settings.local.json\n`);
-  }
-
-  static displayGitConfig() {
-    console.log('📋 Current Git Configuration:\n');
-
-    try {
-      // Global config
-      const globalName = execSync('git config --global user.name', { encoding: 'utf-8' }).trim();
-      const globalEmail = execSync('git config --global user.email', { encoding: 'utf-8' }).trim();
-
-      console.log('   Global:');
-      console.log(`   - user.name:  ${globalName || '(not set)'}`);
-      console.log(`   - user.email: ${globalEmail || '(not set)'}`);
-
-      // Local config (project-specific)
-      try {
-        const localName = execSync('git config --local user.name', { encoding: 'utf-8' }).trim();
-        const localEmail = execSync('git config --local user.email', { encoding: 'utf-8' }).trim();
-
-        if (localName || localEmail) {
-          console.log('\n   Local (this project):');
-          console.log(`   - user.name:  ${localName || '(not set)'}`);
-          console.log(`   - user.email: ${localEmail || '(not set)'}`);
-        }
-      } catch (e) {
-        // No local config, that's ok
-      }
-
-      console.log('');
-    } catch (error) {
-      console.log('   ⚠️  Unable to read git config. Is git installed?\n');
-    }
-  }
-
-  static showConfigCommands() {
-    console.log('\n💡 To configure git, use these commands:\n');
-    console.log('   Global (all projects):');
-    console.log('   git config --global user.name "Your Name"');
-    console.log('   git config --global user.email "your@email.com"\n');
-    console.log('   Local (this project only):');
-    console.log('   git config --local user.name "Your Name"');
-    console.log('   git config --local user.email "your@email.com"\n');
-  }
-
-  static async confirmConfig() {
-    const rl = readline.createInterface({
-      input: process.stdin,
-      output: process.stdout
-    });
-
-    return new Promise((resolve) => {
-      rl.question('Is this configuration correct? (y/n): ', (answer) => {
-        rl.close();
-        resolve(answer.toLowerCase() === 'y' || answer.toLowerCase() === 'yes');
-      });
-    });
-  }
-
-  static async promptToken() {
-    const rl = readline.createInterface({
-      input: process.stdin,
-      output: process.stdout
-    });
-
-    console.log('🔑 GitHub Personal Access Token:');
-    console.log('   Create one at: https://github.com/settings/tokens');
-    console.log('   Required scopes: repo (Full control of private repositories)\n');
-
-    return new Promise((resolve) => {
-      rl.question('Enter your GitHub token: ', (token) => {
-        rl.close();
-        resolve(token.trim());
-      });
-    });
-  }
-
-  static async validateToken(token) {
-    try {
-      const octokit = new Octokit({ auth: token });
-      await octokit.rest.users.getAuthenticated();
-      console.log('\n✓ Token validated successfully');
-      return true;
-    } catch (error) {
-      return false;
-    }
-  }
-
-  static detectRepo() {
-    try {
-      // Get remote URL
-      const remoteUrl = execSync('git config --get remote.origin.url', { encoding: 'utf-8' }).trim();
-
-      // Parse owner/repo from URL
-      // Handles: git@github.com:owner/repo.git or https://github.com/owner/repo.git
-      const match = remoteUrl.match(/github\.com[:/]([^/]+)\/(.+?)(\.git)?$/);
-
-      if (match) {
-        return {
-          owner: match[1],
-          repo: match[2]
-        };
-      }
-    } catch (error) {
-      // Fallback if no remote
-    }
-
-    // Fallback: prompt user
-    console.log('\n⚠️  Could not auto-detect repository from git remote.');
-    console.log('   Using placeholder values. Update .claude/settings.local.json manually.\n');
-
-    return {
-      owner: 'your-github-username',
-      repo: 'your-repo-name'
-    };
-  }
-
-  static saveConfig(token, owner, repo) {
-    const settingsPath = join(process.cwd(), '.claude', 'settings.local.json');
-    const settingsDir = dirname(settingsPath);
-
-    // Ensure directory exists
-    if (!existsSync(settingsDir)) {
-      mkdirSync(settingsDir, { recursive: true });
-    }
-
-    // Read existing settings or create new
-    let settings = {};
-    if (existsSync(settingsPath)) {
-      try {
-        settings = JSON.parse(readFileSync(settingsPath, 'utf-8'));
-      } catch (e) {
-        // Invalid JSON, start fresh
-      }
-    }
-
-    // Update GitHub config
-    settings.github = {
-      enabled: true,
-      autoCreate: true,
-      autoUpdate: true,
-      autoClose: true,
-      token,
-      owner,
-      repo
-    };
-
-    // Write settings
-    writeFileSync(settingsPath, JSON.stringify(settings, null, 2));
-  }
-}
+import { execSync } from 'child_process';
+import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'fs';
+import { join, dirname } from 'path';
+import { fileURLToPath } from 'url';
+import readline from 'readline';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+
+export class AuthCommand {
+  static async setup() {
+    console.log('\n🔐 GitHub Authentication Setup\n');
+
+    // Step 1: Check if gh CLI is installed
+    if (!this.isGhInstalled()) {
+      console.log('❌ GitHub CLI (gh) is not installed\n');
+      console.log('Please install GitHub CLI first:');
+      console.log('  - macOS: brew install gh');
+      console.log('  - Windows: winget install --id GitHub.cli');
+      console.log('  - Linux: https://github.com/cli/cli/blob/trunk/docs/install_linux.md\n');
+      console.log('Or visit: https://cli.github.com/\n');
+      process.exit(1);
+    }
+
+    // Step 2: Check if already authenticated with gh
+    if (!this.isGhAuthenticated()) {
+      console.log('📝 You need to authenticate with GitHub CLI\n');
+      console.log('Running: gh auth login\n');
+
+      try {
+        execSync('gh auth login', { stdio: 'inherit' });
+      } catch (error) {
+        console.log('\n❌ GitHub authentication failed\n');
+        process.exit(1);
+      }
+    } else {
+      console.log('✓ Already authenticated with GitHub CLI\n');
+    }
+
+    // Step 3: Display current git configuration
+    this.displayGitConfig();
+
+    // Step 4: Confirm configuration
+    const configOk = await this.confirmConfig();
+    if (!configOk) {
+      this.showConfigCommands();
+      console.log('\n❌ Setup cancelled. Please configure git first.\n');
+      process.exit(1);
+    }
+
+    // Step 5: Auto-detect owner/repo
+    const { owner, repo } = this.detectRepo();
+
+    // Step 6: Save configuration (no token needed, gh CLI handles auth)
+    this.saveConfig(owner, repo);
+
+    console.log('\n✅ GitHub authentication configured successfully!\n');
+    console.log(`   Owner: ${owner}`);
+    console.log(`   Repo: ${repo}`);
+    console.log(`   Auth: Using gh CLI credentials`);
+    console.log(`   Config: .claude/settings.local.json\n`);
+  }
+
+  static isGhInstalled() {
+    try {
+      execSync('gh --version', { stdio: 'pipe' });
+      return true;
+    } catch (error) {
+      return false;
+    }
+  }
+
+  static isGhAuthenticated() {
+    try {
+      execSync('gh auth status', { stdio: 'pipe' });
+      return true;
+    } catch (error) {
+      return false;
+    }
+  }
+
+  static displayGitConfig() {
+    console.log('📋 Current Git Configuration:\n');
+
+    try {
+      // Global config
+      const globalName = execSync('git config --global user.name', { encoding: 'utf-8' }).trim();
+      const globalEmail = execSync('git config --global user.email', { encoding: 'utf-8' }).trim();
+
+      console.log('   Global:');
+      console.log(`   - user.name:  ${globalName || '(not set)'}`);
+      console.log(`   - user.email: ${globalEmail || '(not set)'}`);
+
+      // Local config (project-specific)
+      try {
+        const localName = execSync('git config --local user.name', { encoding: 'utf-8' }).trim();
+        const localEmail = execSync('git config --local user.email', { encoding: 'utf-8' }).trim();
+
+        if (localName || localEmail) {
+          console.log('\n   Local (this project):');
+          console.log(`   - user.name:  ${localName || '(not set)'}`);
+          console.log(`   - user.email: ${localEmail || '(not set)'}`);
+        }
+      } catch (e) {
+        // No local config, that's ok
+      }
+
+      console.log('');
+    } catch (error) {
+      console.log('   ⚠️  Unable to read git config. Is git installed?\n');
+    }
+  }
+
+  static showConfigCommands() {
+    console.log('\n💡 To configure git, use these commands:\n');
+    console.log('   Global (all projects):');
+    console.log('   git config --global user.name "Your Name"');
+    console.log('   git config --global user.email "your@email.com"\n');
+    console.log('   Local (this project only):');
+    console.log('   git config --local user.name "Your Name"');
+    console.log('   git config --local user.email "your@email.com"\n');
+  }
+
+  static async confirmConfig() {
+    const rl = readline.createInterface({
+      input: process.stdin,
+      output: process.stdout
+    });
+
+    return new Promise((resolve) => {
+      rl.question('Is this configuration correct? (y/n): ', (answer) => {
+        rl.close();
+        resolve(answer.toLowerCase() === 'y' || answer.toLowerCase() === 'yes');
+      });
+    });
+  }
+
+
+  static detectRepo() {
+    try {
+      // Get remote URL
+      const remoteUrl = execSync('git config --get remote.origin.url', { encoding: 'utf-8' }).trim();
+
+      // Parse owner/repo from URL
+      // Handles: git@github.com:owner/repo.git or https://github.com/owner/repo.git
+      const match = remoteUrl.match(/github\.com[:/]([^/]+)\/(.+?)(\.git)?$/);
+
+      if (match) {
+        return {
+          owner: match[1],
+          repo: match[2]
+        };
+      }
+    } catch (error) {
+      // Fallback if no remote
+    }
+
+    // Fallback: prompt user
+    console.log('\n⚠️  Could not auto-detect repository from git remote.');
+    console.log('   Using placeholder values. Update .claude/settings.local.json manually.\n');
+
+    return {
+      owner: 'your-github-username',
+      repo: 'your-repo-name'
+    };
+  }
+
+  static saveConfig(owner, repo) {
+    const settingsPath = join(process.cwd(), '.claude', 'settings.local.json');
+    const settingsDir = dirname(settingsPath);
+
+    // Ensure directory exists
+    if (!existsSync(settingsDir)) {
+      mkdirSync(settingsDir, { recursive: true });
+    }
+
+    // Read existing settings or create new
+    let settings = {};
+    if (existsSync(settingsPath)) {
+      try {
+        settings = JSON.parse(readFileSync(settingsPath, 'utf-8'));
+      } catch (e) {
+        // Invalid JSON, start fresh
+      }
+    }
+
+    // Update GitHub config (no token needed, gh CLI handles auth)
+    settings.github = {
+      enabled: true,
+      autoCreate: false,
+      autoUpdate: false,
+      autoClose: false,
+      owner,
+      repo,
+      comment: "Authentication handled by gh CLI (no token needed)"
+    };
+
+    // Write settings
+    writeFileSync(settingsPath, JSON.stringify(settings, null, 2));
+  }
+}

package/src/cli/CommitCommand.js

@@ -82,11 +82,25 @@ export class CommitCommand {
         // scripts/ might not exist
       }
 
-      // Stage .claude/ framework files (ACTIVE-PLAN, plans/, TASK-TRACKER updates)
+      // Stage user-generated .claude/ files (NOT framework files)
+      // Stage: ACTIVE-PLAN, plans/, and settings.local.json
+      // DO NOT stage: settings.json, POST-INSTALL.md (generated from framework)
       try {
-        execSync('git add .claude/', { stdio: 'pipe' });
+        execSync('git add .claude/ACTIVE-PLAN', { stdio: 'pipe' });
       } catch (e) {
-        // .claude/ might not exist in some projects
+        // File might not exist
+      }
+
+      try {
+        execSync('git add .claude/plans/', { stdio: 'pipe' });
+      } catch (e) {
+        // Directory might not exist
+      }
+
+      try {
+        execSync('git add .claude/settings.local.json', { stdio: 'pipe' });
+      } catch (e) {
+        // File might not exist (it's optional)
       }
 
       // Stage package.json and package-lock.json if they exist

package/src/cli/VisualTestCommand.js

@@ -1,5 +1,5 @@
 import { execSync } from 'child_process';
-import { existsSync, writeFileSync, mkdirSync, unlinkSync } from 'fs';
+import { existsSync, readFileSync, writeFileSync, mkdirSync, unlinkSync } from 'fs';
 import { join } from 'path';
 
 /**
@@ -32,14 +32,23 @@ export class VisualTestCommand {
 
     try {
       // Run Playwright visual test
-      this.runVisualTest(url);
+      const generatedFiles = this.runVisualTest(url);
 
       console.log('\n✅ Visual test complete');
       console.log('   Screenshots saved to: .claude/visual-test/\n');
-      console.log('💡 Next steps:');
-      console.log('   1. Review screenshots in .claude/visual-test/');
-      console.log('   2. Check console-errors.log for JavaScript errors');
-      console.log('   3. Ask Claude to analyze the visual test results\n');
+
+      // Display all generated files for easy copy-paste
+      console.log('📁 Generated Files (copy these paths for Claude):');
+      console.log('─'.repeat(60));
+      generatedFiles.forEach(file => {
+        console.log(file);
+      });
+      console.log('─'.repeat(60));
+
+      console.log('\n💡 Next steps:');
+      console.log('   1. Copy the file paths above');
+      console.log('   2. Ask Claude to analyze the visual test results');
+      console.log('   3. Paste the paths when Claude asks for them\n');
 
       process.exit(0);
     } catch (error) {
@@ -68,7 +77,7 @@ export class VisualTestCommand {
   static runVisualTest(url) {
     const testScript = `
 const { chromium } = require('playwright');
-const { writeFileSync, mkdirSync } = require('fs');
+const { writeFileSync, mkdirSync, existsSync } = require('fs');
 const { join } = require('path');
 
 (async () => {
@@ -77,6 +86,7 @@ const { join } = require('path');
 
   const consoleErrors = [];
   const consoleWarnings = [];
+  const generatedFiles = [];
 
   const browser = await chromium.launch({ headless: true });
   const context = await browser.newContext({
@@ -116,16 +126,20 @@ const { join } = require('path');
     await page.waitForTimeout(2000);
 
     // Capture full page screenshot
+    const fullpagePath = join(outputDir, 'fullpage.png');
     await page.screenshot({
-      path: join(outputDir, 'fullpage.png'),
+      path: fullpagePath,
       fullPage: true
     });
+    generatedFiles.push(fullpagePath);
 
     // Capture viewport screenshot
+    const viewportPath = join(outputDir, 'viewport.png');
     await page.screenshot({
-      path: join(outputDir, 'viewport.png'),
+      path: viewportPath,
       fullPage: false
     });
+    generatedFiles.push(viewportPath);
 
     // Save console errors
     if (consoleErrors.length > 0) {
@@ -133,7 +147,9 @@ const { join } = require('path');
         \`[\${e.timestamp}] \${e.message}\${e.stack ? '\\n' + e.stack : ''}\`
       ).join('\\n\\n');
 
-      writeFileSync(join(outputDir, 'console-errors.log'), errorLog);
+      const errorLogPath = join(outputDir, 'console-errors.log');
+      writeFileSync(errorLogPath, errorLog);
+      generatedFiles.push(errorLogPath);
       console.log(\`❌ Found \${consoleErrors.length} console errors\`);
     } else {
       console.log('✅ No console errors detected');
@@ -145,22 +161,36 @@ const { join } = require('path');
         \`[\${w.timestamp}] \${w.message}\`
       ).join('\\n\\n');
 
-      writeFileSync(join(outputDir, 'console-warnings.log'), warningLog);
+      const warningLogPath = join(outputDir, 'console-warnings.log');
+      writeFileSync(warningLogPath, warningLog);
+      generatedFiles.push(warningLogPath);
       console.log(\`⚠️  Found \${consoleWarnings.length} console warnings\`);
     }
 
     // Capture page HTML for debugging
     const html = await page.content();
-    writeFileSync(join(outputDir, 'page.html'), html);
+    const htmlPath = join(outputDir, 'page.html');
+    writeFileSync(htmlPath, html);
+    generatedFiles.push(htmlPath);
+
+    // Write file list for the parent process to read
+    const fileListPath = join(outputDir, '.file-list.json');
+    writeFileSync(fileListPath, JSON.stringify(generatedFiles, null, 2));
 
   } catch (error) {
     console.error('Error during visual test:', error.message);
 
     // Try to capture screenshot even on error
     try {
+      const errorScreenshotPath = join(outputDir, 'error-screenshot.png');
       await page.screenshot({
-        path: join(outputDir, 'error-screenshot.png')
+        path: errorScreenshotPath
       });
+      generatedFiles.push(errorScreenshotPath);
+
+      // Write file list even on error
+      const fileListPath = join(outputDir, '.file-list.json');
+      writeFileSync(fileListPath, JSON.stringify(generatedFiles, null, 2));
     } catch {}
 
     throw error;
@@ -172,6 +202,7 @@ const { join } = require('path');
 
     // Write test script to temp file and execute
     const tempScriptPath = join(process.cwd(), '.claude', 'temp-visual-test.js');
+    const fileListPath = join(process.cwd(), '.claude', 'visual-test', '.file-list.json');
 
     mkdirSync(join(process.cwd(), '.claude'), { recursive: true });
     writeFileSync(tempScriptPath, testScript);
@@ -181,11 +212,23 @@ const { join } = require('path');
         stdio: 'inherit',
         cwd: process.cwd()
       });
+
+      // Read the generated file list
+      let generatedFiles = [];
+      if (existsSync(fileListPath)) {
+        const fileListContent = readFileSync(fileListPath, 'utf-8');
+        generatedFiles = JSON.parse(fileListContent);
+      }
+
+      return generatedFiles;
     } finally {
-      // Clean up temp script
+      // Clean up temp script and file list
       try {
         unlinkSync(tempScriptPath);
       } catch {}
+      try {
+        unlinkSync(fileListPath);
+      } catch {}
     }
   }
 }