Add GET /api/v1/commitfests endpoint

There is currently no way to discover which commitfests exist. The
needs_ci endpoint only returns the two or three currently relevant ones,
and the patches endpoint requires a commitfest id you already know. API
consumers are left enumerating ids until they get a 404, or scraping the
HTML pages that already show this data.

This returns every commitfest, closed and draft ones included, ordered
by id.

It deliberately does not use CommitFest.relevant_commitfests(), which
has write side effects and can return unsaved objects.

Claude-Session: https://claude.ai/code/session_01QTYvAVjc8amv58QErCwox2

Author: Kevin Rocker

pgcommitfest/commitfest/apiv1.py

@@ -36,6 +36,11 @@ def api_response(payload, status=200, content_type="application/json"):
     return response
 
 
+def all_commitfests(request):
+    """Return all commitfests, including closed ones."""
+    return api_response({"commitfests": list(CommitFest.objects.order_by("id"))})
+
+
 def commitfestst_that_need_ci(request):
     cfs = CommitFest.relevant_commitfests()
 

pgcommitfest/commitfest/tests/test_apiv1.py

@@ -12,6 +12,70 @@
 pytestmark = pytest.mark.django_db
 
 
+def test_commitfests_endpoint(client, commitfests):
+    """Test the /api/v1/commitfests endpoint returns all commitfests."""
+    response = client.get("/api/v1/commitfests")
+
+    assert response.status_code == 200
+    assert response["Content-Type"] == "application/json"
+    assert response["Access-Control-Allow-Origin"] == "*"
+
+    data = json.loads(response.content)
+
+    expected = [
+        {
+            "id": commitfests["open"].id,
+            "name": "2025-01",
+            "status": "Open",
+            "draft": False,
+            "startdate": "2025-01-01",
+            "enddate": "2025-01-31",
+        },
+        {
+            "id": commitfests["in_progress"].id,
+            "name": "2024-11",
+            "status": "In Progress",
+            "draft": False,
+            "startdate": "2024-11-01",
+            "enddate": "2024-11-30",
+        },
+        {
+            "id": commitfests["recent_previous"].id,
+            "name": "2024-09",
+            "status": "Closed",
+            "draft": False,
+            "startdate": "2024-09-01",
+            "enddate": "2024-09-30",
+        },
+        {
+            "id": commitfests["old_previous"].id,
+            "name": "2024-07",
+            "status": "Closed",
+            "draft": False,
+            "startdate": "2024-07-01",
+            "enddate": "2024-07-31",
+        },
+        {
+            "id": commitfests["draft"].id,
+            "name": "2025-03-draft",
+            "status": "Open",
+            "draft": True,
+            "startdate": "2025-03-01",
+            "enddate": "2025-03-31",
+        },
+    ]
+
+    assert data == {"commitfests": sorted(expected, key=lambda cf: cf["id"])}
+
+
+def test_commitfests_endpoint_empty(client):
+    """Test the /api/v1/commitfests endpoint with no commitfests."""
+    response = client.get("/api/v1/commitfests")
+
+    assert response.status_code == 200
+    assert json.loads(response.content) == {"commitfests": []}
+
+
 def test_needs_ci_endpoint(client, commitfests):
     """Test the /api/v1/commitfests/needs_ci endpoint returns correct data."""
     response = client.get("/api/v1/commitfests/needs_ci")

pgcommitfest/urls.py

@@ -16,6 +16,7 @@
 
 urlpatterns = [
     re_path(r"^$", views.home),
+    re_path(r"^api/v1/commitfests$", apiv1.all_commitfests),
     re_path(r"^api/v1/commitfests/needs_ci$", apiv1.commitfestst_that_need_ci),
     re_path(r"^api/v1/commitfests/(\d+)/patches$", apiv1.commitfest_patches),
     re_path(r"^api/v1/patches/(\d+)/threads$", apiv1.patch_threads),