chore: dynamic elements break preset validation

Dynamic elements are not taken into account during preset validation.
We should omit preset validation if it's not accurate

Author: Emyrk

preset.go

@@ -2,7 +2,6 @@ package preview
 
 import (
 	"fmt"
-	"slices"
 
 	"github.com/aquasecurity/trivy/pkg/iac/terraform"
 	"github.com/hashicorp/hcl/v2"
@@ -32,24 +31,6 @@ func presets(modules terraform.Modules, parameters []types.Parameter) []types.Pr
 				defaultPreset = &preset
 			}
 
-			for paramName, paramValue := range preset.Parameters {
-				templateParamIndex := slices.IndexFunc(parameters, func(p types.Parameter) bool {
-					return p.Name == paramName
-				})
-				if templateParamIndex == -1 {
-					preset.Diagnostics = append(preset.Diagnostics, &hcl.Diagnostic{
-						Severity: hcl.DiagError,
-						Summary:  "Undefined Parameter",
-						Detail:   fmt.Sprintf("Preset parameter %q is not defined by the template.", paramName),
-					})
-					continue
-				}
-				templateParam := parameters[templateParamIndex]
-				for _, diag := range templateParam.Valid(types.StringLiteral(paramValue)) {
-					preset.Diagnostics = append(preset.Diagnostics, diag)
-				}
-			}
-
 			foundPresets = append(foundPresets, preset)
 		}
 	}

preview_test.go

@@ -714,6 +714,17 @@ func TestPresetValidation(t *testing.T) {
 				"not_prebuild":           aPre().noDiagnostics().prebuildCount(0),
 			},
 		},
+		{
+			name:  "preset ok",
+			dir:   "presetok",
+			input: preview.Input{},
+			presetAssert: map[string]assertPreset{
+				"valid_preset": aPre().
+					value("use_custom_image", "true").
+					value("custom_image_url", "docker.io/codercom/test:latest").
+					noDiagnostics(),
+			},
+		},
 	} {
 		t.Run(tc.name, func(t *testing.T) {
 			t.Parallel()

testdata/presetok/main.tf

@@ -0,0 +1,32 @@
+terraform {
+  required_providers {
+    coder = {
+      source = "coder/coder"
+      version = "2.8.0"
+    }
+  }
+}
+
+data "coder_parameter" "use_custom_image" {
+  name    = "use_custom_image"
+  type    = "bool"
+  default = "false"
+}
+
+data "coder_parameter" "custom_image_url" {
+  count   = data.coder_parameter.use_custom_image.value == "true" ? 1 : 0
+  name    = "custom_image_url"
+  type    = "string"
+  # No default - required when shown
+}
+
+data "coder_workspace_preset" "valid_preset" {
+  name = "valid_preset"
+  parameters = {
+    "use_custom_image" = "true"
+    "custom_image_url" = "docker.io/codercom/test:latest"
+  }
+  prebuilds {
+    instances = 1
+  }
+}