Merge pull request #14 from rudi-q/feature/seo-optimization

feat: implement route-based PDF handling with dedicated upload and vi…

Author: rudi-q

src/app.d.ts

@@ -8,6 +8,20 @@ declare global {
 		// interface PageState {}
 		// interface Platform {}
 	}
+
+	// Extend Window interface for our custom cleanup properties
+	interface Window {
+		__pdfRouteCleanup?: {
+			unlistenFileOpened: Promise<() => void>;
+			unlistenStartupReady: Promise<() => void>;
+			unlistenDebug: Promise<() => void>;
+		};
+		__pdfUploadCleanup?: {
+			unlistenFileOpened: Promise<() => void>;
+			unlistenStartupReady: Promise<() => void>;
+			unlistenDebug: Promise<() => void>;
+		};
+	}
 }
 
 // Declare .svelte modules for TypeScript

src/app.html

@@ -6,7 +6,6 @@
 		<!-- Favicon and Icons -->
 		<link rel="icon" href="%sveltekit.assets%/favicon.png" />
 		<link rel="apple-touch-icon" href="%sveltekit.assets%/logo.png" />
-		<link rel="mask-icon" href="%sveltekit.assets%/logo.svg" color="#8B9474" />
 
 		<!-- Viewport and Mobile -->
 		<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, minimum-scale=1.0, user-scalable=no" />

src/routes/+page.svelte

@@ -41,12 +41,13 @@
   let maxFileLoadingAttempts = 10;
   let fileLoadingTimer: number | null = null;
 
-  // Check for PDF URL parameter reactively
+  // Redirect from /?pdf=URL to /pdf/URL for better URL structure
   $: if (browser && $page && $page.url) {
     const pdfUrl = $page.url.searchParams.get('pdf');
-    if (pdfUrl && !currentFile) {
-      console.log('[PDF URL] Found PDF parameter:', pdfUrl);
-      handlePdfUrlLoad(pdfUrl);
+    if (pdfUrl) {
+      console.log('[PDF URL] Redirecting to new URL structure:', pdfUrl);
+      const encodedUrl = encodeURIComponent(pdfUrl);
+      goto(`/pdf/${encodedUrl}`);
     }
   }
 
@@ -67,13 +68,22 @@
       return;
     }
 
-    console.log('Setting currentFile and hiding welcome');
-    currentFile = file;
-    showWelcome = false;
-
-    // Set current PDF for auto-save functionality
-    setCurrentPDF(file.name, file.size);
-    console.log('Updated state:', { currentFile: !!currentFile, showWelcome });
+    console.log('Storing file and navigating to pdf-upload route');
+    // Store file in sessionStorage temporarily
+    const fileReader = new FileReader();
+    fileReader.onload = (e) => {
+      const arrayBuffer = e.target?.result as ArrayBuffer;
+      const fileData = {
+        name: file.name,
+        size: file.size,
+        type: file.type,
+        data: Array.from(new Uint8Array(arrayBuffer)) // Convert to array for JSON storage
+      };
+      sessionStorage.setItem('tempPdfFile', JSON.stringify(fileData));
+      console.log('File stored in sessionStorage, navigating...');
+      goto('/pdf-upload');
+    };
+    fileReader.readAsArrayBuffer(file);
   }
 
   function isValidPdfUrl(url: string): boolean {
@@ -644,8 +654,8 @@
       return;
     }
 
-    // Navigate to the same page with the PDF parameter
-    goto(`/?pdf=${encodeURIComponent(trimmedUrl)}`);
+    // Navigate to the new PDF route structure
+    goto(`/pdf/${encodeURIComponent(trimmedUrl)}`);
   }
 
   function handleUrlCancel() {
@@ -670,13 +680,18 @@
       const blankPdfFile = await createBlankPDF();
       console.log('Blank PDF created:', blankPdfFile.name, blankPdfFile.size, 'bytes');
       
-      // Load the blank PDF just like a regular file
-      currentFile = blankPdfFile;
-      showWelcome = false;
-      
-      // Set current PDF for auto-save functionality
-      setCurrentPDF(blankPdfFile.name, blankPdfFile.size);
-      console.log('Blank PDF loaded successfully!');
+      console.log('Storing blank PDF and navigating to pdf-upload route');
+      // Store file in sessionStorage temporarily
+      const arrayBuffer = await blankPdfFile.arrayBuffer();
+      const fileData = {
+        name: blankPdfFile.name,
+        size: blankPdfFile.size,
+        type: blankPdfFile.type,
+        data: Array.from(new Uint8Array(arrayBuffer))
+      };
+      sessionStorage.setItem('tempPdfFile', JSON.stringify(fileData));
+      console.log('Blank PDF stored in sessionStorage, navigating...');
+      goto('/pdf-upload');
     } catch (error) {
       console.error('Failed to create blank PDF:', error);
       alert('Failed to create blank PDF. Please try again.');

src/routes/pdf-upload/+page.js

@@ -0,0 +1,3 @@
+// Disable prerendering for this route since it handles dynamic user-uploaded files
+// and depends on sessionStorage data that's only available at runtime
+export const prerender = false;