@pareto4/pp-core 1.8.2 → 1.9.0

package/pp-core/bin/lib/capability-state.cjs

@@ -249,20 +249,26 @@ function _loadInstalledSkillsManifest(configDir) {
     for (const entry of entries) {
         if (!entry.isDirectory())
             continue;
-        if (!entry.name.startsWith('gsd-'))
+        // Dual-prefix match: current 'pp-' naming plus legacy 'gsd-' directories
+        // (backward compatibility retained per D-11 keep-token policy). Mirrors
+        // the established idiom in src/init.cts's buildSkillManifest (commit
+        // 242aab99).
+        const isPp = entry.name.startsWith('pp-');
+        const isGsd = entry.name.startsWith('gsd-');
+        if (!isPp && !isGsd)
             continue;
-        // Strip the 'gsd-' prefix to get the skill stem
-        const stem = entry.name.slice(4); // 'gsd-'.length === 4
+        // Strip the prefix to get the skill stem ('pp-'.length === 3, 'gsd-'.length === 4)
+        const stem = entry.name.slice(isPp ? 3 : 4);
         if (!stem)
             continue;
         const skillMdPath = node_path_1.default.join(skillsDir, entry.name, 'SKILL.md');
         // Parity with loadSkillsManifest: a stem exists only when its artifact
         // file is present. loadSkillsManifest registers a stem per .md FILE (and
         // tolerates an unreadable file as []), but never invents a stem for which
-        // no file exists. Mirror that here: a stale gsd-<stem>/ directory with no
-        // SKILL.md must NOT register the stem — otherwise the capability would be
-        // wrongly reported surfaced/enabled and a verify:post hook would render
-        // for a skill that cannot run.
+        // no file exists. Mirror that here: a stale pp-<stem>/ or gsd-<stem>/
+        // directory with no SKILL.md must NOT register the stem — otherwise the
+        // capability would be wrongly reported surfaced/enabled and a
+        // verify:post hook would render for a skill that cannot run.
         if (!node_fs_1.default.existsSync(skillMdPath))
             continue;
         let content = '';

package/pp-core/bin/lib/config.cjs

@@ -143,21 +143,6 @@ function validateShipPrBodySections(value) {
 function buildNewProjectConfig(userChoices) {
     const choices = userChoices || {};
     const homedir = node_os_1.default.homedir();
-    // Detect API key availability
-    const braveKeyFile = node_path_1.default.join(homedir, '.gsd', 'brave_api_key');
-    const hasBraveSearch = !!(process.env['BRAVE_API_KEY'] || node_fs_1.default.existsSync(braveKeyFile));
-    const firecrawlKeyFile = node_path_1.default.join(homedir, '.gsd', 'firecrawl_api_key');
-    const hasFirecrawl = !!(process.env['FIRECRAWL_API_KEY'] || node_fs_1.default.existsSync(firecrawlKeyFile));
-    const exaKeyFile = node_path_1.default.join(homedir, '.gsd', 'exa_api_key');
-    const hasExaSearch = !!(process.env['EXA_API_KEY'] || node_fs_1.default.existsSync(exaKeyFile));
-    const tavilyKeyFile = node_path_1.default.join(homedir, '.gsd', 'tavily_api_key');
-    const hasTavilySearch = !!(process.env['TAVILY_API_KEY'] || node_fs_1.default.existsSync(tavilyKeyFile));
-    const refKeyFile = node_path_1.default.join(homedir, '.gsd', 'ref_api_key');
-    const hasRefSearch = !!(process.env['REF_API_KEY'] || node_fs_1.default.existsSync(refKeyFile));
-    const perplexityKeyFile = node_path_1.default.join(homedir, '.gsd', 'perplexity_api_key');
-    const hasPerplexity = !!(process.env['PERPLEXITY_API_KEY'] || node_fs_1.default.existsSync(perplexityKeyFile));
-    const jinaKeyFile = node_path_1.default.join(homedir, '.gsd', 'jina_api_key');
-    const hasJina = !!(process.env['JINA_API_KEY'] || node_fs_1.default.existsSync(jinaKeyFile));
     // Load user-level defaults from ~/.gsd/defaults.json if available
     const globalDefaultsPath = node_path_1.default.join(homedir, '.gsd', 'defaults.json');
     let userDefaults = {};
@@ -184,13 +169,13 @@ function buildNewProjectConfig(userChoices) {
         commit_docs: CONFIG_DEFAULTS.commit_docs,
         parallelization: CONFIG_DEFAULTS.parallelization,
         search_gitignored: CONFIG_DEFAULTS.search_gitignored,
-        brave_search: hasBraveSearch,
-        firecrawl: hasFirecrawl,
-        exa_search: hasExaSearch,
-        tavily_search: hasTavilySearch,
-        ref_search: hasRefSearch,
-        perplexity: hasPerplexity,
-        jina: hasJina,
+        brave_search: false,
+        firecrawl: false,
+        exa_search: false,
+        tavily_search: false,
+        ref_search: false,
+        perplexity: false,
+        jina: false,
         git: {
             branching_strategy: CONFIG_DEFAULTS.branching_strategy,
             create_tag: true,

package/pp-core/bin/lib/loop-resolver.cjs

@@ -461,6 +461,12 @@ function cmdLoopRenderHooks(cwd, point, raw, options = {}) {
     if (state.warnings && state.warnings.length > 0) {
         envelope.warnings = state.warnings;
     }
+    // No rawValue third argument, so --raw emits the JSON envelope rather than
+    // plain text. This is deliberate and load-bearing: 23 workflow call sites do
+    // `HOOKS_JSON=$(pp_run loop render-hooks <point> --raw)` and read `activeHooks`
+    // off the parsed envelope. Callers who want the human-readable text read the
+    // envelope's `rendered` field. Same convention as the deliberate
+    // `output(result, raw, undefined)` sites in src/check-command-router.cts.
     coreOutput(envelope, raw);
 }
 module.exports = {

package/pp-core/bin/lib/profile-output.cjs

@@ -441,8 +441,10 @@ function generateSkillsSection(cwd) {
         for (const entry of entries) {
             if (!entry.isDirectory())
                 continue;
-            // Skip GSD's own installed skills — only surface project-specific skills
-            if (entry.name.startsWith('gsd-'))
+            // Skip pp-core's own installed skills (current 'pp-' naming plus legacy
+            // 'gsd-' directories, backward compatibility retained) — only surface
+            // project-specific skills.
+            if (entry.name.startsWith('pp-') || entry.name.startsWith('gsd-'))
                 continue;
             const skillMdPath = node_path_1.default.join(absDir, entry.name, 'SKILL.md');
             if (!node_fs_1.default.existsSync(skillMdPath))

package/pp-core/bin/shared/config-schema.manifest.json

@@ -10,6 +10,10 @@
     "brave_search",
     "firecrawl",
     "exa_search",
+    "tavily_search",
+    "ref_search",
+    "perplexity",
+    "jina",
     "workflow.plan_check",
     "workflow.verifier",
     "workflow.auto_advance",