@skillsmith/cli 0.6.2 → 0.6.4

package/dist/src/commands/telemetry.helpers.test.d.ts

@@ -0,0 +1,15 @@
+/**
+ * @fileoverview Tests for telemetry.helpers.ts — atomic settings.json merger.
+ * @module @skillsmith/cli/commands/telemetry.helpers.test
+ * @see SMI-5021 Wave 3 Step 2 — plan line 717
+ *
+ * Covers:
+ *   - addSkillHookEntries: empty settings → well-formed hooks block
+ *   - addSkillHookEntries: idempotent (second add is a no-op)
+ *   - addSkillHookEntries: throws on foreign Skill matcher command path
+ *   - removeSkillHookEntries: removes only Skillsmith entries
+ *   - removeSkillHookEntries: on empty settings → no-op
+ *   - writeClaudeSettings: atomic temp-file → rename correctness
+ */
+export {};
+//# sourceMappingURL=telemetry.helpers.test.d.ts.map

package/dist/src/commands/telemetry.helpers.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"telemetry.helpers.test.d.ts","sourceRoot":"","sources":["../../../src/commands/telemetry.helpers.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG"}

package/dist/src/commands/telemetry.helpers.test.js

@@ -0,0 +1,240 @@
+/**
+ * @fileoverview Tests for telemetry.helpers.ts — atomic settings.json merger.
+ * @module @skillsmith/cli/commands/telemetry.helpers.test
+ * @see SMI-5021 Wave 3 Step 2 — plan line 717
+ *
+ * Covers:
+ *   - addSkillHookEntries: empty settings → well-formed hooks block
+ *   - addSkillHookEntries: idempotent (second add is a no-op)
+ *   - addSkillHookEntries: throws on foreign Skill matcher command path
+ *   - removeSkillHookEntries: removes only Skillsmith entries
+ *   - removeSkillHookEntries: on empty settings → no-op
+ *   - writeClaudeSettings: atomic temp-file → rename correctness
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+// ---------------------------------------------------------------------------
+// fs mock — intercepts node:fs (used by telemetry.helpers.ts)
+// ---------------------------------------------------------------------------
+const memfs = {};
+let existsResult = false;
+vi.mock('node:fs', () => ({
+    existsSync: vi.fn((_p) => existsResult),
+    readFileSync: vi.fn((path) => {
+        const content = memfs[path];
+        if (content === undefined)
+            throw Object.assign(new Error(`ENOENT: ${path}`), { code: 'ENOENT' });
+        return content;
+    }),
+    writeFileSync: vi.fn((path, content) => {
+        memfs[path] = content;
+    }),
+    renameSync: vi.fn((src, dst) => {
+        const content = memfs[src];
+        if (content === undefined)
+            throw new Error(`ENOENT: ${src}`);
+        memfs[dst] = content;
+        delete memfs[src];
+    }),
+    mkdirSync: vi.fn(() => undefined),
+    chmodSync: vi.fn(() => undefined),
+}));
+import { loadClaudeSettings, addSkillHookEntries, removeSkillHookEntries, writeClaudeSettings, TelemetryHookError, } from './telemetry.helpers.js';
+// ---------------------------------------------------------------------------
+// Helpers
+// ---------------------------------------------------------------------------
+const HOOK_PATH = '/home/test/.skillsmith/hooks/skill-telemetry.sh';
+function emptySettings() {
+    return {};
+}
+function settingsWithSkillsmithHooks() {
+    return {
+        hooks: {
+            PreToolUse: [
+                {
+                    matcher: 'Skill',
+                    hooks: [{ type: 'command', command: `${HOOK_PATH} pre` }],
+                },
+            ],
+            PostToolUse: [
+                {
+                    matcher: 'Skill',
+                    hooks: [{ type: 'command', command: `${HOOK_PATH} post` }],
+                },
+            ],
+        },
+    };
+}
+function settingsWithForeignSkillHook() {
+    return {
+        hooks: {
+            PreToolUse: [
+                {
+                    matcher: 'Skill',
+                    hooks: [{ type: 'command', command: '/other/tool/skill-hook.sh pre' }],
+                },
+            ],
+            PostToolUse: [],
+        },
+    };
+}
+// ---------------------------------------------------------------------------
+// Setup / teardown
+// ---------------------------------------------------------------------------
+beforeEach(() => {
+    for (const k of Object.keys(memfs))
+        delete memfs[k];
+    existsResult = false;
+    vi.clearAllMocks();
+});
+afterEach(() => {
+    for (const k of Object.keys(memfs))
+        delete memfs[k];
+});
+// ---------------------------------------------------------------------------
+// loadClaudeSettings
+// ---------------------------------------------------------------------------
+describe('loadClaudeSettings', () => {
+    it('returns empty object when file does not exist', () => {
+        existsResult = false;
+        const result = loadClaudeSettings('user');
+        expect(result).toEqual({});
+    });
+    it('returns parsed JSON when file exists and is valid', async () => {
+        existsResult = true;
+        const { existsSync, readFileSync } = await import('node:fs');
+        existsSync.mockReturnValueOnce(true);
+        readFileSync.mockReturnValueOnce(JSON.stringify({ hooks: { PreToolUse: [] } }));
+        const result = loadClaudeSettings('user');
+        expect(result).toHaveProperty('hooks');
+    });
+});
+// ---------------------------------------------------------------------------
+// addSkillHookEntries
+// ---------------------------------------------------------------------------
+describe('addSkillHookEntries', () => {
+    it('adds PreToolUse and PostToolUse Skill matchers to empty settings', () => {
+        const result = addSkillHookEntries(emptySettings(), HOOK_PATH);
+        expect(result.hooks?.PreToolUse).toHaveLength(1);
+        expect(result.hooks?.PreToolUse?.[0]?.matcher).toBe('Skill');
+        expect(result.hooks?.PreToolUse?.[0]?.hooks?.[0]?.command).toBe(`${HOOK_PATH} pre`);
+        expect(result.hooks?.PostToolUse).toHaveLength(1);
+        expect(result.hooks?.PostToolUse?.[0]?.matcher).toBe('Skill');
+        expect(result.hooks?.PostToolUse?.[0]?.hooks?.[0]?.command).toBe(`${HOOK_PATH} post`);
+    });
+    it('is idempotent — calling add twice produces only one entry per array', () => {
+        const once = addSkillHookEntries(emptySettings(), HOOK_PATH);
+        const twice = addSkillHookEntries(once, HOOK_PATH);
+        expect(twice.hooks?.PreToolUse).toHaveLength(1);
+        expect(twice.hooks?.PostToolUse).toHaveLength(1);
+    });
+    it('preserves existing non-Skill hook entries alongside new Skill entry', () => {
+        const existing = {
+            hooks: {
+                PreToolUse: [{ matcher: 'Bash', hooks: [{ type: 'command', command: 'echo pre-bash' }] }],
+                PostToolUse: [],
+            },
+        };
+        const result = addSkillHookEntries(existing, HOOK_PATH);
+        // Should have both Bash and Skill entries
+        expect(result.hooks?.PreToolUse).toHaveLength(2);
+        expect(result.hooks?.PreToolUse?.some((e) => e.matcher === 'Bash')).toBe(true);
+        expect(result.hooks?.PreToolUse?.some((e) => e.matcher === 'Skill')).toBe(true);
+    });
+    it('throws TelemetryHookError with code hook.foreign_skill_matcher when a foreign Skill hook exists', () => {
+        const settings = settingsWithForeignSkillHook();
+        expect(() => addSkillHookEntries(settings, HOOK_PATH)).toThrow(TelemetryHookError);
+        expect(() => addSkillHookEntries(settings, HOOK_PATH)).toThrow(expect.objectContaining({ code: 'hook.foreign_skill_matcher' }));
+    });
+    it('error message from foreign Skill matcher includes removal instructions', () => {
+        const settings = settingsWithForeignSkillHook();
+        let caught;
+        try {
+            addSkillHookEntries(settings, HOOK_PATH);
+        }
+        catch (e) {
+            caught = e;
+        }
+        expect(caught).toBeInstanceOf(TelemetryHookError);
+        const err = caught;
+        expect(err.message).toContain('Remove it manually');
+        expect(err.message).toContain('skillsmith telemetry install-hook');
+    });
+});
+// ---------------------------------------------------------------------------
+// removeSkillHookEntries
+// ---------------------------------------------------------------------------
+describe('removeSkillHookEntries', () => {
+    it('removes Skillsmith Skill entries from both PreToolUse and PostToolUse', () => {
+        const settings = settingsWithSkillsmithHooks();
+        const result = removeSkillHookEntries(settings, HOOK_PATH);
+        expect(result.hooks?.PreToolUse).toHaveLength(0);
+        expect(result.hooks?.PostToolUse).toHaveLength(0);
+    });
+    it('does not touch foreign hooks when removing Skillsmith entries', () => {
+        const settings = {
+            hooks: {
+                PreToolUse: [
+                    { matcher: 'Bash', hooks: [{ type: 'command', command: 'echo bash' }] },
+                    { matcher: 'Skill', hooks: [{ type: 'command', command: `${HOOK_PATH} pre` }] },
+                ],
+                PostToolUse: [
+                    { matcher: 'Skill', hooks: [{ type: 'command', command: `${HOOK_PATH} post` }] },
+                ],
+            },
+        };
+        const result = removeSkillHookEntries(settings, HOOK_PATH);
+        // Bash hook preserved; Skill hook removed
+        expect(result.hooks?.PreToolUse).toHaveLength(1);
+        expect(result.hooks?.PreToolUse?.[0]?.matcher).toBe('Bash');
+        expect(result.hooks?.PostToolUse).toHaveLength(0);
+    });
+    it('is a no-op when settings has no hooks key', () => {
+        const result = removeSkillHookEntries({}, HOOK_PATH);
+        expect(result.hooks).toBeUndefined();
+    });
+    it('is a no-op when no Skill entries exist to remove', () => {
+        const settings = {
+            hooks: {
+                PreToolUse: [{ matcher: 'Bash', hooks: [{ type: 'command', command: 'echo bash' }] }],
+                PostToolUse: [],
+            },
+        };
+        const result = removeSkillHookEntries(settings, HOOK_PATH);
+        expect(result.hooks?.PreToolUse).toHaveLength(1);
+        expect(result.hooks?.PostToolUse).toHaveLength(0);
+    });
+});
+// ---------------------------------------------------------------------------
+// writeClaudeSettings — atomic write correctness
+// ---------------------------------------------------------------------------
+describe('writeClaudeSettings', () => {
+    it('writes the settings to a .tmp file then renames to final path', async () => {
+        const { writeFileSync, renameSync } = await import('node:fs');
+        const settings = { hooks: { PreToolUse: [] } };
+        writeClaudeSettings('user', settings);
+        expect(writeFileSync).toHaveBeenCalledWith(expect.stringContaining('.tmp'), expect.stringContaining('"PreToolUse"'), expect.anything());
+        expect(renameSync).toHaveBeenCalledWith(expect.stringContaining('.tmp'), expect.stringContaining('settings.json'));
+    });
+    it('second write fully replaces first without partial state', async () => {
+        // Simulate sequential write correctness (mirrors manifest.test.ts pattern)
+        const first = { hooks: { PreToolUse: [] } };
+        const second = {
+            hooks: {
+                PreToolUse: [
+                    { matcher: 'Skill', hooks: [{ type: 'command', command: `${HOOK_PATH} pre` }] },
+                ],
+            },
+        };
+        writeClaudeSettings('user', first);
+        writeClaudeSettings('user', second);
+        // After both writes, memfs should contain the second write's content
+        const { resolveSettingsPath } = await import('./telemetry.helpers.js');
+        const finalPath = resolveSettingsPath('user');
+        const raw = memfs[finalPath];
+        expect(raw).toBeDefined();
+        const parsed = JSON.parse(raw);
+        expect(parsed.hooks?.PreToolUse).toHaveLength(1);
+        expect(parsed.hooks?.PreToolUse?.[0]?.matcher).toBe('Skill');
+    });
+});
+//# sourceMappingURL=telemetry.helpers.test.js.map

package/dist/src/commands/telemetry.helpers.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"telemetry.helpers.test.js","sourceRoot":"","sources":["../../../src/commands/telemetry.helpers.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAA;AAExE,8EAA8E;AAC9E,8DAA8D;AAC9D,8EAA8E;AAE9E,MAAM,KAAK,GAA2B,EAAE,CAAA;AACxC,IAAI,YAAY,GAAG,KAAK,CAAA;AAExB,EAAE,CAAC,IAAI,CAAC,SAAS,EAAE,GAAG,EAAE,CAAC,CAAC;IACxB,UAAU,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,EAAU,EAAE,EAAE,CAAC,YAAY,CAAC;IAC/C,YAAY,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,IAAY,EAAE,EAAE;QACnC,MAAM,OAAO,GAAG,KAAK,CAAC,IAAI,CAAC,CAAA;QAC3B,IAAI,OAAO,KAAK,SAAS;YAAE,MAAM,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,WAAW,IAAI,EAAE,CAAC,EAAE,EAAE,IAAI,EAAE,QAAQ,EAAE,CAAC,CAAA;QAChG,OAAO,OAAO,CAAA;IAChB,CAAC,CAAC;IACF,aAAa,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,IAAY,EAAE,OAAe,EAAE,EAAE;QACrD,KAAK,CAAC,IAAI,CAAC,GAAG,OAAO,CAAA;IACvB,CAAC,CAAC;IACF,UAAU,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,GAAW,EAAE,GAAW,EAAE,EAAE;QAC7C,MAAM,OAAO,GAAG,KAAK,CAAC,GAAG,CAAC,CAAA;QAC1B,IAAI,OAAO,KAAK,SAAS;YAAE,MAAM,IAAI,KAAK,CAAC,WAAW,GAAG,EAAE,CAAC,CAAA;QAC5D,KAAK,CAAC,GAAG,CAAC,GAAG,OAAO,CAAA;QACpB,OAAO,KAAK,CAAC,GAAG,CAAC,CAAA;IACnB,CAAC,CAAC;IACF,SAAS,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC;IACjC,SAAS,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC;CAClC,CAAC,CAAC,CAAA;AAEH,OAAO,EACL,kBAAkB,EAClB,mBAAmB,EACnB,sBAAsB,EACtB,mBAAmB,EACnB,kBAAkB,GAEnB,MAAM,wBAAwB,CAAA;AAE/B,8EAA8E;AAC9E,UAAU;AACV,8EAA8E;AAE9E,MAAM,SAAS,GAAG,iDAAiD,CAAA;AAEnE,SAAS,aAAa;IACpB,OAAO,EAAE,CAAA;AACX,CAAC;AAED,SAAS,2BAA2B;IAClC,OAAO;QACL,KAAK,EAAE;YACL,UAAU,EAAE;gBACV;oBACE,OAAO,EAAE,OAAO;oBAChB,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,OAAO,EAAE,GAAG,SAAS,MAAM,EAAE,CAAC;iBAC1D;aACF;YACD,WAAW,EAAE;gBACX;oBACE,OAAO,EAAE,OAAO;oBAChB,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,OAAO,EAAE,GAAG,SAAS,OAAO,EAAE,CAAC;iBAC3D;aACF;SACF;KACF,CAAA;AACH,CAAC;AAED,SAAS,4BAA4B;IACnC,OAAO;QACL,KAAK,EAAE;YACL,UAAU,EAAE;gBACV;oBACE,OAAO,EAAE,OAAO;oBAChB,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,OAAO,EAAE,+BAA+B,EAAE,CAAC;iBACvE;aACF;YACD,WAAW,EAAE,EAAE;SAChB;KACF,CAAA;AACH,CAAC;AAED,8EAA8E;AAC9E,mBAAmB;AACnB,8EAA8E;AAE9E,UAAU,CAAC,GAAG,EAAE;IACd,KAAK,MAAM,CAAC,IAAI,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC,CAAC,CAAC,CAAA;IACnD,YAAY,GAAG,KAAK,CAAA;IACpB,EAAE,CAAC,aAAa,EAAE,CAAA;AACpB,CAAC,CAAC,CAAA;AAEF,SAAS,CAAC,GAAG,EAAE;IACb,KAAK,MAAM,CAAC,IAAI,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC,CAAC,CAAC,CAAA;AACrD,CAAC,CAAC,CAAA;AAEF,8EAA8E;AAC9E,qBAAqB;AACrB,8EAA8E;AAE9E,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;IAClC,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACvD,YAAY,GAAG,KAAK,CAAA;QACpB,MAAM,MAAM,GAAG,kBAAkB,CAAC,MAAM,CAAC,CAAA;QACzC,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC,EAAE,CAAC,CAAA;IAC5B,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QACjE,YAAY,GAAG,IAAI,CAAA;QACnB,MAAM,EAAE,UAAU,EAAE,YAAY,EAAE,GAAG,MAAM,MAAM,CAAC,SAAS,CAAC,CAC3D;QAAC,UAAuC,CAAC,mBAAmB,CAAC,IAAI,CAAC,CAClE;QAAC,YAAyC,CAAC,mBAAmB,CAC7D,IAAI,CAAC,SAAS,CAAC,EAAE,KAAK,EAAE,EAAE,UAAU,EAAE,EAAE,EAAE,EAAE,CAAC,CAC9C,CAAA;QACD,MAAM,MAAM,GAAG,kBAAkB,CAAC,MAAM,CAAC,CAAA;QACzC,MAAM,CAAC,MAAM,CAAC,CAAC,cAAc,CAAC,OAAO,CAAC,CAAA;IACxC,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,8EAA8E;AAC9E,sBAAsB;AACtB,8EAA8E;AAE9E,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,EAAE,CAAC,kEAAkE,EAAE,GAAG,EAAE;QAC1E,MAAM,MAAM,GAAG,mBAAmB,CAAC,aAAa,EAAE,EAAE,SAAS,CAAC,CAAA;QAE9D,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QAChD,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;QAC5D,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,GAAG,SAAS,MAAM,CAAC,CAAA;QAEnF,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,WAAW,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACjD,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,WAAW,EAAE,CAAC,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;QAC7D,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,WAAW,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,GAAG,SAAS,OAAO,CAAC,CAAA;IACvF,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qEAAqE,EAAE,GAAG,EAAE;QAC7E,MAAM,IAAI,GAAG,mBAAmB,CAAC,aAAa,EAAE,EAAE,SAAS,CAAC,CAAA;QAC5D,MAAM,KAAK,GAAG,mBAAmB,CAAC,IAAI,EAAE,SAAS,CAAC,CAAA;QAElD,MAAM,CAAC,KAAK,CAAC,KAAK,EAAE,UAAU,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QAC/C,MAAM,CAAC,KAAK,CAAC,KAAK,EAAE,WAAW,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;IAClD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qEAAqE,EAAE,GAAG,EAAE;QAC7E,MAAM,QAAQ,GAAmB;YAC/B,KAAK,EAAE;gBACL,UAAU,EAAE,CAAC,EAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,OAAO,EAAE,eAAe,EAAE,CAAC,EAAE,CAAC;gBACzF,WAAW,EAAE,EAAE;aAChB;SACF,CAAA;QACD,MAAM,MAAM,GAAG,mBAAmB,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAA;QAEvD,0CAA0C;QAC1C,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QAChD,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,EAAE,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;QAC9E,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,EAAE,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IACjF,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,iGAAiG,EAAE,GAAG,EAAE;QACzG,MAAM,QAAQ,GAAG,4BAA4B,EAAE,CAAA;QAC/C,MAAM,CAAC,GAAG,EAAE,CAAC,mBAAmB,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAA;QAClF,MAAM,CAAC,GAAG,EAAE,CAAC,mBAAmB,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAC,CAAC,OAAO,CAC5D,MAAM,CAAC,gBAAgB,CAAC,EAAE,IAAI,EAAE,4BAA4B,EAAE,CAAC,CAChE,CAAA;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,wEAAwE,EAAE,GAAG,EAAE;QAChF,MAAM,QAAQ,GAAG,4BAA4B,EAAE,CAAA;QAC/C,IAAI,MAAe,CAAA;QACnB,IAAI,CAAC;YACH,mBAAmB,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAA;QAC1C,CAAC;QAAC,OAAO,CAAC,EAAE,CAAC;YACX,MAAM,GAAG,CAAC,CAAA;QACZ,CAAC;QACD,MAAM,CAAC,MAAM,CAAC,CAAC,cAAc,CAAC,kBAAkB,CAAC,CAAA;QACjD,MAAM,GAAG,GAAG,MAA4B,CAAA;QACxC,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAA;QACnD,MAAM,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,mCAAmC,CAAC,CAAA;IACpE,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,8EAA8E;AAC9E,yBAAyB;AACzB,8EAA8E;AAE9E,QAAQ,CAAC,wBAAwB,EAAE,GAAG,EAAE;IACtC,EAAE,CAAC,uEAAuE,EAAE,GAAG,EAAE;QAC/E,MAAM,QAAQ,GAAG,2BAA2B,EAAE,CAAA;QAC9C,MAAM,MAAM,GAAG,sBAAsB,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAA;QAE1D,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QAChD,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,WAAW,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;IACnD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,+DAA+D,EAAE,GAAG,EAAE;QACvE,MAAM,QAAQ,GAAmB;YAC/B,KAAK,EAAE;gBACL,UAAU,EAAE;oBACV,EAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC,EAAE;oBACvE,EAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,OAAO,EAAE,GAAG,SAAS,MAAM,EAAE,CAAC,EAAE;iBAChF;gBACD,WAAW,EAAE;oBACX,EAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,OAAO,EAAE,GAAG,SAAS,OAAO,EAAE,CAAC,EAAE;iBACjF;aACF;SACF,CAAA;QACD,MAAM,MAAM,GAAG,sBAAsB,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAA;QAE1D,0CAA0C;QAC1C,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QAChD,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;QAC3D,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,WAAW,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;IACnD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,MAAM,MAAM,GAAG,sBAAsB,CAAC,EAAE,EAAE,SAAS,CAAC,CAAA;QACpD,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,aAAa,EAAE,CAAA;IACtC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,kDAAkD,EAAE,GAAG,EAAE;QAC1D,MAAM,QAAQ,GAAmB;YAC/B,KAAK,EAAE;gBACL,UAAU,EAAE,CAAC,EAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,OAAO,EAAE,WAAW,EAAE,CAAC,EAAE,CAAC;gBACrF,WAAW,EAAE,EAAE;aAChB;SACF,CAAA;QACD,MAAM,MAAM,GAAG,sBAAsB,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAA;QAC1D,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QAChD,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,WAAW,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;IACnD,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA;AAEF,8EAA8E;AAC9E,iDAAiD;AACjD,8EAA8E;AAE9E,QAAQ,CAAC,qBAAqB,EAAE,GAAG,EAAE;IACnC,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,MAAM,EAAE,aAAa,EAAE,UAAU,EAAE,GAAG,MAAM,MAAM,CAAC,SAAS,CAAC,CAAA;QAC7D,MAAM,QAAQ,GAAmB,EAAE,KAAK,EAAE,EAAE,UAAU,EAAE,EAAE,EAAE,EAAE,CAAA;QAC9D,mBAAmB,CAAC,MAAM,EAAE,QAAQ,CAAC,CAAA;QAErC,MAAM,CAAC,aAAa,CAAC,CAAC,oBAAoB,CACxC,MAAM,CAAC,gBAAgB,CAAC,MAAM,CAAC,EAC/B,MAAM,CAAC,gBAAgB,CAAC,cAAc,CAAC,EACvC,MAAM,CAAC,QAAQ,EAAE,CAClB,CAAA;QACD,MAAM,CAAC,UAAU,CAAC,CAAC,oBAAoB,CACrC,MAAM,CAAC,gBAAgB,CAAC,MAAM,CAAC,EAC/B,MAAM,CAAC,gBAAgB,CAAC,eAAe,CAAC,CACzC,CAAA;IACH,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;QACvE,2EAA2E;QAC3E,MAAM,KAAK,GAAmB,EAAE,KAAK,EAAE,EAAE,UAAU,EAAE,EAAE,EAAE,EAAE,CAAA;QAC3D,MAAM,MAAM,GAAmB;YAC7B,KAAK,EAAE;gBACL,UAAU,EAAE;oBACV,EAAE,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,OAAO,EAAE,GAAG,SAAS,MAAM,EAAE,CAAC,EAAE;iBAChF;aACF;SACF,CAAA;QAED,mBAAmB,CAAC,MAAM,EAAE,KAAK,CAAC,CAAA;QAClC,mBAAmB,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;QAEnC,qEAAqE;QACrE,MAAM,EAAE,mBAAmB,EAAE,GAAG,MAAM,MAAM,CAAC,wBAAwB,CAAC,CAAA;QACtE,MAAM,SAAS,GAAG,mBAAmB,CAAC,MAAM,CAAC,CAAA;QAC7C,MAAM,GAAG,GAAG,KAAK,CAAC,SAAS,CAAC,CAAA;QAC5B,MAAM,CAAC,GAAG,CAAC,CAAC,WAAW,EAAE,CAAA;QACzB,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,GAAI,CAAmB,CAAA;QACjD,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QAChD,MAAM,CAAC,MAAM,CAAC,KAAK,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;IAC9D,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/src/commands/telemetry.js

@@ -0,0 +1,65 @@
+/**
+ * @fileoverview `skillsmith telemetry` command group — opt-in telemetry management.
+ * @module @skillsmith/cli/commands/telemetry
+ * @see SMI-5021 Wave 3 Step 2 — CLI subcommands for telemetry (plan lines 195–196, 576)
+ * @see SMI-5128 batch D — action impls + run* logic moved to telemetry.action.ts
+ *   (sibling-split convention) so the 6 subcommand handlers are withTelemetry-
+ *   wrapped without this file exceeding the 500-LOC gate. This file retains the
+ *   commander factory and re-exports the run* test helpers for back-compat.
+ *
+ * Subcommands:
+ *   skillsmith telemetry enable
+ *   skillsmith telemetry disable
+ *   skillsmith telemetry status
+ *   skillsmith telemetry install-hook   [--scope user|project] [--endpoint <url>]
+ *   skillsmith telemetry uninstall-hook [--scope user|project]
+ *   skillsmith telemetry reset-id
+ *
+ * Privacy invariants (plan line 719):
+ *   - anonymousId is NEVER printed in full. Only the last 8 hex chars appear in stdout.
+ *   - The full SHA-256 hex travels only to the events endpoint in hook payloads.
+ *
+ * Shared-state coordination (plan line 717):
+ *   - install-hook reads then writes ~/.claude/settings.json via telemetry.helpers.ts
+ *   - Refuses on foreign Skill matcher (security gate)
+ *   - manifest read/write always via loadManifest / saveManifest (atomic rename)
+ */
+import { Command } from 'commander';
+import { telemetryEnableAction, telemetryDisableAction, telemetryStatusAction, telemetryInstallHookAction, telemetryUninstallHookAction, telemetryResetIdAction, } from './telemetry.action.js';
+// ---------------------------------------------------------------------------
+// Command factory
+// ---------------------------------------------------------------------------
+export function createTelemetryCommand() {
+    const telemetry = new Command('telemetry').description('Manage Skillsmith telemetry preferences and Claude Code hook installation');
+    telemetry
+        .command('enable')
+        .description('Opt in to anonymous skill-invocation telemetry')
+        .action(telemetryEnableAction);
+    telemetry
+        .command('disable')
+        .description('Opt out of telemetry (anonymous ID is retained for re-enable continuity)')
+        .action(telemetryDisableAction);
+    telemetry
+        .command('status')
+        .description('Show current telemetry state; triggers annual ID rotation if due')
+        .action(telemetryStatusAction);
+    telemetry
+        .command('install-hook')
+        .description('Install the Skill telemetry hook into ~/.claude/settings.json')
+        .option('--scope <scope>', 'Settings scope: user (~/.claude/settings.json) or project (./.claude/settings.json)', 'user')
+        .option('--endpoint <url>', 'Override the telemetry endpoint (default: prod Supabase events)')
+        .action(telemetryInstallHookAction);
+    telemetry
+        .command('uninstall-hook')
+        .description('Remove the Skillsmith Skill hook entries from settings.json')
+        .option('--scope <scope>', 'Settings scope: user or project', 'user')
+        .action(telemetryUninstallHookAction);
+    telemetry
+        .command('reset-id')
+        .description('Immediately rotate the anonymous ID (previous ID kept for 7-day overlap window)')
+        .action(telemetryResetIdAction);
+    return telemetry;
+}
+// Internal exports for tests — sourced from telemetry.action.ts (SMI-5128 split).
+export { runEnable, runDisable, runStatus, runInstallHook, runUninstallHook, runResetId, idTail, } from './telemetry.action.js';
+//# sourceMappingURL=telemetry.js.map

