Refactor publisher creation flow with modular step components and real-time validation

Breaks down monolithic publisher page into reusable step components, extracts validation logic into shared utilities, and adds real-time publisher ID validation API endpoint. Improves code maintainability and user experience.

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>

Author: brandonkachen

web/src/app/api/publishers/route.ts

@@ -7,6 +7,10 @@ import { getServerSession } from 'next-auth'
 
 import { authOptions } from '@/app/api/auth/[...nextauth]/auth-options'
 import { checkOrgPublisherAccess } from '@/lib/publisher-permissions'
+import {
+  validatePublisherName,
+  validatePublisherId,
+} from '@/lib/validators/publisher'
 import { logger } from '@/util/logger'
 
 import type {
@@ -15,44 +19,6 @@ import type {
 } from '@codebuff/common/types/publisher'
 import type { NextRequest } from 'next/server'
 
-function validatePublisherName(name: string): string | null {
-  if (!name || !name.trim()) {
-    return 'Publisher name is required'
-  }
-
-  const trimmedName = name.trim()
-
-  if (trimmedName.length < 2) {
-    return 'Publisher name must be at least 2 characters long'
-  }
-
-  if (trimmedName.length > 50) {
-    return 'Publisher name must be no more than 50 characters long'
-  }
-
-  return null
-}
-function validatePublisherId(id: string): string | null {
-  const result = PublisherIdSchema.safeParse(id)
-  if (!result.success) {
-    return result.error.errors[0]?.message || 'Invalid publisher ID'
-  }
-
-  if (id.length < 3) {
-    return 'Publisher ID must be at least 3 characters long'
-  }
-
-  if (id.length > 30) {
-    return 'Publisher ID must be no more than 30 characters long'
-  }
-
-  if (id.startsWith('-') || id.endsWith('-')) {
-    return 'Publisher ID cannot start or end with a hyphen'
-  }
-
-  return null
-}
-
 export async function GET(): Promise<
   NextResponse<PublisherProfileResponse[] | { error: string }>
 > {

web/src/app/api/publishers/validate/route.ts

@@ -0,0 +1,53 @@
+import { NextRequest, NextResponse } from 'next/server'
+import db from '@codebuff/common/db'
+import * as schema from '@codebuff/common/db/schema'
+import { eq } from 'drizzle-orm'
+import { validatePublisherId } from '@/lib/validators/publisher'
+
+export async function GET(request: NextRequest) {
+  try {
+    const { searchParams } = new URL(request.url)
+    const id = searchParams.get('id')
+
+    if (!id) {
+      return NextResponse.json(
+        { valid: false, error: 'Publisher ID is required' },
+        { status: 400 }
+      )
+    }
+
+    // Validate format first
+    const formatError = validatePublisherId(id)
+    if (formatError) {
+      return NextResponse.json(
+        { valid: false, error: formatError },
+        { status: 200 }
+      )
+    }
+
+    // Check if ID is already taken
+    const existingPublisher = await db
+      .select({ id: schema.publisher.id })
+      .from(schema.publisher)
+      .where(eq(schema.publisher.id, id))
+      .limit(1)
+
+    if (existingPublisher.length > 0) {
+      return NextResponse.json(
+        { valid: false, error: 'This publisher ID is already taken' },
+        { status: 200 }
+      )
+    }
+
+    return NextResponse.json(
+      { valid: true, error: null },
+      { status: 200 }
+    )
+  } catch (error) {
+    console.error('Error validating publisher ID:', error)
+    return NextResponse.json(
+      { valid: false, error: 'Failed to validate publisher ID' },
+      { status: 500 }
+    )
+  }
+}