From abf47be074c65af6233eec3ef2a173c52517bbea Mon Sep 17 00:00:00 2001
From: AmitLY21 <Amit.Levy@appsflyer.com>
Date: Thu, 27 Aug 2026 14:31:40 +0300
Subject: [PATCH] Prevent scenario runner commands from draining parent stdin

Commands executed inside the `pre_actions` loop in `af-scenario-runner.sh` could inherit and consume the parent loop's standard input. This would silently starve subsequent `read` commands, causing them to skip remaining `pre_action` lines.

Redirecting stdin to `/dev/null` for these commands ensures the parent loop's input stream remains intact.
---
 scripts/af-scenario-runner.sh | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/scripts/af-scenario-runner.sh b/scripts/af-scenario-runner.sh
index 746391e5..799b0ce9 100755
--- a/scripts/af-scenario-runner.sh
+++ b/scripts/af-scenario-runner.sh
@@ -547,7 +547,10 @@ run_phase_command() {
 
   log_info "${label}: ${command}"
   set +e
-  output=$(eval "$command" 2>&1)
+  # </dev/null: this runs inside the pre_actions `while read ... done < <(...)` loop, and a
+  # stdin-reading command here (e.g. `adb shell`) would inherit that loop's fd0 and drain it,
+  # silently starving `read` of the remaining pre_action lines (e.g. the following `sleep`).
+  output=$(eval "$command" 2>&1 </dev/null)
   status=$?
   set -e