Merge pull request #172 from seqeralabs/fix-overwrite-to-json

fix: handling for overwrite class without json output

seqerakit/cli.py

@@ -110,8 +110,16 @@ def __init__(self, sp, list_for_add_method):
         """
         self.sp = sp
         self.list_for_add_method = list_for_add_method
-        # Create an instance of Overwrite class
-        self.overwrite_method = overwrite.Overwrite(self.sp)
+
+        # Create a separate instance of the Seqera Platform client without JSON output
+        # This is needed because when checking if resources exist during overwrite operations,
+        # we don't want JSON output mixed with the actual resource creation output
+        sp_without_json = seqeraplatform.SeqeraPlatform(
+            cli_args=sp.cli_args,
+            dryrun=sp.dryrun,
+            json=False,
+        )
+        self.overwrite_method = overwrite.Overwrite(sp_without_json)
 
     def handle_block(self, block, args, destroy=False, dryrun=False):
         # Check if delete is set to True, and call delete handler

seqerakit/seqeraplatform.py

@@ -139,10 +139,12 @@ def _execute_command(self, full_cmd, to_json=False, print_stdout=True):
 
         if self.json or to_json:
             out = json.loads(stdout)
-            print(json.dumps(out))
+            if should_print:
+                print(json.dumps(out))
         else:
             out = stdout
-            print(stdout)
+            if should_print:
+                print(stdout)
 
         return out