feat(yarn2): Support parsing the project's authors

Signed-off-by: Frank Viernau <[email protected]>
oss-review-toolkit · Nov 15, 2024 · 9c12e33 · 9c12e33
1 parent 609903b
commit 9c12e33
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/...ode/src/funTest/assets/projects/synthetic/yarn2/project-with-lockfile-expected-output.yml b/...ode/src/funTest/assets/projects/synthetic/yarn2/project-with-lockfile-expected-output.yml
@@ -2,6 +2,8 @@
 project:
   id: "Yarn2::yarn2-package-with-lockfile:1.0.0"
   definition_file_path: "<REPLACE_DEFINITION_FILE_PATH>"
+  authors:
+  - "The Author"
   declared_licenses:
   - "Apache-2.0"
   declared_licenses_processed:

diff --git a/...s/projects/synthetic/yarn2/project-with-lockfile-skip-excluded-scopes-expected-output.yml b/...s/projects/synthetic/yarn2/project-with-lockfile-skip-excluded-scopes-expected-output.yml
@@ -2,6 +2,8 @@
 project:
   id: "Yarn2::yarn2-package-with-lockfile:1.0.0"
   definition_file_path: "<REPLACE_DEFINITION_FILE_PATH>"
+  authors:
+  - "The Author"
   declared_licenses:
   - "Apache-2.0"
   declared_licenses_processed:

diff --git a/plugins/package-managers/node/src/main/kotlin/yarn2/Yarn2.kt b/plugins/package-managers/node/src/main/kotlin/yarn2/Yarn2.kt
@@ -405,6 +405,9 @@ class Yarn2(
             val projectFile = definitionFile.resolveSibling(header.version).resolve(definitionFile.name)
             val packageJson = parsePackageJson(projectFile)
             val additionalData = processAdditionalPackageInfo(packageJson)
+            val authors = packageJson.authors
+                .flatMap { parseAuthorString(it.name) }
+                .mapNotNullTo(mutableSetOf()) { it.name }
 
             val id = Identifier("Yarn2", namespace, name, version)
             allProjects += id to Project(
@@ -413,7 +416,8 @@ class Yarn2(
                 declaredLicenses = declaredLicenses,
                 vcs = additionalData.vcsFromPackage,
                 vcsProcessed = processProjectVcs(definitionFile.parentFile, additionalData.vcsFromPackage, homepageUrl),
-                homepageUrl = homepageUrl
+                homepageUrl = homepageUrl,
+                authors = authors
             )
             id
         } else {