fix: raise click.Abort on missing token and instance name

mdesmet · mdesmet · commit ace15212e91d · 2025-06-12T15:11:58.000+08:00
diff --git a/src/datapilot/core/knowledge/cli.py b/src/datapilot/core/knowledge/cli.py
@@ -21,7 +21,7 @@ def serve(token, instance_name, backend_url, port):
         click.echo(
             "Error: API token and instance name are required. Use --token and --instance-name options or set them in config.", err=True
         )
-        return
+        raise click.Abort
 
     # Set context data for the handler
     KnowledgeBaseHandler.token = token

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ def serve(token, instance_name, backend_url, port):`
`21`	`21`	`click.echo(`
`22`	`22`	`"Error: API token and instance name are required. Use --token and --instance-name options or set them in config.", err=True`
`23`	`23`	`)`
`24`		`- return`
	`24`	`+ raise click.Abort`
`25`	`25`
`26`	`26`	`# Set context data for the handler`
`27`	`27`	`KnowledgeBaseHandler.token = token`