@prave/cli 1.2.0 → 1.2.1

package/dist/commands/conflicts.js

@@ -1,9 +1,47 @@
+import { stat } from 'node:fs/promises';
+import { join } from 'node:path';
 import chalk from 'chalk';
 import ora from 'ora';
 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 { checkboxPrompt } from '../lib/prompt.js';
+import { isValidSlug } from '../lib/slug.js';
 import { log } from '../utils/logger.js';
+import { uninstallCommand } from './uninstall.js';
+/**
+ * Resolve a `skill_metadata` row back to its on-disk slug, mirroring
+ * the canonical `~/.claude/skills/<slug>/SKILL.md` layout used by
+ * `prave optimize --remove-unused`. Returns `null` when the slug
+ * fails the registry regex (defence against `..` injection via a
+ * malformed `file_path`).
+ */
+function slugOf(s) {
+    const segs = s.file_path?.split('/').filter(Boolean) ?? [];
+    let candidate = null;
+    if (segs.length >= 2) {
+        const last = segs[segs.length - 1];
+        const parent = segs[segs.length - 2];
+        if (last.toLowerCase().endsWith('skill.md'))
+            candidate = parent;
+    }
+    if (!candidate && s.name) {
+        candidate = s.name.trim().toLowerCase().replace(/\s+/g, '-');
+    }
+    if (!candidate)
+        return null;
+    return isValidSlug(candidate) ? candidate : null;
+}
+async function existsOnDisk(slug) {
+    try {
+        const st = await stat(join(CONFIG.skillsDir, slug));
+        return st.isDirectory();
+    }
+    catch {
+        return false;
+    }
+}
 function describe(c) {
     const a = c.skill_a_name ?? c.skill_a_id;
     const b = c.skill_b_name ?? c.skill_b_id;
@@ -44,8 +82,70 @@ export async function conflictsCommand(opts = {}) {
             console.log(`${chalk.yellow('⚠️ ')} ${describe(c)}`);
         }
         if (opts.fix) {
+            // Interactive resolution: the most common (and safest) fix for a
+            // conflict is uninstalling one of the two competing Skills. We
+            // build a checkbox list of the unique slugs that appear in the
+            // conflict set, restricted to ones we can resolve back to a real
+            // on-disk folder (so we never offer to "uninstall" something the
+            // user hasn't actually got installed).
+            let metadata = [];
+            try {
+                const { data } = await api.get('/api/v1/intelligence/skills', true);
+                metadata = data;
+            }
+            catch (err) {
+                log.warn(`Could not fetch skill metadata — ${err.message}`);
+            }
+            // Tally how many conflicts each Skill participates in. Heuristic:
+            // a skill that conflicts with two others is the strongest fix
+            // candidate (uninstalling it resolves multiple rows at once), so
+            // we surface it first.
+            const conflictCount = new Map();
+            for (const c of conflicts) {
+                conflictCount.set(c.skill_a_id, (conflictCount.get(c.skill_a_id) ?? 0) + 1);
+                conflictCount.set(c.skill_b_id, (conflictCount.get(c.skill_b_id) ?? 0) + 1);
+            }
+            const candidates = [];
+            const seen = new Set();
+            for (const m of metadata) {
+                if (!conflictCount.has(m.id))
+                    continue;
+                const slug = slugOf(m);
+                if (!slug || seen.has(slug))
+                    continue;
+                if (!(await existsOnDisk(slug)))
+                    continue;
+                seen.add(slug);
+                candidates.push({
+                    slug,
+                    name: m.name ?? slug,
+                    count: conflictCount.get(m.id) ?? 1,
+                });
+            }
+            candidates.sort((a, b) => b.count - a.count || a.slug.localeCompare(b.slug));
+            if (candidates.length === 0) {
+                console.log();
+                log.dim('Nothing to auto-fix — no conflicting Skills are installed under ~/.claude/skills/. ' +
+                    'Run `prave whatdoes <skill>` to inspect frontmatter and resolve manually.');
+                return;
+            }
+            console.log();
+            const picks = await checkboxPrompt('Select Skills to uninstall:', candidates.map((c) => ({
+                value: c.slug,
+                label: c.name,
+                hint: c.count > 1
+                    ? `${c.count} conflicts · ${c.slug}`
+                    : `1 conflict · ${c.slug}`,
+            })));
+            if (!picks || picks.length === 0) {
+                log.dim('Aborted — nothing removed.');
+                return;
+            }
+            for (const slug of picks) {
+                await uninstallCommand(slug);
+            }
             console.log();
-            log.dim('Interactive fix coming soon — for now run `prave whatdoes <skill>` and adjust frontmatter manually.');
+            log.dim(`Removed ${picks.length} skill(s). Run ${chalk.cyan('prave sync')} to refresh server-side metadata.`);
         }
     }
     catch (err) {

package/dist/commands/optimize.js

@@ -1,9 +1,13 @@
+import { stat } from 'node:fs/promises';
+import { join } from 'node:path';
+import readline from 'node:readline/promises';
 import chalk from 'chalk';
 import ora from 'ora';
-import readline from 'node:readline/promises';
 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 { checkboxPrompt } from '../lib/prompt.js';
 import { isValidSlug } from '../lib/slug.js';
 import { log } from '../utils/logger.js';
 import { uninstallCommand } from './uninstall.js';
@@ -40,6 +44,15 @@ function slugOf(s) {
     // injected via a malformed `file_path`.
     return isValidSlug(candidate) ? candidate : null;
 }
+async function existsOnDisk(slug) {
+    try {
+        const st = await stat(join(CONFIG.skillsDir, slug));
+        return st.isDirectory();
+    }
+    catch {
+        return false;
+    }
+}
 async function confirmYesNo(question) {
     const rl = readline.createInterface({ input: process.stdin, output: process.stdout });
     try {
@@ -121,10 +134,57 @@ export async function optimizeCommand(opts = {}) {
             log.dim(`Removed ${candidates.length} skill(s). Run ${chalk.cyan('prave sync')} to update server-side metadata.`);
         }
         else if (opts.apply) {
+            // `--apply` is the unified interactive cleanup: pick from the
+            // *underused* + *heavy* lists at once and uninstall whatever the
+            // user agrees to. This is strictly safer than blanket-deleting
+            // because every selection is opt-in.
+            //
+            // Merge candidates are intentionally NOT auto-actionable — merging
+            // two Skills is a content-rewrite that the user has to do by hand.
+            // We surface them above and leave them for the suggestions panel.
+            const offer = [];
+            const seen = new Set();
+            const push = (skill, bucket) => {
+                const slug = slugOf(skill);
+                if (!slug || seen.has(slug))
+                    return;
+                seen.add(slug);
+                offer.push({ slug, skill, bucket });
+            };
+            for (const s of data.underused)
+                push(s, 'underused');
+            for (const s of data.heavy)
+                push(s, 'heavy');
+            // Filter to ones actually present on disk — the API returns
+            // metadata for slug-keyed self-heal stubs that may not have a
+            // matching folder.
+            const installed = [];
+            for (const item of offer) {
+                if (await existsOnDisk(item.slug))
+                    installed.push(item);
+            }
+            if (installed.length === 0) {
+                console.log();
+                log.dim('Nothing on disk to clean up — every flagged skill is either uninstalled already or only relevant to merge candidates (which need a manual rewrite).');
+                return;
+            }
+            console.log();
+            const picks = await checkboxPrompt('Pick skills to uninstall:', installed.map((i) => ({
+                value: i.slug,
+                label: nameOf(i.skill),
+                hint: i.bucket === 'underused'
+                    ? `underused · ${formatTokens(i.skill.estimated_tokens)} · ${i.slug}`
+                    : `heavy · ${formatTokens(i.skill.estimated_tokens)} · ${i.slug}`,
+            })));
+            if (!picks || picks.length === 0) {
+                log.dim('Aborted — nothing removed.');
+                return;
+            }
+            for (const slug of picks) {
+                await uninstallCommand(slug);
+            }
             console.log();
-            log.dim('Auto-apply not available — use ' +
-                chalk.cyan('prave optimize --remove-unused') +
-                ' to delete underused skills, or review the suggestions and adjust manually.');
+            log.dim(`Removed ${picks.length} skill(s). Run ${chalk.cyan('prave sync')} to refresh server-side metadata.`);
         }
     }
     catch (err) {

package/dist/commands/usage.js

@@ -104,45 +104,59 @@ export async function usageScanCommand(opts) {
     }
 }
 /**
- * `prave usage report` — invoked by the Claude Code `PostToolUse` hook.
+ * `prave usage report` — invoked by the Claude Code `PostToolUse` hook
+ * (and, when `--source=prompt`, the companion `UserPromptSubmit` hook).
  * Reads the hook payload from stdin, extracts the Skill name, and fires
  * a single-event POST against the slug-keyed self-healing endpoint.
  * Errors are silent (we never want a hook failure to disturb the user's
- * workflow); set `PRAVE_DEBUG=1` to see what's happening in
- * `~/.prave/usage.log`.
+ * workflow); a rotated `~/.prave/hook.log` always captures the last 200
+ * fires so users can verify telemetry without flipping `PRAVE_DEBUG=1`.
+ *
+ * Hook discipline:
+ *   - reads stdin with a 250 ms cap so we never block Claude Code
+ *   - one POST attempt with a hard 4 s deadline (background-friendly)
+ *   - on auth/network failure we log + bail; never throw
  */
-export async function usageReportCommand() {
+export async function usageReportCommand(opts = {}) {
     const debug = process.env.PRAVE_DEBUG === '1' || process.env.PRAVE_DEBUG === 'true';
+    const source = opts.source === 'prompt' ? 'prompt' : 'tool';
+    await rotateHookLog();
     const stdinPayload = await readStdin();
     if (!stdinPayload) {
+        await hookLog(`${source}:no-stdin`);
         if (debug)
             await debugLog('no stdin payload');
         return;
     }
     if (debug)
-        await debugLog(`stdin len=${stdinPayload.length}`);
+        await debugLog(`stdin len=${stdinPayload.length} source=${source}`);
     // Claude Code's PostToolUse payload is documented as `{tool_name,
-    // tool_input, tool_response, ...}`, but we still defend against
-    // alternate shapes (capitalisation, future schema drift) so the hook
-    // doesn't silently break on a single field rename.
+    // tool_input, tool_response, ...}`. UserPromptSubmit ships
+    // `{ prompt, ... }`. We defend against alternate shapes (capitalisation,
+    // future schema drift) so the hook doesn't silently break on a single
+    // field rename.
     let parsed = {};
     try {
         parsed = JSON.parse(stdinPayload);
     }
     catch {
+        await hookLog(`${source}:bad-json`);
         if (debug)
             await debugLog('payload is not valid JSON');
         return;
     }
-    const rawSlug = extractSkillSlug(parsed);
+    const rawSlug = source === 'prompt' ? extractSlashCommand(parsed) : extractSkillSlug(parsed);
     if (!rawSlug) {
+        await hookLog(`${source}:no-slug`);
         if (debug)
             await debugLog(`no skill slug in payload: ${stdinPayload.slice(0, 200)}`);
         return;
     }
     const slug = rawSlug.toLowerCase().split(':').pop()?.trim().replace(/[^a-z0-9_-]+/g, '-');
-    if (!slug)
+    if (!slug) {
+        await hookLog(`${source}:empty-slug`);
         return;
+    }
     if (debug)
         await debugLog(`slug=${slug}`);
     const session = await requireAuthSilent();
@@ -183,18 +197,46 @@ export async function usageReportCommand() {
             slug,
             agent_type: 'claude',
             triggered_at: new Date().toISOString(),
-            meta,
+            meta: { ...meta, source },
         }, true);
+        await hookLog(`${source}:ok slug=${slug} recorded=${data.recorded} stub=${data.created_stub}`);
         if (debug) {
             await debugLog(`ok recorded=${data.recorded} stub=${data.created_stub} meta=${JSON.stringify(meta)}`);
         }
     }
     catch (err) {
+        const msg = err.message;
+        await hookLog(`${source}:err slug=${slug} ${msg.slice(0, 120)}`);
         if (debug)
-            await debugLog(`error: ${err.message}`);
+            await debugLog(`error: ${msg}`);
         /* silent — never break the host shell */
     }
 }
+/**
+ * Extract a slash-command name from a UserPromptSubmit payload. Claude
+ * Code ships `{ prompt: '/graphify ...' }` (and tolerates leading
+ * whitespace). We treat the first whitespace-separated token after `/`
+ * as the slug. Returns `null` for non-slash prompts so we don't waste
+ * an API hop on every keystroke.
+ */
+function extractSlashCommand(payload) {
+    const candidates = [
+        payload.prompt,
+        payload.user_prompt,
+        payload.input,
+    ];
+    for (const c of candidates) {
+        if (typeof c !== 'string')
+            continue;
+        const trimmed = c.trimStart();
+        if (!trimmed.startsWith('/'))
+            continue;
+        const token = trimmed.slice(1).split(/[\s\n\r]/, 1)[0] ?? '';
+        if (token)
+            return token;
+    }
+    return null;
+}
 /**
  * Walk the hook payload looking for the invoked Skill's slug. Tries the
  * documented Claude Code path first, then several plausible aliases so
@@ -226,6 +268,45 @@ async function debugLog(line) {
         /* swallow — debug is best-effort */
     }
 }
