From 8eb0383d488b5a0ca54f9e247819014e7b84d62d Mon Sep 17 00:00:00 2001
From: faligam <faligam@users.noreply.github.com>
Date: Sun, 6 Sep 2026 17:23:11 +0200
Subject: [PATCH] Fix API-key login and research navigation

Validate submitted keys against the protected research endpoint instead of the nonexistent versioned health path. Replace server-only redirects in client code with SvelteKit navigation and route successful logins to the existing new-research screen.

Align frontend research history and creation calls with the backend response contract: list items are wrapped in items and new runs return research_id.
---
 src/lib/sidebar.ts                   |  9 +++++----
 src/routes/+page.svelte              | 13 +++++++------
 src/routes/research/new/+page.svelte |  3 +--
 3 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/src/lib/sidebar.ts b/src/lib/sidebar.ts
index 3e1bc8f..a8f6536 100644
--- a/src/lib/sidebar.ts
+++ b/src/lib/sidebar.ts
@@ -16,7 +16,8 @@ export interface ResearchListItem extends ResearchSummary {
 const HISTORY_LIMIT = 50;
 
 export async function loadResearchHistory(apiKey: string): Promise<ResearchSummary[]> {
-    const list = await apiGet<ResearchSummary[]>('/v1/research', { apiKey });
+    const response = await apiGet<{ items: ResearchSummary[] }>('/v1/research', { apiKey });
+    const list = response.items;
     // Sort descending by created_at (newest first), limit to HISTORY_LIMIT
     const sorted = list
         .slice()
@@ -34,10 +35,10 @@ export async function createResearch(
     query: string,
     language: string,
     depth: string
-): Promise<{ id: string }> {
-    return await apiPost<{ id: string }>(
+): Promise<{ research_id: string }> {
+    return await apiPost<{ research_id: string }>(
         '/v1/research',
         { query, language, depth },
         { apiKey }
     );
-}
\ No newline at end of file
+}
diff --git a/src/routes/+page.svelte b/src/routes/+page.svelte
index 3056bba..fe44e98 100644
--- a/src/routes/+page.svelte
+++ b/src/routes/+page.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-    import { redirect } from '@sveltejs/kit';
+    import { goto } from '$app/navigation';
     import { user } from '$lib/stores';
     import { theme, toggleTheme, applyTheme } from '$lib/theme';
     import { isAuthenticated } from '$lib/auth';
@@ -14,7 +14,7 @@
 
     onMount(() => {
         if (isAuthenticated()) {
-            redirect(302, '/research');
+            void goto('/research/new');
         }
         applyTheme($theme);
     });
@@ -29,11 +29,12 @@
         error = '';
 
         try {
-            // Validate the API key by fetching user info from the backend
-            const info = await apiGet<any>('/v1/health', { apiKey: apiKey.trim() });
+            // Validate against a protected endpoint. `/health` is intentionally
+            // public and is not versioned; using `/v1/health` caused a 404.
+            await apiGet('/v1/research', { apiKey: apiKey.trim() });
             user.set({ apiKey: apiKey.trim() });
             localStorage.setItem('nsct-api-key', apiKey.trim());
-            redirect(302, '/research');
+            await goto('/research/new');
         } catch (e) {
             // If the API key is invalid, the backend will return 401/403
             // or an error response — accept it and show the error
@@ -111,4 +112,4 @@
             </button>
         </div>
     </div>
-</div>
\ No newline at end of file
+</div>
diff --git a/src/routes/research/new/+page.svelte b/src/routes/research/new/+page.svelte
index 46d4819..ffa45bd 100644
--- a/src/routes/research/new/+page.svelte
+++ b/src/routes/research/new/+page.svelte
@@ -1,5 +1,4 @@
 <script lang="ts">
-    import { redirect } from '@sveltejs/kit';
     import { isAuthenticated } from '$lib/auth';
     import { createResearch } from '$lib/sidebar';
     import { errorBanners } from '$lib/components/ErrorBannerStore';
@@ -46,7 +45,7 @@
         try {
             const result = await createResearch(key, query.trim(), language, depth);
             errorBanners.addErrorSuccess('Research wird erstellt…');
-            window.location.href = `/research/${result.id}`;
+            window.location.href = `/research/${result.research_id}`;
         } catch (e) {
             const msg = e instanceof Error ? e.message : 'Fehler beim Erstellen der Recherche.';
             error = msg;