@jaguilar87/gaia-ops 1.3.4 → 1.3.6

package/CHANGELOG.md

@@ -7,6 +7,49 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ---
 
+## [1.3.6] - 2025-11-10
+
+### Fixed
+- **Installer:** Skip questions when project context already has the answers
+- **Smart Detection:** Only ask what's missing or needs confirmation (paths)
+- **User Experience:** Show config summary when context is loaded
+- **Directory Creation:** Auto-create missing directories without prompting
+
+### Changed
+- When project context loads successfully, only asks to confirm/adjust paths
+- Cloud provider, credentials, region, and cluster name auto-applied from context
+- Clearer feedback showing what was loaded from project context
+- Missing directories (gitops, terraform, app-services) now created automatically
+
+### Improved
+- Eliminates ALL redundant questions when context exists
+- Better UX: "Here's what we loaded, just confirm the paths"
+- Faster setup for teams with complete project contexts
+- No interruptions for directory creation confirmations
+
+---
+
+## [1.3.5] - 2025-11-10
+
+### Added
+- **Smart Installer Flow:** Project context repo now asked FIRST, with auto-population of all config
+- **Input Sanitization:** Handles "git clone <url>" pastes automatically (extracts just URL)
+- **Auto-Configuration:** Parses project-context.json and pre-fills all wizard questions
+- **Better Error Messages:** Clear troubleshooting tips for git clone failures (SSH keys, access, URL)
+
+### Changed
+- **Wizard Question Order:** Project context moved from last to first question
+- **User Experience:** Reduced manual input when project context exists
+- **Clone Strategy:** Validates project context early, then sets up in final location
+- **Error Handling:** Installation continues even if project context clone fails
+
+### Improved
+- Eliminates typos and configuration errors by pre-filling from existing context
+- Saves time for users with existing project-context repos
+- Better guidance when git operations fail
+
+---
+
 ## [1.3.4] - 2025-11-10
 
 ### Fixed

package/bin/gaia-init.js

