feat: allow bool + number tag values

Tag values accept bool & number in coder/coder.

Author: Emyrk

preview_test.go

@@ -49,13 +49,28 @@ func Test_Extract(t *testing.T) {
 			dir:         "badparam",
 			failPreview: true,
 		},
+		{
+			name: "sometags",
+			dir:  "sometags",
+			expTags: map[string]string{
+				"string": "foo",
+				"number": "42",
+				"bool":   "true",
+				"extra":  "bar",
+			},
+			unknownTags: []string{
+				"map", "list", "null",
+			},
+		},
 		{
 			name: "simple static values",
 			dir:  "static",
 			expTags: map[string]string{
 				"zone": "developers",
 			},
-			unknownTags: []string{},
+			unknownTags: []string{
+				"list",
+			},
 			params: map[string]assertParam{
 				"region": ap().value("us").
 					def("us").

testdata/sometags/main.tf

@@ -0,0 +1,29 @@
+terraform {
+  required_providers {
+    coder = {
+      source = "coder/coder"
+      version = "2.4.0-pre0"
+    }
+  }
+}
+
+data "coder_workspace_tags" "custom_workspace_tags" {
+  tags = {
+    "string" = "foo"
+    "number" = 42
+    "bool"   = true
+    "list"   = ["a", "b", "c"]
+    "map"    = {
+      "key1" = "value1"
+      "key2" = "value2"
+    }
+    "null"  = null
+  }
+}
+
+
+data "coder_workspace_tags" "custom_workspace_tags" {
+  tags = {
+    "extra" = "bar"
+  }
+}

testdata/sometags/skipe2e

@@ -0,0 +1 @@
+Bad tags are not going to play well with terraform apply.

workspacetags.go

@@ -120,23 +120,6 @@ func newTag(srcRange *hcl.Range, _ map[string]*hcl.File, key, val cty.Value) (ty
 		}
 	}
 
-	if val.IsKnown() && val.Type() != cty.String {
-		fr := "<nil>"
-		if !val.Type().Equals(cty.NilType) {
-			fr = val.Type().FriendlyName()
-		}
-		// r := expr.ValueExpr.Range()
-		return types.Tag{}, &hcl.Diagnostic{
-			Severity: hcl.DiagError,
-			Summary:  "Invalid value type for tag",
-			Detail:   fmt.Sprintf("Value must be a string, but got %s", fr),
-			//Subject:     &r,
-			Context: srcRange,
-			//Expression:  expr.ValueExpr,
-			//EvalContext: evCtx,
-		}
-	}
-
 	tag := types.Tag{
 		Key: types.HCLString{
 			Value: key,
@@ -150,6 +133,25 @@ func newTag(srcRange *hcl.Range, _ map[string]*hcl.File, key, val cty.Value) (ty
 		},
 	}
 
+	// If the value is known, but the type is not a string, bool, or number.
+	// Then throw an error. Only the supported types can safely be converted to a string.
+	if !(val.Type() == cty.String || val.Type() == cty.Bool || val.Type() == cty.Number) {
+		fr := "<nil>"
+		if !val.Type().Equals(cty.NilType) {
+			fr = val.Type().FriendlyName()
+		}
+
+		tag.Value.ValueDiags = tag.Value.ValueDiags.Append(&hcl.Diagnostic{
+			Severity: hcl.DiagError,
+			Summary:  fmt.Sprintf("Invalid value type for tag %q", tag.KeyString()),
+			Detail:   fmt.Sprintf("Value must be a string, but got %s", fr),
+			//Subject:     &r,
+			Context: srcRange,
+			//Expression:  expr.ValueExpr,
+			//EvalContext: evCtx,
+		})
+	}
+
 	// ks, err := source(expr.KeyExpr.Range(), files)
 	// if err == nil {
 	//	src := string(ks)