@balaji003/lantransfer 1.0.3 → 1.0.4

package/package.json

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

package/server.js

@@ -512,24 +512,22 @@ function checkWindowsFirewall() {
   const ruleName = 'LAN Transfer (lantransfer)';
   const nodeExe = process.execPath;
 
-  // Check if a firewall rule already exists for this app
+  // Check if a firewall rule already exists
   exec(`netsh advfirewall firewall show rule name="${ruleName}"`, { windowsHide: true }, (err, stdout) => {
     if (!err && stdout.includes(ruleName)) {
-      // Rule exists, check if it's enabled
       if (stdout.includes('Enabled:') && stdout.includes('Yes')) {
         console.log('  Firewall:  Allowed');
         return;
       }
     }
 
-    // No rule found — prompt the user to allow it
+    // No rule — prompt user via UAC elevation
     console.log('');
     console.log('  !! Firewall rule not found for LAN Transfer.');
-    console.log('  !! Device discovery requires UDP/TCP access through Windows Firewall.');
-    console.log('  !! Attempting to add firewall rule (requires Admin)...');
+    console.log('  !! Device discovery requires UDP/TCP through Windows Firewall.');
+    console.log('  !! Requesting permission (UAC prompt)...');
     console.log('');
 
-    // Try to add the rule via elevated PowerShell — this triggers a UAC prompt
     const addCmd = [
       `netsh advfirewall firewall add rule name="${ruleName}" dir=in action=allow protocol=UDP localport=${DISCOVERY_PORT} program="${nodeExe}" enable=yes`,
       `netsh advfirewall firewall add rule name="${ruleName}" dir=in action=allow protocol=TCP localport=${TRANSFER_PORT} program="${nodeExe}" enable=yes`,
@@ -539,14 +537,14 @@ function checkWindowsFirewall() {
 
     exec(psCmd, { windowsHide: false, timeout: 60_000 }, (err2) => {
       if (err2) {
-        console.log('  !! Could not add firewall rule. Discovery may not work.');
+        console.log('  !! Firewall permission denied. Discovery will NOT work.');
         console.log('  !! To fix manually:');
         console.log('  !!   1. Open Windows Security > Firewall & network protection');
         console.log('  !!   2. Click "Allow an app through firewall"');
         console.log('  !!   3. Add Node.js and allow Private + Public networks');
         console.log('');
       } else {
-        console.log('  Firewall:  Rule added successfully! Discovery should work now.');
+        console.log('  Firewall:  Rule added! Discovery should work now.');
       }
     });
   });