fix(agents): widen file.name to string before comparison to satisfy lint

lonexreb · lonexreb · commit b466f6065e0b · 2026-05-05T00:42:39.000-05:00
Lint flagged String(file.name) as 'type conversion does not change the
type or value'. Widen via .map((f) =&gt; f.name): string[] for the .includes
check, and use file.name as string for the .find predicate.
diff --git a/src/agents/bootstrap-files.test.ts b/src/agents/bootstrap-files.test.ts
@@ -187,9 +187,11 @@ describe("resolveBootstrapFilesForRun", () => {
     // Workspace AGENTS.md is suppressed (skipBootstrap), but hook-injected
     // EXTRA.md still appears. WorkspaceBootstrapFileName is a typed union of
     // canonical bootstrap names; hooks can inject arbitrary file names so
-    // compare via String() to keep the assertion type-portable.
-    expect(files.some((file) => String(file.name) === "AGENTS.md")).toBe(false);
-    const extra = files.find((file) => String(file.name) === "EXTRA.md");
+    // widen each name to string before comparing to keep the assertion
+    // type-portable without forcing the union to admit hook-only names.
+    const fileNames: string[] = files.map((file) => file.name);
+    expect(fileNames.includes("AGENTS.md")).toBe(false);
+    const extra = files.find((file) => (file.name as string) === "EXTRA.md");
     expect(extra?.content).toBe("extra");
   });
 