package/dist/src/commands/telemetry.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"telemetry.js","sourceRoot":"","sources":["../../../src/commands/telemetry.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAA;AAEnC,OAAO,EACL,qBAAqB,EACrB,sBAAsB,EACtB,qBAAqB,EACrB,0BAA0B,EAC1B,4BAA4B,EAC5B,sBAAsB,GACvB,MAAM,uBAAuB,CAAA;AAE9B,8EAA8E;AAC9E,kBAAkB;AAClB,8EAA8E;AAE9E,MAAM,UAAU,sBAAsB;IACpC,MAAM,SAAS,GAAG,IAAI,OAAO,CAAC,WAAW,CAAC,CAAC,WAAW,CACpD,2EAA2E,CAC5E,CAAA;IAED,SAAS;SACN,OAAO,CAAC,QAAQ,CAAC;SACjB,WAAW,CAAC,gDAAgD,CAAC;SAC7D,MAAM,CAAC,qBAAqB,CAAC,CAAA;IAEhC,SAAS;SACN,OAAO,CAAC,SAAS,CAAC;SAClB,WAAW,CAAC,0EAA0E,CAAC;SACvF,MAAM,CAAC,sBAAsB,CAAC,CAAA;IAEjC,SAAS;SACN,OAAO,CAAC,QAAQ,CAAC;SACjB,WAAW,CAAC,kEAAkE,CAAC;SAC/E,MAAM,CAAC,qBAAqB,CAAC,CAAA;IAEhC,SAAS;SACN,OAAO,CAAC,cAAc,CAAC;SACvB,WAAW,CAAC,+DAA+D,CAAC;SAC5E,MAAM,CACL,iBAAiB,EACjB,qFAAqF,EACrF,MAAM,CACP;SACA,MAAM,CAAC,kBAAkB,EAAE,iEAAiE,CAAC;SAC7F,MAAM,CAAC,0BAA0B,CAAC,CAAA;IAErC,SAAS;SACN,OAAO,CAAC,gBAAgB,CAAC;SACzB,WAAW,CAAC,6DAA6D,CAAC;SAC1E,MAAM,CAAC,iBAAiB,EAAE,iCAAiC,EAAE,MAAM,CAAC;SACpE,MAAM,CAAC,4BAA4B,CAAC,CAAA;IAEvC,SAAS;SACN,OAAO,CAAC,UAAU,CAAC;SACnB,WAAW,CAAC,iFAAiF,CAAC;SAC9F,MAAM,CAAC,sBAAsB,CAAC,CAAA;IAEjC,OAAO,SAAS,CAAA;AAClB,CAAC;AAED,kFAAkF;AAClF,OAAO,EACL,SAAS,EACT,UAAU,EACV,SAAS,EACT,cAAc,EACd,gBAAgB,EAChB,UAAU,EACV,MAAM,GACP,MAAM,uBAAuB,CAAA"}

