fix(urlParams): don't split URLs containing commas in query strings

Author: PaulHax

src/utils/urlParams.test.ts

@@ -29,16 +29,6 @@ describe('normalizeUrlParams', () => {
     ]);
   });
 
-  it('handles comma-separated URLs', () => {
-    const result = normalizeUrlParams({
-      urls: 'https://example.com/file1.dcm,https://example.com/file2.dcm',
-    });
-    expect(result.urls).toEqual([
-      'https://example.com/file1.dcm',
-      'https://example.com/file2.dcm',
-    ]);
-  });
-
   it('allows relative URLs (they resolve against base URL)', () => {
     const result = normalizeUrlParams({
       urls: ['https://example.com/valid.dcm', 'relative-path', 'another/path'],
@@ -59,6 +49,13 @@ describe('normalizeUrlParams', () => {
     ]);
   });
 
+  it('preserves commas in query strings', () => {
+    const result = normalizeUrlParams({
+      urls: '/api/v1/folder/123/volview?items=id1,id2,id3',
+    });
+    expect(result.urls).toEqual(['/api/v1/folder/123/volview?items=id1,id2,id3']);
+  });
+
   it('handles config and names parameters', () => {
     const result = normalizeUrlParams({
       config: 'https://example.com/config.json',

src/utils/urlParams.ts

@@ -35,10 +35,6 @@ const parseUrlArray = (value: string | string[]): string[] => {
     return splitAndClean(trimmed.slice(1, -1));
   }
 
-  if (trimmed.includes(',')) {
-    return splitAndClean(trimmed);
-  }
-
   return [trimmed];
 };