Fix service worker mess

package.json

@@ -1,7 +1,7 @@
 {
 	"name": "scan-wave",
 	"private": true,
-	"version": "0.0.1",
+	"version": "0.0.2",
 	"type": "module",
 	"scripts": {
 		"dev": "vite dev",

src/lib/google/sheets/server.ts

@@ -51,25 +51,48 @@ export async function getRecentSpreadsheets(
  * Get data from a Google Sheet
  * @param refreshToken - Google refresh token
  * @param spreadsheetId - ID of the spreadsheet
- * @param range - Cell range to retrieve (default: A1:Z10)
+ * @param range - Optional cell range. If not provided, it will fetch the entire first sheet.
  * @returns Sheet data as a 2D array
  */
 export async function getSpreadsheetData(
-  refreshToken: string,
+	refreshToken: string,
-  spreadsheetId: string,
+	spreadsheetId: string,
-  range: string = 'A1:Z10'
+	range?: string
 ): Promise<SheetData> {
-  const oauth = getAuthenticatedClient(refreshToken);
+	const oauth = getAuthenticatedClient(refreshToken);
-  const sheets = google.sheets({ version: 'v4', auth: oauth });
+	const sheets = google.sheets({ version: 'v4', auth: oauth });
 
-  const response = await sheets.spreadsheets.values.get({
+	let effectiveRange = range;
-    spreadsheetId,
-    range
-  });
 
-  return {
+	// If no range is provided, get the name of the first sheet and use that as the range
-    values: response.data.values || []
+	// to fetch all its content.
-  };
+	if (!effectiveRange) {
+		try {
+			const info = await getSpreadsheetInfo(refreshToken, spreadsheetId);
+			const firstSheetName = info.sheets?.[0]?.properties?.title;
+
+			if (firstSheetName) {
+				// To use a sheet name as a range, it must be quoted if it contains spaces or special characters.
+				effectiveRange = `'${firstSheetName}'`;
+			} else {
+				// Fallback if sheet name can't be determined.
+				effectiveRange = 'A1:Z1000'; // A sensible default for a large preview
+			}
+		} catch (error) {
+			console.error(`Failed to get sheet info for spreadsheet ${spreadsheetId}`, error);
+			// Fallback if the info call fails
+			effectiveRange = 'A1:Z1000';
+		}
+	}
+
+	const response = await sheets.spreadsheets.values.get({
+		spreadsheetId,
+		range: effectiveRange
+	});
+
+	return {
+		values: response.data.values || []
+	};
 }
 
 /**

src/routes/private/api/google/sheets/[sheetId]/data/+server.ts

@@ -2,17 +2,18 @@ import { json } from '@sveltejs/kit';
 import type { RequestHandler } from './$types';
 import { googleSheetsServer } from '$lib/google/sheets/server.js';
 
-export const GET: RequestHandler = async ({ params, request }) => {
+export const GET: RequestHandler = async ({ params, request, url }) => {
 	try {
 		const { sheetId } = params;
 		const authHeader = request.headers.get('authorization');
+		const range = url.searchParams.get('range') || undefined;
 
 		if (!authHeader?.startsWith('Bearer ')) {
 			return json({ error: 'Missing or invalid authorization header' }, { status: 401 });
 		}
 
 		const refreshToken = authHeader.slice(7);
-		const sheetData = await googleSheetsServer.getSpreadsheetData(refreshToken, sheetId, 'A1:Z10');
+		const sheetData = await googleSheetsServer.getSpreadsheetData(refreshToken, sheetId, range);
 
 		return json(sheetData);
 	} catch (error) {

src/routes/private/events/event/new/+page.svelte

@@ -172,8 +172,8 @@
 		}
 
 		try {
-			// Use the new unified API endpoint
+			// Use the new unified API endpoint, requesting only a preview range
-			const response = await fetch(`/private/api/google/sheets/${sheet.id}/data`, {
+			const response = await fetch(`/private/api/google/sheets/${sheet.id}/data?range=A1:Z10`, {
 				method: 'GET',
 				headers: {
 					Authorization: `Bearer ${localStorage.getItem('google_refresh_token')}`

src/routes/private/events/event/view/+page.svelte

@@ -115,6 +115,8 @@
 		}
 
 		syncingParticipants = true;
+		const previousCount = participants.length; // Capture count before sync
+
 		try {
 			// Fetch sheet data
 			const response = await fetch(`/private/api/google/sheets/${event.sheet_id}/data`, {
@@ -136,35 +138,64 @@
 
 			if (rows.length === 0) throw new Error('No data found in sheet');
 
-			// Extract participant data based on column mapping
+			// --- Start of new logic to handle duplicates ---
-			const names: string[] = [];
-			const surnames: string[] = [];
-			const emails: string[] = [];
 
-			// Skip header row (start from index 1)
+			// First, extract all potential participants from the sheet
+			const potentialParticipants = [];
 			for (let i = 1; i < rows.length; i++) {
 				const row = rows[i];
 				if (row.length > 0) {
 					const name = row[event.name_column - 1] || '';
 					const surname = row[event.surname_column - 1] || '';
-					const email = row[event.email_column - 1] || '';
+					const email = (row[event.email_column - 1] || '').trim();
 					const confirmation = row[event.confirmation_column - 1] || '';
 
-					// Only add if the row has meaningful data (not all empty) AND confirmation is TRUE
 					const isConfirmed =
 						confirmation.toString().toLowerCase() === 'true' ||
 						confirmation.toString().toLowerCase() === 'yes' ||
 						confirmation === '1' ||
 						confirmation === 'x';
 
-					if ((name.trim() || surname.trim() || email.trim()) && isConfirmed) {
+					if ((name.trim() || surname.trim() || email) && isConfirmed) {
-						names.push(name.trim());
+						potentialParticipants.push({ name: name.trim(), surname: surname.trim(), email });
-						surnames.push(surname.trim());
-						emails.push(email.trim());
 					}
 				}
 			}
 
+			// Create a map to count occurrences of each unique participant combination
+			const participantCounts = new Map<string, number>();
+			for (const p of potentialParticipants) {
+				const key = `${p.name}|${p.surname}|${p.email}`.toLowerCase(); // Create a unique key
+				participantCounts.set(key, (participantCounts.get(key) || 0) + 1);
+			}
+
+			// Create final arrays, modifying duplicate surnames to be unique
+			const names: string[] = [];
+			const surnames: string[] = [];
+			const emails: string[] = [];
+			const processedParticipants = new Map<string, number>();
+
+			for (const p of potentialParticipants) {
+				const key = `${p.name}|${p.surname}|${p.email}`.toLowerCase();
+				let finalSurname = p.surname;
+
+				// If this participant is a duplicate
+				if (participantCounts.get(key)! > 1) {
+					const count = (processedParticipants.get(key) || 0) + 1;
+					processedParticipants.set(key, count);
+
+					// If it's not the first occurrence, append a counter to the surname
+					if (count > 1) {
+						finalSurname = `${p.surname} (${count})`;
+					}
+				}
+
+				names.push(p.name);
+				surnames.push(finalSurname);
+				emails.push(p.email); // Keep the original email
+			}
+			// --- End of new logic ---
+
 			// Call database function to add participants
 			const { error: syncError } = await data.supabase.rpc('participants_add_bulk', {
 				p_event: eventId,
@@ -178,15 +209,22 @@
 			// Reload participants
 			await loadParticipants();
 
-			// Show success message with count of synced participants
+			// Show success message with accurate count of changes
-			const previousCount = participants.length;
+			const newCount = participants.length;
-			const newCount = names.length;
+			const diff = newCount - previousCount;
-			const addedCount = Math.max(0, participants.length - previousCount);
+			const processedCount = names.length;
 
-			toast.success(
+			let message = `Sync complete. ${processedCount} confirmed entries processed from the sheet.`;
-				`Successfully synced participants. ${newCount} entries processed, ${addedCount} new participants added.`,
+
-				5000
+			if (diff > 0) {
-			);
+				message += ` ${diff} new participants added.`;
+			} else if (diff < 0) {
+				message += ` ${-diff} participants removed.`;
+			} else {
+				message += ` No changes to the participant list.`;
+			}
+
+			toast.success(message, 6000);
 		} catch (err) {
 			console.error('Error syncing participants:', err);
 			toast.error(`Failed to sync participants: ${err instanceof Error ? err.message : 'Unknown error'}`);

src/service-worker.ts

@@ -1,86 +1,71 @@
+/// <reference lib="webworker" />
 /// <reference types="@sveltejs/kit" />
 import { build, files, version } from '$service-worker';
 
-// Create a unique cache name for this deployment
+declare const self: ServiceWorkerGlobalScope;
-const CACHE = `cache-${version}`;
 
+const CACHE = `cache-${version}`;
 const ASSETS = [
-	...build, // the app itself
+    ...build,
-	...files  // everything in `static`
+    ...files
 ];
 
-self.addEventListener('install', (event) => {
+self.addEventListener('install', (event: ExtendableEvent) => {
-	// Create a new cache and add all files to it
+    const addFilesToCache = async () => {
-	async function addFilesToCache() {
+        const cache = await caches.open(CACHE);
-		const cache = await caches.open(CACHE);
+        await cache.addAll(ASSETS);
-		await cache.addAll(ASSETS);
+    };
-	}
 
-	event.waitUntil(addFilesToCache());
+	console.log("[SW] Installing new service worker");
+
+    event.waitUntil(addFilesToCache());
+    self.skipWaiting();
 });
 
-self.addEventListener('activate', (event) => {
+self.addEventListener('activate', (event: ExtendableEvent) => {
-	// Remove previous cached data from disk
+    const deleteOldCaches = async () => {
-	async function deleteOldCaches() {
+        for (const key of await caches.keys()) {
-		for (const key of await caches.keys()) {
+            if (key !== CACHE) await caches.delete(key);
-			if (key !== CACHE) await caches.delete(key);
+			console.log("[SW] Removing old service worker")
-		}
+        }
-	}
+    };
 
-	event.waitUntil(deleteOldCaches());
+    event.waitUntil(deleteOldCaches());
+    self.clients.claim();
 });
 
-self.addEventListener('fetch', (event) => {
+self.addEventListener('fetch', (event: FetchEvent) => {
-	// ignore POST requests etc
+    if (event.request.method !== 'GET') return;
-	if (event.request.method !== 'GET') return;
 
-	async function respond() {
+    const url = new URL(event.request.url);
-		const url = new URL(event.request.url);
 
-		// Skip caching for auth routes
+    // Never cache private routes
-		if (url.pathname.startsWith('/auth/')) {
+    if (url.pathname.startsWith('/private')) {
-			return fetch(event.request);
+        event.respondWith(fetch(event.request));
-		}
+        return;
+    }
 
-		const cache = await caches.open(CACHE);
+    const respond = async () => {
+        const cache = await caches.open(CACHE);
 
-		// `build`/`files` can always be served from the cache
+        if (ASSETS.includes(url.pathname)) {
-		if (ASSETS.includes(url.pathname)) {
+            const cached = await cache.match(url.pathname);
-			const response = await cache.match(url.pathname);
+            if (cached) return cached;
+        }
 
-			if (response) {
+        try {
-				return response;
+            const response = await fetch(event.request);
-			}
+            if (response.status === 200 && build.length > 0 && url.pathname.startsWith(`/${build[0]}/`)) {
-		}
+                cache.put(event.request, response.clone());
+            }
+            return response;
+        } catch {
+            const cached = await cache.match(event.request);
+            if (cached) return cached;
+        }
 
-		// for everything else, try the network first, but
+        return new Response('Not found', { status: 404 });
-		// fall back to the cache if we're offline
+    };
-		try {
-			const response = await fetch(event.request);
 
-			// if we're offline, fetch can return a value that is not a Response
+    event.respondWith(respond());
-			// instead of throwing - and we can't pass this non-Response to respondWith
-			if (!(response instanceof Response)) {
-				throw new Error('invalid response from fetch');
-			}
-
-			if (response.status === 200) {
-				cache.put(event.request, response.clone());
-			}
-
-			return response;
-		} catch (err) {
-			const response = await cache.match(event.request);
-
-			if (response) {
-				return response;
-			}
-
-			// if there's no cache, then just error out
-			// as there is nothing we can do to respond to this request
-			throw err;
-		}
-	}
-
-	event.respondWith(respond());
 });