package/dist/src/commands/telemetry.test.d.ts

@@ -0,0 +1,18 @@
+/**
+ * @fileoverview Tests for `skillsmith telemetry` subcommands.
+ * @module @skillsmith/cli/commands/telemetry.test
+ * @see SMI-5021 Wave 3 Step 2 — plan lines 700, 703, 706
+ *
+ * Covers:
+ *   enable: fresh manifest → generates id, sets enabled=true; idempotent on re-run
+ *   disable: enabled manifest → sets enabled=false; anonymousId preserved
+ *   status: prints expected shape (ID tail only); detects + triggers rotation when backdated >365d
+ *   reset-id: rotates unconditionally; new id distinct; previous id populated
+ *   install-hook (mocked): adds PreToolUse+PostToolUse; idempotent; throws on foreign Skill matcher
+ *   uninstall-hook: removes only Skillsmith entries; preserves foreign hooks
+ *   install-hook --scope project: writes to ./.claude/settings.json
+ *
+ * Privacy invariant: idTail() — last 8 chars only, never full id.
+ */
+export {};
+//# sourceMappingURL=telemetry.test.d.ts.map

package/dist/src/commands/telemetry.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"telemetry.test.d.ts","sourceRoot":"","sources":["../../../src/commands/telemetry.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG"}