add admin leaderboard submission listing

Author: msaroufim

src/kernelbot/api/main.py

@@ -698,13 +698,13 @@ async def admin_list_leaderboard_submissions(
     offset: int = 0,
 ) -> dict:
     with db_context as db:
-        submissions = db.get_leaderboard_submission_history(leaderboard_name, limit, offset)
+        submission_ids = db.get_leaderboard_submission_ids(leaderboard_name, limit, offset)
     return {
         "status": "ok",
         "leaderboard": leaderboard_name,
         "limit": limit,
         "offset": offset,
-        "submissions": submissions,
+        "submission_ids": submission_ids,
     }
 
 

src/libkernelbot/leaderboard_db.py

@@ -1276,43 +1276,30 @@ def get_user_submissions(
             logger.exception("Error fetching user submissions for user %s", user_id, exc_info=e)
             raise KernelBotError("Error fetching user submissions") from e
 
-    def get_leaderboard_submission_history(
+    def get_leaderboard_submission_ids(
         self,
         leaderboard_name: str,
         limit: int = 100,
         offset: int = 0,
-    ) -> list[dict]:
-        """Get all submissions for a leaderboard with admin-oriented metadata."""
-        limit = max(1, min(limit, 1000))
+    ) -> list[int]:
+        """Get submission IDs for a leaderboard in reverse chronological order."""
+        limit = max(1, limit)
         offset = max(0, offset)
 
         try:
             self.get_leaderboard_id(leaderboard_name)
             self.cursor.execute(
                 """
-                SELECT s.id, lb.name, s.file_name, s.user_id, ui.user_name,
-                       s.submission_time, s.done
+                SELECT s.id
                 FROM leaderboard.submission s
                 JOIN leaderboard.leaderboard lb ON s.leaderboard_id = lb.id
-                LEFT JOIN leaderboard.user_info ui ON ui.id = s.user_id
                 WHERE lb.name = %s
                 ORDER BY s.submission_time DESC, s.id DESC
                 LIMIT %s OFFSET %s
                 """,
                 (leaderboard_name, limit, offset),
             )
-            return [
-                {
-                    "id": row[0],
-                    "leaderboard_name": row[1],
-                    "file_name": row[2],
-                    "user_id": row[3],
-                    "user_name": row[4],
-                    "submission_time": row[5],
-                    "done": row[6],
-                }
-                for row in self.cursor.fetchall()
-            ]
+            return [row[0] for row in self.cursor.fetchall()]
         except KernelBotError:
             self.connection.rollback()
             raise

tests/test_admin_api.py

@@ -142,29 +142,10 @@ class TestAdminSubmissions:
     """Test admin submission endpoints."""
 
     def test_list_leaderboard_submissions(self, test_client, mock_backend):
-        """GET /admin/leaderboards/{name}/submissions returns all submission metadata."""
+        """GET /admin/leaderboards/{name}/submissions returns submission IDs."""
         mock_backend.db.__enter__ = MagicMock(return_value=mock_backend.db)
         mock_backend.db.__exit__ = MagicMock(return_value=None)
-        mock_backend.db.get_leaderboard_submission_history = MagicMock(return_value=[
-            {
-                "id": 123,
-                "leaderboard_name": "test-lb",
-                "file_name": "submission.py",
-                "user_id": "42",
-                "user_name": "alice",
-                "submission_time": "2026-04-07T12:00:00Z",
-                "done": True,
-            },
-            {
-                "id": 122,
-                "leaderboard_name": "test-lb",
-                "file_name": "submission_old.py",
-                "user_id": "43",
-                "user_name": "bob",
-                "submission_time": "2026-04-07T11:00:00Z",
-                "done": False,
-            },
-        ])
+        mock_backend.db.get_leaderboard_submission_ids = MagicMock(return_value=[123, 122])
 
         response = test_client.get(
             "/admin/leaderboards/test-lb/submissions?limit=50&offset=10",
@@ -176,30 +157,9 @@ def test_list_leaderboard_submissions(self, test_client, mock_backend):
             "leaderboard": "test-lb",
             "limit": 50,
             "offset": 10,
-            "submissions": [
-                {
-                    "id": 123,
-                    "leaderboard_name": "test-lb",
-                    "file_name": "submission.py",
-                    "user_id": "42",
-                    "user_name": "alice",
-                    "submission_time": "2026-04-07T12:00:00Z",
-                    "done": True,
-                },
-                {
-                    "id": 122,
-                    "leaderboard_name": "test-lb",
-                    "file_name": "submission_old.py",
-                    "user_id": "43",
-                    "user_name": "bob",
-                    "submission_time": "2026-04-07T11:00:00Z",
-                    "done": False,
-                },
-            ],
+            "submission_ids": [123, 122],
         }
-        mock_backend.db.get_leaderboard_submission_history.assert_called_once_with(
-            "test-lb", 50, 10
-        )
+        mock_backend.db.get_leaderboard_submission_ids.assert_called_once_with("test-lb", 50, 10)
 
     def test_get_submission(self, test_client, mock_backend):
         """GET /admin/submissions/{id} returns submission."""