test

Author: Ang-dot

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

@@ -80,36 +80,43 @@ jobs:
           # Function to validate a URL field (supports single values and arrays)
           check_valid_url() {
             local field="$1"
-            local required="${2:-optional}"  # Default to "optional" if no second parameter is provided
-            local value=$(echo "$metadata" | yq ".$field" || true)
+            local field_type="${2:-single_value}"  # Default to "single_value" if no second parameter is provided
+            local required="${3:-optional}"  # Default to "optional" if no third parameter is provided
 
-            # If field is missing or empty
-            if [[ -z "$value" ]]; then
-              if [[ "$required" == "required" ]]; then
-                echo "::error file=$metadata_file::'$field' is required but missing"
-                ((errors++))
+            if [[ "$field_type" == "single_value" ]]; then
+              local value=$(echo "$metadata" | yq ".$field" || true)
+
+              # If field is missing or empty
+              if [[ -z "$value" ]]; then
+                if [[ "$required" == "required" ]]; then
+                  echo "::error file=$metadata_file::'$field' is required but missing"
+                  ((errors++))
+                fi
+                return 0  # Skip validation if it's optional and missing
               fi
-              return 0  # Skip validation if it's optional and missing
-            fi
+            else
+              field_type=$(yq ".${field} | type")
+              echo "The type of '$field' is: $field_type"
+              mapfile -t array < <(echo "$metadata" | yq -r ".$field | select(.) | .[]" || echo "")
+              echo "Array: ${array[@]}"
+
+              local non_empty_entries=0
 
-            # 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
-              # Iterate properly over array elements
-              for url in $(echo "$metadata" | yq -r ".$field[]" || true); do
-                if [[ -z "$url" ]]; then  # Skip empty values inside arrays
+              for url in "${array[@]}"; do
+                if [[ -z "$url" ]]; then
                   continue
-                fi
-                if [[ ! "$url" =~ ^https?:\/\/[a-zA-Z0-9.-]+(\.[a-zA-Z]{2,})+(:[0-9]{1,5})?(\/.*)?$ ]]; then
-                  echo "::error file=$metadata_file::'$field' contains an invalid URL: $url"
-                  ((errors++))
+                else
+                  if [[ "$url" =~ ^(https?:\/\/[a-zA-Z0-9.-]+(\.[a-zA-Z]{2,})+(:[0-9]{1,5})?(\/.*)?)$ ]]; then
+                    ((non_empty_entries++))
+                  else
+                    echo "::error file=$metadata_file::'$field' contains an invalid URL: $url"
+                    ((errors++))
+                  fi
                 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"
+
+              if [[ "$required" == "required" && "$non_empty_entries" -eq 0 ]]; then
+                echo "::error file=$metadata_file::'$field' is required but missing"
                 ((errors++))
               fi
             fi
@@ -128,9 +135,9 @@ jobs:
           check_valid_url "demo_video_url"
           check_valid_url "documentation_url"
           check_valid_url "changelog_url"
-          check_valid_url "screenshots"
 
           # Validate date format (YYYY-MM)
+          echo "The release date is: $(echo "$metadata" | yq '.release_date')"
           release_date=$(echo "$metadata" | yq '.release_date')
           if [[ -z "$release_date" ]]; then
             echo "::error file=$metadata_file::'$release_date' is required but missing"
@@ -143,6 +150,7 @@ jobs:
           fi
 
           # Validate X account handle format (@username)
+          echo "The X account handle is: $(echo "$metadata" | yq '.x_account_handle')"
           x_account_handle=$(echo "$metadata" | yq '.x_account_handle')
           if [[ -z "$x_account_handle" ]]; then
             echo "::error file=$metadata_file::'$x_account_handle' is required but missing"
@@ -155,6 +163,7 @@ jobs:
           fi
 
           # Validate support contact (must be a valid URL or email)
+          echo "The support contact is: $(echo "$metadata" | yq '.support_contact')"
           support_contact=$(echo "$metadata" | yq '.support_contact')
           if [[ -z "$support_contact" ]]; then
             echo "::error file=$metadata_file::'$support_contact' is required but missing"

plugins/plugin_metadata_template.yml

@@ -11,13 +11,13 @@ detailed_description: ""   # Full description with features and benefits
 # Media & Assets
 plugin_logo_url: ""        # URL to the plugin logo (512x512 recommended) (if any or fallback to logo_url)
 screenshots:               # List of screenshots showcasing the plugin (if any)
-  - ""                     # e.g., "https://example.com/screenshot1.png"
-  - ""
+  - "haha"                     # e.g., "https://example.com/screenshot1.png"
+  - "csd"
 demo_video_url: ""         # Link to a demo or walkthrough video (if any)
 documentation_url: ""      # Link to the plugin's official documentation (if any)
 changelog_url: ""          # Link to the changelog (if maintained)
 
 # Contact & Support
 x_account_handle: ""       # X (formerly known as Twitter) account handle (ie: @GAME_Virtuals)
 support_contact: ""        # Email or Slack/Discord link for user support
-community_url: ""          # Forum or community link (if any)
+community_url: "haha"          # Forum or community link (if any)