@skillsmith/cli 0.5.12 → 0.6.1

package/dist/tests/config.test.js

@@ -0,0 +1,240 @@
+/**
+ * @fileoverview SMI-4590 Wave 4 PR 5/6 — `sklx config get/set audit_mode` tests
+ * @module @skillsmith/cli/tests/config
+ *
+ * Coverage (plan §810-813 — load-bearing tier-revalidation security tests):
+ *
+ * Tier revalidation matrix — the `set` path MUST refuse a write when
+ * `tierAllowsAuditMode(tier, mode)` returns false. No file IO occurs on
+ * rejection; the existing config (if any) stays intact.
+ *
+ *   1. community  + power_user   → audit.mode.tier_ineligible, NO write
+ *   2. community  + governance   → audit.mode.tier_ineligible, NO write
+ *   3. individual + power_user   → audit.mode.tier_ineligible, NO write
+ *   4. individual + governance   → audit.mode.tier_ineligible, NO write
+ *   5. team       + power_user   → SUCCESS, file written
+ *   6. team       + governance   → audit.mode.tier_ineligible (Enterprise-only)
+ *   7. enterprise + governance   → SUCCESS, file written
+ *
+ * Value validation:
+ *   8. any tier   + invalid_value → audit.mode.invalid_value, NO write
+ *   9. any tier   + preventative  → SUCCESS for every tier
+ *  10. any tier   + off           → SUCCESS for every tier
+ *
+ * Unsupported keys:
+ *  11. set with unknown key → config.unsupported_key, NO write
+ *  12. get with unknown key → config.unsupported_key
+ *
+ * Get path:
+ *  13. get audit_mode reads existing file value when present
+ *  14. get audit_mode falls back to tier default when unset
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import * as fs from 'node:fs';
+import * as os from 'node:os';
+import { join } from 'node:path';
+const mocks = vi.hoisted(() => ({
+    getLicenseStatus: vi.fn(),
+}));
+vi.mock('../src/utils/license.js', () => ({
+    getLicenseStatus: () => mocks.getLicenseStatus(),
+}));
+import { runConfigGet, runConfigSet, ConfigError, configPath } from '../src/commands/config.js';
+// ============================================================================
+// Test harness — sandbox HOME so config writes are isolated.
+// ============================================================================
+let tmpHome;
+let originalHome;
+let stdoutSpy;
+beforeEach(() => {
+    tmpHome = fs.mkdtempSync(join(os.tmpdir(), 'sklx-config-test-'));
+    originalHome = process.env['HOME'];
+    process.env['HOME'] = tmpHome;
+    stdoutSpy = vi.spyOn(console, 'log').mockImplementation(() => { });
+    vi.clearAllMocks();
+});
+afterEach(() => {
+    if (originalHome !== undefined) {
+        process.env['HOME'] = originalHome;
+    }
+    else {
+        delete process.env['HOME'];
+    }
+    fs.rmSync(tmpHome, { recursive: true, force: true });
+    stdoutSpy.mockRestore();
+});
+function setTier(tier) {
+    mocks.getLicenseStatus.mockResolvedValue({ tier });
+}
+function configFileExists() {
+    return fs.existsSync(configPath());
+}
+function readConfigRaw() {
+    if (!configFileExists())
+        return {};
+    return JSON.parse(fs.readFileSync(configPath(), 'utf-8'));
+}
+// ============================================================================
+// Tests
+// ============================================================================
+describe('SMI-4590 Wave 4 PR 5/6 — sklx config tier revalidation (plan §810-813)', () => {
+    describe('community tier — power_user / governance forbidden', () => {
+        it('community + power_user → tier_ineligible, NO write', async () => {
+            setTier('community');
+            try {
+                await runConfigSet('audit_mode', 'power_user');
+                expect.fail('Expected ConfigError');
+            }
+            catch (error) {
+                expect(error).toBeInstanceOf(ConfigError);
+                expect(error.code).toBe('audit.mode.tier_ineligible');
+            }
+            expect(configFileExists()).toBe(false);
+        });
+        it('community + governance → tier_ineligible, NO write', async () => {
+            setTier('community');
+            try {
+                await runConfigSet('audit_mode', 'governance');
+                expect.fail('Expected ConfigError');
+            }
+            catch (error) {
+                expect(error.code).toBe('audit.mode.tier_ineligible');
+            }
+            expect(configFileExists()).toBe(false);
+        });
+    });
+    describe('individual tier — power_user / governance forbidden', () => {
+        it('individual + power_user → tier_ineligible, NO write', async () => {
+            setTier('individual');
+            try {
+                await runConfigSet('audit_mode', 'power_user');
+                expect.fail('Expected ConfigError');
+            }
+            catch (error) {
+                expect(error.code).toBe('audit.mode.tier_ineligible');
+            }
+            expect(configFileExists()).toBe(false);
+        });
+        it('individual + governance → tier_ineligible, NO write', async () => {
+            setTier('individual');
+            try {
+                await runConfigSet('audit_mode', 'governance');
+                expect.fail('Expected ConfigError');
+            }
+            catch (error) {
+                expect(error.code).toBe('audit.mode.tier_ineligible');
+            }
+            expect(configFileExists()).toBe(false);
+        });
+    });
+    describe('team tier — power_user allowed, governance forbidden', () => {
+        it('team + power_user → SUCCESS, file written', async () => {
+            setTier('team');
+            await runConfigSet('audit_mode', 'power_user');
+            expect(configFileExists()).toBe(true);
+            expect(readConfigRaw()['audit_mode']).toBe('power_user');
+        });
+        it('team + governance → tier_ineligible (Enterprise-only)', async () => {
+            setTier('team');
+            try {
+                await runConfigSet('audit_mode', 'governance');
+                expect.fail('Expected ConfigError');
+            }
+            catch (error) {
+                expect(error.code).toBe('audit.mode.tier_ineligible');
+            }
+            expect(configFileExists()).toBe(false);
+        });
+    });
+    describe('enterprise tier — governance allowed', () => {
+        it('enterprise + governance → SUCCESS, file written', async () => {
+            setTier('enterprise');
+            await runConfigSet('audit_mode', 'governance');
+            expect(configFileExists()).toBe(true);
+            expect(readConfigRaw()['audit_mode']).toBe('governance');
+        });
+    });
+});
+describe('SMI-4590 Wave 4 PR 5/6 — sklx config value validation', () => {
+    it('invalid_value → audit.mode.invalid_value, NO write', async () => {
+        setTier('enterprise');
+        try {
+            await runConfigSet('audit_mode', 'definitely-not-a-mode');
+            expect.fail('Expected ConfigError');
+        }
+        catch (error) {
+            expect(error.code).toBe('audit.mode.invalid_value');
+        }
+        expect(configFileExists()).toBe(false);
+    });
+    it('preventative is allowed for every tier', async () => {
+        for (const tier of ['community', 'individual', 'team', 'enterprise']) {
+            // Reset sandbox between tiers.
+            fs.rmSync(configPath(), { force: true });
+            setTier(tier);
+            await runConfigSet('audit_mode', 'preventative');
+            expect(readConfigRaw()['audit_mode']).toBe('preventative');
+        }
+    });
+    it('off is allowed for every tier', async () => {
+        for (const tier of ['community', 'individual', 'team', 'enterprise']) {
+            fs.rmSync(configPath(), { force: true });
+            setTier(tier);
+            await runConfigSet('audit_mode', 'off');
+            expect(readConfigRaw()['audit_mode']).toBe('off');
+        }
+    });
+});
+describe('SMI-4590 Wave 4 PR 5/6 — sklx config unsupported keys', () => {
+    it('set with unknown key → config.unsupported_key, NO write', async () => {
+        setTier('enterprise');
+        try {
+            await runConfigSet('telemetry_endpoint', 'https://example.com');
+            expect.fail('Expected ConfigError');
+        }
+        catch (error) {
+            expect(error.code).toBe('config.unsupported_key');
+        }
+        expect(configFileExists()).toBe(false);
+    });
+    it('get with unknown key → config.unsupported_key', async () => {
+        setTier('enterprise');
+        try {
+            await runConfigGet('telemetry_endpoint');
+            expect.fail('Expected ConfigError');
+        }
+        catch (error) {
+            expect(error.code).toBe('config.unsupported_key');
+        }
+    });
+});
+describe('SMI-4590 Wave 4 PR 5/6 — sklx config get audit_mode', () => {
+    it('reads existing file value when present', async () => {
+        setTier('team');
+        await runConfigSet('audit_mode', 'power_user');
+        stdoutSpy.mockClear();
+        await runConfigGet('audit_mode');
+        const out = stdoutSpy.mock.calls.map((c) => String(c[0] ?? '')).join('\n');
+        expect(out).toContain('power_user');
+    });
+    it('falls back to tier default when no config file exists', async () => {
+        setTier('community');
+        // No prior write, so file does not exist.
+        expect(configFileExists()).toBe(false);
+        await runConfigGet('audit_mode');
+        const out = stdoutSpy.mock.calls.map((c) => String(c[0] ?? '')).join('\n');
+        // Output should reference the tier default plus the "(tier default)" hint.
+        expect(out).toMatch(/tier default/);
+    });
+    it('preserves unknown keys on read-modify-write (atomic merge)', async () => {
+        setTier('team');
+        // Seed the config with extra keys directly.
+        fs.mkdirSync(join(tmpHome, '.skillsmith'), { recursive: true });
+        fs.writeFileSync(configPath(), JSON.stringify({ extra_key: 'value', audit_mode: 'preventative' }), 'utf-8');
+        await runConfigSet('audit_mode', 'power_user');
+        const after = readConfigRaw();
+        expect(after['audit_mode']).toBe('power_user');
+        expect(after['extra_key']).toBe('value');
+    });
+});
+//# sourceMappingURL=config.test.js.map

package/dist/tests/config.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"config.test.js","sourceRoot":"","sources":["../../tests/config.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA8BG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAA;AACxE,OAAO,KAAK,EAAE,MAAM,SAAS,CAAA;AAC7B,OAAO,KAAK,EAAE,MAAM,SAAS,CAAA;AAC7B,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAA;AAEhC,MAAM,KAAK,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IAC9B,gBAAgB,EAAE,EAAE,CAAC,EAAE,EAAE;CAC1B,CAAC,CAAC,CAAA;AAEH,EAAE,CAAC,IAAI,CAAC,yBAAyB,EAAE,GAAG,EAAE,CAAC,CAAC;IACxC,gBAAgB,EAAE,GAAG,EAAE,CAAC,KAAK,CAAC,gBAAgB,EAAE;CACjD,CAAC,CAAC,CAAA;AAEH,OAAO,EAAE,YAAY,EAAE,YAAY,EAAE,WAAW,EAAE,UAAU,EAAE,MAAM,2BAA2B,CAAA;AAE/F,+EAA+E;AAC/E,6DAA6D;AAC7D,+EAA+E;AAE/E,IAAI,OAAe,CAAA;AACnB,IAAI,YAAgC,CAAA;AACpC,IAAI,SAAsC,CAAA;AAE1C,UAAU,CAAC,GAAG,EAAE;IACd,OAAO,GAAG,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,mBAAmB,CAAC,CAAC,CAAA;IAChE,YAAY,GAAG,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,CAAA;IAClC,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,GAAG,OAAO,CAAA;IAC7B,SAAS,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAA;IACjE,EAAE,CAAC,aAAa,EAAE,CAAA;AACpB,CAAC,CAAC,CAAA;AAEF,SAAS,CAAC,GAAG,EAAE;IACb,IAAI,YAAY,KAAK,SAAS,EAAE,CAAC;QAC/B,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,GAAG,YAAY,CAAA;IACpC,CAAC;SAAM,CAAC;QACN,OAAO,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,CAAA;IAC5B,CAAC;IACD,EAAE,CAAC,MAAM,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAA;IACpD,SAAS,CAAC,WAAW,EAAE,CAAA;AACzB,CAAC,CAAC,CAAA;AAEF,SAAS,OAAO,CAAC,IAAY;IAC3B,KAAK,CAAC,gBAAgB,CAAC,iBAAiB,CAAC,EAAE,IAAI,EAAE,CAAC,CAAA;AACpD,CAAC;AAED,SAAS,gBAAgB;IACvB,OAAO,EAAE,CAAC,UAAU,CAAC,UAAU,EAAE,CAAC,CAAA;AACpC,CAAC;AAED,SAAS,aAAa;IACpB,IAAI,CAAC,gBAAgB,EAAE;QAAE,OAAO,EAAE,CAAA;IAClC,OAAO,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,YAAY,CAAC,UAAU,EAAE,EAAE,OAAO,CAAC,CAA4B,CAAA;AACtF,CAAC;AAED,+EAA+E;AAC/E,QAAQ;AACR,+EAA+E;AAE/E,QAAQ,CAAC,wEAAwE,EAAE,GAAG,EAAE;IACtF,QAAQ,CAAC,oDAAoD,EAAE,GAAG,EAAE;QAClE,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;YAClE,OAAO,CAAC,WAAW,CAAC,CAAA;YACpB,IAAI,CAAC;gBACH,MAAM,YAAY,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;gBAC9C,MAAM,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAA;YACrC,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,MAAM,CAAC,KAAK,CAAC,CAAC,cAAc,CAAC,WAAW,CAAC,CAAA;gBACzC,MAAM,CAAE,KAAqB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAA;YACxE,CAAC;YACD,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACxC,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;YAClE,OAAO,CAAC,WAAW,CAAC,CAAA;YACpB,IAAI,CAAC;gBACH,MAAM,YAAY,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;gBAC9C,MAAM,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAA;YACrC,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,MAAM,CAAE,KAAqB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAA;YACxE,CAAC;YACD,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACxC,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,qDAAqD,EAAE,GAAG,EAAE;QACnE,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;YACnE,OAAO,CAAC,YAAY,CAAC,CAAA;YACrB,IAAI,CAAC;gBACH,MAAM,YAAY,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;gBAC9C,MAAM,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAA;YACrC,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,MAAM,CAAE,KAAqB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAA;YACxE,CAAC;YACD,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACxC,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;YACnE,OAAO,CAAC,YAAY,CAAC,CAAA;YACrB,IAAI,CAAC;gBACH,MAAM,YAAY,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;gBAC9C,MAAM,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAA;YACrC,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,MAAM,CAAE,KAAqB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAA;YACxE,CAAC;YACD,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACxC,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,sDAAsD,EAAE,GAAG,EAAE;QACpE,EAAE,CAAC,2CAA2C,EAAE,KAAK,IAAI,EAAE;YACzD,OAAO,CAAC,MAAM,CAAC,CAAA;YACf,MAAM,YAAY,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;YAC9C,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;YACrC,MAAM,CAAC,aAAa,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAA;QAC1D,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;YACrE,OAAO,CAAC,MAAM,CAAC,CAAA;YACf,IAAI,CAAC;gBACH,MAAM,YAAY,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;gBAC9C,MAAM,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAA;YACrC,CAAC;YAAC,OAAO,KAAK,EAAE,CAAC;gBACf,MAAM,CAAE,KAAqB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAA;YACxE,CAAC;YACD,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACxC,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,sCAAsC,EAAE,GAAG,EAAE;QACpD,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;YAC/D,OAAO,CAAC,YAAY,CAAC,CAAA;YACrB,MAAM,YAAY,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;YAC9C,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;YACrC,MAAM,CAAC,aAAa,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAA;QAC1D,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,uDAAuD,EAAE,GAAG,EAAE;IACrE,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;QAClE,OAAO,CAAC,YAAY,CAAC,CAAA;QACrB,IAAI,CAAC;YACH,MAAM,YAAY,CAAC,YAAY,EAAE,uBAAuB,CAAC,CAAA;YACzD,MAAM,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAA;QACrC,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,CAAE,KAAqB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,0BAA0B,CAAC,CAAA;QACtE,CAAC;QACD,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACxC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,wCAAwC,EAAE,KAAK,IAAI,EAAE;QACtD,KAAK,MAAM,IAAI,IAAI,CAAC,WAAW,EAAE,YAAY,EAAE,MAAM,EAAE,YAAY,CAAC,EAAE,CAAC;YACrE,+BAA+B;YAC/B,EAAE,CAAC,MAAM,CAAC,UAAU,EAAE,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAA;YACxC,OAAO,CAAC,IAAI,CAAC,CAAA;YACb,MAAM,YAAY,CAAC,YAAY,EAAE,cAAc,CAAC,CAAA;YAChD,MAAM,CAAC,aAAa,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAA;QAC5D,CAAC;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,+BAA+B,EAAE,KAAK,IAAI,EAAE;QAC7C,KAAK,MAAM,IAAI,IAAI,CAAC,WAAW,EAAE,YAAY,EAAE,MAAM,EAAE,YAAY,CAAC,EAAE,CAAC;YACrE,EAAE,CAAC,MAAM,CAAC,UAAU,EAAE,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAA;YACxC,OAAO,CAAC,IAAI,CAAC,CAAA;YACb,MAAM,YAAY,CAAC,YAAY,EAAE,KAAK,CAAC,CAAA;YACvC,MAAM,CAAC,aAAa,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACnD,CAAC;IACH,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,uDAAuD,EAAE,GAAG,EAAE;IACrE,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;QACvE,OAAO,CAAC,YAAY,CAAC,CAAA;QACrB,IAAI,CAAC;YACH,MAAM,YAAY,CAAC,oBAAoB,EAAE,qBAAqB,CAAC,CAAA;YAC/D,MAAM,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAA;QACrC,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,CAAE,KAAqB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAA;QACpE,CAAC;QACD,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACxC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;QAC7D,OAAO,CAAC,YAAY,CAAC,CAAA;QACrB,IAAI,CAAC;YACH,MAAM,YAAY,CAAC,oBAAoB,CAAC,CAAA;YACxC,MAAM,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAA;QACrC,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,MAAM,CAAE,KAAqB,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAA;QACpE,CAAC;IACH,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,qDAAqD,EAAE,GAAG,EAAE;IACnE,EAAE,CAAC,wCAAwC,EAAE,KAAK,IAAI,EAAE;QACtD,OAAO,CAAC,MAAM,CAAC,CAAA;QACf,MAAM,YAAY,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;QAC9C,SAAS,CAAC,SAAS,EAAE,CAAA;QACrB,MAAM,YAAY,CAAC,YAAY,CAAC,CAAA;QAChC,MAAM,GAAG,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAY,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACrF,MAAM,CAAC,GAAG,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAA;IACrC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;QACrE,OAAO,CAAC,WAAW,CAAC,CAAA;QACpB,0CAA0C;QAC1C,MAAM,CAAC,gBAAgB,EAAE,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACtC,MAAM,YAAY,CAAC,YAAY,CAAC,CAAA;QAChC,MAAM,GAAG,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAY,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACrF,2EAA2E;QAC3E,MAAM,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC,cAAc,CAAC,CAAA;IACrC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;QAC1E,OAAO,CAAC,MAAM,CAAC,CAAA;QACf,4CAA4C;QAC5C,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,aAAa,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;QAC/D,EAAE,CAAC,aAAa,CACd,UAAU,EAAE,EACZ,IAAI,CAAC,SAAS,CAAC,EAAE,SAAS,EAAE,OAAO,EAAE,UAAU,EAAE,cAAc,EAAE,CAAC,EAClE,OAAO,CACR,CAAA;QACD,MAAM,YAAY,CAAC,YAAY,EAAE,YAAY,CAAC,CAAA;QAC9C,MAAM,KAAK,GAAG,aAAa,EAAE,CAAA;QAC7B,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,CAAA;QAC9C,MAAM,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;IAC1C,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/tests/install-multi-client.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=install-multi-client.test.d.ts.map

package/dist/tests/install-multi-client.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"install-multi-client.test.d.ts","sourceRoot":"","sources":["../../tests/install-multi-client.test.ts"],"names":[],"mappings":""}

package/dist/tests/install-multi-client.test.js

@@ -0,0 +1,202 @@
+/**
+ * SMI-4578 Step 6: install --client / --also-link end-to-end tests.
+ *
+ * Exercises the parseAlsoLink + assertClientId validation surface in
+ * install.ts plus the addLink fan-out behaviour from
+ * @skillsmith/core/install. Each test runs in a temp $HOME so we
+ * never touch the real filesystem.
+ *
+ * Skill installation itself (the SkillInstallationService.install
+ * call) is NOT exercised here — that's covered by install.test.ts,
+ * service unit tests, and the post-merge smoke. These tests focus on
+ * the multi-client surface: target-directory resolution, fan-out
+ * manifest writes, conflict refuse, cycle refuse.
+ */
+import { mkdtemp, mkdir, readFile, rm, stat, lstat, writeFile } from 'node:fs/promises';
+import { tmpdir } from 'node:os';
+import * as path from 'node:path';
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+const ORIGINAL_HOME = process.env['HOME'];
+const ORIGINAL_USERPROFILE = process.env['USERPROFILE'];
+const ORIGINAL_CLIENT = process.env['SKILLSMITH_CLIENT'];
+let homeDir;
+beforeEach(async () => {
+    homeDir = await mkdtemp(path.join(tmpdir(), 'smi4578-multi-'));
+    process.env['HOME'] = homeDir;
+    process.env['USERPROFILE'] = homeDir;
+    delete process.env['SKILLSMITH_CLIENT'];
+    // CLIENT_NATIVE_PATHS computes homedir() at module import time —
+    // reset modules so each test sees its own $HOME.
+    vi.resetModules();
+});
+afterEach(async () => {
+    if (ORIGINAL_HOME === undefined)
+        delete process.env['HOME'];
+    else
+        process.env['HOME'] = ORIGINAL_HOME;
+    if (ORIGINAL_USERPROFILE === undefined)
+        delete process.env['USERPROFILE'];
+    else
+        process.env['USERPROFILE'] = ORIGINAL_USERPROFILE;
+    if (ORIGINAL_CLIENT === undefined)
+        delete process.env['SKILLSMITH_CLIENT'];
+    else
+        process.env['SKILLSMITH_CLIENT'] = ORIGINAL_CLIENT;
+    await rm(homeDir, { recursive: true, force: true });
+});
+async function seedSkill(skillId, body = '# test\n') {
+    const dir = path.join(homeDir, '.claude', 'skills', skillId);
+    await mkdir(dir, { recursive: true });
+    await writeFile(path.join(dir, 'SKILL.md'), `---\nname: ${skillId}\n---\n${body}`, 'utf-8');
+    return dir;
+}
+describe('install --client / --also-link', () => {
+    describe('--client target directory', () => {
+        it('getInstallPath returns the cursor directory for --client cursor', async () => {
+            const { getInstallPath } = await import('@skillsmith/core/install');
+            expect(getInstallPath('cursor')).toBe(path.join(homeDir, '.cursor', 'skills'));
+        });
+        it('SKILLSMITH_CLIENT env var routes resolveClientPath', async () => {
+            process.env['SKILLSMITH_CLIENT'] = 'windsurf';
+            const { resolveClientPath } = await import('@skillsmith/core/install');
+            expect(resolveClientPath()).toBe(path.join(homeDir, '.codeium', 'windsurf', 'skills'));
+        });
+        it('explicit override beats env var', async () => {
+            process.env['SKILLSMITH_CLIENT'] = 'cursor';
+            const { resolveClientPath } = await import('@skillsmith/core/install');
+            expect(resolveClientPath('agents')).toBe(path.join(homeDir, '.agents', 'skills'));
+        });
+        it('rejects invalid SKILLSMITH_CLIENT with a friendly hint', async () => {
+            process.env['SKILLSMITH_CLIENT'] = 'codex';
+            const { resolveClientPath } = await import('@skillsmith/core/install');
+            expect(() => resolveClientPath()).toThrow(/--client agents/);
+        });
+    });
+    describe('--also-link copy default', () => {
+        it('copies the source skill into a secondary client directory', async () => {
+            await seedSkill('foo', '# foo\n');
+            const { addLink, listLinks } = await import('@skillsmith/core/install');
+            const result = await addLink({
+                skillId: 'foo',
+                fromClient: 'claude-code',
+                toClient: 'cursor',
+            });
+            expect(result.record.kind).toBe('copy');
+            const dest = path.join(homeDir, '.cursor', 'skills', 'foo');
+            const skillMd = await readFile(path.join(dest, 'SKILL.md'), 'utf-8');
+            expect(skillMd).toContain('# foo');
+            const links = await listLinks('foo');
+            expect(links).toHaveLength(1);
+            expect(links[0]?.kind).toBe('copy');
+            expect(links[0]?.to).toBe(dest);
+        });
+        it('writes a manifest entry per fan-out target', async () => {
+            await seedSkill('multi');
+            const { addLink, getLinkManifestPath, loadManifest } = await import('@skillsmith/core/install');
+            await addLink({ skillId: 'multi', fromClient: 'claude-code', toClient: 'cursor' });
+            await addLink({ skillId: 'multi', fromClient: 'claude-code', toClient: 'agents' });
+            const manifest = await loadManifest();
+            expect(manifest.links.filter((l) => l.skillId === 'multi')).toHaveLength(2);
+            expect(manifest.version).toBe(1);
+            // manifest path is under ~/.skillsmith/links/manifest.json
+            const manifestPath = getLinkManifestPath();
+            expect(manifestPath).toBe(path.join(homeDir, '.skillsmith', 'links', 'manifest.json'));
+            await expect(stat(manifestPath)).resolves.toBeDefined();
+        });
+    });
+    describe('--also-link --symlink (POSIX opt-in)', () => {
+        it('creates a relative symlink instead of a copy', async () => {
+            await seedSkill('linkme');
+            const { addLink } = await import('@skillsmith/core/install');
+            const result = await addLink({
+                skillId: 'linkme',
+                fromClient: 'claude-code',
+                toClient: 'windsurf',
+                preferSymlink: true,
+            });
+            expect(result.record.kind).toBe('symlink');
+            const dest = path.join(homeDir, '.codeium', 'windsurf', 'skills', 'linkme');
+            const linkStat = await lstat(dest);
+            expect(linkStat.isSymbolicLink()).toBe(true);
+        });
+    });
+    describe('conflict policy', () => {
+        it('refuses to overwrite a pre-existing destination without force', async () => {
+            await seedSkill('clash');
+            // Pre-create destination with different content
+            const destDir = path.join(homeDir, '.cursor', 'skills', 'clash');
+            await mkdir(destDir, { recursive: true });
+            await writeFile(path.join(destDir, 'SKILL.md'), '# DIFFERENT\n', 'utf-8');
+            const { addLink } = await import('@skillsmith/core/install');
+            await expect(addLink({ skillId: 'clash', fromClient: 'claude-code', toClient: 'cursor' })).rejects.toThrow(/already exists/);
+            // Destination contents unchanged
+            const after = await readFile(path.join(destDir, 'SKILL.md'), 'utf-8');
+            expect(after).toBe('# DIFFERENT\n');
+        });
+        it('overwrites when force=true', async () => {
+            await seedSkill('clash');
+            const destDir = path.join(homeDir, '.cursor', 'skills', 'clash');
+            await mkdir(destDir, { recursive: true });
+            await writeFile(path.join(destDir, 'SKILL.md'), '# OLD\n', 'utf-8');
+            const { addLink } = await import('@skillsmith/core/install');
+            await addLink({
+                skillId: 'clash',
+                fromClient: 'claude-code',
+                toClient: 'cursor',
+                force: true,
+            });
+            const after = await readFile(path.join(destDir, 'SKILL.md'), 'utf-8');
+            expect(after).toContain('# test'); // matches seedSkill default body
+        });
+    });
+    describe('cycle detection', () => {
+        it('refuses A→B when an existing B→A entry would form a cycle', async () => {
+            await seedSkill('cycle');
+            const agentsDir = path.join(homeDir, '.agents', 'skills', 'cycle');
+            await mkdir(agentsDir, { recursive: true });
+            await writeFile(path.join(agentsDir, 'SKILL.md'), '# cycle\n', 'utf-8');
+            const { addLink } = await import('@skillsmith/core/install');
+            // Forward: agents → claude-code
+            await addLink({
+                skillId: 'cycle',
+                fromClient: 'agents',
+                toClient: 'claude-code',
+                force: true,
+            });
+            // Reverse: claude-code → agents — should refuse
+            await expect(addLink({
+                skillId: 'cycle',
+                fromClient: 'claude-code',
+                toClient: 'agents',
+                force: true,
+            })).rejects.toThrow(/cycle detected/);
+        });
+    });
+    describe('uninstall fan-out cleanup', () => {
+        it('removeLinks tears down both copies and symlinks', async () => {
+            await seedSkill('teardown');
+            const { addLink, removeLinks, listLinks } = await import('@skillsmith/core/install');
+            await addLink({ skillId: 'teardown', fromClient: 'claude-code', toClient: 'cursor' });
+            await addLink({
+                skillId: 'teardown',
+                fromClient: 'claude-code',
+                toClient: 'windsurf',
+                preferSymlink: true,
+            });
+            const removed = await removeLinks('teardown');
+            expect(removed).toBe(2);
+            expect(await listLinks('teardown')).toEqual([]);
+            // Both destinations gone
+            await expect(stat(path.join(homeDir, '.cursor', 'skills', 'teardown'))).rejects.toThrow(/ENOENT/);
+            await expect(lstat(path.join(homeDir, '.codeium', 'windsurf', 'skills', 'teardown'))).rejects.toThrow(/ENOENT/);
+            // Source untouched
+            await expect(stat(path.join(homeDir, '.claude', 'skills', 'teardown'))).resolves.toBeDefined();
+        });
+        it('removeLinks returns 0 when no manifest exists', async () => {
+            const { removeLinks } = await import('@skillsmith/core/install');
+            const removed = await removeLinks('nothing-installed');
+            expect(removed).toBe(0);
+        });
+    });
+});
+//# sourceMappingURL=install-multi-client.test.js.map

