Update app.py

app.py

@@ -5,7 +5,7 @@ Startup Flow:
   1. Space rebuild (triggered by PR merge) → Docker container starts
   2. Call evaluate.run_evaluation() to scan new submissions in submissions/
   3. Calculate EM/F1 scores for new submissions, update leaderboard_data.json
-  4. Commit updated data back to repository (persistence)
+  4. Commit updated data back to repository (persistence) ONLY if data changed
   5. Start Flask Web server
 """
 
@@ -49,7 +49,8 @@ def startup_evaluation():
       - Re-evaluate all submissions (deduplicate using configuration combinations)
       - Compare with groundtruth.jsonl to calculate scores
       - Update leaderboard_data.json
-      - Commit results back to repository for persistence
+      - Commit results back to repository ONLY if data actually changed
+        (this prevents an infinite rebuild loop)
     
     Note:
       - Every startup re-evaluates all files, making the logic simpler
@@ -61,11 +62,24 @@ def startup_evaluation():
     logger.info("=" * 60)
 
     try:
+        # Read old leaderboard data before evaluation
+        old_data = None
+        if os.path.exists(LEADERBOARD_FILE):
+            with open(LEADERBOARD_FILE, 'r', encoding='utf-8') as f:
+                old_data = f.read()
+
         total, _ = run_evaluation()
 
         if total > 0:
-            logger.info(f"Evaluated all submissions. Committing to repo...")
-            commit_leaderboard_to_repo()
+            # Read new leaderboard data after evaluation
+            with open(LEADERBOARD_FILE, 'r', encoding='utf-8') as f:
+                new_data = f.read()
+
+            if new_data != old_data:
+                logger.info("Leaderboard data changed. Committing to repo...")
+                commit_leaderboard_to_repo()
+            else:
+                logger.info("Leaderboard data unchanged. Skipping commit to avoid rebuild loop.")
         else:
             logger.info("No submissions found.")