Notices Generation Fix - update manifest parser (#9842)

* Notices Generation Fix - update manifest parser

Author: granluo

.github/workflows/notice_generation.yml

@@ -22,7 +22,7 @@ jobs:
     - name: Get all pod names
       run: |
         cd "${GITHUB_WORKSPACE}/ReleaseTooling/"
-        swift run manifest --pod-name-output-file-path ./output.txt
+        swift run manifest --output-file-path ./output.txt --for-notices-generation
         PODS=`cat ./output.txt`
         echo "PODS=${PODS}" >> $GITHUB_ENV
         echo "NOTICES_PATH=${GITHUB_WORKSPACE}/${NOTICES_PATH}" >> $GITHUB_ENV

.github/workflows/prerelease.yml

@@ -35,7 +35,7 @@ jobs:
       id: generate_matrix
       run: |
         cd "${GITHUB_WORKSPACE}/ReleaseTooling"
-        swift run manifest --sdk-repo-url "${GITHUB_WORKSPACE}" --spec-output-file-path ./output.json
+        swift run manifest --sdk-repo-url "${GITHUB_WORKSPACE}" --output-file-path ./output.json --for-gha-matrix-generation
         echo "::set-output name=matrix::{\"include\":$( cat output.json )}"
     - name: Get token
       run: |

.github/workflows/release.yml

@@ -36,7 +36,7 @@ jobs:
       id: generate_matrix
       run: |
         cd "${GITHUB_WORKSPACE}/ReleaseTooling"
-        swift run manifest --sdk-repo-url "${GITHUB_WORKSPACE}" --spec-output-file-path ./output.json
+        swift run manifest --sdk-repo-url "${GITHUB_WORKSPACE}" --output-file-path ./output.json --for-gha-matrix-generation
         echo "::set-output name=matrix::{\"include\":$( cat output.json )}"
     - name: Get token
       run: |

ReleaseTooling/Sources/ManifestParser/main.swift

@@ -19,14 +19,19 @@ import FirebaseManifest
 import Foundation
 import Utils
 
+enum ParsingMode: String, EnumerableFlag {
+  case forNoticesGeneration
+  case forGHAMatrixGeneration
+}
+
 struct ManifestParser: ParsableCommand {
   @Option(help: "The path of the SDK repo.",
           transform: { str in
             if NSString(string: str).isAbsolutePath { return URL(fileURLWithPath: str) }
             let documentDir = URL(fileURLWithPath: FileManager.default.currentDirectoryPath)
             return documentDir.appendingPathComponent(str)
           })
-  var SDKRepoURL: URL
+  var SDKRepoURL: URL?
 
   /// Path of a text file for Firebase Pods' names.
   @Option(help: "An output file with Podspecs",
@@ -35,32 +40,45 @@ struct ManifestParser: ParsableCommand {
             let documentDir = URL(fileURLWithPath: FileManager.default.currentDirectoryPath)
             return documentDir.appendingPathComponent(str)
           })
-  var specOutputFilePath: URL
+  var outputFilePath: URL
 
   @Option(parsing: .upToNextOption, help: "Podspec files that will not be included.")
   var excludedSpecs: [String]
 
+  @Flag(help: "Parsing mode for manifest")
+  var mode: ParsingMode
+
   func parsePodNames(_ manifest: Manifest) throws {
     var output: [String] = []
     for pod in manifest.pods {
       output.append(pod.name)
     }
     do {
       try output.joined(separator: ",")
-        .write(to: specOutputFilePath, atomically: true,
+        .write(to: outputFilePath, atomically: true,
                encoding: String.Encoding.utf8)
-      print("\(output) is written in \n \(specOutputFilePath).")
+      print("\(output) is written in \n \(outputFilePath).")
     } catch {
       throw error
     }
   }
 
   func run() throws {
-    let specCollector = GHAMatrixSpecCollector(
-      SDKRepoURL: SDKRepoURL,
-      outputSpecFileURL: specOutputFilePath
-    )
-    try specCollector.generateMatrixJson(to: specOutputFilePath)
+    switch mode {
+    case .forNoticesGeneration:
+      try parsePodNames(FirebaseManifest.shared)
+    case .forGHAMatrixGeneration:
+      guard let sdkRepoURL = SDKRepoURL else {
+        throw fatalError(
+          "--sdk-repo-url should be specified when --for-gha-matrix-generation is on."
+        )
+      }
+      let specCollector = GHAMatrixSpecCollector(
+        SDKRepoURL: sdkRepoURL,
+        outputSpecFileURL: outputFilePath
+      )
+      try specCollector.generateMatrixJson(to: outputFilePath)
+    }
   }
 }