allagents 0.31.1 → 0.32.0-next.1

package/dist/index.js

@@ -22963,10 +22963,6 @@ async function saveRegistry(registry) {
 `);
 }
 function getSourceLocationKey(source) {
-  if (source.type === "github") {
-    const { owner, repo } = parseLocation(source.location);
-    return `${owner}/${repo}`;
-  }
   return source.location;
 }
 function findBySourceLocation(registry, sourceLocation) {
@@ -23042,10 +23038,15 @@ async function addMarketplace(source, customName, branch) {
       error: `Marketplace '${name}' already exists. Use 'update' to refresh it.`
     };
   }
-  const sourceLocation = parsed.type === "github" ? `${parseLocation(parsed.location).owner}/${parseLocation(parsed.location).repo}` : parsed.location;
+  const sourceLocation = (() => {
+    if (parsed.type !== "github")
+      return parsed.location;
+    const { owner, repo } = parseLocation(parsed.location);
+    return effectiveBranch ? `${owner}/${repo}/${effectiveBranch}` : `${owner}/${repo}`;
+  })();
   const existingBySource = findBySourceLocation(registry, sourceLocation);
   if (existingBySource) {
-    return { success: true, marketplace: existingBySource };
+    return { success: true, marketplace: existingBySource, alreadyRegistered: true };
   }
   let marketplacePath;
   if (parsed.type === "github") {
@@ -23100,7 +23101,8 @@ async function addMarketplace(source, customName, branch) {
         if (existing) {
           return {
             success: true,
-            marketplace: existing
+            marketplace: existing,
+            alreadyRegistered: true
           };
         }
         name = manifestName;
@@ -23489,12 +23491,14 @@ async function autoRegisterMarketplace(source) {
         registeredSourceCache.set(source, existing.name);
         return { success: true, name: existing.name };
       }
-      console.log(`Auto-registering GitHub marketplace: ${source}`);
       const result = await addMarketplace(source);
       if (!result.success) {
         return { success: false, error: result.error || "Unknown error" };
       }
       const name = result.marketplace?.name ?? parts[1];
+      if (!result.alreadyRegistered) {
+        console.log(`Auto-registered GitHub marketplace: ${source}`);
+      }
       registeredSourceCache.set(source, name);
       return { success: true, name };
     }
@@ -24573,6 +24577,7 @@ function executeCommand(binary2, args, options = {}) {
     const proc = spawn2(binary2, args, {
       cwd: options.cwd,
       stdio: ["ignore", "pipe", "pipe"],
+      shell: process.platform === "win32",
       env: { ...process.env }
     });
     let stdout = "";
@@ -24655,6 +24660,9 @@ async function syncCodexMcpServers(validatedPlugins, options) {
     removedServers: [],
     trackedServers: []
   };
+  if (pluginServers.size === 0 && previouslyTracked.size === 0) {
+    return result;
+  }
   const listResult = await exec("codex", ["mcp", "list", "--json"]);
   if (!listResult.success) {
     result.warnings.push(`Codex CLI not available or 'codex mcp list' failed: ${listResult.error ?? "unknown error"}`);
@@ -26346,6 +26354,122 @@ var init_status2 = __esm(() => {
 });
 
 // src/cli/format-sync.ts
+function buildPathLookup() {
+  const entries = [];
+  const seen = new Set;
+  for (const mappings of [CLIENT_MAPPINGS, USER_CLIENT_MAPPINGS]) {
+    for (const [client, mapping] of Object.entries(mappings)) {
+      const paths = [
+        [mapping.skillsPath, "skill"],
+        [mapping.commandsPath, "command"],
+        [mapping.agentsPath, "agent"],
+        [mapping.hooksPath, "hook"]
+      ];
+      for (const [path, artifactType] of paths) {
+        if (!path)
+          continue;
+        const key = `${path}|${artifactType}`;
+        if (seen.has(key))
+          continue;
+        seen.add(key);
+        entries.push({ path, client, artifactType });
+      }
+    }
+  }
+  entries.sort((a, b) => b.path.length - a.path.length);
+  return entries;
+}
+function getPathLookup() {
+  if (!cachedLookup)
+    cachedLookup = buildPathLookup();
+  return cachedLookup;
+}
+function classifyDestination(dest) {
+  const normalized = dest.replace(/\\/g, "/");
+  for (const entry of getPathLookup()) {
+    if (normalized.includes(`/${entry.path}`) || normalized.startsWith(entry.path)) {
+      return { client: entry.client, artifactType: entry.artifactType };
+    }
+  }
+  return null;
+}
+function classifyCopyResults(copyResults) {
+  const clientCounts = new Map;
+  for (const result of copyResults) {
+    if (result.action !== "copied")
+      continue;
+    const classification = classifyDestination(result.destination);
+    if (!classification)
+      continue;
+    const { client, artifactType } = classification;
+    let counts = clientCounts.get(client);
+    if (!counts) {
+      counts = { skills: 0, commands: 0, agents: 0, hooks: 0 };
+      clientCounts.set(client, counts);
+    }
+    switch (artifactType) {
+      case "skill":
+        counts.skills++;
+        break;
+      case "command":
+        counts.commands++;
+        break;
+      case "agent":
+        counts.agents++;
+        break;
+      case "hook":
+        counts.hooks++;
+        break;
+    }
+  }
+  return clientCounts;
+}
+function formatArtifactLines(clientCounts, indent = "  ") {
+  const lines = [];
+  for (const [client, counts] of clientCounts) {
+    const parts = [];
+    if (counts.commands > 0)
+      parts.push(`${counts.commands} ${counts.commands === 1 ? "command" : "commands"}`);
+    if (counts.skills > 0)
+      parts.push(`${counts.skills} ${counts.skills === 1 ? "skill" : "skills"}`);
+    if (counts.agents > 0)
+      parts.push(`${counts.agents} ${counts.agents === 1 ? "agent" : "agents"}`);
+    if (counts.hooks > 0)
+      parts.push(`${counts.hooks} ${counts.hooks === 1 ? "hook" : "hooks"}`);
+    if (parts.length > 0) {
+      lines.push(`${indent}${client}: ${parts.join(", ")}`);
+    }
+  }
+  return lines;
+}
+function formatPluginArtifacts(copyResults, indent = "  ") {
+  const copied = copyResults.filter((r) => r.action === "copied");
+  if (copied.length === 0)
+    return [];
+  const classified = classifyCopyResults(copied);
+  if (classified.size === 0) {
+    return [`${indent}Copied: ${copied.length} ${copied.length === 1 ? "file" : "files"}`];
+  }
+  return formatArtifactLines(classified, indent);
+}
+function formatSyncSummary(result, { dryRun = false, label = "Sync" } = {}) {
+  const lines = [];
+  const allCopied = result.pluginResults.flatMap((pr) => pr.copyResults.filter((r) => r.action === "copied"));
+  lines.push(`${label} complete${dryRun ? " (dry run)" : ""}:`);
+  const classified = classifyCopyResults(allCopied);
+  if (classified.size > 0) {
+    lines.push(...formatArtifactLines(classified));
+  } else if (allCopied.length > 0) {
+    lines.push(`  Total ${dryRun ? "would copy" : "copied"}: ${result.totalCopied}`);
+  }
+  if (result.totalGenerated > 0)
+    lines.push(`  Total generated: ${result.totalGenerated}`);
+  if (result.totalFailed > 0)
+    lines.push(`  Total failed: ${result.totalFailed}`);
+  if (result.totalSkipped > 0)
+    lines.push(`  Total skipped: ${result.totalSkipped}`);
+  return lines;
+}
 function formatMcpResult(mcpResult, scope) {
   const { added, overwritten, removed, skipped } = mcpResult;
   if (added === 0 && overwritten === 0 && removed === 0 && skipped === 0) {
@@ -26435,6 +26559,10 @@ function buildSyncData(result) {
     }
   };
 }
+var cachedLookup = null;
+var init_format_sync = __esm(() => {
+  init_client_mapping();
+});
 
 // node_modules/picocolors/picocolors.js
 var require_picocolors = __commonJS((exports, module) => {
@@ -28815,7 +28943,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "allagents",
-    version: "0.31.1",
+    version: "0.32.0-next.1",
     description: "CLI tool for managing multi-repo AI agent workspaces with plugin synchronization",
     type: "module",
     bin: {
@@ -29103,7 +29231,7 @@ async function runSync(context) {
       } else {
         const lines = result.pluginResults.map((pr) => `${pr.success ? "✓" : "✗"} ${pr.plugin}`);
         lines.push("");
-        lines.push(`Copied: ${result.totalCopied}  Failed: ${result.totalFailed}  Skipped: ${result.totalSkipped}`);
+        lines.push(...formatSyncSummary(result));
         if (result.nativeResult) {
           lines.push(...formatNativeResult(result.nativeResult));
         }
@@ -29120,7 +29248,7 @@ async function runSync(context) {
       } else {
         const lines = userResult.pluginResults.map((pr) => `${pr.success ? "✓" : "✗"} ${pr.plugin}`);
         lines.push("");
-        lines.push(`Copied: ${userResult.totalCopied}  Failed: ${userResult.totalFailed}  Skipped: ${userResult.totalSkipped}`);
+        lines.push(...formatSyncSummary(userResult));
         if (userResult.mcpResults) {
           for (const [scope, mcpResult] of Object.entries(userResult.mcpResults)) {
             if (!mcpResult)
@@ -29143,6 +29271,7 @@ async function runSync(context) {
 var init_sync2 = __esm(() => {
   init_dist2();
   init_sync();
+  init_format_sync();
 });
 
 // src/cli/tui/actions/init.ts
@@ -30429,6 +30558,7 @@ var repoListMeta = {
 };
 
 // src/cli/commands/workspace.ts
+init_format_sync();
 function parseClientEntries(input) {
   const validClients = ClientTypeSchema.options;
   const validModes = InstallModeSchema.options;
@@ -30505,10 +30635,9 @@ Plugin sync results:`);
             }
           }
         }
