Merge pull request #53 from Open330/oac/1772690720042-96b333b728a73b7b

test: add test coverage for file-filters.ts

Author: jiunbae

.oac/contributions/2026-03-05-0607-96b333b728a73b7b.json

@@ -0,0 +1,20 @@
+{
+  "version": "1.0",
+  "runId": "e9c717f6-ebac-4f44-97b3-55ff1d9a41cf",
+  "timestamp": "2026-03-05T06:07:46.285Z",
+  "contributor": "jiun",
+  "task": {
+    "id": "96b333b728a73b7b",
+    "title": "Add tests for file-filters.ts",
+    "source": "test-gap",
+    "complexity": "simple"
+  },
+  "execution": {
+    "success": true,
+    "tokensUsed": 285138,
+    "duration": 77.428,
+    "filesChanged": [
+      "tests/core/file-filters.test.ts"
+    ]
+  }
+}

tests/core/file-filters.test.ts

@@ -0,0 +1,46 @@
+import { describe, expect, it } from "vitest";
+
+import { filterRealChanges, isRealFileChange } from "../../src/core/file-filters.js";
+
+describe("isRealFileChange", () => {
+  it("returns true for regular source files", () => {
+    expect(isRealFileChange("src/index.ts")).toBe(true);
+    expect(isRealFileChange("README.md")).toBe(true);
+    expect(isRealFileChange("package.json")).toBe(true);
+  });
+
+  it("returns false for .oac/ metadata files", () => {
+    expect(isRealFileChange(".oac/config.json")).toBe(false);
+    expect(isRealFileChange(".oac/tasks/task-1.json")).toBe(false);
+  });
+
+  it("returns true for files containing .oac in a non-prefix position", () => {
+    expect(isRealFileChange("src/.oac-utils.ts")).toBe(true);
+    expect(isRealFileChange("docs/oac-guide.md")).toBe(true);
+  });
+
+  it("returns true for an empty string", () => {
+    expect(isRealFileChange("")).toBe(true);
+  });
+});
+
+describe("filterRealChanges", () => {
+  it("returns an empty array when given an empty array", () => {
+    expect(filterRealChanges([])).toEqual([]);
+  });
+
+  it("keeps all files when none are metadata", () => {
+    const files = ["src/index.ts", "package.json", "tests/foo.test.ts"];
+    expect(filterRealChanges(files)).toEqual(files);
+  });
+
+  it("removes .oac/ metadata files", () => {
+    const files = ["src/index.ts", ".oac/config.json", "README.md", ".oac/logs/run.log"];
+    expect(filterRealChanges(files)).toEqual(["src/index.ts", "README.md"]);
+  });
+
+  it("returns an empty array when all files are metadata", () => {
+    const files = [".oac/a", ".oac/b/c"];
+    expect(filterRealChanges(files)).toEqual([]);
+  });
+});