@skillsmith/core 0.5.3 → 0.5.5

package/dist/tests/ScraperAdapters.gitlab-coverage.test.d.ts

@@ -0,0 +1,14 @@
+/**
+ * GitLabSourceAdapter additional coverage (SMI-4290 / closes #595)
+ *
+ * Sidecar to ScraperAdapters.test.ts — fills genuine coverage gaps for
+ * `skillExists`, `getSkillSha`, and the `createGitLabAdapter` factory.
+ *
+ * The primary suite (`ScraperAdapters.test.ts:631`) already has 14 tests
+ * covering init, health, search, pagination, `getRepository`, and
+ * `fetchSkillContent`. This file stays under 500 lines and is kept
+ * separate so pre-commit's file-length check does not block edits to
+ * the main (>500 line) test file.
+ */
+export {};
+//# sourceMappingURL=ScraperAdapters.gitlab-coverage.test.d.ts.map

package/dist/tests/ScraperAdapters.gitlab-coverage.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"ScraperAdapters.gitlab-coverage.test.d.ts","sourceRoot":"","sources":["../../tests/ScraperAdapters.gitlab-coverage.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG"}

package/dist/tests/ScraperAdapters.gitlab-coverage.test.js

@@ -0,0 +1,169 @@
+/**
+ * GitLabSourceAdapter additional coverage (SMI-4290 / closes #595)
+ *
+ * Sidecar to ScraperAdapters.test.ts — fills genuine coverage gaps for
+ * `skillExists`, `getSkillSha`, and the `createGitLabAdapter` factory.
+ *
+ * The primary suite (`ScraperAdapters.test.ts:631`) already has 14 tests
+ * covering init, health, search, pagination, `getRepository`, and
+ * `fetchSkillContent`. This file stays under 500 lines and is kept
+ * separate so pre-commit's file-length check does not block edits to
+ * the main (>500 line) test file.
+ */
+import { describe, it, expect, beforeEach, vi } from 'vitest';
+import { GitLabSourceAdapter, createGitLabAdapter } from '../src/sources/GitLabSourceAdapter.js';
+const mockFetch = vi.fn();
+vi.stubGlobal('fetch', mockFetch);
+function okResponse(body) {
+    return {
+        ok: true,
+        status: 200,
+        headers: new Headers(),
+        json: async () => body,
+    };
+}
+function notFoundResponse() {
+    return {
+        ok: false,
+        status: 404,
+        headers: new Headers(),
+    };
+}
+function rateLimitedResponse() {
+    return {
+        ok: false,
+        status: 429,
+        headers: new Headers(),
+    };
+}
+describe('GitLabSourceAdapter — additional coverage (SMI-4290)', () => {
+    let adapter;
+    beforeEach(() => {
+        mockFetch.mockReset();
+        adapter = new GitLabSourceAdapter({
+            id: 'test-gitlab-coverage',
+            name: 'Test GitLab Coverage',
+            type: 'gitlab',
+            baseUrl: 'https://gitlab.com/api/v4',
+            enabled: true,
+            rateLimit: { maxRequests: 100, windowMs: 60000, minDelayMs: 0 },
+        });
+    });
+    describe('skillExists', () => {
+        it('returns true when the default SKILL.md path resolves 200', async () => {
+            mockFetch.mockResolvedValueOnce(okResponse({}));
+            const exists = await adapter.skillExists({ owner: 'user', repo: 'skill-repo' });
+            expect(exists).toBe(true);
+            expect(mockFetch).toHaveBeenCalledTimes(1);
+            const [url, init] = mockFetch.mock.calls[0];
+            expect(String(url)).toContain('/projects/user%2Fskill-repo/repository/files/');
+            expect(init.method).toBe('HEAD');
+        });
+        it('returns false when every candidate path returns 404', async () => {
+            // GitLabSourceAdapter falls back through SKILL_FILE_PATHS; each lookup 404s.
+            mockFetch.mockResolvedValue(notFoundResponse());
+            const exists = await adapter.skillExists({ owner: 'user', repo: 'missing-repo' });
+            expect(exists).toBe(false);
+            expect(mockFetch.mock.calls.length).toBeGreaterThan(0);
+        });
+        it('returns true when an explicit path is provided and resolves', async () => {
+            mockFetch.mockResolvedValueOnce(okResponse({}));
+            const exists = await adapter.skillExists({
+                owner: 'user',
+                repo: 'skill-repo',
+                path: 'docs/custom-skill.md',
+            });
+            expect(exists).toBe(true);
+            // With an explicit path only one request should be attempted.
+            expect(mockFetch).toHaveBeenCalledTimes(1);
+            expect(String(mockFetch.mock.calls[0][0])).toContain(encodeURIComponent('docs/custom-skill.md'));
+        });
+        it('returns false when the request throws (network error surfaced as continue)', async () => {
+            mockFetch.mockRejectedValue(new Error('ECONNRESET'));
+            const exists = await adapter.skillExists({ owner: 'user', repo: 'broken-repo' });
+            expect(exists).toBe(false);
+        });
+        it('returns false when the rate-limited status is surfaced (not a 200)', async () => {
+            mockFetch.mockResolvedValue(rateLimitedResponse());
+            const exists = await adapter.skillExists({ owner: 'user', repo: 'limited-repo' });
+            expect(exists).toBe(false);
+        });
+    });
+    describe('getSkillSha', () => {
+        it('returns the content SHA-256 when the file resolves', async () => {
+            mockFetch.mockResolvedValueOnce(okResponse({
+                file_name: 'SKILL.md',
+                file_path: 'SKILL.md',
+                content: '',
+                encoding: 'base64',
+                content_sha256: 'sha-abc-123',
+            }));
+            const sha = await adapter.getSkillSha({ owner: 'user', repo: 'skill-repo' });
+            expect(sha).toBe('sha-abc-123');
+        });
+        it('returns null when every candidate path 404s', async () => {
+            mockFetch.mockResolvedValue(notFoundResponse());
+            const sha = await adapter.getSkillSha({ owner: 'user', repo: 'missing-repo' });
+            expect(sha).toBeNull();
+        });
+        it('falls back to the second path when the first 404s', async () => {
+            mockFetch.mockResolvedValueOnce(notFoundResponse()).mockResolvedValueOnce(okResponse({
+                file_name: 'skill.md',
+                file_path: 'skill.md',
+                content: '',
+                encoding: 'base64',
+                content_sha256: 'sha-fallback-789',
+            }));
+            const sha = await adapter.getSkillSha({ owner: 'user', repo: 'skill-repo' });
+            expect(sha).toBe('sha-fallback-789');
+            expect(mockFetch).toHaveBeenCalledTimes(2);
+        });
+        it('returns null when a network error is thrown and no path succeeds', async () => {
+            mockFetch.mockRejectedValue(new Error('ETIMEDOUT'));
+            const sha = await adapter.getSkillSha({ owner: 'user', repo: 'broken-repo' });
+            expect(sha).toBeNull();
+        });
+    });
+    describe('createGitLabAdapter factory', () => {
+        it('returns a GitLabSourceAdapter with type set to gitlab', () => {
+            const built = createGitLabAdapter({
+                id: 'factory-gitlab',
+                name: 'Factory GitLab',
+                enabled: true,
+            });
+            expect(built).toBeInstanceOf(GitLabSourceAdapter);
+            expect(built.type).toBe('gitlab');
+        });
+        it('defaults baseUrl to the public gitlab.com endpoint when not provided', async () => {
+            const built = createGitLabAdapter({
+                id: 'factory-default-url',
+                name: 'Factory Default URL',
+                enabled: true,
+            });
+            mockFetch.mockResolvedValueOnce(okResponse({}));
+            await built.skillExists({ owner: 'user', repo: 'skill-repo' });
+            expect(String(mockFetch.mock.calls[0][0])).toContain('https://gitlab.com/api/v4');
+        });
+        it('respects a custom baseUrl when provided', async () => {
+            const built = createGitLabAdapter({
+                id: 'factory-custom-url',
+                name: 'Factory Custom URL',
+                enabled: true,
+                baseUrl: 'https://gitlab.example.com/api/v4',
+            });
+            mockFetch.mockResolvedValueOnce(okResponse({}));
+            await built.skillExists({ owner: 'user', repo: 'skill-repo' });
+            expect(String(mockFetch.mock.calls[0][0])).toContain('https://gitlab.example.com/api/v4');
+        });
+        it('forwards auth configuration to the constructed adapter', () => {
+            const built = createGitLabAdapter({
+                id: 'factory-auth',
+                name: 'Factory Auth',
+                enabled: true,
+                auth: { type: 'token', credentials: 'glpat-stub' },
+            });
+            expect(built).toBeInstanceOf(GitLabSourceAdapter);
+        });
+    });
+});
+//# sourceMappingURL=ScraperAdapters.gitlab-coverage.test.js.map

