From f2dd9dc6b629e233e40316b7713ecbe399876464 Mon Sep 17 00:00:00 2001
From: Daniel Jurek <djurek@microsoft.com>
Date: Wed, 26 May 2021 09:04:44 -0700
Subject: [PATCH 1/3] Update common engineering system for docs onboarding

---
 eng/common/scripts/Update-DocsMsPackages.ps1 | 31 ++++++++++++++++++++
 eng/common/scripts/common.ps1                |  4 +--
 2 files changed, 33 insertions(+), 2 deletions(-)
 create mode 100644 eng/common/scripts/Update-DocsMsPackages.ps1

diff --git a/eng/common/scripts/Update-DocsMsPackages.ps1 b/eng/common/scripts/Update-DocsMsPackages.ps1
new file mode 100644
index 000000000000..1f160859247f
--- /dev/null
+++ b/eng/common/scripts/Update-DocsMsPackages.ps1
@@ -0,0 +1,31 @@
+# This script is intended to update docs.ms CI configuration (currently supports Java, Python, C#, JS) in nightly build
+# For details on calling, check `docindex.yml`. 
+
+# In this script, we will do the following business logic.
+# 1. Filter out the packages from release csv file by `New=true`, `Hide!=true`
+# 2. Compare current package list with the csv packages, and keep them in sync. Leave other packages as they are.
+# 3. Update the tarage packages back to CI config files. 
+param (
+  [Parameter(Mandatory = $true)]
+  $DocRepoLocation # the location of the cloned doc repo
+)
+
+. (Join-Path $PSScriptRoot common.ps1)
+
+if ($UpdateDocsMsPackagesFn -and (Test-Path "Function:$UpdateDocsMsPackagesFn")) {
+
+  try { 
+    &$UpdateDocsMsPackagesFn -DocsRepoLocation $DocRepoLocation
+  } catch { 
+    LogError "Exception while updating docs.ms packages"
+    LogError $_ 
+    LogError $_.ScriptStackTrace
+    exit 1
+  }
+  
+} else {
+  LogError "The function for '$UpdateFn' was not found.`
+  Make sure it is present in eng/scripts/Language-Settings.ps1 and referenced in eng/common/scripts/common.ps1.`
+  See https://github.com/Azure/azure-sdk-tools/blob/master/doc/common/common_engsys.md#code-structure"
+  exit 1
+}
diff --git a/eng/common/scripts/common.ps1 b/eng/common/scripts/common.ps1
index f772b739d804..b92235c208f9 100644
--- a/eng/common/scripts/common.ps1
+++ b/eng/common/scripts/common.ps1
@@ -39,6 +39,6 @@ if (!(Get-Variable -Name "LanguageDisplayName" -ValueOnly -ErrorAction "Ignore")
 $GetPackageInfoFromRepoFn = "Get-${Language}-PackageInfoFromRepo"
 $GetPackageInfoFromPackageFileFn = "Get-${Language}-PackageInfoFromPackageFile"
 $PublishGithubIODocsFn = "Publish-${Language}-GithubIODocs"
-$UpdateDocCIFn = "Update-${Language}-CIConfig"
+$UpdateDocsMsPackagesFn = "Update-${Language}-DocsMsPackages"
 $GetGithubIoDocIndexFn = "Get-${Language}-GithubIoDocIndex"
-$FindArtifactForApiReviewFn = "Find-${Language}-Artifacts-For-Apireview"
\ No newline at end of file
+$FindArtifactForApiReviewFn = "Find-${Language}-Artifacts-For-Apireview"

From 0ab434228a5e07d495452d480202f8fc5ebc61f7 Mon Sep 17 00:00:00 2001
From: Daniel Jurek <djurek@microsoft.com>
Date: Wed, 26 May 2021 13:52:32 -0700
Subject: [PATCH 2/3] Add updates from review feedback

---
 eng/common/scripts/Update-DocsMsPackages.ps1 | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/eng/common/scripts/Update-DocsMsPackages.ps1 b/eng/common/scripts/Update-DocsMsPackages.ps1
index 1f160859247f..df51e69193bd 100644
--- a/eng/common/scripts/Update-DocsMsPackages.ps1
+++ b/eng/common/scripts/Update-DocsMsPackages.ps1
@@ -12,10 +12,15 @@ param (
 
 . (Join-Path $PSScriptRoot common.ps1)
 
+function GetDocsMetadata() { 
+  (Get-CSVMetadata).Where( { $_.New -eq 'true' -and $_.Hide -ne 'true' })
+}
+
 if ($UpdateDocsMsPackagesFn -and (Test-Path "Function:$UpdateDocsMsPackagesFn")) {
 
-  try { 
-    &$UpdateDocsMsPackagesFn -DocsRepoLocation $DocRepoLocation
+  try {
+    $docsMetadata = GetDocsMetadata
+    &$UpdateDocsMsPackagesFn -DocsRepoLocation $DocRepoLocation -DocsMetadata $docsMetadata
   } catch { 
     LogError "Exception while updating docs.ms packages"
     LogError $_ 

From 9893d830354d82f7db007fc936c42f4e3a04c7ed Mon Sep 17 00:00:00 2001
From: Daniel Jurek <djurek@microsoft.com>
Date: Wed, 26 May 2021 13:55:36 -0700
Subject: [PATCH 3/3] Space nit

---
 eng/common/scripts/Update-DocsMsPackages.ps1 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/eng/common/scripts/Update-DocsMsPackages.ps1 b/eng/common/scripts/Update-DocsMsPackages.ps1
index df51e69193bd..2d43127b2b14 100644
--- a/eng/common/scripts/Update-DocsMsPackages.ps1
+++ b/eng/common/scripts/Update-DocsMsPackages.ps1
@@ -13,7 +13,7 @@ param (
 . (Join-Path $PSScriptRoot common.ps1)
 
 function GetDocsMetadata() { 
-  (Get-CSVMetadata).Where( { $_.New -eq 'true' -and $_.Hide -ne 'true' })
+  (Get-CSVMetadata).Where({ $_.New -eq 'true' -and $_.Hide -ne 'true' })
 }
 
 if ($UpdateDocsMsPackagesFn -and (Test-Path "Function:$UpdateDocsMsPackagesFn")) {