preserve variable interpolation when validating generated compose files (#1379)

Author: jordanstephens

src/pkg/cli/compose/load_content.go

@@ -7,12 +7,20 @@ import (
 	composeTypes "github.com/compose-spec/compose-go/v2/types"
 )
 
-func LoadFromContent(ctx context.Context, content []byte, nameFallback string) (*Project, error) {
+func loadFromContent(ctx context.Context, content []byte, nameFallback string, skipInterpolation bool) (*Project, error) {
 	return loader.LoadWithContext(ctx, composeTypes.ConfigDetails{ConfigFiles: []composeTypes.ConfigFile{{Content: content}}}, func(o *loader.Options) {
 		o.SetProjectName(nameFallback, false)
-		o.SkipConsistencyCheck = true // this matches the WithConsistency(false) option from the loader
-		o.SkipInterpolation = true
+		o.SkipConsistencyCheck = true
+		o.SkipInterpolation = skipInterpolation
 		o.SkipResolveEnvironment = true
 		o.SkipInclude = true
 	})
 }
+
+func LoadFromContent(ctx context.Context, content []byte, nameFallback string) (*Project, error) {
+	return loadFromContent(ctx, content, nameFallback, true)
+}
+
+func LoadFromContentWithInterpolation(ctx context.Context, content []byte, nameFallback string) (*Project, error) {
+	return loadFromContent(ctx, content, nameFallback, false)
+}

src/pkg/migrate/migrate.go

@@ -153,7 +153,7 @@ func generateComposeFile(ctx context.Context, fabric client.FabricClient, platfo
 		}
 
 		// Attempt to load the compose content
-		_, err = compose.LoadFromContent(ctx, []byte(composeContent), projectName)
+		_, err = compose.LoadFromContentWithInterpolation(ctx, []byte(composeContent), projectName)
 		if err != nil {
 			previousError = err.Error()
 			term.Debugf("Invalid compose file received: %v. Retrying...", err)