Use temporary directory

dynamic/main.go

@@ -17,18 +17,16 @@ package main
 import (
 	"context"
 	"encoding/json"
-	"errors"
 	"fmt"
-	"os"
-	"os/exec"
-
 	"github.com/blang/semver"
 	"github.com/opentofu/opentofu/shim/run"
 	"github.com/pulumi/pulumi/pkg/v3/codegen/schema"
 	"github.com/pulumi/pulumi/sdk/v3/go/common/diag"
 	"github.com/pulumi/pulumi/sdk/v3/go/common/diag/colors"
 	"github.com/pulumi/pulumi/sdk/v3/go/common/resource/plugin"
 	"github.com/pulumi/pulumi/sdk/v3/go/common/util/contract"
+	"os"
+	"os/exec"
 
 	"github.com/pulumi/pulumi-terraform-bridge/dynamic/parameterize"
 	"github.com/pulumi/pulumi-terraform-bridge/dynamic/version"
@@ -158,19 +156,20 @@ func initialSetup() (info.Provider, pfbridge.ProviderMetadata, func() error) {
 			}
 			fullDocs = args.Remote.Docs
 			if fullDocs {
-				upstreamRepoDir := "terraform-provider-" + info.Name + "-v" + info.Version
-				// Only clone if the directory doesn't exist in the expected location
-				if _, err := os.Stat(upstreamRepoDir); errors.Is(err, os.ErrNotExist) {
-					versionWithPrefix := "v" + info.Version
-					ghRepo := "https://github.com/" + info.GitHubOrg + "/terraform-provider-" + info.Name
-
-					cmd := exec.Command("git", "clone", "--depth", "1", "-b", versionWithPrefix, ghRepo, upstreamRepoDir)
-					err = cmd.Run()
-					if err != nil {
-						return plugin.ParameterizeResponse{}, err
-					}
+				// Write the upstream files at this version to a temporary directory
+				tmpDir, err := os.MkdirTemp("", "upstreamRepoDir")
+				if err != nil {
+					return plugin.ParameterizeResponse{}, err
+				}
+				versionWithPrefix := "v" + info.Version
+				ghRepo := "https://github.com/" + info.GitHubOrg + "/terraform-provider-" + info.Name
+
+				cmd := exec.Command("git", "clone", "--depth", "1", "-b", versionWithPrefix, ghRepo, tmpDir)
+				err = cmd.Run()
+				if err != nil {
+					return plugin.ParameterizeResponse{}, err
 				}
-				info.UpstreamRepoPath = upstreamRepoDir
+				info.UpstreamRepoPath = tmpDir
 			}
 			return plugin.ParameterizeResponse{
 				Name:    p.Name(),

dynamic/parameterize/args.go

@@ -72,6 +72,6 @@ func ParseArgs(args []string) (Args, error) {
 		remote.Name = args[0]
 		return Args{Remote: &remote}, nil
 	default:
-		return Args{}, fmt.Errorf("expected to be parameterized by 1-3 arguments: <name> [version] fullDocs")
+		return Args{}, fmt.Errorf("expected to be parameterized by 1-3 arguments: <name> [version] [fullDocs]")
 	}
 }

dynamic/parameterize/args_test.go

@@ -51,12 +51,12 @@ func TestParseArgs(t *testing.T) {
 		{
 			name:   "no args",
 			args:   []string{},
-			errMsg: autogold.Expect("expected to be parameterized by 1-3 arguments: <name> [version] fullDocs"),
+			errMsg: autogold.Expect("expected to be parameterized by 1-3 arguments: <name> [version] [fullDocs]"),
 		},
 		{
 			name:   "too many args",
 			args:   []string{"arg1", "arg2", "arg3", "arg4"},
-			errMsg: autogold.Expect("expected to be parameterized by 1-3 arguments: <name> [version] fullDocs"),
+			errMsg: autogold.Expect("expected to be parameterized by 1-3 arguments: <name> [version] [fullDocs]"),
 		},
 		{
 			name:   "invalid third arg",

pkg/tfbridge/info/info.go

@@ -130,7 +130,7 @@ type Provider struct {
 	// - TFProviderModuleVersion
 	//
 	// For dynamically bridged providers, when the fullDocs option is set on the parameterized inputs,
-	// this field will be automatically set to "terraform-provider-<provider>-v<version>".
+	// this field will be automatically set to a temporary directory containing a shallow clone of the upstream repo.
 
 	// This list may change over time.
 	UpstreamRepoPath string

pkg/tfgen/generate.go

@@ -787,7 +787,6 @@ type GenerateSchemaResult struct {
 
 func GenerateSchemaWithOptions(opts GenerateSchemaOptions) (*GenerateSchemaResult, error) {
 	ctx := context.Background()
-
 	info := opts.ProviderInfo
 	sink := opts.DiagnosticsSink
 	g, err := NewGenerator(GeneratorOptions{
@@ -802,6 +801,7 @@ func GenerateSchemaWithOptions(opts GenerateSchemaOptions) (*GenerateSchemaResul
 	if err != nil {
 		return nil, errors.Wrapf(err, "failed to create generator")
 	}
+
 	return g.generateSchemaResult(ctx)
 }