package/dist/tests/ScraperAdapters.gitlab-coverage.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ScraperAdapters.gitlab-coverage.test.js","sourceRoot":"","sources":["../../tests/ScraperAdapters.gitlab-coverage.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAA;AAC7D,OAAO,EAAE,mBAAmB,EAAE,mBAAmB,EAAE,MAAM,uCAAuC,CAAA;AAEhG,MAAM,SAAS,GAAG,EAAE,CAAC,EAAE,EAAE,CAAA;AACzB,EAAE,CAAC,UAAU,CAAC,OAAO,EAAE,SAAS,CAAC,CAAA;AAEjC,SAAS,UAAU,CAAC,IAAa;IAC/B,OAAO;QACL,EAAE,EAAE,IAAI;QACR,MAAM,EAAE,GAAG;QACX,OAAO,EAAE,IAAI,OAAO,EAAE;QACtB,IAAI,EAAE,KAAK,IAAI,EAAE,CAAC,IAAI;KACA,CAAA;AAC1B,CAAC;AAED,SAAS,gBAAgB;IACvB,OAAO;QACL,EAAE,EAAE,KAAK;QACT,MAAM,EAAE,GAAG;QACX,OAAO,EAAE,IAAI,OAAO,EAAE;KACA,CAAA;AAC1B,CAAC;AAED,SAAS,mBAAmB;IAC1B,OAAO;QACL,EAAE,EAAE,KAAK;QACT,MAAM,EAAE,GAAG;QACX,OAAO,EAAE,IAAI,OAAO,EAAE;KACA,CAAA;AAC1B,CAAC;AAED,QAAQ,CAAC,sDAAsD,EAAE,GAAG,EAAE;IACpE,IAAI,OAA4B,CAAA;IAEhC,UAAU,CAAC,GAAG,EAAE;QACd,SAAS,CAAC,SAAS,EAAE,CAAA;QACrB,OAAO,GAAG,IAAI,mBAAmB,CAAC;YAChC,EAAE,EAAE,sBAAsB;YAC1B,IAAI,EAAE,sBAAsB;YAC5B,IAAI,EAAE,QAAQ;YACd,OAAO,EAAE,2BAA2B;YACpC,OAAO,EAAE,IAAI;YACb,SAAS,EAAE,EAAE,WAAW,EAAE,GAAG,EAAE,QAAQ,EAAE,KAAK,EAAE,UAAU,EAAE,CAAC,EAAE;SAChE,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;QAC3B,EAAE,CAAC,0DAA0D,EAAE,KAAK,IAAI,EAAE;YACxE,SAAS,CAAC,qBAAqB,CAAC,UAAU,CAAC,EAAE,CAAC,CAAC,CAAA;YAE/C,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,WAAW,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,YAAY,EAAE,CAAC,CAAA;YAE/E,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;YACzB,MAAM,CAAC,SAAS,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAA;YAC1C,MAAM,CAAC,GAAG,EAAE,IAAI,CAAC,GAAG,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAA;YAC3C,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,SAAS,CAAC,+CAA+C,CAAC,CAAA;YAC9E,MAAM,CAAE,IAAoB,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;QACnD,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;YACnE,6EAA6E;YAC7E,SAAS,CAAC,iBAAiB,CAAC,gBAAgB,EAAE,CAAC,CAAA;YAE/C,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,WAAW,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC,CAAA;YAEjF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;YAC1B,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAA;QACxD,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;YAC3E,SAAS,CAAC,qBAAqB,CAAC,UAAU,CAAC,EAAE,CAAC,CAAC,CAAA;YAE/C,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,WAAW,CAAC;gBACvC,KAAK,EAAE,MAAM;gBACb,IAAI,EAAE,YAAY;gBAClB,IAAI,EAAE,sBAAsB;aAC7B,CAAC,CAAA;YAEF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;YACzB,8DAA8D;YAC9D,MAAM,CAAC,SAAS,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAA;YAC1C,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAClD,kBAAkB,CAAC,sBAAsB,CAAC,CAC3C,CAAA;QACH,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;YAC1F,SAAS,CAAC,iBAAiB,CAAC,IAAI,KAAK,CAAC,YAAY,CAAC,CAAC,CAAA;YAEpD,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,WAAW,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,aAAa,EAAE,CAAC,CAAA;YAEhF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAC5B,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,oEAAoE,EAAE,KAAK,IAAI,EAAE;YAClF,SAAS,CAAC,iBAAiB,CAAC,mBAAmB,EAAE,CAAC,CAAA;YAElD,MAAM,MAAM,GAAG,MAAM,OAAO,CAAC,WAAW,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC,CAAA;YAEjF,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QAC5B,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;QAC3B,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;YAClE,SAAS,CAAC,qBAAqB,CAC7B,UAAU,CAAC;gBACT,SAAS,EAAE,UAAU;gBACrB,SAAS,EAAE,UAAU;gBACrB,OAAO,EAAE,EAAE;gBACX,QAAQ,EAAE,QAAQ;gBAClB,cAAc,EAAE,aAAa;aAC9B,CAAC,CACH,CAAA;YAED,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,WAAW,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,YAAY,EAAE,CAAC,CAAA;YAE5E,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,CAAA;QACjC,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,6CAA6C,EAAE,KAAK,IAAI,EAAE;YAC3D,SAAS,CAAC,iBAAiB,CAAC,gBAAgB,EAAE,CAAC,CAAA;YAE/C,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,WAAW,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,cAAc,EAAE,CAAC,CAAA;YAE9E,MAAM,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE,CAAA;QACxB,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;YACjE,SAAS,CAAC,qBAAqB,CAAC,gBAAgB,EAAE,CAAC,CAAC,qBAAqB,CACvE,UAAU,CAAC;gBACT,SAAS,EAAE,UAAU;gBACrB,SAAS,EAAE,UAAU;gBACrB,OAAO,EAAE,EAAE;gBACX,QAAQ,EAAE,QAAQ;gBAClB,cAAc,EAAE,kBAAkB;aACnC,CAAC,CACH,CAAA;YAED,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,WAAW,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,YAAY,EAAE,CAAC,CAAA;YAE5E,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAA;YACpC,MAAM,CAAC,SAAS,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAA;QAC5C,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,kEAAkE,EAAE,KAAK,IAAI,EAAE;YAChF,SAAS,CAAC,iBAAiB,CAAC,IAAI,KAAK,CAAC,WAAW,CAAC,CAAC,CAAA;YAEnD,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,WAAW,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,aAAa,EAAE,CAAC,CAAA;YAE7E,MAAM,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE,CAAA;QACxB,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;IAEF,QAAQ,CAAC,6BAA6B,EAAE,GAAG,EAAE;QAC3C,EAAE,CAAC,uDAAuD,EAAE,GAAG,EAAE;YAC/D,MAAM,KAAK,GAAG,mBAAmB,CAAC;gBAChC,EAAE,EAAE,gBAAgB;gBACpB,IAAI,EAAE,gBAAgB;gBACtB,OAAO,EAAE,IAAI;aACd,CAAC,CAAA;YAEF,MAAM,CAAC,KAAK,CAAC,CAAC,cAAc,CAAC,mBAAmB,CAAC,CAAA;YACjD,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACnC,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,sEAAsE,EAAE,KAAK,IAAI,EAAE;YACpF,MAAM,KAAK,GAAG,mBAAmB,CAAC;gBAChC,EAAE,EAAE,qBAAqB;gBACzB,IAAI,EAAE,qBAAqB;gBAC3B,OAAO,EAAE,IAAI;aACd,CAAC,CAAA;YAEF,SAAS,CAAC,qBAAqB,CAAC,UAAU,CAAC,EAAE,CAAC,CAAC,CAAA;YAC/C,MAAM,KAAK,CAAC,WAAW,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,YAAY,EAAE,CAAC,CAAA;YAE9D,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,2BAA2B,CAAC,CAAA;QACnF,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,yCAAyC,EAAE,KAAK,IAAI,EAAE;YACvD,MAAM,KAAK,GAAG,mBAAmB,CAAC;gBAChC,EAAE,EAAE,oBAAoB;gBACxB,IAAI,EAAE,oBAAoB;gBAC1B,OAAO,EAAE,IAAI;gBACb,OAAO,EAAE,mCAAmC;aAC7C,CAAC,CAAA;YAEF,SAAS,CAAC,qBAAqB,CAAC,UAAU,CAAC,EAAE,CAAC,CAAC,CAAA;YAC/C,MAAM,KAAK,CAAC,WAAW,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,YAAY,EAAE,CAAC,CAAA;YAE9D,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,mCAAmC,CAAC,CAAA;QAC3F,CAAC,CAAC,CAAA;QAEF,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;YAChE,MAAM,KAAK,GAAG,mBAAmB,CAAC;gBAChC,EAAE,EAAE,cAAc;gBAClB,IAAI,EAAE,cAAc;gBACpB,OAAO,EAAE,IAAI;gBACb,IAAI,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE,WAAW,EAAE,YAAY,EAAE;aACnD,CAAC,CAAA;YAEF,MAAM,CAAC,KAAK,CAAC,CAAC,cAAc,CAAC,mBAAmB,CAAC,CAAA;QACnD,CAAC,CAAC,CAAA;IACJ,CAAC,CAAC,CAAA;AACJ,CAAC,CAAC,CAAA"}

