git-ents.gitmain
⌘K
foforge
commit ba397d9
fix: separate sprite exec flags from the command with `--`

sprite exec parses flags up to the command, so sprite exec -s <s> sh -c '…' consumed -c itself and failed. Pass -- before the command in both the tree sync and the check run. Verified against a live Sprite: stdin forwards, exit codes propagate, and git archive | tar -x lands the tree.

Assisted-by: Claude:claude-opus-4-8

Joseph D. Carpinelli · 1 month ago

Reviews

No reviews of this commit yet — record a verdict below.

Start a review

verdict

crates/git-ents-server/src/checks.rs @@ -145,7 +145,7 @@ let script = format!("rm -rf {WORKDIR} && mkdir -p {WORKDIR} && tar -x -C {WORKDIR}"); let mut child = Command::new("sprite") - .args(["exec", "-s", sprite, "sh", "-c", &script]) + .args(["exec", "-s", sprite, "--", "sh", "-c", &script]) .stdin(Stdio::piped()) .spawn() .map_err(|e| format!("could not run the sprite CLI: {e}"))?; @@ -176,6 +176,7 @@ sprite, "--dir", WORKDIR, + "--", "sh", "-c", &check.command,