command/show: continued work on `terraform show -json` output (#20171)

* command/jsonstate: do not hide SchemaVersion of '0'
* command/jsonconfig: module_calls should be a map
* command/jsonplan: include current terraform version in output
* command/jsonconfig: properly marshal expressions from a module call

Previously this was looking at the root module's variables, instead of
the child module variables, to build the module schema. This fixes that
bug.

command/jsonconfig/config.go

@@ -35,7 +35,7 @@ type module struct {
 	// Resources are sorted in a user-friendly order that is undefined at this
 	// time, but consistent.
 	Resources   []resource            `json:"resources,omitempty"`
-	ModuleCalls []moduleCall `json:"module_calls,omitempty"`
+	ModuleCalls map[string]moduleCall `json:"module_calls,omitempty"`
 }
 
 type moduleCall struct {
@@ -161,37 +161,39 @@ func marshalModule(c *configs.Config, schemas *terraform.Schemas) (module, error
 	return module, nil
 }
 
-func marshalModuleCalls(c *configs.Config, schemas *terraform.Schemas) []moduleCall {
-	var ret []moduleCall
-	for _, v := range c.Module.ModuleCalls {
-		mc := moduleCall{
-			ResolvedSource: v.SourceAddr,
+func marshalModuleCalls(c *configs.Config, schemas *terraform.Schemas) map[string]moduleCall {
+	ret := make(map[string]moduleCall)
+	for _, mc := range c.Module.ModuleCalls {
+		retMC := moduleCall{
+			ResolvedSource: mc.SourceAddr,
 		}
-		cExp := marshalExpression(v.Count)
+		cExp := marshalExpression(mc.Count)
 		if !cExp.Empty() {
-			mc.CountExpression = &cExp
+			retMC.CountExpression = &cExp
 		} else {
-			fExp := marshalExpression(v.ForEach)
+			fExp := marshalExpression(mc.ForEach)
 			if !fExp.Empty() {
-				mc.ForEachExpression = &fExp
+				retMC.ForEachExpression = &fExp
 			}
 		}
 
+		// get the called module's variables so we can build up the expressions
+		childModule := c.Children[mc.Name]
 		schema := &configschema.Block{}
 		schema.Attributes = make(map[string]*configschema.Attribute)
-		for _, variable := range c.Module.Variables {
+		for _, variable := range childModule.Module.Variables {
 			schema.Attributes[variable.Name] = &configschema.Attribute{
 				Required: variable.Default == cty.NilVal,
 			}
 		}
-		mc.Expressions = marshalExpressions(v.Config, schema)
+
+		retMC.Expressions = marshalExpressions(mc.Config, schema)
 
 		for _, cc := range c.Children {
 			childModule, _ := marshalModule(cc, schemas)
-			mc.Module = childModule
+			retMC.Module = childModule
 		}
-		ret = append(ret, mc)
-
+		ret[mc.Name] = retMC
 	}
 
 	return ret

command/jsonplan/plan.go

@@ -16,6 +16,7 @@ import (
 	"github.com/hashicorp/terraform/states"
 	"github.com/hashicorp/terraform/states/statefile"
 	"github.com/hashicorp/terraform/terraform"
+	"github.com/hashicorp/terraform/version"
 )
 
 // FormatVersion represents the version of the json format and will be
@@ -27,6 +28,7 @@ const FormatVersion = "0.1"
 // the complete config and current state.
 type plan struct {
 	FormatVersion    string      `json:"format_version,omitempty"`
+	TerraformVersion string      `json:"terraform_version,omitempty"`
 	PlannedValues    stateValues `json:"planned_values,omitempty"`
 	// ResourceChanges are sorted in a user-friendly order that is undefined at
 	// this time, but consistent.
@@ -83,6 +85,7 @@ func Marshal(
 ) ([]byte, error) {
 
 	output := newPlan()
+	output.TerraformVersion = version.String()
 
 	// output.PlannedValues
 	err := output.marshalPlannedValues(p.Changes, schemas)

command/jsonstate/state.go

@@ -77,7 +77,7 @@ type resource struct {
 
 	// SchemaVersion indicates which version of the resource type schema the
 	// "values" property conforms to.
-	SchemaVersion uint64 `json:"schema_version,omitempty"`
+	SchemaVersion uint64 `json:"schema_version"`
 
 	// AttributeValues is the JSON representation of the attribute values of the
 	// resource, whose structure depends on the resource type schema. Any