@skillsmith/mcp-server 0.4.2 → 0.4.4

package/dist/src/tools/outdated.test.js

@@ -0,0 +1,287 @@
+/**
+ * @fileoverview Unit tests for skill_outdated MCP tool
+ * @see SMI-3138: Wave 5 — Dependency intelligence outdated tool
+ */
+import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
+import { SkillVersionRepository, SkillDependencyRepository } from '@skillsmith/core';
+import { createTestDatabase, closeDatabase } from '../../../core/tests/helpers/database.js';
+import { executeOutdated } from './outdated.js';
+// ============================================================================
+// Mocks
+// ============================================================================
+vi.mock('./install.helpers.js', () => ({
+    loadManifest: vi.fn(),
+}));
+vi.mock('./install.conflict-helpers.js', () => ({
+    hashContent: vi.fn((content) => {
+        // Simple deterministic mock hash
+        if (content === 'latest-content')
+            return 'aabbccdd11223344';
+        if (content === 'old-content')
+            return '11223344aabbccdd';
+        return '0000000000000000';
+    }),
+}));
+vi.mock('fs', async () => {
+    const actual = await vi.importActual('fs');
+    return {
+        ...actual,
+        promises: {
+            ...actual.promises,
+            readFile: vi.fn(),
+        },
+    };
+});
+import { loadManifest } from './install.helpers.js';
+import { promises as fs } from 'fs';
+const mockedLoadManifest = vi.mocked(loadManifest);
+const mockedReadFile = vi.mocked(fs.readFile);
+// ============================================================================
+// Helpers
+// ============================================================================
+function makeContext(db) {
+    return {
+        db,
+        skillDependencyRepository: new SkillDependencyRepository(db),
+    };
+}
+function emptyManifest() {
+    return { version: '1', installedSkills: {} };
+}
+function manifestWithSkills(skills) {
+    const installedSkills = {};
+    for (const s of skills) {
+        installedSkills[s.name] = {
+            id: s.id,
+            name: s.name,
+            version: '1.0.0',
+            source: 'registry',
+            installPath: s.installPath,
+            installedAt: '2026-01-01T00:00:00Z',
+            lastUpdated: '2026-01-01T00:00:00Z',
+        };
+    }
+    return { version: '1', installedSkills };
+}
+// ============================================================================
+// Tests
+// ============================================================================
+describe('executeOutdated', () => {
+    let db;
+    let versionRepo;
+    beforeEach(() => {
+        db = createTestDatabase();
+        versionRepo = new SkillVersionRepository(db);
+        vi.clearAllMocks();
+    });
+    afterEach(() => {
+        closeDatabase(db);
+    });
+    it('returns empty result when manifest has no installed skills', async () => {
+        mockedLoadManifest.mockResolvedValue(emptyManifest());
+        const result = await executeOutdated({ include_deps: true }, makeContext(db));
+        expect(result.skills).toHaveLength(0);
+        expect(result.summary.total_installed).toBe(0);
+        expect(result.summary.outdated).toBe(0);
+        expect(result.summary.up_to_date).toBe(0);
+        expect(result.summary.unknown).toBe(0);
+        expect(result.summary.missing_deps).toBe(0);
+    });
+    it('reports all skills as current when hashes match', async () => {
+        const skillId = 'community/test-skill';
+        mockedLoadManifest.mockResolvedValue(manifestWithSkills([
+            { id: skillId, name: 'test-skill', installPath: '/tmp/skills/test-skill' },
+        ]));
+        mockedReadFile.mockResolvedValue('latest-content');
+        // Insert a version record with the same hash as hashContent('latest-content')
+        await versionRepo.recordVersion(skillId, 'aabbccdd11223344', '1.2.0');
+        const result = await executeOutdated({ include_deps: true }, makeContext(db));
+        expect(result.skills).toHaveLength(1);
+        expect(result.skills[0].status).toBe('current');
+        expect(result.skills[0].installed_hash).toBe('aabbccdd');
+        expect(result.skills[0].latest_hash).toBe('aabbccdd');
+        expect(result.skills[0].semver).toBe('1.2.0');
+        expect(result.summary.up_to_date).toBe(1);
+        expect(result.summary.outdated).toBe(0);
+    });
+    it('reports skill as outdated when hashes differ', async () => {
+        const skillId = 'community/outdated-skill';
+        mockedLoadManifest.mockResolvedValue(manifestWithSkills([
+            { id: skillId, name: 'outdated-skill', installPath: '/tmp/skills/outdated-skill' },
+        ]));
+        // Local content is old → hashContent returns '11223344aabbccdd'
+        mockedReadFile.mockResolvedValue('old-content');
+        // Registry has a different hash
+        await versionRepo.recordVersion(skillId, 'aabbccdd11223344', '2.0.0');
+        const result = await executeOutdated({ include_deps: true }, makeContext(db));
+        expect(result.skills).toHaveLength(1);
+        expect(result.skills[0].status).toBe('outdated');
+        expect(result.skills[0].installed_hash).toBe('11223344');
+        expect(result.skills[0].latest_hash).toBe('aabbccdd');
+        expect(result.skills[0].semver).toBe('2.0.0');
+        expect(result.summary.outdated).toBe(1);
+        expect(result.summary.up_to_date).toBe(0);
+    });
+    it('reports unknown when no version history exists', async () => {
+        const skillId = 'community/new-skill';
+        mockedLoadManifest.mockResolvedValue(manifestWithSkills([{ id: skillId, name: 'new-skill', installPath: '/tmp/skills/new-skill' }]));
+        mockedReadFile.mockResolvedValue('latest-content');
+        // No version records in DB for this skill
+        const result = await executeOutdated({ include_deps: true }, makeContext(db));
+        expect(result.skills).toHaveLength(1);
+        expect(result.skills[0].status).toBe('unknown');
+        expect(result.summary.unknown).toBe(1);
+    });
+    it('includes dependency status when include_deps is true', async () => {
+        const skillId = 'community/dep-skill';
+        const depSkillId = 'community/required-skill';
+        mockedLoadManifest.mockResolvedValue(manifestWithSkills([
+            { id: skillId, name: 'dep-skill', installPath: '/tmp/skills/dep-skill' },
+            { id: depSkillId, name: 'required-skill', installPath: '/tmp/skills/required-skill' },
+        ]));
+        mockedReadFile.mockResolvedValue('latest-content');
+        await versionRepo.recordVersion(skillId, 'aabbccdd11223344', '1.0.0');
+        await versionRepo.recordVersion(depSkillId, 'aabbccdd11223344', '1.0.0');
+        // Add a dependency: dep-skill depends on required-skill (which IS installed)
+        const depRepo = new SkillDependencyRepository(db);
+        depRepo.setDependencies(skillId, [
+            {
+                skill_id: skillId,
+                dep_type: 'skill_hard',
+                dep_target: depSkillId,
+                dep_version: '*',
+                dep_source: 'declared',
+                confidence: 1.0,
+                metadata: null,
+            },
+        ], 'declared');
+        const result = await executeOutdated({ include_deps: true }, makeContext(db));
+        const depSkill = result.skills.find((s) => s.id === skillId);
+        expect(depSkill?.dependencies).toBeDefined();
+        expect(depSkill.dependencies.total).toBe(1);
+        expect(depSkill.dependencies.satisfied).toHaveLength(1);
+        expect(depSkill.dependencies.missing).toHaveLength(0);
+        expect(result.summary.missing_deps).toBe(0);
+    });
+    it('omits dependency status when include_deps is false', async () => {
+        const skillId = 'community/no-dep-check';
+        mockedLoadManifest.mockResolvedValue(manifestWithSkills([
+            { id: skillId, name: 'no-dep-check', installPath: '/tmp/skills/no-dep-check' },
+        ]));
+        mockedReadFile.mockResolvedValue('latest-content');
+        await versionRepo.recordVersion(skillId, 'aabbccdd11223344', '1.0.0');
+        const result = await executeOutdated({ include_deps: false }, makeContext(db));
+        expect(result.skills).toHaveLength(1);
+        expect(result.skills[0].dependencies).toBeUndefined();
+    });
+    it('counts missing deps in summary when a skill dep is not installed', async () => {
+        const skillId = 'community/lonely-skill';
+        mockedLoadManifest.mockResolvedValue(manifestWithSkills([
+            { id: skillId, name: 'lonely-skill', installPath: '/tmp/skills/lonely-skill' },
+        ]));
+        mockedReadFile.mockResolvedValue('latest-content');
+        await versionRepo.recordVersion(skillId, 'aabbccdd11223344', '1.0.0');
+        // Add a dependency on a skill that is NOT installed
+        const depRepo = new SkillDependencyRepository(db);
+        depRepo.setDependencies(skillId, [
+            {
+                skill_id: skillId,
+                dep_type: 'skill_hard',
+                dep_target: 'community/missing-skill',
+                dep_version: '*',
+                dep_source: 'declared',
+                confidence: 1.0,
+                metadata: null,
+            },
+        ], 'declared');
+        const result = await executeOutdated({ include_deps: true }, makeContext(db));
+        const skill = result.skills[0];
+        expect(skill.dependencies.missing).toHaveLength(1);
+        expect(skill.dependencies.missing[0]).toContain('missing-skill');
+        expect(result.summary.missing_deps).toBe(1);
+    });
+    // ===========================================================================
+    // SMI-3177: Corrupt manifest entries (missing installPath)
+    // ===========================================================================
+    it('handles manifest entry with missing installPath gracefully', async () => {
+        // Simulate corrupt manifest entry (runtime JSON, not type-checked)
+        const corruptManifest = {
+            version: '1',
+            installedSkills: {
+                'test-skill': {
+                    id: 'test/test-skill',
+                    name: 'test-skill',
+                    version: '1.0.0',
+                    source: 'registry',
+                    installedAt: '2026-01-01T00:00:00Z',
+                    lastUpdated: '2026-01-01T00:00:00Z',
+                }, // Cast to bypass TS required field
+            },
+        };
+        mockedLoadManifest.mockResolvedValue(corruptManifest);
+        const result = await executeOutdated({ include_deps: true }, makeContext(db));
+        expect(result.skills).toHaveLength(1);
+        expect(result.skills[0].status).toBe('unknown');
+        expect(result.skills[0].installed_hash).toBe('--------');
+        expect(result.skills[0].id).toBe('test/test-skill');
+        expect(result.skills[0].dependencies).toEqual({ total: 0, satisfied: [], missing: [] });
+        expect(result.summary.unknown).toBe(1);
+        expect(result.summary.total_installed).toBe(1);
+    });
+    it('processes valid entries alongside corrupt entries', async () => {
+        const manifest = {
+            version: '1',
+            installedSkills: {
+                'good-skill': {
+                    id: 'community/good-skill',
+                    name: 'good-skill',
+                    version: '1.0.0',
+                    source: 'registry',
+                    installPath: '/tmp/skills/good-skill',
+                    installedAt: '2026-01-01T00:00:00Z',
+                    lastUpdated: '2026-01-01T00:00:00Z',
+                },
+                'bad-skill': {
+                    id: 'test/bad-skill',
+                    name: 'bad-skill',
+                    version: '1.0.0',
+                    source: 'registry',
+                    installedAt: '2026-01-01T00:00:00Z',
+                    lastUpdated: '2026-01-01T00:00:00Z',
+                },
+            },
+        };
+        mockedLoadManifest.mockResolvedValue(manifest);
+        mockedReadFile.mockResolvedValue('latest-content');
+        await versionRepo.recordVersion('community/good-skill', 'aabbccdd11223344', '1.0.0');
+        const result = await executeOutdated({ include_deps: true }, makeContext(db));
+        expect(result.skills).toHaveLength(2);
+        expect(result.summary.total_installed).toBe(2);
+        const good = result.skills.find((s) => s.id === 'community/good-skill');
+        const bad = result.skills.find((s) => s.id === 'test/bad-skill');
+        expect(good?.status).toBe('current');
+        expect(bad?.status).toBe('unknown');
+        expect(bad?.installed_hash).toBe('--------');
+    });
+    it('handles corrupt entry with include_deps false', async () => {
+        const corruptManifest = {
+            version: '1',
+            installedSkills: {
+                broken: {
+                    id: 'test/broken',
+                    name: 'broken',
+                    version: '1.0.0',
+                    source: 'registry',
+                    installedAt: '2026-01-01T00:00:00Z',
+                    lastUpdated: '2026-01-01T00:00:00Z',
+                },
+            },
+        };
+        mockedLoadManifest.mockResolvedValue(corruptManifest);
+        const result = await executeOutdated({ include_deps: false }, makeContext(db));
+        expect(result.skills).toHaveLength(1);
+        expect(result.skills[0].status).toBe('unknown');
+        expect(result.skills[0].dependencies).toBeUndefined();
+    });
+});
+//# sourceMappingURL=outdated.test.js.map

