tools: improve backport review script

Signed-off-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
PR-URL: #62573
Reviewed-By: Filip Skokan <panva.ip@gmail.com>

Author: aduh95

tools/actions/review_backport.sh

@@ -2,11 +2,16 @@
 
 BACKPORT_PR=$1
 
+[ -x "$(command -v gh || true)" ] || {
+    # shellcheck disable=SC2016
+    echo 'Missing required `gh` dependency' >&2
+    exit 1
+}
 [ -n "$BACKPORT_PR" ] || {
-    echo "Usage:"
-    echo 'tools/actions/review_backport.sh https://github.com/nodejs/node/pull/<backport-PR-number> | less'
-    echo 'DIFF_CMD="codium --wait --diff" tools/actions/review_backport.sh https://github.com/nodejs/node/pull/<backport-PR-number>'
-    echo "Limitations: This tools only supports PRs that landed as single commit, e.g. with 'commit-queue-squash' label."
+    echo "Usage:" >&2
+    echo 'tools/actions/review_backport.sh https://github.com/nodejs/node/pull/<backport-PR-number> | less' >&2
+    echo 'DIFF_CMD="codium --wait --diff" tools/actions/review_backport.sh https://github.com/nodejs/node/pull/<backport-PR-number>' >&2
+    echo "Limitations: This tools only supports PRs that landed as single commit, e.g. with 'commit-queue-squash' label." >&2
 
     exit 1
 }
@@ -15,16 +20,28 @@ SED_CMD='s/^index [a-f0-9]\+..[a-f0-9]\+ \([0-7]\{6\}\)$/index eeeeeeeeee..eeeee
 
 set -ex
 
+BACKPORT_PR_URL=$(gh pr view "$BACKPORT_PR" --json url --jq .url)
+
 ORIGINAL=$(mktemp)
 BACKPORT=$(mktemp)
+trap 'set -x; rm -f "$ORIGINAL" "$BACKPORT"; set +x; trap - EXIT; exit' EXIT INT HUP
+
 gh pr view "$BACKPORT_PR" --json commits --jq '.[] | map([ .oid, (.messageBody | match("(?:^|\\n)PR-URL: (https?://.+/pull/\\d+)(?:\\n|$)", "g") | .captures | last | .string)] | @tsv) | .[]' \
 | while read -r LINE; do
   COMMIT_SHA=$(echo "$LINE" | cut -f1)
   PR_URL=$(echo "$LINE" | cut -f2)
 
   curl -fsL "$PR_URL.diff" | sed "$SED_CMD" >> "$ORIGINAL"
-  curl -fsL "$BACKPORT_PR/commits/$COMMIT_SHA.diff" | sed "$SED_CMD" >> "$BACKPORT"
+  curl -fsL "$BACKPORT_PR_URL/commits/$COMMIT_SHA.diff" | sed "$SED_CMD" >> "$BACKPORT"
 done
 
-${DIFF_CMD:-diff} "$ORIGINAL" "$BACKPORT"
-rm "$ORIGINAL" "$BACKPORT"
+${DIFF_CMD:-diff} "$ORIGINAL" "$BACKPORT" || echo "diff command exited with $?" >&2
+
+set +x
+
+printf "Approve the PR using gh? [y/N] "
+read -r r
+[ "$r" != "y" ] || {
+  set -x
+  gh pr review "$BACKPORT_PR" --approve
+}