fixing migrations ::migrate::

Author: kof

packages/prisma-client/prisma/migrations/20251129093846_add_updated_at_to_latest_build_virtual/migration.sql

@@ -1,19 +1,9 @@
 -- Add updatedAt field to latestBuildVirtual virtual table (type definition)
 -- and update both functions to include Build's updatedAt timestamp
--- Add updatedAt column to the virtual table type definition (only if it doesn't exist)
-DO $$
-BEGIN
-  IF NOT EXISTS (
-    SELECT 1
-    FROM information_schema.columns
-    WHERE table_schema = 'public'
-      AND table_name = 'latestBuildVirtual'
-      AND column_name = 'updatedAt'
-  ) THEN
-    ALTER TABLE "latestBuildVirtual"
-    ADD COLUMN "updatedAt" timestamp(3) with time zone NOT NULL;
-  END IF;
-END $$;
+-- Add updatedAt column to the virtual table type definition (uses IF NOT EXISTS for idempotency)
+-- Note: This uses PostgreSQL 9.6+ syntax for IF NOT EXISTS
+ALTER TABLE IF EXISTS "latestBuildVirtual"
+  ADD COLUMN IF NOT EXISTS "updatedAt" timestamp(3) with time zone NOT NULL DEFAULT NOW();
 
 COMMENT ON COLUMN "latestBuildVirtual"."updatedAt" IS 'Timestamp indicating when the Build was last updated';
 
@@ -39,7 +29,7 @@ FROM
   LEFT JOIN "ProjectDomain" pd ON pd."projectId" = p.id
   LEFT JOIN "Domain" d ON d.id = pd."domainId"
 WHERE
-  b."projectId" = $ 1.id
+  b."projectId" = $1.id
   AND b.deployment IS NOT NULL -- 'destination' IS NULL for backward compatibility; 'destination' = 'saas' for non-static builds
   AND (
     (b.deployment :: jsonb ->> 'destination') IS NULL
@@ -74,9 +64,9 @@ SELECT
   b."updatedAt"
 FROM
   "Build" b
-  JOIN "Domain" d ON d.id = $ 1."domainId"
+  JOIN "Domain" d ON d.id = $1."domainId"
 WHERE
-  b."projectId" = $ 1."projectId"
+  b."projectId" = $1."projectId"
   AND b.deployment IS NOT NULL
   AND (b.deployment :: jsonb -> 'domains') @ > to_jsonb(array [d.domain])
 ORDER BY
@@ -105,7 +95,7 @@ FROM
   JOIN "Project" p ON b."projectId" = p.id
   LEFT JOIN "ProjectDomain" pd ON pd."projectId" = p.id
 WHERE
-  b."projectId" = $ 1.id
+  b."projectId" = $1.id
   AND b.deployment IS NOT NULL
   AND (
     (b.deployment :: jsonb ->> 'destination') IS NULL
@@ -122,4 +112,42 @@ LIMIT
 
 $$ STABLE LANGUAGE sql;
 
-COMMENT ON FUNCTION "latestProjectDomainBuildVirtual"("Project") IS 'This function computes the latest build for a project domain, ensuring it is a production (non-static) build, where the domain matches either the Project.domain field or exists in the related Domain table. It provides backward compatibility for older records with a missing "destination" field.';
+COMMENT ON FUNCTION "latestProjectDomainBuildVirtual"("Project") IS 'This function computes the latest build for a project domain, ensuring it is a production (non-static) build, where the domain matches either the Project.domain field or exists in the related Domain table. It provides backward compatibility for older records with a missing "destination" field.';
+
+-- Add latestBuildVirtual function overload for DashboardProject view
+-- This is needed because PostgREST computed fields require a function
+-- that matches the source table/view type. DashboardProject is a view
+-- over Project, so we need this wrapper function.
+CREATE
+OR REPLACE FUNCTION "latestBuildVirtual"("DashboardProject") RETURNS SETOF "latestBuildVirtual" ROWS 1 AS $$
+SELECT
+  *
+FROM
+  "latestBuildVirtual"(
+    (
+      SELECT
+        p
+      FROM
+        "Project" p
+      WHERE
+        p.id = $1.id
+    )
+  );
+
+$$ STABLE LANGUAGE sql;
+
+COMMENT ON FUNCTION "latestBuildVirtual"("DashboardProject") IS 'Wrapper function to make latestBuildVirtual work with DashboardProject view for PostgREST computed fields.';
+
+-- Grant execute permissions to all PostgREST roles
+-- Uses DO block to check if roles exist before granting (prevents errors if roles are missing)
+DO $$
+DECLARE
+  role_name TEXT;
+BEGIN
+  FOREACH role_name IN ARRAY ARRAY['anon', 'authenticated', 'service_role']
+  LOOP
+    IF EXISTS (SELECT 1 FROM pg_roles WHERE rolname = role_name) THEN
+      EXECUTE format('GRANT EXECUTE ON FUNCTION "latestBuildVirtual"("DashboardProject") TO %I', role_name);
+    END IF;
+  END LOOP;
+END $$;

packages/prisma-client/prisma/migrations/20251130131728_add_latest_build_virtual_for_dashboard_project/migration.sql

@@ -22,33 +22,16 @@ $$ STABLE LANGUAGE sql;
 
 COMMENT ON FUNCTION "latestBuildVirtual"("DashboardProject") IS 'Wrapper function to make latestBuildVirtual work with DashboardProject view for PostgREST computed fields.';
 
--- Grant execute permissions to all PostgREST roles (only if they exist)
+-- Grant execute permissions to all PostgREST roles
+-- Uses DO block to check if roles exist before granting (prevents errors if roles are missing)
 DO $$
+DECLARE
+  role_name TEXT;
 BEGIN
-  -- Grant to anon if role exists
-  IF EXISTS (
-    SELECT 1
-    FROM pg_roles
-    WHERE rolname = 'anon'
-  ) THEN
-    GRANT EXECUTE ON FUNCTION "latestBuildVirtual"("DashboardProject") TO anon;
-  END IF;
-  
-  -- Grant to authenticated if role exists
-  IF EXISTS (
-    SELECT 1
-    FROM pg_roles
-    WHERE rolname = 'authenticated'
-  ) THEN
-    GRANT EXECUTE ON FUNCTION "latestBuildVirtual"("DashboardProject") TO authenticated;
-  END IF;
-  
-  -- Grant to service_role if role exists
-  IF EXISTS (
-    SELECT 1
-    FROM pg_roles
-    WHERE rolname = 'service_role'
-  ) THEN
-    GRANT EXECUTE ON FUNCTION "latestBuildVirtual"("DashboardProject") TO service_role;
-  END IF;
+  FOREACH role_name IN ARRAY ARRAY['anon', 'authenticated', 'service_role']
+  LOOP
+    IF EXISTS (SELECT 1 FROM pg_roles WHERE rolname = role_name) THEN
+      EXECUTE format('GRANT EXECUTE ON FUNCTION "latestBuildVirtual"("DashboardProject") TO %I', role_name);
+    END IF;
+  END LOOP;
 END $$;