-        console.log(`
-Sync complete: ${syncResult.totalCopied} files copied`);
-        if (syncResult.totalFailed > 0) {
-          console.log(`  Failed: ${syncResult.totalFailed}`);
+        console.log("");
+        for (const line of formatSyncSummary(syncResult)) {
+          console.log(line);
         }
       }
     } catch (error) {
@@ -30608,11 +30737,11 @@ var syncCmd = import_cmd_ts2.command({
         if (pluginResult.error) {
           console.log(`  Error: ${pluginResult.error}`);
         }
-        const copied = pluginResult.copyResults.filter((r) => r.action === "copied").length;
+        for (const line of formatPluginArtifacts(pluginResult.copyResults)) {
+          console.log(line);
+        }
         const generated = pluginResult.copyResults.filter((r) => r.action === "generated").length;
         const failed = pluginResult.copyResults.filter((r) => r.action === "failed").length;
-        if (copied > 0)
-          console.log(`  Copied: ${copied} files`);
         if (generated > 0)
           console.log(`  Generated: ${generated} files`);
         if (failed > 0) {
@@ -30658,15 +30787,10 @@ native:`);
           }
         }
       }
-      console.log(`
-Sync complete${dryRun ? " (dry run)" : ""}:`);
-      console.log(`  Total ${dryRun ? "would copy" : "copied"}: ${result.totalCopied}`);
-      if (result.totalGenerated > 0)
-        console.log(`  Total generated: ${result.totalGenerated}`);
-      if (result.totalFailed > 0)
-        console.log(`  Total failed: ${result.totalFailed}`);
-      if (result.totalSkipped > 0)
-        console.log(`  Total skipped: ${result.totalSkipped}`);
+      console.log("");
+      for (const line of formatSyncSummary(result, { dryRun })) {
+        console.log(line);
+      }
       if (!result.success || result.totalFailed > 0) {
         process.exit(1);
       }
@@ -31641,6 +31765,7 @@ var skillsCmd = conciseSubcommands({
 });
 
 // src/cli/commands/plugin.ts
+init_format_sync();
 init_workspace_config();
 init_constants();
 init_js_yaml();
@@ -31668,11 +31793,11 @@ Syncing workspace...
       if (pluginResult.error) {
         console.log(`  Error: ${pluginResult.error}`);
       }
-      const copied = pluginResult.copyResults.filter((r) => r.action === "copied").length;
+      for (const line of formatPluginArtifacts(pluginResult.copyResults)) {
+        console.log(line);
+      }
       const generated = pluginResult.copyResults.filter((r) => r.action === "generated").length;
       const failed = pluginResult.copyResults.filter((r) => r.action === "failed").length;
-      if (copied > 0)
-        console.log(`  Copied: ${copied} files`);
       if (generated > 0)
         console.log(`  Generated: ${generated} files`);
       if (failed > 0) {
@@ -31692,17 +31817,9 @@ native:`);
         }
       }
     }
