Skip to content

Commit fe276dd

Browse files
sjarmakclaude
andcommitted
fix: check FULL_CONFIG string instead of VERIFIER_MODE for artifact detection
VERIFIER_MODE is set by config_to_mcp_type() but that function runs inside $() subshells at lines 151-152, so the global never propagates to the parent shell. Check FULL_CONFIG directly instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 72a0502 commit fe276dd

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

configs/run_selected_tasks.sh

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -376,9 +376,11 @@ _launch_task_pair() {
376376
local _mcp_temp_dir=""
377377
local _bl_temp_dir=""
378378

379-
# Determine which Dockerfile variant to use for each config
379+
# Determine which Dockerfile variant to use for each config.
380+
# Note: cannot rely on VERIFIER_MODE because config_to_mcp_type runs in $()
381+
# subshells (lines 151-152), so its side-effect globals don't propagate.
380382
local _is_artifact=false
381-
[[ "$VERIFIER_MODE" == "artifact" ]] && _is_artifact=true
383+
[[ "$FULL_CONFIG" == *artifact* ]] && _is_artifact=true
382384

383385
# Launch baseline config
384386
if [ "$RUN_BASELINE" = true ]; then

0 commit comments

Comments
 (0)