mirror of
https://github.com/garrytan/gstack.git
synced 2026-09-22 21:00:51 +02:00
fix(sandbox-doctor): keep both heredoc bodies under the 512B pipe-deadlock window
The cycle-2 additions pushed the python-patch and bashrc heredocs into the 512-65536B window test/heredoc-pipe-deadlock.test.ts guards (sh scripts get no BASH_COMPAT escape hatch). Same content, tighter prose; the drift warning now reuses the patch pattern variable instead of a second literal. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Fable 5
parent
643bbfeac9
commit
064216c91b
@@ -73,12 +73,8 @@ new = 'exit 0\nelse\n\tstatus=$?\nfi'
|
|||||||
if old in src:
|
if old in src:
|
||||||
open('/conductor/bin/git', 'w').write(src.replace(old, new))
|
open('/conductor/bin/git', 'w').write(src.replace(old, new))
|
||||||
print('sandbox-doctor: patched /conductor/bin/git exit-code laundering')
|
print('sandbox-doctor: patched /conductor/bin/git exit-code laundering')
|
||||||
elif 'else\n\tstatus=$?\nfi' not in src:
|
elif new not in src:
|
||||||
# Loud on format drift: the outer guard matched but the byte-exact patch
|
print('sandbox-doctor: WARNING git shim drifted from the patch pattern; laundering NOT fixed, patch by hand')
|
||||||
# pattern did not — silence here would read as "fixed" while phantom git
|
|
||||||
# successes persist.
|
|
||||||
print('sandbox-doctor: WARNING /conductor/bin/git matched the laundering guard '
|
|
||||||
'but not the patch pattern — exit-code laundering NOT fixed; patch it by hand')
|
|
||||||
EOF
|
EOF
|
||||||
fi
|
fi
|
||||||
|
|
||||||
@@ -87,11 +83,9 @@ if ! grep -q 'GSTACK sandbox test env' "$HOME/.bashrc" 2>/dev/null; then
|
|||||||
cat >> "$HOME/.bashrc" <<'EOF'
|
cat >> "$HOME/.bashrc" <<'EOF'
|
||||||
|
|
||||||
# GSTACK sandbox test env (written by scripts/sandbox-doctor.sh)
|
# GSTACK sandbox test env (written by scripts/sandbox-doctor.sh)
|
||||||
# NOTE: GSTACK_FREE_RETRY_FLAKY=1 deliberately overrides the runner's
|
# RETRY_FLAKY=1 overrides the runner's default-OFF contract on purpose:
|
||||||
# default-OFF contract ("dev boxes should see flakes, not absorb them") —
|
# the seccomp supervisor injects spurious one-off failures under load.
|
||||||
# this sandbox's seccomp supervisor injects spurious one-off failures under
|
# Delete this block to restore the default.
|
||||||
# load, which the serial retry absorbs while still failing on reproducible
|
|
||||||
# breakage. Delete this block from ~/.bashrc to restore the default.
|
|
||||||
export TMPDIR="$HOME/tmp"
|
export TMPDIR="$HOME/tmp"
|
||||||
export GSTACK_FREE_JOBS=2
|
export GSTACK_FREE_JOBS=2
|
||||||
export GSTACK_FREE_RETRY_FLAKY=1
|
export GSTACK_FREE_RETRY_FLAKY=1
|
||||||
|
|||||||
Reference in New Issue
Block a user