devtronic 1.2.4 → 1.2.5

package/README.md

@@ -20,8 +20,8 @@ The CLI analyzes your project (framework, architecture, stack) and generates per
 | `doctor [--fix]` | Run health diagnostics |
 | `list [skills\|agents]` | List installed skills and agents |
 | `config` | View or manage project configuration |
-| `addon enable <name>` | Add an addon skill pack |
-| `addon disable <name>` | Remove an addon skill pack |
+| `addon add <name>` | Add an addon skill pack |
+| `addon remove <name>` | Remove an addon skill pack |
 | `addon list` | List available and installed addons |
 | `addon sync` | Regenerate addon files for current agents |
 | `add <ide>` | Add another IDE |
@@ -45,11 +45,11 @@ The CLI analyzes your project (framework, architecture, stack) and generates per
 Three optional addon packs extend the core toolkit. Select them during `init` or manage them at any time:
 
 ```bash
-npx devtronic addon list                          # See available addons and status
-npx devtronic addon enable orchestration          # Install
-npx devtronic addon enable design-best-practices
-npx devtronic addon enable auto-devtronic
-npx devtronic addon disable <name>                # Uninstall
+npx devtronic addon list                        # See available addons and status
+npx devtronic addon add orchestration           # Install
+npx devtronic addon add design-best-practices
+npx devtronic addon add auto-devtronic
+npx devtronic addon remove <name>               # Uninstall
 ```
 
 | Addon | Skills | Description |

package/dist/index.js

