@prave/cli 1.4.15 → 1.5.1

package/README.md

@@ -63,7 +63,7 @@ Syncing 12 Skills — this takes about 15 seconds.
 Synced 12 · failed 0
 ```
 
-The `last_sync_at` watermark persists at `~/.prave/state.json`. Pass `--yes` to skip the prompt in CI scripts.
+The `last_sync_at` watermark persists at `~/.prave/state.json`.
 
 ## Commands
 

package/dist/commands/sync.js

@@ -3,9 +3,9 @@ import { join } from 'node:path';
 import { createInterface } from 'node:readline/promises';
 import chalk from 'chalk';
 import ora from 'ora';
+import { resolveAgentTargets } from '../lib/agent-paths.js';
 import { track } from '../lib/analytics.js';
 import { api, ApiError } from '../lib/api.js';
-import { CONFIG } from '../lib/config.js';
 import { requireAuth } from '../lib/credentials.js';
 import { fetchMyPlan, formatUpgradeHint } from '../lib/plan.js';
 import { readState, writeState } from '../lib/state.js';
@@ -111,28 +111,49 @@ export async function syncCommand() {
             return;
         }
     }
+    // Resolve every on-disk target the user has enabled in web settings.
+    // Pre-2026-06-05 sync was hard-pinned to ~/.claude/skills/ — users
+    // with Codex / Cline / Amp enabled saw nothing land in those dirs.
+    // Now we read the server-side agent settings and write to every
+    // enabled (non-conversion) path. Cursor is excluded for now because
+    // it needs format conversion.
+    const targets = await resolveAgentTargets();
+    const targetSummary = targets.map((t) => t.agent).join(', ');
     const spinner = ora('Scanning local Skills…').start();
-    let entries = [];
-    try {
-        entries = await readdir(CONFIG.skillsDir);
+    // Union of slugs across all enabled targets — a skill that only
+    // exists in ~/.claude/skills is still "installed", and one we
+    // pulled into ~/.codex/skills the last sync should re-sync too.
+    const slugSet = new Set();
+    let scannedAny = false;
+    for (const target of targets) {
+        let entries = [];
+        try {
+            entries = await readdir(target.dir);
+            scannedAny = true;
+        }
+        catch {
+            // Missing dir → user enabled the agent in settings but never
+            // installed a skill there. Sync will create it on first write.
+            continue;
+        }
+        for (const name of entries) {
+            const dir = join(target.dir, name);
+            if ((await stat(dir).catch(() => null))?.isDirectory())
+                slugSet.add(name);
+        }
     }
-    catch {
-        spinner.warn(`No Skills directory at ${CONFIG.skillsDir}`);
+    if (!scannedAny && slugSet.size === 0) {
+        spinner.warn(`No Skills directory found across ${targetSummary || 'claude'}.`);
         return;
     }
-    const slugs = [];
-    for (const name of entries) {
-        const dir = join(CONFIG.skillsDir, name);
-        if ((await stat(dir).catch(() => null))?.isDirectory())
-            slugs.push(name);
-    }
+    const slugs = Array.from(slugSet);
     if (!slugs.length) {
         spinner.warn('No installed Skills to sync.');
         return;
     }
-    spinner.succeed(`Found ${slugs.length} installed Skill${slugs.length === 1 ? '' : 's'}.`);
+    spinner.succeed(`Found ${slugs.length} installed Skill${slugs.length === 1 ? '' : 's'} across ${targetSummary || 'claude'}.`);
     const estSeconds = estimateSeconds(slugs.length);
-    console.log(chalk.dim(`Syncing ${slugs.length} Skill${slugs.length === 1 ? '' : 's'} — this takes about ${estSeconds} seconds.`));
+    console.log(chalk.dim(`Syncing ${slugs.length} Skill${slugs.length === 1 ? '' : 's'} → ${targets.length} agent dir${targets.length === 1 ? '' : 's'} — this takes about ${estSeconds} seconds.`));
     const progress = ora(`Synced 0 / ${slugs.length}`).start();
     let updated = 0;
     let paywalled = 0;
@@ -170,7 +191,10 @@ export async function syncCommand() {
         // Track per-slug verdicts for the final summary line.
         missing += response.missing.length;
         missingSlugs.push(...response.missing);
-        // Write files in parallel, bounded.
+        // Write files in parallel, bounded. Each successful item gets
+        // written to EVERY target dir (claude + codex + …) — failures on
+        // a single target don't abort the others; we just count a skill
+        // as updated when at least one write succeeded.
         await runWithConcurrency(response.items, WRITE_CONCURRENCY, async (item) => {
             if (item.error === 'paid_unpurchased') {
                 paywalled += 1;
@@ -181,15 +205,23 @@ export async function syncCommand() {
                 noContent += 1;
                 return;
             }
-            const targetDir = join(CONFIG.skillsDir, item.slug);
-            try {
-                await mkdir(targetDir, { recursive: true });
-                await writeFile(join(targetDir, 'SKILL.md'), item.content, 'utf8');
+            let anyWritten = false;
+            for (const target of targets) {
+                const dir = join(target.dir, item.slug);
+                try {
+                    await mkdir(dir, { recursive: true });
+                    await writeFile(join(dir, 'SKILL.md'), item.content, 'utf8');
+                    anyWritten = true;
+                }
+                catch {
+                    /* per-target write failure — continue with other targets */
+                }
+            }
+            if (anyWritten) {
                 updated += 1;
                 progress.text = `Synced ${updated} / ${slugs.length} — ${item.slug}`;
             }
-            catch {
-                // Disk-level failure — surface in the summary, don't crash.
+            else {
                 noContent += 1;
             }
         });

package/dist/lib/agent-paths.js

@@ -0,0 +1,56 @@
+import { homedir } from 'node:os';
+import { AGENT_REGISTRY } from '@prave/shared';
+import { api, ApiError } from './api.js';
+import { CONFIG } from './config.js';
+const isWindows = process.platform === 'win32';
+function expandTilde(p) {
+    if (!p)
+        return p;
+    if (p === '~')
+        return homedir();
+    if (p.startsWith('~/') || p.startsWith('~\\')) {
+        return homedir() + p.slice(1);
+    }
+    return p;
+}
+function pickOsPath(paths) {
+    return isWindows ? paths.windows : paths.mac;
+}
+export async function resolveAgentTargets() {
+    try {
+        const { data: settings } = await api.get('/api/v1/settings/agents', true);
+        const enabled = settings.enabled_agents ?? [];
+        if (enabled.length === 0) {
+            return [{ agent: 'claude', dir: CONFIG.skillsDir }];
+        }
+        const seen = new Set();
+        const targets = [];
+        for (const agent of enabled) {
+            const meta = AGENT_REGISTRY[agent];
+            if (!meta)
+                continue;
+            if (meta.conversionRequired)
+                continue;
+            const stored = settings.skill_paths?.[agent];
+            const raw = stored
+                ? pickOsPath(stored)
+                : pickOsPath(meta.defaultPath);
+            const dir = expandTilde(raw).replace(/[\\/]+$/, '');
+            if (!dir || seen.has(dir))
+                continue;
+            seen.add(dir);
+            targets.push({ agent, dir });
+        }
+        if (targets.length === 0) {
+            return [{ agent: 'claude', dir: CONFIG.skillsDir }];
+        }
+        return targets;
+    }
+    catch (err) {
+        // Fail-safe path: legacy single-claude behaviour. A 401 here just
+        // means the user isn't authed and the caller will surface that
+        // through requireAuth() anyway.
+        void (err instanceof ApiError);
+        return [{ agent: 'claude', dir: CONFIG.skillsDir }];
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prave/cli",
-  "version": "1.4.15",
+  "version": "1.5.1",
   "description": "Prave CLI — discover, install, version, test, and ship Claude Skills. The developer platform for the complete Skill lifecycle.",
   "type": "module",
   "keywords": [
@@ -54,7 +54,7 @@
     "ora": "^8.0.1",
     "tar": "^7.4.3",
     "undici": "^6.18.0",
-    "@prave/shared": "1.4.15"
+    "@prave/shared": "1.4.16"
   },
   "devDependencies": {
     "@types/node": "^20.12.7",