fix(cli): extend suppress_sdk_output scope for image upload and version download

Wrap all SDK calls (including single_upload, upload_dataset, versions,
download) inside suppress_sdk_output context to prevent "loading..."
noise from corrupting --json output. Also consolidate try/except to
catch errors from SDK operations, not just initialization.

256 tests pass, all linting clean.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: yeldarby

roboflow/cli/handlers/image.py

@@ -102,28 +102,23 @@ def _handle_upload_directory(args: argparse.Namespace, api_key: str, path: str)
     import roboflow
     from roboflow.cli._output import suppress_sdk_output
 
+    retries = getattr(args, "retries", None) or getattr(args, "num_retries", 0) or 0
+
     with suppress_sdk_output(args):
         try:
             rf = roboflow.Roboflow(api_key)
             workspace = rf.workspace(args.workspace)
+            workspace.upload_dataset(
+                dataset_path=path,
+                project_name=args.project,
+                num_workers=args.concurrency,
+                batch_name=getattr(args, "batch", None),
+                num_retries=retries,
+            )
         except Exception as exc:
             output_error(args, str(exc), exit_code=3)
             return
 
-    retries = getattr(args, "retries", None) or getattr(args, "num_retries", 0) or 0
-
-    try:
-        workspace.upload_dataset(
-            dataset_path=path,
-            project_name=args.project,
-            num_workers=args.concurrency,
-            batch_name=getattr(args, "batch", None),
-            num_retries=retries,
-        )
-    except Exception as exc:
-        output_error(args, str(exc))
-        return
-
     # Count files uploaded (approximate via image extensions)
     count = 0
     image_exts = {".jpg", ".jpeg", ".png", ".bmp", ".gif", ".tiff", ".webp"}

roboflow/cli/handlers/version.py

@@ -172,25 +172,27 @@ def _download(args: argparse.Namespace) -> None:
         try:
             rf = roboflow.Roboflow()
             project = rf.workspace(w).project(p)
+
+            if not v:
+                versions = project.versions()
+                if not versions:
+                    output_error(args, f"Project {p} does not have any versions.")
+                    return
+                version_obj = versions[-1]
+            else:
+                version_obj = project.version(int(v))
+
+            version_obj.download(args.format, location=args.location, overwrite=True)
+        except SystemExit:
+            raise
         except Exception as exc:
             output_error(args, str(exc), exit_code=3)
             return
 
-    if not v:
-        versions = project.versions()
-        if not versions:
-            output_error(args, f"Project {p} does not have any versions.")
-            return
-        version = versions[-1]
-    else:
-        version = project.version(int(v))
-
-    version.download(args.format, location=args.location, overwrite=True)
-
     data = {
         "workspace": w,
         "project": p,
-        "version": int(v) if v else version.version,
+        "version": int(v) if v else version_obj.version,
         "format": args.format,
         "location": args.location or "",
     }