+/**
+ * Always-on per-fire breadcrumb log so users can answer "did the hook
+ * run?" without flipping `PRAVE_DEBUG=1`. Capped at 200 lines (rotated
+ * lazily by `rotateHookLog`). Lives at `~/.prave/hook.log`.
+ */
+async function hookLog(line) {
+    try {
+        const { mkdir, appendFile } = await import('node:fs/promises');
+        const { join } = await import('node:path');
+        await mkdir(CONFIG.praveDir, { recursive: true });
+        const stamp = new Date().toISOString();
+        await appendFile(join(CONFIG.praveDir, 'hook.log'), `${stamp} ${line}\n`);
+    }
+    catch {
+        /* swallow */
+    }
+}
+const HOOK_LOG_MAX_LINES = 200;
+async function rotateHookLog() {
+    try {
+        const { stat, readFile, writeFile } = await import('node:fs/promises');
+        const { join } = await import('node:path');
+        const path = join(CONFIG.praveDir, 'hook.log');
+        const info = await stat(path).catch(() => null);
+        // Only do the read-rewrite dance occasionally — when the file gets
+        // big enough that we suspect overflow. 32 KB ≈ 200 short lines.
+        if (!info || info.size < 32 * 1024)
+            return;
+        const raw = await readFile(path, 'utf8');
+        const lines = raw.split('\n');
+        if (lines.length <= HOOK_LOG_MAX_LINES)
+            return;
+        const trimmed = lines.slice(-HOOK_LOG_MAX_LINES).join('\n');
+        await writeFile(path, trimmed, 'utf8');
+    }
+    catch {
+        /* swallow */
+    }
+}
 export async function usageHookInstallCommand() {
     track('cli_usage_hook_install');
     const session = await requireAuth('prave usage hook install');
@@ -269,16 +350,22 @@ export async function usageStatusCommand() {
     const { readFile } = await import('node:fs/promises');
     const { join } = await import('node:path');
     const { homedir } = await import('node:os');
-    // 1. Hook installed?
+    // 1. Hook installed? Check both channels we manage.
     const settingsPath = join(homedir(), '.claude', 'settings.json');
-    let hookInstalled = false;
+    let toolHookInstalled = false;
+    let promptHookInstalled = false;
     try {
         const raw = await readFile(settingsPath, 'utf8');
-        hookInstalled = raw.includes('__prave_managed') && raw.includes('Skill');
+        const parsed = JSON.parse(raw);
+        toolHookInstalled =
+            (parsed.hooks?.PostToolUse ?? []).some((b) => b.matcher === 'Skill' && (b.hooks ?? []).some((h) => h.__prave_managed === true));
+        promptHookInstalled =
+            (parsed.hooks?.UserPromptSubmit ?? []).some((b) => (b.hooks ?? []).some((h) => h.__prave_managed === true));
     }
     catch {
         /* no settings.json yet */
     }
+    const hookInstalled = toolHookInstalled && promptHookInstalled;
     // 2. Last 7 days of recorded events from the API.
     let recent7 = 0;
     let topSlugs = [];
@@ -300,14 +387,33 @@ export async function usageStatusCommand() {
     catch {
         /* never scanned */
     }
-    // 4. Debug log tail.
-    const logPath = join(CONFIG.praveDir, 'usage.log');
-    const debugAvailable = existsSync(logPath);
+    // 4. Debug log + hook log tails.
+    const debugLogPath = join(CONFIG.praveDir, 'usage.log');
+    const hookLogPath = join(CONFIG.praveDir, 'hook.log');
+    const debugAvailable = existsSync(debugLogPath);
+    const hookLogAvailable = existsSync(hookLogPath);
+    // 5. API reachability — token-validating ping. We already passed
+    // `requireAuth` above so the credentials file exists; this round-trip
+    // just confirms the access_token is still accepted server-side. If the
+    // hook has been silently 401-ing for a week, this is the only way the
+    // user finds out without flipping `PRAVE_DEBUG=1`.
+    let apiReachable = false;
+    let apiMessage = '';
+    try {
+        await api.get('/api/v1/intelligence/usage/recent?days=1', true);
+        apiReachable = true;
+    }
+    catch (err) {
+        apiMessage =
+            err instanceof ApiError ? `${err.status} ${err.message}` : err.message;
+    }
     // Render.
-    const checkmark = (ok) => (ok ? chalk.green('✓') : chalk.dim('—'));
+    const checkmark = (ok) => (ok ? chalk.green('✓') : chalk.red('✗'));
     log.info(chalk.bold('Usage tracking status'));
     console.log();
-    console.log(`  ${checkmark(hookInstalled)} Real-time hook (Claude Code): ${hookInstalled ? chalk.green('installed') : chalk.yellow('not installed — `prave usage hook install`')}`);
+    console.log(`  ${checkmark(toolHookInstalled)} PostToolUse hook (Skill tool fires): ${toolHookInstalled ? chalk.green('installed') : chalk.yellow('missing — `prave usage hook install`')}`);
+    console.log(`  ${checkmark(promptHookInstalled)} UserPromptSubmit hook (slash commands like /graphify): ${promptHookInstalled ? chalk.green('installed') : chalk.yellow('missing — `prave usage hook install`')}`);
+    console.log(`  ${checkmark(apiReachable)} API reachable + auth valid: ${apiReachable ? chalk.green('yes') : chalk.red(apiMessage || 'no')}`);
     console.log(`  ${checkmark(Boolean(lastScanAt))} Transcript scanner watermark: ${lastScanAt ?? chalk.dim('never run — `prave sync` includes it')}`);
     console.log(`  ${checkmark(recent7 > 0)} Events in last 7 days: ${chalk.cyan(String(recent7))}`);
     if (topSlugs.length) {
@@ -317,20 +423,41 @@ export async function usageStatusCommand() {
             console.log(`    ${chalk.cyan(s.count.toString().padStart(4))}  ${s.name}`);
         }
     }
-    console.log();
-    if (debugAvailable) {
-        log.dim(`Debug log: ${logPath}`);
-        log.dim('Set PRAVE_DEBUG=1 in your shell to enable verbose hook logging.');
-    }
-    else {
-        log.dim('No debug log yet. Set PRAVE_DEBUG=1 to capture every hook fire to ~/.prave/usage.log');
+    // Last 5 hook fires — the breadcrumb trail that answers "did the hook
+    // even run when I typed /graphify?". Read the bottom of hook.log.
+    if (hookLogAvailable) {
+        try {
+            const raw = await readFile(hookLogPath, 'utf8');
+            const tail = raw.trimEnd().split('\n').slice(-5);
+            if (tail.length) {
+                console.log();
+                console.log(chalk.dim('  Last hook fires:'));
+                for (const line of tail)
+                    console.log(`    ${chalk.dim(line)}`);
+            }
+        }
+        catch {
+            /* swallow */
+        }
     }
-    if (recent7 === 0) {
+    console.log();
+    log.dim(`Hook breadcrumb log: ${hookLogPath}`);
+    if (debugAvailable)
+        log.dim(`Verbose debug log:    ${debugLogPath}`);
+    log.dim('Set PRAVE_DEBUG=1 in your shell to enable verbose hook logging.');
+    if (!hookInstalled || !apiReachable || recent7 === 0) {
         console.log();
-        log.warn('No events recorded yet. Quick checks:');
-        log.dim('  1. Is the hook installed? See checkmarks above.');
-        log.dim('  2. Run a Skill in Claude Code, then `tail ~/.prave/usage.log` (with PRAVE_DEBUG=1)');
-        log.dim('  3. Or run `prave usage scan --since 7d` to backfill from transcripts.');
+        log.warn('Telemetry may be incomplete. Suggested fixes:');
+        if (!toolHookInstalled || !promptHookInstalled) {
+            log.dim('  • Run `prave usage hook install` to wire BOTH PostToolUse and UserPromptSubmit.');
+        }
+        if (!apiReachable) {
+            log.dim('  • Run `prave login` — your access token may have expired (silently 401-ing).');
+        }
+        if (recent7 === 0 && hookInstalled && apiReachable) {
+            log.dim('  • Run a Skill in Claude Code, then re-run `prave usage status`.');
+            log.dim('  • Or run `prave usage scan --since 7d` to backfill from transcripts.');
+        }
     }
 }
 async function fetchEnabledAgents() {
@@ -414,13 +541,37 @@ function parseSinceFlag(raw) {
 async function readStdin() {
     if (process.stdin.isTTY)
         return '';
-    const chunks = [];
-    for await (const chunk of process.stdin) {
-        chunks.push(typeof chunk === 'string' ? Buffer.from(chunk) : chunk);
-        if (Buffer.concat(chunks).length > 1_000_000)
-            break;
-    }
-    return Buffer.concat(chunks).toString('utf8');
+    // Hard 1.5 s cap so a stuck pipe never blocks Claude Code's tool flow.
+    // 1 MB payload cap so a runaway stream can't OOM the hook either.
+    return new Promise((resolve) => {
+        const chunks = [];
+        let total = 0;
+        let settled = false;
+        const finish = () => {
+            if (settled)
+                return;
+            settled = true;
+            resolve(Buffer.concat(chunks).toString('utf8'));
+        };
+        const timer = setTimeout(finish, 1500);
+        process.stdin.on('data', (chunk) => {
+            const buf = typeof chunk === 'string' ? Buffer.from(chunk) : chunk;
+            chunks.push(buf);
+            total += buf.length;
+            if (total > 1_000_000) {
+                clearTimeout(timer);
+                finish();
+            }
+        });
+        process.stdin.once('end', () => {
+            clearTimeout(timer);
+            finish();
+        });
+        process.stdin.once('error', () => {
+            clearTimeout(timer);
+            finish();
+        });
+    });
 }
 /**
  * Auth check with no UX side-effects. Used by the hook so a logged-out

package/dist/index.js

@@ -119,12 +119,12 @@ program
 program
     .command('conflicts')
     .description('Detect overlap, collisions, and missing dependencies')
-    .option('--fix', 'placeholder for interactive fix flow')
+    .option('--fix', 'after listing conflicts, interactively pick installed Skills to uninstall to resolve them')
     .action(conflictsCommand);
 program
     .command('optimize')
     .description('Recommendations: underused, mergeable, and heavy skills')
-    .option('--apply', 'placeholder for auto-apply')
+    .option('--apply', 'after listing recommendations, interactively pick underused + heavy skills to uninstall')
     .option('--remove-unused', 'interactively delete skills that have not fired in 30+ days from ~/.claude/skills/')
     .option('--yes', 'with --remove-unused: skip the confirmation prompt')
     .action(optimizeCommand);
@@ -143,8 +143,9 @@ usage
     .action(usageStatusCommand);
 usage
     .command('report')
-    .description('Internal: invoked by the Claude Code PostToolUse hook (reads stdin)')
-    .action(usageReportCommand);
+    .description('Internal: invoked by the Claude Code PostToolUse / UserPromptSubmit hook (reads stdin)')
+    .option('--source <kind>', 'hook channel that fired this report ("tool" or "prompt")', 'tool')
+    .action((opts) => usageReportCommand(opts));
 const hook = usage.command('hook').description('Install/uninstall the Claude Code real-time usage hook');
 hook
     .command('install')

package/dist/lib/hook.js

@@ -22,25 +22,50 @@ const HOOK_MARKER = '__prave_managed';
  */
 export const HOOK_SUPPORTED = ['claude'];
 const HOOK_COMMAND = 'prave usage report';
+// Companion command for the UserPromptSubmit channel so a typed-slash
+// `/graphify` is captured even when the Skill tool path doesn't fire a
+// matching PostToolUse with a populated `tool_input.skill` field.
+const PROMPT_HOOK_COMMAND = 'prave usage report --source=prompt';
+/**
+ * Install on BOTH `PostToolUse` (matcher `Skill`) and `UserPromptSubmit`
+ * (catches slash commands like `/graphify` before tool dispatch). The two
+ * channels are deduped server-side by per-minute bucket, so double-fires
+ * for the same Skill in the same minute land as a single row — but every
+ * additional minute of activity is preserved. Idempotent + atomic.
+ */
 export async function installSkillHook() {
     const settings = await readSettings();
     settings.hooks ??= {};
     settings.hooks.PostToolUse ??= [];
-    const blocks = settings.hooks.PostToolUse;
-    const existingIdx = blocks.findIndex((b) => b.matcher === 'Skill' && b.hooks?.some((h) => h[HOOK_MARKER]));
-    const fresh = {
+    settings.hooks.UserPromptSubmit ??= [];
+    const postBlocks = settings.hooks.PostToolUse;
+    const promptBlocks = settings.hooks.UserPromptSubmit;
+    const postFresh = {
         matcher: 'Skill',
         hooks: [{ type: 'command', command: HOOK_COMMAND, [HOOK_MARKER]: true }],
     };
-    if (existingIdx >= 0) {
-        const existingCmd = blocks[existingIdx]?.hooks?.[0]?.command;
-        if (existingCmd === HOOK_COMMAND) {
-            return { installed: false, alreadyPresent: true, settingsPath: SETTINGS_PATH };
+    const promptFresh = {
+        // UserPromptSubmit has no matcher concept in Claude Code today — the
+        // hook script itself filters for `prompt.startsWith('/')` before
+        // doing any work.
+        hooks: [{ type: 'command', command: PROMPT_HOOK_COMMAND, [HOOK_MARKER]: true }],
+    };
+    const upsert = (blocks, fresh, expectedCmd) => {
+        const idx = blocks.findIndex((b) => b.matcher === fresh.matcher && b.hooks?.some((h) => h[HOOK_MARKER]));
+        if (idx >= 0) {
+            const existingCmd = blocks[idx]?.hooks?.[0]?.command;
+            if (existingCmd === expectedCmd)
+                return false;
+            blocks[idx] = fresh;
+            return true;
         }
-        blocks[existingIdx] = fresh;
-    }
-    else {
         blocks.push(fresh);
+        return true;
+    };
+    const changedPost = upsert(postBlocks, postFresh, HOOK_COMMAND);
+    const changedPrompt = upsert(promptBlocks, promptFresh, PROMPT_HOOK_COMMAND);
+    if (!changedPost && !changedPrompt) {
+        return { installed: false, alreadyPresent: true, settingsPath: SETTINGS_PATH };
     }
     await writeSettings(settings);
     return { installed: true, alreadyPresent: false, settingsPath: SETTINGS_PATH };
@@ -93,25 +118,35 @@ export async function uninstallHooksForAgents(agents) {
 }
 export async function uninstallSkillHook() {
     const settings = await readSettings();
-    const blocks = settings.hooks?.PostToolUse;
-    if (!blocks?.length)
+    if (!settings.hooks)
         return { removed: false, settingsPath: SETTINGS_PATH };
-    const before = blocks.length;
-    const filtered = blocks
-        .map((b) => ({
-        ...b,
-        hooks: b.hooks?.filter((h) => !h[HOOK_MARKER]),
-    }))
-        .filter((b) => (b.hooks?.length ?? 0) > 0);
-    if (filtered.length === before && filtered.every((b, i) => b.hooks?.length === blocks[i]?.hooks?.length)) {
+    let touched = false;
+    const stripChannel = (channel) => {
+        const blocks = settings.hooks?.[channel];
+        if (!blocks?.length)
+            return;
+        const beforeCounts = blocks.map((b) => b.hooks?.length ?? 0);
+        const filtered = blocks
+            .map((b) => ({ ...b, hooks: b.hooks?.filter((h) => !h[HOOK_MARKER]) }))
+            .filter((b) => (b.hooks?.length ?? 0) > 0);
+        const lengthChanged = filtered.length !== blocks.length;
+        const innerChanged = !lengthChanged &&
+            filtered.some((b, i) => (b.hooks?.length ?? 0) !== beforeCounts[i]);
+        if (!lengthChanged && !innerChanged)
+            return;
+        touched = true;
+        if (settings.hooks) {
+            settings.hooks[channel] = filtered.length ? filtered : undefined;
+            if (!filtered.length)
+                delete settings.hooks[channel];
+        }
+    };
+    stripChannel('PostToolUse');
+    stripChannel('UserPromptSubmit');
+    if (!touched)
         return { removed: false, settingsPath: SETTINGS_PATH };
-    }
-    if (settings.hooks) {
-        settings.hooks.PostToolUse = filtered;
-        if (!filtered.length)
-            delete settings.hooks.PostToolUse;
-        if (Object.keys(settings.hooks).length === 0)
-            delete settings.hooks;
+    if (settings.hooks && Object.keys(settings.hooks).length === 0) {
+        delete settings.hooks;
     }
     await writeSettings(settings);
     return { removed: true, settingsPath: SETTINGS_PATH };

package/package.json

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