allagents 0.32.1 → 0.32.2

package/dist/index.js

@@ -22498,7 +22498,6 @@ var init_marketplace_manifest_parser = __esm(() => {
 var exports_user_workspace = {};
 __export(exports_user_workspace, {
   setUserClients: () => setUserClients,
-  resolveGitHubIdentity: () => resolveGitHubIdentity,
   removeUserPluginsForMarketplace: () => removeUserPluginsForMarketplace,
   removeUserPlugin: () => removeUserPlugin,
   removeUserEnabledSkill: () => removeUserEnabledSkill,
@@ -22669,29 +22668,6 @@ async function removeUserPluginsForMarketplace(marketplaceName) {
   await writeFile2(configPath, dump(config, { lineWidth: -1 }), "utf-8");
   return matching.map((entry) => getPluginSource(entry));
 }
-async function resolveGitHubIdentity(pluginSource) {
-  if (isGitHubUrl(pluginSource)) {
-    const parsed = parseGitHubUrl(pluginSource);
-    return parsed ? `${parsed.owner}/${parsed.repo}`.toLowerCase() : null;
-  }
-  if (isPluginSpec(pluginSource)) {
-    const parsed = parsePluginSpec(pluginSource);
-    if (!parsed)
-      return null;
-    const marketplace = await getMarketplace(parsed.marketplaceName);
-    if (!marketplace)
-      return null;
-    const manifestResult = await parseMarketplaceManifest(marketplace.path);
-    if (!manifestResult.success)
-      return null;
-    const entry = manifestResult.data.plugins.find((p) => p.name === parsed.plugin);
-    if (!entry || typeof entry.source === "string")
-      return null;
-    const parsedUrl = parseGitHubUrl(entry.source.url);
-    return parsedUrl ? `${parsedUrl.owner}/${parsedUrl.repo}`.toLowerCase() : null;
-  }
-  return null;
-}
 async function addPluginToUserConfig(plugin, configPath, autoRegistered) {
   try {
     const content = await readFile4(configPath, "utf-8");
@@ -22912,7 +22888,6 @@ var init_user_workspace = __esm(() => {
   init_js_yaml();
   init_constants();
   init_workspace_config();
-  init_marketplace_manifest_parser();
   init_plugin_path();
   init_marketplace();
   init_marketplace();
@@ -23488,7 +23463,7 @@ async function resolvePluginSpecWithAutoRegister(spec, options = {}) {
   Expected at: ${join9(marketplace.path, expectedSubpath, pluginName)}`
     };
   }
-  const shouldReturnRegisteredAs = didAutoRegister || marketplace.name !== marketplaceName;
+  const shouldReturnRegisteredAs = didAutoRegister || marketplace.name !== marketplaceName || owner != null && subpath == null;
   const marketplaceSource = marketplace.source.type === "github" ? marketplace.source.location : undefined;
   return {
     success: true,
@@ -23670,6 +23645,19 @@ async function addPluginToConfig(plugin, configPath, autoRegistered) {
         error: `Plugin already exists in .allagents/workspace.yaml: ${plugin}`
       };
     }
+    const newIdentity = await resolveGitHubIdentity(plugin);
+    if (newIdentity) {
+      for (const existing of config.plugins) {
+        const existingSource = getPluginSource(existing);
+        const existingIdentity = await resolveGitHubIdentity(existingSource);
+        if (existingIdentity === newIdentity) {
+          return {
+            success: false,
+            error: `Plugin duplicates existing entry '${existingSource}': both resolve to ${newIdentity}`
+          };
+        }
+      }
+    }
     config.plugins.push(plugin);
     const newContent = dump(config, { lineWidth: -1 });
     await writeFile4(configPath, newContent, "utf-8");
@@ -23724,6 +23712,21 @@ async function removePlugin(plugin, workspacePath = process.cwd()) {
         return source.startsWith(`${plugin}@`) || source === plugin;
       });
     }
+    if (index === -1) {
+      const identity = await resolveGitHubIdentity(plugin);
+      if (identity) {
+        for (let i2 = 0;i2 < config.plugins.length; i2++) {
+          const p = config.plugins[i2];
+          if (!p)
+            continue;
+          const existing = await resolveGitHubIdentity(getPluginSource(p));
+          if (existing === identity) {
+            index = i2;
+            break;
+          }
+        }
+      }
+    }
     if (index === -1) {
       return {
         success: false,
@@ -23909,6 +23912,29 @@ function pruneEnabledSkillsForPlugin(config, pluginEntry) {
     config.enabledSkills = undefined;
   }
 }
+async function resolveGitHubIdentity(pluginSource) {
+  if (isGitHubUrl(pluginSource)) {
+    const parsed = parseGitHubUrl(pluginSource);
+    return parsed ? `${parsed.owner}/${parsed.repo}`.toLowerCase() : null;
+  }
+  if (isPluginSpec(pluginSource)) {
+    const parsed = parsePluginSpec(pluginSource);
+    if (!parsed)
+      return null;
+    const marketplace = await getMarketplace(parsed.marketplaceName);
+    if (!marketplace)
+      return null;
+    const manifestResult = await parseMarketplaceManifest(marketplace.path);
+    if (!manifestResult.success)
+      return null;
+    const entry = manifestResult.data.plugins.find((p) => p.name === parsed.plugin);
+    if (!entry || typeof entry.source === "string")
+      return null;
+    const parsedUrl = parseGitHubUrl(entry.source.url);
+    return parsedUrl ? `${parsedUrl.owner}/${parsedUrl.repo}`.toLowerCase() : null;
+  }
+  return null;
+}
 async function updateRepositories(changes, workspacePath = process.cwd()) {
   if (changes.remove.length === 0 && changes.add.length === 0) {
     return { success: true };
@@ -23935,6 +23961,7 @@ var init_workspace_modify = __esm(() => {
   init_constants();
   init_workspace_config();
   init_plugin_path();
+  init_marketplace_manifest_parser();
   init_marketplace();
   DEFAULT_PROJECT_CLIENTS = ["universal"];
 });
@@ -28981,7 +29008,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "allagents",
-    version: "0.32.1",
+    version: "0.32.2",
     description: "CLI tool for managing multi-repo AI agent workspaces with plugin synchronization",
     type: "module",
     bin: {

package/package.json

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