fix(commandline):

- Add strict mode definition to load option
- Give strict mode option a default
- Sanitize campaign name

Author: tcjennings

alembic/seed/seed_ci_campaign.yaml

@@ -14,7 +14,6 @@ metadata:
 # information parameters for a specific manifest type here, but the campaign
 # comes last in the configuration lookup hierarchy!
 spec:
-  # set the initial state of the campaign to "paused" so it can be step-advanced
   butlerSelector:
     instrument: lsstcam
     embargo: "false"

packages/cm-commandline/src/lsst/cmservice/commandline/arguments.py

@@ -18,9 +18,10 @@ def preprocess_campaign_name(ctx: TypedContext, campaign: str) -> str:
     """Preprocesses a campaign NAME by translating it to a campaign ID and
     storing the result in the application context
     """
-    ctx.obj.campaign_name = campaign
-    ctx.obj.campaign_id = str(uuid5(settings.default_namespace, campaign))
-    return campaign
+    sanitized_campaign_name = as_snake_case(campaign)
+    ctx.obj.campaign_name = sanitized_campaign_name
+    ctx.obj.campaign_id = str(uuid5(settings.default_namespace, sanitized_campaign_name))
+    return sanitized_campaign_name
 
 
 campaign_name = Annotated[

packages/cm-commandline/src/lsst/cmservice/commandline/loader/app.py

@@ -18,7 +18,9 @@ def load_from_yaml(
     filename: str,
     campaign: arguments.campaign_name,
     *,
-    strict: Annotated[bool, typer.Option("--strict", help="Load YAML in strict mode")],
+    strict: Annotated[
+        bool, typer.Option(help="Load YAML in strict mode (file must have only a single campaign)")
+    ] = False,
 ) -> None:
     """Load manifests from a YAML file"""
     try: