skillvault 0.9.1 → 0.9.3

package/dist/cli.js

@@ -20,7 +20,7 @@
 import { existsSync, readFileSync, writeFileSync, mkdirSync, readdirSync, rmSync } from 'node:fs';
 import { join } from 'node:path';
 import { createDecipheriv, createHmac, createPublicKey, diffieHellman, hkdfSync, generateKeyPairSync, } from 'node:crypto';
-const VERSION = '0.9.1';
+const VERSION = '0.9.3';
 const HOME = process.env.HOME || process.env.USERPROFILE || '~';
 const API_URL = process.env.SKILLVAULT_API_URL || 'https://api.getskillvault.com';
 const CONFIG_DIR = join(HOME, '.skillvault');
@@ -246,6 +246,17 @@ function cleanupMCPConfig() {
  *
  * Also removes legacy PostToolCall/PreToolCall hooks from older versions.
  */
+/** Check if a hook entry (old or new format) contains a skillvault command. */
+function hasSkillvaultCommand(entry) {
+    // Old format: { command: "npx skillvault ..." }
+    if (entry.command?.includes('skillvault'))
+        return true;
+    // New format: { hooks: [{ type: "command", command: "npx skillvault ..." }] }
+    if (Array.isArray(entry.hooks)) {
+        return entry.hooks.some((h) => h.command?.includes('skillvault'));
+    }
+    return false;
+}
 function configureSessionHook() {
     const settingsPath = join(HOME, '.claude', 'settings.json');
     try {
@@ -259,50 +270,52 @@ function configureSessionHook() {
             settings.hooks.SessionStart = [];
         if (!settings.hooks.SessionEnd)
             settings.hooks.SessionEnd = [];
-        // ── SessionStart: auto-sync (permanent) ──
-        const hasStartHook = settings.hooks.SessionStart.some((group) => group.matcher === 'startup' &&
-            group.hooks?.some((h) => h.command?.includes('skillvault')));
-        if (!hasStartHook) {
-            settings.hooks.SessionStart.push({
-                matcher: 'startup',
-                hooks: [{
-                        type: 'command',
-                        command: `npx skillvault@${VERSION} --sync`,
-                        timeout: 30,
-                    }],
-            });
-        }
-        // ── SessionEnd: cleanup (permanent) ──
-        const hasEndHook = settings.hooks.SessionEnd.some((h) => h.command?.includes('skillvault') && h.command?.includes('session-cleanup'));
-        if (!hasEndHook) {
-            settings.hooks.SessionEnd.push({
-                command: 'npx skillvault --session-cleanup',
-            });
-        }
-        // ── Remove legacy PostToolCall + PreToolCall hooks ──
+        // ── SessionStart: auto-sync (permanent, unpinned so it always gets latest) ──
+        // Remove all existing skillvault SessionStart hooks and re-add with current format
+        settings.hooks.SessionStart = settings.hooks.SessionStart.filter((h) => !hasSkillvaultCommand(h));
+        settings.hooks.SessionStart.push({
+            matcher: 'startup',
+            hooks: [{
+                    type: 'command',
+                    command: 'npx -y skillvault --sync',
+                    timeout: 30,
+                }],
+        });
+        // ── SessionEnd: cleanup (permanent, pinned to current version) ──
+        // Remove all existing skillvault SessionEnd hooks and re-add with current version
+        settings.hooks.SessionEnd = settings.hooks.SessionEnd.filter((h) => !hasSkillvaultCommand(h));
+        settings.hooks.SessionEnd.push({
+            hooks: [{
+                    type: 'command',
+                    command: `npx -y skillvault@${VERSION} --session-cleanup`,
+                }],
+        });
+        // ── Remove legacy PostToolCall + PreToolCall hooks (invalid event names) ──
         let removedLegacy = false;
-        if (Array.isArray(settings.hooks.PostToolCall)) {
-            const before = settings.hooks.PostToolCall.length;
-            settings.hooks.PostToolCall = settings.hooks.PostToolCall.filter((h) => !h.command?.includes('skillvault'));
-            if (settings.hooks.PostToolCall.length < before)
-                removedLegacy = true;
-            if (settings.hooks.PostToolCall.length === 0)
-                delete settings.hooks.PostToolCall;
+        for (const legacyKey of ['PostToolCall', 'PreToolCall']) {
+            if (Array.isArray(settings.hooks[legacyKey])) {
+                const before = settings.hooks[legacyKey].length;
+                settings.hooks[legacyKey] = settings.hooks[legacyKey].filter((h) => !hasSkillvaultCommand(h));
+                if (settings.hooks[legacyKey].length < before)
+                    removedLegacy = true;
+                if (settings.hooks[legacyKey].length === 0)
+                    delete settings.hooks[legacyKey];
+            }
         }
-        if (Array.isArray(settings.hooks.PreToolCall)) {
-            const before = settings.hooks.PreToolCall.length;
-            settings.hooks.PreToolCall = settings.hooks.PreToolCall.filter((h) => !h.command?.includes('skillvault'));
-            if (settings.hooks.PreToolCall.length < before)
-                removedLegacy = true;
-            if (settings.hooks.PreToolCall.length === 0)
-                delete settings.hooks.PreToolCall;
+        // ── Remove old-format hooks (command on entry instead of hooks array) ──
+        for (const key of ['SessionEnd', 'SessionStart', 'UserPromptSubmit', 'PostToolUse', 'Stop']) {
+            if (Array.isArray(settings.hooks[key])) {
+                const before = settings.hooks[key].length;
+                settings.hooks[key] = settings.hooks[key].filter((h) => !(h.command?.includes('skillvault') && !h.hooks));
+                if (settings.hooks[key].length < before)
+                    removedLegacy = true;
+                if (settings.hooks[key].length === 0)
+                    delete settings.hooks[key];
+            }
         }
         mkdirSync(join(HOME, '.claude'), { recursive: true });
         writeFileSync(settingsPath, JSON.stringify(settings, null, 2), { mode: 0o600 });
-        if (!hasStartHook)
-            console.error('   ✅ Auto-sync hook installed');
-        if (!hasEndHook)
-            console.error('   ✅ Session cleanup hook installed');
+        console.error('   ✅ Hooks updated');
         if (removedLegacy)
             console.error('   🧹 Removed legacy monitoring hooks (now session-scoped)');
     }
@@ -341,34 +354,45 @@ function injectMonitoringHooks(skillName) {
         };
         mkdirSync(CONFIG_DIR, { recursive: true });
         writeFileSync(activeSessionPath, JSON.stringify(session, null, 2), { mode: 0o600 });
-        // ── UserPromptSubmit hook ──
-        if (!Array.isArray(settings.hooks.UserPromptSubmit))
-            settings.hooks.UserPromptSubmit = [];
-        const hasPromptHook = settings.hooks.UserPromptSubmit.some((h) => h.command?.includes('skillvault') && h.command?.includes('session-keepalive'));
-        if (!hasPromptHook) {
-            settings.hooks.UserPromptSubmit.push({
-                command: 'npx skillvault --session-keepalive --event-type prompt-submit',
-            });
+        // ── Remove old-format monitoring hooks (command on entry, no hooks array) ──
+        for (const key of ['UserPromptSubmit', 'PostToolUse', 'Stop']) {
+            if (Array.isArray(settings.hooks[key])) {
+                settings.hooks[key] = settings.hooks[key].filter((h) => !(h.command?.includes('skillvault') && !h.hooks));
+            }
         }
-        // ── PostToolUse hook ──
-        if (!Array.isArray(settings.hooks.PostToolUse))
-            settings.hooks.PostToolUse = [];
-        const hasToolHook = settings.hooks.PostToolUse.some((h) => h.command?.includes('skillvault') && h.command?.includes('session-keepalive'));
-        if (!hasToolHook) {
-            settings.hooks.PostToolUse.push({
-                matcher: 'Write|Bash|Edit',
-                command: 'npx skillvault --session-keepalive --event-type tool-use',
-            });
+        // ── Remove legacy PostToolCall/PreToolCall keys ──
+        for (const legacyKey of ['PostToolCall', 'PreToolCall']) {
+            if (Array.isArray(settings.hooks[legacyKey])) {
+                settings.hooks[legacyKey] = settings.hooks[legacyKey].filter((h) => !hasSkillvaultCommand(h));
+                if (settings.hooks[legacyKey].length === 0)
+                    delete settings.hooks[legacyKey];
+            }
         }
-        // ── Stop hook ──
-        if (!Array.isArray(settings.hooks.Stop))
-            settings.hooks.Stop = [];
-        const hasStopHook = settings.hooks.Stop.some((h) => h.command?.includes('skillvault') && h.command?.includes('session-keepalive'));
-        if (!hasStopHook) {
-            settings.hooks.Stop.push({
-                command: 'npx skillvault --session-keepalive --event-type stop',
-            });
+        // ── Monitoring hooks: remove existing and re-add with current version ──
+        for (const key of ['UserPromptSubmit', 'PostToolUse', 'Stop']) {
+            if (!Array.isArray(settings.hooks[key]))
+                settings.hooks[key] = [];
+            settings.hooks[key] = settings.hooks[key].filter((h) => !hasSkillvaultCommand(h));
         }
+        settings.hooks.UserPromptSubmit.push({
+            hooks: [{
+                    type: 'command',
+                    command: `npx -y skillvault@${VERSION} --session-keepalive --event-type prompt-submit`,
+                }],
+        });
+        settings.hooks.PostToolUse.push({
+            matcher: 'Write|Bash|Edit',
+            hooks: [{
+                    type: 'command',
+                    command: `npx -y skillvault@${VERSION} --session-keepalive --event-type tool-use`,
+                }],
+        });
+        settings.hooks.Stop.push({
+            hooks: [{
+                    type: 'command',
+                    command: `npx -y skillvault@${VERSION} --session-keepalive --event-type stop`,
+                }],
+        });
         writeFileSync(settingsPath, JSON.stringify(settings, null, 2), { mode: 0o600 });
     }
     catch {
@@ -1004,10 +1028,10 @@ function watermarkLayer5(content, id, skillName) {
     const timestamp = new Date().toISOString();
     const secret = 'skillvault-heartbeat-v1'; // shared secret for HMAC
     // HMAC-based 5-digit values
-    const hmac1 = createHmac('sha256', secret);
+    const hmac1 = createHmac('sha256', secret); // nosemgrep: hardcoded-hmac-key — intentional forensic fingerprint key, not a secret
     hmac1.update(`${id}:${timestamp}:hb1`);
     const hb1 = (parseInt(hmac1.digest('hex').slice(0, 8), 16) % 100000).toString().padStart(5, '0');
-    const hmac2 = createHmac('sha256', secret);
+    const hmac2 = createHmac('sha256', secret); // nosemgrep: hardcoded-hmac-key — intentional forensic fingerprint key, not a secret
     hmac2.update(`${id}:${timestamp}:hb2`);
     const hb2 = (parseInt(hmac2.digest('hex').slice(0, 8), 16) % 100000).toString().padStart(5, '0');
     // Detect language from code blocks
@@ -1053,7 +1077,7 @@ function watermarkLayer5(content, id, skillName) {
     // Context templates (selected by HMAC of skill name)
     const ctx1Templates = ['Initialize workspace:', 'Allow environment to settle:', 'Brief initialization delay:', 'Pre-flight pause:'];
     const ctx2Templates = ['Brief pause between operations:', 'Allow buffers to flush:', 'Cooldown before next step:', 'Rate limit compliance delay:'];
-    const ctxHmac = createHmac('sha256', secret);
+    const ctxHmac = createHmac('sha256', secret); // nosemgrep: hardcoded-hmac-key — intentional context template selector
     ctxHmac.update(skillName);
     const ctxHash = ctxHmac.digest();
     const ctx1 = ctx1Templates[ctxHash[0] % ctx1Templates.length];
@@ -1714,49 +1738,27 @@ function handleSessionCleanup() {
             const settings = JSON.parse(readFileSync(settingsPath, 'utf8'));
             if (settings.hooks) {
                 let modified = false;
-                // Remove UserPromptSubmit skillvault hooks
-                if (Array.isArray(settings.hooks.UserPromptSubmit)) {
-                    const before = settings.hooks.UserPromptSubmit.length;
-                    settings.hooks.UserPromptSubmit = settings.hooks.UserPromptSubmit.filter((h) => !h.command?.includes('skillvault'));
-                    if (settings.hooks.UserPromptSubmit.length < before)
-                        modified = true;
-                    if (settings.hooks.UserPromptSubmit.length === 0)
-                        delete settings.hooks.UserPromptSubmit;
-                }
-                // Remove PostToolUse skillvault session-keepalive hooks
-                if (Array.isArray(settings.hooks.PostToolUse)) {
-                    const before = settings.hooks.PostToolUse.length;
-                    settings.hooks.PostToolUse = settings.hooks.PostToolUse.filter((h) => !h.command?.includes('skillvault') || !h.command?.includes('session-keepalive'));
-                    if (settings.hooks.PostToolUse.length < before)
-                        modified = true;
-                    if (settings.hooks.PostToolUse.length === 0)
-                        delete settings.hooks.PostToolUse;
-                }
-                // Remove Stop skillvault session-keepalive hooks
-                if (Array.isArray(settings.hooks.Stop)) {
-                    const before = settings.hooks.Stop.length;
-                    settings.hooks.Stop = settings.hooks.Stop.filter((h) => !h.command?.includes('skillvault') || !h.command?.includes('session-keepalive'));
-                    if (settings.hooks.Stop.length < before)
-                        modified = true;
-                    if (settings.hooks.Stop.length === 0)
-                        delete settings.hooks.Stop;
-                }
-                // Also remove legacy PostToolCall / PreToolCall hooks
-                if (Array.isArray(settings.hooks.PostToolCall)) {
-                    const before = settings.hooks.PostToolCall.length;
-                    settings.hooks.PostToolCall = settings.hooks.PostToolCall.filter((h) => !h.command?.includes('skillvault'));
-                    if (settings.hooks.PostToolCall.length < before)
-                        modified = true;
-                    if (settings.hooks.PostToolCall.length === 0)
-                        delete settings.hooks.PostToolCall;
+                // Remove session-scoped monitoring hooks (both old and new format)
+                for (const key of ['UserPromptSubmit', 'PostToolUse', 'Stop']) {
+                    if (Array.isArray(settings.hooks[key])) {
+                        const before = settings.hooks[key].length;
+                        settings.hooks[key] = settings.hooks[key].filter((h) => !hasSkillvaultCommand(h));
+                        if (settings.hooks[key].length < before)
+                            modified = true;
+                        if (settings.hooks[key].length === 0)
+                            delete settings.hooks[key];
+                    }
                 }
-                if (Array.isArray(settings.hooks.PreToolCall)) {
-                    const before = settings.hooks.PreToolCall.length;
-                    settings.hooks.PreToolCall = settings.hooks.PreToolCall.filter((h) => !h.command?.includes('skillvault'));
-                    if (settings.hooks.PreToolCall.length < before)
-                        modified = true;
-                    if (settings.hooks.PreToolCall.length === 0)
-                        delete settings.hooks.PreToolCall;
+                // Remove legacy PostToolCall / PreToolCall keys (invalid event names)
+                for (const legacyKey of ['PostToolCall', 'PreToolCall']) {
+                    if (Array.isArray(settings.hooks[legacyKey])) {
+                        const before = settings.hooks[legacyKey].length;
+                        settings.hooks[legacyKey] = settings.hooks[legacyKey].filter((h) => !hasSkillvaultCommand(h));
+                        if (settings.hooks[legacyKey].length < before)
+                            modified = true;
+                        if (settings.hooks[legacyKey].length === 0)
+                            delete settings.hooks[legacyKey];
+                    }
                 }
                 if (modified) {
                     writeFileSync(settingsPath, JSON.stringify(settings, null, 2), { mode: 0o600 });
@@ -1850,6 +1852,8 @@ async function main() {
             process.exit(1);
         }
         console.error('🔐 SkillVault Sync\n');
+        // Re-install hooks with current version (auto-upgrades pinned commands)
+        configureSessionHook();
         await syncSkills();
         const result = await installSkillStubs();
         console.error(`\n   ✅ ${result.installed} installed, ${result.skipped} up to date\n`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillvault",
-  "version": "0.9.1",
+  "version": "0.9.3",
   "description": "SkillVault — secure skill distribution for Claude Code",
   "type": "module",
   "bin": {
@@ -11,7 +11,7 @@
     "start": "node dist/cli.js"
   },
   "files": [
-    "dist"
+    "dist/**/*.js"
   ],
   "keywords": [
     "skillvault",

package/dist/cli.d.ts

@@ -1,20 +0,0 @@
-#!/usr/bin/env node
-/**
- * SkillVault — Secure skill distribution for Claude Code.
- *
- * Usage:
- *   npx skillvault --invite CODE          # Setup: redeem invite + sync + install
- *   npx skillvault --load SKILL           # Decrypt a skill (used by Claude Code)
- *   npx skillvault --status               # Show publishers & skills
- *   npx skillvault --refresh              # Re-authenticate tokens + sync
- *   npx skillvault --sync                 # Sync vaults + update stubs
- *
- * How it works:
- *   1. Customer runs --invite once → vaults downloaded, stubs installed
- *   2. Claude Code discovers stub SKILL.md files in ~/.claude/skills/
- *   3. When a skill is triggered, Claude runs: npx skillvault --load <name>
- *   4. The CLI decrypts the vault (license checked) and outputs to stdout
- *   5. Claude reads the output and follows the instructions
- *   6. Decrypted content is never written to disk
- */
-export {};