@@ -1469,7 +1469,7 @@ Valid addons: ${validAddons.join(", ")}`);
         p3.log.warn(`Template not found: ${templateName}`);
         continue;
       }
-      const resolution = conflictResolutions.get(ide) || "overwrite";
+      const resolution = conflictResolutions.get(ide) || "replace";
       const files = getAllFilesRecursive(templateDir);
       const dynamicFiles = DYNAMIC_RULE_FILES[ide] || [];
       for (const file of files) {
@@ -1507,7 +1507,7 @@ Valid addons: ${validAddons.join(", ")}`);
         const rulePath = dynamicFiles[0];
         if (rulePath) {
           const destPath = join7(targetDir, rulePath);
-          const resolution2 = conflictResolutions.get(ide) || "overwrite";
+          const resolution2 = conflictResolutions.get(ide) || "replace";
           if (fileExists(destPath) && resolution2 === "keep") {
             skippedFiles.push(rulePath);
           } else if (fileExists(destPath) && resolution2 === "merge") {
@@ -1795,14 +1795,18 @@ function readAddonConfig(targetDir) {
   if (!existsSync7(configPath)) {
     return { agents: ["claude"], installed: {} };
   }
-  const raw = JSON.parse(readFileSync3(configPath, "utf-8"));
-  const data = raw.addons ?? raw;
-  return {
-    version: 1,
-    mode: data.mode,
-    agents: data.agents ?? ["claude"],
-    installed: data.installed ?? {}
-  };
+  try {
+    const raw = JSON.parse(readFileSync3(configPath, "utf-8"));
+    const data = raw.addons ?? raw;
+    return {
+      version: 1,
+      mode: data.mode,
+      agents: data.agents ?? ["claude"],
+      installed: data.installed ?? {}
+    };
+  } catch {
+    return { agents: ["claude"], installed: {} };
+  }
 }
 function writeAddonConfig(targetDir, config) {
   const configPath = getConfigPath(targetDir);
@@ -1990,9 +1994,9 @@ function generateAddonFiles(projectDir, addonSourceDir, agents) {
         const existing = readFileSync4(destPath, "utf-8");
         if (existing === content) {
           result.skipped++;
-          continue;
+        } else {
+          result.conflicts.push(relPath);
         }
-        result.skipped++;
         continue;
       }
       ensureDir2(dirname5(destPath));
@@ -2120,8 +2124,10 @@ function removeAddonFiles(projectDir, addonName, agents, addonSourceDir) {
       }
     }
   }
-  const noticePath = join10(projectDir, "NOTICE.md");
-  if (existsSync8(noticePath)) unlinkSync(noticePath);
+  if (manifest.attribution) {
+    const noticePath = join10(projectDir, "NOTICE.md");
+    if (existsSync8(noticePath)) unlinkSync(noticePath);
+  }
 }
 function syncAddonFiles(projectDir, addonSourceDir, agents) {
   const fileMap = buildFileMap(addonSourceDir);
@@ -2276,18 +2282,22 @@ function syncAddonFiles(projectDir, addonSourceDir, agents) {
 }
 function detectModifiedAddonFiles(projectDir, addonName) {
   let installedChecksums = {};
+  let agents = ["claude"];
   try {
     const config = readAddonConfig(projectDir);
     const installed = config.installed?.[addonName];
     if (!installed?.checksums) return [];
     installedChecksums = installed.checksums;
+    agents = config.agents ?? ["claude"];
   } catch {
     return [];
   }
   const modified = [];
-  for (const agentDir of Object.values(AGENT_PATHS)) {
+  for (const agent of agents) {
+    const spec = RUNTIME_SPECS[agent];
+    const baseDir = spec?.baseDir ?? AGENT_PATHS[agent] ?? `.${agent}`;
     for (const [relPath, originalHash] of Object.entries(installedChecksums)) {
-      const absPath = join10(projectDir, agentDir, relPath);
+      const absPath = join10(projectDir, baseDir, relPath);
       if (!existsSync8(absPath)) continue;
       const current = checksum(readFileSync4(absPath, "utf-8"));
       if (current !== originalHash) {
@@ -2629,6 +2639,7 @@ async function updateCommand(options) {
     syncSpinner.start("Updating enabled addon files...");
     let totalUpdated = 0;
     for (const name of enabledAddons) {
+      if (name === "orchestration") continue;
       const addonSourceDir = getAddonSourceDir(name);
       const result = syncAddonFiles(targetDir, addonSourceDir, addonConfig.agents);
       totalUpdated += (result.updated ?? 0) + result.written;
@@ -4407,7 +4418,7 @@ function readdirSafe(dir) {
 
 // src/commands/addon.ts
 import { resolve as resolve14, join as join20, dirname as dirname10 } from "path";
-import { existsSync as existsSync17, unlinkSync as unlinkSync3, rmSync as rmSync3, readFileSync as readFileSync6 } from "fs";
+import { existsSync as existsSync17, unlinkSync as unlinkSync3, rmSync as rmSync3 } from "fs";
 import * as p14 from "@clack/prompts";
 import chalk15 from "chalk";
 function isFileBasedAddon(addonName) {
@@ -4425,8 +4436,8 @@ Valid addons: ${validAddons.join(", ")}`);
   }
   const typedName = addonName;
   const canonicalAction = action === "enable" ? "add" : action === "disable" ? "remove" : action;
-  if (action === "add" || action === "remove") {
-    const canonical = action === "add" ? "enable" : "disable";
+  if (action === "enable" || action === "disable") {
+    const canonical = action === "enable" ? "add" : "remove";
     p14.log.warn(
       `"addon ${action}" is deprecated. Use "addon ${canonical}" instead.`
     );
@@ -4624,16 +4635,14 @@ async function addFileBasedAddon(targetDir, addonName, _options) {
   spinner8.start(`Adding ${addon.label}...`);
   const addonSourceDir = getAddonSourceDir(addonName);
   const result = generateAddonFiles(targetDir, addonSourceDir, config.agents);
-  const addonManifest = JSON.parse(
-    readFileSync6(join20(addonSourceDir, "manifest.json"), "utf-8")
-  );
+  const addonMeta = getAddonManifest(addonName);
   const fileList = [
-    ...(addonManifest.files.skills ?? []).map((s) => `skills/${s}`),
-    ...(addonManifest.files.agents ?? []).map((a) => `agents/${a}.md`),
-    ...(addonManifest.files.rules ?? []).map((r) => `rules/${r}`)
+    ...(addonMeta.files.skills ?? []).map((s) => `skills/${s}`),
+    ...(addonMeta.files.agents ?? []).map((a) => `agents/${a}.md`),
+    ...(addonMeta.files.rules ?? []).map((r) => `rules/${r}`)
   ];
   writeAddonToConfig(targetDir, addonName, {
-    version: addonManifest.version,
+    version: addonMeta.version,
     files: fileList,
     checksums: result.checksums ?? {}
   });
@@ -4706,7 +4715,24 @@ async function addonSyncCommand(options) {
   const targetDir = resolve14(options.path || ".");
   p14.intro(introTitle("Addon Sync"));
   const config = readAddonConfig(targetDir);
-  const installedNames = Object.keys(config.installed);
+  const manifest = readManifest(targetDir);
+  const manifestAddons = manifest?.projectConfig?.enabledAddons ?? [];
+  for (const name of manifestAddons) {
+    if (!config.installed[name] && isFileBasedAddon(name)) {
+      const addonMeta = getAddonManifest(name);
+      const fileList = [
+        ...(addonMeta.files.skills ?? []).map((s) => `skills/${s}`),
+        ...(addonMeta.files.agents ?? []).map((a) => `agents/${a}.md`),
+        ...(addonMeta.files.rules ?? []).map((r) => `rules/${r}`)
+      ];
+      writeAddonToConfig(targetDir, name, {
+        version: addonMeta.version,
+        files: fileList
+      });
+    }
+  }
+  const freshConfig = readAddonConfig(targetDir);
+  const installedNames = Object.keys(freshConfig.installed);
   if (installedNames.length === 0) {
     p14.log.info("No addons installed. Nothing to sync.");
     p14.outro("");
@@ -4717,8 +4743,9 @@ async function addonSyncCommand(options) {
   let totalWritten = 0;
   let totalConflicts = [];
   for (const name of installedNames) {
+    if (!isFileBasedAddon(name)) continue;
     const addonSourceDir = getAddonSourceDir(name);
-    const result = syncAddonFiles(targetDir, addonSourceDir, config.agents);
+    const result = syncAddonFiles(targetDir, addonSourceDir, freshConfig.agents);
     totalWritten += result.written + (result.updated ?? 0);
     totalConflicts = totalConflicts.concat(result.conflicts);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devtronic",
-  "version": "1.2.4",
+  "version": "1.2.5",
   "description": "AI-assisted development toolkit — skills, agents, quality gates, and rules for Claude Code, Cursor, Copilot, and Antigravity",
   "type": "module",
   "bin": {