env_run: fold argv into result logs, drop pre-run debug line

The pre-run `logger.debug("env-run: %s", argv)` was redundant on the
success path (every successful call produced two debug lines for the
same command). Move the argv into the post-run logs so each run
produces exactly one log line, with the full command alongside the
captured streams.

The pre-run log's only unique value was surviving the
`check=True → CalledProcessError` path, but `CalledProcessError`
already carries `cmd`, `returncode`, `stdout`, and `stderr`, so any
caller that handles the exception has access to the same information.

Author: calvinp0

arc/job/env_run.py

@@ -122,16 +122,16 @@ def run_in_conda_env(
         "python", script_path,
         *script_args,
     ]
-    logger.debug("env-run: %s", " ".join(argv))
     result = subprocess.run(argv, check=check, capture_output=True, text=True)
     if result.returncode:
         logger.warning(
-            "env-run: %s exited with %d\nstdout:\n%s\nstderr:\n%s",
-            script_path, result.returncode, result.stdout, result.stderr,
+            "env-run: %s exited with %d\ncmd: %s\nstdout:\n%s\nstderr:\n%s",
+            script_path, result.returncode, " ".join(argv),
+            result.stdout, result.stderr,
         )
     else:
         logger.debug(
-            "env-run: %s exited 0\nstdout:\n%s\nstderr:\n%s",
-            script_path, result.stdout, result.stderr,
+            "env-run: %s exited 0\ncmd: %s\nstdout:\n%s\nstderr:\n%s",
+            script_path, " ".join(argv), result.stdout, result.stderr,
         )
     return result