fix: ignore scanner metadata files

.github/workflows/ci.yml

@@ -21,3 +21,5 @@ jobs:
         run: bun run build
       - name: Lint
         run: bun run lint
+      - name: Test
+        run: bun test

package.json

@@ -6,7 +6,8 @@
   "scripts": {
     "dev": "node esbuild.config.mjs",
     "build": "node esbuild.config.mjs production",
-    "lint": "eslint src/"
+    "lint": "eslint src/",
+    "test": "bun test"
   },
   "devDependencies": {
     "@codemirror/autocomplete": "^6.20.1",

src/scanner-ignore.test.ts

@@ -0,0 +1,19 @@
+import { describe, expect, test } from "bun:test";
+import { isScannableMarkdownFile } from "./scanner-ignore";
+
+describe("scanner metadata files", () => {
+	test("excludes common package metadata case-insensitively", () => {
+		const metadata = [
+			"HISTORY.md",
+			"changes.md",
+			"AUTHORS",
+			"notice",
+			".gitignore",
+			".gitkeep",
+		];
+
+		const scanResults = [...metadata, "my-skill.md"].filter(isScannableMarkdownFile);
+
+		expect(scanResults).toEqual(["my-skill.md"]);
+	});
+});

src/scanner-ignore.ts

@@ -0,0 +1,19 @@
+const IGNORED_FILES = new Set([
+	"readme.md",
+	"license",
+	"license.md",
+	"changelog.md",
+	"history.md",
+	"changes.md",
+	"authors",
+	"notice",
+	".gitignore",
+	".gitkeep",
+	".ds_store",
+	"thumbs.db",
+]);
+
+export function isScannableMarkdownFile(fileName: string): boolean {
+	const normalized = fileName.toLowerCase();
+	return normalized.endsWith(".md") && !IGNORED_FILES.has(normalized);
+}

src/scanner.ts

@@ -10,17 +10,9 @@ import { homedir } from "os";
 import { parseYaml } from "obsidian";
 import { createHash } from "crypto";
 import { TOOL_CONFIGS, clearInstallCache } from "./tool-configs";
+import { isScannableMarkdownFile } from "./scanner-ignore";
 import type { SkillItem, SkillPath, SkillType, NamingMode, ScanPattern, ChopsSettings, ToolConfig } from "./types";
 
-const IGNORED_FILES = new Set([
-	"readme.md",
-	"license",
-	"license.md",
-	"changelog.md",
-	".ds_store",
-	"thumbs.db",
-]);
-
 function hashPath(p: string): string {
 	return createHash("sha256").update(p).digest("hex").slice(0, 12);
 }
@@ -115,9 +107,7 @@ function scanFlatMd(
 				if (item) items.push(item);
 				continue;
 			}
-			const mdFiles = readdirSync(fullPath).filter(
-				(f) => f.endsWith(".md") && !IGNORED_FILES.has(f.toLowerCase())
-			);
+			const mdFiles = readdirSync(fullPath).filter(isScannableMarkdownFile);
 			const preferred =
 				mdFiles.find(
 					(f) => f.toLowerCase() === `${entry.name.toLowerCase()}.md`
@@ -135,8 +125,7 @@ function scanFlatMd(
 			continue;
 		}
 
-		const fname = entry.name.toLowerCase();
-		if (!fname.endsWith(".md") || IGNORED_FILES.has(fname)) continue;
+		if (!isScannableMarkdownFile(entry.name)) continue;
 		const item = parseSkillFile(fullPath, type, toolId, "flat-md", namingMode);
 		if (item) items.push(item);
 	}