@promptcellar/pc 0.5.0 → 0.5.2

package/bin/pc.js

@@ -1,6 +1,7 @@
 #!/usr/bin/env node
 
 import { program } from 'commander';
+import { createRequire } from 'module';
 import { login } from '../src/commands/login.js';
 import { logout } from '../src/commands/logout.js';
 import { status } from '../src/commands/status.js';
@@ -10,10 +11,13 @@ import { push } from '../src/commands/push.js';
 import { config } from '../src/commands/config.js';
 import { update } from '../src/commands/update.js';
 
+const require = createRequire(import.meta.url);
+const pkg = require('../package.json');
+
 program
   .name('pc')
   .description('PromptCellar CLI - sync prompts between your terminal and the cloud')
-  .version('0.4.0');
+  .version(pkg.version);
 
 program
   .command('login')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@promptcellar/pc",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "description": "CLI for PromptCellar - sync prompts between your terminal and the cloud",
   "type": "module",
   "main": "src/index.js",

package/src/commands/setup.js

@@ -64,6 +64,23 @@ function isClaudeHookInstalled(settings) {
   );
 }
 
+// Check for legacy Stop hook (pre-0.5.0)
+function hasLegacyClaudeHook(settings) {
+  const stopMatchers = settings.hooks?.Stop || [];
+  return stopMatchers.some(matcher =>
+    matcher.hooks?.some(hook => hook.command?.includes(HOOK_SCRIPT_NAME))
+  );
+}
+
+// Remove legacy Stop hook if present
+function removeLegacyClaudeHook(settings) {
+  if (settings.hooks?.Stop) {
+    settings.hooks.Stop = settings.hooks.Stop.filter(matcher =>
+      !matcher.hooks?.some(hook => hook.command?.includes(HOOK_SCRIPT_NAME))
+    );
+  }
+}
+
 // Codex config (TOML)
 function getCodexConfig() {
   if (existsSync(CODEX_CONFIG_PATH)) {
@@ -168,8 +185,10 @@ async function setupClaudeCode() {
   console.log(chalk.cyan('\nConfiguring Claude Code...'));
 
   const settings = getClaudeSettings();
+  const hasLegacy = hasLegacyClaudeHook(settings);
+  const hasCurrent = isClaudeHookInstalled(settings);
 
-  if (isClaudeHookInstalled(settings)) {
+  if (hasCurrent) {
     console.log(chalk.yellow('  Hook already installed.'));
 
     const { reinstall } = await inquirer.prompt([{
@@ -180,6 +199,12 @@ async function setupClaudeCode() {
     }]);
 
     if (!reinstall) {
+      // Still remove legacy hook if present
+      if (hasLegacy) {
+        removeLegacyClaudeHook(settings);
+        saveClaudeSettings(settings);
+        console.log(chalk.green('  Removed legacy Stop hook.'));
+      }
       return;
     }
 
@@ -187,8 +212,13 @@ async function setupClaudeCode() {
     settings.hooks.UserPromptSubmit = (settings.hooks.UserPromptSubmit || []).filter(matcher =>
       !matcher.hooks?.some(hook => hook.command?.includes(HOOK_SCRIPT_NAME))
     );
+  } else if (hasLegacy) {
+    console.log(chalk.yellow('  Migrating from legacy Stop hook to UserPromptSubmit...'));
   }
 
+  // Always remove legacy Stop hook if present
+  removeLegacyClaudeHook(settings);
+
   // Ensure hooks.UserPromptSubmit exists
   if (!settings.hooks) {
     settings.hooks = {};
@@ -313,12 +343,20 @@ export async function unsetup() {
 
   let removed = false;
 
-  // Remove Claude hook
+  // Remove Claude hook (both current and legacy)
   const claudeSettings = getClaudeSettings();
-  if (isClaudeHookInstalled(claudeSettings)) {
-    claudeSettings.hooks.UserPromptSubmit = (claudeSettings.hooks.UserPromptSubmit || []).filter(matcher =>
-      !matcher.hooks?.some(hook => hook.command?.includes(HOOK_SCRIPT_NAME))
-    );
+  const hasCurrent = isClaudeHookInstalled(claudeSettings);
+  const hasLegacy = hasLegacyClaudeHook(claudeSettings);
+
+  if (hasCurrent || hasLegacy) {
+    if (hasCurrent) {
+      claudeSettings.hooks.UserPromptSubmit = (claudeSettings.hooks.UserPromptSubmit || []).filter(matcher =>
+        !matcher.hooks?.some(hook => hook.command?.includes(HOOK_SCRIPT_NAME))
+      );
+    }
+    if (hasLegacy) {
+      removeLegacyClaudeHook(claudeSettings);
+    }
     saveClaudeSettings(claudeSettings);
     console.log(chalk.green('  Removed Claude Code hook.'));
     removed = true;