fix: replace stale NEXT: grep with RUN: in validate-phase.sh (2731-c62d) (merge worktree-20260324-091358)

Author: JoeOakhartNava

plugins/dso/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "dso",
-  "version": "0.25.14",
+  "version": "0.25.15",
   "description": "Workflow infrastructure plugin for Claude Code projects",
   "commands": "./commands/",
   "skills": "./skills/",

plugins/dso/scripts/validate-phase.sh

@@ -170,14 +170,14 @@ run_check() {
 
 # run_test_batched: time-bounded test runner using test-batched.sh.
 # When test-batched.sh is available, delegates test execution to it.
-# If test-batched.sh outputs "NEXT:", sets any_fail_ref=2 (pending).
+# If test-batched.sh outputs "RUN:", sets any_fail_ref=2 (pending).
 # Falls back to direct eval when test-batched.sh is not available.
 #
 # Usage: run_test_batched <any_fail_ref>
 #   any_fail_ref — name of the integer variable to set on failure or pending
 #
 # Outputs a TESTS: PASS / FAIL / PENDING line to stdout.
-# Sets $any_fail_ref to 1 on failure, 2 on pending (NEXT: detected).
+# Sets $any_fail_ref to 1 on failure, 2 on pending (RUN: detected).
 run_test_batched() {
     local -n _any_fail="$1"
     local batched_script="$VALIDATE_TEST_BATCHED_SCRIPT"
@@ -193,9 +193,9 @@ run_test_batched() {
             bash "$batched_script" --timeout="$batched_timeout" "$CMD_TEST_UNIT" 2>&1
         ) || rc=$?
 
-        # Detect partial run: test-batched.sh prints "NEXT:" when time budget exhausted.
+        # Detect partial run: test-batched.sh prints "RUN:" when time budget exhausted.
         # In that case it exits 0, but tests are not done — emit TESTS: PENDING.
-        if [ "$rc" = "0" ] && echo "$batched_output" | grep -q "^NEXT:"; then
+        if [ "$rc" = "0" ] && echo "$batched_output" | grep -q "^RUN:"; then
             echo "TESTS: PENDING (run validate-phase.sh again to continue)"
             _any_fail=2
             return

tests/test-validate-phase-portability.sh

@@ -352,20 +352,20 @@ else
 fi
 
 # ---------------------------------------------------------------------------
-# Test i: post-batch with NEXT: output from test-batched.sh exits 2 (pending)
+# Test i: post-batch with RUN: output from test-batched.sh exits 2 (pending)
 # ---------------------------------------------------------------------------
 echo ""
-echo "Test i: post-batch / NEXT: output from test-batched.sh → exit 2 (dso-qlgq)"
+echo "Test i: post-batch / RUN: output from test-batched.sh → exit 2 (dso-qlgq, 2731-c62d)"
 write_all_pass_config
 
 STUB_NEXT="$TMPDIR/stub-test-batched-next.sh"
 STUB_SENTINEL_I="$TMPDIR/test-batched-called-i"
 cat > "$STUB_NEXT" << 'STUB'
 #!/usr/bin/env bash
-# Stub: simulate a partial (time-bounded) run — emit NEXT: and exit 0
+# Stub: simulate a partial (time-bounded) run — emit RUN: and exit 0
 touch "$STUB_SENTINEL_PATH"
 echo "0/1 tests completed."
-echo "NEXT: TEST_BATCHED_STATE_FILE=/tmp/state.json bash /path/to/test-batched.sh 'bash tests/run-all.sh'"
+echo "RUN: TEST_BATCHED_STATE_FILE=/tmp/state.json bash /path/to/test-batched.sh 'bash tests/run-all.sh'"
 exit 0
 STUB
 chmod +x "$STUB_NEXT"
@@ -380,15 +380,15 @@ RUN_OUTPUT=$(
 ) || RUN_EXIT=$?
 
 if [ "$RUN_EXIT" -eq 2 ]; then
-    pass "post-batch NEXT: pending: exit 2"
+    pass "post-batch RUN: pending: exit 2"
 else
-    fail "post-batch NEXT: pending: expected exit 2 (pending), got $RUN_EXIT (output: $RUN_OUTPUT)"
+    fail "post-batch RUN: pending: expected exit 2 (pending), got $RUN_EXIT (output: $RUN_OUTPUT)"
 fi
 
 if echo "$RUN_OUTPUT" | grep -q "TESTS: PENDING"; then
-    pass "post-batch NEXT: pending: output has TESTS: PENDING"
+    pass "post-batch RUN: pending: output has TESTS: PENDING"
 else
-    fail "post-batch NEXT: pending: output missing TESTS: PENDING (output: $RUN_OUTPUT)"
+    fail "post-batch RUN: pending: output missing TESTS: PENDING (output: $RUN_OUTPUT)"
 fi
 
 # ---------------------------------------------------------------------------