allagents 0.32.1 → 0.33.0-next.1

package/README.md

@@ -155,7 +155,7 @@ Create `.allagents/template.code-workspace` for VSCode-specific settings, launch
   "settings": {
     "cSpell.words": ["myterm"],
     "chat.agent.maxRequests": 999,
-    "chat.useClaudeSkills": true
+    "chat.useAgentSkills": true
   },
   "launch": {
     "configurations": [

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"];
 });
@@ -26060,7 +26087,7 @@ var init_github_fetch = __esm(() => {
 });
 
 // src/core/workspace.ts
-import { mkdir as mkdir8, readFile as readFile9, writeFile as writeFile7, copyFile as copyFile2 } from "node:fs/promises";
+import { mkdir as mkdir8, readFile as readFile9, writeFile as writeFile7, copyFile as copyFile2, unlink as unlink3 } from "node:fs/promises";
 import { existsSync as existsSync13 } from "node:fs";
 import { join as join16, resolve as resolve10, dirname as dirname8, relative as relative5, sep as sep2, isAbsolute as isAbsolute4 } from "node:path";
 import { fileURLToPath } from "node:url";
@@ -26069,8 +26096,12 @@ async function initWorkspace(targetPath = ".", options = {}) {
   const configDir = join16(absoluteTarget, CONFIG_DIR);
   const configPath = join16(configDir, WORKSPACE_CONFIG_FILE);
   if (existsSync13(configPath)) {
-    throw new Error(`Workspace already exists: ${absoluteTarget}
+    if (options.force) {
+      await unlink3(configPath);
+    } else {
+      throw new Error(`Workspace already exists: ${absoluteTarget}
   Found existing ${CONFIG_DIR}/${WORKSPACE_CONFIG_FILE}`);
+    }
   }
   const currentFilePath = fileURLToPath(import.meta.url);
   const currentFileDir = dirname8(currentFilePath);
@@ -28981,7 +29012,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "allagents",
-    version: "0.32.1",
+    version: "0.33.0-next.1",
     description: "CLI tool for managing multi-repo AI agent workspaces with plugin synchronization",
     type: "module",
     bin: {
@@ -30630,9 +30661,10 @@ var initCmd = import_cmd_ts2.command({
   args: {
     path: import_cmd_ts2.positional({ type: import_cmd_ts2.optional(import_cmd_ts2.string), displayName: "path" }),
     from: import_cmd_ts2.option({ type: import_cmd_ts2.optional(import_cmd_ts2.string), long: "from", description: "Copy workspace.yaml from existing template/workspace" }),
-    client: import_cmd_ts2.option({ type: import_cmd_ts2.optional(import_cmd_ts2.string), long: "client", short: "c", description: "Comma-separated clients with optional :mode (e.g., claude:native,copilot,cursor)" })
+    client: import_cmd_ts2.option({ type: import_cmd_ts2.optional(import_cmd_ts2.string), long: "client", short: "c", description: "Comma-separated clients with optional :mode (e.g., claude:native,copilot,cursor)" }),
+    force: import_cmd_ts2.flag({ long: "force", short: "f", description: "Overwrite existing workspace.yaml" })
   },
-  handler: async ({ path, from, client }) => {
+  handler: async ({ path, from, client, force }) => {
     try {
       const targetPath = path ?? ".";
       let clients = client ? parseClientEntries(client) : undefined;
@@ -30649,7 +30681,8 @@ var initCmd = import_cmd_ts2.command({
       }
       const result = await initWorkspace(targetPath, {
         ...from ? { from } : {},
-        ...clients ? { clients } : {}
+        ...clients ? { clients } : {},
+        ...force ? { force } : {}
       });
       if (isJsonMode()) {
         const syncData = result.syncResult ? buildSyncData(result.syncResult) : null;
@@ -32281,6 +32314,9 @@ var pluginListCmd = import_cmd_ts4.command({
   args: {},
   handler: async () => {
     try {
+      let canonicalKey = function(name, marketplace, scope) {
+        return `${name}:${marketplace}:${scope}`;
+      };
       const pluginClients = new Map;
       async function loadConfigClients(configPath, scope) {
         if (!existsSync19(configPath))
@@ -32309,13 +32345,13 @@ var pluginListCmd = import_cmd_ts4.command({
       const projectSyncState = await loadSyncState(process.cwd());
       const merged = new Map;
       for (const p of allInstalled) {
-        const key = `${p.spec}:${p.scope}`;
+        const key = canonicalKey(p.name, p.marketplace, p.scope);
         merged.set(key, {
           spec: p.spec,
           name: p.name,
           marketplace: p.marketplace,
           scope: p.scope,
-          fileClients: pluginClients.get(key) ?? [],
+          fileClients: pluginClients.get(`${p.spec}:${p.scope}`) ?? [],
           nativeClients: []
         });
       }
@@ -32325,7 +32361,8 @@ var pluginListCmd = import_cmd_ts4.command({
       ]) {
         for (const [client, specs] of Object.entries(state?.nativePlugins ?? {})) {
           for (const spec of specs) {
-            const key = `${spec}:${scope}`;
+            const parsed = parsePluginSpec(spec);
+            const key = parsed ? canonicalKey(parsed.plugin, parsed.marketplaceName, scope) : `${spec}::${scope}`;
             const existing = merged.get(key);
             if (existing) {
               if (!existing.nativeClients.includes(client)) {
@@ -32334,8 +32371,8 @@ var pluginListCmd = import_cmd_ts4.command({
             } else {
               merged.set(key, {
                 spec,
-                name: spec.split("@")[0] ?? spec,
-                marketplace: spec.split("@")[1] ?? "",
+                name: parsed?.plugin ?? spec,
+                marketplace: parsed?.marketplaceName ?? "",
                 scope,
                 fileClients: [],
                 nativeClients: [client]

package/package.json

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