fix(hygiene): handle TypeScript generics in importActual regex and fix suppression placement

Author: waleedlatif1

apps/sim/lib/copilot/request/go/stream.test.ts

@@ -11,8 +11,8 @@ import {
   MothershipStreamV1ToolPhase,
 } from '@/lib/copilot/generated/mothership-stream-v1'
 
-// hygiene-suppress: session re-exports many real domain functions (eventToStreamEvent, parsePersistedStreamEventEnvelope) used by stream.ts under test — must spread real implementations and override only hasAbortMarker
 vi.mock('@/lib/copilot/request/session', async () => {
+  // hygiene-suppress: session re-exports real domain functions (eventToStreamEvent, parsePersistedStreamEventEnvelope) used by stream.ts under test — must spread real implementations and override only hasAbortMarker
   const actual = await vi.importActual<typeof import('@/lib/copilot/request/session')>(
     '@/lib/copilot/request/session'
   )

scripts/check-hygiene.ts

@@ -158,7 +158,7 @@ const VITEST_ANTI_PATTERNS: Array<{ pattern: RegExp; description: string }> = [
       'vi.resetModules() — use vi.hoisted() + vi.mock() + static imports instead (exception: singleton modules that cache state)',
   },
   {
-    pattern: /\bvi\.importActual\s*\(/,
+    pattern: /\bvi\.importActual\s*(?:<[^>]*>)?\s*\(/,
     description: 'vi.importActual() — mock everything explicitly instead',
   },
 ]