Publish api json file and enable api change detection

eng/pipelines/templates/steps/build.yml

@@ -81,7 +81,6 @@ steps:
         New-Item -Type Directory -Name $artifactName -Path $(Build.ArtifactStagingDirectory) > $null
         Copy-Item sdk/${{parameters.ServiceDirectory}}/**/$artifactName-[0-9]*.[0-9]*.[0-9]*.tgz $(Build.ArtifactStagingDirectory)/$artifactName
         Copy-Item sdk/${{parameters.ServiceDirectory}}/**/browser/$artifactName-[0-9]*.[0-9]*.[0-9]*.zip $(Build.ArtifactStagingDirectory)/$artifactName
-        Copy-Item sdk/${{parameters.ServiceDirectory}}/**/temp/*.api.json $(Build.ArtifactStagingDirectory)/$artifactName
         if ($${{ parameters.IncludeRelease }} -eq $true)
         {
           New-Item -Type Directory -Name documentation -Path $(Build.ArtifactStagingDirectory)/$artifactName > $null
@@ -90,6 +89,10 @@ steps:
       }
     displayName: 'Copy Packages'
 
+  - script: |
+      eng/scripts/stage-api-review-file.ps1 -PackageInfoPath $(Build.ArtifactStagingDirectory)/PackageInfo -StagingDirectory $(Build.ArtifactStagingDirectory)
+    displayName: 'Copy API extracted files'
+
   - template: /eng/common/pipelines/templates/steps/publish-artifact.yml
     parameters:
       ArtifactPath: '$(Build.ArtifactStagingDirectory)'

eng/scripts/Language-Settings.ps1

@@ -413,14 +413,14 @@ function Find-javascript-Artifacts-For-Apireview($artifactDir, $packageName)
     $files = Get-ChildItem "${artifactPath}" | Where-Object -FilterScript { $_.Name.EndsWith(".api.json") }
     if (!$files)
     {
-      Write-Host "$($artifactPath) does not have api review json for package"
+      Write-Host "$($packageName) does not have api review json"
       Write-Host "API Extractor must be enabled for $($packageName). Please ensure api-extractor.json is present in package directory and api extract script included in build script"
       return $null
     }
     elseif ($files.Count -ne 1)
     {
       Write-Host "$($artifactPath) should contain only one api review for $($packageName)"
-      Write-Host "No of Packages $($files.Count)"
+      Write-Host "No of files $($files.Count)"
       return $null
     }
   }

eng/scripts/stage-api-review-file.ps1

@@ -0,0 +1,50 @@
+param (
+  [Parameter(mandatory = $true)]
+  $PackageInfoPath,
+  [Parameter(mandatory = $true)]
+  $StagingDirectory
+)
+
+if (!((Test-Path $PackageInfoPath) -and (Test-Path $StagingDirectory)))
+{
+  Write-Error "Invalid parameter values. Pleaes verify values for these parameters."
+  exit 1
+}
+
+foreach($pkg in (Get-ChildItem -Path $PackageInfoPath "*.json"))
+{
+  $info = Get-Content -Path $pkg.FullName | ConvertFrom-Json
+  $apiFilePath = Join-Path $info.DirectoryPath "temp"
+  if (Test-Path $apiFilePath)
+  {
+    $apiFile = Get-ChildItem -Path $apiFilePath "*.api.json"
+    if ($apiFile)
+    {
+      if ($apiFile.Count -ne 1)
+      {
+        # Unlikely, but handling to avoid any issue in the future if more than one api file is present here
+        Write-Error "Detected more than one api extracted file in $apiFilePath"
+        exit 1
+      }
+
+      $pkgStagingDir = Join-Path $StagingDirectory $info.ArtifactName
+      if (!(Test-Path $pkgStagingDir))
+      {
+        New-Item -Type Directory -Name $info.ArtifactName -Path $StagingDirectory > $null
+      }
+      $sourceFilePath = $apiFile[0].FullName
+      $targetFilePath = "$($pkgStagingDir)/$($info.ArtifactName)_$($info.Version).api.json"
+      Write-Host "Copying $($sourceFilePath) to $($targetFilePath)"
+      Copy-Item $sourceFilePath $targetFilePath
+    }
+    else
+    {
+      # Not an error is api-extractor is not cofigured/ required for a package
+      Write-Host "API extracted file is not present for package $($info.Name)"
+    }
+  }
+  else
+  {
+    Write-Host "Directory $($apiFilePath) is not present in package root to search for api-extracted file"
+  }
+}