Wrap check errors with distinct error for scorecard-action to ignore. (…

…ossf#2250)

Signed-off-by: Spencer Schrock <sschrock@google.com>

Signed-off-by: Spencer Schrock <sschrock@google.com>

cmd/root.go

@@ -28,6 +28,7 @@ import (
 	"github.com/ossf/scorecard/v4/checker"
 	"github.com/ossf/scorecard/v4/clients"
 	docs "github.com/ossf/scorecard/v4/docs/checks"
+	sce "github.com/ossf/scorecard/v4/errors"
 	sclog "github.com/ossf/scorecard/v4/log"
 	"github.com/ossf/scorecard/v4/options"
 	"github.com/ossf/scorecard/v4/pkg"
@@ -162,7 +163,7 @@ func rootCmd(o *options.Options) error {
 	// intentionally placed at end to preserve outputting results, even if a check has a runtime error
 	for _, result := range repoResult.Checks {
 		if result.Error != nil {
-			return fmt.Errorf("one or more checks had a runtime error: %w", result.Error)
+			return sce.WithMessage(sce.ErrorCheckRuntime, fmt.Sprintf("%s: %v", result.Name, result.Error))
 		}
 	}
 	return nil

errors/public.go

@@ -30,8 +30,10 @@ var (
 	ErrorInvalidURL = errors.New("invalid repo flag")
 	// ErrorShellParsing indicates there was an error when parsing shell code.
 	ErrorShellParsing = errors.New("error parsing shell code")
-	// ErrorUnsupportedCheck indicates check caanot be run for given request.
+	// ErrorUnsupportedCheck indicates check cannot be run for given request.
 	ErrorUnsupportedCheck = errors.New("check is not supported for this request")
+	// ErrorCheckRuntime indicates an individual check had a runtime error.
+	ErrorCheckRuntime = errors.New("check runtime error")
 )
 
 // WithMessage wraps any of the errors listed above.