fix: add --ignore-buildable option to pull command to skip services with build sections

Author: mstrhakr

source/compose.manager/scripts/compose.sh

@@ -243,10 +243,10 @@ case $command in
 
   pull)
     if [ "$debug" = true ]; then
-      log_msg "DEBUG" "${compose_base[*]} -p $name pull"
+      log_msg "DEBUG" "${compose_base[*]} -p $name pull --ignore-buildable"
     fi
 
-    "${compose_base[@]}" -p "$name" pull
+    "${compose_base[@]}" -p "$name" pull --ignore-buildable
     exit_code=$?
 
     if [ $exit_code -eq 0 ]; then
@@ -264,7 +264,7 @@ case $command in
   update)
     if [ "$debug" = true ]; then
       log_msg "DEBUG" "${compose_base[*]} -p $name images -q"
-      log_msg "DEBUG" "${compose_base[*]} -p $name pull"
+      log_msg "DEBUG" "${compose_base[*]} -p $name pull --ignore-buildable"
       log_msg "DEBUG" "${compose_base[*]} -p $name up -d --build"
     fi
 
@@ -291,9 +291,9 @@ case $command in
       images=( "${images[@]##sha256:}" )
     fi
 
-    # Pull latest images
+    # Pull latest images (--ignore-buildable: skip services with build sections, they are rebuilt by up --build)
     echo "Pulling latest images..."
-    "${compose_base[@]}" -p "$name" pull
+    "${compose_base[@]}" -p "$name" pull --ignore-buildable
     pull_exit=$?
 
     if [ $pull_exit -ne 0 ]; then

source/compose.manager/scripts/compose_autoupdate.sh

@@ -59,7 +59,8 @@ get_image_digests() {
 OLD_DIGESTS=$(get_image_digests || true)
 
 # Run pull and capture output (timeout prevents indefinite hangs on unresponsive registries)
-timeout "$COMMAND_TIMEOUT" docker compose -f "$COMPOSE_FILE" -p "$PROJECT_NAME" pull > "$OUT" 2>&1 || RC=$?
+# --ignore-buildable: skip services with build sections (they should be rebuilt, not pulled)
+timeout "$COMMAND_TIMEOUT" docker compose -f "$COMPOSE_FILE" -p "$PROJECT_NAME" pull --ignore-buildable > "$OUT" 2>&1 || RC=$?
 
 if [ "$RC" -ne 0 ]; then
   ERRMSG="Auto-update pull failed for '$PROJECT_NAME'. Recent output: $(summarize_output)"

tests/unit/compose.bats

@@ -84,6 +84,32 @@ test_setup() {
     assert_failure
 }
 
+# ============================================================
+# --ignore-buildable Flag Tests
+# ============================================================
+
+@test "compose.sh pull action uses --ignore-buildable" {
+    # Ensure the pull action always passes --ignore-buildable to skip build-only services
+    run grep -E '^\s*"\$\{compose_base\[@\]\}" -p "\$name" pull --ignore-buildable' "$COMPOSE_SCRIPT"
+    assert_success
+}
+
+@test "compose.sh update action pull step uses --ignore-buildable" {
+    # The update action pulls before 'up -d --build'; buildable services are handled by --build
+    run grep -E 'pull --ignore-buildable' "$COMPOSE_SCRIPT"
+    assert_success
+    # Should appear at least twice (pull action + update action)
+    local count
+    count=$(grep -cE 'pull --ignore-buildable' "$COMPOSE_SCRIPT")
+    [ "$count" -ge 2 ]
+}
+
+@test "compose_autoupdate.sh uses --ignore-buildable" {
+    local autoupdate_script="$BATS_TEST_DIRNAME/../../source/compose.manager/scripts/compose_autoupdate.sh"
+    run grep -E 'pull --ignore-buildable' "$autoupdate_script"
+    assert_success
+}
+
 # ============================================================
 # Stack Directory Tests
 # ============================================================