package/dist/tests/install-multi-client.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"install-multi-client.test.js","sourceRoot":"","sources":["../../tests/install-multi-client.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AACH,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,QAAQ,EAAE,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAA;AACvF,OAAO,EAAE,MAAM,EAAE,MAAM,SAAS,CAAA;AAChC,OAAO,KAAK,IAAI,MAAM,WAAW,CAAA;AACjC,OAAO,EAAE,SAAS,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAA;AAExE,MAAM,aAAa,GAAG,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,CAAA;AACzC,MAAM,oBAAoB,GAAG,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC,CAAA;AACvD,MAAM,eAAe,GAAG,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,CAAA;AAExD,IAAI,OAAe,CAAA;AAEnB,UAAU,CAAC,KAAK,IAAI,EAAE;IACpB,OAAO,GAAG,MAAM,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,gBAAgB,CAAC,CAAC,CAAA;IAC9D,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,GAAG,OAAO,CAAA;IAC7B,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC,GAAG,OAAO,CAAA;IACpC,OAAO,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,CAAA;IACvC,iEAAiE;IACjE,iDAAiD;IACjD,EAAE,CAAC,YAAY,EAAE,CAAA;AACnB,CAAC,CAAC,CAAA;AAEF,SAAS,CAAC,KAAK,IAAI,EAAE;IACnB,IAAI,aAAa,KAAK,SAAS;QAAE,OAAO,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,CAAA;;QACtD,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,GAAG,aAAa,CAAA;IACxC,IAAI,oBAAoB,KAAK,SAAS;QAAE,OAAO,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC,CAAA;;QACpE,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC,GAAG,oBAAoB,CAAA;IACtD,IAAI,eAAe,KAAK,SAAS;QAAE,OAAO,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,CAAA;;QACrE,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,GAAG,eAAe,CAAA;IACvD,MAAM,EAAE,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAA;AACrD,CAAC,CAAC,CAAA;AAEF,KAAK,UAAU,SAAS,CAAC,OAAe,EAAE,OAAe,UAAU;IACjE,MAAM,GAAG,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,OAAO,CAAC,CAAA;IAC5D,MAAM,KAAK,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;IACrC,MAAM,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,UAAU,CAAC,EAAE,cAAc,OAAO,UAAU,IAAI,EAAE,EAAE,OAAO,CAAC,CAAA;IAC3F,OAAO,GAAG,CAAA;AACZ,CAAC;AAED,QAAQ,CAAC,gCAAgC,EAAE,GAAG,EAAE;IAC9C,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACzC,EAAE,CAAC,iEAAiE,EAAE,KAAK,IAAI,EAAE;YAC/E,MAAM,EAAE,cAAc,EAAE,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YACnE,MAAM,CAAC,cAAc,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC,CAAA;QAChF,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;YAClE,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,GAAG,UAAU,CAAA;YAC7C,MAAM,EAAE,iBAAiB,EAAE,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YACtE,MAAM,CAAC,iBAAiB,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,EAAE,UAAU,EAAE,QAAQ,CAAC,CAAC,CAAA;QACxF,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,iCAAiC,EAAE,KAAK,IAAI,EAAE;YAC/C,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,GAAG,QAAQ,CAAA;YAC3C,MAAM,EAAE,iBAAiB,EAAE,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YACtE,MAAM,CAAC,iBAAiB,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,QAAQ,CAAC,CAAC,CAAA;QACnF,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;YACtE,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,GAAG,OAAO,CAAA;YAC1C,MAAM,EAAE,iBAAiB,EAAE,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YACtE,MAAM,CAAC,GAAG,EAAE,CAAC,iBAAiB,EAAE,CAAC,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAA;QAC9D,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,0BAA0B,EAAE,GAAG,EAAE;QACxC,EAAE,CAAC,2DAA2D,EAAE,KAAK,IAAI,EAAE;YACzE,MAAM,SAAS,CAAC,KAAK,EAAE,SAAS,CAAC,CAAA;YACjC,MAAM,EAAE,OAAO,EAAE,SAAS,EAAE,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YAEvE,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC;gBAC3B,OAAO,EAAE,KAAK;gBACd,UAAU,EAAE,aAAa;gBACzB,QAAQ,EAAE,QAAQ;aACnB,CAAC,CAAA;YAEF,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;YACvC,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,KAAK,CAAC,CAAA;YAC3D,MAAM,OAAO,GAAG,MAAM,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,UAAU,CAAC,EAAE,OAAO,CAAC,CAAA;YACpE,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,OAAO,CAAC,CAAA;YAElC,MAAM,KAAK,GAAG,MAAM,SAAS,CAAC,KAAK,CAAC,CAAA;YACpC,MAAM,CAAC,KAAK,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;YAC7B,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;YACnC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QACjC,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,4CAA4C,EAAE,KAAK,IAAI,EAAE;YAC1D,MAAM,SAAS,CAAC,OAAO,CAAC,CAAA;YACxB,MAAM,EAAE,OAAO,EAAE,mBAAmB,EAAE,YAAY,EAAE,GAClD,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YAE1C,MAAM,OAAO,CAAC,EAAE,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,aAAa,EAAE,QAAQ,EAAE,QAAQ,EAAE,CAAC,CAAA;YAClF,MAAM,OAAO,CAAC,EAAE,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,aAAa,EAAE,QAAQ,EAAE,QAAQ,EAAE,CAAC,CAAA;YAElF,MAAM,QAAQ,GAAG,MAAM,YAAY,EAAE,CAAA;YACrC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,OAAO,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;YAC3E,MAAM,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;YAEhC,2DAA2D;YAC3D,MAAM,YAAY,GAAG,mBAAmB,EAAE,CAAA;YAC1C,MAAM,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,aAAa,EAAE,OAAO,EAAE,eAAe,CAAC,CAAC,CAAA;YACtF,MAAM,MAAM,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAA;QACzD,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,sCAAsC,EAAE,GAAG,EAAE;QACpD,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;YAC5D,MAAM,SAAS,CAAC,QAAQ,CAAC,CAAA;YACzB,MAAM,EAAE,OAAO,EAAE,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YAE5D,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC;gBAC3B,OAAO,EAAE,QAAQ;gBACjB,UAAU,EAAE,aAAa;gBACzB,QAAQ,EAAE,UAAU;gBACpB,aAAa,EAAE,IAAI;aACpB,CAAC,CAAA;YAEF,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;YAC1C,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,EAAE,UAAU,EAAE,QAAQ,EAAE,QAAQ,CAAC,CAAA;YAC3E,MAAM,QAAQ,GAAG,MAAM,KAAK,CAAC,IAAI,CAAC,CAAA;YAClC,MAAM,CAAC,QAAQ,CAAC,cAAc,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAC9C,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;QAC/B,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;YAC7E,MAAM,SAAS,CAAC,OAAO,CAAC,CAAA;YACxB,gDAAgD;YAChD,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,OAAO,CAAC,CAAA;YAChE,MAAM,KAAK,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;YACzC,MAAM,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,CAAC,EAAE,eAAe,EAAE,OAAO,CAAC,CAAA;YAEzE,MAAM,EAAE,OAAO,EAAE,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YAC5D,MAAM,MAAM,CACV,OAAO,CAAC,EAAE,OAAO,EAAE,OAAO,EAAE,UAAU,EAAE,aAAa,EAAE,QAAQ,EAAE,QAAQ,EAAE,CAAC,CAC7E,CAAC,OAAO,CAAC,OAAO,CAAC,gBAAgB,CAAC,CAAA;YAEnC,iCAAiC;YACjC,MAAM,KAAK,GAAG,MAAM,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,CAAC,EAAE,OAAO,CAAC,CAAA;YACrE,MAAM,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,eAAe,CAAC,CAAA;QACrC,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,4BAA4B,EAAE,KAAK,IAAI,EAAE;YAC1C,MAAM,SAAS,CAAC,OAAO,CAAC,CAAA;YACxB,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,OAAO,CAAC,CAAA;YAChE,MAAM,KAAK,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;YACzC,MAAM,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,CAAC,EAAE,SAAS,EAAE,OAAO,CAAC,CAAA;YAEnE,MAAM,EAAE,OAAO,EAAE,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YAC5D,MAAM,OAAO,CAAC;gBACZ,OAAO,EAAE,OAAO;gBAChB,UAAU,EAAE,aAAa;gBACzB,QAAQ,EAAE,QAAQ;gBAClB,KAAK,EAAE,IAAI;aACZ,CAAC,CAAA;YACF,MAAM,KAAK,GAAG,MAAM,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,CAAC,EAAE,OAAO,CAAC,CAAA;YACrE,MAAM,CAAC,KAAK,CAAC,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAA,CAAC,iCAAiC;QACrE,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;QAC/B,EAAE,CAAC,2DAA2D,EAAE,KAAK,IAAI,EAAE;YACzE,MAAM,SAAS,CAAC,OAAO,CAAC,CAAA;YACxB,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,OAAO,CAAC,CAAA;YAClE,MAAM,KAAK,CAAC,SAAS,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;YAC3C,MAAM,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,UAAU,CAAC,EAAE,WAAW,EAAE,OAAO,CAAC,CAAA;YAEvE,MAAM,EAAE,OAAO,EAAE,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YAC5D,gCAAgC;YAChC,MAAM,OAAO,CAAC;gBACZ,OAAO,EAAE,OAAO;gBAChB,UAAU,EAAE,QAAQ;gBACpB,QAAQ,EAAE,aAAa;gBACvB,KAAK,EAAE,IAAI;aACZ,CAAC,CAAA;YACF,gDAAgD;YAChD,MAAM,MAAM,CACV,OAAO,CAAC;gBACN,OAAO,EAAE,OAAO;gBAChB,UAAU,EAAE,aAAa;gBACzB,QAAQ,EAAE,QAAQ;gBAClB,KAAK,EAAE,IAAI;aACZ,CAAC,CACH,CAAC,OAAO,CAAC,OAAO,CAAC,gBAAgB,CAAC,CAAA;QACrC,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,2BAA2B,EAAE,GAAG,EAAE;QACzC,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;YAC/D,MAAM,SAAS,CAAC,UAAU,CAAC,CAAA;YAC3B,MAAM,EAAE,OAAO,EAAE,WAAW,EAAE,SAAS,EAAE,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YAEpF,MAAM,OAAO,CAAC,EAAE,OAAO,EAAE,UAAU,EAAE,UAAU,EAAE,aAAa,EAAE,QAAQ,EAAE,QAAQ,EAAE,CAAC,CAAA;YACrF,MAAM,OAAO,CAAC;gBACZ,OAAO,EAAE,UAAU;gBACnB,UAAU,EAAE,aAAa;gBACzB,QAAQ,EAAE,UAAU;gBACpB,aAAa,EAAE,IAAI;aACpB,CAAC,CAAA;YAEF,MAAM,OAAO,GAAG,MAAM,WAAW,CAAC,UAAU,CAAC,CAAA;YAC7C,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;YACvB,MAAM,CAAC,MAAM,SAAS,CAAC,UAAU,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAA;YAE/C,yBAAyB;YACzB,MAAM,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,UAAU,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CACrF,QAAQ,CACT,CAAA;YACD,MAAM,MAAM,CACV,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,UAAU,EAAE,UAAU,EAAE,QAAQ,EAAE,UAAU,CAAC,CAAC,CACxE,CAAC,OAAO,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAA;YAE3B,mBAAmB;YACnB,MAAM,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,UAAU,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAA;QAChG,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;YAC7D,MAAM,EAAE,WAAW,EAAE,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAA;YAChE,MAAM,OAAO,GAAG,MAAM,WAAW,CAAC,mBAAmB,CAAC,CAAA;YACtD,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACzB,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/tests/integration/import-local.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=import-local.test.d.ts.map

package/dist/tests/integration/import-local.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"import-local.test.d.ts","sourceRoot":"","sources":["../../../tests/integration/import-local.test.ts"],"names":[],"mappings":""}

