test

Author: Ang-dot

.github/workflows/validate-new-plugin-metadata.yml

@@ -94,11 +94,12 @@ jobs:
 
             # Handle arrays of URLs (e.g., screenshots)
             # Check if the field is an array using yq length check
-            is_array=$(echo "$metadata" | yq -r ".${field} | length")
-            
-            if [[ "$is_array" =~ ^[0-9]+$ && "$is_array" -gt 0 ]]; then
+            local field_type
+            field_type=$(echo "$metadata" | yq -r ".${field} | type")
+
+            if [[ "$field_type" == "!!seq" ]]; then
               # Iterate properly over array elements
-              for url in $(echo "$metadata" | yq -r ".$field[]" || true); do
+              for url in $(echo "$metadata" | yq ".${field} | .[]"); do
                 if [[ -z "$url" ]]; then  # Skip empty values inside arrays
                   continue
                 fi
@@ -107,6 +108,15 @@ jobs:
                   ((errors++))
                 fi
               done
+              for screenshot in $screenshots; do
+                if [[ -z "$screenshot" ]]; then
+                  echo "::error file=$metadata_file::'screenshots' contains an empty string"
+                  ((errors++))
+                elif [[ ! "$screenshot" =~ ^https?:\/\/[a-zA-Z0-9.-]+(\.[a-zA-Z]{2,})+(:[0-9]{1,5})?(\/.*)?\.(png|jpg|jpeg|gif|webp)$ ]]; then
+                  echo "::error file=$metadata_file::'screenshots' contains an invalid URL: $screenshot"
+                  ((errors++))
+                fi
+              done
             else
               if [[ ! "$value" =~ ^https?:\/\/[a-zA-Z0-9.-]+(\.[a-zA-Z]{2,})+(:[0-9]{1,5})?(\/.*)?$ ]]; then
                 echo "::error file=$metadata_file::'$field' is not a valid URL"