chore: shift over index page creation to collection (#1552)

* chore: shift over index page creation to collection

* chore: fix duplicate import error

* chore: swap from 2 -> 3 audit logs for tests

* chore: rename to indexPage

* chore: update wording for default subtitle

Author: seaerchin

apps/studio/src/server/modules/collection/__tests__/collection.router.test.ts

@@ -218,7 +218,7 @@ describe("collection.router", async () => {
       })
       expect(result).toMatchObject({ id: actualCollection.id })
       expect(auditSpy).toHaveBeenCalled()
-      await assertAuditLogRows(2)
+      await assertAuditLogRows(3)
       const auditEntry = await db
         .selectFrom("AuditLog")
         .where("eventType", "=", "ResourceCreate")
@@ -252,7 +252,7 @@ describe("collection.router", async () => {
       })
       expect(result).toMatchObject({ id: actualCollection.id })
       expect(auditSpy).toHaveBeenCalled()
-      await assertAuditLogRows(2)
+      await assertAuditLogRows(3)
       const auditEntry = await db
         .selectFrom("AuditLog")
         .where("eventType", "=", "ResourceCreate")
@@ -286,7 +286,7 @@ describe("collection.router", async () => {
       })
       expect(actualCollection.parentId).toEqual(parent.id)
       expect(result).toMatchObject({ id: actualCollection.id })
-      await assertAuditLogRows(2)
+      await assertAuditLogRows(3)
       expect(auditSpy).toHaveBeenCalled()
       const auditEntry = await db
         .selectFrom("AuditLog")
@@ -297,7 +297,7 @@ describe("collection.router", async () => {
       expect(auditEntry.userId).toBe(session.userId)
     })
 
-    it("should create a nested collection if `parentFolderId` is provided and hte user is not an admin", async () => {
+    it("should create a nested collection if `parentFolderId` is provided and the user is not an admin", async () => {
       // Arrange
       const permalinkToUse = "test-collection-777"
       const { folder: parent, site } = await setupFolder()
@@ -321,7 +321,7 @@ describe("collection.router", async () => {
       })
       expect(actualCollection.parentId).toEqual(parent.id)
       expect(result).toMatchObject({ id: actualCollection.id })
-      await assertAuditLogRows(2)
+      await assertAuditLogRows(3)
       expect(auditSpy).toHaveBeenCalled()
       const auditEntry = await db
         .selectFrom("AuditLog")

apps/studio/src/server/modules/collection/collection.router.ts

@@ -1,8 +1,8 @@
 import type { UnwrapTagged } from "type-fest"
 import { TRPCError } from "@trpc/server"
-import { AuditLogEvent } from "~prisma/generated/generatedEnums"
 import { get, pick } from "lodash"
 
+import { INDEX_PAGE_PERMALINK } from "~/constants/sitemap"
 import {
   createCollectionSchema,
   editLinkSchema,
@@ -13,7 +13,13 @@ import { readFolderSchema } from "~/schemas/folder"
 import { createCollectionPageSchema } from "~/schemas/page"
 import { protectedProcedure, router } from "~/server/trpc"
 import { logResourceEvent } from "../audit/audit.service"
-import { db, jsonb, ResourceState, ResourceType } from "../database"
+import {
+  AuditLogEvent,
+  db,
+  jsonb,
+  ResourceState,
+  ResourceType,
+} from "../database"
 import { PG_ERROR_CODES } from "../database/constants"
 import { bulkValidateUserPermissionsForResources } from "../permissions/permissions.service"
 import {
@@ -26,6 +32,7 @@ import {
 import { validateUserPermissionsForSite } from "../site/site.service"
 import { defaultCollectionSelect } from "./collection.select"
 import {
+  createCollectionIndexJson,
   createCollectionLinkJson,
   createCollectionPageJson,
 } from "./collection.service"
@@ -127,6 +134,44 @@ export const collectionRouter = router({
             by: user,
           })
 
+          const indexJson = createCollectionIndexJson(collection.title)
+
+          const blob = await tx
+            .insertInto("Blob")
+            .values({ content: jsonb(indexJson) })
+            .returning("Blob.id")
+            .executeTakeFirstOrThrow()
+
+          const indexPage = await tx
+            .insertInto("Resource")
+            .values({
+              title: collection.title,
+              permalink: INDEX_PAGE_PERMALINK,
+              siteId,
+              parentId: collection.id,
+              draftBlobId: blob.id,
+              type: ResourceType.IndexPage,
+              state: ResourceState.Draft,
+            })
+            .returningAll()
+            .executeTakeFirstOrThrow()
+            .catch((err) => {
+              if (get(err, "code") === PG_ERROR_CODES.uniqueViolation) {
+                throw new TRPCError({
+                  code: "CONFLICT",
+                  message: "A resource with the same permalink already exists",
+                })
+              }
+              throw err
+            })
+
+          await logResourceEvent(tx, {
+            siteId,
+            by: user,
+            delta: { before: null, after: indexPage },
+            eventType: AuditLogEvent.ResourceCreate,
+          })
+
           return collection
         })
 

apps/studio/src/server/modules/collection/collection.service.ts

@@ -1,4 +1,10 @@
 import type { UnwrapTagged } from "type-fest"
+import {
+  COLLECTION_PAGE_DEFAULT_SORT_BY,
+  COLLECTION_PAGE_DEFAULT_SORT_DIRECTION,
+  CollectionPagePageProps,
+  ISOMER_USABLE_PAGE_LAYOUTS,
+} from "@opengovsg/isomer-components"
 import { format } from "date-fns"
 
 import type { ResourceType } from "../database"
@@ -38,3 +44,17 @@ export const createCollectionLinkJson = ({}: {
     version: "0.1.0",
   } satisfies UnwrapTagged<PrismaJson.BlobJsonContent>
 }
+
+export const createCollectionIndexJson = (title: string) => {
+  return {
+    layout: ISOMER_USABLE_PAGE_LAYOUTS.Collection,
+    page: {
+      title,
+      subtitle: `Read up-to-date news articles, speeches, and press releases here.`,
+      defaultSortBy: COLLECTION_PAGE_DEFAULT_SORT_BY,
+      defaultSortDirection: COLLECTION_PAGE_DEFAULT_SORT_DIRECTION,
+    } as CollectionPagePageProps,
+    content: [],
+    version: "0.1.0",
+  }
+}