check more specifically

johnstcn · johnstcn · commit 22cd79a0aa71 · 2025-01-03T12:11:05.000Z
diff --git a/provisioner/terraform/tfparse/tfparse.go b/provisioner/terraform/tfparse/tfparse.go
@@ -268,7 +268,7 @@ func (p *Parser) VariableDefaults(ctx context.Context, tags map[string]string) (
 			}
 		}
 		if !found {
-			skipped = append(skipped, v.Name)
+			skipped = append(skipped, "var."+v.Name)
 			continue
 		}
 		sv, err := interfaceToString(v.Default)
@@ -306,13 +306,14 @@ func (p *Parser) CoderParameterDefaults(ctx context.Context, varsDefaults map[st
 		}
 
 		var found bool
+		needle := strings.Join([]string{"data", dataResource.Type, dataResource.Name}, ".")
 		for _, tv := range tags {
-			if strings.Contains(tv, dataResource.Name) {
+			if strings.Contains(tv, needle) {
 				found = true
 			}
 		}
 		if !found {
-			skipped = append(skipped, dataResource.Name)
+			skipped = append(skipped, needle)
 			continue
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -268,7 +268,7 @@ func (p *Parser) VariableDefaults(ctx context.Context, tags map[string]string) (`
`268`	`268`	`}`
`269`	`269`	`}`
`270`	`270`	`if !found {`
`271`		`- skipped = append(skipped, v.Name)`
	`271`	`+ skipped = append(skipped, "var."+v.Name)`
`272`	`272`	`continue`
`273`	`273`	`}`
`274`	`274`	`sv, err := interfaceToString(v.Default)`
`@@ -306,13 +306,14 @@ func (p *Parser) CoderParameterDefaults(ctx context.Context, varsDefaults map[st`
`306`	`306`	`}`
`307`	`307`
`308`	`308`	`var found bool`
	`309`	`+ needle := strings.Join([]string{"data", dataResource.Type, dataResource.Name}, ".")`
`309`	`310`	`for _, tv := range tags {`
`310`		`- if strings.Contains(tv, dataResource.Name) {`
	`311`	`+ if strings.Contains(tv, needle) {`
`311`	`312`	`found = true`
`312`	`313`	`}`
`313`	`314`	`}`
`314`	`315`	`if !found {`
`315`		`- skipped = append(skipped, dataResource.Name)`
	`316`	`+ skipped = append(skipped, needle)`
`316`	`317`	`continue`
`317`	`318`	`}`
`318`	`319`