johba/harb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: OptimizerV3.sol mutation has no CI guard (#631)

Browse source 
batch-eval.sh mutates OptimizerV3.sol by injecting Push3 candidates but
never restores it on exit. Add a backup/restore trap so the file is
always returned to its committed state, and add a CI step that fails
loudly if OptimizerV3.sol is left dirty after any pipeline step.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
openhands 2026-03-19 22:36:49 +00:00
parent 9cfffa5cea
commit af2f7e6115
2 changed files with 23 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

11
tools/push3-evolution/revm-evaluator/batch-eval.sh
View file

@ -104,10 +104,19 @@ fi
MANIFEST_DIR="$(mktemp -d)"
# Back up OptimizerV3.sol so we can restore it on exit — batch-eval mutates
# this file by injecting transpiled Push3 candidates (via inject.sh).
cp "$OPTIMIZERV3_SOL" "${OPTIMIZERV3_SOL}.batch-backup"
cleanup() {
# Restore OptimizerV3.sol to its pre-run state
if [ -f "${OPTIMIZERV3_SOL}.batch-backup" ]; then
cp "${OPTIMIZERV3_SOL}.batch-backup" "$OPTIMIZERV3_SOL"
rm -f "${OPTIMIZERV3_SOL}.batch-backup"
fi
[ -d "${MANIFEST_DIR:-}" ] && rm -rf "$MANIFEST_DIR"
}
trap cleanup EXIT
trap cleanup EXIT INT TERM
IDS_FILE="$MANIFEST_DIR/ids.txt"
BYTECODES_FILE="$MANIFEST_DIR/bytecodes.txt"