Update app.py

app.py

@@ -9,10 +9,6 @@ from langchain.agents import AgentExecutor
 from langchain_experimental.tools import PythonREPLTool
 from langchain_community.tools.youtube.search import YouTubeSearchTool
 from langchain_community.tools.tavily_search import TavilySearchResults
-
-# (Optional) Tiedostohallinnan työkalut
-
-# LLM
 from langchain_openai import ChatOpenAI
 from langgraph.graph import StateGraph, END
 from langgraph.prebuilt import ToolNode, tools_condition
@@ -26,7 +22,7 @@ class AgentState(TypedDict):
 # Agentin rakentajafunktio
 def create_langgraph_agent():
     print("Initializing Advanced LangGraph Agent...")
-    
+
     # 1. System prompt GAIA-tyyliin
     system_prompt = """
 You are a general AI assistant. I will ask you a question. Report your thoughts, and finish your answer with the following template:
@@ -39,15 +35,14 @@ If you are asked for a comma separated list, apply the above rules depending on
 """
     llm = ChatOpenAI(model="gpt-4o", temperature=0, system_message=system_prompt)
 
-    # 2. Työkalut: Tavily, PythonREPL, YouTube
+    # 2. Perustyökalut: Tavily, PythonREPL, YouTube
     tools = [
         TavilySearchResults(max_results=3),
         PythonREPLTool(),
         YouTubeSearchTool(),
     ]
 
-    # 3. Lisätään valinnaiset Playwright-selain- ja tiedostotyökalut
-    # Playwright: yritetään lisätä, jos riippuvuus saatavilla ja selaimet asennettu
+    # 3. Valinnainen Playwright-selain toolkit
     try:
         from langchain_community.tools.playwright.utils import create_sync_playwright_browser
         from langchain_community.agent_toolkits.playwright.toolkit import PlayWrightBrowserToolkit
@@ -59,28 +54,29 @@ If you are asked for a comma separated list, apply the above rules depending on
     except Exception as e:
         print("Playwright not available, skipping browser tools:", e)
 
-    # FileManagementToolkit on kevyt, voidaan lisätä suoraan
+    # 4. Valinnainen FileManagement toolkit
     try:
         from langchain_community.agent_toolkits.file_management.toolkit import FileManagementToolkit
         file_toolkit = FileManagementToolkit(root_dir=".")
         tools.extend(file_toolkit.get_tools())
+        print("FileManagement tools loaded.")
     except Exception as e:
-        print("FileManagement toolkit unavailable:", e) = FileManagementToolkit(root_dir=".")
-    tools.extend(file_toolkit.get_tools())
+        print("FileManagement toolkit unavailable:", e)
 
+    # Bind tools to LLM
     llm_with_tools = llm.bind_tools(tools)
     print("LLM and tools initialized.")
 
-    # 4. Agentin solmu (kutsuu kielimallia)
+    # 5. Agentin solmu (kutsuu kielimallia)
     def agent_node(state):
         print("Calling agent node...")
         return {"messages": [llm_with_tools.invoke(state["messages"])]}
-    
-    # 5. Työkalusolmu
+
+    # 6. Työkalusolmu
     tool_node = ToolNode(tools)
     print("Graph nodes defined.")
 
-    # 6. Graafin määritys
+    # 7. Graafin määritys
     graph = StateGraph(AgentState)
     graph.add_node("agent", agent_node)
     graph.add_node("tools", tool_node)
@@ -88,7 +84,7 @@ If you are asked for a comma separated list, apply the above rules depending on
     graph.add_conditional_edges("agent", tools_condition)
     graph.add_edge("tools", "agent")
 
-    # 7. Graafin kääntäminen
+    # 8. Graafin kääntäminen
     app = graph.compile()  # rekursion raja määritellään invoke-kutsussa
     print("LangGraph agent compiled and ready.")
     return app
@@ -110,7 +106,7 @@ def run_agent(agent_executor, question: str) -> str:
     except Exception as e:
         print(f"Error during agent execution: {e}")
         final_answer = f"Error: Agent failed to execute. {e}"
-    
+
     print(f"Agent returning answer: {final_answer}")
     return str(final_answer)
 
@@ -120,15 +116,15 @@ def run_and_submit_all(profile: gr.OAuthProfile | None):
     if not profile:
         return "Please Login to Hugging Face with the button.", None
     username = f"{profile.username}"
-    
+
     if not os.getenv("TAVILY_API_KEY") or not os.getenv("OPENAI_API_KEY"):
-         return "One or more API keys (TAVILY_API_KEY, OPENAI_API_KEY) are not set.", None
+        return "One or more API keys (TAVILY_API_KEY, OPENAI_API_KEY) are not set.", None
 
     try:
         agent_executor = create_langgraph_agent()
     except Exception as e:
         return f"Error initializing agent: {e}", None
-        
+
     agent_code = f"https://huggingface.co/spaces/{space_id}/tree/main"
     questions_url = f"https://agents-course-unit4-scoring.hf.space/questions"
     try:
@@ -156,7 +152,7 @@ def run_and_submit_all(profile: gr.OAuthProfile | None):
             f"User: {result_data.get('username')}\n"
             f"Overall Score: {result_data.get('score', 'N/A')}% "
             f"({result_data.get('correct_count', '?')}/{result_data.get('total_attempted', '?')} correct)\n"
-            f"Message: {result_data.get('message', 'No message received.')}"
+            f"Message: {result_data.get('message', 'No message received.') }"
         )
         return final_status, pd.DataFrame(answers_payload)
     except Exception as e: