fix: import isTestFile from production instead of reimplementing

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: superlowburn

convex/llmEval.test.ts

@@ -1,34 +1,5 @@
 import { describe, expect, it } from 'vitest'
-
-/**
- * Helper to test the isTestFile function (not exported, so we recreate it for testing)
- */
-function isTestFile(path: string): boolean {
-  const lower = path.toLowerCase()
-
-  // Common test file patterns
-  if (lower.endsWith('.test.ts')) return true
-  if (lower.endsWith('.test.js')) return true
-  if (lower.endsWith('.test.tsx')) return true
-  if (lower.endsWith('.test.jsx')) return true
-  if (lower.endsWith('.spec.ts')) return true
-  if (lower.endsWith('.spec.js')) return true
-  if (lower.endsWith('.spec.tsx')) return true
-  if (lower.endsWith('.spec.jsx')) return true
-
-  // Common test directories
-  const parts = path.split('/')
-  for (const part of parts) {
-    const lowerPart = part.toLowerCase()
-    if (lowerPart === '__tests__') return true
-    if (lowerPart === 'tests') return true
-    if (lowerPart === 'test') return true
-    if (lowerPart === '__mocks__') return true
-    if (lowerPart === 'mocks') return true
-  }
-
-  return false
-}
+import { isTestFile } from './llmEval'
 
 describe('llmEval test file filtering', () => {
   describe('isTestFile', () => {

convex/llmEval.ts

@@ -20,7 +20,7 @@ import {
  * Check if a file path is a test file that should be excluded from security scanning.
  * Test files often contain malicious code patterns for testing purposes (e.g., security skills).
  */
-function isTestFile(path: string): boolean {
+export function isTestFile(path: string): boolean {
   const lower = path.toLowerCase()
 
   // Common test file patterns