fix(academy): reset exerciseComplete on lesson navigation, remove unused useAcademyCertificate hook

Author: waleedlatif1

apps/sim/app/academy/[courseSlug]/[lessonSlug]/page.tsx

@@ -1,6 +1,6 @@
 'use client'
 
-import { use, useCallback, useEffect, useMemo, useState } from 'react'
+import { use, useCallback, useEffect, useMemo, useRef, useState } from 'react'
 import { ChevronLeft, ChevronRight } from 'lucide-react'
 import Image from 'next/image'
 import Link from 'next/link'
@@ -22,6 +22,12 @@ export default function LessonPage({ params }: LessonPageProps) {
   const { courseSlug, lessonSlug } = use(params)
   const course = getCourse(courseSlug)
   const [exerciseComplete, setExerciseComplete] = useState(false)
+  // Reset completion state when the lesson changes (Next.js reuses the component across navigations).
+  const prevLessonSlugRef = useRef(lessonSlug)
+  if (prevLessonSlugRef.current !== lessonSlug) {
+    prevLessonSlugRef.current = lessonSlug
+    setExerciseComplete(false)
+  }
 
   const allLessons = useMemo<Lesson[]>(
     () => course?.modules.flatMap((m) => m.lessons) ?? [],

apps/sim/hooks/queries/academy.ts

@@ -1,25 +1,10 @@
-import { useMutation, useQuery, useQueryClient } from '@tanstack/react-query'
+import { useMutation, useQueryClient } from '@tanstack/react-query'
 import type { AcademyCertificate } from '@/lib/academy/types'
 import { fetchJson } from '@/hooks/selectors/helpers'
 
 export const academyKeys = {
   all: ['academy'] as const,
   certificates: () => [...academyKeys.all, 'certificate'] as const,
-  certificate: (certificateNumber?: string) =>
-    [...academyKeys.certificates(), certificateNumber ?? ''] as const,
-}
-
-export function useAcademyCertificate(certificateNumber?: string, options?: { enabled?: boolean }) {
-  return useQuery({
-    queryKey: academyKeys.certificate(certificateNumber),
-    queryFn: ({ signal }) =>
-      fetchJson<{ certificate: AcademyCertificate }>(
-        `/api/academy/certificates?certificateNumber=${encodeURIComponent(certificateNumber as string)}`,
-        { signal }
-      ).then((d) => d.certificate),
-    enabled: (options?.enabled ?? true) && Boolean(certificateNumber),
-    staleTime: 10 * 60 * 1000,
-  })
 }
 
 export function useIssueCertificate() {