Merge branch 'main' into devin/1745584099-add-organization-id-overall-reviews

Author: NoritakaIkeda

.github/workflows/notify_supabase_failure.yml

@@ -0,0 +1,44 @@
+name: Notify Supabase Failure
+
+on:
+  check_run:
+    types: [completed]
+
+jobs:
+  notify_slack:
+    runs-on: ubuntu-latest
+    if: |
+      github.event.check_run.app.name == 'Supabase' &&
+      github.event.check_run.conclusion == 'failure'
+      # github.event.check_run.check_suite.head_branch == 'main'
+    steps:
+      - name: Slack Notification for Supabase Failure
+        uses: tokorom/action-slack-incoming-webhook@d57bf1eb618f3dae9509afefa70d5774ad3d42bf # v1.3.0
+        env:
+          INCOMING_WEBHOOK_URL: ${{ secrets.SLACK_CLI_CI_WEBHOOK_URL }}
+        with:
+          text: "Supabase Preview Deployment Failure"
+          attachments: |
+            [
+              {
+                "color": "bad",
+                "fields": [
+                  {
+                    "title": "Branch",
+                    "value": "main"
+                  },
+                  {
+                    "title": "Check Name",
+                    "value": "${{ github.event.check_run.name }}"
+                  },
+                  {
+                    "title": "Result",
+                    "value": "failure"
+                  },
+                  {
+                    "title": "Check Run URL",
+                    "value": "${{ github.event.check_run.html_url }}"
+                  }
+                ]
+              }
+            ]

frontend/apps/app/app/(app)/app/(root)/invitations/tokens/[token]/page.tsx

@@ -1,7 +1,7 @@
 import type { PageProps } from '@/app/types'
 import { getInvitationData } from '@/features/invitations/actions'
 import { InvitationCard } from '@/features/invitations/components/InvitationCard'
