Initial analysis and plan for fixing workflow API and UI build errors

Copilot · Gerome-Elassaad · Copilot · commit be1d216fa0a8 · 2025-07-19T10:18:53.000Z
Co-authored-by: Gerome-Elassaad &lt;186273274+Gerome-Elassaad@users.noreply.github.com&gt;
diff --git a/app/api/chat/codeInterpreter.ts b/app/api/chat/codeInterpreter.ts
@@ -1,18 +1,19 @@
-import 'server-only';
-
-import { Sandbox } from '@e2b/code-interpreter';
-
+import 'server-only';
+
+import { Sandbox } from '@e2b/code-interpreter';
+
 const E2B_API_KEY = process.env.E2B_API_KEY;
-if (!E2B_API_KEY) {
-  throw new Error('E2B_API_KEY environment variable not found');
-}
 
 const sandboxTimeout = 10 * 60 * 1000;
 
-export async function evaluateCode(
-  sessionID: string,
-  code: string,
-) {
+export async function evaluateCode(
+  sessionID: string,
+  code: string,
+) {
+  if (!E2B_API_KEY) {
+    throw new Error('E2B_API_KEY environment variable not found');
+  }
+
   const sandbox = await getSandbox(sessionID);
 
   // Execute the code in a Jupyter Notebook in the sandbox.
@@ -34,7 +35,11 @@ export async function evaluateCode(
 }
 
 
-async function getSandbox(sessionID: string) {
+async function getSandbox(sessionID: string) {
+  if (!E2B_API_KEY) {
+    throw new Error('E2B_API_KEY environment variable not found');
+  }
+
   const sandboxes = await Sandbox.list();
 
   const sandboxID = sandboxes.find(sandbox => sandbox.metadata?.sessionID === sessionID)?.sandboxId;
diff --git a/app/api/files/content/route.ts b/app/api/files/content/route.ts
@@ -2,9 +2,6 @@ import { NextResponse } from 'next/server'
 import { Sandbox } from '@e2b/code-interpreter'
 
 const E2B_API_KEY = process.env.E2B_API_KEY
-if (!E2B_API_KEY) {
-  throw new Error('E2B_API_KEY environment variable not found')
-}
 
 const sandboxTimeout = 10 * 60 * 1000
 
@@ -22,6 +19,13 @@ async function getSandbox(sessionID: string, template?: string) {
 
 export async function GET(req: Request) {
   try {
+    if (!E2B_API_KEY) {
+      return NextResponse.json(
+        { error: 'E2B_API_KEY environment variable not found' },
+        { status: 500 },
+      )
+    }
+
     const { searchParams } = new URL(req.url)
     const sessionID = searchParams.get('sessionID')
     const path = searchParams.get('path')
diff --git a/app/api/files/route.ts b/app/api/files/route.ts
@@ -29,13 +29,14 @@ async function listFilesRecursively(
 }
 
 const E2B_API_KEY = process.env.E2B_API_KEY
-if (!E2B_API_KEY) {
-  throw new Error('E2B_API_KEY environment variable not found')
-}
 
 const sandboxTimeout = 10 * 60 * 1000
 
 async function getSandbox(sessionID: string, template?: string) {
+  if (!E2B_API_KEY) {
+    throw new Error('E2B_API_KEY environment variable not found')
+  }
+
   const sandbox = await Sandbox.create(template || 'code-interpreter-v1', {
     apiKey: E2B_API_KEY,
     metadata: {
diff --git a/app/api/files/sandbox/list/route.ts b/app/api/files/sandbox/list/route.ts
@@ -29,24 +29,28 @@ async function listFilesRecursively(
 }
 
 const E2B_API_KEY = process.env.E2B_API_KEY
-if (!E2B_API_KEY) {
-  throw new Error('E2B_API_KEY environment variable not found')
-}
 
 const sandboxTimeout = 10 * 60 * 1000
 
 export async function GET(request: NextRequest) {
-  const searchParams = request.nextUrl.searchParams
-  const sandboxId = searchParams.get('sandboxId')
+  try {
+    if (!E2B_API_KEY) {
+      return NextResponse.json(
+        { error: 'E2B_API_KEY environment variable not found' },
+        { status: 500 },
+      )
+    }
 
-  if (!sandboxId) {
-    return NextResponse.json(
-      { error: 'sandboxId is required' },
-      { status: 400 },
-    )
-  }
+    const searchParams = request.nextUrl.searchParams
+    const sandboxId = searchParams.get('sandboxId')
+
+    if (!sandboxId) {
+      return NextResponse.json(
+        { error: 'sandboxId is required' },
+        { status: 400 },
+      )
+    }
 
-  try {
     // Connect to existing sandbox by ID
     const sandbox = await Sandbox.connect(sandboxId, {
       apiKey: E2B_API_KEY,
diff --git a/app/api/files/sandbox/route.ts b/app/api/files/sandbox/route.ts
@@ -2,14 +2,18 @@ import { NextResponse } from 'next/server'
 import { Sandbox } from '@e2b/code-interpreter'
 
 const E2B_API_KEY = process.env.E2B_API_KEY
-if (!E2B_API_KEY) {
-  throw new Error('E2B_API_KEY environment variable not found')
-}
 
 const sandboxTimeout = 10 * 60 * 1000
 
 export async function GET(req: Request) {
   try {
+    if (!E2B_API_KEY) {
+      return NextResponse.json(
+        { error: 'E2B_API_KEY environment variable not found' },
+        { status: 500 },
+      )
+    }
+
     const { searchParams } = new URL(req.url)
     const sandboxId = searchParams.get('sandboxId')
     const path = searchParams.get('path')
diff --git a/app/api/import-dataset/route.ts b/app/api/import-dataset/route.ts
@@ -7,11 +7,17 @@ const DATASET = 'bigcode/the-stack';
 const EMBEDDING_MODEL = 'Xenova/all-MiniLM-L6-v2';
 const HUGGING_FACE_API_URL = 'https://huggingface.co/api/datasets';
 
-// Initialize Supabase client with the service role key for admin access
-const supabase = createClient(
-  process.env.NEXT_PUBLIC_SUPABASE_URL!,
-  process.env.SUPABASE_SERVICE_ROLE_KEY!
-);
+// Initialize Supabase client with the service role key for admin access
+const supabaseUrl = process.env.NEXT_PUBLIC_SUPABASE_URL
+const supabaseServiceKey = process.env.SUPABASE_SERVICE_ROLE_KEY
+
+// Check if Supabase is configured
+let supabase: any = null
+if (supabaseUrl && supabaseServiceKey) {
+  supabase = createClient(supabaseUrl, supabaseServiceKey)
+} else {
+  console.warn('Supabase configuration is incomplete. Import dataset feature is disabled.')
+}
 
 // Function to process and embed a single file's content
 async function processAndEmbedFile(content: string, embeddingPipeline: any) {
@@ -80,8 +86,15 @@ async function importDataset(subset: string) {
 }
 
 // The API Route Handler
-export async function POST(request: Request) {
-  try {
+export async function POST(request: Request) {
+  try {
+    if (!supabase) {
+      return NextResponse.json(
+        { error: 'Supabase is not configured. Import dataset feature is disabled.' },
+        { status: 503 }
+      )
+    }
+
     const { subset } = await request.json();
 
     if (!subset) {
diff --git a/app/deployments/page.tsx b/app/deployments/page.tsx
@@ -24,9 +24,12 @@ import {
 } from 'lucide-react'
 import { toast } from '@/components/ui/use-toast'
 import { useAuth } from '@/lib/auth'
+import { ViewType } from '@/components/auth'
 
 export default function DeploymentsPage() {
-  const { session } = useAuth(() => {}, () => {})
+  const [authDialog, setAuthDialog] = useState(false)
+  const [authView, setAuthView] = useState<ViewType>('sign_in')
+  const { session } = useAuth(setAuthDialog, setAuthView)
   const [deployments, setDeployments] = useState<DeploymentResult[]>([])
   const [activeDeployments, setActiveDeployments] = useState<DeploymentStatus[]>([])
   const [selectedFragment, setSelectedFragment] = useState<FragmentSchema | null>(null)
diff --git a/app/workflows/page.tsx b/app/workflows/page.tsx
@@ -11,9 +11,12 @@ import { WorkflowSchema } from '@/lib/workflow-engine'
 import { Plus, Search, Play, Edit, Trash2, Copy, GitBranch } from 'lucide-react'
 import { toast } from '@/components/ui/use-toast'
 import { useAuth } from '@/lib/auth'
+import { ViewType } from '@/components/auth'
 
 export default function WorkflowsPage() {
-  const { session } = useAuth(() => {}, () => {})
+  const [authDialog, setAuthDialog] = useState(false)
+  const [authView, setAuthView] = useState<ViewType>('sign_in')
+  const { session } = useAuth(setAuthDialog, setAuthView)
   const [workflows, setWorkflows] = useState<WorkflowSchema[]>([])
   const [selectedWorkflow, setSelectedWorkflow] = useState<WorkflowSchema | null>(null)
   const [searchTerm, setSearchTerm] = useState('')
diff --git a/lib/workflow-persistence.ts b/lib/workflow-persistence.ts
@@ -127,10 +127,23 @@ export class WorkflowPersistence {
 
   constructor() {
     const supabaseUrl = process.env.NEXT_PUBLIC_SUPABASE_URL
-    const supabaseServiceKey = process.env.NEXT_PUBLIC_SUPABASE_SERVICE_ROLE_KEY
+    const supabaseServiceKey = process.env.SUPABASE_SERVICE_ROLE_KEY || process.env.NEXT_PUBLIC_SUPABASE_ANON_KEY
 
     if (!supabaseUrl || !supabaseServiceKey) {
-      throw new Error('Supabase URL and service key are required.')
+      console.warn('Supabase configuration is incomplete. Some features may not work.')
+      // Create a mock client to prevent build errors
+      this.supabase = {
+        from: () => ({
+          select: () => ({ single: () => Promise.resolve({ data: null, error: new Error('Supabase not configured') }) }),
+          insert: () => ({ select: () => ({ single: () => Promise.resolve({ data: null, error: new Error('Supabase not configured') }) }) }),
+          update: () => ({ eq: () => ({ select: () => ({ single: () => Promise.resolve({ data: null, error: new Error('Supabase not configured') }) }) }) }),
+          delete: () => ({ eq: () => Promise.resolve({ error: new Error('Supabase not configured') }) }),
+          eq: () => ({ single: () => Promise.resolve({ data: null, error: new Error('Supabase not configured') }) }),
+          order: () => ({ range: () => Promise.resolve({ data: [], error: new Error('Supabase not configured'), count: 0 }) }),
+          range: () => Promise.resolve({ data: [], error: new Error('Supabase not configured'), count: 0 })
+        })
+      } as any
+      return
     }
     this.supabase = createClient<Database>(supabaseUrl, supabaseServiceKey)
   }
