fix(node): Deduplicate issue lines before collapsing them

Duplicate lines might confuse the common prefix finder logic.

Signed-off-by: Sebastian Schuberth <[email protected]>

plugins/package-managers/node/src/funTest/assets/projects/synthetic/npm-version-urls-expected-output.yml

@@ -5161,10 +5161,6 @@ issues:
   source: "NPM"
   message: "deprecated [email protected]: Please upgrade to v1.0.1"
   severity: "WARNING"
-- timestamp: "1970-01-01T00:00:00Z"
-  source: "NPM"
-  message: "deprecated [email protected]: Please upgrade to v1.0.1"
-  severity: "WARNING"
 - timestamp: "1970-01-01T00:00:00Z"
   source: "NPM"
   message: "deprecated [email protected]: Please upgrade to v0.1.5"
@@ -5173,10 +5169,6 @@ issues:
   source: "NPM"
   message: "deprecated [email protected]: Please upgrade to v1.0.1"
   severity: "WARNING"
-- timestamp: "1970-01-01T00:00:00Z"
-  source: "NPM"
-  message: "deprecated [email protected]: Please upgrade to v1.0.1"
-  severity: "WARNING"
 - timestamp: "1970-01-01T00:00:00Z"
   source: "NPM"
   message: "deprecated [email protected]: Legacy versions of mkdirp are no longer supported.\

plugins/package-managers/node/src/main/kotlin/Npm.kt

@@ -622,7 +622,7 @@ open class Npm(
             var commonPrefix: String
             var previousPrefix = ""
 
-            val collapsedLines = issueLines.fold(mutableListOf<String>()) { messages, line ->
+            val collapsedLines = issueLines.distinct().fold(mutableListOf<String>()) { messages, line ->
                 if (messages.isEmpty()) {
                     // The first line is always added including the prefix. The prefix will be removed later.
                     messages += line