skillvault 0.9.2 → 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.2';
+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');
@@ -270,33 +270,26 @@ function configureSessionHook() {
             settings.hooks.SessionStart = [];
         if (!settings.hooks.SessionEnd)
             settings.hooks.SessionEnd = [];
-        // ── SessionStart: auto-sync (permanent) ──
-        // Remove any old-format SessionStart entries before checking
-        settings.hooks.SessionStart = settings.hooks.SessionStart.filter((h) => !(h.command?.includes('skillvault') && !h.hooks));
-        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 -y skillvault@${VERSION} --sync`,
-                        timeout: 30,
-                    }],
-            });
-        }
-        // ── SessionEnd: cleanup (permanent) ──
-        // Remove any old-format SessionEnd entries before checking
-        settings.hooks.SessionEnd = settings.hooks.SessionEnd.filter((h) => !(h.command?.includes('skillvault') && !h.hooks));
-        const hasEndHook = settings.hooks.SessionEnd.some((g) => g.hooks?.some((h) => h.command?.includes('skillvault') && h.command?.includes('session-cleanup')));
-        if (!hasEndHook) {
-            settings.hooks.SessionEnd.push({
-                hooks: [{
-                        type: 'command',
-                        command: `npx -y skillvault@${VERSION} --session-cleanup`,
-                    }],
-            });
-        }
+        // ── 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;
         for (const legacyKey of ['PostToolCall', 'PreToolCall']) {
@@ -322,10 +315,7 @@ function configureSessionHook() {
         }
         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)');
     }
@@ -378,43 +368,31 @@ function injectMonitoringHooks(skillName) {
                     delete settings.hooks[legacyKey];
             }
         }
-        // ── UserPromptSubmit hook ──
-        if (!Array.isArray(settings.hooks.UserPromptSubmit))
-            settings.hooks.UserPromptSubmit = [];
-        const hasPromptHook = settings.hooks.UserPromptSubmit.some((g) => g.hooks?.some((h) => h.command?.includes('skillvault') && h.command?.includes('session-keepalive')));
-        if (!hasPromptHook) {
-            settings.hooks.UserPromptSubmit.push({
-                hooks: [{
-                        type: 'command',
-                        command: `npx -y skillvault@${VERSION} --session-keepalive --event-type prompt-submit`,
-                    }],
-            });
-        }
-        // ── PostToolUse hook ──
-        if (!Array.isArray(settings.hooks.PostToolUse))
-            settings.hooks.PostToolUse = [];
-        const hasToolHook = settings.hooks.PostToolUse.some((g) => g.hooks?.some((h) => h.command?.includes('skillvault') && h.command?.includes('session-keepalive')));
-        if (!hasToolHook) {
-            settings.hooks.PostToolUse.push({
-                matcher: 'Write|Bash|Edit',
-                hooks: [{
-                        type: 'command',
-                        command: `npx -y skillvault@${VERSION} --session-keepalive --event-type tool-use`,
-                    }],
-            });
-        }
-        // ── Stop hook ──
-        if (!Array.isArray(settings.hooks.Stop))
-            settings.hooks.Stop = [];
-        const hasStopHook = settings.hooks.Stop.some((g) => g.hooks?.some((h) => h.command?.includes('skillvault') && h.command?.includes('session-keepalive')));
-        if (!hasStopHook) {
-            settings.hooks.Stop.push({
-                hooks: [{
-                        type: 'command',
-                        command: `npx -y skillvault@${VERSION} --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 {
@@ -1874,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.2",
+  "version": "0.9.3",
   "description": "SkillVault — secure skill distribution for Claude Code",
   "type": "module",
   "bin": {