ai-agent-config 2.6.4 → 2.6.6

package/bin/cli.js

@@ -75,8 +75,8 @@ Usage: ai-agent <command> [options]
   # Update all skills
   ai-agent update
 
-  # Install to specific platform
-  ai-agent install --platform claude
+  # Force reinstall all skills
+  ai-agent install --force
 
   # Export your config to share with team
   ai-agent config export my-config.json

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ai-agent-config",
-  "version": "2.6.4",
+  "version": "2.6.6",
   "description": "Universal skill & workflow manager for AI coding assistants with bi-directional GitHub sync",
   "main": "index.js",
   "bin": {

package/scripts/config-manager.js

@@ -102,11 +102,11 @@ function migrateConfig(oldConfig) {
   const newConfig = {
     version: "2.3",
     repository: {
-      url: null,
-      branch: "main",
-      local: path.join(os.homedir(), ".ai-agent", "sync-repo"),
-      lastSync: null,
-      autoSync: true,
+      url: oldConfig.repository?.url || null,
+      branch: oldConfig.repository?.branch || "main",
+      local: oldConfig.repository?.local || path.join(os.homedir(), ".ai-agent", "sync-repo"),
+      lastSync: oldConfig.repository?.lastSync || null,
+      autoSync: oldConfig.repository?.autoSync !== undefined ? oldConfig.repository.autoSync : true,
     },
     sync: {
       conflictResolution: "pull-first",
@@ -359,13 +359,15 @@ function importConfig(inputPath, merge = false) {
   }
 
   if (merge) {
-    // Merge with existing config
+    // Merge with existing config, deduplicating by source name
     const existing = loadConfig();
+    const existingNames = new Set(existing.sources.custom.map(s => s.name));
+    const newCustom = (importData.sources.custom || []).filter(s => !existingNames.has(s.name));
     const merged = {
       ...existing,
       sources: {
         official: existing.sources.official,
-        custom: [...existing.sources.custom, ...(importData.sources.custom || [])],
+        custom: [...existing.sources.custom, ...newCustom],
       },
     };
     saveConfig(merged);

package/scripts/external-sync.js

@@ -6,7 +6,7 @@
 
 const fs = require("fs");
 const path = require("path");
-const { execSync } = require("child_process");
+const { spawnSync } = require("child_process");
 const configManager = require("./config-manager");
 
 const CACHE_DIR = path.join(require("os").homedir(), ".ai-agent-external-cache");
@@ -46,13 +46,19 @@ function syncRepo(source) {
   try {
     if (fs.existsSync(repoDir)) {
       console.log(`   Updating ${source.name}...`);
-      execSync(`git -C "${repoDir}" fetch origin`, { stdio: "pipe" });
-      execSync(`git -C "${repoDir}" reset --hard origin/${source.branch}`, { stdio: "pipe" });
+      spawnSync("git", ["-C", repoDir, "fetch", "origin"], { stdio: "pipe" });
+      const resetResult = spawnSync("git", ["-C", repoDir, "reset", "--hard", `origin/${source.branch}`], { stdio: "pipe" });
+      if (resetResult.status !== 0) {
+        throw new Error(resetResult.stderr?.toString() || "git reset failed");
+      }
     } else {
       console.log(`   Cloning ${source.name}...`);
-      execSync(`git clone --branch ${source.branch} "${source.repo}" "${repoDir}"`, {
+      const cloneResult = spawnSync("git", ["clone", "--branch", source.branch, source.repo, repoDir], {
         stdio: "pipe",
       });
+      if (cloneResult.status !== 0) {
+        throw new Error(cloneResult.stderr?.toString() || "git clone failed");
+      }
     }
     return true;
   } catch (error) {

package/scripts/secret-manager.js

@@ -228,11 +228,14 @@ function fetchSecretsFromBitwarden(sessionKey, secretNames) {
 
     try {
         // List all items in the vault (across all folders)
-        const itemsJson = execSync(`bw list items --session ${sessionKey}`, {
+        const listResult = spawnSync("bw", ["list", "items", "--session", sessionKey], {
             encoding: "utf-8",
             stdio: ["pipe", "pipe", "pipe"],
         });
-        const items = JSON.parse(itemsJson);
+        if (listResult.status !== 0) {
+            throw new Error(listResult.stderr || "Failed to list Bitwarden items");
+        }
+        const items = JSON.parse(listResult.stdout || "[]");
 
         // Match secrets by name
         for (const secretName of secretNames) {

package/scripts/sync-manager.js

@@ -5,7 +5,7 @@
 
 const fs = require("fs");
 const path = require("path");
-const { execSync } = require("child_process");
+const { execSync, spawnSync } = require("child_process");
 
 class SyncManager {
     constructor(config) {
@@ -149,10 +149,11 @@ class SyncManager {
             const branch = this.config.repository.branch || "main";
 
             // Compare local vs remote
-            const diff = execSync(`git diff HEAD origin/${branch} --name-only`, {
+            const diffResult = spawnSync("git", ["diff", "HEAD", `origin/${branch}`, "--name-only"], {
                 cwd: this.repoPath,
                 encoding: "utf-8",
             });
+            const diff = diffResult.stdout || "";
 
             return diff.trim().split("\n").filter(Boolean);
         } catch (error) {
@@ -182,7 +183,7 @@ class SyncManager {
                 const skills = fs.readdirSync(skillsDir);
                 skills.forEach(skill => {
                     if (!bundledSkills.includes(skill)) {
-                        execSync(`git add .agent/skills/${skill}`, {
+                        spawnSync("git", ["add", `.agent/skills/${skill}`], {
                             cwd: this.repoPath,
                             stdio: "pipe"
                         });
@@ -190,7 +191,13 @@ class SyncManager {
                 });
             }
 
-            execSync(`git commit -m "${message}"`, { cwd: this.repoPath, stdio: "pipe" });
+            const commitResult = spawnSync("git", ["commit", "-m", message], { cwd: this.repoPath, stdio: "pipe" });
+            if (commitResult.status !== 0) {
+                const output = (commitResult.stdout?.toString() || "") + (commitResult.stderr?.toString() || "");
+                if (!output.includes("nothing to commit")) {
+                    throw new Error(output || "git commit failed");
+                }
+            }
         } catch (error) {
             // Ignore commit errors if nothing to commit
             if (!error.message.includes("nothing to commit")) {
@@ -204,7 +211,10 @@ class SyncManager {
      */
     gitPush() {
         const branch = this.config.repository.branch || "main";
-        execSync(`git push origin ${branch}`, { cwd: this.repoPath, stdio: "inherit" });
+        const result = spawnSync("git", ["push", "origin", branch], { cwd: this.repoPath, stdio: "inherit" });
+        if (result.status !== 0) {
+            throw new Error("git push failed");
+        }
     }
 
     /**