Clarify workaround conditions

Author: guineveresaenger

pkg/tfbridge/typechecker/typechecker.go

@@ -162,7 +162,8 @@ func (v *TypeChecker) validatePropertyValue(
 	switch typeSpec.Type {
 	case "boolean":
 		// Check for strings that are values "true" or "false".
-		// These are handled as booleans in the bridge, so they should be skipped by the type checker.
+		//TODO: Remove the boolString condition when https://github.com/pulumi/pulumi-terraform-bridge/issues/2520
+		// is resolved. This is a workaround for the config encoding not honoring type overrides.
 		var boolString bool
 		if propertyValue.IsString() {
 			if propertyValue.StringValue() == "true" || propertyValue.StringValue() == "false" {

pkg/tfbridge/typechecker/typechecker_test.go

@@ -1632,6 +1632,9 @@ func TestValidateConfigType(t *testing.T) {
 			}),
 		},
 		{
+			//TODO: Remove this test when https://github.com/pulumi/pulumi-terraform-bridge/issues/2520 is resolved.
+			// This tests a workaround path to keep the type checker from tripping on missing functionality in the
+			// config encoding and will fail once that is fixed.
 			name:      "allows_bool_strings",
 			inputName: "skipMetadataApiCheck",
 			input:     resource.PropertyValue{V: "true"},