cadet-agent 0.15.1 → 0.15.4

package/package.json

@@ -1,35 +1,35 @@
-{
-  "name": "cadet-agent",
-  "version": "0.15.1",
-  "description": "Cross-IDE agent framework for Unity/C# game-development — one-command install",
-  "type": "module",
-  "bin": {
-    "cadet-agent": "bin/cli.mjs"
-  },
-  "scripts": {
-    "test": "node --test test/*.test.mjs"
-  },
-  "files": [
-    "bin/",
-    "src/"
-  ],
-  "keywords": [
-    "cadet",
-    "cadet-agent",
-    "unity",
-    "game-development",
-    "ai-agent",
-    "copilot",
-    "cursor",
-    "claude-code"
-  ],
-  "license": "CC-BY-4.0",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/naishtech/cadet-agent.git"
-  },
-  "homepage": "https://github.com/naishtech/cadet-agent#readme",
-  "engines": {
-    "node": ">=18.0.0"
-  }
-}
+{
+  "name": "cadet-agent",
+  "version": "0.15.4",
+  "description": "Cross-IDE agent framework for Unity/C# game-development — one-command install",
+  "type": "module",
+  "bin": {
+    "cadet-agent": "bin/cli.mjs"
+  },
+  "scripts": {
+    "test": "node --test test/*.test.mjs"
+  },
+  "files": [
+    "bin/",
+    "src/"
+  ],
+  "keywords": [
+    "cadet",
+    "cadet-agent",
+    "unity",
+    "game-development",
+    "ai-agent",
+    "copilot",
+    "cursor",
+    "claude-code"
+  ],
+  "license": "CC-BY-4.0",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/naishtech/cadet-agent.git"
+  },
+  "homepage": "https://github.com/naishtech/cadet-agent#readme",
+  "engines": {
+    "node": ">=18.0.0"
+  }
+}

package/src/install.mjs

