PR review: dstack run|fleet get --json

peterschmidt85 · peterschmidt85 · commit 1148df90c590 · 2026-01-08T09:42:24.000+01:00
Better handling edge case (empty ID)
diff --git a/src/dstack/_internal/cli/commands/fleet.py b/src/dstack/_internal/cli/commands/fleet.py
@@ -141,14 +141,14 @@ def _delete(self, args: argparse.Namespace):
     def _get(self, args: argparse.Namespace):
         # TODO: Implement non-json output format
         fleet_id = None
-        if args.id:
+        if args.id is not None:
             try:
                 fleet_id = UUID(args.id)
             except ValueError:
                 raise CLIError(f"Invalid UUID format: {args.id}")
 
         try:
-            if args.id:
+            if args.id is not None:
                 fleet = self.api.client.fleets.get(
                     project_name=self.api.project, fleet_id=fleet_id
                 )
diff --git a/src/dstack/_internal/cli/commands/run.py b/src/dstack/_internal/cli/commands/run.py
@@ -51,14 +51,14 @@ def _command(self, args: argparse.Namespace):
     def _get(self, args: argparse.Namespace):
         # TODO: Implement non-json output format
         run_id = None
-        if args.id:
+        if args.id is not None:
             try:
                 run_id = UUID(args.id)
             except ValueError:
                 raise CLIError(f"Invalid UUID format: {args.id}")
 
         try:
-            if args.id:
+            if args.id is not None:
                 run = self.api.client.runs.get(project_name=self.api.project, run_id=run_id)
             else:
                 run = self.api.client.runs.get(project_name=self.api.project, run_name=args.name)
