fix(cli_args): format cli deploy update clean args (#307)

Author: floritange

veadk/cli/cli_clean.py

@@ -24,7 +24,7 @@
 
 @click.command()
 @click.option(
-    "--vefaas-application-name",
+    "--vefaas-app-name",
     required=True,
     help="VeFaaS application name to clean",
 )
@@ -39,7 +39,7 @@
     help="Volcengine secret key, if not set, will use the value of environment variable VOLCENGINE_SECRET_KEY",
 )
 def clean(
-    vefaas_application_name: str, volcengine_access_key: str, volcengine_secret_key: str
+    vefaas_app_name: str, volcengine_access_key: str, volcengine_secret_key: str
 ) -> None:
     """
     Clean and delete a VeFaaS application from the cloud.
@@ -49,7 +49,7 @@ def clean(
     and monitor the deletion process until completion.
 
     Args:
-        vefaas_application_name (str): The name of the VeFaaS application to delete
+        vefaas_app_name (str): The name of the VeFaaS application to delete
         volcengine_access_key (str): Volcengine access key for authentication.
             If None, will use VOLCENGINE_ACCESS_KEY environment variable
         volcengine_secret_key (str): Volcengine secret key for authentication.
@@ -63,24 +63,22 @@ def clean(
     if not volcengine_secret_key:
         volcengine_secret_key = getenv("VOLCENGINE_SECRET_KEY")
 
-    confirm = input(f"Confirm delete cloud app {vefaas_application_name}? (y/N): ")
+    confirm = input(f"Confirm delete cloud app {vefaas_app_name}? (y/N): ")
     if confirm.lower() != "y":
         click.echo("Delete cancelled.")
         return
     else:
         vefaas_client = VeFaaS(
             access_key=volcengine_access_key, secret_key=volcengine_secret_key
         )
-        vefaas_application_id = vefaas_client.find_app_id_by_name(
-            vefaas_application_name
-        )
+        vefaas_application_id = vefaas_client.find_app_id_by_name(vefaas_app_name)
         vefaas_client.delete(vefaas_application_id)
         click.echo(
-            f"Cloud app {vefaas_application_name} delete request has been sent to VeFaaS"
+            f"Cloud app {vefaas_app_name} delete request has been sent to VeFaaS"
         )
         while True:
             try:
-                id = vefaas_client.find_app_id_by_name(vefaas_application_name)
+                id = vefaas_client.find_app_id_by_name(vefaas_app_name)
                 if not id:
                     break
                 time.sleep(3)

veadk/cli/cli_deploy.py

@@ -22,12 +22,12 @@
 
 @click.command()
 @click.option(
-    "--access-key",
+    "--volcengine-access-key",
     default=None,
     help="Volcengine access key",
 )
 @click.option(
-    "--secret-key",
+    "--volcengine-secret-key",
     default=None,
     help="Volcengine secret key",
 )
@@ -52,8 +52,8 @@
 @click.option("--use-adk-web", is_flag=True, help="Whether to use ADK Web")
 @click.option("--path", default=".", help="Local project path")
 def deploy(
-    access_key: str,
-    secret_key: str,
+    volcengine_access_key: str,
+    volcengine_secret_key: str,
     vefaas_app_name: str,
     veapig_instance_name: str,
     veapig_service_name: str,
@@ -77,9 +77,9 @@ def deploy(
     5. Cleaning up temporary files
 
     Args:
-        access_key: Volcengine access key for API authentication. If not provided,
+        volcengine_access_key: Volcengine access key for API authentication. If not provided,
             will use VOLCENGINE_ACCESS_KEY environment variable
-        secret_key: Volcengine secret key for API authentication. If not provided,
+        volcengine_secret_key: Volcengine secret key for API authentication. If not provided,
             will use VOLCENGINE_SECRET_KEY environment variable
         vefaas_app_name: Name of the target Volcengine FaaS application where the
             project will be deployed
@@ -111,10 +111,10 @@ def deploy(
 
     logger = get_logger(__name__)
 
-    if not access_key:
-        access_key = getenv("VOLCENGINE_ACCESS_KEY")
-    if not secret_key:
-        secret_key = getenv("VOLCENGINE_SECRET_KEY")
+    if not volcengine_access_key:
+        volcengine_access_key = getenv("VOLCENGINE_ACCESS_KEY")
+    if not volcengine_secret_key:
+        volcengine_secret_key = getenv("VOLCENGINE_SECRET_KEY")
 
     user_proj_abs_path = Path(path).resolve()
     template_dir_path = Path(vefaas.__file__).parent / "template"

veadk/cli/cli_update.py

@@ -33,7 +33,7 @@
     help="Volcengine secret key for authentication. Defaults to VOLCENGINE_SECRET_KEY environment variable.",
 )
 @click.option(
-    "--application-name",
+    "--vefaas-app-name",
     required=True,
     help="Name of the cloud application to update.",
 )
@@ -45,7 +45,7 @@
 def update(
     volcengine_access_key: str,
     volcengine_secret_key: str,
-    application_name: str,
+    vefaas_app_name: str,
     path: str,
 ) -> None:
     """Update function code of a deployed cloud application on Volcengine FaaS.
@@ -65,7 +65,7 @@ def update(
             If not provided, uses VOLCENGINE_ACCESS_KEY environment variable.
         volcengine_secret_key: Volcengine platform secret key for authentication.
             If not provided, uses VOLCENGINE_SECRET_KEY environment variable.
-        application_name: Name of the existing cloud application to update.
+        vefaas_app_name: Name of the existing cloud application to update.
         path: Local directory path containing the updated agent project.
             Defaults to current directory if not specified.
 
@@ -93,11 +93,11 @@ def update(
     try:
         # Update function code
         updated_app = engine.update_function_code(
-            application_name=application_name,
+            application_name=vefaas_app_name,
             path=path,
         )
 
-        logger.info(f"Successfully updated cloud application '{application_name}'")
+        logger.info(f"Successfully updated cloud application '{vefaas_app_name}'")
         logger.info(f"Endpoint: {updated_app.vefaas_endpoint}")
         logger.info(f"Application ID: {updated_app.vefaas_application_id}")