topdown: Include both input and error details in PURL parse errors

Anivar A Aravind · Anivar A Aravind · commit ce5cc8d576c4 · 2025-08-24T23:12:58.000+05:30
As suggested by @srenatus, error messages now include both the invalid input string and the underlying parser error using %w for proper error wrapping. This provides better debugging information when parsing fails.
diff --git a/v1/topdown/purl.go b/v1/topdown/purl.go
@@ -31,7 +31,7 @@ func builtinPurlParse(_ BuiltinContext, operands []*ast.Term, iter func(*ast.Ter
 
 	purl, err := packageurl.FromString(string(str))
 	if err != nil {
-		return fmt.Errorf("invalid PURL: %s", str)
+		return fmt.Errorf("invalid PURL: %s: %w", str, err)
 	}
 
 	// Handle qualifiers - always return an object (empty if no qualifiers)

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ func builtinPurlParse(_ BuiltinContext, operands []ast.Term, iter func(ast.Ter`
`31`	`31`
`32`	`32`	`purl, err := packageurl.FromString(string(str))`
`33`	`33`	`if err != nil {`
`34`		`- return fmt.Errorf("invalid PURL: %s", str)`
	`34`	`+ return fmt.Errorf("invalid PURL: %s: %w", str, err)`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`// Handle qualifiers - always return an object (empty if no qualifiers)`