Merge pull request #20 from padok-team/feat/required-provider-version

feat(static): add required provider version operator check

checks/required_provider_version_operator.go

@@ -0,0 +1,55 @@
+package checks
+
+import (
+	"guacamole/data"
+	"guacamole/helpers"
+	"regexp"
+
+	"github.com/hashicorp/terraform-config-inspect/tfconfig"
+)
+
+func RequiredProviderVersionOperatorInModules() (data.Check, error) {
+	name := "Required provider versions in modules should be set with ~> operator"
+	relatedGuidelines := "https://t.ly/0bxoy"
+	modules, err := helpers.GetModules()
+	if err != nil {
+		return data.Check{}, err
+	}
+	requiredProvidersInError := []string{}
+
+	pattern := `~>`
+	matcher, err := regexp.Compile(pattern)
+	if err != nil {
+		return data.Check{}, err
+	}
+
+	// For each module, check if the provider is defined
+	for _, module := range modules {
+		moduleConf, diags := tfconfig.LoadModule(module.FullPath)
+		if diags.HasErrors() {
+			return data.Check{}, diags.Err()
+		}
+
+		for _, requiredProvider := range moduleConf.RequiredProviders {
+			for _, versionConstraint := range requiredProvider.VersionConstraints {
+				matched := matcher.MatchString(versionConstraint)
+				if !matched {
+					requiredProvidersInError = append(requiredProvidersInError, requiredProvider.Source+" --> "+versionConstraint)
+				}
+			}
+		}
+	}
+
+	dataCheck := data.Check{
+		Name:              name,
+		RelatedGuidelines: relatedGuidelines,
+		Status:            "✅",
+		Errors:            requiredProvidersInError,
+	}
+
+	if len(requiredProvidersInError) > 0 {
+		dataCheck.Status = "❌"
+	}
+
+	return dataCheck, nil
+}

checks/static_checks.go

@@ -8,13 +8,14 @@ import (
 func StaticChecks() []data.Check {
 	// Add static checks here
 	checks := map[string]func() (data.Check, error){
-		"ProviderInModule":      ProviderInModule,
-		"Stuttering":            Stuttering,
-		"SnakeCase":             SnakeCase,
-		"MissingVarDescription": MissingVarDescription,
-		"VarNumberMatchesType":  VarNumberMatchesType,
-		"VariableTypeAny":       VariableTypeAny,
-		"RemoteModuleVersion":   RemoteModuleVersion,
+		"ProviderInModule":                         ProviderInModule,
+		"Stuttering":                               Stuttering,
+		"SnakeCase":                                SnakeCase,
+		"MissingVarDescription":                    MissingVarDescription,
+		"VarNumberMatchesType":                     VarNumberMatchesType,
+		"VariableTypeAny":                          VariableTypeAny,
+		"RemoteModuleVersion":                      RemoteModuleVersion,
+		"RequiredProviderVersionOperatorInModules": RequiredProviderVersionOperatorInModules,
 	}
 
 	var checkResults []data.Check