package/dist/tests/integration/import-local.test.js

@@ -0,0 +1,158 @@
+/**
+ * @fileoverview Integration tests for `skillsmith import-local` (SMI-4665)
+ *
+ * Exercises the action handler against a real SQLite DB on a tmpdir fixture.
+ * Round-trip:
+ *   1. Import — fixture skill lands in DB with source='local', trust_tier='local'
+ *   2. Re-import is idempotent — same id, no duplicates
+ *   3. SyncEngine.upsertSkills with force=true does NOT clobber the local row
+ */
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import { promises as fs } from 'node:fs';
+import { join } from 'node:path';
+import { tmpdir } from 'node:os';
+import { createDatabaseAsync, initializeSchema, SkillRepository, SyncEngine, SyncConfigRepository, SyncHistoryRepository, } from '@skillsmith/core';
+import { runImportLocal } from '../../src/commands/import-local.js';
+import { vi } from 'vitest';
+let workDir;
+let dbPath;
+beforeEach(async () => {
+    workDir = await fs.mkdtemp(join(tmpdir(), 'skillsmith-il-'));
+    dbPath = join(workDir, 'skills.db');
+});
+afterEach(async () => {
+    await fs.rm(workDir, { recursive: true, force: true });
+});
+async function writeSkill(name) {
+    const dir = join(workDir, 'skills', name);
+    await fs.mkdir(dir, { recursive: true });
+    const file = join(dir, 'SKILL.md');
+    await fs.writeFile(file, `---\nname: ${name}\ndescription: A test skill named ${name}\ntags:\n  - test\n---\n# ${name}\n\nBody.\n`);
+    return file;
+}
+describe('skillsmith import-local — round-trip', () => {
+    it('imports SKILL.md files into the DB with source=local', async () => {
+        await writeSkill('alpha');
+        await writeSkill('beta');
+        const result = await runImportLocal({
+            path: join(workDir, 'skills'),
+            dbPath,
+        });
+        expect(result.errors).toEqual([]);
+        expect(result.imported).toBe(2);
+        expect(result.scanned).toBe(2);
+        const db = await createDatabaseAsync(dbPath);
+        initializeSchema(db);
+        try {
+            const repo = new SkillRepository(db);
+            const all = repo.findAll({ limit: 100 });
+            expect(all.items).toHaveLength(2);
+            for (const skill of all.items) {
+                expect(skill.source).toBe('local');
+                expect(skill.trustTier).toBe('local');
+            }
+        }
+        finally {
+            db.close();
+        }
+    });
+    it('re-importing the same directory is idempotent (same id, no duplicates)', async () => {
+        await writeSkill('gamma');
+        const first = await runImportLocal({ path: join(workDir, 'skills'), dbPath });
+        expect(first.imported).toBe(1);
+        expect(first.updated).toBe(0);
+        const second = await runImportLocal({ path: join(workDir, 'skills'), dbPath });
+        expect(second.imported).toBe(0);
+        expect(second.updated).toBe(1); // path is the same, id matches → update path
+        const db = await createDatabaseAsync(dbPath);
+        initializeSchema(db);
+        try {
+            const repo = new SkillRepository(db);
+            expect(repo.count()).toBe(1);
+        }
+        finally {
+            db.close();
+        }
+    });
+    it('sync --force does NOT clobber locally-imported rows', async () => {
+        const skillFile = await writeSkill('delta');
+        await runImportLocal({ path: join(workDir, 'skills'), dbPath });
+        const db = await createDatabaseAsync(dbPath);
+        try {
+            const skillRepo = new SkillRepository(db);
+            const localSkills = skillRepo.findAll({ limit: 10 }).items;
+            expect(localSkills).toHaveLength(1);
+            const local = localSkills[0];
+            // Build a SyncEngine that returns a colliding registry skill.
+            const apiClient = {
+                isOffline: vi.fn().mockReturnValue(false),
+                checkHealth: vi.fn().mockResolvedValue({ status: 'healthy' }),
+                search: vi.fn().mockResolvedValue({
+                    data: [
+                        {
+                            id: local.id,
+                            name: 'CLOBBERED REGISTRY NAME',
+                            description: 'should not appear',
+                            author: 'reg',
+                            repo_url: 'https://example.com/registry-collision',
+                            quality_score: 0.5,
+                            trust_tier: 'community',
+                            tags: [],
+                            stars: 0,
+                            installable: true,
+                            created_at: new Date().toISOString(),
+                            updated_at: new Date(Date.now() + 60_000).toISOString(),
+                        },
+                    ],
+                    total: 1,
+                }),
+                getSkill: vi.fn(),
+                getHealthStatus: vi.fn(),
+            };
+            const syncConfigRepo = new SyncConfigRepository(db);
+            const syncHistoryRepo = new SyncHistoryRepository(db);
+            const skillVersionRepo = {
+                recordVersion: vi.fn().mockResolvedValue(undefined),
+                pruneVersions: vi.fn().mockResolvedValue(undefined),
+                getLatestVersion: vi.fn().mockResolvedValue(null),
+                getVersionHistory: vi.fn().mockResolvedValue([]),
+                getVersionByHash: vi.fn().mockResolvedValue(null),
+            };
+            const engine = new SyncEngine(apiClient, skillRepo, syncConfigRepo, syncHistoryRepo, skillVersionRepo);
+            const result = await engine.sync({ force: true });
+            expect(result.success).toBe(true);
+            expect(result.skillsUpdated).toBe(0);
+            const after = skillRepo.findById(local.id);
+            expect(after).not.toBeNull();
+            expect(after.name).toBe('delta');
+            expect(after.source).toBe('local');
+        }
+        finally {
+            db.close();
+        }
+        // Sanity: the fixture file is still there (we didn't accidentally delete it).
+        await fs.access(skillFile);
+    });
+    it('--dry-run does not mutate the DB', async () => {
+        await writeSkill('echo');
+        const result = await runImportLocal({
+            path: join(workDir, 'skills'),
+            dbPath,
+            dryRun: true,
+        });
+        expect(result.dryRun).toBe(true);
+        expect(result.imported).toBe(1);
+        // The dry-run pass opens the DB only to read; it still initializes the
+        // schema so subsequent reads work. Verify no skill rows landed.
+        const db = await createDatabaseAsync(dbPath);
+        initializeSchema(db);
+        try {
+            const repo = new SkillRepository(db);
+            expect(repo.count()).toBe(0);
+        }
+        finally {
+            db.close();
+        }
+    });
+});
+//# sourceMappingURL=import-local.test.js.map