@ztffn/presentation-generator-plugin 1.4.2 → 1.4.3

package/bin/index.js

@@ -244,39 +244,56 @@ function cleanOldCacheVersions(keepVersion) {
   }
 }
 
-function purgeInstalledPluginsJson() {
-  // Directly remove all entries for this plugin from Claude's global registry
-  // so that a subsequent `claude plugin install` always writes a fresh entry.
+function writeInstalledPluginsJson(version, installDir, scope) {
+  // Claude Code's `claude plugin install` writes stale version/installPath data
+  // (a known bug: https://github.com/anthropics/claude-code/issues/15642).
+  // We overwrite the entry directly with the correct values after install.
   const registryPath = path.join(
     os.homedir(), ".claude", "plugins", "installed_plugins.json"
   );
-  if (!fs.existsSync(registryPath)) return;
-  try {
-    const registry = JSON.parse(fs.readFileSync(registryPath, "utf8"));
-    const key = `${PLUGIN_NAME}@${MARKETPLACE_NAME}`;
-    if (registry.plugins && key in registry.plugins) {
-      delete registry.plugins[key];
-      fs.writeFileSync(registryPath, JSON.stringify(registry, null, 2) + "\n");
-    }
-  } catch {
-    // registry unreadable/corrupt — leave it alone
+
+  // Ensure the cache dir exists and contains the plugin files.
+  const cacheDir = path.join(
+    os.homedir(), ".claude", "plugins", "cache", MARKETPLACE_NAME, PLUGIN_NAME, version
+  );
+  if (!fs.existsSync(cacheDir)) {
+    execSync(`cp -r "${installDir}/." "${cacheDir}"`, { stdio: "pipe" });
+  }
+
+  let registry = { version: 2, plugins: {} };
+  if (fs.existsSync(registryPath)) {
+    try { registry = JSON.parse(fs.readFileSync(registryPath, "utf8")); } catch {}
   }
+
+  const key = `${PLUGIN_NAME}@${MARKETPLACE_NAME}`;
+  const now = new Date().toISOString();
+  const existing = (registry.plugins[key] || []).find((e) => e.scope === scope);
+  const entry = {
+    scope,
+    installPath: cacheDir,
+    version,
+    installedAt: existing?.installedAt || now,
+    lastUpdated: now,
+    projectPath: process.cwd(),
+  };
+
+  // Replace all entries for this scope, keep entries for other scopes.
+  const others = (registry.plugins[key] || []).filter((e) => e.scope !== scope);
+  registry.plugins[key] = [...others, entry];
+  fs.mkdirSync(path.dirname(registryPath), { recursive: true });
+  fs.writeFileSync(registryPath, JSON.stringify(registry, null, 2) + "\n");
 }
 
-function registerWithClaude(installDir, scope) {
+function registerWithClaude(installDir, scope, version) {
   const pluginsDir = path.dirname(installDir);
   ensureMarketplaceJson(pluginsDir);
 
-  // Remove any stale registration (e.g. from a previous install at a different path)
-  // before adding the current one. Errors are ignored — it may not exist yet.
+  // Remove any stale marketplace registration before re-adding.
   try {
     execSync(`claude plugin marketplace remove "${MARKETPLACE_NAME}"`, { stdio: "pipe" });
-  } catch {
-    // didn't exist or not removable — ignore
-  }
+  } catch {}
 
-  // Uninstall any stale plugin entries from both scopes before re-installing.
-  // This prevents Claude from reusing a cached entry pointing at an old version path.
+  // Uninstall stale plugin entries from both scopes before re-installing.
   try {
     execSync(`claude plugin uninstall ${PLUGIN_NAME}@${MARKETPLACE_NAME} --scope project`, { stdio: "pipe" });
   } catch {}
@@ -284,14 +301,13 @@ function registerWithClaude(installDir, scope) {
     execSync(`claude plugin uninstall ${PLUGIN_NAME}@${MARKETPLACE_NAME} --scope user`, { stdio: "pipe" });
   } catch {}
 
-  // Belt-and-suspenders: also purge the registry JSON directly in case
-  // `claude plugin uninstall` doesn't clean installed_plugins.json reliably.
-  purgeInstalledPluginsJson();
-
   execSync(`claude plugin marketplace add "${pluginsDir}"`, { stdio: "pipe" });
 
   const scopeFlag = scope === "project" ? "--scope project" : "--scope user";
   execSync(`claude plugin install ${PLUGIN_NAME}@${MARKETPLACE_NAME} ${scopeFlag}`, { stdio: "pipe" });
+
+  // Overwrite whatever Claude wrote with the correct version and cache path.
+  writeInstalledPluginsJson(version, installDir, scope);
 }
 
 function removeFromSettings(settingsPath) {
@@ -352,7 +368,7 @@ async function install() {
   console.log(`\nInstalled v${meta.version} to: ${label}`);
 
   console.log("\nRegistering plugin with Claude Code...");
-  registerWithClaude(installDir, scope);
+  registerWithClaude(installDir, scope, meta.version);
   cleanOldCacheVersions(meta.version);
 
   const settingsLabel = scope === "project" ? ".claude/settings.json" : "~/.claude/settings.json";
@@ -390,7 +406,7 @@ async function update() {
     console.log(`\n${label}: updating v${before || "unknown"} → v${meta.version}...`);
     await downloadAndExtract(meta.version, meta.tarball, installDir);
     const updateScope = installDir === GLOBAL_INSTALL_DIR ? "user" : "project";
-    registerWithClaude(installDir, updateScope);
+    registerWithClaude(installDir, updateScope, meta.version);
     cleanOldCacheVersions(meta.version);
     console.log(`${label}: updated to v${meta.version} ✓`);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ztffn/presentation-generator-plugin",
-  "version": "1.4.2",
+  "version": "1.4.3",
   "description": "Claude Code plugin for generating graph-based presentations",
   "bin": {
     "presentation-generator-plugin": "bin/index.js"