package/dist/tests/ScraperAdapters.test.d.ts

@@ -1,7 +1,11 @@
 /**
  * Scraper Adapters Tests (SMI-591)
  *
- * Tests for RawUrlSourceAdapter, LocalFilesystemAdapter, and GitLabSourceAdapter
+ * Tests for RawUrlSourceAdapter and GitLabSourceAdapter.
+ *
+ * LocalFilesystemAdapter tests live in `LocalFilesystemAdapter.test.ts`
+ * (extracted by SMI-4287 to keep this file under the 500-line governance
+ * ceiling and avoid a duplicate `describe` block — see SMI-4286 retro).
  */
 export {};
 //# sourceMappingURL=ScraperAdapters.test.d.ts.map

package/dist/tests/ScraperAdapters.test.d.ts.map

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

package/dist/tests/ScraperAdapters.test.js

@@ -1,15 +1,15 @@
 /**
  * Scraper Adapters Tests (SMI-591)
  *
- * Tests for RawUrlSourceAdapter, LocalFilesystemAdapter, and GitLabSourceAdapter
+ * Tests for RawUrlSourceAdapter and GitLabSourceAdapter.
+ *
+ * LocalFilesystemAdapter tests live in `LocalFilesystemAdapter.test.ts`
+ * (extracted by SMI-4287 to keep this file under the 500-line governance
+ * ceiling and avoid a duplicate `describe` block — see SMI-4286 retro).
  */
