fix(core): remove deprecated uuid mock helpers that use vi.doMock

Author: waleedlatif1

packages/testing/src/index.ts

@@ -76,7 +76,6 @@ export {
   mockDrizzleOrm,
   mockHybridAuth,
   mockKnowledgeSchemas,
-  mockUuid,
   requestUtilsMock,
   setupCommonApiMocks,
   setupGlobalFetchMock,

packages/testing/src/mocks/index.ts

@@ -82,5 +82,3 @@ export {
 export { clearStorageMocks, createMockStorage, setupGlobalStorageMocks } from './storage.mock'
 // Telemetry mocks
 export { telemetryMock } from './telemetry.mock'
-// UUID mocks
-export { mockGenerateId, mockUuid } from './uuid.mock'

packages/testing/src/mocks/uuid.mock.ts

@@ -1,25 +1,6 @@
 /**
- * UUID mock utilities for testing.
+ * UUID mock utilities — intentionally empty.
  *
- * @deprecated Prefer `vi.hoisted()` + `vi.mock('@/lib/core/utils/uuid', ...)` directly
- * in test files. These helpers use `vi.doMock` which violates project testing rules.
+ * All test files should mock `@/lib/core/utils/uuid` directly using
+ * `vi.hoisted()` + `vi.mock()` per project testing rules.
  */
-import { vi } from 'vitest'
-
-/**
- * @deprecated Use `vi.hoisted()` + `vi.mock('@/lib/core/utils/uuid', ...)` instead.
- */
-export function mockGenerateId(mockValue = 'test-uuid') {
-  vi.doMock('@/lib/core/utils/uuid', () => ({
-    generateId: vi.fn().mockReturnValue(mockValue),
-    generateShortId: vi.fn().mockImplementation((size = 21) => mockValue.slice(0, size)),
-    isValidUuid: vi
-      .fn()
-      .mockImplementation((v: string) =>
-        /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(v)
-      ),
-  }))
-}
-
-/** @deprecated Use `mockGenerateId` instead. */
-export const mockUuid = mockGenerateId