✨ Allow incomplete local checks (#4423)

lharrison13 · web-flow · commit cdfb58b4930d · 2024-12-02T21:12:51.000Z
* ✨ allow incomplete localdir checks (#3832) Signed-off-by: Luke Harrison <luke.harrison1@ibm.com> * ✨ fixes as per @spencerschrock (#3832) Signed-off-by: Luke Harrison <Luke.Harrison1@ibm.com> * ✨ fixed linting issues (#3832) Signed-off-by: Luke Harrison <Luke.Harrison1@ibm.com> --------- Signed-off-by: Luke Harrison <luke.harrison1@ibm.com> Signed-off-by: Luke Harrison <Luke.Harrison1@ibm.com>
diff --git a/checks/dangerous_workflow.go b/checks/dangerous_workflow.go
@@ -29,8 +29,8 @@ const CheckDangerousWorkflow = "Dangerous-Workflow"
 //nolint:gochecknoinits
 func init() {
 	supportedRequestTypes := []checker.RequestType{
-		checker.FileBased,
 		checker.CommitBased,
+		checker.FileBased,
 	}
 	if err := registerCheck(CheckDangerousWorkflow, DangerousWorkflow, supportedRequestTypes); err != nil {
 		// this should never happen
diff --git a/checks/fuzzing.go b/checks/fuzzing.go
@@ -28,7 +28,10 @@ const CheckFuzzing = "Fuzzing"
 
 //nolint:gochecknoinits
 func init() {
-	if err := registerCheck(CheckFuzzing, Fuzzing, nil); err != nil {
+	supportedRequestTypes := []checker.RequestType{
+		checker.FileBased,
+	}
+	if err := registerCheck(CheckFuzzing, Fuzzing, supportedRequestTypes); err != nil {
 		// this should never happen
 		panic(err)
 	}
diff --git a/checks/license.go b/checks/license.go
@@ -30,6 +30,7 @@ const CheckLicense = "License"
 func init() {
 	supportedRequestTypes := []checker.RequestType{
 		checker.CommitBased,
+		checker.FileBased,
 	}
 	if err := registerCheck(CheckLicense, License, supportedRequestTypes); err != nil {
 		// this should never happen
diff --git a/checks/packaging.go b/checks/packaging.go
@@ -21,6 +21,7 @@ import (
 	"github.com/ossf/scorecard/v5/checks/raw/gitlab"
 	"github.com/ossf/scorecard/v5/clients/githubrepo"
 	"github.com/ossf/scorecard/v5/clients/gitlabrepo"
+	"github.com/ossf/scorecard/v5/clients/localdir"
 	sce "github.com/ossf/scorecard/v5/errors"
 	"github.com/ossf/scorecard/v5/probes"
 	"github.com/ossf/scorecard/v5/probes/zrunner"
@@ -31,18 +32,34 @@ const CheckPackaging = "Packaging"
 
 //nolint:gochecknoinits
 func init() {
-	if err := registerCheck(CheckPackaging, Packaging, nil); err != nil {
+	supportedRequestTypes := []checker.RequestType{
+		checker.FileBased,
+	}
+	if err := registerCheck(CheckPackaging, Packaging, supportedRequestTypes); err != nil {
 		// this should never happen
 		panic(err)
 	}
 }
 
 // Packaging runs Packaging check.
 func Packaging(c *checker.CheckRequest) checker.CheckResult {
-	var rawData checker.PackagingData
-	var err error
+	var rawData, rawDataGithub, rawDataGitlab checker.PackagingData
+	var err, errGithub, errGitlab error
 
 	switch v := c.RepoClient.(type) {
+	case *localdir.Client:
+		// Performing both packaging checks since we dont know when local
+		rawDataGithub, errGithub = github.Packaging(c)
+		rawDataGitlab, errGitlab = gitlab.Packaging(c)
+		// Appending results of checks
+		rawData.Packages = append(rawData.Packages, rawDataGithub.Packages...)
+		rawData.Packages = append(rawData.Packages, rawDataGitlab.Packages...)
+		// checking for errors
+		if errGithub != nil {
+			err = errGithub
+		} else if errGitlab != nil {
+			err = errGitlab
+		}
 	case *githubrepo.Client:
 		rawData, err = github.Packaging(c)
 	case *gitlabrepo.Client:
diff --git a/checks/permissions.go b/checks/permissions.go
@@ -29,8 +29,8 @@ const CheckTokenPermissions = "Token-Permissions"
 //nolint:gochecknoinits
 func init() {
 	supportedRequestTypes := []checker.RequestType{
-		checker.FileBased,
 		checker.CommitBased,
+		checker.FileBased,
 	}
 	if err := registerCheck(CheckTokenPermissions, TokenPermissions, supportedRequestTypes); err != nil {
 		// This should never happen.
diff --git a/checks/pinned_dependencies.go b/checks/pinned_dependencies.go
@@ -29,8 +29,8 @@ const CheckPinnedDependencies = "Pinned-Dependencies"
 //nolint:gochecknoinits
 func init() {
 	supportedRequestTypes := []checker.RequestType{
-		checker.FileBased,
 		checker.CommitBased,
+		checker.FileBased,
 	}
 	if err := registerCheck(CheckPinnedDependencies, PinningDependencies, supportedRequestTypes); err != nil {
 		// This should never happen.
diff --git a/checks/raw/fuzzing.go b/checks/raw/fuzzing.go
@@ -338,6 +338,8 @@ func getProminentLanguages(langs []clients.Language) []clients.LanguageName {
 	numLangs := len(langs)
 	if numLangs == 0 {
 		return nil
+	} else if len(langs) == 1 && langs[0].Name == clients.All {
+		return getAllLanguages()
 	}
 	totalLoC := 0
 	// Use a map to record languages and their lines of code to drop potential duplicates.
@@ -361,6 +363,14 @@ func getProminentLanguages(langs []clients.Language) []clients.LanguageName {
 	return ret
 }
 
+func getAllLanguages() []clients.LanguageName {
+	allLanguages := make([]clients.LanguageName, 0, len(languageFuzzSpecs))
+	for l := range languageFuzzSpecs {
+		allLanguages = append(allLanguages, l)
+	}
+	return allLanguages
+}
+
 func propertyBasedDescription(language string) *string {
 	s := fmt.Sprintf("Property-based testing in %s generates test instances randomly or exhaustively "+
 		"and test that specific properties are satisfied.", language)
diff --git a/checks/raw/github/packaging.go b/checks/raw/github/packaging.go
@@ -15,6 +15,7 @@
 package github
 
 import (
+	"errors"
 	"fmt"
 	"io"
 	"path/filepath"
@@ -23,6 +24,7 @@ import (
 
 	"github.com/ossf/scorecard/v5/checker"
 	"github.com/ossf/scorecard/v5/checks/fileparser"
+	"github.com/ossf/scorecard/v5/clients"
 	"github.com/ossf/scorecard/v5/finding"
 )
 
@@ -73,7 +75,12 @@ func Packaging(c *checker.CheckRequest) (checker.PackagingData, error) {
 
 		runs, err := c.RepoClient.ListSuccessfulWorkflowRuns(filepath.Base(fp))
 		if err != nil {
-			return data, fmt.Errorf("Client.Actions.ListWorkflowRunsByFileName: %w", err)
+			// assume the workflow will have run for localdir client
+			if errors.Is(err, clients.ErrUnsupportedFeature) {
+				runs = append(runs, clients.WorkflowRun{})
+			} else {
+				return data, fmt.Errorf("Client.Actions.ListWorkflowRunsByFileName: %w", err)
+			}
 		}
 
 		if len(runs) > 0 {
diff --git a/checks/raw/sast.go b/checks/raw/sast.go
@@ -28,6 +28,7 @@ import (
 
 	"github.com/ossf/scorecard/v5/checker"
 	"github.com/ossf/scorecard/v5/checks/fileparser"
+	"github.com/ossf/scorecard/v5/clients"
 	sce "github.com/ossf/scorecard/v5/errors"
 	"github.com/ossf/scorecard/v5/finding"
 )
@@ -92,6 +93,10 @@ func sastToolInCheckRuns(c *checker.CheckRequest) ([]checker.SASTCommit, error)
 	var sastCommits []checker.SASTCommit
 	commits, err := c.RepoClient.ListCommits()
 	if err != nil {
+		// ignoring check for local dir
+		if errors.Is(err, clients.ErrUnsupportedFeature) {
+			return sastCommits, nil
+		}
 		return sastCommits,
 			sce.WithMessage(sce.ErrScorecardInternal, fmt.Sprintf("RepoClient.ListCommits: %v", err))
 	}
diff --git a/checks/sast.go b/checks/sast.go
@@ -28,7 +28,10 @@ const CheckSAST = "SAST"
 
 //nolint:gochecknoinits
 func init() {
-	if err := registerCheck(CheckSAST, SAST, nil); err != nil {
+	supportedRequestTypes := []checker.RequestType{
+		checker.FileBased,
+	}
+	if err := registerCheck(CheckSAST, SAST, supportedRequestTypes); err != nil {
 		// This should never happen.
 		panic(err)
 	}
diff --git a/checks/security_policy.go b/checks/security_policy.go
@@ -30,6 +30,7 @@ const CheckSecurityPolicy = "Security-Policy"
 func init() {
 	supportedRequestTypes := []checker.RequestType{
 		checker.CommitBased,
+		checker.FileBased,
 	}
 	if err := registerCheck(CheckSecurityPolicy, SecurityPolicy, supportedRequestTypes); err != nil {
 		// This should never happen.
diff --git a/clients/localdir/client.go b/clients/localdir/client.go
@@ -34,12 +34,12 @@ import (
 )
 
 var (
-	_                clients.RepoClient = &localDirClient{}
+	_                clients.RepoClient = &Client{}
 	errInputRepoType                    = errors.New("input repo should be of type repoLocal")
 )
 
 //nolint:govet
-type localDirClient struct {
+type Client struct {
 	logger      *log.Logger
 	ctx         context.Context
 	path        string
@@ -50,7 +50,7 @@ type localDirClient struct {
 }
 
 // InitRepo sets up the local repo.
-func (client *localDirClient) InitRepo(inputRepo clients.Repo, commitSHA string, commitDepth int) error {
+func (client *Client) InitRepo(inputRepo clients.Repo, commitSHA string, commitDepth int) error {
 	localRepo, ok := inputRepo.(*Repo)
 	if !ok {
 		return fmt.Errorf("%w: %v", errInputRepoType, inputRepo)
@@ -66,12 +66,12 @@ func (client *localDirClient) InitRepo(inputRepo clients.Repo, commitSHA string,
 }
 
 // URI implements RepoClient.URI.
-func (client *localDirClient) URI() string {
+func (client *Client) URI() string {
 	return fmt.Sprintf("file://%s", client.path)
 }
 
 // IsArchived implements RepoClient.IsArchived.
-func (client *localDirClient) IsArchived() (bool, error) {
+func (client *Client) IsArchived() (bool, error) {
 	return false, fmt.Errorf("IsArchived: %w", clients.ErrUnsupportedFeature)
 }
 
@@ -148,7 +148,7 @@ func applyPredicate(
 }
 
 // LocalPath implements RepoClient.LocalPath.
-func (client *localDirClient) LocalPath() (string, error) {
+func (client *Client) LocalPath() (string, error) {
 	clientPath, err := filepath.Abs(client.path)
 	if err != nil {
 		return "", fmt.Errorf("error during filepath.Abs: %w", err)
@@ -157,7 +157,7 @@ func (client *localDirClient) LocalPath() (string, error) {
 }
 
 // ListFiles implements RepoClient.ListFiles.
-func (client *localDirClient) ListFiles(predicate func(string) (bool, error)) ([]string, error) {
+func (client *Client) ListFiles(predicate func(string) (bool, error)) ([]string, error) {
 	client.once.Do(func() {
 		client.files, client.errFiles = listFiles(client.path)
 	})
@@ -175,102 +175,103 @@ func getFile(clientpath, filename string) (*os.File, error) {
 }
 
 // GetFileReader implements RepoClient.GetFileReader.
-func (client *localDirClient) GetFileReader(filename string) (io.ReadCloser, error) {
+func (client *Client) GetFileReader(filename string) (io.ReadCloser, error) {
 	return getFile(client.path, filename)
 }
 
 // GetBranch implements RepoClient.GetBranch.
-func (client *localDirClient) GetBranch(branch string) (*clients.BranchRef, error) {
+func (client *Client) GetBranch(branch string) (*clients.BranchRef, error) {
 	return nil, fmt.Errorf("ListBranches: %w", clients.ErrUnsupportedFeature)
 }
 
 // GetDefaultBranch implements RepoClient.GetDefaultBranch.
-func (client *localDirClient) GetDefaultBranch() (*clients.BranchRef, error) {
+func (client *Client) GetDefaultBranch() (*clients.BranchRef, error) {
 	return nil, fmt.Errorf("GetDefaultBranch: %w", clients.ErrUnsupportedFeature)
 }
 
 // GetDefaultBranchName implements RepoClient.GetDefaultBranchName.
-func (client *localDirClient) GetDefaultBranchName() (string, error) {
+func (client *Client) GetDefaultBranchName() (string, error) {
 	return "", fmt.Errorf("GetDefaultBranchName: %w", clients.ErrUnsupportedFeature)
 }
 
 // ListCommits implements RepoClient.ListCommits.
-func (client *localDirClient) ListCommits() ([]clients.Commit, error) {
+func (client *Client) ListCommits() ([]clients.Commit, error) {
 	return nil, fmt.Errorf("ListCommits: %w", clients.ErrUnsupportedFeature)
 }
 
 // ListIssues implements RepoClient.ListIssues.
-func (client *localDirClient) ListIssues() ([]clients.Issue, error) {
+func (client *Client) ListIssues() ([]clients.Issue, error) {
 	return nil, fmt.Errorf("ListIssues: %w", clients.ErrUnsupportedFeature)
 }
 
 // ListReleases implements RepoClient.ListReleases.
-func (client *localDirClient) ListReleases() ([]clients.Release, error) {
+func (client *Client) ListReleases() ([]clients.Release, error) {
 	return nil, fmt.Errorf("ListReleases: %w", clients.ErrUnsupportedFeature)
 }
 
 // ListContributors implements RepoClient.ListContributors.
-func (client *localDirClient) ListContributors() ([]clients.User, error) {
+func (client *Client) ListContributors() ([]clients.User, error) {
 	return nil, fmt.Errorf("ListContributors: %w", clients.ErrUnsupportedFeature)
 }
 
 // ListSuccessfulWorkflowRuns implements RepoClient.WorkflowRunsByFilename.
-func (client *localDirClient) ListSuccessfulWorkflowRuns(filename string) ([]clients.WorkflowRun, error) {
+func (client *Client) ListSuccessfulWorkflowRuns(filename string) ([]clients.WorkflowRun, error) {
 	return nil, fmt.Errorf("ListSuccessfulWorkflowRuns: %w", clients.ErrUnsupportedFeature)
 }
 
 // ListCheckRunsForRef implements RepoClient.ListCheckRunsForRef.
-func (client *localDirClient) ListCheckRunsForRef(ref string) ([]clients.CheckRun, error) {
+func (client *Client) ListCheckRunsForRef(ref string) ([]clients.CheckRun, error) {
 	return nil, fmt.Errorf("ListCheckRunsForRef: %w", clients.ErrUnsupportedFeature)
 }
 
 // ListStatuses implements RepoClient.ListStatuses.
-func (client *localDirClient) ListStatuses(ref string) ([]clients.Status, error) {
+func (client *Client) ListStatuses(ref string) ([]clients.Status, error) {
 	return nil, fmt.Errorf("ListStatuses: %w", clients.ErrUnsupportedFeature)
 }
 
 // ListWebhooks implements RepoClient.ListWebhooks.
-func (client *localDirClient) ListWebhooks() ([]clients.Webhook, error) {
+func (client *Client) ListWebhooks() ([]clients.Webhook, error) {
 	return nil, fmt.Errorf("ListWebhooks: %w", clients.ErrUnsupportedFeature)
 }
 
 // Search implements RepoClient.Search.
-func (client *localDirClient) Search(request clients.SearchRequest) (clients.SearchResponse, error) {
+func (client *Client) Search(request clients.SearchRequest) (clients.SearchResponse, error) {
 	return clients.SearchResponse{}, fmt.Errorf("Search: %w", clients.ErrUnsupportedFeature)
 }
 
 // SearchCommits implements RepoClient.SearchCommits.
-func (client *localDirClient) SearchCommits(request clients.SearchCommitsOptions) ([]clients.Commit, error) {
+func (client *Client) SearchCommits(request clients.SearchCommitsOptions) ([]clients.Commit, error) {
 	return nil, fmt.Errorf("Search: %w", clients.ErrUnsupportedFeature)
 }
 
-func (client *localDirClient) Close() error {
+func (client *Client) Close() error {
 	return nil
 }
 
 // ListProgrammingLanguages implements RepoClient.ListProgrammingLanguages.
 // TODO: add ListProgrammingLanguages support for local directories.
-func (client *localDirClient) ListProgrammingLanguages() ([]clients.Language, error) {
-	return nil, fmt.Errorf("ListProgrammingLanguages: %w", clients.ErrUnsupportedFeature)
+func (client *Client) ListProgrammingLanguages() ([]clients.Language, error) {
+	// for now just return all programming languages
+	return []clients.Language{{Name: clients.All, NumLines: 1}}, nil
 }
 
 // ListLicenses implements RepoClient.ListLicenses.
 // TODO: add ListLicenses support for local directories.
-func (client *localDirClient) ListLicenses() ([]clients.License, error) {
+func (client *Client) ListLicenses() ([]clients.License, error) {
 	return nil, fmt.Errorf("ListLicenses: %w", clients.ErrUnsupportedFeature)
 }
 
-func (client *localDirClient) GetCreatedAt() (time.Time, error) {
+func (client *Client) GetCreatedAt() (time.Time, error) {
 	return time.Time{}, fmt.Errorf("GetCreatedAt: %w", clients.ErrUnsupportedFeature)
 }
 
-func (client *localDirClient) GetOrgRepoClient(ctx context.Context) (clients.RepoClient, error) {
+func (client *Client) GetOrgRepoClient(ctx context.Context) (clients.RepoClient, error) {
 	return nil, fmt.Errorf("GetOrgRepoClient: %w", clients.ErrUnsupportedFeature)
 }
 
 // CreateLocalDirClient returns a client which implements RepoClient interface.
 func CreateLocalDirClient(ctx context.Context, logger *log.Logger) clients.RepoClient {
-	return &localDirClient{
+	return &Client{
 		ctx:    ctx,
 		logger: logger,
 	}
diff --git a/cmd/root.go b/cmd/root.go
@@ -118,9 +118,11 @@ func rootCmd(o *options.Options) error {
 	}
 
 	var requiredRequestTypes []checker.RequestType
+	// if local option not set add file based
 	if o.Local != "" {
 		requiredRequestTypes = append(requiredRequestTypes, checker.FileBased)
 	}
+	// if commit option set to anything other than HEAD add commit based
 	if !strings.EqualFold(o.Commit, clients.HeadSHA) {
 		requiredRequestTypes = append(requiredRequestTypes, checker.CommitBased)
 	}
diff --git a/policy/policy_test.go b/policy/policy_test.go
@@ -333,7 +333,7 @@ func TestGetEnabled(t *testing.T) {
 			name:                  "request types limit enabled checks",
 			argsChecks:            []string{},
 			requiredRequestTypes:  []checker.RequestType{checker.FileBased, checker.CommitBased},
-			expectedEnabledChecks: 5, // All checks which are FileBased and CommitBased
+			expectedEnabledChecks: 7, // All checks which are FileBased and CommitBased
 			expectedError:         false,
 		},
 		{

Original file line number	Diff line number	Diff line change
`@@ -30,6 +30,7 @@ const CheckLicense = "License"`
`30`	`30`	`func init() {`
`31`	`31`	`supportedRequestTypes := []checker.RequestType{`
`32`	`32`	`checker.CommitBased,`
	`33`	`+ checker.FileBased,`
`33`	`34`	`}`
`34`	`35`	`if err := registerCheck(CheckLicense, License, supportedRequestTypes); err != nil {`
`35`	`36`	`// this should never happen`
Original file line number	Diff line number	Diff line change
`@@ -30,6 +30,7 @@ const CheckSecurityPolicy = "Security-Policy"`
`30`	`30`	`func init() {`
`31`	`31`	`supportedRequestTypes := []checker.RequestType{`
`32`	`32`	`checker.CommitBased,`
	`33`	`+ checker.FileBased,`
`33`	`34`	`}`
`34`	`35`	`if err := registerCheck(CheckSecurityPolicy, SecurityPolicy, supportedRequestTypes); err != nil {`
`35`	`36`	`// This should never happen.`
Original file line number	Diff line number	Diff line change
`@@ -34,12 +34,12 @@ import (`
`34`	`34`	`)`
`35`	`35`
`36`	`36`	`var (`
`37`		`- _ clients.RepoClient = &localDirClient{}`
	`37`	`+ _ clients.RepoClient = &Client{}`
`38`	`38`	`errInputRepoType = errors.New("input repo should be of type repoLocal")`
`39`	`39`	`)`
`40`	`40`
`41`	`41`	`//nolint:govet`
`42`		`-type localDirClient struct {`
	`42`	`+type Client struct {`
`43`	`43`	`logger *log.Logger`
`44`	`44`	`ctx context.Context`
`45`	`45`	`path string`
`@@ -50,7 +50,7 @@ type localDirClient struct {`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`// InitRepo sets up the local repo.`
`53`		`-func (client *localDirClient) InitRepo(inputRepo clients.Repo, commitSHA string, commitDepth int) error {`
	`53`	`+func (client *Client) InitRepo(inputRepo clients.Repo, commitSHA string, commitDepth int) error {`
`54`	`54`	`localRepo, ok := inputRepo.(*Repo)`
`55`	`55`	`if !ok {`
`56`	`56`	`return fmt.Errorf("%w: %v", errInputRepoType, inputRepo)`
`@@ -66,12 +66,12 @@ func (client *localDirClient) InitRepo(inputRepo clients.Repo, commitSHA string,`
`66`	`66`	`}`
`67`	`67`
`68`	`68`	`// URI implements RepoClient.URI.`
`69`		`-func (client *localDirClient) URI() string {`
	`69`	`+func (client *Client) URI() string {`
`70`	`70`	`return fmt.Sprintf("file://%s", client.path)`
`71`	`71`	`}`
`72`	`72`
`73`	`73`	`// IsArchived implements RepoClient.IsArchived.`
`74`		`-func (client *localDirClient) IsArchived() (bool, error) {`
	`74`	`+func (client *Client) IsArchived() (bool, error) {`
`75`	`75`	`return false, fmt.Errorf("IsArchived: %w", clients.ErrUnsupportedFeature)`
`76`	`76`	`}`
`77`	`77`
`@@ -148,7 +148,7 @@ func applyPredicate(`
`148`	`148`	`}`
`149`	`149`
`150`	`150`	`// LocalPath implements RepoClient.LocalPath.`
`151`		`-func (client *localDirClient) LocalPath() (string, error) {`
	`151`	`+func (client *Client) LocalPath() (string, error) {`
`152`	`152`	`clientPath, err := filepath.Abs(client.path)`
`153`	`153`	`if err != nil {`
`154`	`154`	`return "", fmt.Errorf("error during filepath.Abs: %w", err)`
`@@ -157,7 +157,7 @@ func (client *localDirClient) LocalPath() (string, error) {`
`157`	`157`	`}`
`158`	`158`
`159`	`159`	`// ListFiles implements RepoClient.ListFiles.`
`160`		`-func (client *localDirClient) ListFiles(predicate func(string) (bool, error)) ([]string, error) {`
	`160`	`+func (client *Client) ListFiles(predicate func(string) (bool, error)) ([]string, error) {`
`161`	`161`	`client.once.Do(func() {`
`162`	`162`	`client.files, client.errFiles = listFiles(client.path)`
`163`	`163`	`})`
`@@ -175,102 +175,103 @@ func getFile(clientpath, filename string) (*os.File, error) {`
`175`	`175`	`}`
`176`	`176`
`177`	`177`	`// GetFileReader implements RepoClient.GetFileReader.`
`178`		`-func (client *localDirClient) GetFileReader(filename string) (io.ReadCloser, error) {`
	`178`	`+func (client *Client) GetFileReader(filename string) (io.ReadCloser, error) {`
`179`	`179`	`return getFile(client.path, filename)`
`180`	`180`	`}`
`181`	`181`
`182`	`182`	`// GetBranch implements RepoClient.GetBranch.`
`183`		`-func (client localDirClient) GetBranch(branch string) (clients.BranchRef, error) {`
	`183`	`+func (client Client) GetBranch(branch string) (clients.BranchRef, error) {`
`184`	`184`	`return nil, fmt.Errorf("ListBranches: %w", clients.ErrUnsupportedFeature)`
`185`	`185`	`}`
`186`	`186`
`187`	`187`	`// GetDefaultBranch implements RepoClient.GetDefaultBranch.`
`188`		`-func (client localDirClient) GetDefaultBranch() (clients.BranchRef, error) {`
	`188`	`+func (client Client) GetDefaultBranch() (clients.BranchRef, error) {`
`189`	`189`	`return nil, fmt.Errorf("GetDefaultBranch: %w", clients.ErrUnsupportedFeature)`
`190`	`190`	`}`
`191`	`191`
`192`	`192`	`// GetDefaultBranchName implements RepoClient.GetDefaultBranchName.`
`193`		`-func (client *localDirClient) GetDefaultBranchName() (string, error) {`
	`193`	`+func (client *Client) GetDefaultBranchName() (string, error) {`
`194`	`194`	`return "", fmt.Errorf("GetDefaultBranchName: %w", clients.ErrUnsupportedFeature)`
`195`	`195`	`}`
`196`	`196`
`197`	`197`	`// ListCommits implements RepoClient.ListCommits.`
`198`		`-func (client *localDirClient) ListCommits() ([]clients.Commit, error) {`
	`198`	`+func (client *Client) ListCommits() ([]clients.Commit, error) {`
`199`	`199`	`return nil, fmt.Errorf("ListCommits: %w", clients.ErrUnsupportedFeature)`
`200`	`200`	`}`
`201`	`201`
`202`	`202`	`// ListIssues implements RepoClient.ListIssues.`
`203`		`-func (client *localDirClient) ListIssues() ([]clients.Issue, error) {`
	`203`	`+func (client *Client) ListIssues() ([]clients.Issue, error) {`
`204`	`204`	`return nil, fmt.Errorf("ListIssues: %w", clients.ErrUnsupportedFeature)`
`205`	`205`	`}`
`206`	`206`
`207`	`207`	`// ListReleases implements RepoClient.ListReleases.`
`208`		`-func (client *localDirClient) ListReleases() ([]clients.Release, error) {`
	`208`	`+func (client *Client) ListReleases() ([]clients.Release, error) {`
`209`	`209`	`return nil, fmt.Errorf("ListReleases: %w", clients.ErrUnsupportedFeature)`
`210`	`210`	`}`
`211`	`211`
`212`	`212`	`// ListContributors implements RepoClient.ListContributors.`
`213`		`-func (client *localDirClient) ListContributors() ([]clients.User, error) {`
	`213`	`+func (client *Client) ListContributors() ([]clients.User, error) {`
`214`	`214`	`return nil, fmt.Errorf("ListContributors: %w", clients.ErrUnsupportedFeature)`
`215`	`215`	`}`
`216`	`216`
`217`	`217`	`// ListSuccessfulWorkflowRuns implements RepoClient.WorkflowRunsByFilename.`
`218`		`-func (client *localDirClient) ListSuccessfulWorkflowRuns(filename string) ([]clients.WorkflowRun, error) {`
	`218`	`+func (client *Client) ListSuccessfulWorkflowRuns(filename string) ([]clients.WorkflowRun, error) {`
`219`	`219`	`return nil, fmt.Errorf("ListSuccessfulWorkflowRuns: %w", clients.ErrUnsupportedFeature)`
`220`	`220`	`}`
`221`	`221`
`222`	`222`	`// ListCheckRunsForRef implements RepoClient.ListCheckRunsForRef.`
`223`		`-func (client *localDirClient) ListCheckRunsForRef(ref string) ([]clients.CheckRun, error) {`
	`223`	`+func (client *Client) ListCheckRunsForRef(ref string) ([]clients.CheckRun, error) {`
`224`	`224`	`return nil, fmt.Errorf("ListCheckRunsForRef: %w", clients.ErrUnsupportedFeature)`
`225`	`225`	`}`
`226`	`226`
`227`	`227`	`// ListStatuses implements RepoClient.ListStatuses.`
`228`		`-func (client *localDirClient) ListStatuses(ref string) ([]clients.Status, error) {`
	`228`	`+func (client *Client) ListStatuses(ref string) ([]clients.Status, error) {`
`229`	`229`	`return nil, fmt.Errorf("ListStatuses: %w", clients.ErrUnsupportedFeature)`
`230`	`230`	`}`
`231`	`231`
`232`	`232`	`// ListWebhooks implements RepoClient.ListWebhooks.`
`233`		`-func (client *localDirClient) ListWebhooks() ([]clients.Webhook, error) {`
	`233`	`+func (client *Client) ListWebhooks() ([]clients.Webhook, error) {`
`234`	`234`	`return nil, fmt.Errorf("ListWebhooks: %w", clients.ErrUnsupportedFeature)`
`235`	`235`	`}`
`236`	`236`
`237`	`237`	`// Search implements RepoClient.Search.`
`238`		`-func (client *localDirClient) Search(request clients.SearchRequest) (clients.SearchResponse, error) {`
	`238`	`+func (client *Client) Search(request clients.SearchRequest) (clients.SearchResponse, error) {`
`239`	`239`	`return clients.SearchResponse{}, fmt.Errorf("Search: %w", clients.ErrUnsupportedFeature)`
`240`	`240`	`}`
`241`	`241`
`242`	`242`	`// SearchCommits implements RepoClient.SearchCommits.`
`243`		`-func (client *localDirClient) SearchCommits(request clients.SearchCommitsOptions) ([]clients.Commit, error) {`
	`243`	`+func (client *Client) SearchCommits(request clients.SearchCommitsOptions) ([]clients.Commit, error) {`
`244`	`244`	`return nil, fmt.Errorf("Search: %w", clients.ErrUnsupportedFeature)`
`245`	`245`	`}`
`246`	`246`
`247`		`-func (client *localDirClient) Close() error {`
	`247`	`+func (client *Client) Close() error {`
`248`	`248`	`return nil`
`249`	`249`	`}`
`250`	`250`
`251`	`251`	`// ListProgrammingLanguages implements RepoClient.ListProgrammingLanguages.`
`252`	`252`	`// TODO: add ListProgrammingLanguages support for local directories.`
`253`		`-func (client *localDirClient) ListProgrammingLanguages() ([]clients.Language, error) {`
`254`		`- return nil, fmt.Errorf("ListProgrammingLanguages: %w", clients.ErrUnsupportedFeature)`
	`253`	`+func (client *Client) ListProgrammingLanguages() ([]clients.Language, error) {`
	`254`	`+ // for now just return all programming languages`
	`255`	`+ return []clients.Language{{Name: clients.All, NumLines: 1}}, nil`
`255`	`256`	`}`
`256`	`257`
`257`	`258`	`// ListLicenses implements RepoClient.ListLicenses.`
`258`	`259`	`// TODO: add ListLicenses support for local directories.`
`259`		`-func (client *localDirClient) ListLicenses() ([]clients.License, error) {`
	`260`	`+func (client *Client) ListLicenses() ([]clients.License, error) {`
`260`	`261`	`return nil, fmt.Errorf("ListLicenses: %w", clients.ErrUnsupportedFeature)`
`261`	`262`	`}`
`262`	`263`
`263`		`-func (client *localDirClient) GetCreatedAt() (time.Time, error) {`
	`264`	`+func (client *Client) GetCreatedAt() (time.Time, error) {`
`264`	`265`	`return time.Time{}, fmt.Errorf("GetCreatedAt: %w", clients.ErrUnsupportedFeature)`
`265`	`266`	`}`
`266`	`267`
`267`		`-func (client *localDirClient) GetOrgRepoClient(ctx context.Context) (clients.RepoClient, error) {`
	`268`	`+func (client *Client) GetOrgRepoClient(ctx context.Context) (clients.RepoClient, error) {`
`268`	`269`	`return nil, fmt.Errorf("GetOrgRepoClient: %w", clients.ErrUnsupportedFeature)`
`269`	`270`	`}`
`270`	`271`
`271`	`272`	`// CreateLocalDirClient returns a client which implements RepoClient interface.`
`272`	`273`	`func CreateLocalDirClient(ctx context.Context, logger *log.Logger) clients.RepoClient {`
`273`		`- return &localDirClient{`
	`274`	`+ return &Client{`
`274`	`275`	`ctx: ctx,`
`275`	`276`	`logger: logger,`
`276`	`277`	`}`
Original file line number	Diff line number	Diff line change
`@@ -118,9 +118,11 @@ func rootCmd(o *options.Options) error {`
`118`	`118`	`}`
`119`	`119`
`120`	`120`	`var requiredRequestTypes []checker.RequestType`
	`121`	`+ // if local option not set add file based`
`121`	`122`	`if o.Local != "" {`
`122`	`123`	`requiredRequestTypes = append(requiredRequestTypes, checker.FileBased)`
`123`	`124`	`}`
	`125`	`+ // if commit option set to anything other than HEAD add commit based`
`124`	`126`	`if !strings.EqualFold(o.Commit, clients.HeadSHA) {`
`125`	`127`	`requiredRequestTypes = append(requiredRequestTypes, checker.CommitBased)`
`126`	`128`	`}`
Original file line number	Diff line number	Diff line change
`@@ -333,7 +333,7 @@ func TestGetEnabled(t *testing.T) {`
`333`	`333`	`name: "request types limit enabled checks",`
`334`	`334`	`argsChecks: []string{},`
`335`	`335`	`requiredRequestTypes: []checker.RequestType{checker.FileBased, checker.CommitBased},`
`336`		`- expectedEnabledChecks: 5, // All checks which are FileBased and CommitBased`
	`336`	`+ expectedEnabledChecks: 7, // All checks which are FileBased and CommitBased`
`337`	`337`	`expectedError: false,`
`338`	`338`	`},`
`339`	`339`	`{`