-import { describe, it, expect, beforeEach, vi, afterEach } from 'vitest';
+import { describe, it, expect, beforeEach, vi } from 'vitest';
 import { RawUrlSourceAdapter } from '../src/sources/RawUrlSourceAdapter.js';
-import { LocalFilesystemAdapter } from '../src/sources/LocalFilesystemAdapter.js';
 import { GitLabSourceAdapter } from '../src/sources/GitLabSourceAdapter.js';
-import { promises as fs } from 'fs';
-import { join } from 'path';
-import { tmpdir } from 'os';
 // Mock fetch for network adapters
 const mockFetch = vi.fn();
 vi.stubGlobal('fetch', mockFetch);
@@ -191,336 +191,6 @@ describe('RawUrlSourceAdapter (SMI-591)', () => {
         });
     });
 });
-describe('LocalFilesystemAdapter (SMI-591)', () => {
-    let adapter;
-    let testDir;
-    beforeEach(async () => {
-        // Create test directory structure
-        testDir = join(tmpdir(), `skillsmith-test-${Date.now()}`);
-        await fs.mkdir(testDir, { recursive: true });
-        // Create test skill files
-        await fs.mkdir(join(testDir, 'skill-one'), { recursive: true });
-        await fs.writeFile(join(testDir, 'skill-one', 'SKILL.md'), '---\nname: Skill One\ndescription: First skill\n---\n# Skill One');
-        await fs.mkdir(join(testDir, 'skill-two'), { recursive: true });
-        await fs.writeFile(join(testDir, 'skill-two', 'SKILL.md'), '---\nname: Skill Two\n---\n# Skill Two');
-        // Create node_modules (should be excluded)
-        await fs.mkdir(join(testDir, 'node_modules', 'some-module'), { recursive: true });
-        await fs.writeFile(join(testDir, 'node_modules', 'some-module', 'SKILL.md'), '# Should be excluded');
-        adapter = new LocalFilesystemAdapter({
-            id: 'test-local',
-            name: 'Test Local',
-            type: 'local',
-            baseUrl: 'file://',
-            enabled: true,
-            rootDir: testDir,
-            rateLimit: { maxRequests: 100, windowMs: 60000, minDelayMs: 0 },
-        });
-        await adapter.initialize();
-    });
-    afterEach(async () => {
-        // Clean up test directory
-        try {
-            await fs.rm(testDir, { recursive: true, force: true });
-        }
-        catch {
-            // Ignore cleanup errors
-        }
-    });
-    describe('Initialization', () => {
-        it('should discover skill files in root directory', () => {
-            expect(adapter.skillCount).toBe(2);
-        });
-        it('should exclude node_modules', () => {
-            expect(adapter.skillCount).toBe(2); // Only skill-one and skill-two
-        });
-    });
-    describe('Health Check', () => {
-        it('should return healthy when root directory exists', async () => {
-            const health = await adapter.checkHealth();
-            expect(health.healthy).toBe(true);
-        });
-        it('should return unhealthy when root directory does not exist', async () => {
-            const badAdapter = new LocalFilesystemAdapter({
-                id: 'bad',
-                name: 'Bad',
-                type: 'local',
-                baseUrl: 'file://',
-                enabled: true,
-                rootDir: '/nonexistent/path',
-                rateLimit: { maxRequests: 100, windowMs: 60000, minDelayMs: 0 },
-            });
-            const health = await badAdapter.checkHealth();
-            expect(health.healthy).toBe(false);
-        });
-    });
-    describe('Search', () => {
-        it('should return all discovered skills', async () => {
-            const result = await adapter.search({});
-            expect(result.repositories).toHaveLength(2);
-        });
-        it('should filter by query', async () => {
-            const result = await adapter.search({ query: 'skill-one' });
-            expect(result.repositories).toHaveLength(1);
-            expect(result.repositories[0].name).toBe('Skill One');
-        });
-        it('should extract name from frontmatter', async () => {
-            const result = await adapter.search({});
-            const names = result.repositories.map((r) => r.name);
-            expect(names).toContain('Skill One');
-            expect(names).toContain('Skill Two');
-        });
-        it('should extract description from frontmatter', async () => {
-            const result = await adapter.search({});
-            const skillOne = result.repositories.find((r) => r.name === 'Skill One');
-            expect(skillOne?.description).toBe('First skill');
-        });
-    });
-    describe('Fetch Skill Content', () => {
-        it('should fetch skill content by path', async () => {
-            const content = await adapter.fetchSkillContent({
-                path: join(testDir, 'skill-one', 'SKILL.md'),
-            });
-            expect(content.rawContent).toContain('# Skill One');
-            expect(content.sha).toBeDefined();
-        });
-        it('should throw error for non-existent skill', async () => {
-            await expect(adapter.fetchSkillContent({
-                path: join(testDir, 'nonexistent', 'SKILL.md'),
-            })).rejects.toThrow('Failed to read skill file');
-        });
-    });
-    describe('Skill Exists', () => {
-        it('should return true for existing skill', async () => {
-            const exists = await adapter.skillExists({
-                path: join(testDir, 'skill-one', 'SKILL.md'),
-            });
-            expect(exists).toBe(true);
-        });
-        it('should return false for non-existent skill', async () => {
-            const exists = await adapter.skillExists({
-                path: join(testDir, 'nonexistent', 'SKILL.md'),
-            });
-            expect(exists).toBe(false);
-        });
-    });
-    describe('Rescan', () => {
-        it('should discover newly added skills', async () => {
-            // Add a new skill
-            await fs.mkdir(join(testDir, 'skill-three'), { recursive: true });
-            await fs.writeFile(join(testDir, 'skill-three', 'SKILL.md'), '# Skill Three');
-            // Rescan
-            const count = await adapter.rescan();
-            expect(count).toBe(3);
-        });
-    });
-    describe('Path Traversal Prevention (SMI-720)', () => {
-        it('should reject relative path traversal with ../', async () => {
-            await expect(adapter.fetchSkillContent({
-                path: '../../../etc/passwd',
-            })).rejects.toThrow('Path traversal detected');
-        });
-        it('should reject deeply nested path traversal', async () => {
-            await expect(adapter.fetchSkillContent({
-                path: 'skill-one/../../../../../../etc/shadow',
-            })).rejects.toThrow('Path traversal detected');
-        });
-        it('should reject absolute paths outside rootDir', async () => {
-            await expect(adapter.fetchSkillContent({
-                path: '/etc/passwd',
-            })).rejects.toThrow('Path traversal detected');
-        });
-        it('should reject path traversal via owner/repo', async () => {
-            await expect(adapter.getRepository({
-                owner: '..',
-                repo: '../../../etc',
-            })).rejects.toThrow('Path traversal detected');
-        });
-        it('should reject path traversal via repo only', async () => {
-            await expect(adapter.skillExists({
-                repo: '../../../etc/passwd',
-            })).rejects.toThrow('Path traversal detected');
-        });
-        it('should allow valid paths within rootDir', async () => {
-            const exists = await adapter.skillExists({
-                path: join(testDir, 'skill-one', 'SKILL.md'),
-            });
-            expect(exists).toBe(true);
-        });
-        it('should allow valid relative paths that stay within rootDir', async () => {
-            const content = await adapter.fetchSkillContent({
-                path: 'skill-one/SKILL.md',
-            });
-            expect(content.rawContent).toContain('# Skill One');
-        });
-    });
-    describe('Symlink Handling (SMI-724)', () => {
-        it('should skip symlinks by default', async () => {
-            // Create a symlink to external directory
-            const externalDir = join(tmpdir(), `external-${Date.now()}`);
-            await fs.mkdir(externalDir, { recursive: true });
-            await fs.writeFile(join(externalDir, 'SKILL.md'), '# External Skill');
-            try {
-                await fs.symlink(externalDir, join(testDir, 'symlink-skill'));
-            }
-            catch {
-                // Skip test on platforms that don't support symlinks
-                return;
-            }
-            // Create new adapter (symlinks disabled by default)
-            const newAdapter = new LocalFilesystemAdapter({
-                id: 'test-symlink',
-                name: 'Test Symlink',
-                type: 'local',
-                baseUrl: 'file://',
-                enabled: true,
-                rootDir: testDir,
-                followSymlinks: false,
-                rateLimit: { maxRequests: 100, windowMs: 60000, minDelayMs: 0 },
-            });
-            await newAdapter.initialize();
-            // Should not include symlinked skill
-            expect(newAdapter.skillCount).toBe(2);
-            // Cleanup
-            await fs.rm(externalDir, { recursive: true, force: true });
-        });
-        it('should follow symlinks when enabled', async () => {
-            // Create a symlink to skill directory
-            const externalDir = join(tmpdir(), `external-follow-${Date.now()}`);
-            await fs.mkdir(externalDir, { recursive: true });
-            await fs.writeFile(join(externalDir, 'SKILL.md'), '# Symlinked Skill');
-            try {
-                await fs.symlink(externalDir, join(testDir, 'symlink-follow'));
-            }
-            catch {
-                // Skip test on platforms that don't support symlinks
-                return;
-            }
-            const followAdapter = new LocalFilesystemAdapter({
-                id: 'test-follow-symlink',
-                name: 'Test Follow Symlink',
-                type: 'local',
-                baseUrl: 'file://',
-                enabled: true,
-                rootDir: testDir,
-                followSymlinks: true,
-                rateLimit: { maxRequests: 100, windowMs: 60000, minDelayMs: 0 },
-            });
-            await followAdapter.initialize();
-            // Should include symlinked skill
-            expect(followAdapter.skillCount).toBe(3);
-            // Cleanup
-            await fs.rm(externalDir, { recursive: true, force: true });
-        });
-    });
-    describe('Deep Directory Structures (SMI-724)', () => {
-        it('should respect maxDepth limit', async () => {
-            // Create deeply nested skill
-            const deepPath = join(testDir, 'level1', 'level2', 'level3', 'level4', 'level5', 'level6');
-            await fs.mkdir(deepPath, { recursive: true });
-            await fs.writeFile(join(deepPath, 'SKILL.md'), '# Deep Skill');
-            const shallowAdapter = new LocalFilesystemAdapter({
-                id: 'test-shallow',
-                name: 'Test Shallow',
-                type: 'local',
-                baseUrl: 'file://',
-                enabled: true,
-                rootDir: testDir,
-                maxDepth: 3, // Only go 3 levels deep
-                rateLimit: { maxRequests: 100, windowMs: 60000, minDelayMs: 0 },
-            });
-            await shallowAdapter.initialize();
-            // Should not find the deep skill (at level 6)
-            expect(shallowAdapter.skillCount).toBe(2);
-        });
-        it('should find skills within maxDepth', async () => {
-            // Create skill at level 3
-            const level3Path = join(testDir, 'a', 'b', 'c');
-            await fs.mkdir(level3Path, { recursive: true });
-            await fs.writeFile(join(level3Path, 'SKILL.md'), '# Level 3 Skill');
-            const deepAdapter = new LocalFilesystemAdapter({
-                id: 'test-deep',
-                name: 'Test Deep',
-                type: 'local',
-                baseUrl: 'file://',
-                enabled: true,
-                rootDir: testDir,
-                maxDepth: 5,
-                rateLimit: { maxRequests: 100, windowMs: 60000, minDelayMs: 0 },
-            });
-            await deepAdapter.initialize();
-            // Should find the level 3 skill
-            expect(deepAdapter.skillCount).toBe(3);
-        });
-    });
-    describe('Invalid Regex Patterns (SMI-722)', () => {
-        it('should not crash with invalid regex patterns like unclosed parenthesis', async () => {
-            // Create adapter with invalid regex pattern that would crash without fix
-            const adapterWithInvalidPattern = new LocalFilesystemAdapter({
-                id: 'test-invalid-regex',
-                name: 'Test Invalid Regex',
-                type: 'local',
-                baseUrl: 'file://',
-                enabled: true,
-                rootDir: testDir,
-                excludePatterns: ['(', 'node_modules'], // '(' is invalid regex
-                rateLimit: { maxRequests: 100, windowMs: 60000, minDelayMs: 0 },
-            });
-            // Should not throw during initialization
-            await expect(adapterWithInvalidPattern.initialize()).resolves.not.toThrow();
-            expect(adapterWithInvalidPattern.skillCount).toBe(2);
-        });
-        it('should fall back to includes check for invalid regex patterns', async () => {
-            // Create a directory that contains the invalid pattern as substring
-            await fs.mkdir(join(testDir, 'test(dir'), { recursive: true });
-            await fs.writeFile(join(testDir, 'test(dir', 'SKILL.md'), '# Test Paren Dir');
-            const adapterWithInvalidPattern = new LocalFilesystemAdapter({
-                id: 'test-includes-fallback',
-                name: 'Test Includes Fallback',
-                type: 'local',
-                baseUrl: 'file://',
-                enabled: true,
-                rootDir: testDir,
-                excludePatterns: ['(', 'node_modules'], // '(' should match via includes
-                rateLimit: { maxRequests: 100, windowMs: 60000, minDelayMs: 0 },
-            });
-            await adapterWithInvalidPattern.initialize();
-            // test(dir should be excluded because it includes '('
-            expect(adapterWithInvalidPattern.skillCount).toBe(2);
-        });
-        it('should handle multiple invalid regex patterns', async () => {
-            const adapterWithMultipleInvalid = new LocalFilesystemAdapter({
-                id: 'test-multiple-invalid',
-                name: 'Test Multiple Invalid',
-                type: 'local',
-                baseUrl: 'file://',
-                enabled: true,
-                rootDir: testDir,
-                excludePatterns: ['[invalid', '(unclosed', '*bad', 'node_modules'],
-                rateLimit: { maxRequests: 100, windowMs: 60000, minDelayMs: 0 },
-            });
-            await expect(adapterWithMultipleInvalid.initialize()).resolves.not.toThrow();
-            expect(adapterWithMultipleInvalid.skillCount).toBe(2);
-        });
-        it('should still work with valid regex patterns', async () => {
-            // Create a directory matching a valid regex pattern
-            await fs.mkdir(join(testDir, 'test-temp-123'), { recursive: true });
-            await fs.writeFile(join(testDir, 'test-temp-123', 'SKILL.md'), '# Temp Skill');
-            const adapterWithValidRegex = new LocalFilesystemAdapter({
-                id: 'test-valid-regex',
-                name: 'Test Valid Regex',
-                type: 'local',
-                baseUrl: 'file://',
-                enabled: true,
-                rootDir: testDir,
-                excludePatterns: ['test-temp-\\d+', 'node_modules'], // Valid regex
-                rateLimit: { maxRequests: 100, windowMs: 60000, minDelayMs: 0 },
-            });
-            await adapterWithValidRegex.initialize();
-            // test-temp-123 should be excluded by valid regex
-            expect(adapterWithValidRegex.skillCount).toBe(2);
-        });
-    });
-});
 describe('GitLabSourceAdapter (SMI-591)', () => {
     let adapter;
     beforeEach(() => {