fix: Address CodeRabbit review comments

- Configure Biome to enforce noExplicitAny: "error"
- Fix vi.mock path to include .js extension
- Add proper type signatures to all Mock types
- Fix AGENTS.md test fixture documentation
- Convert bare URLs to markdown links

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: JustinBeckwith

AGENTS.md

@@ -96,7 +96,7 @@ The action is tested end-to-end on every PR using `test-prompts/`:
 5. Verifies action can execute, parse configs, and post results
 
 **Test Files:**
-- `test-prompts/promptfooconfig.yaml` - Minimal config (gpt-4o-mini, 1 test)
+- `test-prompts/promptfooconfig.yaml` - Minimal config (echo provider, 1 test)
 - `test-prompts/test-prompt.txt` - Simple prompt with variable
 - `test-prompts/README.md` - Documentation
 
@@ -478,10 +478,10 @@ git add lib/
 
 ## Additional Resources
 
-- **Promptfoo Documentation:** https://www.promptfoo.dev/docs/
-- **GitHub Actions Toolkit:** https://github.com/actions/toolkit
-- **Vitest Documentation:** https://vitest.dev/
-- **Biome Documentation:** https://biomejs.dev/
+- [Promptfoo Documentation](https://www.promptfoo.dev/docs/)
+- [GitHub Actions Toolkit](https://github.com/actions/toolkit)
+- [Vitest Documentation](https://vitest.dev/)
+- [Biome Documentation](https://biomejs.dev/)
 
 ## Debugging Tips
 

__tests__/main.test.ts

@@ -22,7 +22,7 @@ const actualFs = await vi.importActual<typeof import('fs')>('fs');
 type MockOctokit = {
   rest: {
     issues: {
-      createComment: Mock;
+      createComment: Mock<() => Promise<unknown>>;
     };
   };
 };
@@ -53,7 +53,7 @@ vi.mock('simple-git', () => ({
 }));
 
 // Mock auth utilities
-vi.mock('../src/utils/auth');
+vi.mock('../src/utils/auth.js');
 
 import { handleError, run } from '../src/main.js';
 import * as auth from '../src/utils/auth.js';

__tests__/utils/cache.test.ts

@@ -48,15 +48,18 @@ vi.mock('fs', async () => {
   };
 });
 
-const mockCore = core as { info: Mock; debug: Mock };
+const mockCore = core as {
+  info: Mock<(message: string) => void>;
+  debug: Mock<(message: string) => void>;
+};
 const mockFs = fs as unknown as {
-  existsSync: Mock;
-  mkdirSync: Mock;
-  readdirSync: Mock;
-  statSync: Mock;
-  unlinkSync: Mock;
-  rmdirSync: Mock;
-  writeFileSync: Mock;
+  existsSync: Mock<(path: string) => boolean>;
+  mkdirSync: Mock<(path: string, options?: unknown) => void>;
+  readdirSync: Mock<(path: string) => string[]>;
+  statSync: Mock<(path: string) => { mtime: Date; size: number }>;
+  unlinkSync: Mock<(path: string) => void>;
+  rmdirSync: Mock<(path: string) => void>;
+  writeFileSync: Mock<(path: string, data: string) => void>;
 };
 
 describe('Cache Utilities', () => {

biome.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "https://biomejs.dev/schemas/2.3.13/schema.json",
+  "$schema": "https://biomejs.dev/schemas/2.3.12/schema.json",
   "files": {
     "maxSize": 2097152,
     "ignoreUnknown": true,
@@ -9,17 +9,13 @@
       "__tests__/**/*.ts",
       "*.js",
       "*.ts"
-    ],
-    "experimentalScannerIgnores": [
-      "node_modules",
-      "dist/**",
-      "build",
-      "coverage",
-      "lib/**",
-      ".next",
-      "*.min.js"
     ]
   },
+  "vcs": {
+    "enabled": true,
+    "clientKind": "git",
+    "useIgnoreFile": true
+  },
   "linter": {
     "enabled": true,
     "rules": {
@@ -39,7 +35,7 @@
         "useNodejsImportProtocol": "off"
       },
       "suspicious": {
-        "noExplicitAny": "off"
+        "noExplicitAny": "error"
       }
     }
   },
@@ -59,10 +55,5 @@
       "jsxQuoteStyle": "double",
       "arrowParentheses": "always"
     }
-  },
-  "vcs": {
-    "enabled": true,
-    "clientKind": "git",
-    "useIgnoreFile": true
   }
 }