fix: prevent component duplication

jkowalleck · jkowalleck · commit 21258b7e7e5a · 2025-11-07T13:36:19.000+01:00
Signed-off-by: Jan Kowalleck &lt;jan.kowalleck@gmail.com&gt;
diff --git a/src/extractor.ts b/src/extractor.ts
@@ -51,7 +51,7 @@ export class Extractor {
   }
 
   generateComponents (modules: Iterable<Module>, collectEvidence: boolean, logger?: WebpackLogger): Iterable<CDX.Models.Component> {
-    const pkgs: Record<string, CDX.Models.Component | undefined> = {}
+    const pkgs: Record<string, CDX.Models.Component> = {}
     const components = new Map<Module, CDX.Models.Component>()
 
     logger?.log('start building Components from modules...')
@@ -85,7 +85,7 @@ export class Extractor {
     this.#linkDependencies(components)
 
     logger?.log('done building Components from modules...')
-    return components.values()
+    return Object.values(pkgs)
   }
 
   /**

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ export class Extractor {`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`generateComponents (modules: Iterable<Module>, collectEvidence: boolean, logger?: WebpackLogger): Iterable<CDX.Models.Component> {`
`54`		`- const pkgs: Record<string, CDX.Models.Component \| undefined> = {}`
	`54`	`+ const pkgs: Record<string, CDX.Models.Component> = {}`
`55`	`55`	`const components = new Map<Module, CDX.Models.Component>()`
`56`	`56`
`57`	`57`	`logger?.log('start building Components from modules...')`
`@@ -85,7 +85,7 @@ export class Extractor {`
`85`	`85`	`this.#linkDependencies(components)`
`86`	`86`
`87`	`87`	`logger?.log('done building Components from modules...')`
`88`		`- return components.values()`
	`88`	`+ return Object.values(pkgs)`
`89`	`89`	`}`
`90`	`90`
`91`	`91`	`/**`