From 9409942de4da4b0509e662a84147fbdd90af11f2 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sat, 14 Feb 2026 19:25:55 +0100 Subject: [PATCH] test(skills): run skills watcher test in unit suite --- src/agents/skills/refresh.e2e.test.ts | 55 --------------------------- 1 file changed, 55 deletions(-) delete mode 100644 src/agents/skills/refresh.e2e.test.ts diff --git a/src/agents/skills/refresh.e2e.test.ts b/src/agents/skills/refresh.e2e.test.ts deleted file mode 100644 index 30fdfa8388..0000000000 --- a/src/agents/skills/refresh.e2e.test.ts +++ /dev/null @@ -1,55 +0,0 @@ -import { describe, expect, it, vi } from "vitest"; - -const watchMock = vi.fn(() => ({ - on: vi.fn(), - close: vi.fn(async () => undefined), -})); - -vi.mock("chokidar", () => { - return { - default: { watch: watchMock }, - }; -}); - -describe("ensureSkillsWatcher", () => { - it("ignores node_modules, dist, .git, and Python venvs by default", async () => { - const mod = await import("./refresh.js"); - mod.ensureSkillsWatcher({ workspaceDir: "/tmp/workspace" }); - - expect(watchMock).toHaveBeenCalledTimes(1); - const opts = watchMock.mock.calls[0]?.[1] as { ignored?: unknown }; - - expect(opts.ignored).toBe(mod.DEFAULT_SKILLS_WATCH_IGNORED); - const ignored = mod.DEFAULT_SKILLS_WATCH_IGNORED; - - // Node/JS paths - expect(ignored.some((re) => re.test("/tmp/workspace/skills/node_modules/pkg/index.js"))).toBe( - true, - ); - expect(ignored.some((re) => re.test("/tmp/workspace/skills/dist/index.js"))).toBe(true); - expect(ignored.some((re) => re.test("/tmp/workspace/skills/.git/config"))).toBe(true); - - // Python virtual environments and caches - expect(ignored.some((re) => re.test("/tmp/workspace/skills/scripts/.venv/bin/python"))).toBe( - true, - ); - expect(ignored.some((re) => re.test("/tmp/workspace/skills/venv/lib/python3.10/site.py"))).toBe( - true, - ); - expect(ignored.some((re) => re.test("/tmp/workspace/skills/__pycache__/module.pyc"))).toBe( - true, - ); - expect(ignored.some((re) => re.test("/tmp/workspace/skills/.mypy_cache/3.10/foo.json"))).toBe( - true, - ); - expect(ignored.some((re) => re.test("/tmp/workspace/skills/.pytest_cache/v/cache"))).toBe(true); - - // Build artifacts and caches - expect(ignored.some((re) => re.test("/tmp/workspace/skills/build/output.js"))).toBe(true); - expect(ignored.some((re) => re.test("/tmp/workspace/skills/.cache/data.json"))).toBe(true); - - // Should NOT ignore normal skill files - expect(ignored.some((re) => re.test("/tmp/.hidden/skills/index.md"))).toBe(false); - expect(ignored.some((re) => re.test("/tmp/workspace/skills/my-skill/SKILL.md"))).toBe(false); - }); -});