package/dist/src/tools/outdated.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"outdated.test.js","sourceRoot":"","sources":["../../../src/tools/outdated.test.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAA;AACxE,OAAO,EAAE,sBAAsB,EAAE,yBAAyB,EAAE,MAAM,kBAAkB,CAAA;AACpF,OAAO,EAAE,kBAAkB,EAAE,aAAa,EAAE,MAAM,yCAAyC,CAAA;AAC3F,OAAO,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AAK/C,+EAA+E;AAC/E,QAAQ;AACR,+EAA+E;AAE/E,EAAE,CAAC,IAAI,CAAC,sBAAsB,EAAE,GAAG,EAAE,CAAC,CAAC;IACrC,YAAY,EAAE,EAAE,CAAC,EAAE,EAAE;CACtB,CAAC,CAAC,CAAA;AAEH,EAAE,CAAC,IAAI,CAAC,+BAA+B,EAAE,GAAG,EAAE,CAAC,CAAC;IAC9C,WAAW,EAAE,EAAE,CAAC,EAAE,CAAC,CAAC,OAAe,EAAE,EAAE;QACrC,iCAAiC;QACjC,IAAI,OAAO,KAAK,gBAAgB;YAAE,OAAO,kBAAkB,CAAA;QAC3D,IAAI,OAAO,KAAK,aAAa;YAAE,OAAO,kBAAkB,CAAA;QACxD,OAAO,kBAAkB,CAAA;IAC3B,CAAC,CAAC;CACH,CAAC,CAAC,CAAA;AAEH,EAAE,CAAC,IAAI,CAAC,IAAI,EAAE,KAAK,IAAI,EAAE;IACvB,MAAM,MAAM,GAAG,MAAM,EAAE,CAAC,YAAY,CAAsB,IAAI,CAAC,CAAA;IAC/D,OAAO;QACL,GAAG,MAAM;QACT,QAAQ,EAAE;YACR,GAAG,MAAM,CAAC,QAAQ;YAClB,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;SAClB;KACF,CAAA;AACH,CAAC,CAAC,CAAA;AAEF,OAAO,EAAE,YAAY,EAAE,MAAM,sBAAsB,CAAA;AACnD,OAAO,EAAE,QAAQ,IAAI,EAAE,EAAE,MAAM,IAAI,CAAA;AAEnC,MAAM,kBAAkB,GAAG,EAAE,CAAC,MAAM,CAAC,YAAY,CAAC,CAAA;AAClD,MAAM,cAAc,GAAG,EAAE,CAAC,MAAM,CAAC,EAAE,CAAC,QAAQ,CAAC,CAAA;AAE7C,+EAA+E;AAC/E,UAAU;AACV,+EAA+E;AAE/E,SAAS,WAAW,CAAC,EAAY;IAC/B,OAAO;QACL,EAAE;QACF,yBAAyB,EAAE,IAAI,yBAAyB,CAAC,EAAE,CAAC;KACnC,CAAA;AAC7B,CAAC;AAED,SAAS,aAAa;IACpB,OAAO,EAAE,OAAO,EAAE,GAAG,EAAE,eAAe,EAAE,EAAE,EAAE,CAAA;AAC9C,CAAC;AAED,SAAS,kBAAkB,CACzB,MAAgE;IAEhE,MAAM,eAAe,GAAqC,EAAE,CAAA;IAC5D,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;QACvB,eAAe,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG;YACxB,EAAE,EAAE,CAAC,CAAC,EAAE;YACR,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,OAAO,EAAE,OAAO;YAChB,MAAM,EAAE,UAAU;YAClB,WAAW,EAAE,CAAC,CAAC,WAAW;YAC1B,WAAW,EAAE,sBAAsB;YACnC,WAAW,EAAE,sBAAsB;SACpC,CAAA;IACH,CAAC;IACD,OAAO,EAAE,OAAO,EAAE,GAAG,EAAE,eAAe,EAAE,CAAA;AAC1C,CAAC;AAED,+EAA+E;AAC/E,QAAQ;AACR,+EAA+E;AAE/E,QAAQ,CAAC,iBAAiB,EAAE,GAAG,EAAE;IAC/B,IAAI,EAAY,CAAA;IAChB,IAAI,WAAmC,CAAA;IAEvC,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,GAAG,kBAAkB,EAAE,CAAA;QACzB,WAAW,GAAG,IAAI,sBAAsB,CAAC,EAAE,CAAC,CAAA;QAC5C,EAAE,CAAC,aAAa,EAAE,CAAA;IACpB,CAAC,CAAC,CAAA;IAEF,SAAS,CAAC,GAAG,EAAE;QACb,aAAa,CAAC,EAAE,CAAC,CAAA;IACnB,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;QAC1E,kBAAkB,CAAC,iBAAiB,CAAC,aAAa,EAAE,CAAC,CAAA;QAErD,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,EAAE,YAAY,EAAE,IAAI,EAAE,EAAE,WAAW,CAAC,EAAE,CAAC,CAAC,CAAA;QAE7E,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACrC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC9C,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACvC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACzC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACtC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IAC7C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,MAAM,OAAO,GAAG,sBAAsB,CAAA;QACtC,kBAAkB,CAAC,iBAAiB,CAClC,kBAAkB,CAAC;YACjB,EAAE,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,YAAY,EAAE,WAAW,EAAE,wBAAwB,EAAE;SAC3E,CAAC,CACH,CAAA;QAED,cAAc,CAAC,iBAAiB,CAAC,gBAAgB,CAAC,CAAA;QAElD,8EAA8E;QAC9E,MAAM,WAAW,CAAC,aAAa,CAAC,OAAO,EAAE,kBAAkB,EAAE,OAAO,CAAC,CAAA;QAErE,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,EAAE,YAAY,EAAE,IAAI,EAAE,EAAE,WAAW,CAAC,EAAE,CAAC,CAAC,CAAA;QAE7E,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACrC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QAC/C,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;QACxD,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;QACrD,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;QAC7C,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACzC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACzC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,MAAM,OAAO,GAAG,0BAA0B,CAAA;QAC1C,kBAAkB,CAAC,iBAAiB,CAClC,kBAAkB,CAAC;YACjB,EAAE,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,gBAAgB,EAAE,WAAW,EAAE,4BAA4B,EAAE;SACnF,CAAC,CACH,CAAA;QAED,gEAAgE;QAChE,cAAc,CAAC,iBAAiB,CAAC,aAAa,CAAC,CAAA;QAE/C,gCAAgC;QAChC,MAAM,WAAW,CAAC,aAAa,CAAC,OAAO,EAAE,kBAAkB,EAAE,OAAO,CAAC,CAAA;QAErE,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,EAAE,YAAY,EAAE,IAAI,EAAE,EAAE,WAAW,CAAC,EAAE,CAAC,CAAC,CAAA;QAE7E,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACrC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;QAChD,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;QACxD,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;QACrD,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;QAC7C,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACvC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IAC3C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAC9D,MAAM,OAAO,GAAG,qBAAqB,CAAA;QACrC,kBAAkB,CAAC,iBAAiB,CAClC,kBAAkB,CAAC,CAAC,EAAE,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,WAAW,EAAE,uBAAuB,EAAE,CAAC,CAAC,CAC/F,CAAA;QACD,cAAc,CAAC,iBAAiB,CAAC,gBAAgB,CAAC,CAAA;QAElD,0CAA0C;QAE1C,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,EAAE,YAAY,EAAE,IAAI,EAAE,EAAE,WAAW,CAAC,EAAE,CAAC,CAAC,CAAA;QAE7E,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACrC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QAC/C,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IACxC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;QACpE,MAAM,OAAO,GAAG,qBAAqB,CAAA;QACrC,MAAM,UAAU,GAAG,0BAA0B,CAAA;QAC7C,kBAAkB,CAAC,iBAAiB,CAClC,kBAAkB,CAAC;YACjB,EAAE,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,WAAW,EAAE,uBAAuB,EAAE;YACxE,EAAE,EAAE,EAAE,UAAU,EAAE,IAAI,EAAE,gBAAgB,EAAE,WAAW,EAAE,4BAA4B,EAAE;SACtF,CAAC,CACH,CAAA;QACD,cAAc,CAAC,iBAAiB,CAAC,gBAAgB,CAAC,CAAA;QAElD,MAAM,WAAW,CAAC,aAAa,CAAC,OAAO,EAAE,kBAAkB,EAAE,OAAO,CAAC,CAAA;QACrE,MAAM,WAAW,CAAC,aAAa,CAAC,UAAU,EAAE,kBAAkB,EAAE,OAAO,CAAC,CAAA;QAExE,6EAA6E;QAC7E,MAAM,OAAO,GAAG,IAAI,yBAAyB,CAAC,EAAE,CAAC,CAAA;QACjD,OAAO,CAAC,eAAe,CACrB,OAAO,EACP;YACE;gBACE,QAAQ,EAAE,OAAO;gBACjB,QAAQ,EAAE,YAAY;gBACtB,UAAU,EAAE,UAAU;gBACtB,WAAW,EAAE,GAAG;gBAChB,UAAU,EAAE,UAAU;gBACtB,UAAU,EAAE,GAAG;gBACf,QAAQ,EAAE,IAAI;aACf;SACF,EACD,UAAU,CACX,CAAA;QAED,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,EAAE,YAAY,EAAE,IAAI,EAAE,EAAE,WAAW,CAAC,EAAE,CAAC,CAAC,CAAA;QAE7E,MAAM,QAAQ,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,OAAO,CAAC,CAAA;QAC5D,MAAM,CAAC,QAAQ,EAAE,YAAY,CAAC,CAAC,WAAW,EAAE,CAAA;QAC5C,MAAM,CAAC,QAAS,CAAC,YAAa,CAAC,KAAK,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAC7C,MAAM,CAAC,QAAS,CAAC,YAAa,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACzD,MAAM,CAAC,QAAS,CAAC,YAAa,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACvD,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IAC7C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;QAClE,MAAM,OAAO,GAAG,wBAAwB,CAAA;QACxC,kBAAkB,CAAC,iBAAiB,CAClC,kBAAkB,CAAC;YACjB,EAAE,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,cAAc,EAAE,WAAW,EAAE,0BAA0B,EAAE;SAC/E,CAAC,CACH,CAAA;QACD,cAAc,CAAC,iBAAiB,CAAC,gBAAgB,CAAC,CAAA;QAElD,MAAM,WAAW,CAAC,aAAa,CAAC,OAAO,EAAE,kBAAkB,EAAE,OAAO,CAAC,CAAA;QAErE,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,EAAE,YAAY,EAAE,KAAK,EAAE,EAAE,WAAW,CAAC,EAAE,CAAC,CAAC,CAAA;QAE9E,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACrC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,aAAa,EAAE,CAAA;IACvD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,kEAAkE,EAAE,KAAK,IAAI,EAAE;QAChF,MAAM,OAAO,GAAG,wBAAwB,CAAA;QACxC,kBAAkB,CAAC,iBAAiB,CAClC,kBAAkB,CAAC;YACjB,EAAE,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,cAAc,EAAE,WAAW,EAAE,0BAA0B,EAAE;SAC/E,CAAC,CACH,CAAA;QACD,cAAc,CAAC,iBAAiB,CAAC,gBAAgB,CAAC,CAAA;QAElD,MAAM,WAAW,CAAC,aAAa,CAAC,OAAO,EAAE,kBAAkB,EAAE,OAAO,CAAC,CAAA;QAErE,oDAAoD;QACpD,MAAM,OAAO,GAAG,IAAI,yBAAyB,CAAC,EAAE,CAAC,CAAA;QACjD,OAAO,CAAC,eAAe,CACrB,OAAO,EACP;YACE;gBACE,QAAQ,EAAE,OAAO;gBACjB,QAAQ,EAAE,YAAY;gBACtB,UAAU,EAAE,yBAAyB;gBACrC,WAAW,EAAE,GAAG;gBAChB,UAAU,EAAE,UAAU;gBACtB,UAAU,EAAE,GAAG;gBACf,QAAQ,EAAE,IAAI;aACf;SACF,EACD,UAAU,CACX,CAAA;QAED,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,EAAE,YAAY,EAAE,IAAI,EAAE,EAAE,WAAW,CAAC,EAAE,CAAC,CAAC,CAAA;QAE7E,MAAM,KAAK,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAA;QAC9B,MAAM,CAAC,KAAK,CAAC,YAAa,CAAC,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACnD,MAAM,CAAC,KAAK,CAAC,YAAa,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,eAAe,CAAC,CAAA;QACjE,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IAC7C,CAAC,CAAC,CAAA;IAEF,8EAA8E;IAC9E,2DAA2D;IAC3D,8EAA8E;IAE9E,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;QAC1E,mEAAmE;QACnE,MAAM,eAAe,GAAkB;YACrC,OAAO,EAAE,GAAG;YACZ,eAAe,EAAE;gBACf,YAAY,EAAE;oBACZ,EAAE,EAAE,iBAAiB;oBACrB,IAAI,EAAE,YAAY;oBAClB,OAAO,EAAE,OAAO;oBAChB,MAAM,EAAE,UAAU;oBAClB,WAAW,EAAE,sBAAsB;oBACnC,WAAW,EAAE,sBAAsB;iBACd,EAAE,mCAAmC;aAC7D;SACF,CAAA;QACD,kBAAkB,CAAC,iBAAiB,CAAC,eAAe,CAAC,CAAA;QAErD,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,EAAE,YAAY,EAAE,IAAI,EAAE,EAAE,WAAW,CAAC,EAAE,CAAC,CAAC,CAAA;QAE7E,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACrC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QAC/C,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;QACxD,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAA;QACnD,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC,EAAE,KAAK,EAAE,CAAC,EAAE,SAAS,EAAE,EAAE,EAAE,OAAO,EAAE,EAAE,EAAE,CAAC,CAAA;QACvF,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QACtC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;IAChD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QACjE,MAAM,QAAQ,GAAkB;YAC9B,OAAO,EAAE,GAAG;YACZ,eAAe,EAAE;gBACf,YAAY,EAAE;oBACZ,EAAE,EAAE,sBAAsB;oBAC1B,IAAI,EAAE,YAAY;oBAClB,OAAO,EAAE,OAAO;oBAChB,MAAM,EAAE,UAAU;oBAClB,WAAW,EAAE,wBAAwB;oBACrC,WAAW,EAAE,sBAAsB;oBACnC,WAAW,EAAE,sBAAsB;iBACpC;gBACD,WAAW,EAAE;oBACX,EAAE,EAAE,gBAAgB;oBACpB,IAAI,EAAE,WAAW;oBACjB,OAAO,EAAE,OAAO;oBAChB,MAAM,EAAE,UAAU;oBAClB,WAAW,EAAE,sBAAsB;oBACnC,WAAW,EAAE,sBAAsB;iBACd;aACxB;SACF,CAAA;QACD,kBAAkB,CAAC,iBAAiB,CAAC,QAAQ,CAAC,CAAA;QAC9C,cAAc,CAAC,iBAAiB,CAAC,gBAAgB,CAAC,CAAA;QAElD,MAAM,WAAW,CAAC,aAAa,CAAC,sBAAsB,EAAE,kBAAkB,EAAE,OAAO,CAAC,CAAA;QAEpF,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,EAAE,YAAY,EAAE,IAAI,EAAE,EAAE,WAAW,CAAC,EAAE,CAAC,CAAC,CAAA;QAE7E,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACrC,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAA;QAE9C,MAAM,IAAI,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,sBAAsB,CAAC,CAAA;QACvE,MAAM,GAAG,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,gBAAgB,CAAC,CAAA;QAEhE,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QACpC,MAAM,CAAC,GAAG,EAAE,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QACnC,MAAM,CAAC,GAAG,EAAE,cAAc,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAA;IAC9C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,+CAA+C,EAAE,KAAK,IAAI,EAAE;QAC7D,MAAM,eAAe,GAAkB;YACrC,OAAO,EAAE,GAAG;YACZ,eAAe,EAAE;gBACf,MAAM,EAAE;oBACN,EAAE,EAAE,aAAa;oBACjB,IAAI,EAAE,QAAQ;oBACd,OAAO,EAAE,OAAO;oBAChB,MAAM,EAAE,UAAU;oBAClB,WAAW,EAAE,sBAAsB;oBACnC,WAAW,EAAE,sBAAsB;iBACd;aACxB;SACF,CAAA;QACD,kBAAkB,CAAC,iBAAiB,CAAC,eAAe,CAAC,CAAA;QAErD,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,EAAE,YAAY,EAAE,KAAK,EAAE,EAAE,WAAW,CAAC,EAAE,CAAC,CAAC,CAAA;QAE9E,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAA;QACrC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QAC/C,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,aAAa,EAAE,CAAA;IACvD,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/src/tools/recommend.d.ts

@@ -5,10 +5,12 @@
  * @see SMI-602: Integrate semantic matching with EmbeddingService
  * @see SMI-604: Add trigger phrase overlap detection
  * @see SMI-1837: Include local skills in recommendations (parallel search)
+ * @see SMI-2741: Split to meet 500-line standard
  */
 import type { ToolContext } from '../context.js';
 import { type RecommendInput, type RecommendResponse } from './recommend.types.js';
 export { recommendInputSchema, recommendToolSchema, type RecommendInput, type RecommendResponse, type SkillRecommendation, } from './recommend.types.js';
+export { formatRecommendations, mergeAndDeduplicateRecommendations } from './recommend.format.js';
 /**
  * Execute skill recommendation based on installed skills and context.
  *
@@ -17,8 +19,4 @@ export { recommendInputSchema, recommendToolSchema, type RecommendInput, type Re
  * - Falls back to local semantic matching if API is offline or fails
  */
 export declare function executeRecommend(input: RecommendInput, context: ToolContext): Promise<RecommendResponse>;
-/**
- * Format recommendations for terminal display
- */
-export declare function formatRecommendations(response: RecommendResponse): string;
 //# sourceMappingURL=recommend.d.ts.map

package/dist/src/tools/recommend.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"recommend.d.ts","sourceRoot":"","sources":["../../../src/tools/recommend.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAGH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,eAAe,CAAA;AAKhD,OAAO,EAEL,KAAK,cAAc,EACnB,KAAK,iBAAiB,EAGvB,MAAM,sBAAsB,CAAA;AAc7B,OAAO,EACL,oBAAoB,EACpB,mBAAmB,EACnB,KAAK,cAAc,EACnB,KAAK,iBAAiB,EACtB,KAAK,mBAAmB,GACzB,MAAM,sBAAsB,CAAA;AAsH7B;;;;;;GAMG;AACH,wBAAsB,gBAAgB,CACpC,KAAK,EAAE,cAAc,EACrB,OAAO,EAAE,WAAW,GACnB,OAAO,CAAC,iBAAiB,CAAC,CA0T5B;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,QAAQ,EAAE,iBAAiB,GAAG,MAAM,CAyDzE"}
+{"version":3,"file":"recommend.d.ts","sourceRoot":"","sources":["../../../src/tools/recommend.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAGH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,eAAe,CAAA;AAKhD,OAAO,EAEL,KAAK,cAAc,EACnB,KAAK,iBAAiB,EAGvB,MAAM,sBAAsB,CAAA;AAiB7B,OAAO,EACL,oBAAoB,EACpB,mBAAmB,EACnB,KAAK,cAAc,EACnB,KAAK,iBAAiB,EACtB,KAAK,mBAAmB,GACzB,MAAM,sBAAsB,CAAA;AAG7B,OAAO,EAAE,qBAAqB,EAAE,kCAAkC,EAAE,MAAM,uBAAuB,CAAA;AA0DjG;;;;;;GAMG;AACH,wBAAsB,gBAAgB,CACpC,KAAK,EAAE,cAAc,EACrB,OAAO,EAAE,WAAW,GACnB,OAAO,CAAC,iBAAiB,CAAC,CA0T5B"}

package/dist/src/tools/recommend.format.d.ts

@@ -0,0 +1,28 @@
+/**
+ * @fileoverview Recommendation Formatting and Deduplication Utilities
+ * @module @skillsmith/mcp-server/tools/recommend.format
+ * @see SMI-2741: Split from recommend.ts to meet 500-line standard
+ *
+ * Standalone utilities extracted from executeRecommend:
+ * - mergeAndDeduplicateRecommendations: Merge API and local results, removing duplicates
+ * - formatRecommendations: Format recommendation response for terminal display
+ */
+import type { SkillRecommendation, RecommendResponse } from './recommend.types.js';
+/**
+ * Merge and deduplicate API and local skill recommendations.
+ * API results take priority over local results with the same name.
+ *
+ * @param apiResults - Results from API
+ * @param localResults - Results from local skill search
+ * @param limit - Maximum combined results
+ * @returns Merged and deduplicated recommendations
+ */
+export declare function mergeAndDeduplicateRecommendations(apiResults: SkillRecommendation[], localResults: SkillRecommendation[], limit: number): SkillRecommendation[];
+/**
+ * Format recommendations for terminal display
+ *
+ * @param response - Recommendation response to format
+ * @returns Formatted string for terminal output
+ */
+export declare function formatRecommendations(response: RecommendResponse): string;
+//# sourceMappingURL=recommend.format.d.ts.map

package/dist/src/tools/recommend.format.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"recommend.format.d.ts","sourceRoot":"","sources":["../../../src/tools/recommend.format.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAGH,OAAO,KAAK,EAAE,mBAAmB,EAAE,iBAAiB,EAAE,MAAM,sBAAsB,CAAA;AAElF;;;;;;;;GAQG;AACH,wBAAgB,kCAAkC,CAChD,UAAU,EAAE,mBAAmB,EAAE,EACjC,YAAY,EAAE,mBAAmB,EAAE,EACnC,KAAK,EAAE,MAAM,GACZ,mBAAmB,EAAE,CA8CvB;AAED;;;;;GAKG;AACH,wBAAgB,qBAAqB,CAAC,QAAQ,EAAE,iBAAiB,GAAG,MAAM,CAyDzE"}

package/dist/src/tools/recommend.format.js

@@ -0,0 +1,111 @@
+/**
+ * @fileoverview Recommendation Formatting and Deduplication Utilities
+ * @module @skillsmith/mcp-server/tools/recommend.format
+ * @see SMI-2741: Split from recommend.ts to meet 500-line standard
+ *
+ * Standalone utilities extracted from executeRecommend:
+ * - mergeAndDeduplicateRecommendations: Merge API and local results, removing duplicates
+ * - formatRecommendations: Format recommendation response for terminal display
+ */
+import { getTrustBadge } from '../utils/validation.js';
+/**
+ * Merge and deduplicate API and local skill recommendations.
+ * API results take priority over local results with the same name.
+ *
+ * @param apiResults - Results from API
+ * @param localResults - Results from local skill search
+ * @param limit - Maximum combined results
+ * @returns Merged and deduplicated recommendations
+ */
+export function mergeAndDeduplicateRecommendations(apiResults, localResults, limit) {
+    // Build a Set of names from API results for deduplication
+    const apiSkillNames = new Set(apiResults.map((r) => r.name.toLowerCase()));
+    // Also track skill IDs (without the author prefix)
+    const apiSkillIdNames = new Set(apiResults.map((r) => r.skill_id.split('/').pop()?.toLowerCase() || ''));
+    // Filter local results to exclude duplicates
+    const uniqueLocalResults = localResults.filter((local) => {
+        const localName = local.name.toLowerCase();
+        const localIdName = local.skill_id.split('/').pop()?.toLowerCase() || '';
+        // Exclude if name matches an API result
+        if (apiSkillNames.has(localName)) {
+            return false;
+        }
+        // Exclude if ID name matches an API result
+        if (apiSkillIdNames.has(localIdName)) {
+            return false;
+        }
+        // Exclude if local name is contained in or contains an API skill name
+        for (const apiName of apiSkillNames) {
+            if (localName.includes(apiName) || apiName.includes(localName)) {
+                return false;
+            }
+        }
+        return true;
+    });
+    // Combine API results first (higher priority), then unique local results
+    const combined = [...apiResults, ...uniqueLocalResults];
+    // Sort by quality score descending, then by similarity score
+    combined.sort((a, b) => {
+        if (b.quality_score !== a.quality_score) {
+            return b.quality_score - a.quality_score;
+        }
+        return b.similarity_score - a.similarity_score;
+    });
+    return combined.slice(0, limit);
+}
+/**
+ * Format recommendations for terminal display
+ *
+ * @param response - Recommendation response to format
+ * @returns Formatted string for terminal output
+ */
+export function formatRecommendations(response) {
+    const lines = [];
+    lines.push('\n=== Skill Recommendations ===\n');
+    if (response.recommendations.length === 0) {
+        lines.push('No recommendations found.');
+        lines.push('');
+        lines.push('Suggestions:');
+        lines.push('  - Try adding more installed skills for better matching');
+        lines.push('  - Provide a project context for more relevant results');
+        // SMI-1631: Suggest removing role filter if one was applied
+        if (response.context.role_filter) {
+            lines.push(`  - Try removing the role filter (currently: ${response.context.role_filter})`);
+        }
+    }
+    else {
+        lines.push(`Found ${response.recommendations.length} recommendation(s):\n`);
+        response.recommendations.forEach((rec, index) => {
+            const trustBadge = getTrustBadge(rec.trust_tier);
+            // SMI-1631: Show roles if present
+            const rolesDisplay = rec.roles?.length ? ` [${rec.roles.join(', ')}]` : '';
+            lines.push(`${index + 1}. ${rec.name} ${trustBadge}${rolesDisplay}`);
+            lines.push(`   Score: ${rec.quality_score}/100 | Relevance: ${Math.round(rec.similarity_score * 100)}%`);
+            lines.push(`   ${rec.reason}`);
+            lines.push(`   ID: ${rec.skill_id}`);
+            lines.push('');
+        });
+    }
+    lines.push('---');
+    lines.push(`Candidates considered: ${response.candidates_considered}`);
+    if (response.overlap_filtered > 0) {
+        lines.push(`Filtered for overlap: ${response.overlap_filtered}`);
+    }
+    // SMI-1631: Show role filter stats
+    if (response.role_filtered > 0) {
+        lines.push(`Filtered for role: ${response.role_filtered}`);
+    }
+    if (response.context.role_filter) {
+        lines.push(`Role filter: ${response.context.role_filter}`);
+    }
+    if (response.context.auto_detected) {
+        lines.push(`Installed skills: ${response.context.installed_count} (auto-detected from ~/.claude/skills/)`);
+    }
+    else {
+        lines.push(`Installed skills: ${response.context.installed_count}`);
+    }
+    lines.push(`Semantic matching: ${response.context.using_semantic_matching ? 'enabled' : 'disabled'}`);
+    lines.push(`Completed in ${response.timing.totalMs}ms`);
+    return lines.join('\n');
+}
+//# sourceMappingURL=recommend.format.js.map

package/dist/src/tools/recommend.format.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"recommend.format.js","sourceRoot":"","sources":["../../../src/tools/recommend.format.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAA;AAGtD;;;;;;;;GAQG;AACH,MAAM,UAAU,kCAAkC,CAChD,UAAiC,EACjC,YAAmC,EACnC,KAAa;IAEb,0DAA0D;IAC1D,MAAM,aAAa,GAAG,IAAI,GAAG,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC,CAAC,CAAA;IAE1E,mDAAmD;IACnD,MAAM,eAAe,GAAG,IAAI,GAAG,CAC7B,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,WAAW,EAAE,IAAI,EAAE,CAAC,CACxE,CAAA;IAED,6CAA6C;IAC7C,MAAM,kBAAkB,GAAG,YAAY,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE;QACvD,MAAM,SAAS,GAAG,KAAK,CAAC,IAAI,CAAC,WAAW,EAAE,CAAA;QAC1C,MAAM,WAAW,GAAG,KAAK,CAAC,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,WAAW,EAAE,IAAI,EAAE,CAAA;QAExE,wCAAwC;QACxC,IAAI,aAAa,CAAC,GAAG,CAAC,SAAS,CAAC,EAAE,CAAC;YACjC,OAAO,KAAK,CAAA;QACd,CAAC;QAED,2CAA2C;QAC3C,IAAI,eAAe,CAAC,GAAG,CAAC,WAAW,CAAC,EAAE,CAAC;YACrC,OAAO,KAAK,CAAA;QACd,CAAC;QAED,sEAAsE;QACtE,KAAK,MAAM,OAAO,IAAI,aAAa,EAAE,CAAC;YACpC,IAAI,SAAS,CAAC,QAAQ,CAAC,OAAO,CAAC,IAAI,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAC,EAAE,CAAC;gBAC/D,OAAO,KAAK,CAAA;YACd,CAAC;QACH,CAAC;QAED,OAAO,IAAI,CAAA;IACb,CAAC,CAAC,CAAA;IAEF,yEAAyE;IACzE,MAAM,QAAQ,GAAG,CAAC,GAAG,UAAU,EAAE,GAAG,kBAAkB,CAAC,CAAA;IAEvD,6DAA6D;IAC7D,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE;QACrB,IAAI,CAAC,CAAC,aAAa,KAAK,CAAC,CAAC,aAAa,EAAE,CAAC;YACxC,OAAO,CAAC,CAAC,aAAa,GAAG,CAAC,CAAC,aAAa,CAAA;QAC1C,CAAC;QACD,OAAO,CAAC,CAAC,gBAAgB,GAAG,CAAC,CAAC,gBAAgB,CAAA;IAChD,CAAC,CAAC,CAAA;IAEF,OAAO,QAAQ,CAAC,KAAK,CAAC,CAAC,EAAE,KAAK,CAAC,CAAA;AACjC,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,qBAAqB,CAAC,QAA2B;IAC/D,MAAM,KAAK,GAAa,EAAE,CAAA;IAE1B,KAAK,CAAC,IAAI,CAAC,mCAAmC,CAAC,CAAA;IAE/C,IAAI,QAAQ,CAAC,eAAe,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC1C,KAAK,CAAC,IAAI,CAAC,2BAA2B,CAAC,CAAA;QACvC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QACd,KAAK,CAAC,IAAI,CAAC,cAAc,CAAC,CAAA;QAC1B,KAAK,CAAC,IAAI,CAAC,0DAA0D,CAAC,CAAA;QACtE,KAAK,CAAC,IAAI,CAAC,yDAAyD,CAAC,CAAA;QACrE,4DAA4D;QAC5D,IAAI,QAAQ,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC;YACjC,KAAK,CAAC,IAAI,CAAC,gDAAgD,QAAQ,CAAC,OAAO,CAAC,WAAW,GAAG,CAAC,CAAA;QAC7F,CAAC;IACH,CAAC;SAAM,CAAC;QACN,KAAK,CAAC,IAAI,CAAC,SAAS,QAAQ,CAAC,eAAe,CAAC,MAAM,uBAAuB,CAAC,CAAA;QAE3E,QAAQ,CAAC,eAAe,CAAC,OAAO,CAAC,CAAC,GAAG,EAAE,KAAK,EAAE,EAAE;YAC9C,MAAM,UAAU,GAAG,aAAa,CAAC,GAAG,CAAC,UAAU,CAAC,CAAA;YAChD,kCAAkC;YAClC,MAAM,YAAY,GAAG,GAAG,CAAC,KAAK,EAAE,MAAM,CAAC,CAAC,CAAC,KAAK,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAA;YAC1E,KAAK,CAAC,IAAI,CAAC,GAAG,KAAK,GAAG,CAAC,KAAK,GAAG,CAAC,IAAI,IAAI,UAAU,GAAG,YAAY,EAAE,CAAC,CAAA;YACpE,KAAK,CAAC,IAAI,CACR,aAAa,GAAG,CAAC,aAAa,qBAAqB,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,gBAAgB,GAAG,GAAG,CAAC,GAAG,CAC7F,CAAA;YACD,KAAK,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,MAAM,EAAE,CAAC,CAAA;YAC9B,KAAK,CAAC,IAAI,CAAC,UAAU,GAAG,CAAC,QAAQ,EAAE,CAAC,CAAA;YACpC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QAChB,CAAC,CAAC,CAAA;IACJ,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACjB,KAAK,CAAC,IAAI,CAAC,0BAA0B,QAAQ,CAAC,qBAAqB,EAAE,CAAC,CAAA;IACtE,IAAI,QAAQ,CAAC,gBAAgB,GAAG,CAAC,EAAE,CAAC;QAClC,KAAK,CAAC,IAAI,CAAC,yBAAyB,QAAQ,CAAC,gBAAgB,EAAE,CAAC,CAAA;IAClE,CAAC;IACD,mCAAmC;IACnC,IAAI,QAAQ,CAAC,aAAa,GAAG,CAAC,EAAE,CAAC;QAC/B,KAAK,CAAC,IAAI,CAAC,sBAAsB,QAAQ,CAAC,aAAa,EAAE,CAAC,CAAA;IAC5D,CAAC;IACD,IAAI,QAAQ,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC;QACjC,KAAK,CAAC,IAAI,CAAC,gBAAgB,QAAQ,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,CAAA;IAC5D,CAAC;IACD,IAAI,QAAQ,CAAC,OAAO,CAAC,aAAa,EAAE,CAAC;QACnC,KAAK,CAAC,IAAI,CACR,qBAAqB,QAAQ,CAAC,OAAO,CAAC,eAAe,yCAAyC,CAC/F,CAAA;IACH,CAAC;SAAM,CAAC;QACN,KAAK,CAAC,IAAI,CAAC,qBAAqB,QAAQ,CAAC,OAAO,CAAC,eAAe,EAAE,CAAC,CAAA;IACrE,CAAC;IACD,KAAK,CAAC,IAAI,CACR,sBAAsB,QAAQ,CAAC,OAAO,CAAC,uBAAuB,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,UAAU,EAAE,CAC1F,CAAA;IACD,KAAK,CAAC,IAAI,CAAC,gBAAgB,QAAQ,CAAC,MAAM,CAAC,OAAO,IAAI,CAAC,CAAA;IAEvD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACzB,CAAC"}

package/dist/src/tools/recommend.js

@@ -5,18 +5,23 @@
  * @see SMI-602: Integrate semantic matching with EmbeddingService
  * @see SMI-604: Add trigger phrase overlap detection
  * @see SMI-1837: Include local skills in recommendations (parallel search)
+ * @see SMI-2741: Split to meet 500-line standard
  */
 import { SkillMatcher, OverlapDetector, trackEvent } from '@skillsmith/core';
 import { getInstalledSkills } from '../utils/installed-skills.js';
-import { mapTrustTierFromDb, getTrustBadge } from '../utils/validation.js';
+import { mapTrustTierFromDb } from '../utils/validation.js';
 // Import types
 import { recommendInputSchema, } from './recommend.types.js';
 // Import helpers
 import { inferRolesFromTags, loadSkillsFromDatabase, isSkillCollection, } from './recommend.helpers.js';
+// SMI-2741: Formatting and deduplication extracted to companion file
+import { mergeAndDeduplicateRecommendations } from './recommend.format.js';
 // SMI-1837: Import local skill search for parallel querying
 import { getLocalIndexer } from './LocalSkillSearch.js';
 // Re-export only public API types (SMI-1718: trimmed internal exports)
 export { recommendInputSchema, recommendToolSchema, } from './recommend.types.js';
+// Re-export formatting utilities (SMI-2741)
+export { formatRecommendations, mergeAndDeduplicateRecommendations } from './recommend.format.js';
 /**
  * SMI-1837: Convert a LocalSkill to SkillRecommendation format
  * @param skill - The local skill to convert
@@ -61,50 +66,6 @@ async function searchLocalSkillsForRecommend(query, limit) {
         return [];
     }
 }
-/**
- * SMI-1837: Merge and deduplicate API and local skill recommendations
- * API results take priority over local results with the same name
- * @param apiResults - Results from API
- * @param localResults - Results from local skill search
- * @param limit - Maximum combined results
- * @returns Merged and deduplicated recommendations
- */
-function mergeAndDeduplicateRecommendations(apiResults, localResults, limit) {
-    // Build a Set of names from API results for deduplication
-    const apiSkillNames = new Set(apiResults.map((r) => r.name.toLowerCase()));
-    // Also track skill IDs (without the author prefix)
-    const apiSkillIdNames = new Set(apiResults.map((r) => r.skill_id.split('/').pop()?.toLowerCase() || ''));
-    // Filter local results to exclude duplicates
-    const uniqueLocalResults = localResults.filter((local) => {
-        const localName = local.name.toLowerCase();
-        const localIdName = local.skill_id.split('/').pop()?.toLowerCase() || '';
-        // Exclude if name matches an API result
-        if (apiSkillNames.has(localName)) {
-            return false;
-        }
-        // Exclude if ID name matches an API result
-        if (apiSkillIdNames.has(localIdName)) {
-            return false;
-        }
-        // Exclude if local name is contained in or contains an API skill name
-        for (const apiName of apiSkillNames) {
-            if (localName.includes(apiName) || apiName.includes(localName)) {
-                return false;
-            }
-        }
-        return true;
-    });
-    // Combine API results first (higher priority), then unique local results
-    const combined = [...apiResults, ...uniqueLocalResults];
-    // Sort by quality score descending, then by similarity score
-    combined.sort((a, b) => {
-        if (b.quality_score !== a.quality_score) {
-            return b.quality_score - a.quality_score;
-        }
-        return b.similarity_score - a.similarity_score;
-    });
-    return combined.slice(0, limit);
-}
 /**
  * Execute skill recommendation based on installed skills and context.
  *
@@ -364,56 +325,4 @@ export async function executeRecommend(input, context) {
     }
     return response;
 }
-/**
- * Format recommendations for terminal display
- */
-export function formatRecommendations(response) {
-    const lines = [];
-    lines.push('\n=== Skill Recommendations ===\n');
-    if (response.recommendations.length === 0) {
-        lines.push('No recommendations found.');
-        lines.push('');
-        lines.push('Suggestions:');
-        lines.push('  - Try adding more installed skills for better matching');
-        lines.push('  - Provide a project context for more relevant results');
-        // SMI-1631: Suggest removing role filter if one was applied
-        if (response.context.role_filter) {
-            lines.push(`  - Try removing the role filter (currently: ${response.context.role_filter})`);
-        }
-    }
-    else {
-        lines.push(`Found ${response.recommendations.length} recommendation(s):\n`);
-        response.recommendations.forEach((rec, index) => {
-            const trustBadge = getTrustBadge(rec.trust_tier);
-            // SMI-1631: Show roles if present
-            const rolesDisplay = rec.roles?.length ? ` [${rec.roles.join(', ')}]` : '';
-            lines.push(`${index + 1}. ${rec.name} ${trustBadge}${rolesDisplay}`);
-            lines.push(`   Score: ${rec.quality_score}/100 | Relevance: ${Math.round(rec.similarity_score * 100)}%`);
-            lines.push(`   ${rec.reason}`);
-            lines.push(`   ID: ${rec.skill_id}`);
-            lines.push('');
-        });
-    }
-    lines.push('---');
-    lines.push(`Candidates considered: ${response.candidates_considered}`);
-    if (response.overlap_filtered > 0) {
-        lines.push(`Filtered for overlap: ${response.overlap_filtered}`);
-    }
-    // SMI-1631: Show role filter stats
-    if (response.role_filtered > 0) {
-        lines.push(`Filtered for role: ${response.role_filtered}`);
-    }
-    if (response.context.role_filter) {
-        lines.push(`Role filter: ${response.context.role_filter}`);
-    }
-    if (response.context.auto_detected) {
-        lines.push(`Installed skills: ${response.context.installed_count} (auto-detected from ~/.claude/skills/)`);
-    }
-    else {
-        lines.push(`Installed skills: ${response.context.installed_count}`);
-    }
-    lines.push(`Semantic matching: ${response.context.using_semantic_matching ? 'enabled' : 'disabled'}`);
-    lines.push(`Completed in ${response.timing.totalMs}ms`);
-    return lines.join('\n');
-}
 //# sourceMappingURL=recommend.js.map