allagents 1.0.3 → 1.0.4

package/dist/index.js

@@ -29572,7 +29572,7 @@ function classifyDeletedPath(path, client, mapping) {
   }
   return null;
 }
-function computeDeletedArtifacts(previousState, newStatePaths, clients, clientMappings) {
+function computeDeletedArtifacts(previousState, newStatePaths, clients, clientMappings, availableSkillNames) {
   if (!previousState)
     return [];
   const deleted = [];
@@ -29589,6 +29589,8 @@ function computeDeletedArtifacts(previousState, newStatePaths, clients, clientMa
       const artifact = classifyDeletedPath(path, client, mapping);
       if (!artifact)
         continue;
+      if (artifact.type === "skill" && availableSkillNames?.has(artifact.name))
+        continue;
       const key = `${client}:${artifact.type}:${artifact.name}`;
       if (!seen.has(key)) {
         seen.add(key);
@@ -29598,6 +29600,16 @@ function computeDeletedArtifacts(previousState, newStatePaths, clients, clientMa
   }
   return deleted;
 }
+async function collectAvailableSkillNames(validPlugins) {
+  const names = new Set;
+  for (const plugin of validPlugins) {
+    const skills = await collectPluginSkills(plugin.resolved, plugin.plugin);
+    for (const skill of skills) {
+      names.add(skill.folderName);
+    }
+  }
+  return names;
+}
 async function validatePlugin(pluginSource, workspacePath, offline) {
   if (isPluginSpec(pluginSource)) {
     const resolved = await resolvePluginSpecWithAutoRegister(pluginSource, {
@@ -30146,10 +30158,11 @@ ${fileValidationErrors.map((e) => `  - ${e}`).join(`
   }
   const { totalCopied, totalFailed, totalSkipped, totalGenerated } = countCopyResults(pluginResults, workspaceFileResults);
   const hasFailures = pluginResults.some((r) => !r.success) || totalFailed > 0;
+  const availableSkillNames = await collectAvailableSkillNames(validPlugins);
   const allCopyResultsForState = [...pluginResults.flatMap((r) => r.copyResults), ...workspaceFileResults];
   const resolvedMappings = resolveClientMappings(syncClients, CLIENT_MAPPINGS);
   const newStatePaths = collectSyncedPaths(allCopyResultsForState, workspacePath, syncClients, resolvedMappings);
-  const deletedArtifacts = computeDeletedArtifacts(previousState, newStatePaths, syncClients, resolvedMappings);
+  const deletedArtifacts = computeDeletedArtifacts(previousState, newStatePaths, syncClients, resolvedMappings, availableSkillNames);
   const { pluginsByClient: nativePluginsByClient } = collectNativePluginSources(validPlugins);
   if (!dryRun) {
     await persistSyncState(workspacePath, pluginResults, workspaceFileResults, syncClients, nativePluginsByClient, nativeResult, vscodeState ? { vscodeState } : undefined);
@@ -30235,10 +30248,11 @@ ${failedValidations.map((v) => `  - ${v.plugin}: ${v.error}`).join(`
     mcpResults.codex = codexMcp;
   }
   const nativeResult = await syncNativePlugins(validPlugins, previousState, "user", homeDir, dryRun, warnings, messages);
+  const availableUserSkillNames = await collectAvailableSkillNames(validPlugins);
   const allCopyResultsForState = pluginResults.flatMap((r) => r.copyResults);
   const resolvedUserMappings = resolveClientMappings(syncClients, USER_CLIENT_MAPPINGS);
   const newStatePaths = collectSyncedPaths(allCopyResultsForState, homeDir, syncClients, resolvedUserMappings);
-  const deletedArtifacts = computeDeletedArtifacts(previousState, newStatePaths, syncClients, resolvedUserMappings);
+  const deletedArtifacts = computeDeletedArtifacts(previousState, newStatePaths, syncClients, resolvedUserMappings, availableUserSkillNames);
   if (!dryRun) {
     const { pluginsByClient: nativePluginsByClient } = collectNativePluginSources(validPlugins);
     await persistSyncState(homeDir, pluginResults, [], syncClients, nativePluginsByClient, nativeResult, {
@@ -30841,28 +30855,19 @@ function formatSyncSummary(result, { dryRun = false, label = "Sync" } = {}) {
   return lines;
 }
 function formatDeletedArtifacts(artifacts) {
-  const byClient = new Map;
-  for (const artifact of artifacts) {
-    const displayName = getDisplayName(artifact.client);
-    let list = byClient.get(displayName);
-    if (!list) {
-      list = [];
-      byClient.set(displayName, list);
-    }
-    list.push(artifact);
-  }
-  return Array.from(byClient.entries()).map(([displayClient, items]) => {
-    const seen = new Set;
-    const unique = items.filter((a) => {
-      const key = `${a.type}:${a.name}`;
-      if (seen.has(key))
-        return false;
-      seen.add(key);
-      return true;
-    });
-    const names = unique.map((a) => `${a.type} '${a.name}'`).join(", ");
-    return `  Deleted (${displayClient}): ${names}`;
-  });
+  const seen = new Set;
+  const unique = [];
+  for (const a of artifacts) {
+    const key = `${a.type}:${a.name}`;
+    if (seen.has(key))
+      continue;
+    seen.add(key);
+    unique.push(a);
+  }
+  if (unique.length === 0)
+    return [];
+  const names = unique.map((a) => `${a.type} '${a.name}'`).join(", ");
+  return [`  Deleted: ${names}`];
 }
 function formatMcpResult(mcpResult, scope) {
   const { added, overwritten, removed, skipped } = mcpResult;
@@ -33411,7 +33416,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "allagents",
-    version: "1.0.3",
+    version: "1.0.4",
     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": "1.0.3",
+  "version": "1.0.4",
   "description": "CLI tool for managing multi-repo AI agent workspaces with plugin synchronization",
   "type": "module",
   "bin": {