fix: address Copilot review round 2 on PR #2476

- Remove unused parserRustBinary constant (dispatch returns parserGoBinary
  for all executables, language determined at parse time)
- Make suffix-based dispatch case-insensitive (.deps.json, gradle.lockfile)
  for cross-platform compatibility
- Aggregate parse errors into l.warns so they surface in scan results
- Remove unused fanal blank imports from golden test file

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: kotakanbe

scanner/analyze_golden_test.go

@@ -11,31 +11,6 @@ import (
 	"strings"
 	"testing"
 
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/c/conan"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/dart/pub"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/dotnet/deps"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/dotnet/nuget"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/dotnet/packagesprops"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/elixir/mix"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/golang/binary"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/golang/mod"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/java/gradle"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/java/pom"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/nodejs/bun"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/nodejs/npm"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/nodejs/pnpm"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/nodejs/yarn"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/php/composer"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/python/pip"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/python/pipenv"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/python/poetry"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/python/uv"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/ruby/bundler"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/rust/cargo"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/swift/cocoapods"
-	_ "github.com/aquasecurity/trivy/pkg/fanal/analyzer/language/swift/swift"
-	_ "github.com/future-architect/vuls/scanner/trivy/jar"
-
 	"github.com/future-architect/vuls/models"
 )
 

scanner/base.go

@@ -722,6 +722,7 @@ func (l *base) scanLibraries() (err error) {
 		libraryScanners, err := AnalyzeLibrary(context.Background(), abspath, contents, filemode, l.ServerInfo.Mode.IsOffline())
 		if err != nil {
 			l.log.Warnf("Failed to analyze library %s: %+v", abspath, err)
+			l.warns = append(l.warns, err)
 			continue
 		}
 		for _, libscanner := range libraryScanners {

scanner/dispatch.go

@@ -39,7 +39,6 @@ const (
 	parserMix           parserType = "mix"
 	parserCocoapods     parserType = "cocoapods"
 	parserSwift         parserType = "swift"
-	parserRustBinary    parserType = "rust-binary"
 )
 
 // detectParserType determines which parser should handle a given file,
@@ -127,14 +126,15 @@ func detectParserType(filePath string, filemode os.FileMode) parserType {
 		return parserSwift
 	}
 
-	// Suffix-based matches
-	if strings.HasSuffix(filePath, ".deps.json") {
+	// Suffix-based matches (case-insensitive for cross-platform compatibility)
+	lowerPath := strings.ToLower(filePath)
+	if strings.HasSuffix(lowerPath, ".deps.json") {
 		return parserDotnetDeps
 	}
-	if strings.HasSuffix(strings.ToLower(filePath), "packages.props") {
+	if strings.HasSuffix(lowerPath, "packages.props") {
 		return parserPackagesProps
 	}
-	if strings.HasSuffix(filePath, "gradle.lockfile") {
+	if strings.HasSuffix(lowerPath, "gradle.lockfile") {
 		return parserGradle
 	}