@skillsmith/core 0.4.13 → 0.4.15

package/dist/tests/routing/LanguageRouter.test.js

@@ -0,0 +1,150 @@
+/**
+ * SMI-2756: Wave 3 — LanguageRouter tests
+ *
+ * Tests the LanguageRouter dispatch layer: adapter registration,
+ * extension-based lookup, tryGetAdapter, canHandle, getLanguage,
+ * getSupportedLanguages, framework rules aggregation, and dispose.
+ */
+import { describe, it, expect } from 'vitest';
+import { LanguageRouter } from '../../src/analysis/router.js';
+describe('LanguageRouter', () => {
+    // -------------------------------------------------------------------------
+    // createWithAllAdapters factory
+    // -------------------------------------------------------------------------
+    describe('createWithAllAdapters', () => {
+        it('creates a router with multiple adapters pre-registered', () => {
+            const router = LanguageRouter.createWithAllAdapters();
+            expect(router.adapterCount).toBeGreaterThan(0);
+        });
+        it('creates a router with only specified languages', () => {
+            const router = LanguageRouter.createWithAllAdapters({ languages: ['typescript'] });
+            expect(router.getSupportedLanguages()).toContain('typescript');
+            // Should have fewer adapters than the full set
+            const fullRouter = LanguageRouter.createWithAllAdapters();
+            expect(router.adapterCount).toBeLessThanOrEqual(fullRouter.adapterCount);
+        });
+    });
+    // -------------------------------------------------------------------------
+    // registerAdapter / unregisterAdapter
+    // -------------------------------------------------------------------------
+    describe('registerAdapter / unregisterAdapter', () => {
+        it('registers an adapter and makes its extension routable', () => {
+            const router = LanguageRouter.createWithAllAdapters({ languages: ['typescript'] });
+            expect(router.canHandle('main.ts')).toBe(true);
+        });
+        it('unregisterAdapter removes extension mappings', () => {
+            const router = LanguageRouter.createWithAllAdapters({ languages: ['typescript'] });
+            expect(router.canHandle('main.ts')).toBe(true);
+            const removed = router.unregisterAdapter('typescript');
+            expect(removed).toBe(true);
+            expect(router.canHandle('main.ts')).toBe(false);
+        });
+        it('unregisterAdapter returns false for unknown language', () => {
+            const router = new LanguageRouter();
+            const result = router.unregisterAdapter('typescript');
+            expect(result).toBe(false);
+        });
+        it('re-registering an adapter replaces the old one', () => {
+            const router = LanguageRouter.createWithAllAdapters({ languages: ['typescript'] });
+            const before = router.adapterCount;
+            // Re-register the same language
+            const tsAdapter = router.getAdapterByLanguage('typescript');
+            router.registerAdapter(tsAdapter);
+            // Count should not grow
+            expect(router.adapterCount).toBe(before);
+        });
+    });
+    // -------------------------------------------------------------------------
+    // getAdapter / tryGetAdapter / canHandle / getLanguage
+    // -------------------------------------------------------------------------
+    describe('getAdapter', () => {
+        it('returns adapter for a known extension', () => {
+            const router = LanguageRouter.createWithAllAdapters({ languages: ['typescript'] });
+            const adapter = router.tryGetAdapter('main.ts');
+            expect(adapter).not.toBeNull();
+            expect(adapter?.language).toBe('typescript');
+        });
+        it('throws for unknown extension', () => {
+            const router = new LanguageRouter();
+            expect(() => router.getAdapter('unknown.xyz')).toThrow(/No adapter registered/);
+        });
+        it('tryGetAdapter returns null for unknown extension', () => {
+            const router = new LanguageRouter();
+            expect(router.tryGetAdapter('unknown.xyz')).toBeNull();
+        });
+    });
+    describe('canHandle', () => {
+        it('returns true for registered extension', () => {
+            const router = LanguageRouter.createWithAllAdapters({ languages: ['python'] });
+            expect(router.canHandle('app.py')).toBe(true);
+        });
+        it('returns false for unregistered extension', () => {
+            const router = new LanguageRouter();
+            expect(router.canHandle('app.cobol')).toBe(false);
+        });
+    });
+    describe('getLanguage', () => {
+        it('returns language for a known file path', () => {
+            const router = LanguageRouter.createWithAllAdapters({ languages: ['python'] });
+            expect(router.getLanguage('script.py')).toBe('python');
+        });
+        it('returns null for unsupported extension', () => {
+            const router = new LanguageRouter();
+            expect(router.getLanguage('file.cobol')).toBeNull();
+        });
+    });
+    // -------------------------------------------------------------------------
+    // getSupportedLanguages / getSupportedExtensions
+    // -------------------------------------------------------------------------
+    describe('getSupportedLanguages / getSupportedExtensions', () => {
+        it('getSupportedLanguages returns registered language keys', () => {
+            const router = LanguageRouter.createWithAllAdapters({ languages: ['typescript', 'python'] });
+            const langs = router.getSupportedLanguages();
+            expect(langs).toContain('typescript');
+            expect(langs).toContain('python');
+        });
+        it('getSupportedExtensions includes dot-prefixed extensions', () => {
+            const router = LanguageRouter.createWithAllAdapters({ languages: ['typescript'] });
+            const exts = router.getSupportedExtensions();
+            expect(exts.some((e) => e.startsWith('.'))).toBe(true);
+        });
+    });
+    // -------------------------------------------------------------------------
+    // getAllFrameworkRules / getFrameworkRules
+    // -------------------------------------------------------------------------
+    describe('framework rules', () => {
+        it('getAllFrameworkRules returns an array', () => {
+            const router = LanguageRouter.createWithAllAdapters();
+            const rules = router.getAllFrameworkRules();
+            expect(Array.isArray(rules)).toBe(true);
+        });
+        it('getFrameworkRules returns empty array for unregistered language', () => {
+            const router = new LanguageRouter();
+            const rules = router.getFrameworkRules('typescript');
+            expect(rules).toEqual([]);
+        });
+    });
+    // -------------------------------------------------------------------------
+    // dispose
+    // -------------------------------------------------------------------------
+    describe('dispose', () => {
+        it('clears adapter registry after dispose', () => {
+            const router = LanguageRouter.createWithAllAdapters();
+            expect(router.adapterCount).toBeGreaterThan(0);
+            router.dispose();
+            expect(router.adapterCount).toBe(0);
+        });
+    });
+    // -------------------------------------------------------------------------
+    // detectLanguageFromContent (disabled)
+    // -------------------------------------------------------------------------
+    describe('detectLanguageFromContent', () => {
+        it('returns language:null and method:none when detection is disabled', () => {
+            const router = new LanguageRouter({ enableLanguageDetection: false });
+            const result = router.detectLanguageFromContent('#!/usr/bin/env python3\nprint("hello")');
+            expect(result.language).toBeNull();
+            expect(result.method).toBe('none');
+        });
+    });
+});
+//# sourceMappingURL=LanguageRouter.test.js.map

