From 95ac5b27fa1471e3481ef2dc0159a9f94b1c22c0 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Tue, 18 Aug 2026 22:35:54 -0700 Subject: [PATCH] test(skills): remove scanner cache reset seam (#126199) --- src/skills/security/scanner.test.ts | 15 ++------------- src/skills/security/scanner.ts | 5 ----- 2 files changed, 2 insertions(+), 18 deletions(-) diff --git a/src/skills/security/scanner.test.ts b/src/skills/security/scanner.test.ts index db0b89b7aff6..dc162acbd089 100644 --- a/src/skills/security/scanner.test.ts +++ b/src/skills/security/scanner.test.ts @@ -3,14 +3,8 @@ import fsSync from "node:fs"; import fs from "node:fs/promises"; import os from "node:os"; import path from "node:path"; -import { afterAll, afterEach, describe, expect, it, vi } from "vitest"; -import { - clearSkillScanCacheForTest, - isScannable, - scanDirectoryWithSummary, - scanSkillContent, - scanSource, -} from "./scanner.js"; +import { afterAll, describe, expect, it, vi } from "vitest"; +import { isScannable, scanDirectoryWithSummary, scanSkillContent, scanSource } from "./scanner.js"; import type { SkillScanOptions } from "./scanner.js"; // --------------------------------------------------------------------------- @@ -140,10 +134,6 @@ type SummaryCase = { }; }; -afterEach(() => { - clearSkillScanCacheForTest(); -}); - // --------------------------------------------------------------------------- // scanSource // --------------------------------------------------------------------------- @@ -815,7 +805,6 @@ describe("scanDirectoryWithSummary", () => { testCase.expected.expectedPresent, ); } - clearSkillScanCacheForTest(); }); } }); diff --git a/src/skills/security/scanner.ts b/src/skills/security/scanner.ts index bd650386694a..517bee579e10 100644 --- a/src/skills/security/scanner.ts +++ b/src/skills/security/scanner.ts @@ -124,11 +124,6 @@ function setCachedDirEntries(dirPath: string, entry: DirEntryCacheEntry): void { DIR_ENTRY_CACHE.set(dirPath, entry); } -export function clearSkillScanCacheForTest(): void { - FILE_SCAN_CACHE.clear(); - DIR_ENTRY_CACHE.clear(); -} - // --------------------------------------------------------------------------- // Rule definitions // ---------------------------------------------------------------------------