Update app.py

app.py

@@ -21,6 +21,17 @@ def sanitize_input(text: str) -> str:
     """Sanitize user input to prevent XSS and injection attacks."""
     return html.escape(text.strip())
 
+def validate_name(name: str) -> str:
+    """Validate name input."""
+    name = name.strip()
+    if not name:
+        raise gr.Error("Name cannot be empty")
+    if len(name) > 100:
+        raise gr.Error("Name is too long (max 100 characters)")
+    if any(c.isdigit() for c in name):
+        raise gr.Error("Name cannot contain numbers")
+    return name
+
 def validate_age(age: Union[int, float, str]) -> int:
     """Validate and convert age input."""
     try:
@@ -36,12 +47,13 @@ def validate_file(file_obj) -> None:
     if not file_obj:
         raise gr.Error("No file uploaded")
     
-    if not any(file_obj.name.endswith(ext) for ext in ALLOWED_FILE_TYPES):
-        raise gr.Error(f"Only {', '.join(ALLOWED_FILE_TYPES)} files are allowed")
+    file_ext = os.path.splitext(file_obj.name)[1].lower()
+    if file_ext not in ALLOWED_FILE_TYPES:
+        raise gr.Error(f"Invalid file type. Allowed: {', '.join(ALLOWED_FILE_TYPES)}")
     
     file_size = os.path.getsize(file_obj.name) / (1024 * 1024)  # MB
     if file_size > MAX_FILE_SIZE_MB:
-        raise gr.Error(f"File size must be less than {MAX_FILE_SIZE_MB}MB")
+        raise gr.Error(f"File too large. Max size: {MAX_FILE_SIZE_MB}MB")
 
 # ========== TRANSCRIPT PARSING ==========
 def extract_gpa(text: str, gpa_type: str) -> str:
@@ -116,6 +128,9 @@ def extract_courses_from_table(text: str) -> Dict[str, List[Dict]]:
 def parse_transcript(file_obj) -> Tuple[str, Optional[Dict]]:
     """Parse transcript file with robust error handling."""
     try:
+        if not file_obj:
+            raise gr.Error("Please upload a file first")
+            
         validate_file(file_obj)
         
         text = ''
@@ -335,7 +350,7 @@ learning_style_quiz = LearningStyleQuiz()
 class ProfileManager:
     def __init__(self):
         self.profiles_dir = Path(PROFILES_DIR)
-        self.profiles_dir.mkdir(exist_ok=True)
+        self.profiles_dir.mkdir(exist_ok=True, parents=True)
     
     def save_profile(self, name: str, age: Union[int, str], interests: str, 
                     transcript: Dict, learning_style: str, 
@@ -345,10 +360,7 @@ class ProfileManager:
         """Save student profile with validation."""
         try:
             # Validate required fields
-            if not name.strip():
-                raise gr.Error("Name is required")
-            
-            name = sanitize_input(name)
+            name = validate_name(name)
             age = validate_age(age)
             interests = sanitize_input(interests)
             
@@ -840,11 +852,11 @@ def create_interface():
                     )
                 
                 quiz_submit.click(
-                    fn=lambda *answers: learning_style_quiz.evaluate_quiz(answers),
+                    fn=lambda *answers: learning_style_quiz.evaluate_quiz(*answers),
                     inputs=quiz_components,
                     outputs=learning_output
                 ).then(
-                    fn=lambda: gr.update(visible=True),
+                    fn=lambda: gr.Markdown(visible=True),
                     outputs=learning_output
                 )
             
@@ -907,6 +919,7 @@ def create_interface():
                             visible=bool(profile_manager.list_profiles())
                         )
                         load_btn = gr.Button("Load Profile", visible=bool(profile_manager.list_profiles()))
+                        clear_btn = gr.Button("Clear Form")
                     
                     with gr.Column(scale=2):
                         output_summary = gr.Markdown(
@@ -929,6 +942,16 @@ def create_interface():
                     inputs=load_profile_dropdown,
                     outputs=output_summary
                 )
+                
+                clear_btn.click(
+                    fn=lambda: [gr.update(value="") for _ in range(12)],
+                    outputs=[
+                        name, age, interests, 
+                        movie, movie_reason, show, show_reason,
+                        book, book_reason, character, character_reason,
+                        blog_text
+                    ]
+                )
             
             # ===== TAB 5: AI Teaching Assistant =====
             with gr.Tab("AI Assistant", id=4) as tab5:
@@ -949,27 +972,27 @@ def create_interface():
         
         # Tab navigation logic
         def navigate_to_tab(tab_index: int):
-            return {tabs: gr.update(selected=tab_index)}
+            return gr.Tabs(selected=tab_index)
         
         step1.click(
             fn=lambda: navigate_to_tab(0),
-            outputs={tabs: tabs}
+            outputs=tabs
         )
         step2.click(
             fn=lambda: navigate_to_tab(1),
-            outputs={tabs: tabs}
+            outputs=tabs
         )
         step3.click(
             fn=lambda: navigate_to_tab(2),
-            outputs={tabs: tabs}
+            outputs=tabs
         )
         step4.click(
             fn=lambda: navigate_to_tab(3),
-            outputs={tabs: tabs}
+            outputs=tabs
         )
         step5.click(
             fn=lambda: navigate_to_tab(4),
-            outputs={tabs: tabs}
+            outputs=tabs
         )
     
     return app