@jmylchreest/aide-plugin 0.0.58 → 0.0.59

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jmylchreest/aide-plugin",
-  "version": "0.0.58",
+  "version": "0.0.59",
   "description": "aide plugin for OpenCode and Codex CLI — multi-agent orchestration, memory, skills, and persistence",
   "type": "module",
   "main": "./src/opencode/index.ts",

package/src/core/mcp-sync.ts

@@ -806,6 +806,13 @@ function updateJournalAndResolve(
   const now = new Date().toISOString();
   const activePaths = new Set(sources.map((s) => s.path));
 
+  // Capture servers already in the journal before we process state
+  // transitions. The bootstrap phase should only backfill absence for
+  // these pre-existing servers — not for servers discovered for the
+  // first time in this run — to avoid treating "never existed in
+  // another platform's config" as "intentionally removed".
+  const preExistingServers = new Set(Object.keys(journal.servers));
+
   // Detect state transitions per server per source
   for (const source of sources) {
     const currentNames = new Set(Object.keys(source.servers));
@@ -841,12 +848,14 @@ function updateJournalAndResolve(
     }
   }
 
-  // Bootstrap: for servers found in some sources but absent from others,
-  // record "not present" entries for the missing sources. Without this,
-  // a server deleted from one file before the v2 journal existed would
-  // have no removal event to counterbalance presence in other files.
-  const allServerNames = new Set(Object.keys(journal.servers));
-  for (const serverName of allServerNames) {
+  // Bootstrap: for servers that were already tracked in the journal
+  // before this run, record "not present" entries for sources that
+  // have never been tracked for that server. Without this, a server
+  // deleted from one file before the v2 journal existed would have
+  // no removal event to counterbalance presence in other files.
+  // We only bootstrap pre-existing servers to avoid treating "never
+  // existed in another platform's config" as an intentional removal.
+  for (const serverName of preExistingServers) {
     const sourceMap = journal.servers[serverName];
     for (const source of sources) {
       if (!sourceMap[source.path]) {

package/src/lib/aide-downloader.ts

@@ -161,25 +161,44 @@ export function compareVersions(a: string, b: string): number {
 }
 
 /**
- * Get the download URL for the current platform
+ * Get the platform-specific binary name
  */
-export function getDownloadUrl(): string {
-  const platform = process.platform; // 'darwin', 'linux', 'win32'
-  const arch = process.arch; // 'x64', 'arm64'
-
+export function getBinaryName(): string {
+  const platform = process.platform;
+  const arch = process.arch;
   const goos = platform === "win32" ? "windows" : platform;
   const goarch = arch === "x64" ? "amd64" : arch;
   const ext = platform === "win32" ? ".exe" : "";
+  return `aide-${goos}-${goarch}${ext}`;
+}
 
-  const binaryName = `aide-${goos}-${goarch}${ext}`;
-
+/**
+ * Get download URLs for the current platform, in priority order.
+ * Returns the versioned URL first (exact match), then the latest release
+ * URL as a fallback. The fallback handles the race condition where the
+ * marketplace pulls the new plugin version before the release action
+ * has finished publishing binary artifacts.
+ */
+export function getDownloadUrls(): string[] {
+  const binaryName = getBinaryName();
   const version = getPluginVersion();
+
   if (version) {
-    return `https://github.com/jmylchreest/aide/releases/download/v${version}/${binaryName}`;
+    return [
+      `https://github.com/jmylchreest/aide/releases/download/v${version}/${binaryName}`,
+      `https://github.com/jmylchreest/aide/releases/latest/download/${binaryName}`,
+    ];
   }
 
-  // Fallback to latest if version can't be determined
-  return `https://github.com/jmylchreest/aide/releases/latest/download/${binaryName}`;
+  return [`https://github.com/jmylchreest/aide/releases/latest/download/${binaryName}`];
+}
+
+/**
+ * Get the download URL for the current platform (first priority URL).
+ * @deprecated Use getDownloadUrls() for fallback support.
+ */
+export function getDownloadUrl(): string {
+  return getDownloadUrls()[0];
 }
 
 /**
@@ -250,11 +269,7 @@ export async function downloadAideBinary(
     }
   }
 
-  const url = getDownloadUrl();
-
-  if (!quiet) {
-    log(`[aide] Downloading from: ${url}`);
-  }
+  const urls = getDownloadUrls();
 
   try {
     // Create bin directory
@@ -262,13 +277,35 @@ export async function downloadAideBinary(
       mkdirSync(destDir, { recursive: true });
     }
 
-    // Download using native fetch (follows redirects by default)
-    const response = await fetch(url, {
-      headers: { "User-Agent": "aide-plugin" },
-    });
+    // Try each URL in priority order, falling back on 404/network errors.
+    // This handles the race where the marketplace pulls a new plugin version
+    // before the release action has finished publishing binaries.
+    let response: Response | null = null;
+    let usedUrl = urls[0];
 
-    if (!response.ok) {
-      throw new Error(`HTTP ${response.status}: ${response.statusText}`);
+    for (const url of urls) {
+      if (!quiet) {
+        log(`[aide] Downloading from: ${url}`);
+      }
+      const resp = await fetch(url, {
+        headers: { "User-Agent": "aide-plugin" },
+      });
+      if (resp.ok) {
+        response = resp;
+        usedUrl = url;
+        break;
+      }
+      if (resp.status === 404 && url !== urls[urls.length - 1]) {
+        if (!quiet) {
+          log(`[aide] Release not found (HTTP 404), trying fallback...`);
+        }
+        continue;
+      }
+      throw new Error(`HTTP ${resp.status}: ${resp.statusText}`);
+    }
+
+    if (!response || !response.ok) {
+      throw new Error("All download URLs failed");
     }
 
     if (!response.body) {