-    console.log(`
-Sync complete:`);
-    console.log(`  Total copied: ${result.totalCopied}`);
-    if (result.totalGenerated > 0) {
-      console.log(`  Total generated: ${result.totalGenerated}`);
-    }
-    if (result.totalFailed > 0) {
-      console.log(`  Total failed: ${result.totalFailed}`);
-    }
-    if (result.totalSkipped > 0) {
-      console.log(`  Total skipped: ${result.totalSkipped}`);
+    console.log("");
+    for (const line of formatSyncSummary(result)) {
+      console.log(line);
     }
   }
   return { ok: result.success && result.totalFailed === 0, syncData };
@@ -31728,11 +31845,11 @@ Syncing user workspace...
       if (pluginResult.error) {
         console.log(`  Error: ${pluginResult.error}`);
       }
-      const copied = pluginResult.copyResults.filter((r) => r.action === "copied").length;
+      for (const line of formatPluginArtifacts(pluginResult.copyResults)) {
+        console.log(line);
+      }
       const generated = pluginResult.copyResults.filter((r) => r.action === "generated").length;
       const failed = pluginResult.copyResults.filter((r) => r.action === "failed").length;
-      if (copied > 0)
-        console.log(`  Copied: ${copied} files`);
       if (generated > 0)
         console.log(`  Generated: ${generated} files`);
       if (failed > 0) {
@@ -31765,17 +31882,9 @@ native:`);
         }
       }
     }
-    console.log(`
-User sync complete:`);
-    console.log(`  Total copied: ${result.totalCopied}`);
-    if (result.totalGenerated > 0) {
-      console.log(`  Total generated: ${result.totalGenerated}`);
-    }
-    if (result.totalFailed > 0) {
-      console.log(`  Total failed: ${result.totalFailed}`);
-    }
-    if (result.totalSkipped > 0) {
-      console.log(`  Total skipped: ${result.totalSkipped}`);
+    console.log("");
+    for (const line of formatSyncSummary(result, { label: "User sync" })) {
+      console.log(line);
     }
   }
   return { ok: result.success && result.totalFailed === 0, syncData };
@@ -32336,7 +32445,7 @@ Enabled skills: ${skills.join(", ")}`);
       }
       if (!isJsonMode()) {
         if (result.autoRegistered) {
-          console.log(`✓ Auto-registered marketplace: ${result.autoRegistered}`);
+          console.log(`  Resolved marketplace: ${result.autoRegistered}`);
         }
         console.log(`✓ Installed plugin (${isUser ? "user" : "project"} scope): ${displayPlugin}`);
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "allagents",
-  "version": "0.31.1",
+  "version": "0.32.0-next.1",
   "description": "CLI tool for managing multi-repo AI agent workspaces with plugin synchronization",
   "type": "module",
   "bin": {