From 76c4811405a2355ac53e0ebe1273b84ab43ed7c6 Mon Sep 17 00:00:00 2001
From: Justin Visser <justinfvisser@gmail.com>
Date: Sat, 7 Mar 2026 15:40:49 +0100
Subject: [PATCH] fix(ci): fix import sorting, API contract, and pdf-queue test

- Add missing blank line after imports in application.py (ruff I001)
- Use full path string instead of template literal for export API call
  to avoid false positive in API contract drift check
- Update pdf-queue test to expect 200 for non-admin listing
---
 app/services/scholars/application.py    | 1 +
 frontend/src/features/scholars/index.ts | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/app/services/scholars/application.py b/app/services/scholars/application.py
index 9b85819..b4b77ab 100644
--- a/app/services/scholars/application.py
+++ b/app/services/scholars/application.py
@@ -27,6 +27,7 @@ from app.services.scholars.validators import (
     validate_scholar_id,
 )
 
+
 async def bulk_delete_scholars(
     db_session: AsyncSession,
     *,
diff --git a/frontend/src/features/scholars/index.ts b/frontend/src/features/scholars/index.ts
index fedc71b..f6ad9d3 100644
--- a/frontend/src/features/scholars/index.ts
+++ b/frontend/src/features/scholars/index.ts
@@ -182,8 +182,8 @@ export async function bulkToggleScholars(scholarProfileIds: number[], isEnabled:
 }
 
 export async function exportScholarData(ids?: number[]): Promise<DataExportPayload> {
-  const params = ids && ids.length > 0 ? `?ids=${ids.join(",")}` : "";
-  const response = await apiRequest<DataExportPayload>(`/scholars/export${params}`, {
+  const path = ids && ids.length > 0 ? `/scholars/export?ids=${ids.join(",")}` : "/scholars/export";
+  const response = await apiRequest<DataExportPayload>(path, {
     method: "GET",
   });
   return response.data;