Merge pull request #913 from appwrite/improve-cli-qa

Improving CLI 2.0 qa

Author: eldadfux

src/SDK/Language/CLI.php

@@ -2,6 +2,7 @@
 
 namespace Appwrite\SDK\Language;
 
+use Twig\TwigFilter;
 use Twig\TwigFunction;
 
 class CLI extends Node
@@ -364,6 +365,14 @@ public function getParamExample(array $param): string
         return $output;
     }
 
+    public function getFilters(): array
+    {
+        return array_merge(parent::getFilters(), [
+            new TwigFilter('caseKebab', function ($value) {
+                return strtolower(preg_replace('/(?<!^)([A-Z][a-z]|(?<=[a-z])[^a-z]|(?<=[A-Z])[0-9_])/', '-$1', $value));
+            })
+        ]);
+    }
     /**
      * Language specific filters.
      * @return array

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

@@ -86,11 +86,11 @@ const {{ service.name | caseLower }}{{ method.name | caseUcfirst }} = async ({
 {% endfor %}
 {% for method in service.methods %}
 {{service.name | caseLower }}
-    .command(`{{ method.name | caseCamel }}`)
+    .command(`{{ method.name | caseKebab }}`)
 {% autoescape false %}
     .description(`{{ method.description | replace({'`':'\''}) | replace({'\n':' '}) | replace({'\n \n':' '}) }}`)
 {% for parameter in method.parameters.all %}
-    .{% if parameter.required and not parameter.nullable %}requiredOption{% else %}option{% endif %}(`--{{ parameter.name | escapeKeyword }} {% if parameter.array.type|length > 0 %}[{% else %}<{% endif %}{{ parameter.name | escapeKeyword }}{% if parameter.array.type|length > 0 %}...{% endif %}{% if parameter.array.type|length > 0 %}]{% else %}>{% endif %}`, `{{ parameter.description | replace({'`':'\''}) | replace({'\n':' '}) | replace({'\n \n':' '}) }}`{% if parameter | typeName == 'boolean' %}, parseBool{% elseif parameter | typeName == 'number' %}, parseInteger{% endif %})
+    .{% if parameter.required and not parameter.nullable %}requiredOption{% else %}option{% endif %}(`--{{ parameter.name | escapeKeyword | caseKebab }} {% if parameter.array.type|length > 0 %}[{% else %}<{% endif %}{{ parameter.name | escapeKeyword | caseKebab }}{% if parameter.array.type|length > 0 %}...{% endif %}{% if parameter.array.type|length > 0 %}]{% else %}>{% endif %}`, `{{ parameter.description | replace({'`':'\''}) | replace({'\n':' '}) | replace({'\n \n':' '}) }}`{% if parameter | typeName == 'boolean' %}, parseBool{% elseif parameter | typeName == 'number' %}, parseInteger{% endif %})
 {% endfor %}
 {% if method.type == 'location' %}
     .requiredOption(`--destination <path>`, `output file path.`)

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

@@ -226,10 +226,10 @@ const client = new Command("client")
     .configureHelp({
         helpWidth: process.stdout.columns || 80
     })
-    .option("-ss, --selfSigned <value>", "Configure the CLI to use a self-signed certificate ( true or false )", parseBool)
-    .option("-e, --endpoint <endpoint>", "Set your Appwrite server endpoint")
-    .option("-p, --projectId <projectId>", "Set your Appwrite project ID")
-    .option("-k, --key <key>", "Set your Appwrite server's API key")
+    .option("-ss, --self-signed <value>", "Configure the CLI to use a self-signed certificate ( true or false )", parseBool)
+    .option("-e, --endpoint <endpoint>", "Set your {{ spec.title|caseUcfirst }} server endpoint")
+    .option("-p, --project-id <project-id>", "Set your {{ spec.title|caseUcfirst }} project ID")
+    .option("-k, --key <key>", "Set your {{ spec.title|caseUcfirst }} server's API key")
     .option("-d, --debug", "Print CLI debug information")
     .option("-r, --reset", "Reset the CLI configuration")
     .action(actionRunner(async ({ selfSigned, endpoint, projectId, key, debug, reset }, command) => {

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

@@ -334,9 +334,9 @@ const init = new Command("init")
 init
     .command("project")
     .description("Init a new {{ spec.title|caseUcfirst }} project")
-    .option("--organizationId <organizationId>", "{{ spec.title|caseUcfirst }} organization ID")
-    .option("--projectId <projectId>", "{{ spec.title|caseUcfirst }} project ID")
-    .option("--projectName <projectName>", "{{ spec.title|caseUcfirst }} project ID")
+    .option("--organization-id <organization-id>", "{{ spec.title|caseUcfirst }} organization ID")
+    .option("--project-id <project-id>", "{{ spec.title|caseUcfirst }} project ID")
+    .option("--project-name <project-name>", "{{ spec.title|caseUcfirst }} project ID")
     .action(actionRunner(initProject));
 
 init