More PR review changes

Author: Meldiron

templates/cli/lib/commands/run.js.twig

@@ -49,14 +49,28 @@ const runFunction = async ({ port, functionId, noVariables, noReload, userId } =
         const taken = await isPortTaken(port);
 
         if(taken) {
-            log(`Port ${port} is already in use by another process.`);
-            port = null;
+            error(`Port ${port} is already in use by another process.`);
+            return;
         }
     }
 
     if(!port) {
-        const answers = await inquirer.prompt(questionsRunFunctions[1]);
-        port = answers.port;
+        let portFound = fale;
+        port = 3000;
+        while(port < 3100) {
+            const taken = await isPortTaken(port);
+            if(!taken) {
+                portFound = true;
+                break;
+            }
+
+            port++;
+        }
+
+        if(!portFound) {
+            error('Could not auto-suggest an available port. Please configure port with `appwrite run --port YOUR_PORT` command.');
+            return;
+        }
     }
 
     // Configuration: Engine

templates/cli/lib/questions.js.twig

@@ -791,50 +791,7 @@ const questionsRunFunctions = [
             })
             return choices;
         }
-    },
-    {
-        type: "input",
-        name: "port",
-        message: 'Which port would you like function to listen on?',
-        default: async () => {
-            let port = 3000;
-            while(port < 3100) {
-                const taken = await isPortTaken(port);
-                if(!taken) {
-                    return port;
-                }
-
-                port++;
-            }
-
-            return 3000;
-        },
-        validate: function(value) {
-            const done = this.async();
-
-            (async () => {
-                if (typeof value !== 'number' && isNaN(+value)) {
-                    throw Error(`Port needs to be a valid integer between 1024 and 65536.`);
-                }
-
-                value = +value;
-
-                if(value < 1024 || value > 65536) {
-                    throw Error(`Port needs to be a valid integer between 1024 and 65536.`);
-                }
-
-                const taken = await isPortTaken(value);
-
-                if(taken) {
-                    throw Error(`Port ${value} is in use by another process. Pick a different one.`);
-                }
-            })().then(() => {
-                done(null, true);
-            }).catch((err) => {
-                done(err.message);
-            });
-        },
-    },
+    }
 ];
 
 module.exports = {