Fix TypeScript syntax error and standardize Supabase client usage

Co-authored-by: Gerome-Elassaad <186273274+Gerome-Elassaad@users.noreply.github.com>

Author: Copilot

app/api/deployments/[id]/rollback/route.ts

@@ -10,7 +10,7 @@ export async function POST(
   { params }: { params: { id: string } }
 ) {
   try {
-    const supabase = createServerClient()
+    const supabase = createServerClient(true)
     const { data: { session } } = await supabase.auth.getSession()
 
     if (!session?.user?.id) {

app/api/deployments/[id]/route.ts

@@ -10,7 +10,7 @@ export async function GET(
   { params }: { params: { id: string } }
 ) {
   try {
-    const supabase = createServerClient()
+    const supabase = createServerClient(true)
     const { data: { session } } = await supabase.auth.getSession()
 
     if (!session?.user?.id) {
@@ -42,7 +42,7 @@ export async function DELETE(
   { params }: { params: { id: string } }
 ) {
   try {
-    const supabase = createServerClient()
+    const supabase = createServerClient(true)
     const { data: { session } } = await supabase.auth.getSession()
 
     if (!session?.user?.id) {

app/api/deployments/route.ts

@@ -8,7 +8,7 @@ export const dynamic = 'force-dynamic'
 // POST /api/deployments - Deploy fragment
 export async function POST(request: NextRequest) {
   try {
-    const supabase = createServerClient()
+    const supabase = createServerClient(true)
     const { data: { session } } = await supabase.auth.getSession()
 
     if (!session?.user?.id) {
@@ -50,7 +50,7 @@ export async function POST(request: NextRequest) {
 // GET /api/deployments - List deployments
 export async function GET(request: NextRequest) {
   try {
-    const supabase = createServerClient()
+    const supabase = createServerClient(true)
     const { data: { session } } = await supabase.auth.getSession()
 
     if (!session?.user?.id) {

lib/deployment/deployment-engine.ts

@@ -320,7 +320,6 @@ export const deploymentProviders: DeploymentProvider[] = [
 ]
 
 export class DeploymentEngine {
-  [x: string]: any
   private deployments = new Map<string, DeploymentStatus>()
   private deploymentHistory = new Map<string, DeploymentResult[]>()
 
@@ -738,48 +737,58 @@ gatherUsageStats = false
     // - Resource caching
   }
 
-  private async deployToProvider(
+  private async deployToActualProvider(
     deployment: DeploymentStatus,
     fragment: FragmentSchema,
     config: DeploymentConfig
   ): Promise<DeploymentResult> {
+    const provider = config.provider
+    const deploymentId = deployment.deploymentId
+    
     deployment.status = 'deploying'
     deployment.progress = 80
     deployment.currentStep = `Deploying to ${config.provider.name}`
     deployment.logs.push(`🚀 Deploying to ${config.provider.name}...`)
-
-    // Simulate deployment based on provider
-    const result = await this.simulateProviderDeployment(deployment, fragment, config)
-    
-    deployment.progress = 100
-    deployment.status = 'success'
-    deployment.deployedAt = new Date()
-    deployment.url = result.url
-    deployment.previewUrl = result.previewUrl
-    
-    return result
-  }
-
-  private async deployToActualProvider(
-    deployment: DeploymentStatus,
-    fragment: FragmentSchema,
-    config: DeploymentConfig
-  ): Promise<DeploymentResult> {
-    const provider = config.provider
-    const deploymentId = deployment.deploymentId
 
     try {
       switch (provider.id) {
         case 'vercel':
-          return await this.deployToVercel(deployment, fragment, config)
+          const result = await this.deployToVercel(deployment, fragment, config)
+          deployment.progress = 100
+          deployment.status = 'success'
+          deployment.deployedAt = new Date()
+          deployment.url = result.url
+          deployment.previewUrl = result.previewUrl
+          return result
         case 'netlify':
-          return await this.deployToNetlify(deployment, fragment, config)
+          const netlifyResult = await this.deployToNetlify(deployment, fragment, config)
+          deployment.progress = 100
+          deployment.status = 'success'
+          deployment.deployedAt = new Date()
+          deployment.url = netlifyResult.url
+          deployment.previewUrl = netlifyResult.previewUrl
+          return netlifyResult
         case 'railway':
-          return await this.deployToRailway(deployment, fragment, config)
+          const railwayResult = await this.deployToRailway(deployment, fragment, config)
+          deployment.progress = 100
+          deployment.status = 'success'
+          deployment.deployedAt = new Date()
+          deployment.url = railwayResult.url
+          return railwayResult
         case 'render':
-          return await this.deployToRender(deployment, fragment, config)
+          const renderResult = await this.deployToRender(deployment, fragment, config)
+          deployment.progress = 100
+          deployment.status = 'success'
+          deployment.deployedAt = new Date()
+          deployment.url = renderResult.url
+          return renderResult
         case 'fly-io':
-          return await this.deployToFly(deployment, fragment, config)
+          const flyResult = await this.deployToFly(deployment, fragment, config)
+          deployment.progress = 100
+          deployment.status = 'success'
+          deployment.deployedAt = new Date()
+          deployment.url = flyResult.url
+          return flyResult
         default:
           throw new Error(`Provider ${provider.id} not implemented`)
       }