refactor: update comment

DmitriyLewen · DmitriyLewen · commit b8e52c499fa9 · 2024-06-06T10:01:07.000+06:00
diff --git a/pkg/dependency/parser/nodejs/npm/parse.go b/pkg/dependency/parser/nodejs/npm/parse.go
@@ -216,7 +216,8 @@ func (p *Parser) resolveLinks(packages map[string]Package) {
 	}
 
 	workspaces := rootPkg.Workspaces
-	// Clone packages to avoid cases when we check already updated packages
+	// Changing the map during the map iteration causes unexpected behavior,
+	// so we need to iterate over the cloned `packages` map, but change the original `packages` map.
 	for pkgPath, pkg := range maps.Clone(packages) {
 		for linkPath, link := range links {
 			if !strings.HasPrefix(pkgPath, link.Resolved) {

Original file line number	Diff line number	Diff line change
`@@ -216,7 +216,8 @@ func (p *Parser) resolveLinks(packages map[string]Package) {`
`216`	`216`	`}`
`217`	`217`
`218`	`218`	`workspaces := rootPkg.Workspaces`
`219`		`- // Clone packages to avoid cases when we check already updated packages`
	`219`	`+ // Changing the map during the map iteration causes unexpected behavior,`
	`220`	+ // so we need to iterate over the cloned `packages` map, but change the original `packages` map.
`220`	`221`	`for pkgPath, pkg := range maps.Clone(packages) {`
`221`	`222`	`for linkPath, link := range links {`
`222`	`223`	`if !strings.HasPrefix(pkgPath, link.Resolved) {`