@@ -254,6 +254,81 @@ function validateConfiguration(config, nonInteractive) {
 // Interactive prompts
 // ============================================================================
 
+/**
+ * Sanitize git repo URL input
+ * Handles cases where user pastes "git clone <url>" instead of just "<url>"
+ */
+function sanitizeGitUrl(input) {
+  if (!input || typeof input !== 'string') return '';
+
+  // Trim whitespace
+  let sanitized = input.trim();
+
+  // Remove "git clone" prefix if present
+  if (sanitized.toLowerCase().startsWith('git clone ')) {
+    sanitized = sanitized.substring('git clone '.length).trim();
+  }
+
+  // Remove quotes if present
+  sanitized = sanitized.replace(/^["']|["']$/g, '');
+
+  return sanitized;
+}
+
+/**
+ * Try to clone project context repo early and parse config
+ * Returns parsed config or null if clone fails
+ */
+async function tryCloneProjectContext(repoUrl) {
+  if (!repoUrl || repoUrl.trim() === '') {
+    return null;
+  }
+
+  const sanitizedUrl = sanitizeGitUrl(repoUrl);
+
+  const spinner = ora('Cloning project context repository...').start();
+
+  try {
+    const tempDir = join(CWD, '.claude-temp-context');
+
+    // Remove temp dir if exists
+    if (existsSync(tempDir)) {
+      await fs.rm(tempDir, { recursive: true, force: true });
+    }
+
+    // Clone to temp directory
+    await execAsync(`git clone ${sanitizedUrl} ${tempDir}`, { timeout: 30000 });
+
+    // Try to read project-context.json
+    const contextPath = join(tempDir, 'project-context.json');
+
+    if (existsSync(contextPath)) {
+      const contextData = JSON.parse(await fs.readFile(contextPath, 'utf-8'));
+
+      // Clean up temp dir
+      await fs.rm(tempDir, { recursive: true, force: true });
+
+      spinner.succeed('Project context loaded successfully');
+      console.log(chalk.green('  ✓ Auto-populated configuration from project context\n'));
+
+      return {
+        contextData,
+        repoUrl: sanitizedUrl
+      };
+    } else {
+      // No project-context.json found
+      await fs.rm(tempDir, { recursive: true, force: true });
+      spinner.warn('Repository cloned but no project-context.json found');
+      return null;
+    }
+  } catch (error) {
+    spinner.fail('Failed to clone project context repository');
+    console.log(chalk.yellow(`\n⚠️  Error: ${error.message}`));
+    console.log(chalk.gray('  Continuing with manual configuration...\n'));
+    return null;
+  }
+}
+
 /**
  * Present interactive wizard to user
  */
@@ -276,36 +351,97 @@ async function runInteractiveWizard(detected) {
 
   console.log(chalk.gray('This wizard will set up the Gaia-Ops agent system for your project.\n'));
 
-  console.log(chalk.yellow('📍 Directory Configuration'));
-  console.log(chalk.gray('Gaia-Ops agents need to know where your code lives:\n'));
-  console.log(chalk.gray('  • GitOps: Kubernetes manifests that agents will monitor and deploy'));
-  console.log(chalk.gray('  • Terraform: Infrastructure code that agents will plan and apply'));
-  console.log(chalk.gray('  • App Services: Application code that agents will build and test\n'));
+  // =========================================================================
+  // STEP 1: Ask for project context repo first (if available)
+  // =========================================================================
+  console.log(chalk.yellow('🔗 Project Context (Optional but Recommended)'));
+  console.log(chalk.gray('If you have a project context repo, we can auto-populate your configuration.\n'));
+
+  const contextQuestion = await prompts({
+    type: 'text',
+    name: 'projectContextRepo',
+    message: '📦 Project context Git repo (e.g., git@bitbucket.org:org/context.git):',
+    initial: ''
+  }, {
+    onCancel: () => {
+      console.log(chalk.yellow('\n⚠️  Installation cancelled by user\n'));
+      process.exit(0);
+    }
+  });
+
+  // Try to clone and parse project context
+  let projectContext = null;
+  if (contextQuestion.projectContextRepo) {
+    projectContext = await tryCloneProjectContext(contextQuestion.projectContextRepo);
+  }
+
+  // Extract defaults from project context if available
+  const defaults = projectContext ? {
+    gitops: projectContext.contextData.paths?.gitops || detected.gitops || './gitops',
+    terraform: projectContext.contextData.paths?.terraform || detected.terraform || './terraform',
+    appServices: projectContext.contextData.paths?.app_services || detected.appServices || './app-services',
+    cloudProvider: projectContext.contextData.sections?.project_details?.cloud_provider || 'gcp',
+    gcpProjectId: projectContext.contextData.sections?.project_details?.project_id || projectContext.contextData.metadata?.project_id || '',
+    awsAccountId: projectContext.contextData.sections?.project_details?.aws_account || projectContext.contextData.metadata?.aws_account || '',
+    region: projectContext.contextData.sections?.project_details?.region || projectContext.contextData.metadata?.primary_region || 'us-central1',
+    clusterName: projectContext.contextData.sections?.project_details?.cluster_name || '',
+    repoUrl: projectContext?.repoUrl || contextQuestion.projectContextRepo
+  } : {
+    gitops: detected.gitops || './gitops',
+    terraform: detected.terraform || './terraform',
+    appServices: detected.appServices || './app-services',
+    cloudProvider: 'gcp',
+    gcpProjectId: '',
+    awsAccountId: '',
+    region: 'us-central1',
+    clusterName: '',
+    repoUrl: contextQuestion.projectContextRepo
+  };
+
+  // =========================================================================
+  // STEP 2: Ask remaining questions (with smart defaults from context)
+  // =========================================================================
+
+  // If we have project context, show a summary and only ask to confirm paths
+  if (projectContext) {
+    console.log(chalk.green('\n✅ Configuration loaded from project context:'));
+    console.log(chalk.gray(`  • Cloud: ${defaults.cloudProvider.toUpperCase()}`));
+    if (defaults.gcpProjectId) console.log(chalk.gray(`  • GCP Project: ${defaults.gcpProjectId}`));
+    if (defaults.awsAccountId) console.log(chalk.gray(`  • AWS Account: ${defaults.awsAccountId}`));
+    console.log(chalk.gray(`  • Region: ${defaults.region}`));
+    console.log(chalk.gray(`  • Cluster: ${defaults.clusterName}`));
+    console.log(chalk.yellow('\n📍 Directory Configuration'));
+    console.log(chalk.gray('Verify or adjust paths if needed:\n'));
+  } else {
+    console.log(chalk.yellow('\n📍 Directory Configuration'));
+    console.log(chalk.gray('Please provide your project configuration:\n'));
+  }
 
   const questions = [
     {
       type: 'text',
       name: 'gitops',
       message: '📦 GitOps directory:',
-      initial: detected.gitops || './gitops',
+      initial: defaults.gitops,
       validate: value => value.trim().length > 0
     },
     {
       type: 'text',
       name: 'terraform',
       message: '🔧 Terraform directory:',
-      initial: detected.terraform || './terraform',
+      initial: defaults.terraform,
       validate: value => value.trim().length > 0
     },
     {
       type: 'text',
       name: 'appServices',
       message: '🚀 App Services directory:',
-      initial: detected.appServices || './app-services',
+      initial: defaults.appServices,
       validate: value => value.trim().length > 0
     },
+    // Only ask cloud provider if not loaded from context
     {
-      type: 'select',
+      type: projectContext ? null : 'select',
       name: 'cloudProvider',
       message: '☁️  Cloud provider:',
       choices: [
@@ -313,31 +449,48 @@ async function runInteractiveWizard(detected) {
         { title: 'AWS (Amazon Web Services)', value: 'aws' },
         { title: 'Multi-cloud (AWS + GCP)', value: 'multi-cloud' }
       ],
-      initial: 0
+      initial: defaults.cloudProvider === 'gcp' ? 0 : defaults.cloudProvider === 'aws' ? 1 : 2
     },
+    // Only ask GCP Project ID if not loaded from context
     {
-      type: (prev, values) => ['gcp', 'multi-cloud'].includes(values.cloudProvider) ? 'text' : null,
+      type: (prev, values) => {
+        const provider = values.cloudProvider || defaults.cloudProvider;
+        const needsGcp = ['gcp', 'multi-cloud'].includes(provider);
+        const hasValue = projectContext && defaults.gcpProjectId;
+        return (needsGcp && !hasValue) ? 'text' : null;
+      },
       name: 'gcpProjectId',
       message: '🌐 GCP Project ID (e.g., aaxis-rnd-non-prod):',
+      initial: defaults.gcpProjectId,
       validate: value => value.trim().length > 0
     },
+    // Only ask AWS Account if not loaded from context
     {
-      type: (prev, values) => ['aws', 'multi-cloud'].includes(values.cloudProvider) ? 'text' : null,
+      type: (prev, values) => {
+        const provider = values.cloudProvider || defaults.cloudProvider;
+        const needsAws = ['aws', 'multi-cloud'].includes(provider);
+        const hasValue = projectContext && defaults.awsAccountId;
+        return (needsAws && !hasValue) ? 'text' : null;
+      },
       name: 'awsAccountId',
       message: '🌐 AWS Account ID (e.g., 929914624686):',
+      initial: defaults.awsAccountId,
       validate: value => value.trim().length > 0
     },
+    // Only ask region if not loaded from context
     {
-      type: 'text',
+      type: projectContext && defaults.region ? null : 'text',
       name: 'region',
       message: '🌍 Primary Region (e.g., us-central1 for GCP, us-east-1 for AWS):',
-      initial: 'us-central1',
+      initial: defaults.region,
       validate: value => value.trim().length > 0
     },
+    // Only ask cluster name if not loaded from context
     {
-      type: 'text',
+      type: projectContext && defaults.clusterName ? null : 'text',
       name: 'clusterName',
       message: '☸️  Cluster Name (e.g., rnd-gke-nonprod or digital-eks-prod):',
+      initial: defaults.clusterName,
       validate: value => value.trim().length > 0
     },
     {
@@ -345,12 +498,6 @@ async function runInteractiveWizard(detected) {
       name: 'installClaudeCode',
       message: '📥 Install Claude Code if not present?',
       initial: true
-    },
-    {
-      type: 'text',
-      name: 'projectContextRepo',
-      message: '🔗 Project context Git repo (optional, e.g., git@bitbucket.org:org/context.git):',
-      initial: ''
     }
   ];
 
@@ -361,13 +508,21 @@ async function runInteractiveWizard(detected) {
     }
   });
 
-  // Verify critical responses are present (some questions are conditional)
-  if (!responses.cloudProvider || !responses.clusterName) {
+  // Merge responses with defaults (for values skipped because they were in context)
+  const finalConfig = {
+    ...defaults,
+    ...responses,
+    projectContextRepo: defaults.repoUrl,
+    projectContextAlreadyCloned: !!projectContext
+  };
+
+  // Verify critical responses are present
+  if (!finalConfig.cloudProvider || !finalConfig.clusterName) {
     console.log(chalk.yellow('\n⚠️  Installation cancelled or incomplete\n'));
     process.exit(0);
   }
 
-  return responses;
+  return finalConfig;
 }
 
 // ============================================================================
@@ -507,9 +662,9 @@ async function generateAgentsMd() {
 
 /**
  * Validate and setup project paths (gitops, terraform, app-services)
- * Creates directories if they don't exist (with user confirmation in interactive mode)
+ * Creates directories automatically if they don't exist
  */
-async function validateAndSetupProjectPaths(config, nonInteractive) {
+async function validateAndSetupProjectPaths(config) {
   console.log(chalk.cyan('\n📁 Setting up project directories...\n'));
 
   const paths = {
@@ -527,30 +682,13 @@ async function validateAndSetupProjectPaths(config, nonInteractive) {
       continue;
     }
 
-    // Path doesn't exist - decide what to do
-    let shouldCreate = nonInteractive; // Auto-create in non-interactive mode
-
-    if (!nonInteractive) {
-      // Ask user in interactive mode
-      const response = await prompts({
-        type: 'confirm',
-        name: 'create',
-        message: `Directory ${chalk.yellow(userPath)} doesn't exist. Create it?`,
-        initial: true
-      });
-      shouldCreate = response.create;
-    }
-
-    if (shouldCreate) {
-      await fs.mkdir(absPath, { recursive: true });
-      console.log(chalk.green(`  ✓ ${name}: ${userPath} (created)`));
-    } else {
-      console.log(chalk.yellow(`  ⚠ ${name}: ${userPath} (skipped - agents may create it later if needed)`));
-    }
+    // Create directory automatically
+    await fs.mkdir(absPath, { recursive: true });
+    console.log(chalk.green(`  ✓ ${name}: ${userPath} (created)`));
 
     // Warn about absolute paths (portability concern)
     if (isAbsolute(userPath)) {
-      console.log(chalk.yellow(`  ⚠ Warning: Absolute path "${userPath}" may not work on other machines`));
+      console.log(chalk.yellow(`    ⚠ Note: Absolute path may not work on other machines`));
     }
   }
 
@@ -698,13 +836,58 @@ async function installClaudeAgentsPackage() {
 
 /**
  * Clone project context repository (optional)
+ * If already cloned during wizard, skip clone and just set it up in final location
  */
-async function cloneProjectContextRepo(repoUrl) {
+async function cloneProjectContextRepo(repoUrl, alreadyCloned = false) {
   if (!repoUrl || repoUrl.trim() === '') {
     console.log(chalk.gray('\n✓ Skipping project context repo clone (not provided)\n'));
     return;
   }
 
+  const sanitizedUrl = sanitizeGitUrl(repoUrl);
+
+  if (alreadyCloned) {
+    // Context was already cloned during wizard, just re-clone to final location
+    const spinner = ora('Setting up project context...').start();
+
+    try {
+      const projectContextDir = join(CWD, '.claude', 'project-context');
+
+      // Remove the generated project-context.json as it will be replaced by the cloned repo
+      const generatedFile = join(projectContextDir, 'project-context.json');
+      if (existsSync(generatedFile)) {
+        await fs.unlink(generatedFile);
+      }
+
+      // Clone fresh to final location (we already validated it works)
+      const tempDir = `${projectContextDir}-temp`;
+      await execAsync(`git clone ${sanitizedUrl} ${tempDir}`, { timeout: 30000 });
+
+      // Move contents from temp to project-context
+      const files = await fs.readdir(tempDir);
+      for (const file of files) {
+        const src = join(tempDir, file);
+        const dest = join(projectContextDir, file);
+        await fs.rename(src, dest);
+      }
+
+      // Remove temp directory
+      await fs.rm(tempDir, { recursive: true, force: true });
+
+      spinner.succeed('Project context repository configured');
+      console.log(chalk.green(`  → Location: .claude/project-context/\n`));
+    } catch (error) {
+      spinner.fail('Failed to setup project context repository');
+      console.log(chalk.yellow(`\n⚠️  You can clone it manually with:`));
+      console.log(chalk.gray(`  cd .claude`));
+      console.log(chalk.gray(`  rm -rf project-context`));
+      console.log(chalk.gray(`  git clone ${sanitizedUrl} project-context\n`));
+      // Don't throw - allow installation to continue
+    }
+    return;
+  }
+
+  // Not cloned yet during wizard, try to clone now
   const spinner = ora('Cloning project context repository...').start();
 
   try {
@@ -716,11 +899,11 @@ async function cloneProjectContextRepo(repoUrl) {
       await fs.unlink(generatedFile);
     }
 
-    // Clone repo directly into project-context directory
-    await execAsync(`git clone ${repoUrl} ${projectContextDir}-temp`);
+    // Clone repo
+    const tempDir = `${projectContextDir}-temp`;
+    await execAsync(`git clone ${sanitizedUrl} ${tempDir}`, { timeout: 30000 });
 
     // Move contents from temp to project-context
-    const tempDir = `${projectContextDir}-temp`;
     const files = await fs.readdir(tempDir);
     for (const file of files) {
       const src = join(tempDir, file);
@@ -729,17 +912,22 @@ async function cloneProjectContextRepo(repoUrl) {
     }
 
     // Remove temp directory
-    await fs.rmdir(tempDir);
+    await fs.rm(tempDir, { recursive: true, force: true });
 
     spinner.succeed('Project context repository cloned');
-    console.log(chalk.green(`  → Cloned from: ${repoUrl}`));
+    console.log(chalk.green(`  → Cloned from: ${sanitizedUrl}`));
     console.log(chalk.gray(`  → Location: .claude/project-context/\n`));
   } catch (error) {
     spinner.fail('Failed to clone project context repository');
-    console.log(chalk.yellow(`\n⚠️  You can clone it manually later with:`));
-    console.log(chalk.gray(`  cd .claude`));
-    console.log(chalk.gray(`  rm -rf project-context`));
-    console.log(chalk.gray(`  git clone ${repoUrl} project-context\n`));
+    console.log(chalk.yellow(`\n⚠️  Error: ${error.message}`));
+    console.log(chalk.gray('\n  Common issues:'));
+    console.log(chalk.gray('  • Check SSH keys are configured: ssh -T git@bitbucket.org'));
+    console.log(chalk.gray('  • Verify repository URL is correct'));
+    console.log(chalk.gray('  • Ensure you have access to the repository\n'));
+    console.log(chalk.yellow(`  You can clone it manually later with:`));
+    console.log(chalk.gray(`    cd .claude`));
+    console.log(chalk.gray(`    rm -rf project-context`));
+    console.log(chalk.gray(`    git clone ${sanitizedUrl} project-context\n`));
     // Don't throw - allow installation to continue
   }
 }
@@ -798,7 +986,7 @@ async function main() {
     await installClaudeAgentsPackage();
 
     // Step 5.5: Validate and setup project paths (gitops, terraform, app-services)
-    await validateAndSetupProjectPaths(config, args.nonInteractive);
+    await validateAndSetupProjectPaths(config);
 
     // Step 6: Create .claude/ directory with symlinks
     await createClaudeDirectory();
@@ -814,7 +1002,7 @@ async function main() {
 
     // Step 10: Clone project context repository (optional)
     if (config.projectContextRepo) {
-      await cloneProjectContextRepo(config.projectContextRepo);
+      await cloneProjectContextRepo(config.projectContextRepo, config.projectContextAlreadyCloned);
     }
 
     // Success message

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jaguilar87/gaia-ops",
-  "version": "1.3.4",
+  "version": "1.3.6",
   "description": "Multi-agent orchestration system for Claude Code - DevOps automation toolkit",
   "main": "index.js",
   "type": "module",