refactor: improve URL handling and simplify types

- Add error handling for URL parameter parsing in App.vue

- Improve extension detection in openUriStream.ts
  - Use basename() and lastIndexOf('.') for robust extension check
  - Prevents false positives from hidden files and URL domains

Author: PaulHax

src/components/App.vue

@@ -147,9 +147,15 @@ export default defineComponent({
     populateAuthorizationToken();
     stripTokenFromUrl();
 
-    const urlParams = normalizeUrlParams(
-      vtkURLExtract.extractURLParameters() as UrlParams
-    );
+    let urlParams: ReturnType<typeof normalizeUrlParams>;
+    try {
+      urlParams = normalizeUrlParams(
+        vtkURLExtract.extractURLParameters() as UrlParams
+      );
+    } catch (error) {
+      console.error('Failed to parse URL parameters:', error);
+      urlParams = {};
+    }
 
     onMounted(() => {
       loadUrls(urlParams);

src/io/import/processors/openUriStream.ts

@@ -4,6 +4,7 @@ import { getRequestPool } from '@/src/core/streaming/requestPool';
 import { ImportHandler, asIntermediateResult } from '@/src/io/import/common';
 import { canFetchUrl } from '@/src/utils/fetch';
 import { extractFilenameFromContentDisposition } from '@/src/utils/parseContentDispositionHeader';
+import { basename } from '@/src/utils/path';
 
 const openUriStream: ImportHandler = async (dataSource, context) => {
   if (dataSource.type !== 'uri' || !canFetchUrl(dataSource.uri)) {
@@ -26,7 +27,7 @@ const openUriStream: ImportHandler = async (dataSource, context) => {
 
   // Only use Content-Disposition if current name lacks an extension
   // (indicating it's likely auto-derived from URL like "download" or "getImage")
-  const hasExtension = dataSource.name.includes('.');
+  const hasExtension = basename(dataSource.name).lastIndexOf('.') > 0;
   const finalName =
     !hasExtension && filenameFromHeader ? filenameFromHeader : dataSource.name;
 

src/utils/parseContentDispositionHeader.ts

@@ -1,10 +1,10 @@
 const CONTENT_DISPOSITION_FILENAME_REGEXP =
   /filename\s*=\s*(?:"([^"]*)"|([^;\s]*))/i;
 
-export type ContentDisposition =
-  | { type: 'invalid'; filename: null }
-  | { type: 'inline'; filename: string | null }
-  | { type: 'attachment'; filename: string | null };
+export type ContentDisposition = {
+  type: 'inline' | 'attachment' | 'invalid';
+  filename: string | null;
+};
 
 export function parseContentDispositionHeader(
   headerValue: string | null

src/utils/urlParams.ts

@@ -18,6 +18,12 @@ const isValidUrl = (str: string) => {
   }
 };
 
+const splitAndClean = (str: string) =>
+  str
+    .split(',')
+    .map((url) => url.trim())
+    .filter(Boolean);
+
 const parseUrlArray = (value: string | string[]): string[] => {
   if (Array.isArray(value)) {
     return value.flatMap((v) => parseUrlArray(v));
@@ -28,18 +34,11 @@ const parseUrlArray = (value: string | string[]): string[] => {
   if (!trimmed) return [];
 
   if (trimmed.startsWith('[') && trimmed.endsWith(']')) {
-    const inner = trimmed.slice(1, -1);
-    return inner
-      .split(',')
-      .map((url) => url.trim())
-      .filter(Boolean);
+    return splitAndClean(trimmed.slice(1, -1));
   }
 
   if (trimmed.includes(',')) {
-    return trimmed
-      .split(',')
-      .map((url) => url.trim())
-      .filter(Boolean);
+    return splitAndClean(trimmed);
   }
 
   return [trimmed];