fix(academy): course not-found metadata title; try-finally guard in mock run loop

Author: waleedlatif1

apps/sim/app/academy/(catalog)/[courseSlug]/page.tsx

@@ -16,7 +16,7 @@ export function generateStaticParams() {
 export async function generateMetadata({ params }: CourseDetailPageProps): Promise<Metadata> {
   const { courseSlug } = await params
   const course = getCourse(courseSlug)
-  if (!course) return {}
+  if (!course) return { title: 'Course Not Found' }
   return {
     title: course.title,
     description: course.description,

apps/sim/app/academy/components/sandbox-canvas-provider.tsx

@@ -306,25 +306,27 @@ export function SandboxCanvasProvider({
     clearWorkflowConsole(workflowId)
     useTerminalConsoleStore.setState({ isOpen: true })
 
-    for (let i = 0; i < plan.length; i++) {
-      const step = plan[i]
-      setActiveBlocks(workflowId, new Set([step.blockId]))
-      await new Promise((resolve) => setTimeout(resolve, step.delay))
-      addConsole({
-        workflowId,
-        blockId: step.blockId,
-        blockName: workflowBlocks[step.blockId]?.name ?? step.blockType,
-        blockType: step.blockType,
-        executionOrder: i,
-        output: step.output,
-        success: true,
-        durationMs: step.delay,
-      })
-      setActiveBlocks(workflowId, new Set())
+    try {
+      for (let i = 0; i < plan.length; i++) {
+        const step = plan[i]
+        setActiveBlocks(workflowId, new Set([step.blockId]))
+        await new Promise((resolve) => setTimeout(resolve, step.delay))
+        addConsole({
+          workflowId,
+          blockId: step.blockId,
+          blockName: workflowBlocks[step.blockId]?.name ?? step.blockType,
+          blockType: step.blockType,
+          executionOrder: i,
+          output: step.output,
+          success: true,
+          durationMs: step.delay,
+        })
+        setActiveBlocks(workflowId, new Set())
+      }
+    } finally {
+      setIsExecuting(workflowId, false)
+      isMockRunningRef.current = false
     }
-
-    setIsExecuting(workflowId, false)
-    isMockRunningRef.current = false
   }, [workflowId, exerciseConfig.validationRules, exerciseConfig.mockOutputs])
   handleMockRunRef.current = handleMockRun