@balaji003/lantransfer 1.0.6 → 1.0.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@balaji003/lantransfer",
-  "version": "1.0.6",
+  "version": "1.0.8",
   "description": "LAN File Transfer — peer-to-peer file sharing over local network. Zero dependencies.",
   "main": "server.js",
   "bin": {

package/public/index.html

@@ -158,6 +158,34 @@
     }
     .file-item-remove:hover { color: var(--red); }
 
+    .transfer-actions {
+      display: flex; align-items: center; gap: 10px;
+      margin-top: 8px; font-size: 12px;
+    }
+    .transfer-size { color: var(--text-dim); }
+    .btn-open-folder {
+      background: none; border: 1px solid var(--border); color: var(--accent);
+      padding: 3px 10px; border-radius: 4px; cursor: pointer;
+      font-size: 12px; transition: all 0.12s;
+    }
+    .btn-open-folder:hover { background: var(--surface-hover); border-color: var(--accent); }
+
+    .download-dir {
+      display: flex; align-items: center; gap: 8px; padding: 8px 24px;
+      background: var(--surface); border-bottom: 1px solid var(--border);
+      font-size: 12px; color: var(--text-dim); flex-shrink: 0;
+    }
+    .download-dir-path {
+      color: var(--text); font-weight: 500; overflow: hidden;
+      text-overflow: ellipsis; white-space: nowrap; max-width: 400px;
+    }
+    .btn-change-dir {
+      background: none; border: 1px solid var(--border); color: var(--accent);
+      padding: 2px 8px; border-radius: 4px; cursor: pointer;
+      font-size: 11px; transition: all 0.12s; white-space: nowrap;
+    }
+    .btn-change-dir:hover { background: var(--surface-hover); border-color: var(--accent); }
+
     /* ── Transfers ── */
     .transfers-section { flex: 1; overflow-y: auto; padding: 16px 24px; }
     .transfer {
@@ -256,6 +284,13 @@
     </div>
   </div>
 
+  <!-- Download location -->
+  <div class="download-dir">
+    <span>Save to:</span>
+    <span class="download-dir-path" id="download-dir" title=""></span>
+    <button class="btn-change-dir" onclick="api('set-download-dir')">Change</button>
+  </div>
+
   <!-- Main -->
   <div class="main">
     <!-- Sidebar: peers -->
@@ -312,6 +347,8 @@
     evtSource = new EventSource('/events');
     evtSource.onmessage = e => {
       state = JSON.parse(e.data);
+      console.log('[sse] State update — peers:', state.peers?.length, 'files:', state.sharedFiles?.length,
+        'transfers:', state.transfers?.length, 'incoming:', !!state.incomingRequest);
       // Auto-select the latest file if none selected
       if (!selectedFileId && state.sharedFiles && state.sharedFiles.length > 0) {
         selectedFileId = state.sharedFiles[state.sharedFiles.length - 1].id;
@@ -319,9 +356,11 @@
       render();
     };
     evtSource.onopen = () => {
+      console.log('[sse] Connected');
       document.getElementById('conn-status').textContent = '';
     };
-    evtSource.onerror = () => {
+    evtSource.onerror = (e) => {
+      console.error('[sse] Error/disconnected', e);
       document.getElementById('conn-status').textContent = 'Reconnecting...';
     };
   }
@@ -331,10 +370,16 @@
   // API
   // ──────────────────────────────────────────────────────────────────────────
   function api(endpoint, data) {
+    console.log('[api]', endpoint, data || '');
     return fetch('/api/' + endpoint, {
       method: 'POST',
       headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify(data || {}),
+    }).then(r => {
+      console.log('[api]', endpoint, 'response:', r.status);
+      return r;
+    }).catch(err => {
+      console.error('[api]', endpoint, 'error:', err);
     });
   }
 
@@ -353,13 +398,9 @@
     if (selectedFileId === id) selectedFileId = null;
   }
 
-  async function doBrowse() {
-    if (browsing) return;
-    browsing = true;
-    document.getElementById('browse-btn').textContent = 'Opening...';
-    await api('browse');
-    browsing = false;
-    document.getElementById('browse-btn').textContent = 'Browse';
+  function doBrowse() {
+    console.log('[browse] Clicked');
+    api('browse');
   }
 
   function doSend() {
@@ -368,8 +409,14 @@
     }
   }
 
-  function respondTransfer(requestId, accepted) {
-    api('respond', { requestId: requestId, accepted: accepted });
+  function respondTransfer(accepted) {
+    console.log('[respondTransfer] accepted:', accepted);
+    api('respond', { accepted: accepted });
+  }
+
+  function openFolder(transferId) {
+    console.log('[openFolder] transferId:', transferId);
+    api('open-folder', { transferId: transferId });
   }
 
   function doShutdown() {
@@ -403,6 +450,11 @@
     }
     document.getElementById('local-ip').textContent = state.localIP || '';
 
+    // Download dir
+    const dirEl = document.getElementById('download-dir');
+    dirEl.textContent = state.downloadDir || '';
+    dirEl.title = state.downloadDir || '';
+
     // Toggle
     const toggle = document.getElementById('toggle');
     const toggleText = document.getElementById('toggle-text');
@@ -487,8 +539,8 @@
     // Modal
     const overlay = document.getElementById('modal-overlay');
     const modal = document.getElementById('modal');
-    if (state.incomingRequests && state.incomingRequests.length > 0) {
-      const req = state.incomingRequests[0];
+    if (state.incomingRequest) {
+      const req = state.incomingRequest;
       overlay.classList.add('show');
       modal.innerHTML =
         '<div class="modal-title">Incoming File</div>'
@@ -496,8 +548,8 @@
         + '<div class="modal-file">' + esc(req.filename) + '</div>'
         + '<div class="modal-size">' + formatSize(req.size) + '</div>'
         + '<div class="modal-buttons">'
-        + '<button class="btn-accept" onclick="respondTransfer(\'' + esc(req.id) + '\', true)">Accept</button>'
-        + '<button class="btn-reject" onclick="respondTransfer(\'' + esc(req.id) + '\', false)">Reject</button>'
+        + '<button class="btn-accept" onclick="respondTransfer(true)">Accept</button>'
+        + '<button class="btn-reject" onclick="respondTransfer(false)">Reject</button>'
         + '</div>';
     } else {
       overlay.classList.remove('show');
@@ -525,6 +577,13 @@
         + '<span>' + formatSize(t.size) + '</span>'
         + '<span>' + formatSpeed(t.speed) + '</span>'
         + '</div>';
+    } else if (t.status === 'completed' || t.status === 'failed') {
+      extra = '<div class="transfer-actions">'
+        + '<span class="transfer-size">' + formatSize(t.size) + '</span>';
+      if (t.status === 'completed' && t.direction === 'receive' && t.savePath) {
+        extra += '<button class="btn-open-folder" onclick="openFolder(\'' + esc(t.id) + '\')">Open Folder</button>';
+      }
+      extra += '</div>';
     }
 
     return '<div class="transfer">'

package/server.js

@@ -58,10 +58,11 @@ const config = loadConfig();
 // ──────────────────────────────────────────────────────────────────────────────
 let isDiscoverable = true;
 let deviceName = config.deviceName || os.hostname();
+let downloadDir = config.downloadDir || path.join(os.homedir(), 'Downloads');
 const sharedFiles = [];               // [ { id, path, name, size } ]
 const peers = new Map();              // id → { device_name, ip, tcp_port, last_seen }
 const transfers = [];                 // [ TransferEntry ]
-const pendingRequests = new Map();    // id → { filename, size, senderName, resolve }
+let pendingRequest = null;            // { filename, size, senderName, resolve } or null
 let idCounter = 0;
 let browseInProgress = false;
 
@@ -99,6 +100,7 @@ function serializeState() {
   return JSON.stringify({
     isDiscoverable,
     deviceName,
+    downloadDir,
     localIP: getLocalIP(),
     peers: [...peers.entries()].map(([id, p]) => ({
       id, name: p.device_name, ip: p.ip,
@@ -110,10 +112,11 @@ function serializeState() {
       direction: t.direction, peerName: t.peerName,
       status: t.status, progress: t.progress,
       speed: t.speed, error: t.error,
+      savePath: t.savePath || null,
     })),
-    incomingRequests: [...pendingRequests.entries()].map(([id, r]) => ({
-      id, filename: r.filename, size: r.size, senderName: r.senderName,
-    })),
+    incomingRequest: pendingRequest
+      ? { filename: pendingRequest.filename, size: pendingRequest.size, senderName: pendingRequest.senderName }
+      : null,
   });
 }
 
@@ -177,6 +180,8 @@ function readExact(socket, n) {
 const htmlPath = path.join(__dirname, 'public', 'index.html');
 
 const server = http.createServer(async (req, res) => {
+  console.log(`  [http] ${req.method} ${req.url}`);
+
   // ── Serve the web UI ──
   if (req.method === 'GET' && (req.url === '/' || req.url === '/index.html')) {
     fs.readFile(htmlPath, (err, data) => {
@@ -214,7 +219,11 @@ const server = http.createServer(async (req, res) => {
       'Connection': 'keep-alive',
     });
     sseClients.add(res);
-    req.on('close', () => sseClients.delete(res));
+    console.log(`  [sse] Client connected (total: ${sseClients.size})`);
+    req.on('close', () => {
+      sseClients.delete(res);
+      console.log(`  [sse] Client disconnected (total: ${sseClients.size})`);
+    });
     res.write(`data: ${serializeState()}\n\n`);
     return;
   }
@@ -230,32 +239,40 @@ const server = http.createServer(async (req, res) => {
       console.log(`  [toggle] Discoverable: ${isDiscoverable}`);
     }
     else if (req.url === '/api/browse') {
+      // Respond immediately so we don't block the HTTP connection
+      res.writeHead(200, { 'Content-Type': 'text/plain' });
+      res.end('ok');
       if (browseInProgress) {
         console.log('  [browse] Dialog already open, ignoring');
       } else {
         browseInProgress = true;
         console.log('  [browse] Opening file dialog...');
-        const fp = await openFileDialog();
-        browseInProgress = false;
-        if (fp) {
-          try {
+        try {
+          const fp = await openFileDialog();
+          browseInProgress = false;
+          if (fp) {
             const s = fs.statSync(fp);
             const file = { id: String(idCounter++), path: fp, name: path.basename(fp), size: s.size };
             sharedFiles.push(file);
             console.log(`  [browse] Added: ${file.name} (${formatSize(file.size)})`);
-          } catch (e) {
-            console.error(`  [browse] Error reading file: ${e.message}`);
+          } else {
+            console.log('  [browse] Cancelled by user');
           }
-        } else {
-          console.log('  [browse] Cancelled');
+        } catch (e) {
+          browseInProgress = false;
+          console.error(`  [browse] Error: ${e.message}`);
         }
+        broadcast();
       }
+      return; // already responded
     }
     else if (req.url === '/api/remove-file') {
       const idx = sharedFiles.findIndex(f => f.id === data.fileId);
       if (idx !== -1) {
         console.log(`  [files] Removed: ${sharedFiles[idx].name}`);
         sharedFiles.splice(idx, 1);
+      } else {
+        console.log(`  [files] Remove failed — fileId not found: ${data.fileId}`);
       }
     }
     else if (req.url === '/api/send') {
@@ -263,9 +280,9 @@ const server = http.createServer(async (req, res) => {
       const file = sharedFiles.find(f => f.id === data.fileId);
       if (peer && file) {
         console.log(`  [send] ${file.name} -> ${peer.device_name} (${peer.ip})`);
-        sendFile(peer, { ...file });
+        sendFile(peer, { ...file }).catch(e => console.error(`  [send] Unhandled: ${e.message}`));
       } else {
-        console.log(`  [send] Failed — peer: ${!!peer}, file: ${!!file}`);
+        console.log(`  [send] Failed — peer: ${!!peer} (${data.peerId}), file: ${!!file} (${data.fileId})`);
       }
     }
     else if (req.url === '/api/rename') {
@@ -275,14 +292,62 @@ const server = http.createServer(async (req, res) => {
         config.deviceName = newName;
         saveConfig(config);
         console.log(`  [rename] Device name set to: ${deviceName}`);
+      } else {
+        console.log(`  [rename] Invalid name: "${data.name}"`);
       }
     }
+    else if (req.url === '/api/set-download-dir') {
+      // Respond immediately so we don't block the HTTP connection
+      res.writeHead(200, { 'Content-Type': 'text/plain' });
+      res.end('ok');
+      console.log('  [config] Opening folder dialog...');
+      try {
+        const dir = await openFolderDialog();
+        if (dir) {
+          downloadDir = dir;
+          config.downloadDir = dir;
+          saveConfig(config);
+          console.log(`  [config] Download dir set to: ${downloadDir}`);
+        } else {
+          console.log('  [config] Folder dialog cancelled');
+        }
+      } catch (e) {
+        console.error(`  [config] Folder dialog error: ${e.message}`);
+      }
+      broadcast();
+      return; // already responded
+    }
     else if (req.url === '/api/respond') {
-      const pending = pendingRequests.get(data.requestId);
-      if (pending) {
-        pending.resolve(!!data.accepted);
-        pendingRequests.delete(data.requestId);
-        console.log(`  [respond] ${data.accepted ? 'Accepted' : 'Rejected'} transfer ${data.requestId}`);
+      console.log(`  [respond] Received: accepted=${data.accepted}`);
+      if (pendingRequest) {
+        pendingRequest.resolve(!!data.accepted);
+        console.log(`  [respond] ${data.accepted ? 'Accepted' : 'Rejected'}: ${pendingRequest.filename}`);
+        pendingRequest = null;
+      } else {
+        console.log(`  [respond] WARNING: No pending request`);
+      }
+    }
+    else if (req.url === '/api/open-folder') {
+      const transferId = data.transferId;
+      const t = transfers.find(tr => tr.id === transferId);
+      if (t && t.savePath) {
+        console.log(`  [open] Opening folder for: ${t.savePath}`);
+        if (fs.existsSync(t.savePath)) {
+          if (process.platform === 'win32') exec(`explorer /select,"${t.savePath}"`);
+          else if (process.platform === 'darwin') exec(`open -R "${t.savePath}"`);
+          else exec(`xdg-open "${path.dirname(t.savePath)}"`);
+        } else {
+          console.log(`  [open] File no longer exists: ${t.savePath}`);
+          // Open the directory instead
+          const dir = path.dirname(t.savePath);
+          if (fs.existsSync(dir)) {
+            if (process.platform === 'win32') exec(`explorer "${dir}"`);
+            else if (process.platform === 'darwin') exec(`open "${dir}"`);
+            else exec(`xdg-open "${dir}"`);
+          }
+        }
+      } else {
+        console.log(`  [open] Failed — transferId: ${transferId}, found: ${!!t}, savePath: ${t ? t.savePath : 'N/A'}`);
       }
     }
     else if (req.url === '/api/shutdown') {
@@ -292,6 +357,9 @@ const server = http.createServer(async (req, res) => {
       setTimeout(() => process.exit(0), 200);
       return;
     }
+    else {
+      console.log(`  [http] Unknown POST endpoint: ${req.url}`);
+    }
 
     res.writeHead(200, { 'Content-Type': 'text/plain' });
     res.end('ok');
@@ -322,6 +390,22 @@ function openFileDialog() {
   });
 }
 
+function openFolderDialog() {
+  return new Promise(resolve => {
+    let cmd;
+    if (process.platform === 'win32') {
+      cmd = 'powershell -sta -command "Add-Type -AssemblyName System.Windows.Forms; $f = New-Object System.Windows.Forms.FolderBrowserDialog; $f.Description = \'Select download folder\'; if($f.ShowDialog() -eq \'OK\'){Write-Output $f.SelectedPath}"';
+    } else if (process.platform === 'darwin') {
+      cmd = 'osascript -e \'POSIX path of (choose folder with prompt "Select download folder")\'';
+    } else {
+      cmd = 'zenity --file-selection --directory --title="Select download folder" 2>/dev/null || kdialog --getexistingdirectory . 2>/dev/null';
+    }
+    exec(cmd, { encoding: 'utf-8', windowsHide: true, timeout: 120_000 }, (err, stdout) => {
+      resolve(err ? null : (stdout.trim() || null));
+    });
+  });
+}
+
 // ──────────────────────────────────────────────────────────────────────────────
 // UDP discovery
 // ──────────────────────────────────────────────────────────────────────────────
@@ -389,33 +473,41 @@ tcpServer.on('error', err => {
 tcpServer.listen(TRANSFER_PORT);
 
 async function handleIncoming(socket) {
+  const remoteAddr = `${socket.remoteAddress}:${socket.remotePort}`;
+  console.log(`  [recv] TCP connection from ${remoteAddr}`);
+
   // Read header length (8 bytes, little-endian u64)
   const lenBuf = await readExact(socket, 8);
   const headerLen = Number(lenBuf.readBigUInt64LE(0));
+  console.log(`  [recv] Header length: ${headerLen} bytes`);
 
   // Read header JSON
   const headerBuf = await readExact(socket, headerLen);
   const header = JSON.parse(headerBuf.toString('utf-8'));
   const { filename, size, sender_name } = header;
 
-  console.log(`  Incoming: ${filename} (${formatSize(size)}) from ${sender_name}`);
+  console.log(`  [recv] Incoming: "${filename}" (${formatSize(size)}) from ${sender_name}`);
 
   // Prompt user for accept / reject
-  const reqId = String(idCounter++);
+  console.log(`  [recv] Waiting for user decision...`);
+
   const accepted = await new Promise(resolve => {
-    pendingRequests.set(reqId, { filename, size, senderName: sender_name, resolve });
+    pendingRequest = { filename, size, senderName: sender_name, resolve };
     broadcast();
 
     // Auto-reject if sender disconnects while waiting
     socket.once('close', () => {
-      if (pendingRequests.has(reqId)) {
-        pendingRequests.delete(reqId);
+      if (pendingRequest && pendingRequest.resolve === resolve) {
+        console.log(`  [recv] Sender disconnected while waiting, auto-rejecting`);
+        pendingRequest = null;
         resolve(false);
         broadcast();
       }
     });
   });
 
+  console.log(`  [recv] User decision: ${accepted ? 'ACCEPTED' : 'REJECTED'}`);
+
   // Send decision byte
   socket.write(Buffer.from([accepted ? 1 : 0]));
   if (!accepted) { socket.end(); return; }
@@ -430,8 +522,8 @@ async function handleIncoming(socket) {
   transfers.push(t);
   broadcast();
 
-  // Unique save path in Downloads
-  const dl = path.join(os.homedir(), 'Downloads');
+  // Unique save path in download directory
+  const dl = downloadDir;
   if (!fs.existsSync(dl)) fs.mkdirSync(dl, { recursive: true });
   let savePath = path.join(dl, filename);
   let counter = 1;
@@ -441,6 +533,9 @@ async function handleIncoming(socket) {
     savePath = path.join(dl, `${base} (${counter++})${ext}`);
   }
 
+  t.savePath = savePath;
+  console.log(`  [recv] Saving to: ${savePath}`);
+
   const ws = fs.createWriteStream(savePath);
   let received = 0;
   let lastBc = 0;
@@ -467,7 +562,7 @@ async function handleIncoming(socket) {
       t.status = 'completed';
       t.progress = 100;
       broadcast();
-      console.log(`  Saved: ${savePath}`);
+      console.log(`  [recv] Complete: ${savePath} (${formatSize(received)})`);
       resolve();
     });
     socket.on('error', err => {
@@ -475,6 +570,7 @@ async function handleIncoming(socket) {
       t.status = 'failed';
       t.error = err.message;
       broadcast();
+      console.error(`  [recv] Socket error: ${err.message}`);
       reject(err);
     });
   });