fix(tests): replace await import with vi.importMock in fetch-threat-feed tests

- Add beforeEach hook to clear mocks between tests
- Replace all `await import()` with `vi.importMock()` for mocked dependencies
- Ensures consistent mock state across all test cases

All 8 tests in fetch-threat-feed.test.mts now pass.

Author: jdalton

packages/cli/src/commands/threat-feed/fetch-threat-feed.test.mts

@@ -1,4 +1,4 @@
-import { describe, expect, it, vi } from 'vitest'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
 
 import { fetchThreatFeed } from './fetch-threat-feed.mts'
 import {
@@ -12,8 +12,12 @@ vi.mock('../../utils/socket/api.mts', () => ({
 }))
 
 describe('fetchThreatFeed', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+  })
+
   it('fetches threat feed successfully', async () => {
-    const { queryApiSafeJson } = await import('../../utils/socket/api.mts')
+    const { queryApiSafeJson } = await vi.importMock('../../utils/socket/api.mts')
     const mockQueryApi = vi.mocked(queryApiSafeJson)
 
     const mockData = {
@@ -61,7 +65,7 @@ describe('fetchThreatFeed', () => {
   })
 
   it('handles SDK setup failure', async () => {
-    const { queryApiSafeJson } = await import('../../utils/socket/api.mts')
+    const { queryApiSafeJson } = await vi.importMock('../../utils/socket/api.mts')
     const mockQueryApi = vi.mocked(queryApiSafeJson)
 
     const error = createErrorResult('Failed to fetch threat feed', {
@@ -85,7 +89,7 @@ describe('fetchThreatFeed', () => {
   })
 
   it('handles API call failure', async () => {
-    const { queryApiSafeJson } = await import('../../utils/socket/api.mts')
+    const { queryApiSafeJson } = await vi.importMock('../../utils/socket/api.mts')
     const mockQueryApi = vi.mocked(queryApiSafeJson)
 
     mockQueryApi.mockResolvedValue(
@@ -108,7 +112,7 @@ describe('fetchThreatFeed', () => {
   })
 
   it('passes custom SDK options', async () => {
-    const { queryApiSafeJson } = await import('../../utils/socket/api.mts')
+    const { queryApiSafeJson } = await vi.importMock('../../utils/socket/api.mts')
     const mockQueryApi = vi.mocked(queryApiSafeJson)
 
     mockQueryApi.mockResolvedValue(createSuccessResult({}))
@@ -131,7 +135,7 @@ describe('fetchThreatFeed', () => {
   })
 
   it('handles filtering by severity levels', async () => {
-    const { queryApiSafeJson } = await import('../../utils/socket/api.mts')
+    const { queryApiSafeJson } = await vi.importMock('../../utils/socket/api.mts')
     const mockQueryApi = vi.mocked(queryApiSafeJson)
 
     mockQueryApi.mockResolvedValue(createSuccessResult({ threats: [] }))
@@ -154,7 +158,7 @@ describe('fetchThreatFeed', () => {
   })
 
   it('handles pagination parameters', async () => {
-    const { queryApiSafeJson } = await import('../../utils/socket/api.mts')
+    const { queryApiSafeJson } = await vi.importMock('../../utils/socket/api.mts')
     const mockQueryApi = vi.mocked(queryApiSafeJson)
 
     mockQueryApi.mockResolvedValue(createSuccessResult({ threats: [] }))
@@ -177,7 +181,7 @@ describe('fetchThreatFeed', () => {
   })
 
   it('handles date range filtering', async () => {
-    const { queryApiSafeJson } = await import('../../utils/socket/api.mts')
+    const { queryApiSafeJson } = await vi.importMock('../../utils/socket/api.mts')
     const mockQueryApi = vi.mocked(queryApiSafeJson)
 
     mockQueryApi.mockResolvedValue(createSuccessResult({ threats: [] }))
@@ -200,7 +204,7 @@ describe('fetchThreatFeed', () => {
   })
 
   it('uses null prototype for options', async () => {
-    const { queryApiSafeJson } = await import('../../utils/socket/api.mts')
+    const { queryApiSafeJson } = await vi.importMock('../../utils/socket/api.mts')
     const mockQueryApi = vi.mocked(queryApiSafeJson)
 
     mockQueryApi.mockResolvedValue(createSuccessResult({}))