@pareto4/pp-core 1.5.0 → 1.5.2

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pp-core",
   "displayName": "PP Core",
-  "version": "1.5.0",
+  "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.0",
+  "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,7 +1,7 @@
 {
   "name": "@pareto4/pp-core",
-  "version": "1.5.0",
-  "description": "GSD Core is a meta-prompting, context engineering, and spec-driven development system for AI coding agents.",
+  "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",
     "pp-tools": "pp-core/bin/pp-tools.cjs",
@@ -31,7 +31,7 @@
     "codex",
     "codex-cli"
   ],
-  "author": "OpenGSD",
+  "author": "The Pareto Project",
   "license": "MIT",
   "repository": {
     "type": "git",

package/scripts/verify-opengsd-sweep.cjs

@@ -41,6 +41,16 @@ const ROOT = path.resolve(__dirname, '..');
 /** Matches any recognised OpenGSD spelling in a line of text. */
 const OPENGSD_RE = /opengsd|open-gsd/i;
 
+/**
+ * Broader brand pattern for npm-facing manifest metadata (package.json
+ * `description` / `author`). Catches the upstream product name ("GSD Core")
+ * and the org spelling ("OpenGSD") that leaked into the published 1.5.0
+ * description/author. Applied ONLY to those manifest fields — not the code/docs
+ * surface — so legitimate `gsd-core` references in carved-out provenance files
+ * are unaffected.
+ */
+const MANIFEST_BRAND_RE = /opengsd|open-gsd|gsd[\s-]?core/i;
+
 // ── Carve-out allowlists (all paths are absolute) ───────────────────────────
 
 /**
@@ -224,6 +234,44 @@ function collectDocsFiles() {
   return files;
 }
 
+// ── Manifest metadata surface ────────────────────────────────────────────────
+
+const PACKAGE_JSON = path.join(ROOT, 'package.json');
+
+/**
+ * Scan package.json's npm-facing branding fields (`description`, `author`) for
+ * leftover upstream branding. npm renders these on the public package page, and
+ * a published version's metadata is immutable — so a leak here can only be fixed
+ * by a new release. Guarding it keeps the class from recurring (the 1.5.0
+ * description shipped "GSD Core ..." / author "OpenGSD" past the file-surface
+ * sweep, which never read package.json).
+ *
+ * @returns {Array<{file: string, line: number, text: string}>}
+ */
+function scanManifestBranding() {
+  const hits = [];
+  let pkg;
+  try {
+    pkg = JSON.parse(fs.readFileSync(PACKAGE_JSON, 'utf8'));
+  } catch {
+    return hits; // no/unreadable package.json — nothing to scan
+  }
+
+  const fields = [
+    ['description', pkg.description],
+    // `author` may be a string or an { name, ... } object
+    ['author', typeof pkg.author === 'object' && pkg.author ? pkg.author.name : pkg.author],
+  ];
+
+  for (const [field, value] of fields) {
+    if (typeof value === 'string' && MANIFEST_BRAND_RE.test(value)) {
+      hits.push({ file: 'package.json', line: `${field} field`, text: value });
+    }
+  }
+
+  return hits;
+}
+
 // ── Main sweep logic ─────────────────────────────────────────────────────────
 
 /**
@@ -247,7 +295,8 @@ function runSweep(opts) {
 
   const codeHits = scanFiles(codeFiles);
   const docsHits = scanFiles(docsFiles);
-  const hits = [...codeHits, ...docsHits];
+  const manifestHits = scanManifestBranding();
+  const hits = [...codeHits, ...docsHits, ...manifestHits];
 
   return { hits, clean: hits.length === 0 };
 }
@@ -259,7 +308,7 @@ function cliMain() {
 
   if (clean) {
     process.stdout.write(
-      '[verify-opengsd-sweep] PASS — live code+docs surface is OpenGSD-free (carve-outs intact).\n'
+      '[verify-opengsd-sweep] PASS — live code+docs surface and package.json branding are OpenGSD-free (carve-outs intact).\n'
     );
     return 0;
   }
@@ -281,7 +330,7 @@ function cliMain() {
 
 // ── Exports + guard ──────────────────────────────────────────────────────────
 
-module.exports = { runSweep, OPENGSD_RE, isCarvedOut };
+module.exports = { runSweep, OPENGSD_RE, MANIFEST_BRAND_RE, isCarvedOut, scanManifestBranding };
 
 if (require.main === module) {
   runMain(cliMain);