fix: reviews

Author: byawitz

templates/cli/index.js.twig

@@ -12,7 +12,7 @@ const { commandDescriptions, cliConfig } = require("./lib/parser");
 const { client } = require("./lib/commands/generic");
 const inquirer = require("inquirer");
 {% if sdk.test != "true" %}
-const { login, register, logout, whoami, migrate } = require("./lib/commands/generic");
+const { login, logout, whoami, migrate } = require("./lib/commands/generic");
 const { init } = require("./lib/commands/init");
 const { pull } = require("./lib/commands/pull");
 const { push } = require("./lib/commands/push");
@@ -62,7 +62,6 @@ program
 {% if sdk.test != "true" %}
   .addCommand(whoami)
   .addCommand(login)
-  .addCommand(register)
   .addCommand(init)
   .addCommand(pull)
   .addCommand(push)

templates/cli/lib/client.js.twig

@@ -4,7 +4,7 @@ const { fetch, FormData, Agent } = require("undici");
 const JSONbig = require("json-bigint")({ storeAsString: false });
 const {{spec.title | caseUcfirst}}Exception = require("./exception.js");
 const { globalConfig } = require("./config.js");
-const {log} = require('./parser');
+const { log } = require('./parser');
 
 class Client {
   CHUNK_SIZE = 5*1024*1024; // 5MB
@@ -146,7 +146,7 @@ class Client {
         throw new {{spec.title | caseUcfirst}}Exception(text, response.status, "", text);
       }
 
-      if(path !== '/account' && json.code === 401 && json.type ==='user_more_factors_required'){
+      if (path !== '/account' && json.code === 401 && json.type === 'user_more_factors_required') {
         log('Unusable account found, removing...')
         const current = globalConfig.getCurrentLogin();
         globalConfig.setCurrentLogin('');

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

@@ -6,42 +6,11 @@ const { globalConfig, localConfig } = require("../config");
 const { actionRunner, success, parseBool, commandDescriptions, error, parse, log, drawTable } = require("../parser");
 const ID = require("../id");
 {% if sdk.test != "true" %}
-const { questionsRegister, questionsRegisterWithEndpoint, questionsLogin, questionLoginWithEndpoint, questionsLogout, questionsListFactors, questionsMfaChallenge } = require("../questions");
+const { questionsLogin, questionLoginWithEndpoint, questionsLogout, questionsListFactors, questionsMfaChallenge } = require("../questions");
 const { accountCreate, accountUpdateMfaChallenge, accountCreateMfaChallenge, accountGet, accountCreateEmailPasswordSession, accountDeleteSession } = require("./account");
 
 const DEFAULT_ENDPOINT = 'https://cloud.appwrite.io/v1';
 
-const registerCommand = async ({ selfHosted, email, password, endpoint, name }) => {
-    let answers;
-    let configEndpoint = DEFAULT_ENDPOINT;
-
-    if (selfHosted) {
-        answers = endpoint && email && password ? { endpoint, email, password } : await inquirer.prompt(questionsRegisterWithEndpoint);
-        configEndpoint = answers.endpoint;
-    } else {
-        answers = email && password ? { email, password } : await inquirer.prompt(questionsRegister);
-    }
-
-    globalConfig.setEndpoint(configEndpoint);
-
-    let client = await sdkForConsole(false);
-
-    try {
-        await accountCreate({
-            userId: ID.unique(),
-            email: answers.email,
-            password: answers.password,
-            parseOutput: false,
-            name: answers.name,
-            sdk: client,
-        })
-
-        success();
-    } catch (e) {
-        throw e;
-    }
-
-}
 const loginCommand = async ({ selfHosted, email, password, endpoint, mfa, code }) => {
     const oldCurrent = globalConfig.getCurrentLogin();
     let answers = {};
@@ -180,15 +149,6 @@ const login = new Command("login")
     })
     .action(actionRunner(loginCommand));
 
-const register = new Command("register")
-    .description(commandDescriptions['login'])
-    .option(`-sh, --self-hosted`, `Flag for enabling custom endpoint for self hosted instances`)
-    .option(`--name [name]`, `User name`)
-    .option(`--email [email]`, `User email`)
-    .option(`--password [password]`, `User password`)
-    .option(`--endpoint [endpoint]`, `Appwrite endpoint for self hosted instances`)
-    .action(actionRunner(registerCommand));
-
 const singleLogout = async (accountId) => {
     try {
         let client = await sdkForConsole();
@@ -349,7 +309,6 @@ module.exports = {
     loginCommand,
     whoami,
     login,
-    register,
     logout,
     {% endif %}
     migrate,