fix(yarn): exclude root packages (#320)

Author: G-Rath

pkg/lockfile/parse-yarn-lock-v2_test.go

@@ -180,12 +180,6 @@ func TestParseYarnLock_v2_WithPrerelease(t *testing.T) {
 			Ecosystem: lockfile.YarnEcosystem,
 			CompareAs: lockfile.YarnEcosystem,
 		},
-		{
-			Name:      "eslint-plugin-jest",
-			Version:   "0.0.0-use.local",
-			Ecosystem: lockfile.YarnEcosystem,
-			CompareAs: lockfile.YarnEcosystem,
-		},
 	})
 }
 
@@ -212,12 +206,6 @@ func TestParseYarnLock_v2_WithBuildString(t *testing.T) {
 			Ecosystem: lockfile.YarnEcosystem,
 			CompareAs: lockfile.YarnEcosystem,
 		},
-		{
-			Name:      "zone.js",
-			Version:   "0.0.0-use.local",
-			Ecosystem: lockfile.YarnEcosystem,
-			CompareAs: lockfile.YarnEcosystem,
-		},
 	})
 }
 
@@ -331,11 +319,30 @@ func TestParseYarnLock_v2_WithAliases(t *testing.T) {
 			Ecosystem: lockfile.YarnEcosystem,
 			CompareAs: lockfile.YarnEcosystem,
 		},
+	})
+}
+
+func TestParseYarnLock_v2_ExcludeRoot(t *testing.T) {
+	t.Parallel()
+
+	packages, err := lockfile.ParseYarnLock("testdata/yarn/exclude-root.v2.lock")
+
+	if err != nil {
+		t.Errorf("Got unexpected error: %v", err)
+	}
+
+	expectPackages(t, packages, []lockfile.PackageDetails{
 		{
-			Name:      "mine",
+			Name:      "@ws/ansi-regex",
 			Version:   "0.0.0-use.local",
 			Ecosystem: lockfile.YarnEcosystem,
 			CompareAs: lockfile.YarnEcosystem,
 		},
+		{
+			Name:      "ansi-regex",
+			Version:   "6.1.0",
+			Ecosystem: lockfile.YarnEcosystem,
+			CompareAs: lockfile.YarnEcosystem,
+		},
 	})
 }

pkg/lockfile/parse-yarn-lock.go

@@ -189,7 +189,7 @@ func ParseYarnLock(pathToLockfile string) ([]PackageDetails, error) {
 	packages := make([]PackageDetails, 0, len(packageGroups))
 
 	for _, group := range packageGroups {
-		if group[0] == "__metadata:" {
+		if group[0] == "__metadata:" || strings.HasSuffix(group[0], "@workspace:.\":") {
 			continue
 		}
 

pkg/lockfile/testdata/yarn/exclude-root.v2.lock

@@ -0,0 +1,30 @@
+# This file is generated by running "yarn install" inside your project.
+# Manual changes might be lost - proceed with caution!
+
+__metadata:
+  version: 8
+  cacheKey: 10c0
+
+"@ws/ansi-regex@workspace:^, @ws/ansi-regex@workspace:ansi-regex":
+  version: 0.0.0-use.local
+  resolution: "@ws/ansi-regex@workspace:ansi-regex"
+  dependencies:
+    ansi-regex: "npm:^6.1.0"
+  languageName: unknown
+  linkType: soft
+
+"ansi-regex@npm:^6.1.0":
+  version: 6.1.0
+  resolution: "ansi-regex@npm:6.1.0"
+  checksum: 10c0/a91daeddd54746338478eef88af3439a7edf30f8e23196e2d6ed182da9add559c601266dbef01c2efa46a958ad6f1f8b176799657616c702b5b02e799e7fd8dc
+  languageName: node
+  linkType: hard
+
+"ansi-regex@workspace:.":
+  version: 0.0.0-use.local
+  resolution: "ansi-regex@workspace:."
+  dependencies:
+    "@ws/ansi-regex": "workspace:^"
+    ansi-regex: "npm:^6.1.0"
+  languageName: unknown
+  linkType: soft