@pareto4/pp-core 1.8.1 → 1.9.0

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

@@ -1825,9 +1825,9 @@ function buildSkillManifest(cwd, skillsDir = null) {
             // Nested layout: <entry>/skills/<stem>/SKILL.md
             // Used by cline, qwen, hermes, augment, trae, antigravity (#69 nested=true).
             // Descend exactly one level into <entry>/skills/ — no deeper recursion.
-            // Scope to gsd-ns-* routers only: never vacuum up an unrelated user skill
-            // that happens to have its own `skills/` subdirectory.
-            if (!entry.name.startsWith('gsd-ns-'))
+            // Scope to pp-ns-* routers (plus legacy gsd-ns-*) only: never vacuum up an
+            // unrelated user skill that happens to have its own `skills/` subdirectory.
+            if (!entry.name.startsWith('pp-ns-') && !entry.name.startsWith('gsd-ns-'))
                 continue;
             const nestedSkillsDir = node_path_1.default.join(rootPath, entry.name, 'skills');
             let nestedEntries = [];
@@ -1859,7 +1859,7 @@ function buildSkillManifest(cwd, skillsDir = null) {
         const rootCmp = a.root.localeCompare(b.root);
         return rootCmp !== 0 ? rootCmp : a.name.localeCompare(b.name);
     });
-    const gsdSkillsInstalled = skills.some((skill) => skill.name.startsWith('gsd-'));
+    const gsdSkillsInstalled = skills.some((skill) => skill.name.startsWith('pp-') || skill.name.startsWith('gsd-'));
     return {
         skills,
         roots,

package/pp-core/bin/lib/installer-migration-report.cjs

@@ -151,7 +151,7 @@ function classifyPromptUserAction(action) {
     if (/^pp-core\/sdk\/(dist|src)\//.test(relPath)) {
         return { category: 'stale-sdk-build-artifact', choice: 'remove' };
     }
-    if (/^skills\/gsd-[^/]+\/SKILL\.md$/.test(relPath)) {
+    if (/^skills\/(?:pp|gsd)-[^/]+\/SKILL\.md$/.test(relPath)) {
         return { category: 'user-facing-skill', choice: 'keep' };
     }
     // #3610 / #3628: bundled GSD hooks shipped under `hooks/`. The whitelist

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/lib/runtime-name-policy.cjs

@@ -43,7 +43,6 @@ function normalizeRuntimeToken(value) {
 function loadAliasManifest() {
     const manifestCandidates = [
         node_path_1.default.resolve(__dirname, '..', 'shared', 'runtime-aliases.manifest.json'),
-        node_path_1.default.resolve(__dirname, '../../../sdk/shared/runtime-aliases.manifest.json'),
     ];
     for (const manifestPath of manifestCandidates) {
         try {

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

@@ -439,11 +439,11 @@ function _syncGsdDir(stagedDir, destDir, kind, manifest) {
         }
         // Prune stale GSD-owned files not in the staged set, preserving user-owned files
         // (mirrors install's prefix-scoped _removeGsdEntries):
-        //   - agents: only gsd-* are GSD-owned
+        //   - agents: only `${kindPrefix}`-prefixed (plus legacy gsd-*) are GSD-owned
         //   - flat command dirs: only `${kindPrefix}`-prefixed are GSD-owned
         //   - namespaced command dirs: the whole dir is GSD-owned
         for (const file of node_fs_1.default.readdirSync(destDir).filter(f => f.endsWith('.md'))) {
-            if (kindName === 'agents' && !file.startsWith('gsd-'))
+            if (kindName === 'agents' && !(kindPrefix && file.startsWith(kindPrefix)) && !file.startsWith('gsd-'))
                 continue;
             if (kindName === 'commands' && !namespacedByDir && kindPrefix && !file.startsWith(kindPrefix))
                 continue;

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",

package/pp-core/references/workstream-flag.md

@@ -68,6 +68,19 @@ All workflow routing commands include `${GSD_WS}` which:
 This ensures workstream scope chains automatically through the workflow:
 `new-milestone → discuss-phase → plan-phase → execute-phase → transition`
 
+### Substitution rule (user-facing output and Skill/Task args)
+
+Outside executed bash blocks, `${GSD_WS}` / `{GSD_WS}` is a placeholder, not a
+shell variable — nothing expands it automatically. When a workflow emits a
+routing or "Next:" command, or passes args to `Skill()`/`Task()`, Claude must
+substitute the placeholder itself:
+
+- Workstream active → replace with `--ws <name>`
+- Flat mode → remove the placeholder entirely (and any doubled whitespace)
+
+Never print the literal token `${GSD_WS}` in a user-facing command — a pasted
+command containing it breaks in the shell.
+
 ## Directory Structure
 
 ```