-import { notFound } from 'next/navigation'
+
 import * as v from 'valibot'
 
 const paramsSchema = v.object({
@@ -10,7 +10,7 @@ const paramsSchema = v.object({
 
 export default async function Page({ params }: PageProps) {
   const parsedParams = v.safeParse(paramsSchema, await params)
-  if (!parsedParams.success) return notFound()
+  if (!parsedParams.success) throw new Error('Invalid token parameters')
 
   const { token } = parsedParams.output
   const { organizationName, currentUser } = await getInvitationData(token)

frontend/apps/app/app/(app)/app/(root)/projects/new/page.tsx

@@ -2,7 +2,6 @@ import { getOrganizationId } from '@/features/organizations/services/getOrganiza
 import { ProjectNewPage } from '@/features/projects/pages'
 import { createClient } from '@/libs/db/server'
 import { getInstallations } from '@liam-hq/github'
-import { notFound } from 'next/navigation'
 
 export default async function NewProjectPage() {
   const organizationId = await getOrganizationId()
@@ -19,13 +18,13 @@ export default async function NewProjectPage() {
   } = await supabase.auth.getUser()
   if (error || !user) {
     console.error('Error fetching user:', error)
-    return notFound()
+    throw new Error('User not authenticated')
   }
 
   const { data } = await supabase.auth.getSession()
 
   if (data.session === null) {
-    return notFound()
+    throw new Error('Session not found')
   }
 
   const { installations } = await getInstallations(data.session)

frontend/apps/app/app/(app)/app/(root)/projects/page.tsx

@@ -1,7 +1,6 @@
 import { getOrganizationId } from '@/features/organizations/services/getOrganizationId'
 import { ProjectsPage } from '@/features/projects/pages'
 import { createClient } from '@/libs/db/server'
-import { notFound } from 'next/navigation'
 
 export default async function Page() {
   const organizationId = await getOrganizationId()
@@ -20,10 +19,10 @@ export default async function Page() {
 
   if (error || !user) {
     console.error('Error fetching user:', error)
-    return notFound()
+    throw new Error('User not authenticated')
   }
   if (data.session === null) {
-    return notFound()
+    throw new Error('User not authenticated')
   }
 
   return <ProjectsPage organizationId={organizationId} />

frontend/apps/app/app/(app)/app/(root)/settings/general/GeneralPage.tsx

@@ -1,5 +1,5 @@
 import { getOrganizationDetails } from '@/features/organizations/services/organizationService'
-import { notFound } from 'next/navigation'
+
 import { GeneralPageClient } from './GeneralPageClient'
 
 export async function GeneralPage({
@@ -11,7 +11,7 @@ export async function GeneralPage({
   const organization = await getOrganizationDetails(organizationId)
 
   if (!organization) {
-    return notFound()
+    throw new Error('Organization not found')
   }
 
   // Use the client component with useActionState for toast notifications

frontend/apps/app/app/(app)/app/(root)/settings/members/page.tsx

@@ -1,7 +1,6 @@
 import { OrganizationMembersPage } from '@/features/organizations/pages/OrganizationMembersPage'
 import { getOrganizationId } from '@/features/organizations/services/getOrganizationId'
 import { createClient } from '@/libs/db/server'
-import { notFound } from 'next/navigation'
 
 export default async function MembersPage() {
   const organizationId = await getOrganizationId()
@@ -21,7 +20,7 @@ export default async function MembersPage() {
 
   if (error || !organization) {
     console.error('Error fetching organization:', error)
-    notFound()
+    throw new Error('Organization not found')
   }
 
   return <OrganizationMembersPage organization={organization} />

frontend/apps/app/app/(app)/app/(with-project)/projects/[projectId]/branches/page.tsx

@@ -1,6 +1,6 @@
 import type { PageProps } from '@/app/types'
 import { ProjectBranchesListPage } from '@/features/projects/pages/ProjectBranchesListPage/ProjectBranchesListPage'
-import { notFound } from 'next/navigation'
+
 import * as v from 'valibot'
 
 const paramsSchema = v.object({
@@ -9,6 +9,6 @@ const paramsSchema = v.object({
 
 export default async function BranchesPage({ params }: PageProps) {
   const parsedParams = v.safeParse(paramsSchema, await params)
-  if (!parsedParams.success) return notFound()
+  if (!parsedParams.success) throw new Error('Invalid parameters')
   return <ProjectBranchesListPage projectId={parsedParams.output.projectId} />
 }

frontend/apps/app/app/(app)/app/(with-project)/projects/[projectId]/page.tsx

@@ -1,5 +1,5 @@
 import type { PageProps } from '@/app/types'
-import { notFound, redirect } from 'next/navigation'
+import { redirect } from 'next/navigation'
 import * as v from 'valibot'
 
 const paramsSchema = v.object({
@@ -8,7 +8,7 @@ const paramsSchema = v.object({
 
 export default async function Page({ params }: PageProps) {
   const parsedParams = v.safeParse(paramsSchema, await params)
-  if (!parsedParams.success) return notFound()
+  if (!parsedParams.success) throw new Error('Invalid project parameters')
 
   const { projectId } = parsedParams.output
 

frontend/apps/app/app/(app)/app/(with-project)/projects/[projectId]/rule/page.tsx

@@ -1,5 +1,5 @@
 import type { PageProps } from '@/app/types'
-import { notFound } from 'next/navigation'
+
 import * as v from 'valibot'
 import { RulePage } from './RulePage'
 
@@ -11,7 +11,7 @@ const paramsSchema = v.object({
 export default async function Page({ params }: PageProps) {
   // Validate and parse params
   const parsedParams = v.safeParse(paramsSchema, await params)
-  if (!parsedParams.success) return notFound()
+  if (!parsedParams.success) throw new Error('Invalid parameters')
 
   const { projectId } = parsedParams.output
 

frontend/apps/app/app/(app)/app/(with-project)/projects/[projectId]/settings/page.tsx

@@ -1,5 +1,5 @@
 import type { PageProps } from '@/app/types'
-import { notFound } from 'next/navigation'
+
 import * as v from 'valibot'
 import { SettingsPage } from './SettingsPage'
 
@@ -11,7 +11,7 @@ const paramsSchema = v.object({
 export default async function Page({ params }: PageProps) {
   // Validate and parse params
   const parsedParams = v.safeParse(paramsSchema, await params)
-  if (!parsedParams.success) return notFound()
+  if (!parsedParams.success) throw new Error('Invalid parameters')
 
   const { projectId } = parsedParams.output