Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 30 additions & 2 deletions src/lib/components/csvImportBox.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,12 @@
*/
let importItems = $state<ImportItemsMap>(new Map());

/**
* Tracks removal timeouts for completed/failed imports.
* Used to prevent memory leaks and race conditions.
*/
let removalTimeouts = new Map<string, ReturnType<typeof setTimeout>>();

async function showCompletionNotification(database: string, table: string, payload: Payload) {
const isSuccess = payload.status === 'completed';
const isError = !isSuccess && !!payload.errors;
Expand Down Expand Up @@ -117,11 +123,27 @@
}

if (status === 'completed' || status === 'failed') {
await showCompletionNotification(databaseId, tableId, importData);
try {
await showCompletionNotification(databaseId, tableId, importData);
} finally {
const existingTimeout = removalTimeouts.get(importData.$id);
if (existingTimeout) {
clearTimeout(existingTimeout);
}
const timeoutId = setTimeout(() => {
const next = new Map(importItems);
next.delete(importData.$id);
importItems = next;
removalTimeouts.delete(importData.$id);
}, 5000);
removalTimeouts.set(importData.$id, timeoutId);
}
}
}

function clear() {
removalTimeouts.forEach((timeout) => clearTimeout(timeout));
removalTimeouts = new Map();
importItems = new Map();
}

Expand Down Expand Up @@ -187,12 +209,18 @@
migrations.migrations.forEach(updateOrAddItem);
});

return realtime.forConsole(page.params.region, 'console', (response) => {
const unsubscribe = realtime.forConsole(page.params.region, 'console', (response) => {
if (!response.channels.includes(`projects.${getProjectId()}`)) return;
if (response.events.includes('migrations.*')) {
updateOrAddItem(response.payload as Payload);
}
});

return () => {
unsubscribe();
removalTimeouts.forEach((timeout) => clearTimeout(timeout));
removalTimeouts = new Map();
};
});

let isOpen = $state(true);
Expand Down