(CAT-2413) Defaults puppetcore agent version to newest if none given

When puppetcore is selected without a specified version, this change defaults to the latest 8.x version available.

This prevents errors caused by the download site requiring a version parameter.

Relates to CAT-2413

Author: david22swan

tasks/install_powershell.ps1

@@ -120,6 +120,16 @@ if (($collection -like '*nightly*') -And -Not ($PSBoundParameters.ContainsKey('w
   $windows_source = 'https://nightlies.puppet.com/downloads'
 } elseif (($collection -like '*puppetcore*') -And -Not ($PSBoundParameters.ContainsKey('windows_source'))) {
   $windows_source = 'https://artifacts-puppetcore.puppet.com/v1/download'
+  
+  # Puppetcore requires a version to be specified, so we will use the latest version if not specified.
+  # Or if the version is set to "latest".
+  if ($version -eq "" || !$version || $version -eq "latest") {
+    $response = Invoke-WebRequest -Uri "https://forgeapi.puppet.com/private/versions/puppet-agent" -UseBasicParsing
+    $jsonData = $response.Content | ConvertFrom-Json
+    $allVersions = $jsonData.PSObject.Properties.Name
+    $version8x = $allVersions | Where-Object { $_ -like "8.*" }
+    $version = $version8x | Sort-Object { [Version]$_ } | Select-Object -Last 1
+  }
 }
 
 if ($absolute_source) {
@@ -131,6 +141,7 @@ if ($absolute_source) {
     } else {
         $dev = ''
     }
+  
     $msi_source = "${windows_source}?type=native&version=${version}&os_name=windows&os_version=${major_os_version}&os_arch=${arch}&fips=${fips}${dev}"
 } else {
     $msi_source = "$windows_source/windows/${collection}/${msi_name}"

tasks/install_shell.sh

@@ -823,24 +823,33 @@ case $platform in
   "mac_os_x")
     info "Mac platform! Lets get you a DMG..."
     filetype="dmg"
-    if test "$version" = "latest"; then
-      filename="puppet-agent-latest.dmg"
-    else
-      filename="puppet-agent-${version}-1.osx${platform_version}.dmg"
-    fi
-
     arch="x86_64"
     if [[ $(uname -p) == "arm" ]]; then
         arch="arm64"
     fi
     if [[ "$collection" =~ "puppetcore" ]]; then
+      # Puppetcore requires a version to be specified, so we will use the latest version if not specified.
+      # Or if the version is set to "latest".
+      if [[ -z "$version" || "$version" == "latest" ]]; then
+        version=$(curl -sL https://forgeapi.puppet.com/private/versions/puppet-agent | \
+                  jq -r 'keys_unsorted | 
+                        map(select(startswith("8."))) | 
+                        max_by(split(".") | map(tonumber))')
+      fi
+
       dots=$(echo "${version}" | grep -o '\.' | wc -l)
       if (( dots >= 3 )); then
-        download_url="${mac_source}?version=${version}&os_name=osx&os_version=${platform_version}&os_arch=${arch}&dev=true"
+        download_url="${mac_source}?type=native&version=${version}&os_name=osx&os_version=${platform_version}&os_arch=${arch}&dev=true"
       else
-        download_url="${mac_source}?version=${version}&os_name=osx&os_version=${platform_version}&os_arch=${arch}"
+        download_url="${mac_source}?type=native&version=${version}&os_name=osx&os_version=${platform_version}&os_arch=${arch}"
       fi
     else
+      if test "$version" = "latest"; then
+        filename="puppet-agent-latest.dmg"
+      else
+        filename="puppet-agent-${version}-1.osx${platform_version}.dmg"
+      fi
+
       download_url="${mac_source}/mac/${collection}/${platform_version}/${arch}/${filename}"
     fi
     ;;