remove analyze_transcript as not needed

app.py

@@ -453,28 +453,28 @@ def get_youtube_transcript(video_id: str, language: str = "en") -> str:
         return f"Error retrieving transcript: {str(e)}"
 
 # Tool to analyze the transcript and answer a question
-@tool
-def analyze_transcript(transcript: str, question: str) -> str:
-    """
-    Analyze the provided video transcript to answer a specific question.
+# @tool
+# def analyze_transcript(transcript: str, question: str) -> str:
+#     """
+#     Analyze the provided video transcript to answer a specific question.
     
-    Args:
-        transcript: The full transcript text of the video
-        question: The specific question to answer based on the transcript
+#     Args:
+#         transcript: The full transcript text of the video
+#         question: The specific question to answer based on the transcript
     
-    Returns:
-        The answer to the question based on the transcript content
-    """
-    # This tool leverages the LLM's understanding capability
-    # The implementation is simple because the LLM will do the analysis
-    if not transcript or transcript.startswith("Error"):
-        return f"Unable to analyze transcript: {transcript}"
+#     Returns:
+#         The answer to the question based on the transcript content
+#     """
+#     # This tool leverages the LLM's understanding capability
+#     # The implementation is simple because the LLM will do the analysis
+#     if not transcript or transcript.startswith("Error"):
+#         return f"Unable to analyze transcript: {transcript}"
         
-    # Check if transcript has enough content to analyze
-    if len(transcript.split()) < 10:
-        return "The transcript is too short or incomplete to properly analyze."
+#     # Check if transcript has enough content to analyze
+#     if len(transcript.split()) < 10:
+#         return "The transcript is too short or incomplete to properly analyze."
     
-    return "TRANSCRIPT_ANALYSIS_PLACEHOLDER"  # This will be replaced by LLM reasoning
+#     return "TRANSCRIPT_ANALYSIS_PLACEHOLDER"  # This will be replaced by LLM reasoning
 
 
 
@@ -599,7 +599,7 @@ class BasicAgent:
         self.youtube_qa_agent = ToolCallingAgent(
             model=reasoning_model, #self.model,
             tools=[extract_youtube_id, get_youtube_transcript,
-                   analyze_transcript, FinalAnswerTool()],
+                   FinalAnswerTool()],
             name="youtube_qa_agent",
             planning_interval=2,
             max_steps=5,
@@ -610,9 +610,8 @@ class BasicAgent:
         When given a YouTube URL and a question, follow these steps IN ORDER:
         1. Extract the video ID from the URL using the `extract_youtube_id` tool
         2. Retrieve the transcript of the video using the `get_youtube_transcript` tool
-        3. ALWAYS analyze the transcript to find the answer to the question using the `analyze_transcript` tool
-        4. Provide a clear and concise answer based solely on the transcript content
-        5. Return your final answer using the `final_answer` tool
+        3. Provide a clear and concise answer based solely on the transcript content
+        4. Return your final answer using the `final_answer` tool
         
         IMPORTANT INSTRUCTIONS:
         - After getting the transcript, you MUST use the analyze_transcript tool. DO NOT call get_youtube_transcript twice.