package/dist/tests/routing/LanguageRouter.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"LanguageRouter.test.js","sourceRoot":"","sources":["../../../tests/routing/LanguageRouter.test.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAA;AAC7C,OAAO,EAAE,cAAc,EAAE,MAAM,8BAA8B,CAAA;AAE7D,QAAQ,CAAC,gBAAgB,EAAE,GAAG,EAAE;IAC9B,4EAA4E;IAC5E,gCAAgC;IAChC,4EAA4E;IAE5E,QAAQ,CAAC,uBAAuB,EAAE,GAAG,EAAE;QACrC,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;YAChE,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,EAAE,CAAA;YACrD,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAA;QAChD,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;YACxD,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,CAAC,EAAE,SAAS,EAAE,CAAC,YAAY,CAAC,EAAE,CAAC,CAAA;YAClF,MAAM,CAAC,MAAM,CAAC,qBAAqB,EAAE,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAA;YAC9D,+CAA+C;YAC/C,MAAM,UAAU,GAAG,cAAc,CAAC,qBAAqB,EAAE,CAAA;YACzD,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,mBAAmB,CAAC,UAAU,CAAC,YAAY,CAAC,CAAA;QAC1E,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,4EAA4E;IAC5E,sCAAsC;IACtC,4EAA4E;IAE5E,QAAQ,CAAC,qCAAqC,EAAE,GAAG,EAAE;QACnD,EAAE,CAAC,uDAAuD,EAAE,GAAG,EAAE;YAC/D,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,CAAC,EAAE,SAAS,EAAE,CAAC,YAAY,CAAC,EAAE,CAAC,CAAA;YAClF,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAChD,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,8CAA8C,EAAE,GAAG,EAAE;YACtD,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,CAAC,EAAE,SAAS,EAAE,CAAC,YAAY,CAAC,EAAE,CAAC,CAAA;YAClF,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;YAE9C,MAAM,OAAO,GAAG,MAAM,CAAC,iBAAiB,CAAC,YAAY,CAAC,CAAA;YAEtD,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;YAC1B,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACjD,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,sDAAsD,EAAE,GAAG,EAAE;YAC9D,MAAM,MAAM,GAAG,IAAI,cAAc,EAAE,CAAA;YACnC,MAAM,MAAM,GAAG,MAAM,CAAC,iBAAiB,CAAC,YAAY,CAAC,CAAA;YACrD,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAC5B,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;YACxD,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,CAAC,EAAE,SAAS,EAAE,CAAC,YAAY,CAAC,EAAE,CAAC,CAAA;YAClF,MAAM,MAAM,GAAG,MAAM,CAAC,YAAY,CAAA;YAElC,gCAAgC;YAChC,MAAM,SAAS,GAAG,MAAM,CAAC,oBAAoB,CAAC,YAAY,CAAE,CAAA;YAC5D,MAAM,CAAC,eAAe,CAAC,SAAS,CAAC,CAAA;YAEjC,wBAAwB;YACxB,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;QAC1C,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,4EAA4E;IAC5E,uDAAuD;IACvD,4EAA4E;IAE5E,QAAQ,CAAC,YAAY,EAAE,GAAG,EAAE;QAC1B,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,CAAC,EAAE,SAAS,EAAE,CAAC,YAAY,CAAC,EAAE,CAAC,CAAA;YAClF,MAAM,OAAO,GAAG,MAAM,CAAC,aAAa,CAAC,SAAS,CAAC,CAAA;YAC/C,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,QAAQ,EAAE,CAAA;YAC9B,MAAM,CAAC,OAAO,EAAE,QAAQ,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAA;QAC9C,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,8BAA8B,EAAE,GAAG,EAAE;YACtC,MAAM,MAAM,GAAG,IAAI,cAAc,EAAE,CAAA;YACnC,MAAM,CAAC,GAAG,EAAE,CAAC,MAAM,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC,CAAC,OAAO,CAAC,uBAAuB,CAAC,CAAA;QACjF,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;YAC1D,MAAM,MAAM,GAAG,IAAI,cAAc,EAAE,CAAA;YACnC,MAAM,CAAC,MAAM,CAAC,aAAa,CAAC,aAAa,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAA;QACxD,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,WAAW,EAAE,GAAG,EAAE;QACzB,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,CAAC,EAAE,SAAS,EAAE,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAA;YAC9E,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAC/C,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;YAClD,MAAM,MAAM,GAAG,IAAI,cAAc,EAAE,CAAA;YACnC,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,WAAW,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACnD,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;QAC3B,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;YAChD,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,CAAC,EAAE,SAAS,EAAE,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAA;YAC9E,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,WAAW,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACxD,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,wCAAwC,EAAE,GAAG,EAAE;YAChD,MAAM,MAAM,GAAG,IAAI,cAAc,EAAE,CAAA;YACnC,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,YAAY,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAA;QACrD,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,4EAA4E;IAC5E,iDAAiD;IACjD,4EAA4E;IAE5E,QAAQ,CAAC,gDAAgD,EAAE,GAAG,EAAE;QAC9D,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;YAChE,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,CAAC,EAAE,SAAS,EAAE,CAAC,YAAY,EAAE,QAAQ,CAAC,EAAE,CAAC,CAAA;YAC5F,MAAM,KAAK,GAAG,MAAM,CAAC,qBAAqB,EAAE,CAAA;YAC5C,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAA;YACrC,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAA;QACnC,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,yDAAyD,EAAE,GAAG,EAAE;YACjE,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,CAAC,EAAE,SAAS,EAAE,CAAC,YAAY,CAAC,EAAE,CAAC,CAAA;YAClF,MAAM,IAAI,GAAG,MAAM,CAAC,sBAAsB,EAAE,CAAA;YAC5C,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACxD,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,4EAA4E;IAC5E,2CAA2C;IAC3C,4EAA4E;IAE5E,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;QAC/B,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,EAAE,CAAA;YACrD,MAAM,KAAK,GAAG,MAAM,CAAC,oBAAoB,EAAE,CAAA;YAC3C,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACzC,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;YACzE,MAAM,MAAM,GAAG,IAAI,cAAc,EAAE,CAAA;YACnC,MAAM,KAAK,GAAG,MAAM,CAAC,iBAAiB,CAAC,YAAY,CAAC,CAAA;YACpD,MAAM,CAAC,KAAK,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAA;QAC3B,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,4EAA4E;IAC5E,UAAU;IACV,4EAA4E;IAE5E,QAAQ,CAAC,SAAS,EAAE,GAAG,EAAE;QACvB,EAAE,CAAC,uCAAuC,EAAE,GAAG,EAAE;YAC/C,MAAM,MAAM,GAAG,cAAc,CAAC,qBAAqB,EAAE,CAAA;YACrD,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAA;YAE9C,MAAM,CAAC,OAAO,EAAE,CAAA;YAEhB,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACrC,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,4EAA4E;IAC5E,uCAAuC;IACvC,4EAA4E;IAE5E,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACzC,EAAE,CAAC,kEAAkE,EAAE,GAAG,EAAE;YAC1E,MAAM,MAAM,GAAG,IAAI,cAAc,CAAC,EAAE,uBAAuB,EAAE,KAAK,EAAE,CAAC,CAAA;YACrE,MAAM,MAAM,GAAG,MAAM,CAAC,yBAAyB,CAAC,wCAAwC,CAAC,CAAA;YACzF,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,QAAQ,EAAE,CAAA;YAClC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;QACpC,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/tests/session/SessionManager.memory.test.d.ts

@@ -0,0 +1,11 @@
+/**
+ * SMI-2754: SessionManager Memory Operation Tests
+ *
+ * Tests for storeMemoryEntry, retrieveMemoryEntry, deleteMemoryEntry,
+ * runPreTaskHook, and runPostTaskHook.
+ *
+ * Since USE_V3_API is evaluated at module load time, we mock the helpers
+ * module to control getClaudeFlowMemory / getClaudeFlowMcp / USE_V3_API.
+ */
+export {};
+//# sourceMappingURL=SessionManager.memory.test.d.ts.map

package/dist/tests/session/SessionManager.memory.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"SessionManager.memory.test.d.ts","sourceRoot":"","sources":["../../../tests/session/SessionManager.memory.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG"}

package/dist/tests/session/SessionManager.memory.test.js

@@ -0,0 +1,219 @@
+/**
+ * SMI-2754: SessionManager Memory Operation Tests
+ *
+ * Tests for storeMemoryEntry, retrieveMemoryEntry, deleteMemoryEntry,
+ * runPreTaskHook, and runPostTaskHook.
+ *
+ * Since USE_V3_API is evaluated at module load time, we mock the helpers
+ * module to control getClaudeFlowMemory / getClaudeFlowMcp / USE_V3_API.
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+// ============================================================================
+// We mock the helpers module to control USE_V3_API and module loading
+// ============================================================================
+vi.mock('../../src/session/SessionManager.helpers.js', () => {
+    return {
+        validateMemoryKey: (key) => /^[a-zA-Z0-9/_-]+$/.test(key) && key.length <= 256,
+        MEMORY_KEYS: { CURRENT: 'session/current' },
+        MEMORY_NAMESPACE: 'skillsmith-sessions',
+        USE_V3_API: false, // Start with spawn path; override per test
+        getClaudeFlowMemory: vi.fn(),
+        getClaudeFlowMcp: vi.fn(),
+    };
+});
+import { storeMemoryEntry, retrieveMemoryEntry, deleteMemoryEntry, runPreTaskHook, runPostTaskHook, } from '../../src/session/SessionManager.memory.js';
+import * as helpers from '../../src/session/SessionManager.helpers.js';
+// ============================================================================
+// Helpers
+// ============================================================================
+function makeSpawnExecutor(overrides = {}) {
+    return {
+        execute: vi.fn().mockResolvedValue({ stdout: '', stderr: '' }),
+        spawn: vi.fn().mockResolvedValue({ stdout: overrides.stdout ?? '', stderr: '' }),
+    };
+}
+function enableV3(enabled) {
+    // Override the USE_V3_API value in the mocked module
+    // @ts-expect-error - overriding readonly for testing
+    helpers.USE_V3_API = enabled;
+}
+beforeEach(() => {
+    enableV3(false);
+    vi.mocked(helpers.getClaudeFlowMemory).mockReset();
+    vi.mocked(helpers.getClaudeFlowMcp).mockReset();
+});
+afterEach(() => {
+    vi.clearAllMocks();
+});
+// ============================================================================
+// storeMemoryEntry
+// ============================================================================
+describe('storeMemoryEntry', () => {
+    it('returns { success: false, error: "Invalid memory key" } for an invalid key', async () => {
+        const executor = makeSpawnExecutor();
+        const result = await storeMemoryEntry('bad key!', 'value', executor);
+        expect(result).toEqual({ success: false, error: 'Invalid memory key' });
+        expect(executor.spawn).not.toHaveBeenCalled();
+    });
+    it('uses V3 storeEntry when USE_V3_API is true and module is available', async () => {
+        enableV3(true);
+        const storeEntry = vi.fn().mockResolvedValue({ success: true });
+        vi.mocked(helpers.getClaudeFlowMemory).mockResolvedValue({ storeEntry });
+        const executor = makeSpawnExecutor();
+        const result = await storeMemoryEntry('session/test', 'my-value', executor);
+        expect(result).toEqual({ success: true });
+        expect(storeEntry).toHaveBeenCalledWith({
+            key: 'session/test',
+            value: 'my-value',
+            namespace: 'skillsmith-sessions',
+        });
+        expect(executor.spawn).not.toHaveBeenCalled();
+    });
+    it('falls back to spawn when V3 storeEntry returns success:false', async () => {
+        enableV3(true);
+        const storeEntry = vi.fn().mockResolvedValue({ success: false, error: 'V3 error' });
+        vi.mocked(helpers.getClaudeFlowMemory).mockResolvedValue({ storeEntry });
+        const executor = makeSpawnExecutor();
+        const result = await storeMemoryEntry('session/fallback', 'value', executor);
+        expect(result).toEqual({ success: true });
+        expect(executor.spawn).toHaveBeenCalled();
+    });
+    it('uses spawn directly when USE_V3_API is false', async () => {
+        enableV3(false);
+        const executor = makeSpawnExecutor();
+        const result = await storeMemoryEntry('session/key', 'val', executor);
+        expect(result).toEqual({ success: true });
+        expect(executor.spawn).toHaveBeenCalledWith('npx', expect.arrayContaining(['claude-flow', 'memory', 'store', '--key', 'session/key']));
+        expect(helpers.getClaudeFlowMemory).not.toHaveBeenCalled();
+    });
+});
+// ============================================================================
+// retrieveMemoryEntry
+// ============================================================================
+describe('retrieveMemoryEntry', () => {
+    it('returns { success: false, error: "Invalid memory key" } for an invalid key', async () => {
+        const executor = makeSpawnExecutor();
+        const result = await retrieveMemoryEntry('bad key!', executor);
+        expect(result).toEqual({ success: false, error: 'Invalid memory key' });
+    });
+    it('returns data from V3 getEntry when found', async () => {
+        enableV3(true);
+        const getEntry = vi.fn().mockResolvedValue({
+            success: true,
+            found: true,
+            entry: { content: 'stored-value' },
+        });
+        vi.mocked(helpers.getClaudeFlowMemory).mockResolvedValue({ getEntry });
+        const executor = makeSpawnExecutor();
+        const result = await retrieveMemoryEntry('session/found', executor);
+        expect(result).toEqual({ success: true, data: 'stored-value' });
+        expect(executor.spawn).not.toHaveBeenCalled();
+    });
+    it('returns not-found error from V3 when key does not exist', async () => {
+        enableV3(true);
+        const getEntry = vi.fn().mockResolvedValue({
+            success: true,
+            found: false,
+        });
+        vi.mocked(helpers.getClaudeFlowMemory).mockResolvedValue({ getEntry });
+        const executor = makeSpawnExecutor();
+        const result = await retrieveMemoryEntry('session/missing', executor);
+        expect(result).toEqual({ success: false, error: 'Key not found' });
+        expect(executor.spawn).not.toHaveBeenCalled();
+    });
+    it('falls back to spawn when V3 getEntry throws an exception', async () => {
+        enableV3(true);
+        vi.mocked(helpers.getClaudeFlowMemory).mockRejectedValue(new Error('module load error'));
+        const executor = makeSpawnExecutor({ stdout: 'spawned-data' });
+        const result = await retrieveMemoryEntry('session/key', executor);
+        expect(result).toEqual({ success: true, data: 'spawned-data' });
+        expect(executor.spawn).toHaveBeenCalled();
+    });
+    it('uses spawn directly when USE_V3_API is false', async () => {
+        enableV3(false);
+        const executor = makeSpawnExecutor({ stdout: 'value-from-spawn\n' });
+        const result = await retrieveMemoryEntry('session/key', executor);
+        expect(result).toEqual({ success: true, data: 'value-from-spawn' });
+        expect(helpers.getClaudeFlowMemory).not.toHaveBeenCalled();
+    });
+});
+// ============================================================================
+// deleteMemoryEntry
+// ============================================================================
+describe('deleteMemoryEntry', () => {
+    it('returns { success: false, error: "Invalid memory key" } for an invalid key', async () => {
+        const executor = makeSpawnExecutor();
+        const result = await deleteMemoryEntry('bad key!', executor);
+        expect(result).toEqual({ success: false, error: 'Invalid memory key' });
+    });
+    it('returns success when V3 callMCPTool succeeds', async () => {
+        enableV3(true);
+        const callMCPTool = vi.fn().mockResolvedValue({ success: true, deleted: true });
+        vi.mocked(helpers.getClaudeFlowMcp).mockResolvedValue({ callMCPTool });
+        const executor = makeSpawnExecutor();
+        const result = await deleteMemoryEntry('session/to-delete', executor);
+        expect(result).toEqual({ success: true });
+        expect(callMCPTool).toHaveBeenCalledWith('memory/delete', { key: 'session/to-delete' });
+    });
+    it('falls back to spawn (and returns success) when V3 callMCPTool fails', async () => {
+        enableV3(true);
+        const callMCPTool = vi.fn().mockResolvedValue({ success: false });
+        vi.mocked(helpers.getClaudeFlowMcp).mockResolvedValue({ callMCPTool });
+        const executor = makeSpawnExecutor();
+        const result = await deleteMemoryEntry('session/key', executor);
+        expect(result).toEqual({ success: true });
+        expect(executor.spawn).toHaveBeenCalled();
+    });
+});
+// ============================================================================
+// runPreTaskHook
+// ============================================================================
+describe('runPreTaskHook', () => {
+    it('calls V3 MCP pre-task hook when USE_V3_API is true', async () => {
+        enableV3(true);
+        const callMCPTool = vi.fn().mockResolvedValue({});
+        vi.mocked(helpers.getClaudeFlowMcp).mockResolvedValue({ callMCPTool });
+        const executor = makeSpawnExecutor();
+        await runPreTaskHook('test description', executor);
+        expect(callMCPTool).toHaveBeenCalledWith('hooks/pre-task', expect.objectContaining({
+            description: 'test description',
+        }));
+        expect(executor.spawn).not.toHaveBeenCalled();
+    });
+    it('uses spawn when USE_V3_API is false', async () => {
+        enableV3(false);
+        const executor = makeSpawnExecutor();
+        await runPreTaskHook('test description', executor);
+        expect(executor.spawn).toHaveBeenCalledWith('npx', expect.arrayContaining(['pre-task', '--description', 'test description']));
+    });
+    it('swallows errors and does not throw', async () => {
+        enableV3(false);
+        const executor = {
+            execute: vi.fn().mockResolvedValue({ stdout: '', stderr: '' }),
+            spawn: vi.fn().mockRejectedValue(new Error('hook failed')),
+        };
+        // Should not throw
+        await expect(runPreTaskHook('test', executor)).resolves.toBeUndefined();
+    });
+});
+// ============================================================================
+// runPostTaskHook
+// ============================================================================
+describe('runPostTaskHook', () => {
+    it('calls V3 MCP post-task hook when USE_V3_API is true', async () => {
+        enableV3(true);
+        const callMCPTool = vi.fn().mockResolvedValue({});
+        vi.mocked(helpers.getClaudeFlowMcp).mockResolvedValue({ callMCPTool });
+        const executor = makeSpawnExecutor();
+        await runPostTaskHook('task-123', executor);
+        expect(callMCPTool).toHaveBeenCalledWith('hooks/post-task', { taskId: 'task-123' });
+        expect(executor.spawn).not.toHaveBeenCalled();
+    });
+    it('uses spawn when USE_V3_API is false', async () => {
+        enableV3(false);
+        const executor = makeSpawnExecutor();
+        await runPostTaskHook('task-456', executor);
+        expect(executor.spawn).toHaveBeenCalledWith('npx', expect.arrayContaining(['post-task', '--task-id', 'task-456']));
+    });
+});
+//# sourceMappingURL=SessionManager.memory.test.js.map

package/dist/tests/session/SessionManager.memory.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"SessionManager.memory.test.js","sourceRoot":"","sources":["../../../tests/session/SessionManager.memory.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAA;AAExE,+EAA+E;AAC/E,sEAAsE;AACtE,+EAA+E;AAE/E,EAAE,CAAC,IAAI,CAAC,6CAA6C,EAAE,GAAG,EAAE;IAC1D,OAAO;QACL,iBAAiB,EAAE,CAAC,GAAW,EAAE,EAAE,CAAC,mBAAmB,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,GAAG,CAAC,MAAM,IAAI,GAAG;QACtF,WAAW,EAAE,EAAE,OAAO,EAAE,iBAAiB,EAAE;QAC3C,gBAAgB,EAAE,qBAAqB;QACvC,UAAU,EAAE,KAAK,EAAE,2CAA2C;QAC9D,mBAAmB,EAAE,EAAE,CAAC,EAAE,EAAE;QAC5B,gBAAgB,EAAE,EAAE,CAAC,EAAE,EAAE;KAC1B,CAAA;AACH,CAAC,CAAC,CAAA;AAEF,OAAO,EACL,gBAAgB,EAChB,mBAAmB,EACnB,iBAAiB,EACjB,cAAc,EACd,eAAe,GAChB,MAAM,4CAA4C,CAAA;AACnD,OAAO,KAAK,OAAO,MAAM,6CAA6C,CAAA;AAGtE,+EAA+E;AAC/E,UAAU;AACV,+EAA+E;AAE/E,SAAS,iBAAiB,CAAC,YAAyC,EAAE;IACpE,OAAO;QACL,OAAO,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;QAC9D,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,MAAM,EAAE,SAAS,CAAC,MAAM,IAAI,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;KACjF,CAAA;AACH,CAAC;AAED,SAAS,QAAQ,CAAC,OAAgB;IAChC,qDAAqD;IACrD,qDAAqD;IACrD,OAAO,CAAC,UAAU,GAAG,OAAO,CAAA;AAC9B,CAAC;AAED,UAAU,CAAC,GAAG,EAAE;IACd,QAAQ,CAAC,KAAK,CAAC,CAAA;IACf,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAC,SAAS,EAAE,CAAA;IAClD,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,gBAAgB,CAAC,CAAC,SAAS,EAAE,CAAA;AACjD,CAAC,CAAC,CAAA;AAEF,SAAS,CAAC,GAAG,EAAE;IACb,EAAE,CAAC,aAAa,EAAE,CAAA;AACpB,CAAC,CAAC,CAAA;AAEF,+EAA+E;AAC/E,mBAAmB;AACnB,+EAA+E;AAE/E,QAAQ,CAAC,kBAAkB,EAAE,GAAG,EAAE;IAChC,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,MAAM,GAAG,MAAM,gBAAgB,CAAC,UAAU,EAAE,OAAO,EAAE,QAAQ,CAAC,CAAA;QACpE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,oBAAoB,EAAE,CAAC,CAAA;QACvE,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,oEAAoE,EAAE,KAAK,IAAI,EAAE;QAClF,QAAQ,CAAC,IAAI,CAAC,CAAA;QACd,MAAM,UAAU,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAA;QAC/D,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAC,iBAAiB,CAAC,EAAE,UAAU,EAAE,CAAC,CAAA;QAExE,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,MAAM,GAAG,MAAM,gBAAgB,CAAC,cAAc,EAAE,UAAU,EAAE,QAAQ,CAAC,CAAA;QAE3E,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAA;QACzC,MAAM,CAAC,UAAU,CAAC,CAAC,oBAAoB,CAAC;YACtC,GAAG,EAAE,cAAc;YACnB,KAAK,EAAE,UAAU;YACjB,SAAS,EAAE,qBAAqB;SACjC,CAAC,CAAA;QACF,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,QAAQ,CAAC,IAAI,CAAC,CAAA;QACd,MAAM,UAAU,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,UAAU,EAAE,CAAC,CAAA;QACnF,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAC,iBAAiB,CAAC,EAAE,UAAU,EAAE,CAAC,CAAA;QAExE,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,MAAM,GAAG,MAAM,gBAAgB,CAAC,kBAAkB,EAAE,OAAO,EAAE,QAAQ,CAAC,CAAA;QAE5E,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAA;QACzC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,gBAAgB,EAAE,CAAA;IAC3C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,QAAQ,CAAC,KAAK,CAAC,CAAA;QACf,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,MAAM,GAAG,MAAM,gBAAgB,CAAC,aAAa,EAAE,KAAK,EAAE,QAAQ,CAAC,CAAA;QAErE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAA;QACzC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,oBAAoB,CACzC,KAAK,EACL,MAAM,CAAC,eAAe,CAAC,CAAC,aAAa,EAAE,QAAQ,EAAE,OAAO,EAAE,OAAO,EAAE,aAAa,CAAC,CAAC,CACnF,CAAA;QACD,MAAM,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;IAC5D,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,+EAA+E;AAC/E,sBAAsB;AACtB,+EAA+E;AAE/E,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,UAAU,EAAE,QAAQ,CAAC,CAAA;QAC9D,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,oBAAoB,EAAE,CAAC,CAAA;IACzE,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0CAA0C,EAAE,KAAK,IAAI,EAAE;QACxD,QAAQ,CAAC,IAAI,CAAC,CAAA;QACd,MAAM,QAAQ,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC;YACzC,OAAO,EAAE,IAAI;YACb,KAAK,EAAE,IAAI;YACX,KAAK,EAAE,EAAE,OAAO,EAAE,cAAc,EAAE;SACnC,CAAC,CAAA;QACF,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAC,iBAAiB,CAAC,EAAE,QAAQ,EAAE,CAAC,CAAA;QAEtE,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,eAAe,EAAE,QAAQ,CAAC,CAAA;QAEnE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC,CAAA;QAC/D,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;QACvE,QAAQ,CAAC,IAAI,CAAC,CAAA;QACd,MAAM,QAAQ,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC;YACzC,OAAO,EAAE,IAAI;YACb,KAAK,EAAE,KAAK;SACb,CAAC,CAAA;QACF,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAC,iBAAiB,CAAC,EAAE,QAAQ,EAAE,CAAC,CAAA;QAEtE,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,iBAAiB,EAAE,QAAQ,CAAC,CAAA;QAErE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,eAAe,EAAE,CAAC,CAAA;QAClE,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0DAA0D,EAAE,KAAK,IAAI,EAAE;QACxE,QAAQ,CAAC,IAAI,CAAC,CAAA;QACd,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAC,iBAAiB,CAAC,IAAI,KAAK,CAAC,mBAAmB,CAAC,CAAC,CAAA;QAExF,MAAM,QAAQ,GAAG,iBAAiB,CAAC,EAAE,MAAM,EAAE,cAAc,EAAE,CAAC,CAAA;QAC9D,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,aAAa,EAAE,QAAQ,CAAC,CAAA;QAEjE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC,CAAA;QAC/D,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,gBAAgB,EAAE,CAAA;IAC3C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,QAAQ,CAAC,KAAK,CAAC,CAAA;QACf,MAAM,QAAQ,GAAG,iBAAiB,CAAC,EAAE,MAAM,EAAE,oBAAoB,EAAE,CAAC,CAAA;QACpE,MAAM,MAAM,GAAG,MAAM,mBAAmB,CAAC,aAAa,EAAE,QAAQ,CAAC,CAAA;QAEjE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,IAAI,EAAE,kBAAkB,EAAE,CAAC,CAAA;QACnE,MAAM,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;IAC5D,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,+EAA+E;AAC/E,oBAAoB;AACpB,+EAA+E;AAE/E,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;IACjC,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,MAAM,GAAG,MAAM,iBAAiB,CAAC,UAAU,EAAE,QAAQ,CAAC,CAAA;QAC5D,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,oBAAoB,EAAE,CAAC,CAAA;IACzE,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,QAAQ,CAAC,IAAI,CAAC,CAAA;QACd,MAAM,WAAW,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAA;QAC/E,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,gBAAgB,CAAC,CAAC,iBAAiB,CAAC,EAAE,WAAW,EAAE,CAAC,CAAA;QAEtE,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,MAAM,GAAG,MAAM,iBAAiB,CAAC,mBAAmB,EAAE,QAAQ,CAAC,CAAA;QAErE,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAA;QACzC,MAAM,CAAC,WAAW,CAAC,CAAC,oBAAoB,CAAC,eAAe,EAAE,EAAE,GAAG,EAAE,mBAAmB,EAAE,CAAC,CAAA;IACzF,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qEAAqE,EAAE,KAAK,IAAI,EAAE;QACnF,QAAQ,CAAC,IAAI,CAAC,CAAA;QACd,MAAM,WAAW,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC,CAAA;QACjE,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,gBAAgB,CAAC,CAAC,iBAAiB,CAAC,EAAE,WAAW,EAAE,CAAC,CAAA;QAEtE,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,MAAM,GAAG,MAAM,iBAAiB,CAAC,aAAa,EAAE,QAAQ,CAAC,CAAA;QAE/D,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC,CAAA;QACzC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,gBAAgB,EAAE,CAAA;IAC3C,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,+EAA+E;AAC/E,iBAAiB;AACjB,+EAA+E;AAE/E,QAAQ,CAAC,gBAAgB,EAAE,GAAG,EAAE;IAC9B,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;QAClE,QAAQ,CAAC,IAAI,CAAC,CAAA;QACd,MAAM,WAAW,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC,CAAA;QACjD,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,gBAAgB,CAAC,CAAC,iBAAiB,CAAC,EAAE,WAAW,EAAE,CAAC,CAAA;QAEtE,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,cAAc,CAAC,kBAAkB,EAAE,QAAQ,CAAC,CAAA;QAElD,MAAM,CAAC,WAAW,CAAC,CAAC,oBAAoB,CACtC,gBAAgB,EAChB,MAAM,CAAC,gBAAgB,CAAC;YACtB,WAAW,EAAE,kBAAkB;SAChC,CAAC,CACH,CAAA;QACD,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qCAAqC,EAAE,KAAK,IAAI,EAAE;QACnD,QAAQ,CAAC,KAAK,CAAC,CAAA;QACf,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,cAAc,CAAC,kBAAkB,EAAE,QAAQ,CAAC,CAAA;QAElD,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,oBAAoB,CACzC,KAAK,EACL,MAAM,CAAC,eAAe,CAAC,CAAC,UAAU,EAAE,eAAe,EAAE,kBAAkB,CAAC,CAAC,CAC1E,CAAA;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,oCAAoC,EAAE,KAAK,IAAI,EAAE;QAClD,QAAQ,CAAC,KAAK,CAAC,CAAA;QACf,MAAM,QAAQ,GAAoB;YAChC,OAAO,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;YAC9D,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,IAAI,KAAK,CAAC,aAAa,CAAC,CAAC;SAC3D,CAAA;QACD,mBAAmB;QACnB,MAAM,MAAM,CAAC,cAAc,CAAC,MAAM,EAAE,QAAQ,CAAC,CAAC,CAAC,QAAQ,CAAC,aAAa,EAAE,CAAA;IACzE,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,+EAA+E;AAC/E,kBAAkB;AAClB,+EAA+E;AAE/E,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAC/B,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,QAAQ,CAAC,IAAI,CAAC,CAAA;QACd,MAAM,WAAW,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,CAAC,CAAA;QACjD,EAAE,CAAC,MAAM,CAAC,OAAO,CAAC,gBAAgB,CAAC,CAAC,iBAAiB,CAAC,EAAE,WAAW,EAAE,CAAC,CAAA;QAEtE,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,eAAe,CAAC,UAAU,EAAE,QAAQ,CAAC,CAAA;QAE3C,MAAM,CAAC,WAAW,CAAC,CAAC,oBAAoB,CAAC,iBAAiB,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAA;QACnF,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qCAAqC,EAAE,KAAK,IAAI,EAAE;QACnD,QAAQ,CAAC,KAAK,CAAC,CAAA;QACf,MAAM,QAAQ,GAAG,iBAAiB,EAAE,CAAA;QACpC,MAAM,eAAe,CAAC,UAAU,EAAE,QAAQ,CAAC,CAAA;QAE3C,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,oBAAoB,CACzC,KAAK,EACL,MAAM,CAAC,eAAe,CAAC,CAAC,WAAW,EAAE,WAAW,EAAE,UAAU,CAAC,CAAC,CAC/D,CAAA;IACH,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/tests/sync/SyncEngine.test.js

@@ -9,6 +9,20 @@ import { SyncConfigRepository } from '../../src/repositories/SyncConfigRepositor
 import { SyncHistoryRepository } from '../../src/repositories/SyncHistoryRepository.js';
 import { SkillRepository } from '../../src/repositories/SkillRepository.js';
 import { SyncEngine } from '../../src/sync/SyncEngine.js';
+/**
+ * Create a mock SkillVersionRepository for testing.
+ * All methods return resolved promises so SyncEngine can call recordVersion
+ * after each upsert without errors.
+ */
+function createMockSkillVersionRepo() {
+    return {
+        recordVersion: vi.fn().mockResolvedValue(undefined),
+        pruneVersions: vi.fn().mockResolvedValue(undefined),
+        getLatestVersion: vi.fn().mockResolvedValue(null),
+        getVersionHistory: vi.fn().mockResolvedValue([]),
+        getVersionByHash: vi.fn().mockResolvedValue(null),
+    };
+}
 /**
  * Create a mock skill for testing
  * Note: quality_score must be between 0 and 1 (database constraint)
@@ -71,21 +85,21 @@ describe('SyncEngine', () => {
     describe('constructor', () => {
         it('should create sync engine with all dependencies', () => {
             const apiClient = createMockApiClient();
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             expect(engine).toBeDefined();
         });
     });
     describe('sync - offline and health checks', () => {
         it('should fail when API client is offline', async () => {
             const apiClient = createMockApiClient({ offline: true });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const result = await engine.sync();
             expect(result.success).toBe(false);
             expect(result.errors).toContain('API client is in offline mode. Cannot sync.');
         });
         it('should fail when API health check fails', async () => {
             const apiClient = createMockApiClient({ healthStatus: 'unhealthy' });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const result = await engine.sync();
             expect(result.success).toBe(false);
             expect(result.errors).toContain('API is unhealthy. Try again later.');
@@ -99,7 +113,7 @@ describe('SyncEngine', () => {
                 createMockSkill('test/skill-3'),
             ];
             const apiClient = createMockApiClient({ skills });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const result = await engine.sync();
             expect(result.success).toBe(true);
             expect(result.skillsAdded).toBe(3);
@@ -119,7 +133,7 @@ describe('SyncEngine', () => {
             // API returns updated version with new updated_at
             const skills = [createMockSkill('test/skill-1', new Date(Date.now() + 1000).toISOString())];
             const apiClient = createMockApiClient({ skills });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const result = await engine.sync();
             expect(result.success).toBe(true);
             expect(result.skillsAdded).toBe(0);
@@ -139,7 +153,7 @@ describe('SyncEngine', () => {
             // API returns skill with same timestamp
             const skills = [createMockSkill('test/skill-1', timestamp)];
             const apiClient = createMockApiClient({ skills });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const result = await engine.sync();
             expect(result.success).toBe(true);
             expect(result.skillsUnchanged).toBe(1);
@@ -156,7 +170,7 @@ describe('SyncEngine', () => {
             const oldSkill = createMockSkill('test/old-skill', new Date(Date.now() - 2 * 60 * 60 * 1000).toISOString());
             const newSkill = createMockSkill('test/new-skill', new Date().toISOString());
             const apiClient = createMockApiClient({ skills: [oldSkill, newSkill] });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const result = await engine.sync();
             expect(result.success).toBe(true);
             // Only the new skill should be added (old one filtered out by differential)
@@ -173,7 +187,7 @@ describe('SyncEngine', () => {
                 createMockSkill('test/skill-2', new Date(Date.now() - 2 * 60 * 60 * 1000).toISOString()),
             ];
             const apiClient = createMockApiClient({ skills });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const result = await engine.sync({ force: true });
             expect(result.success).toBe(true);
             expect(result.skillsAdded).toBe(2); // All skills processed because force=true
@@ -183,7 +197,7 @@ describe('SyncEngine', () => {
         it('should not modify database in dry run mode', async () => {
             const skills = [createMockSkill('test/skill-1')];
             const apiClient = createMockApiClient({ skills });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const result = await engine.sync({ dryRun: true });
             expect(result.success).toBe(true);
             expect(result.dryRun).toBe(true);
@@ -200,7 +214,7 @@ describe('SyncEngine', () => {
         it('should call onProgress callback with phases', async () => {
             const skills = [createMockSkill('test/skill-1')];
             const apiClient = createMockApiClient({ skills });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const progressCalls = [];
             const onProgress = vi.fn((progress) => {
                 progressCalls.push(progress.phase);
@@ -219,7 +233,7 @@ describe('SyncEngine', () => {
             // Create 150 skills (more than one page of 100)
             const skills = Array.from({ length: 150 }, (_, i) => createMockSkill(`test/skill-${i}`));
             const apiClient = createMockApiClient({ skills });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const result = await engine.sync({ pageSize: 100 });
             expect(result.success).toBe(true);
             expect(result.skillsAdded).toBe(150);
@@ -234,7 +248,7 @@ describe('SyncEngine', () => {
         it('should record sync history on success', async () => {
             const skills = [createMockSkill('test/skill-1')];
             const apiClient = createMockApiClient({ skills });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             await engine.sync();
             const history = syncHistoryRepo.getHistory();
             expect(history).toHaveLength(1);
@@ -245,7 +259,7 @@ describe('SyncEngine', () => {
             const apiClient = createMockApiClient({
                 throwOnSearch: new Error('Network failure'),
             });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             await engine.sync();
             const history = syncHistoryRepo.getHistory();
             expect(history).toHaveLength(1);
@@ -255,7 +269,7 @@ describe('SyncEngine', () => {
         it('should update sync config on success', async () => {
             const skills = [createMockSkill('test/skill-1')];
             const apiClient = createMockApiClient({ skills });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             await engine.sync();
             const config = syncConfigRepo.getConfig();
             expect(config.lastSyncAt).not.toBeNull();
@@ -264,7 +278,7 @@ describe('SyncEngine', () => {
         });
         it('should set error in config on failure', async () => {
             const apiClient = createMockApiClient({ offline: true });
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             await engine.sync();
             const config = syncConfigRepo.getConfig();
             expect(config.lastSyncError).toBe('API client is in offline mode. Cannot sync.');
@@ -273,7 +287,7 @@ describe('SyncEngine', () => {
     describe('getStatus', () => {
         it('should return sync status summary', () => {
             const apiClient = createMockApiClient();
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const status = engine.getStatus();
             expect(status).toHaveProperty('config');
             expect(status).toHaveProperty('lastRun');
@@ -282,14 +296,14 @@ describe('SyncEngine', () => {
         });
         it('should show sync is due when never synced', () => {
             const apiClient = createMockApiClient();
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             const status = engine.getStatus();
             expect(status.isDue).toBe(true);
             expect(status.lastRun).toBeNull();
         });
         it('should reflect running state', async () => {
             const apiClient = createMockApiClient();
-            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo);
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, createMockSkillVersionRepo());
             // Start a run manually
             syncHistoryRepo.startRun();
             const status = engine.getStatus();