fix: normalize discovered source URLs

Author: Programcode

src/lib/ai/client.ts

@@ -113,9 +113,10 @@ function parseDiscoveredSources(value: unknown): DiscoveredSource[] {
 
     const title = typeof candidate.title === 'string' ? candidate.title.trim() : '';
     const url = typeof candidate.url === 'string' ? candidate.url.trim() : '';
-    if (!title || !validateUrl(url)) return [];
+    const normalizedUrl = validateUrl(url);
+    if (!title || !normalizedUrl) return [];
 
-    const source: DiscoveredSource = { title, url };
+    const source: DiscoveredSource = { title, url: normalizedUrl };
     if (typeof candidate.content === 'string') source.content = candidate.content;
 
     if (typeof candidate.sourceType === 'string' && candidate.sourceType.trim()) {

src/test/ai/client-branches.test.ts

@@ -259,10 +259,10 @@ describe('AI client branch coverage', () => {
       title: 'Source A',
       content: 'A content',
       type: 'url',
-      origin: 'https://a.com',
+      origin: 'https://a.com/',
       folder: 'Auto-saved',
       isActive: true,
-      metadata: expect.objectContaining({ capturedVia: 'generate', url: 'https://a.com' }),
+      metadata: expect.objectContaining({ capturedVia: 'generate', url: 'https://a.com/' }),
     });
   });
 
@@ -311,6 +311,7 @@ describe('AI client branch coverage', () => {
             qualityLabel: 'unknown',
             query: 42,
           },
+          { title: 'Protocol-less source', url: 'example.com/source-two' },
           { title: '', url: 'https://example.com/rejected' },
           { title: 'Unsafe protocol', url: 'httpx://example.com/rejected' },
         ],
@@ -322,6 +323,7 @@ describe('AI client branch coverage', () => {
 
     expect(outcome.discoveredSources).toEqual([
       { title: 'Source with malformed metadata', url: 'https://example.com/source' },
+      { title: 'Protocol-less source', url: 'https://example.com/source-two' },
     ]);
     expect(addSourceMock).toHaveBeenCalledWith(expect.objectContaining({
       title: 'Source with malformed metadata',
@@ -428,10 +430,10 @@ describe('AI client branch coverage', () => {
       title: 'Source C',
       content: 'C content',
       type: 'url',
-      origin: 'https://c.com',
+      origin: 'https://c.com/',
       folder: 'Auto-saved',
       isActive: true,
-      metadata: expect.objectContaining({ capturedVia: 'brainstorm', url: 'https://c.com' }),
+      metadata: expect.objectContaining({ capturedVia: 'brainstorm', url: 'https://c.com/' }),
     });
   });