chore: Set the stage for a primary key with value uuid() (#18245)

* chore: Set the stage for a primary key  with value uuid()

* fix: Test and ensure it validates the id value is set and defined

* Add unique index manually, prisma doesn't support this

* revert unique

packages/app-store/googlecalendar/lib/CalendarService.test.ts

@@ -136,18 +136,23 @@ test("Calendar can be watched and unwatched", async () => {
       integration: "google_calendar",
     },
   });
-  expect(watchedCalendar).toEqual({
-    userId: 1,
-    integration: "google_calendar",
-    externalId: "example@cal.com",
-    credentialId: 1,
-    domainWideDelegationCredentialId: null,
-    googleChannelId: "mock-channel-id",
-    googleChannelKind: "api#channel",
-    googleChannelResourceId: "mock-resource-id",
-    googleChannelResourceUri: "mock-resource-uri",
-    googleChannelExpiration: "1111111111",
-  });
+  expect(watchedCalendar).toEqual(
+    expect.objectContaining({
+      userId: 1,
+      integration: "google_calendar",
+      externalId: "example@cal.com",
+      credentialId: 1,
+      domainWideDelegationCredentialId: null,
+      googleChannelId: "mock-channel-id",
+      googleChannelKind: "api#channel",
+      googleChannelResourceId: "mock-resource-id",
+      googleChannelResourceUri: "mock-resource-uri",
+      googleChannelExpiration: "1111111111",
+    })
+  );
+
+  expect(watchedCalendar?.id).toBeDefined();
+
   await calendarCache.unwatchCalendar({ calendarId: testSelectedCalendar.externalId });
   // There's a bug in prismock where upsert creates duplicate records so we need to acces the second element
   const [, unWatchedCalendar] = await prismock.selectedCalendar.findMany({
@@ -158,18 +163,22 @@ test("Calendar can be watched and unwatched", async () => {
     },
   });
 
-  expect(unWatchedCalendar).toEqual({
-    userId: 1,
-    integration: "google_calendar",
-    externalId: "example@cal.com",
-    credentialId: 1,
-    domainWideDelegationCredentialId: null,
-    googleChannelId: null,
-    googleChannelKind: null,
-    googleChannelResourceId: null,
-    googleChannelResourceUri: null,
-    googleChannelExpiration: null,
-  });
+  expect(unWatchedCalendar).toEqual(
+    expect.objectContaining({
+      userId: 1,
+      integration: "google_calendar",
+      externalId: "example@cal.com",
+      credentialId: 1,
+      domainWideDelegationCredentialId: null,
+      googleChannelId: null,
+      googleChannelKind: null,
+      googleChannelResourceId: null,
+      googleChannelResourceUri: null,
+      googleChannelExpiration: null,
+    })
+  );
+
+  expect(unWatchedCalendar?.id).toBeDefined();
 });
 
 test("`updateTokenObject` should update credential in DB as well as myGoogleAuth", async () => {

packages/prisma/migrations/20241218125130_add_optional_uuid_two_step_primary_key/migration.sql

@@ -0,0 +1,5 @@
+-- AlterTable
+ALTER TABLE "SelectedCalendar" ADD COLUMN     "id" TEXT;
+
+-- Update all historical rows (id) to `gen_random_uuid`
+UPDATE "SelectedCalendar" SET "id" = gen_random_uuid() WHERE "id" IS NULL;

packages/prisma/schema.prisma

@@ -709,6 +709,8 @@ model Availability {
 }
 
 model SelectedCalendar {
+  // Will become required and @id in a seperate PR
+  id                               String?               @default(uuid())
   user                             User                  @relation(fields: [userId], references: [id], onDelete: Cascade)
   userId                           Int
   integration                      String