fix(cli+sdk): minor fixes and typo

Author: LevinCeglie

cli/kleinkram/api/routes.py

@@ -527,7 +527,8 @@ def _update_trigger(
     type_: Optional[TriggerType] = None,
     config: Optional[TriggerConfig] = None,
 ) -> None:
-    if not any([name, description, template_uuid, mission_uuid, type_, config]):
+
+    if all(v is None for v in [name, description, template_uuid, mission_uuid, type_, config]):
         raise ValueError("at least one field must be updated")
 
     body = {}

cli/kleinkram/cli/_triggers.py

@@ -159,7 +159,7 @@ def update_trigger_cli(
         typer.Option(
             "--file-patterns",
             help="File patterns, provide multiples via '--file-patterns *.bag "
-            "--file-patters date.bag' (only for FILE triggers)",
+            "--file-patterns date.bag' (only for FILE triggers)",
         ),
     ] = None,
     file_events: Annotated[
@@ -198,6 +198,8 @@ def update_trigger_cli(
                 updated_fields["config"] = TimeConfig(cron=cron_expression)
             case TriggerType.WEBHOOK:
                 updated_fields["config"] = WebhookConfig()
+    elif any(v is not None for v in [file_patterns, file_events, cron_expression]):
+        raise typer.BadParameter("Trigger type must be specified when updating config fields. Please provide --type option.")
 
     if not updated_fields:
         typer.secho("No fields to update. Please provide at least one field to update.", fg=typer.colors.GREEN)

cli/kleinkram/printing.py

@@ -28,6 +28,7 @@
 
 import kleinkram
 from kleinkram.api.client import AuthenticatedClient
+from kleinkram.config import get_config
 from kleinkram.config import get_shared_state
 from kleinkram.core import FileVerificationStatus
 from kleinkram.models import ActionTemplate
@@ -504,7 +505,7 @@ def trigger_info_table(trigger: ActionTrigger) -> Table:
         case TimeConfig(cron=cron):
             table.add_row("time trigger cron expression", cron)
         case WebhookConfig():
-            curr_config = kleinkram.config.get_config()
+            curr_config = get_config()
             url = curr_config.endpoints[curr_config.selected_endpoint].api + f"/hooks/actions/{trigger.uuid}"
             table.add_row("webhook trigger url", url)
     return table