Added guard for empty region (#1423)

Author: KevyVo

src/pkg/mcp/prompts/awsBYOC.go

@@ -66,9 +66,11 @@ func setupAWSBYOPrompt(s *server.MCPServer, cluster string, providerId *client.P
 			}
 
 			region := getStringArg(req.Params.Arguments, "AWS_REGION", "")
-			err = os.Setenv("AWS_REGION", region)
-			if err != nil {
-				return nil, err
+			if region != "" {
+				err = os.Setenv("AWS_REGION", region)
+				if err != nil {
+					return nil, err
+				}
 			}
 		}
 

src/pkg/mcp/tools/estimate.go

@@ -47,12 +47,6 @@ func setupEstimateTool(s *server.MCPServer, cluster string, providerId *cliClien
 		term.Debug("Estimate tool called")
 		track.Evt("MCP Estimate Tool")
 
-		if *providerId == cliClient.ProviderDefang || *providerId == cliClient.ProviderAuto {
-			// We only support estimates for AWS and GCP, not playground; suggest use setup prompt for another provider
-			err := errors.New("estimates are only supported for AWS and GCP; please configure another provider using the appropriate prompts and type /mcp.defang.AWS_Setup for AWS or /mcp.defang.GCP_Setup for GCP")
-			return mcp.NewToolResultErrorFromErr("No compatible provider configured", err), err
-		}
-
 		wd, err := request.RequireString("working_directory")
 		if err != nil || wd == "" {
 			term.Error("Invalid working directory", "error", errors.New("working_directory is required"))