From 3d1d44839073238c92641637fceff1d3219d89b3 Mon Sep 17 00:00:00 2001
From: Karl Kirch <karl@hashicorp.com>
Date: Fri, 12 Jul 2024 09:55:02 -0500
Subject: [PATCH] Remove resource ID validations

These validations exists at the API level and are not useful in the provider itself
---
 internal/provider/provider_next.go                   | 10 ----------
 internal/provider/resource_tfe_project.go            |  2 --
 .../provider/resource_tfe_team_project_access.go     |  4 ----
 internal/provider/resource_tfe_workspace_settings.go | 12 ------------
 4 files changed, 28 deletions(-)

diff --git a/internal/provider/provider_next.go b/internal/provider/provider_next.go
index 5268194fb..3eb81c6fb 100644
--- a/internal/provider/provider_next.go
+++ b/internal/provider/provider_next.go
@@ -5,9 +5,7 @@ package provider
 
 import (
 	"context"
-	"fmt"
 	"os"
-	"regexp"
 
 	"github.com/hashicorp/terraform-plugin-framework/datasource"
 	"github.com/hashicorp/terraform-plugin-framework/provider"
@@ -25,14 +23,6 @@ type frameworkProvider struct{}
 // Compile-time interface check
 var _ provider.Provider = &frameworkProvider{}
 
-// Can be used to construct ID regexp patterns
-var base58Alphabet = "[1-9A-HJ-NP-Za-km-z]"
-
-// IDPattern constructs a regexp pattern for HCP Terraform with the given prefix
-func IDPattern(prefix string) *regexp.Regexp {
-	return regexp.MustCompile(fmt.Sprintf("^%s-%s{16}$", prefix, base58Alphabet))
-}
-
 // FrameworkProviderConfig is a helper type for extracting the provider
 // configuration from the provider block.
 type FrameworkProviderConfig struct {
diff --git a/internal/provider/resource_tfe_project.go b/internal/provider/resource_tfe_project.go
index a7b1b6abe..6f4c3e4be 100644
--- a/internal/provider/resource_tfe_project.go
+++ b/internal/provider/resource_tfe_project.go
@@ -21,8 +21,6 @@ import (
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
 )
 
-var projectIDRegexp = regexp.MustCompile("^prj-[a-zA-Z0-9]{16}$")
-
 func resourceTFEProject() *schema.Resource {
 	return &schema.Resource{
 		CreateContext: resourceTFEProjectCreate,
diff --git a/internal/provider/resource_tfe_team_project_access.go b/internal/provider/resource_tfe_team_project_access.go
index ab1abe49e..c50c3bddc 100644
--- a/internal/provider/resource_tfe_team_project_access.go
+++ b/internal/provider/resource_tfe_team_project_access.go
@@ -59,10 +59,6 @@ func resourceTFETeamProjectAccess() *schema.Resource {
 				Type:     schema.TypeString,
 				Required: true,
 				ForceNew: true,
-				ValidateFunc: validation.StringMatch(
-					projectIDRegexp,
-					"must be a valid project ID (prj-<RANDOM STRING>)",
-				),
 			},
 
 			"project_access": {
diff --git a/internal/provider/resource_tfe_workspace_settings.go b/internal/provider/resource_tfe_workspace_settings.go
index f6fabaff6..91ebd0d8d 100644
--- a/internal/provider/resource_tfe_workspace_settings.go
+++ b/internal/provider/resource_tfe_workspace_settings.go
@@ -202,12 +202,6 @@ func (r *workspaceSettings) Schema(ctx context.Context, req resource.SchemaReque
 					stringplanmodifier.RequiresReplace(),
 					stringplanmodifier.UseStateForUnknown(),
 				},
-				Validators: []validator.String{
-					stringvalidator.RegexMatches(
-						IDPattern("ws"),
-						"must be a valid workspace ID (ws-<RANDOM STRING>)",
-					),
-				},
 			},
 
 			"execution_mode": schema.StringAttribute{
@@ -228,12 +222,6 @@ func (r *workspaceSettings) Schema(ctx context.Context, req resource.SchemaReque
 					unknownIfExecutionModeUnset{},
 					validateAgentExecutionMode{},
 				},
-				Validators: []validator.String{
-					stringvalidator.RegexMatches(
-						IDPattern("apool"),
-						"must be a valid workspace ID (apool-<RANDOM STRING>)",
-					),
-				},
 			},
 
 			// ListAttribute was required here because we are still using plugin protocol v5.