Fix type definitions in malware mock API

Extend types to include API fields not in base types:
- Add category and file fields to ExtendedAlert
- Add size and batchIndex to CompactSocketArtifactWithScore
- Remove unused vi import

Author: jdalton

test/mock-malware-api.mts

@@ -3,15 +3,30 @@
  * Provides utilities to mock Socket API responses with malware alerts.
  */
 
-import { vi } from 'vitest'
+import type {
+  CompactSocketArtifact,
+  CompactSocketArtifactAlert,
+} from '../src/utils/alert/artifact.mts'
 
-import type { CompactSocketArtifact } from '../src/utils/alert/artifact.mts'
+/**
+ * Extended alert type with additional API fields not in base type.
+ */
+export type ExtendedAlert = CompactSocketArtifactAlert & {
+  category?: string
+  file?: string
+}
 
 /**
- * Extended CompactSocketArtifact type with score property for testing.
- * In practice, the API returns score data but it's not in the base type.
+ * Extended CompactSocketArtifact type with additional API fields for testing.
+ * The API returns score, size, and batchIndex data not included in the base type.
  */
-export type CompactSocketArtifactWithScore = CompactSocketArtifact & {
+export type CompactSocketArtifactWithScore = Omit<
+  CompactSocketArtifact,
+  'alerts'
+> & {
+  alerts: ExtendedAlert[]
+  batchIndex?: number
+  size?: number
   score?: {
     license: number
     maintenance: number