plugin-updater 1.0.33 → 1.0.35

package/dist/cli.js

@@ -16,19 +16,31 @@ function parseArgs(argv) {
     }
     return parsed;
 }
+function binaryExists(name) {
+    try {
+        const probe = process.platform === "win32" ? `where ${name}` : `command -v ${name}`;
+        require("child_process").execSync(probe, { stdio: "ignore" });
+        return true;
+    }
+    catch {
+        return false;
+    }
+}
 function detectApp(explicit) {
     if (explicit === "claude" || explicit === "opencode")
         return explicit;
     if (explicit)
         throw new Error(`Unknown app "${explicit}" - use claude or opencode`);
-    const hasClaude = fs.existsSync(path.join(os.homedir(), ".claude"));
-    const hasOpencode = fs.existsSync(path.join(os.homedir(), ".opencode"))
+    const hasClaudeDir = fs.existsSync(path.join(os.homedir(), ".claude"));
+    const hasOpencodeDir = fs.existsSync(path.join(os.homedir(), ".opencode"))
         || fs.existsSync(path.join(os.homedir(), ".config", "opencode"));
-    if (hasClaude && !hasOpencode)
-        return "claude";
-    if (hasOpencode && !hasClaude)
-        return "opencode";
-    throw new Error("Cannot detect the app automatically - pass --app claude or --app opencode");
+    if (hasClaudeDir !== hasOpencodeDir)
+        return hasClaudeDir ? "claude" : "opencode";
+    const hasClaudeBin = binaryExists("claude");
+    const hasOpencodeBin = binaryExists("opencode");
+    if (hasClaudeBin !== hasOpencodeBin)
+        return hasClaudeBin ? "claude" : "opencode";
+    throw new Error("Both apps (or neither) found - pass --app claude or --app opencode");
 }
 function getConfigDir(app) {
     const home = os.homedir();
@@ -59,8 +71,9 @@ function registerClaudeHook(configDir) {
     const settings = (fs.existsSync(settingsPath) ? readJson(settingsPath) : {}) ?? {};
     const hooks = (settings.hooks ?? {});
     const sessionStart = (hooks.SessionStart ?? []);
-    if (!JSON.stringify(sessionStart).includes("plugin-updater run")) {
-        sessionStart.push({ hooks: [{ type: "command", command: "npx -y plugin-updater run --app claude" }] });
+    if (!JSON.stringify(sessionStart).includes("plugin-updater")) {
+        // @latest so npx re-resolves the tag instead of pinning its first cached copy
+        sessionStart.push({ hooks: [{ type: "command", command: "npx -y plugin-updater@latest run --app claude" }] });
     }
     hooks.SessionStart = sessionStart;
     settings.hooks = hooks;
@@ -99,6 +112,22 @@ function addPluginEntry(configDir, url, branch) {
     }
     return { name, url: cleanUrl, branch };
 }
+function removePluginEntry(configDir, name) {
+    const file = pluginsJsonPath(configDir);
+    const entries = readJson(file) ?? [];
+    fs.writeFileSync(file, JSON.stringify(entries.filter((e) => e.name !== name), null, 2), "utf8");
+}
+async function setupEntry(updater, configDir, url, branch) {
+    const entry = addPluginEntry(configDir, url, branch);
+    console.log(`Setting up ${entry.name}...`);
+    try {
+        await updater.updatePluginPublic(entry.name, entry.url, entry.branch);
+    }
+    catch (e) {
+        removePluginEntry(configDir, entry.name);
+        throw e;
+    }
+}
 async function main() {
     const parsed = parseArgs(process.argv.slice(2));
     if (!["init", "add", "run"].includes(parsed.command)) {
@@ -119,9 +148,7 @@ async function main() {
         else
             registerOpencodePlugin(configDir);
         for (const url of parsed.urls) {
-            const entry = addPluginEntry(configDir, url, parsed.branch);
-            console.log(`Setting up ${entry.name}...`);
-            await updater.updatePluginPublic(entry.name, entry.url, entry.branch);
+            await setupEntry(updater, configDir, url, parsed.branch);
         }
         console.log("Init complete.");
     }
@@ -129,9 +156,7 @@ async function main() {
         if (parsed.urls.length === 0)
             throw new Error("add requires at least one git url");
         for (const url of parsed.urls) {
-            const entry = addPluginEntry(configDir, url, parsed.branch);
-            console.log(`Setting up ${entry.name}...`);
-            await updater.updatePluginPublic(entry.name, entry.url, entry.branch);
+            await setupEntry(updater, configDir, url, parsed.branch);
         }
     }
     else {

package/dist/index.js

@@ -243,7 +243,9 @@ function updatePlugin(pluginName, gitUrl, branch, commitHash, updateInterval = 1
         if (!fs.existsSync(reposDir))
             fs.mkdirSync(reposDir, { recursive: true });
         const branchFlag = branch ? `--branch ${branch}` : "";
-        executeGit(`git clone --recurse-submodules ${branchFlag} ${gitUrl} ${pluginName}`, reposDir);
+        const cloned = executeGit(`git clone --recurse-submodules ${branchFlag} ${gitUrl} ${pluginName}`, reposDir);
+        if (!cloned)
+            return { success: false, changed: false };
         fs.writeFileSync(lastCheckFile, Date.now().toString());
         didChange = true;
     }
@@ -481,6 +483,8 @@ export async function updatePluginPublic(pluginName, gitUrl, branch, commitHash)
     const configDir = getAppConfigDir(getAppName());
     // interval 0: an explicit update request must never fast-path-skip
     const result = updatePlugin(pluginName, gitUrl, branch, commitHash ?? null, 0);
+    if (!result.success)
+        throw new Error(`could not set up ${pluginName} - see the updater log`);
     await deployToExecutionDir(pluginName, path.join(configDir, "plugin"), result.changed, configDir);
 }
 export async function earlyLaunch(configDir, plugins) {
@@ -526,6 +530,10 @@ export async function earlyLaunch(configDir, plugins) {
         writeLog(`Processing earlyLaunch for ${plugin.name}`);
         try {
             const updateResult = updatePlugin(plugin.name, plugin.url, plugin.branch, null, plugin.updateInterval ?? 1);
+            if (!updateResult.success) {
+                writeLog(`Skipping deploy for ${plugin.name}: update failed`, true);
+                continue;
+            }
             await deployToExecutionDir(plugin.name, path.join(configDir, "plugin"), updateResult.changed, configDir);
         }
         catch (e) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "plugin-updater",
-  "version": "1.0.33",
+  "version": "1.0.35",
   "description": "Plugin lifecycle manager for OpenCode and Claude Code launchers",
   "type": "module",
   "main": "dist/index.js",