that's correct! well done message should always have a tickmark.

src/app/journey/[journeyId]/page.tsx

@@ -99,16 +99,24 @@ export default function JourneyPage() {
 
 
   const handleOpenSummaryDialog = useCallback(() => {
-    if (!imageDataUri) {
+    if (!imageDataUri && !journey?.imageUrl) { // Check if journey itself lacks an image
       toast({
-        title: "Image Not Ready",
-        description: "The image data is still loading or failed to load. Please wait or try refreshing.",
+        title: "Summary Not Available",
+        description: "The journey image is missing, so a summary cannot be generated.",
+        variant: "destructive",
+      });
+      return;
+    }
+     if (!imageDataUri && journey?.imageUrl) { // Image exists for journey but failed to load as data URI
+      toast({
+        title: "Image Not Ready for Summary",
+        description: "The image data is still loading or failed to load. Please wait or try refreshing to view the summary.",
         variant: "destructive",
       });
       return;
     }
     setIsSummaryDialogOpen(true);
-  }, [imageDataUri, toast]);
+  }, [imageDataUri, journey, toast]);
 
 
   if (error) {
@@ -145,18 +153,18 @@ export default function JourneyPage() {
         showBackButton
         backHref="/select-journey"
         title={journeyTitle}
-        showDetailsButton={!!journey && !!imageDataUri}
+        showDetailsButton={!!journey} // Show details button if journey exists, even if image data is loading/failed
         onDetailsClick={handleOpenSummaryDialog}
       />
       <main className="flex-1 overflow-hidden p-2 md:p-4">
         <div className="grid h-full grid-cols-1 gap-2 md:grid-cols-2 md:gap-4">
           <div className="relative h-full w-full overflow-hidden rounded-lg shadow-lg bg-muted">
-            {isLoadingJourney || !journey?.imageUrl ? (
+            {isLoadingJourney && journey?.imageUrl ? ( // Show loader only if an image URL exists and is loading
               <div className="flex h-full w-full flex-col items-center justify-center">
                  <Loader2 className="h-16 w-16 animate-spin text-primary mb-4" />
                  <p className="text-muted-foreground">Loading image for {journey?.title || 'journey'}...</p>
               </div>
-            ) : (
+            ) : journey?.imageUrl && imageDataUri ? ( // Image URL exists and data URI is loaded
               <Image
                 src={journey.imageUrl}
                 alt={journey.title}
@@ -166,8 +174,7 @@ export default function JourneyPage() {
                 data-ai-hint={journey.imageHint}
                 priority
               />
-            )}
-             {!isLoadingJourney && !journey?.imageUrl && journey && (
+            ) : ( // No image URL or imageDataUri failed to load (after isLoadingJourney is false)
               <div className="flex h-full w-full flex-col items-center justify-center p-4 text-center">
                  <AlertTriangle className="h-16 w-16 text-destructive mb-4" />
                  <p className="text-destructive-foreground font-semibold">Image Not Available</p>
@@ -177,16 +184,20 @@ export default function JourneyPage() {
           </div>
           
           <div className="h-full overflow-y-auto">
-            {isLoadingJourney && !imageDataUri && (
+            {isLoadingJourney && !imageDataUri && journey?.imageUrl && ( // Show AI loading if journey has an image but data URI is not ready
                 <div className="flex h-full flex-col items-center justify-center rounded-lg border bg-card p-4 shadow-xl">
                     <Loader2 className="h-12 w-12 animate-spin text-primary mb-4" />
                     <p className="text-muted-foreground">Preparing AI interactions...</p>
                 </div>
             )}
             {!isLoadingJourney && imageDataUri && journey && (
-              <Chatbot imageDataUri={imageDataUri} journeyTitle={journey.title} />
+              <Chatbot 
+                imageDataUri={imageDataUri} 
+                journeyTitle={journey.title}
+                onOpenSummaryDialog={handleOpenSummaryDialog} 
+              />
             )}
-            {!isLoadingJourney && !imageDataUri && journey && (
+            {((!isLoadingJourney && !imageDataUri && journey) || (!journey?.imageUrl && !isLoadingJourney)) && ( // Covers both failed data URI load AND no image URL in journey
                <div className="flex h-full flex-col items-center justify-center rounded-lg border bg-card p-4 shadow-xl text-center">
                     <AlertTriangle className="h-12 w-12 text-destructive mb-4" />
                     <p className="font-semibold text-destructive">AI Features Unavailable</p>
@@ -200,7 +211,7 @@ export default function JourneyPage() {
         <SummaryReportDialog
             isOpen={isSummaryDialogOpen}
             onOpenChange={setIsSummaryDialogOpen}
-            imageDataUri={imageDataUri}
+            imageDataUri={imageDataUri} // Pass imageDataUri, dialog handles null
             journeyTitle={journey.title}
         />
       )}

src/components/chatbot/chatbot.tsx

@@ -12,20 +12,24 @@ import { explainCorrectAnswer } from '@/ai/flows/explain-correct-answer-flow';
 import type { ChatMessage as ChatMessageType, ActiveMCQ } from '@/types';
 import { ChatMessage } from './chat-message';
 import { useToast } from '@/hooks/use-toast';
-import { Send } from 'lucide-react';
+import { Send, FileText } from 'lucide-react';
+
+const MAX_QUESTIONS = 5;
 
 interface ChatbotProps {
   imageDataUri: string | null;
   journeyTitle: string;
+  onOpenSummaryDialog: () => void;
 }
 
-export function Chatbot({ imageDataUri, journeyTitle }: ChatbotProps) {
+export function Chatbot({ imageDataUri, journeyTitle, onOpenSummaryDialog }: ChatbotProps) {
   const [messages, setMessages] = useState<ChatMessageType[]>([]);
   const [currentMCQ, setCurrentMCQ] = useState<ActiveMCQ | null>(null);
   const [isLoading, setIsLoading] = useState(false);
   const [isAwaitingAnswer, setIsAwaitingAnswer] = useState(false);
   const [hasAnsweredCorrectly, setHasAnsweredCorrectly] = useState(false);
   const [incorrectAttempts, setIncorrectAttempts] = useState<string[]>([]);
+  const [questionCount, setQuestionCount] = useState(0);
   const scrollAreaRef = useRef<HTMLDivElement>(null);
   const { toast } = useToast();
   const messageIdCounter = useRef(0);
@@ -41,20 +45,20 @@ export function Chatbot({ imageDataUri, journeyTitle }: ChatbotProps) {
   useEffect(() => {
     let ignore = false;
 
-    // Comprehensive reset for new journey/image
     setMessages([]);
     setCurrentMCQ(null);
     setIsLoading(false);
     setIsAwaitingAnswer(false);
     setHasAnsweredCorrectly(false);
     setIncorrectAttempts([]);
+    setQuestionCount(0);
     isInitialQuestionRef.current = true;
     messageIdCounter.current = 0;
 
 
     const performInitialFetch = async () => {
       if (!imageDataUri) {
-        if (!ignore && messages.length === 0) { // Only add if no messages and not ignoring
+        if (!ignore && messages.length === 0) { 
           addMessage({ sender: 'ai', type: 'text', text: "Preparing your journey..." });
         }
         return;
@@ -69,7 +73,7 @@ export function Chatbot({ imageDataUri, journeyTitle }: ChatbotProps) {
 
         let activeMcqData: ActiveMCQ = {
           ...mcqResult,
-          originalQuestionTextForFlow: mcqResult.mcq, // Store original question text
+          originalQuestionTextForFlow: mcqResult.mcq, 
         };
         
         if (isInitialQuestionRef.current) {
@@ -99,21 +103,19 @@ export function Chatbot({ imageDataUri, journeyTitle }: ChatbotProps) {
     };
     
     if (imageDataUri) {
-        // If there's an existing "Preparing..." message, clear it before fetching
         if (messages.length === 1 && messages[0].text === "Preparing your journey...") {
             setMessages([]);
         }
         performInitialFetch();
-    } else if (messages.length === 0) { // Only add "Preparing..." if no messages exist
+    } else if (messages.length === 0) { 
         addMessage({ sender: 'ai', type: 'text', text: "Preparing your journey..." });
     }
 
-
     return () => {
       ignore = true;
     };
   // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [imageDataUri, journeyTitle]); // addMessage and toast are stable, so not needed in deps
+  }, [imageDataUri, journeyTitle]); 
 
 
   useEffect(() => {
@@ -153,12 +155,17 @@ export function Chatbot({ imageDataUri, journeyTitle }: ChatbotProps) {
       addMessage({ sender: 'ai', type: 'feedback', text: combinedMessage, isCorrect: true });
       setHasAnsweredCorrectly(true);
       setIncorrectAttempts([]);
+      const newQuestionCount = questionCount + 1;
+      setQuestionCount(newQuestionCount);
+      if (newQuestionCount >= MAX_QUESTIONS) {
+        addMessage({ sender: 'ai', type: 'text', text: "You've completed all 5 questions! Please proceed to the summary report." });
+      }
+
     } else { // Incorrect answer
       const updatedIncorrectAttempts = [...incorrectAttempts, option];
       setIncorrectAttempts(updatedIncorrectAttempts);
 
       if (updatedIncorrectAttempts.length >= currentMCQ.options.length - 1) {
-        // All incorrect options have been tried
         let combinedText = "";
         try {
           const incorrectExplanationResult = await explainIncorrectAnswer({
@@ -197,6 +204,11 @@ export function Chatbot({ imageDataUri, journeyTitle }: ChatbotProps) {
         });
         setHasAnsweredCorrectly(true);
         setIncorrectAttempts([]);
+        const newQuestionCount = questionCount + 1;
+        setQuestionCount(newQuestionCount);
+        if (newQuestionCount >= MAX_QUESTIONS) {
+            addMessage({ sender: 'ai', type: 'text', text: "You've completed all 5 questions! Please proceed to the summary report." });
+        }
 
       } else { // Still other options left to try
         try {
@@ -235,6 +247,13 @@ export function Chatbot({ imageDataUri, journeyTitle }: ChatbotProps) {
   };
 
   const handleNextQuestionClick = async () => {
+    if (questionCount >= MAX_QUESTIONS) {
+      addMessage({ sender: 'ai', type: 'text', text: "You've completed all the questions! Please proceed to the summary." });
+      setIsLoading(false);
+      setHasAnsweredCorrectly(true); // Keep UI consistent for summary button
+      return;
+    }
+
     setIsAwaitingAnswer(false);
     setHasAnsweredCorrectly(false);
     setCurrentMCQ(null); 
@@ -272,35 +291,21 @@ export function Chatbot({ imageDataUri, journeyTitle }: ChatbotProps) {
   return (
     <div className="flex h-full flex-col rounded-lg border bg-card shadow-xl">
       <div className="border-b p-4">
-        <h2 className="font-headline text-lg font-semibold text-center">AI Chat Helper</h2>
+        <h2 className="font-headline text-lg font-semibold text-center">AI Chat Helper ({questionCount}/{MAX_QUESTIONS} Questions)</h2>
       </div>
       <ScrollArea className="flex-1 p-4" ref={scrollAreaRef}>
         <div className="space-y-4">
           {messages.map((msg, index) => {
-            const isThisMessageTheCurrentMCQ =
+             const isLastMessageTheActiveMCQ =
               msg.type === 'mcq' &&
-              msg.mcq &&
               currentMCQ &&
-              msg.mcq.mcq === currentMCQ.mcq && 
-              JSON.stringify(msg.mcq.options) === JSON.stringify(currentMCQ.options) &&
-              msg.mcq.correctAnswer === currentMCQ.correctAnswer;
-            
-            const lastInstanceOfCurrentMCQIndex = messages.findLastIndex(
-              (m) =>
-                m.type === 'mcq' &&
-                m.mcq &&
-                currentMCQ &&
-                m.mcq.mcq === currentMCQ.mcq &&
-                JSON.stringify(m.mcq.options) === JSON.stringify(currentMCQ.options) &&
-                m.mcq.correctAnswer === currentMCQ.correctAnswer
-            );
+              msg.id === messages.findLast(m => m.type === 'mcq' && m.mcq?.mcq === currentMCQ.mcq && JSON.stringify(m.mcq?.options) === JSON.stringify(currentMCQ.options))?.id;
 
             const shouldThisMessageBeInteractive =
-              isThisMessageTheCurrentMCQ &&
-              index === lastInstanceOfCurrentMCQIndex && 
-              isAwaitingAnswer && 
+              isLastMessageTheActiveMCQ &&
+              isAwaitingAnswer &&
               !hasAnsweredCorrectly;
-
+            
             return (
               <ChatMessage
                 key={msg.id}
@@ -330,7 +335,13 @@ export function Chatbot({ imageDataUri, journeyTitle }: ChatbotProps) {
       <div className="border-t bg-background/80 p-4">
         {isLoading && <p className="text-center text-sm text-muted-foreground">AI is thinking...</p>}
 
-        {!isLoading && currentMCQ && hasAnsweredCorrectly && (
+        {!isLoading && questionCount >= MAX_QUESTIONS && hasAnsweredCorrectly && (
+          <Button onClick={onOpenSummaryDialog} className="w-full" variant="default">
+            Go to Report Summary <FileText className="ml-2 h-4 w-4" />
+          </Button>
+        )}
+
+        {!isLoading && questionCount < MAX_QUESTIONS && currentMCQ && hasAnsweredCorrectly && (
            <Button onClick={handleNextQuestionClick} className="w-full" variant="default" disabled={isLoading}>
             Next Question <Send className="ml-2 h-4 w-4" />
           </Button>