fix: correctly handle exiting the argument parsing loop

2025-11-16 01:24:28 -05:00
parent 531bb20519
commit a9127e3abd
4 changed files with 20 additions and 8 deletions
--- a/plugins/builder-dockerfile/builder-build
+++ b/plugins/builder-dockerfile/builder-build
@@ -236,8 +236,11 @@ trigger-builder-dockerfile-builder-build() {
        shift 1
        ;;
      *)
-        shift
-        continue
+        if [[ -n "$1" ]]; then
+          shift
+          continue
+        fi
+        break
        ;;
    esac
  done
--- a/plugins/builder-nixpacks/builder-build
+++ b/plugins/builder-nixpacks/builder-build
@@ -232,8 +232,11 @@ trigger-builder-nixpacks-builder-build() {
        shift 1
        ;;
      *)
-        shift
-        continue
+        if [[ -n "$1" ]]; then
+          shift
+          continue
+        fi
+        break
        ;;
    esac
  done
--- a/plugins/builder-pack/builder-build
+++ b/plugins/builder-pack/builder-build
@@ -270,8 +270,11 @@ trigger-builder-pack-builder-build() {
        shift 1
        ;;
      *)
-        shift
-        continue
+        if [[ -n "$1" ]]; then
+          shift
+          continue
+        fi
+        break
        ;;
    esac
  done
--- a/plugins/builder-railpack/builder-build
+++ b/plugins/builder-railpack/builder-build
@@ -114,8 +114,11 @@ trigger-builder-railpack-builder-build() {
        shift 1
        ;;
      *)
-        shift
-        continue
+        if [[ -n "$1" ]]; then
+          shift
+          continue
+        fi
+        break
        ;;
    esac
  done