@@ -2,6 +2,7 @@ import { createWriteStream, mkdirSync, readFileSync, unlinkSync, existsSync, rea
 import { join, dirname, relative } from 'node:path';
 import { inflateRawSync } from 'node:zlib';
 import { Readable } from 'node:stream';
+import { runUpgrades } from './upgrades.mjs';
 
 // ── Constants ────────────────────────────────────────────────────────────────
 
@@ -274,13 +275,13 @@ function walkDir(dir, fn) {
 
 function deleteObsoleteManagedFiles(targetDir, managedPaths, zipFilenames) {
   const normalizedZip = new Set(
-    zipFilenames.map(f => f.replace(/^\.?\/?/, '').replace(/\\/g, '/'))
+    zipFilenames.map(f => f.replace(/^\.\//, '').replace(/\\/g, '/'))
   );
   const deleted = [];
 
   for (const managed of managedPaths) {
-    const mn = managed.replace(/^\.?\/?/, '').replace(/\\/g, '/');
-    const managedPath = join(targetDir, managed.replace(/^\.?\/?/, ''));
+    const mn = managed.replace(/^\.\//, '').replace(/\\/g, '/');
+    const managedPath = join(targetDir, managed.replace(/^\.\//, ''));
     const stat = (() => { try { return statSync(managedPath); } catch { return null; } })();
     if (!stat) continue;
 
@@ -340,7 +341,61 @@ async function extractZipWithManifest(buf, targetDir, { preserved, managed }) {
   // Delete obsolete managed files no longer in the zip (renamed/removed managed paths)
   const deleted = deleteObsoleteManagedFiles(targetDir, managed, zipFilenames);
 
-  return { updated, preserved: preserved_list, added, deleted };
+  return { updated, preserved: preserved_list, added, deleted, zipFilenames };
+}
+
+// ── Removed-managed-path cleanup ─────────────────────────────────────────────
+
+export function findManagedPathsInZip(buf) {
+  try {
+    const eocdOff = findEocd(buf);
+    const cdSize  = read32(buf, eocdOff + 12);
+    const cdOff   = read32(buf, eocdOff + 16);
+
+    for (const entry of centralDirectoryEntries(buf, cdOff, cdSize)) {
+      const fn = entry.filename.replace(/^\.\//, '').replace(/\\/g, '/');
+      if (fn === '.cadet/agent/core/FrameworkManifest.json') {
+        // Extract just this one entry to read managedPaths
+        const lfhFilenameLen = read16(buf, entry.localHeaderOff + 26);
+        const lfhExtraLen    = read16(buf, entry.localHeaderOff + 28);
+        const dataStart = entry.localHeaderOff + 30 + lfhFilenameLen + lfhExtraLen;
+        const compressed = buf.subarray(dataStart, dataStart + entry.compressedSize);
+        let data;
+        if (entry.method === 0) data = compressed;
+        else if (entry.method === 8) data = inflateRawSync(compressed);
+        else break;
+        const manifest = JSON.parse(data.toString('utf-8'));
+        return manifest.managedPaths || [];
+      }
+    }
+  } catch {
+    // Not a valid zip or manifest not found
+  }
+  return [];
+}
+
+export function deleteRemovedManagedPaths(targetDir, oldManaged, newManaged) {
+  const newSet = new Set(newManaged.map(p => p.replace(/^\.\//, '').replace(/\\/g, '/')));
+  const deleted = [];
+
+  for (const old of oldManaged) {
+    const oldNorm = old.replace(/^\.\//, '').replace(/\\/g, '/');
+    if (newSet.has(oldNorm)) continue;
+
+    // This path was in the old manifest but is absent from the new one — remove it
+    const fullPath = join(targetDir, old.replace(/^\.\//, ''));
+    const st = (() => { try { return statSync(fullPath); } catch { return null; } })();
+    if (!st) continue;
+
+    if (st.isFile()) {
+      try { unlinkSync(fullPath); deleted.push(fullPath); } catch {}
+    } else if (st.isDirectory()) {
+      walkDir(fullPath, (filePath) => {
+        try { unlinkSync(filePath); deleted.push(filePath); } catch {}
+      });
+    }
+  }
+  return deleted;
 }
 
 // ── Public sync entry ───────────────────────────────────────────────────────
@@ -384,6 +439,23 @@ export async function sync(targetDir, opts = {}) {
     managed: existingManifest.managedPaths || [],
   });
 
+  // 4b. Find new managed paths from the zip and delete any old paths that were removed
+  const newManagedPaths = findManagedPathsInZip(zipBuf);
+  const removedDeleted = deleteRemovedManagedPaths(
+    targetDir,
+    existingManifest.managedPaths || [],
+    newManagedPaths
+  );
+  if (removedDeleted.length > 0) {
+    result.deleted.push(...removedDeleted);
+  }
+
+  // 4c. Run version-based upgrades (migrations for renamed/removed files)
+  const upgradeDeleted = runUpgrades(targetDir, oldVersionNorm, newVersion);
+  if (upgradeDeleted.length > 0) {
+    result.deleted.push(...upgradeDeleted);
+  }
+
   // 5. Report
   console.log('');
   console.log(`✅ Cadet-Agent synced: v${oldVersionNorm} → v${newVersion}`);

package/src/upgrades.mjs

@@ -0,0 +1,77 @@
+import { unlinkSync, statSync, readdirSync } from 'node:fs';
+import { join } from 'node:path';
+
+// ── Path helpers ────────────────────────────────────────────────────────────
+
+function walkDir(dir, fn) {
+  let entries;
+  try { entries = readdirSync(dir, { withFileTypes: true }); } catch { return; }
+  for (const entry of entries) {
+    const fullPath = join(dir, entry.name);
+    if (entry.isDirectory()) {
+      walkDir(fullPath, fn);
+    } else {
+      fn(fullPath);
+    }
+  }
+}
+
+function deletePath(targetDir, relativePath) {
+  const norm = relativePath.replace(/^\.\//, '').replace(/\\/g, '/');
+  const fullPath = join(targetDir, norm);
+  let st;
+  try { st = statSync(fullPath); } catch { return []; }
+
+  const deleted = [];
+  if (st.isFile()) {
+    try { unlinkSync(fullPath); deleted.push(fullPath); } catch {}
+  } else if (st.isDirectory()) {
+    walkDir(fullPath, (filePath) => {
+      try { unlinkSync(filePath); deleted.push(filePath); } catch {}
+    });
+  }
+  return deleted;
+}
+
+// ── Upgrade registry ────────────────────────────────────────────────────────
+
+// Key = FROM version. Each upgrade runs when upgrading FROM a version
+// older than or equal to the key, TO a version newer than the key.
+// Upgrade functions receive (targetDir) and return string[] of deleted paths.
+
+const upgrades = {
+  '0.15.3': (targetDir) => [
+    ...deletePath(targetDir, '.cadet/orchestrator'),
+    ...deletePath(targetDir, '.github/prompts'),
+    ...deletePath(targetDir, 'AGENTS.md'),
+  ],
+};
+
+// ── Runner ──────────────────────────────────────────────────────────────────
+
+function semverGt(a, b) {
+  const pa = a.split('.').map(Number);
+  const pb = b.split('.').map(Number);
+  for (let i = 0; i < 3; i++) {
+    if ((pa[i] || 0) > (pb[i] || 0)) return true;
+    if ((pa[i] || 0) < (pb[i] || 0)) return false;
+  }
+  return false;
+}
+
+export function runUpgrades(targetDir, fromVersion, toVersion) {
+  const deleted = [];
+
+  for (const [ver, fn] of Object.entries(upgrades)) {
+    if (semverGt(ver, fromVersion) && !semverGt(ver, toVersion)) {
+      try {
+        const result = fn(targetDir);
+        if (Array.isArray(result)) deleted.push(...result);
+      } catch {
+        // Upgrade failure shouldn't block the sync
+      }
+    }
+  }
+
+  return deleted;
+}