unexport methods that are internal

Author: Emyrk

owner.go

@@ -10,7 +10,7 @@ import (
 	"golang.org/x/xerrors"
 )
 
-func WorkspaceOwnerHook(dfs fs.FS, input Input) (func(ctx *tfcontext.Context, blocks terraform.Blocks, inputVars map[string]cty.Value), error) {
+func workspaceOwnerHook(dfs fs.FS, input Input) (func(ctx *tfcontext.Context, blocks terraform.Blocks, inputVars map[string]cty.Value), error) {
 	if input.Owner.Groups == nil {
 		input.Owner.Groups = []string{}
 	}

parameter.go

@@ -11,7 +11,7 @@ import (
 	"github.com/coder/preview/types"
 )
 
-func RichParameters(modules terraform.Modules) ([]types.Parameter, hcl.Diagnostics) {
+func parameters(modules terraform.Modules) ([]types.Parameter, hcl.Diagnostics) {
 	diags := make(hcl.Diagnostics, 0)
 	params := make([]types.Parameter, 0)
 	exists := make(map[string][]types.Parameter)

paramhook.go

@@ -8,13 +8,13 @@ import (
 	"github.com/coder/preview/hclext"
 )
 
-// ParameterContextsEvalHook is called in a loop, so if parameters affect
+// parameterContextsEvalHook is called in a loop, so if parameters affect
 // other parameters, this can solve the problem 1 "evaluation" at a time.
 //
 // Omitting to set a default value is OK, as long as at least 1 parameter
 // is resolvable. The resolvable parameter will be accessible on the next
 // iteration.
-func ParameterContextsEvalHook(input Input) func(ctx *tfcontext.Context, blocks terraform.Blocks, inputVars map[string]cty.Value) {
+func parameterContextsEvalHook(input Input) func(ctx *tfcontext.Context, blocks terraform.Blocks, inputVars map[string]cty.Value) {
 	return func(ctx *tfcontext.Context, blocks terraform.Blocks, inputVars map[string]cty.Value) {
 		data := blocks.OfType("data")
 		for _, block := range data {

plan.go

@@ -10,7 +10,6 @@ import (
 	"slices"
 	"strings"
 
-	"github.com/aquasecurity/trivy/pkg/iac/scanners/terraformplan/tfjson/parser"
 	"github.com/aquasecurity/trivy/pkg/iac/terraform"
 	tfcontext "github.com/aquasecurity/trivy/pkg/iac/terraform/context"
 	tfjson "github.com/hashicorp/terraform-json"
@@ -20,7 +19,7 @@ import (
 	"github.com/coder/preview/hclext"
 )
 
-func PlanJSONHook(dfs fs.FS, input Input) (func(ctx *tfcontext.Context, blocks terraform.Blocks, inputVars map[string]cty.Value), error) {
+func planJSONHook(dfs fs.FS, input Input) (func(ctx *tfcontext.Context, blocks terraform.Blocks, inputVars map[string]cty.Value), error) {
 	var contents io.Reader = bytes.NewReader(input.PlanJSON)
 	// Also accept `{}` as an empty plan. If this is stored in postgres or another json
 	// type, then `{}` is the "empty" value.
@@ -36,7 +35,7 @@ func PlanJSONHook(dfs fs.FS, input Input) (func(ctx *tfcontext.Context, blocks t
 		}
 	}
 
-	plan, err := ParsePlanJSON(contents)
+	plan, err := parsePlanJSON(contents)
 	if err != nil {
 		return nil, fmt.Errorf("unable to parse plan JSON: %w", err)
 	}
@@ -216,28 +215,15 @@ func toCtyValue(a any) (cty.Value, error) {
 	}
 }
 
-// ParsePlanJSON can parse the JSON output of a Terraform plan.
+// parsePlanJSON can parse the JSON output of a Terraform plan.
 // terraform plan out.plan
 // terraform show -json out.plan
-func ParsePlanJSON(reader io.Reader) (*tfjson.Plan, error) {
+func parsePlanJSON(reader io.Reader) (*tfjson.Plan, error) {
 	plan := new(tfjson.Plan)
 	plan.FormatVersion = tfjson.PlanFormatVersionConstraints
 	return plan, json.NewDecoder(reader).Decode(plan)
 }
 
-// ParsePlanJSON can parse the JSON output of a Terraform plan.
-// terraform plan out.plan
-// terraform show -json out.plan
-func TrivyParsePlanJSON(reader io.Reader) (*tfjson.Plan, error) {
-	p := parser.New()
-	plan, err := p.Parse(reader)
-	var _ = plan
-
-	plan.ToFS()
-
-	return nil, err
-}
-
 func keyMatcher(key cty.Value) func(to any) bool {
 	switch {
 	case key.Type().Equals(cty.Number):

preview.go

@@ -64,7 +64,7 @@ func Preview(ctx context.Context, input Input, dir fs.FS) (mainOutput *Output, d
 		}
 	}
 
-	planHook, err := PlanJSONHook(dir, input)
+	planHook, err := planJSONHook(dir, input)
 	if err != nil {
 		return nil, hcl.Diagnostics{
 			{
@@ -75,7 +75,7 @@ func Preview(ctx context.Context, input Input, dir fs.FS) (mainOutput *Output, d
 		}
 	}
 
-	ownerHook, err := WorkspaceOwnerHook(dir, input)
+	ownerHook, err := workspaceOwnerHook(dir, input)
 	if err != nil {
 		return nil, hcl.Diagnostics{
 			{
@@ -95,7 +95,7 @@ func Preview(ctx context.Context, input Input, dir fs.FS) (mainOutput *Output, d
 		parser.OptionWithTFVarsPaths(varFiles...),
 		parser.OptionWithEvalHook(planHook),
 		parser.OptionWithEvalHook(ownerHook),
-		parser.OptionWithEvalHook(ParameterContextsEvalHook(input)),
+		parser.OptionWithEvalHook(parameterContextsEvalHook(input)),
 	)
 
 	err = p.ParseFS(ctx, ".")
@@ -121,8 +121,8 @@ func Preview(ctx context.Context, input Input, dir fs.FS) (mainOutput *Output, d
 	}
 
 	diags := make(hcl.Diagnostics, 0)
-	rp, rpDiags := RichParameters(modules)
-	tags, tagDiags := WorkspaceTags(modules, p.Files())
+	rp, rpDiags := parameters(modules)
+	tags, tagDiags := workspaceTags(modules, p.Files())
 
 	// Add warnings
 	diags = diags.Extend(warnings(modules))

workspacetags.go

@@ -10,7 +10,7 @@ import (
 	"github.com/coder/preview/types"
 )
 
-func WorkspaceTags(modules terraform.Modules, files map[string]*hcl.File) (types.TagBlocks, hcl.Diagnostics) {
+func workspaceTags(modules terraform.Modules, files map[string]*hcl.File) (types.TagBlocks, hcl.Diagnostics) {
 	diags := make(hcl.Diagnostics, 0)
 	tagBlocks := make(types.TagBlocks, 0)
 
@@ -65,7 +65,7 @@ func WorkspaceTags(modules terraform.Modules, files map[string]*hcl.File) (types
 			var tags []types.Tag
 			tagsValue.ForEachElement(func(key cty.Value, val cty.Value) (stop bool) {
 				r := tagsAttr.HCLAttribute().Expr.Range()
-				tag, tagDiag := NewTag(&r, files, key, val)
+				tag, tagDiag := newTag(&r, files, key, val)
 				if tagDiag != nil {
 					diags = diags.Append(tagDiag)
 					return false
@@ -76,7 +76,7 @@ func WorkspaceTags(modules terraform.Modules, files map[string]*hcl.File) (types
 				return false
 			})
 			//for _, item := range tagsObj.Items {
-			//	tag, tagDiag := NewTag(tagsObj, files, item, evCtx)
+			//	tag, tagDiag := newTag(tagsObj, files, item, evCtx)
 			//	if tagDiag != nil {
 			//		diags = diags.Append(tagDiag)
 			//		continue
@@ -94,8 +94,8 @@ func WorkspaceTags(modules terraform.Modules, files map[string]*hcl.File) (types
 	return tagBlocks, diags
 }
 
-// NewTag creates a workspace tag from its hcl expression.
-func NewTag(srcRange *hcl.Range, files map[string]*hcl.File, key, val cty.Value) (types.Tag, *hcl.Diagnostic) {
+// newTag creates a workspace tag from its hcl expression.
+func newTag(srcRange *hcl.Range, files map[string]*hcl.File, key, val cty.Value) (types.Tag, *hcl.Diagnostic) {
 	//key, kdiags := expr.KeyExpr.Value(evCtx)
 	//val, vdiags := expr.ValueExpr.Value(evCtx)