proagents 1.0.4 → 1.0.5

package/lib/commands/init.js

@@ -2,6 +2,7 @@ import { existsSync, mkdirSync, cpSync, writeFileSync, readFileSync, readdirSync
 import { join, dirname } from 'path';
 import { fileURLToPath } from 'url';
 import chalk from 'chalk';
+import yaml from 'js-yaml';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
@@ -9,11 +10,13 @@ const __dirname = dirname(__filename);
 // Files/folders to preserve during update (user customizations)
 const PRESERVE_PATHS = [
   'active-features',           // User's work in progress
-  'proagents.config.yaml',     // User's configuration
   '.learning',                 // Learned patterns
   'cache',                     // Cached analysis
 ];
 
+// Config file is handled specially - merged not preserved
+const CONFIG_FILE = 'proagents.config.yaml';
+
 // Files that should always be updated (framework files)
 const FRAMEWORK_FOLDERS = [
   'prompts',
@@ -109,7 +112,7 @@ export async function initCommand(options = {}) {
       console.log(chalk.green('\n✓ ProAgents updated successfully!\n'));
       console.log(chalk.gray('Preserved:'));
       console.log(chalk.gray('  • active-features/ (your work in progress)'));
-      console.log(chalk.gray('  • proagents.config.yaml (your configuration)'));
+      console.log(chalk.gray('  • proagents.config.yaml (your values + new options merged)'));
       console.log(chalk.gray('  • .learning/ (learned patterns)'));
       console.log(chalk.gray('  • cache/ (analysis cache)\n'));
       return;
@@ -314,6 +317,22 @@ async function smartUpdate(sourceDir, targetDir) {
   }
   console.log(chalk.green(`✓ Updated ${filesUpdated} framework files`));
 
+  // Merge config file - keep user values, add new options
+  const userConfigPath = join(targetDir, CONFIG_FILE);
+  const newConfigPath = join(sourceDir, CONFIG_FILE);
+
+  if (existsSync(newConfigPath)) {
+    const mergeResult = mergeConfigs(userConfigPath, newConfigPath);
+    if (mergeResult.newOptions > 0) {
+      console.log(chalk.green(`✓ Config merged: ${mergeResult.newOptions} new options added`));
+      if (mergeResult.newKeys.length > 0 && mergeResult.newKeys.length <= 5) {
+        console.log(chalk.gray(`  New: ${mergeResult.newKeys.join(', ')}`));
+      }
+    } else {
+      console.log(chalk.green(`✓ Config up to date`));
+    }
+  }
+
   // Ensure preserved paths still exist (they should, but just in case)
   for (const [path, fullPath] of Object.entries(backups)) {
     if (!existsSync(fullPath)) {
@@ -327,3 +346,84 @@ async function smartUpdate(sourceDir, targetDir) {
     }
   }
 }
+
+/**
+ * Deep merge two objects - source values override target, but target values are preserved
+ * Returns the merged object and list of new keys added
+ */
+function deepMerge(target, source, path = '') {
+  const result = { ...target };
+  const newKeys = [];
+
+  for (const key of Object.keys(source)) {
+    const currentPath = path ? `${path}.${key}` : key;
+
+    if (!(key in target)) {
+      // New key - add it
+      result[key] = source[key];
+      newKeys.push(currentPath);
+    } else if (
+      typeof source[key] === 'object' &&
+      source[key] !== null &&
+      !Array.isArray(source[key]) &&
+      typeof target[key] === 'object' &&
+      target[key] !== null &&
+      !Array.isArray(target[key])
+    ) {
+      // Both are objects - recurse
+      const merged = deepMerge(target[key], source[key], currentPath);
+      result[key] = merged.result;
+      newKeys.push(...merged.newKeys);
+    }
+    // If key exists in target and is not an object, keep target value
+  }
+
+  return { result, newKeys };
+}
+
+/**
+ * Merge user config with new framework config
+ * Preserves user values, adds new options from framework
+ */
+function mergeConfigs(userConfigPath, newConfigPath) {
+  try {
+    const newConfigContent = readFileSync(newConfigPath, 'utf-8');
+    const newConfig = yaml.load(newConfigContent);
+
+    if (!existsSync(userConfigPath)) {
+      // No user config - just copy the new one
+      writeFileSync(userConfigPath, newConfigContent);
+      return { newOptions: Object.keys(newConfig).length, newKeys: ['(new config created)'] };
+    }
+
+    const userConfigContent = readFileSync(userConfigPath, 'utf-8');
+    const userConfig = yaml.load(userConfigContent);
+
+    if (!userConfig || typeof userConfig !== 'object') {
+      // User config is empty or invalid - use new config
+      writeFileSync(userConfigPath, newConfigContent);
+      return { newOptions: Object.keys(newConfig).length, newKeys: ['(config replaced)'] };
+    }
+
+    // Deep merge - user values take precedence, new keys added
+    const { result: mergedConfig, newKeys } = deepMerge(userConfig, newConfig);
+
+    if (newKeys.length > 0) {
+      // Write merged config with comment
+      const header = `# ProAgents Configuration\n# User values preserved, new options added during update\n# Last updated: ${new Date().toISOString().split('T')[0]}\n\n`;
+      const mergedYaml = yaml.dump(mergedConfig, {
+        indent: 2,
+        lineWidth: 120,
+        noRefs: true,
+        sortKeys: false
+      });
+      writeFileSync(userConfigPath, header + mergedYaml);
+    }
+
+    return { newOptions: newKeys.length, newKeys };
+  } catch (error) {
+    // If merge fails, don't break the update - just warn
+    console.log(chalk.yellow(`⚠️  Could not merge config: ${error.message}`));
+    return { newOptions: 0, newKeys: [] };
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "proagents",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "description": "AI-agnostic development workflow framework that automates the full software development lifecycle",
   "type": "module",
   "main": "lib/index.js",
@@ -47,6 +47,7 @@
   },
   "dependencies": {
     "commander": "^11.1.0",
-    "chalk": "^5.3.0"
+    "chalk": "^5.3.0",
+    "js-yaml": "^4.1.0"
   }
 }