lib/config-store.js

@@ -277,12 +277,7 @@ class ConfigStore {
             if (fs.existsSync(this.filePath)) {
               const existing = fs.readFileSync(this.filePath, 'utf-8');
               if (existing && existing.trim().length > 2) {
-                let isValid = false;
-                try {
-                  const parsed = JSON.parse(existing);
-                  isValid = parsed && typeof parsed === 'object' && (parsed.hosters || parsed.hosterSettings || parsed.globalSettings);
-                } catch {}
-                if (isValid) fs.writeFileSync(backupPath, existing, 'utf-8');
+                fs.writeFileSync(backupPath, existing, 'utf-8');
               }
             }
           } catch {}

lib/upload-manager.js

@@ -316,7 +316,6 @@ class UploadManager extends EventEmitter {
 
     const DEDUP_CHUNK = 200;
     for (let i = 0; i < tasks.length; i += DEDUP_CHUNK) {
-      if (signal.aborted) break;
       const end = Math.min(i + DEDUP_CHUNK, tasks.length);
       for (let j = i; j < end; j++) {
         const task = tasks[j];
@@ -335,7 +334,6 @@ class UploadManager extends EventEmitter {
     const SPAWN_CHUNK = 100;
     const promises = [];
     for (let i = 0; i < tasks.length; i += SPAWN_CHUNK) {
-      if (signal.aborted) break;
       const end = Math.min(i + SPAWN_CHUNK, tasks.length);
       for (let j = i; j < end; j++) promises.push(this._runJob(tasks[j], results, signal));
       if (end < tasks.length) await new Promise(setImmediate);

lib/vidmoly-upload.js

@@ -382,7 +382,7 @@ class VidmolyUploader {
           }
         }
 
-        if (best && bestScore > 0) {
+        if (best && (bestScore > 0 || newFiles.length === 1)) {
           return this._buildUrlsFromCode(best.file_code);
         }
       }

main.js

@@ -231,7 +231,7 @@ function rotLog(msg, ts) {
 function safeSend(channel, data) {
   if (!mainWindow || mainWindow.isDestroyed()) return false;
   try {
-    mainWindow.webContents.send(channel, data);
+    safeSend(channel, data);
     return true;
   } catch (err) {
     debugLog(`safeSend(${channel}) failed: ${err && err.message ? err.message : err}`);
@@ -1155,8 +1155,7 @@ app.on('before-quit', () => {
     if (remoteServer) { remoteServer.stop(); remoteServer = null; }
     destroyCaptureWindow();
   } catch {}
-  try { destroyDropTargetWindow(); } catch {}
-  try { if (tray && !tray.isDestroyed()) { tray.destroy(); tray = null; } } catch {}
+  destroyDropTargetWindow();
   // Flush pending log buffers synchronously so no lines are lost.
   try {
     if (_debugLogBuffer.length) {
@@ -1379,17 +1378,6 @@ ipcMain.handle('select-folder', async () => {
   });
   if (result.canceled || !result.filePaths.length) return null;
 
-  const files = [];
-  for (const folder of result.filePaths) await walkFolderAsync(folder, files);
-  return files.length > 0 ? files.map(f => f.path) : null;
-});
-
-ipcMain.handle('select-folder-with-sizes', async () => {
-  const result = await dialog.showOpenDialog(mainWindow, {
-    properties: ['openDirectory', 'multiSelections']
-  });
-  if (result.canceled || !result.filePaths.length) return null;
-
   const files = [];
   for (const folder of result.filePaths) await walkFolderAsync(folder, files);
   return files.length > 0 ? files : null;
@@ -1407,11 +1395,7 @@ async function walkFolderAsync(rootDir, outFiles) {
     for (const entry of entries) {
       const full = path.join(dir, entry.name);
       if (entry.isDirectory()) stack.push(full);
-      else if (entry.isFile()) {
-        let size = 0;
-        try { size = (await fsp.stat(full)).size; } catch {}
-        outFiles.push({ path: full, name: entry.name, size });
-      }
+      else if (entry.isFile()) outFiles.push(full);
     }
     if ((++scanned % 8) === 0) await new Promise(setImmediate);
   }
@@ -1423,18 +1407,6 @@ ipcMain.handle('resolve-folder-files', async (_event, folderPath) => {
   return files;
 });
 
-ipcMain.handle('get-file-sizes', async (_event, paths) => {
-  if (!Array.isArray(paths)) return {};
-  const fsp = fs.promises;
-  const out = {};
-  let i = 0;
-  for (const p of paths) {
-    try { out[p] = (await fsp.stat(p)).size; } catch { out[p] = 0; }
-    if ((++i % 32) === 0) await new Promise(setImmediate);
-  }
-  return out;
-});
-
 ipcMain.handle('start-upload', (_event, payload) => {
   const config = configStore.load();
   const files = payload && Array.isArray(payload.files) ? payload.files : [];

package.json

@@ -1,6 +1,6 @@
 {
   "name": "multi-hoster-uploader",
-  "version": "3.3.58",
+  "version": "3.3.53",
   "description": "Upload files to doodstream, voe, vidmoly, byse simultaneously",
   "main": "main.js",
   "scripts": {

preload.js

@@ -30,9 +30,7 @@ contextBridge.exposeInMainWorld('api', {
   // File selection
   selectFiles: () => ipcRenderer.invoke('select-files'),
   selectFolder: () => ipcRenderer.invoke('select-folder'),
-  selectFolderWithSizes: () => ipcRenderer.invoke('select-folder-with-sizes'),
   resolveFolderFiles: (folderPath) => ipcRenderer.invoke('resolve-folder-files', folderPath),
-  getFileSizes: (paths) => ipcRenderer.invoke('get-file-sizes', paths),
 
   // Upload control
   startUpload: (payload) => ipcRenderer.invoke('start-upload', payload),

renderer/app.js

@@ -99,7 +99,6 @@ async function init() {
   syncSelectedUploadHosters();
   restoreQueueStateFromConfig();
   await _autoDeduplicateFromLog();
-  _hydrateMissingJobSizes();
   renderHosterSummary();
   renderHosterModal();
   renderSettings();
@@ -696,12 +695,11 @@ async function addDroppedFiles(fileList) {
           const folderFiles = await window.api.resolveFolderFiles(filePath);
           if (folderFiles && folderFiles.length > 0) {
             for (const fp of folderFiles) {
-              const p = typeof fp === 'string' ? fp : (fp && fp.path);
-              if (!p || existingPaths.has(p)) continue;
-              const name = typeof fp === 'string' ? p.split('\\').pop().split('/').pop() : (fp.name || p.split('\\').pop().split('/').pop());
-              const size = typeof fp === 'string' ? null : (fp.size || 0);
-              newFiles.push({ path: p, name, size });
-              existingPaths.add(p);
+              if (!existingPaths.has(fp)) {
+                const name = fp.split('\\').pop().split('/').pop();
+                newFiles.push({ path: fp, name, size: null });
+                existingPaths.add(fp);
+              }
             }
             continue;
           }
@@ -732,52 +730,28 @@ async function pickFiles() {
 }
 
 async function pickFolder() {
-  const richFiles = window.api.selectFolderWithSizes ? await window.api.selectFolderWithSizes() : null;
-  if (richFiles && Array.isArray(richFiles)) { addPathsToQueue(richFiles); return; }
   const paths = await window.api.selectFolder();
   if (!paths) return;
   addPathsToQueue(paths);
 }
 
 function addPathsToQueue(paths) {
+  // Build path-Set once so dedup is O(1) per candidate instead of O(n+m).
+  // Matters when the user picks a folder with thousands of files.
   const existing = new Set();
   for (const f of selectedFiles) existing.add(f.path);
   for (const f of _pendingFiles) existing.add(f.path);
 
   const newFiles = [];
-  const pendingSizeFetch = [];
-  for (const entry of paths) {
-    const p = typeof entry === 'string' ? entry : (entry && entry.path);
-    if (!p || existing.has(p)) continue;
+  for (const p of paths) {
+    if (existing.has(p)) continue;
     existing.add(p);
-    const name = typeof entry === 'string' ? p.split('\\').pop().split('/').pop() : (entry.name || p.split('\\').pop().split('/').pop());
-    const size = typeof entry === 'string' ? null : (entry.size || 0);
-    newFiles.push({ path: p, name, size });
-    if (size === null || size === undefined || size === 0) pendingSizeFetch.push(p);
+    const name = p.split('\\').pop().split('/').pop();
+    newFiles.push({ path: p, name, size: null });
   }
   if (newFiles.length > 0) {
     _pendingFiles.push(...newFiles);
     openHosterModal();
-    if (pendingSizeFetch.length > 0 && window.api.getFileSizes) {
-      window.api.getFileSizes(pendingSizeFetch).then((sizeMap) => {
-        if (!sizeMap || typeof sizeMap !== 'object') return;
-        let changed = false;
-        for (const f of _pendingFiles) {
-          if (sizeMap[f.path] && (!f.size || f.size === 0)) { f.size = sizeMap[f.path]; changed = true; }
-        }
-        for (const f of selectedFiles) {
-          if (sizeMap[f.path] && (!f.size || f.size === 0)) { f.size = sizeMap[f.path]; changed = true; }
-        }
-        for (const j of queueJobs) {
-          if (sizeMap[j.file] && (!j.bytesTotal || j.bytesTotal === 0)) { j.bytesTotal = sizeMap[j.file]; changed = true; }
-        }
-        if (changed) {
-          _queueStatsCache = null;
-          if (typeof renderQueueTable === 'function') renderQueueTable();
-          if (typeof updateStatusBar === 'function') updateStatusBar();
-        }
-      }).catch(() => {});
-    }
   }
 }
 
@@ -998,54 +972,12 @@ function scheduleStatusChangeUpdate() {
   });
 }
 
-function _hydrateMissingJobSizes(jobsLike) {
-  if (!window.api || !window.api.getFileSizes) return;
-  const paths = [];
-  const seen = new Set();
-  const source = Array.isArray(jobsLike) ? jobsLike : queueJobs;
-  for (const j of source) {
-    if (!j || !j.file) continue;
-    if (j.bytesTotal && j.bytesTotal > 0) continue;
-    if (seen.has(j.file)) continue;
-    seen.add(j.file);
-    paths.push(j.file);
-  }
-  if (paths.length === 0) return;
-  window.api.getFileSizes(paths).then((sizeMap) => {
-    if (!sizeMap || typeof sizeMap !== 'object') return;
-    let changed = false;
-    for (const j of queueJobs) {
-      if (sizeMap[j.file] && (!j.bytesTotal || j.bytesTotal === 0)) {
-        j.bytesTotal = sizeMap[j.file];
-        changed = true;
-      }
-    }
-    for (const f of selectedFiles) {
-      if (sizeMap[f.path] && (!f.size || f.size === 0)) f.size = sizeMap[f.path];
-    }
-    if (changed) {
-      _queueStatsCache = null;
-      if (typeof renderQueueTable === 'function') renderQueueTable();
-      if (typeof updateStatusBar === 'function') updateStatusBar();
-    }
-  }).catch(() => {});
-}
-
-function _formatUploadedSize(job) {
-  const bt = job.bytesTotal || 0;
-  const bu = job.bytesUploaded || 0;
-  const s = job.status;
-  if (s === 'preview') return bt > 0 ? formatSize(bt) : '...';
-  if (s === 'queued' || s === 'getting-server' || s === 'retrying') {
-    return bt > 0 ? `${formatSize(bu)} / ${formatSize(bt)}` : '...';
-  }
-  return `${formatSize(bu)} / ${formatSize(bt)}`;
-}
-
 function buildRowHtml(job) {
   const statusClass = `status-${job.status}`;
   const rowClass = `queue-row ${statusClass}${selectedJobIds.has(job.id) ? ' selected' : ''}`;
-  const uploadedSize = _formatUploadedSize(job);
+  const uploadedSize = job.status === 'preview'
+    ? (job.bytesTotal > 0 ? formatSize(job.bytesTotal) : '...')
+    : `${formatSize(job.bytesUploaded)} / ${formatSize(job.bytesTotal)}`;
   const statusText = getStatusText(job);
   const elapsed = formatTime(job.elapsed);
   const remaining = formatTime(job.remaining);
@@ -1075,7 +1007,9 @@ function buildRowHtml(job) {
 // In-place update of a single row's cells (avoids full innerHTML rebuild)
 function _updateRowInPlace(tr, job) {
   const statusClass = `status-${job.status}`;
-  const uploadedSize = _formatUploadedSize(job);
+  const uploadedSize = job.status === 'preview'
+    ? (job.bytesTotal > 0 ? formatSize(job.bytesTotal) : '...')
+    : `${formatSize(job.bytesUploaded)} / ${formatSize(job.bytesTotal)}`;
   const statusText = getStatusText(job);
   const elapsed = formatTime(job.elapsed);
   const remaining = formatTime(job.remaining);
@@ -1769,7 +1703,6 @@ async function startUpload() {
   if (uploading) return;
   uploading = true; // set immediately to prevent double-click race
   updateQueueActionButtons();
-  _hydrateMissingJobSizes();
 
   const hosters = getSelectedHosters();
   if (queueJobs.length === 0 && selectedFiles.length > 0) {
@@ -1842,13 +1775,10 @@ function _markSkippedJobs(result) {
 
 async function startSelectedUpload() {
   if (uploading) {
-    _hydrateMissingJobSizes();
-    const addable = queueJobs.filter(j => selectedJobIds.has(j.id) && isStartableQueueStatus(j.status));
-    if (addable.length === 0) {
-      if (selectedJobIds.size > 0) showCopyToast('Keine startbaren Jobs ausgewählt (alle laufen schon oder sind fertig).');
-      return;
-    }
-    {
+    // Batch already running — add selected jobs (queued/error/aborted/skipped) to running batch
+    // Upload-manager has duplicate protection (skips jobs already tracked)
+    const addable = queueJobs.filter(j => selectedJobIds.has(j.id) && ['queued', 'error', 'aborted', 'skipped'].includes(j.status));
+    if (addable.length > 0) {
       addable.forEach(j => {
         j.status = 'queued'; j.error = null; j.result = null;
         j.bytesUploaded = 0; j.speedKbs = 0; j.progress = 0; j.uploadId = null;
@@ -1893,6 +1823,7 @@ async function startSelectedUpload() {
       }
       return;
     }
+    return;
   }
   uploading = true; // set immediately to prevent double-click race
   updateQueueActionButtons();
@@ -2152,6 +2083,7 @@ function handleBatchDone(summary) {
 
   lastUploadStats = { state: 'idle', globalSpeedKbs: 0, totalBytes: lastUploadStats.totalBytes, elapsed: lastUploadStats.elapsed, activeJobs: 0 };
   updateStatusBar();
+  _maybeShowBatchSummary(summary);
   _refreshSessionFailedSnapshot();
 }
 
@@ -2568,7 +2500,7 @@ function _computeQueueStats() {
   }
 
   _queueStatsCache = { total, remaining, inProgress, done, errors, bytesRemaining, totalSize, remainingSize, inProgressBytes };
-  (typeof queueMicrotask === 'function' ? queueMicrotask : (fn) => Promise.resolve().then(fn))(() => { _queueStatsCache = null; });
+  queueMicrotask(() => { _queueStatsCache = null; });
   return _queueStatsCache;
 }
 
@@ -3759,10 +3691,7 @@ async function deleteAccount(accountId) {
   // Fire-and-forget the persist. The earlier `await getConfig()` round-trip
   // was redundant (we already have the truth in memory) and was the main
   // source of perceived lag on add/delete.
-  window.api.saveConfig({ hosters: config.hosters }).catch((err) => {
-    if (window.api && window.api.debugLog) window.api.debugLog(`deleteAccount saveConfig failed: ${err && err.message ? err.message : err}`);
-    showCopyToast('Account-Löschung konnte nicht persistiert werden — bitte erneut versuchen.');
-  });
+  window.api.saveConfig({ hosters: config.hosters }).catch(() => {});
 }
 
 function readAccountCredsFromModal(authType) {
@@ -3968,15 +3897,6 @@ async function _commitAccount(ctx, creds, validatedStatus, validatedMessage) {
     const idx = config.hosters[ctx.hosterName].findIndex(a => a.id === accountId);
     if (idx >= 0) {
       config.hosters[ctx.hosterName][idx] = { ...config.hosters[ctx.hosterName][idx], ...creds };
-    } else {
-      _accountModalBusy = false;
-      const _sb = document.getElementById('saveAccountBtn'); if (_sb) _sb.disabled = false;
-      const _st = document.getElementById('accountModalStatus');
-      if (_st) {
-        _st.textContent = 'Account nicht mehr in der Config — wurde extern gelöscht. Modal schließen und neu anlegen.';
-        _st.className = 'account-modal-status error';
-      }
-      return;
     }
   } else {
     accountId = `${ctx.hosterName}-${Date.now()}-${Math.random().toString(36).slice(2, 6)}`;
@@ -4924,15 +4844,4 @@ function updateStatsPanel() {
 }
 
 // --- Start ---
-init().catch((err) => {
-  try {
-    if (window.api && window.api.debugLog) window.api.debugLog(`init failed: ${err && err.stack ? err.stack : err}`);
-    const root = document.getElementById('app') || document.body;
-    if (root) {
-      const banner = document.createElement('div');
-      banner.style.cssText = 'position:fixed;top:0;left:0;right:0;background:#5a1e1e;color:#fff;padding:8px;z-index:99999;font-family:sans-serif;font-size:13px';
-      banner.textContent = 'Initialisierung fehlgeschlagen: ' + (err && err.message ? err.message : err) + ' — bitte Diagnose-Paket exportieren oder Programm neu starten.';
-      root.appendChild(banner);
-    }
-  } catch {}
-});
+init();

renderer/index.html

@@ -342,6 +342,22 @@
     </div>
   </div>
 
+  <div class="modal" id="batchSummaryModal" style="display:none">
+    <div class="modal-content" style="max-width:680px">
+      <div class="modal-header">
+        <h2>Batch-Zusammenfassung</h2>
+        <button class="icon-btn" id="batchSummaryClose" aria-label="Schließen">&times;</button>
+      </div>
+      <div class="modal-body">
+        <div id="batchSummaryList"></div>
+      </div>
+      <div class="modal-footer">
+        <button class="btn btn-secondary" id="batchSummaryRetryTransient">Transiente erneut hochladen</button>
+        <button class="btn btn-primary" id="batchSummaryRetryAll">Alle Fehler erneut versuchen</button>
+      </div>
+    </div>
+  </div>
+
   <script src="../lib/queue-prune.js"></script>
   <script src="../lib/queue-dedup.js"></script>
   <script src="../lib/log-mode.js"></script>