Apply a few changes to Get-PRPackageProperties to make it more customizable for our users.

Remove wonky gap + extra spacing that should pass the patch application process now that we've manually aligned common.ps1

Author: scbedd

eng/common/scripts/Package-Properties.ps1

@@ -15,7 +15,7 @@ class PackageProps
     [boolean]$IsNewSdk
     [string]$ArtifactName
     [string]$ReleaseStatus
-    [string[]]$DependentPackages
+    [string[]]$CanaryPackages
 
     PackageProps([string]$name, [string]$version, [string]$directoryPath, [string]$serviceDirectory)
     {
@@ -113,6 +113,11 @@ function Get-PrPkgProperties([string]$InputDiffJson) {
     $diff = Get-Content $InputDiffJson | ConvertFrom-Json
     $targetedFiles = $diff.ChangedFiles
 
+    $customDependentPackagesHandler = $false
+    if ($DependentPackagesFromPackageSetFn -and (Test-Path "Function:$DependentPackagesFromPackageSetFn")) {
+        $customDependentPackagesHandler = $true
+    }
+
     $dependentPackagesForInclusion = @()
     $lookup = @{}
 
@@ -129,16 +134,25 @@ function Get-PrPkgProperties([string]$InputDiffJson) {
             if ($shouldInclude) {
                 $packagesWithChanges += $pkg
 
-                if ($pkg.DependentPackages) {
-                    $dependentPackagesForInclusion += $pkg.DependentPackages
+                if ($customDependentPackagesHandler -and $pkg.CanaryPackages) {
+                    $dependentPackagesForInclusion += $pkg.CanaryPackages
                 }
             }
         }
     }
 
-    foreach ($addition in $dependentPackagesForInclusion) {
-        if ($lookup[$addition]) {
-            $packagesWithChanges += $lookup[$addition]
+    # there is a custom function to get dependent packages from a changed package set and diff
+    if ($customDependentPackagesHandler)
+    {
+        $packagesWithChanges += &$DependentPackagesFromPackageSetFn $packagesWithChanges $diff
+    }
+    else {
+        foreach ($addition in $dependentPackagesForInclusion) {
+            $key = $addition.Replace($RepoRoot, "").SubString(1)
+
+            if ($lookup[$key]) {
+                $packagesWithChanges += $lookup[$key]
+            }
         }
     }
 

eng/common/scripts/common.ps1

@@ -62,7 +62,8 @@ $GetEmitterAdditionalOptionsFn = "Get-${Language}-EmitterAdditionalOptions"
 $GetEmitterNameFn = "Get-${Language}-EmitterName"
 $GetDirectoriesForGenerationFn = "Get-${Language}-DirectoriesForGeneration"
 $UpdateGeneratedSdksFn = "Update-${Language}-GeneratedSdks"
-$IsApiviewStatusCheckRequiredFn = "Get-${Language}-ApiviewStatusCheckRequirement" 
+$IsApiviewStatusCheckRequiredFn = "Get-${Language}-ApiviewStatusCheckRequirement"
+$DependentPackagesFromPackageSetFn = "Get-${Language}-DependentPackagesFromPackageSet"
 
 # Expected to be set in eng/scripts/docs/Docs-Onboarding.ps1
 $SetDocsPackageOnboarding = "Set-${Language}-DocsPackageOnboarding"