fix: dont capture IO for docker pass-through commands

Author: miki725

CHANGELOG.md

@@ -1,5 +1,14 @@
 # Chalk Release Notes
 
+## On the `main` branch
+
+### Fixes
+
+- Docker pass-through commands (non build/push) commands were capturing all
+  IO which could possibly fail with OOM. Standard in/out is no longer captured
+  for pass-through commands to resolve that.
+  ([#514](https://github.com/crashappsec/chalk/pull/514))
+
 ## 0.5.6
 
 **Apr 30, 2025**

chalk.nimble

@@ -11,7 +11,7 @@ bin           = @["chalk"]
 
 # Dependencies
 requires "nim >= 2.0.8"
-requires "https://github.com/crashappsec/con4m#0487d36db4432a800138d4f69ee94bc98d881493"
+requires "https://github.com/crashappsec/con4m#8faa1db2b2e591814a5114f152c81cd51e5ea90f"
 requires "https://github.com/viega/zippy == 0.10.7" # MIT
 requires "https://github.com/NimParsers/parsetoml == 0.7.1" # MIT
 

src/util.nim

@@ -250,12 +250,7 @@ proc runCmdNoOutputCapture*(exe:       string,
                             newStdIn = ""): int {.discardable.} =
   let execOutput = runCmdGetEverything(exe, args, newStdIn,
                                        passthrough = true,
-                                       timeoutUsec = 0) # No timeout
-  result = execOutput.getExit()
-
-proc runCmdExitCode*(exe: string, args: seq[string]): int {.discardable } =
-  let execOutput = runCmdGetEverything(exe, args,
-                                       passthrough = false,
+                                       capture = SPIoNone,
                                        timeoutUsec = 0) # No timeout
   result = execOutput.getExit()