@pareto4/pp-core 1.5.1 → 1.5.2

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pp-core",
   "displayName": "PP Core",
-  "version": "1.5.1",
+  "version": "1.5.2",
   "description": "PP Core is a meta-prompting, context engineering, and spec-driven development system for AI coding agents.",
   "author": {
     "name": "pareto4",

package/bin/install.js

@@ -7714,6 +7714,15 @@ function cleanupOrphanedHooks(settings) {
     'gsd-intel-prune.js',  // Removed in v1.9.2
   ];
 
+  // pareto4/pp-core#1: prune the entire predecessor OpenGSD hook namespace.
+  // pp-core ships hooks/pp-*.{js,sh} and deletes the old hooks/gsd-*.{js,sh}
+  // files on install — but the matching settings.json entries were left behind,
+  // dangling (target file removed) and duplicating the freshly-added pp-* hooks.
+  // Any command pointing into hooks/gsd-*.{js,sh,cmd} is therefore stale: pp-core
+  // never registers gsd-* hooks. Match generically rather than enumerating each
+  // basename so future predecessor hooks are covered automatically.
+  const legacyGsdManagedHookRe = /hooks[\\/]gsd-[a-z0-9-]+\.(?:js|sh|cmd)(?=$|["'`\s])/i;
+
   let cleanedHooks = false;
 
   // Check all hook event types (Stop, SessionStart, etc.)
@@ -7724,9 +7733,13 @@ function cleanupOrphanedHooks(settings) {
         // Filter out entries that contain orphaned hooks
         const filtered = hookEntries.filter(entry => {
           if (entry.hooks && Array.isArray(entry.hooks)) {
-            // Check if any hook in this entry matches orphaned patterns
+            // Check if any hook in this entry matches orphaned patterns or the
+            // predecessor gsd-* managed-hook namespace.
             const hasOrphaned = entry.hooks.some(h =>
-              h.command && orphanedHookPatterns.some(pattern => h.command.includes(pattern))
+              h.command && (
+                orphanedHookPatterns.some(pattern => h.command.includes(pattern)) ||
+                legacyGsdManagedHookRe.test(h.command)
+              )
             );
             if (hasOrphaned) {
               cleanedHooks = true;
@@ -7736,6 +7749,11 @@ function cleanupOrphanedHooks(settings) {
           return true;  // Keep this entry
         });
         settings.hooks[eventType] = filtered;
+        // Drop the event key entirely once all its matcher groups are gone, so
+        // no empty hook arrays linger in settings.json.
+        if (settings.hooks[eventType].length === 0) {
+          delete settings.hooks[eventType];
+        }
       }
     }
   }
@@ -7744,16 +7762,21 @@ function cleanupOrphanedHooks(settings) {
     console.log(`  ${green}✓${reset} Removed orphaned hook registrations`);
   }
 
-  // Fix #330: Update statusLine if it points to old GSD statusline.js path
-  // Only match the specific old GSD path pattern (hooks/statusline.js),
-  // not third-party statusline scripts that happen to contain 'statusline.js'
+  // Fix #330 + pareto4/pp-core#1: Update statusLine if it points to a stale
+  // managed statusline path — both the pre-rename `hooks/statusline.js` and the
+  // OpenGSD-era `hooks/gsd-statusline.js` (deleted on the pp-core upgrade, which
+  // otherwise leaves the statusline pointing at a missing file). Match only
+  // these specific managed basenames, not third-party scripts that merely
+  // contain 'statusline.js'.
   if (settings.statusLine && settings.statusLine.command &&
-    /hooks[\/\\]statusline\.js/.test(settings.statusLine.command)) {
+    /hooks[\/\\](?:gsd-)?statusline\.js/.test(settings.statusLine.command)) {
+    const wasGsd = /hooks[\/\\]gsd-statusline\.js/.test(settings.statusLine.command);
     settings.statusLine.command = settings.statusLine.command.replace(
-      /hooks([\/\\])statusline\.js/,
+      /hooks([\/\\])(?:gsd-)?statusline\.js/,
       'hooks$1pp-statusline.js'
     );
-    console.log(`  ${green}✓${reset} Updated statusline path (hooks/statusline.js → hooks/pp-statusline.js)`);
+    const from = wasGsd ? 'hooks/gsd-statusline.js' : 'hooks/statusline.js';
+    console.log(`  ${green}✓${reset} Updated statusline path (${from} → hooks/pp-statusline.js)`);
   }
 
   return settings;
@@ -12167,6 +12190,7 @@ module.exports = {
     stripGsdFromCodexConfig,
     migrateCodexHooksMapFormat,
     stripStaleGsdHookBlocks,
+    cleanupOrphanedHooks,
     hasUserNamespacedAotHooks,
     parseTomlToObject,
     validateCodexConfigSchema,

package/gemini-extension.json

@@ -1,6 +1,6 @@
 {
   "name": "pp-core",
-  "version": "1.5.1",
+  "version": "1.5.2",
   "description": "PP Core — a meta-prompting, context engineering, and spec-driven development system for AI coding agents. Loads pp-core's operating context into every Gemini CLI session.",
   "contextFileName": "GEMINI.md"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pareto4/pp-core",
-  "version": "1.5.1",
+  "version": "1.5.2",
   "description": "Pareto pp-core — The Pareto Project's AI-assisted engineering framework: meta-prompting, context engineering, and spec-driven development for AI coding agents.",
   "bin": {
     "pp-core": "bin/install.js",