Switch condition and rule from ListNestedAttribute to ListNestedBlock

Fixes "Unsupported block type" error by using schema.ListNestedBlock
for rule and condition, which supports the natural HCL block syntax.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: 2 people

internal/provider/asset_routing_common.go

@@ -18,26 +18,31 @@ type AssetRoutingConditionModel struct {
 	Key      types.String `tfsdk:"key"`
 }
 
-// assetRoutingConditionSchemaAttributes returns the schema attributes for a routing condition,
+// assetRoutingConditionSchemaBlock returns a ListNestedBlock for routing conditions,
 // reusable by both the table and rule resources.
-func assetRoutingConditionSchemaAttributes() map[string]schema.Attribute {
-	return map[string]schema.Attribute{
-		"field": schema.StringAttribute{
-			MarkdownDescription: "The field to match on. Valid values: `HOSTNAME`, `PLATFORM`, `LABEL`.",
-			Required:            true,
-		},
-		"operator": schema.StringAttribute{
-			MarkdownDescription: "The comparison operator. Valid values: `EQUAL`, `NOT_EQUAL`, `CONTAINS`, `MATCHES`.",
-			Required:            true,
-		},
-		"values": schema.ListAttribute{
-			MarkdownDescription: "List of values to match against. A condition matches if the field matches any of the listed values (OR logic).",
-			Required:            true,
-			ElementType:         types.StringType,
-		},
-		"key": schema.StringAttribute{
-			MarkdownDescription: "The label key to match on. Required when `field` is `LABEL`.",
-			Optional:            true,
+func assetRoutingConditionSchemaBlock() schema.ListNestedBlock {
+	return schema.ListNestedBlock{
+		MarkdownDescription: "Conditions that must all match for this rule to apply (AND logic). If empty, the rule matches all assets (catch-all).",
+		NestedObject: schema.NestedBlockObject{
+			Attributes: map[string]schema.Attribute{
+				"field": schema.StringAttribute{
+					MarkdownDescription: "The field to match on. Valid values: `HOSTNAME`, `PLATFORM`, `LABEL`.",
+					Required:            true,
+				},
+				"operator": schema.StringAttribute{
+					MarkdownDescription: "The comparison operator. Valid values: `EQUAL`, `NOT_EQUAL`, `CONTAINS`, `MATCHES`.",
+					Required:            true,
+				},
+				"values": schema.ListAttribute{
+					MarkdownDescription: "List of values to match against. A condition matches if the field matches any of the listed values (OR logic).",
+					Required:            true,
+					ElementType:         types.StringType,
+				},
+				"key": schema.StringAttribute{
+					MarkdownDescription: "The label key to match on. Required when `field` is `LABEL`.",
+					Optional:            true,
+				},
+			},
 		},
 	}
 }

internal/provider/asset_routing_rule_resource.go

@@ -71,13 +71,9 @@ func (r *AssetRoutingRuleResource) Schema(_ context.Context, _ resource.SchemaRe
 					int64planmodifier.UseStateForUnknown(),
 				},
 			},
-			"condition": schema.ListNestedAttribute{
-				MarkdownDescription: "Conditions that must all match for this rule to apply (AND logic). If empty, the rule matches all assets (catch-all).",
-				Optional:            true,
-				NestedObject: schema.NestedAttributeObject{
-					Attributes: assetRoutingConditionSchemaAttributes(),
-				},
-			},
+		},
+		Blocks: map[string]schema.Block{
+			"condition": assetRoutingConditionSchemaBlock(),
 		},
 	}
 }

internal/provider/asset_routing_table_resource.go

@@ -54,22 +54,19 @@ func (r *AssetRoutingTableResource) Schema(_ context.Context, _ resource.SchemaR
 					stringplanmodifier.RequiresReplace(),
 				},
 			},
-			"rule": schema.ListNestedAttribute{
+		},
+		Blocks: map[string]schema.Block{
+			"rule": schema.ListNestedBlock{
 				MarkdownDescription: "Ordered list of routing rules. Priority is determined by position (first = highest priority). A rule with no conditions acts as a catch-all.",
-				Required:            true,
-				NestedObject: schema.NestedAttributeObject{
+				NestedObject: schema.NestedBlockObject{
 					Attributes: map[string]schema.Attribute{
 						"target_space_mrn": schema.StringAttribute{
 							MarkdownDescription: "The MRN of the space where matching assets will be routed.",
 							Required:            true,
 						},
-						"condition": schema.ListNestedAttribute{
-							MarkdownDescription: "Conditions that must all match for this rule to apply (AND logic). If empty, the rule matches all assets (catch-all).",
-							Optional:            true,
-							NestedObject: schema.NestedAttributeObject{
-								Attributes: assetRoutingConditionSchemaAttributes(),
-							},
-						},
+					},
+					Blocks: map[string]schema.Block{
+						"condition": assetRoutingConditionSchemaBlock(),
 					},
 				},
 			},