Update app.py

app.py

@@ -122,7 +122,7 @@ def load_qa_history():
         return []
 
 def save_qa_history(history_entry):
-    """Save QA history entry to local JSON file and push to both Hugging Face and GitHub"""
+    """Save QA history entry to local JSON file and push to GitHub"""
     try:
         history_file = Path("qa_history.json")
         
@@ -132,6 +132,7 @@ def save_qa_history(history_entry):
                 try:
                     history_data = json.load(f)
                 except json.JSONDecodeError:
+                    st.error("Error parsing existing JSON file")
                     history_data = []
         else:
             history_data = []
@@ -142,9 +143,15 @@ def save_qa_history(history_entry):
         # Save updated history locally
         with open("qa_history.json", "w", encoding="utf-8") as f:
             json.dump(history_data, f, ensure_ascii=False, indent=2)
+        
+        st.info("Saved locally, attempting GitHub upload...")
 
         # Push to GitHub
         github_token = os.getenv('GITHUB_TOKEN') or st.secrets['GITHUB_TOKEN']
+        if not github_token:
+            st.error("GitHub token not found!")
+            return
+            
         from github import Github
         g = Github(github_token)
         
@@ -154,22 +161,27 @@ def save_qa_history(history_entry):
         try:
             # Try to get the file first
             contents = repo.get_contents("qa_history.json")
-            repo.update_file(
+            response = repo.update_file(
                 path="qa_history.json",
                 message="Update QA history",
                 content=json.dumps(history_data, ensure_ascii=False, indent=2),
                 sha=contents.sha
             )
-        except Exception:
+            st.success(f"Successfully updated file on GitHub: {response.commit.html_url}")
+        except Exception as e:
+            st.info(f"File doesn't exist yet, creating new one... Error was: {str(e)}")
             # File doesn't exist, create it
-            repo.create_file(
+            response = repo.create_file(
                 path="qa_history.json",
                 message="Create QA history",
                 content=json.dumps(history_data, ensure_ascii=False, indent=2)
             )
+            st.success(f"Successfully created file on GitHub: {response['commit'].html_url}")
             
     except Exception as e:
         st.error(f"Error saving QA history: {str(e)}")
+        import traceback
+        st.error(f"Full error: {traceback.format_exc()}")
 
 def add_to_qa_history(query: str, answer: str):
     """Add new QA pair to history"""
@@ -185,12 +197,21 @@ def add_to_qa_history(query: str, answer: str):
 def clear_qa_history():
     """Clear QA history file"""
     try:
+        # Create empty list for history data
+        history_data = []
+        
         # Write empty list to history file
         with open("qa_history.json", "w", encoding="utf-8") as f:
-            json.dump([], f, ensure_ascii=False, indent=2)
+            json.dump(history_data, f, ensure_ascii=False, indent=2)
+        
+        st.info("Cleared local file, attempting GitHub update...")
         
         # Push to GitHub
         github_token = os.getenv('GITHUB_TOKEN') or st.secrets['GITHUB_TOKEN']
+        if not github_token:
+            st.error("GitHub token not found!")
+            return
+            
         from github import Github
         g = Github(github_token)
         
@@ -200,21 +221,27 @@ def clear_qa_history():
         try:
             # Try to get the file first
             contents = repo.get_contents("qa_history.json")
-            repo.update_file(
+            response = repo.update_file(
                 path="qa_history.json",
-                message="Update QA history",
+                message="Clear QA history",
                 content=json.dumps(history_data, ensure_ascii=False, indent=2),
                 sha=contents.sha
             )
-        except Exception:
+            st.success(f"Successfully cleared file on GitHub: {response.commit.html_url}")
+        except Exception as e:
+            st.info(f"File doesn't exist yet, creating empty one... Error was: {str(e)}")
             # File doesn't exist, create it
-            repo.create_file(
+            response = repo.create_file(
                 path="qa_history.json",
-                message="Create QA history",
+                message="Initialize empty QA history",
                 content=json.dumps(history_data, ensure_ascii=False, indent=2)
             )
+            st.success(f"Successfully created empty file on GitHub: {response['commit'].html_url}")
+            
     except Exception as e:
         st.error(f"Error clearing QA history: {str(e)}")
+        import traceback
+        st.error(f"Full error: {traceback.format_exc()}")
         
 def add_to_history(role: str, message: str):
     """Add message to chat history and save if it's a complete QA pair"""