Update app.py

app.py

@@ -141,10 +141,11 @@ def load_qa_history():
 
 def save_qa_history(history_entry):
     """
-    Save QA history to Git with improved authentication handling.
+    Save QA history to Git with improved error handling and logging.
     """
     try:
-        repo_path = Path(os.getenv('GITHUB_REPO_PATH', '')).resolve()
+        # Get repository path
+        repo_path = Path(REPO_PATH)  # Use the global REPO_PATH instead of env variable
         history_file = repo_path / "qa_history.json"
         
         # Load existing history
@@ -167,38 +168,40 @@ def save_qa_history(history_entry):
             json.dump(history_data, f, ensure_ascii=False, indent=2)
         
         # Git operations
-        try:
-            repo = Repo(str(repo_path))
+        repo = Repo(str(repo_path))
+        
+        # Check if the file is tracked
+        current_files = repo.git.ls_files().split('\n')
+        if str(history_file.relative_to(repo_path)) not in current_files:
+            repo.index.add([str(history_file)])
+            print(f"Added new file: {history_file}")
+        else:
             repo.index.add([str(history_file)])
+            print(f"Updated existing file: {history_file}")
+        
+        if repo.is_dirty(untracked_files=True):
+            commit = repo.index.commit(f"Update QA history at {datetime.now().isoformat()}")
+            origin = repo.remote('origin')
             
-            if repo.is_dirty():
-                commit = repo.index.commit("Update QA history")
-                origin = repo.remote('origin')
-                
-                try:
-                    origin.push()
-                    print(f"Successfully pushed commit {commit.hexsha}")
-                    return True
-                except git.GitCommandError as push_error:
-                    if "Authentication failed" in str(push_error):
-                        print("Authentication failed - please check your Git credentials")
-                    else:
-                        print(f"Push failed: {push_error}")
-                    return False
-            else:
-                print("No changes to commit")
+            try:
+                origin.push()
+                print(f"Successfully pushed commit {commit.hexsha}")
                 return True
-                
-        except Exception as git_error:
-            print(f"Git operation failed: {str(git_error)}")
-            return False
+            except git.GitCommandError as push_error:
+                print(f"Push error: {str(push_error)}")
+                if "Authentication failed" in str(push_error):
+                    print("Authentication failed - please check your Git credentials")
+                return False
+        else:
+            print("No changes to commit")
+            return True
             
     except Exception as e:
         print(f"Save operation failed: {str(e)}")
         return False
 
 def add_to_qa_history(query: str, answer: str):
-    """Add new QA pair to history and store it in session state"""
+    """Add new QA pair to history with immediate save attempt"""
     history_entry = {
         "timestamp": (datetime.now() + timedelta(hours=7)).isoformat(),
         "query": query,
@@ -206,7 +209,16 @@ def add_to_qa_history(query: str, answer: str):
     }
     
     # Append to session state history
+    if 'all_history' not in st.session_state:
+        st.session_state.all_history = []
     st.session_state.all_history.append(history_entry)
+    
+    # Immediately try to save if GitHub sync is enabled
+    if st.session_state.github_sync_enabled:
+        success = save_qa_history(history_entry)
+        if not success:
+            print("Failed to save QA history to GitHub")
+    
     return history_entry
 
 def fix_repo_path():