chore: make update deleted package script put one entry per line

Author: rickeylev

tools/update_deleted_packages.sh

@@ -13,27 +13,57 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# For integration tests, we want to be able to glob() up the sources inside a nested package
-# See explanation in .bazelrc
-#
-# This script ensures that we only delete subtrees that have something a file
-# signifying a new bazel workspace, whether it be bzlmod or classic. Generic
-# algorithm:
-#   1. Get all directories where a WORKSPACE or MODULE.bazel exists.
-#   2. For each of the directories, get all directories that contains a BUILD.bazel file.
-#   3. Sort and remove duplicates.
-
-set -euxo pipefail
-
-DIR="$(dirname $0)/.."
-cd $DIR
-
-# The sed -i.bak pattern is compatible between macos and linux
-sed -i.bak "/^[^#].*--deleted_packages/s#=.*#=$(\
-    find examples/*/* tests/*/* \( -name WORKSPACE -or -name MODULE.bazel \) |
-        xargs -n 1 dirname |
-        xargs -n 1 -I{} find {} \( -name BUILD -or -name BUILD.bazel \) |
-        xargs -n 1 dirname |
-        sort -u |
-        paste -sd, -\
-)#" $DIR/.bazelrc && rm .bazelrc.bak
+# This script finds Bazel sub-workspaces and marks them, and all their BUILD
+# containing directories, as ignored by the root level bazel project, via the
+# `--deleted_packages` flag.
+
+set -euo pipefail
+set -x
+
+DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" >/dev/null 2>&1 && pwd)"
+ROOT_DIR="$DIR/.."
+BAZELRC="$ROOT_DIR/.bazelrc"
+
+# Find all packages in sub-workspaces
+found_packages() {
+    cd "$ROOT_DIR"
+    (
+        find . -mindepth 2 \( -name WORKSPACE -o -name MODULE.bazel \) |
+            while read -r marker; do
+                workspace_dir="$(dirname "$marker")"
+                echo "$workspace_dir"
+                find "$workspace_dir" \( -name BUILD -o -name BUILD.bazel \) -exec dirname {} \;
+            done
+    ) | sed 's#^\./##'
+}
+
+# Update .bazelrc
+update_bazelrc() {
+    local packages
+    packages=$(found_packages | sort -u)
+
+    local start_marker="# GENERATED_DELETED_PACKAGES_START"
+    local end_marker="# GENERATED_DELETED_PACKAGES_END"
+
+    # Create a temporary file
+    local tmpfile
+    tmpfile=$(mktemp)
+
+    # Write the content before the start marker
+    sed "/$start_marker/q" "$BAZELRC" > "$tmpfile"
+
+    # Write the generated packages
+    echo "$start_marker" >> "$tmpfile"
+    for pkg in $packages; do
+        echo "common --deleted_packages=$pkg" >> "$tmpfile"
+    done
+    echo "$end_marker" >> "$tmpfile"
+
+    # Write the content after the end marker
+    sed "1,/$end_marker/d" "$BAZELRC" >> "$tmpfile"
+
+    # Replace the original file
+    mv "$tmpfile" "$BAZELRC"
+}
+
+update_bazelrc