terraform init should pick up action providers

Author: DanielMSchmidt

internal/configs/config.go

@@ -483,6 +483,15 @@ func (c *Config) addProviderRequirements(reqs providerreqs.Requirements, recurse
 		reqs[fqn] = nil
 	}
 
+	for _, rc := range c.Module.Actions {
+		fqn := rc.Provider
+		if _, exists := reqs[fqn]; exists {
+			// Explicit dependency already present
+			continue
+		}
+		reqs[fqn] = nil
+	}
+
 	// Import blocks that are generating config may have a custom provider
 	// meta-argument. Like the provider meta-argument used in resource blocks,
 	// we use this opportunity to load any implicit providers.
@@ -1022,6 +1031,9 @@ func (c *Config) EffectiveRequiredProviderConfigs() addrs.Map[addrs.RootProvider
 	for _, rc := range c.Module.DataResources {
 		maybePutLocal(rc.ProviderConfigAddr(), false)
 	}
+	for _, rc := range c.Module.Actions {
+		maybePutLocal(rc.ProviderConfigAddr(), false)
+	}
 	for _, ic := range c.Module.Import {
 		if ic.ProviderConfigRef != nil {
 			maybePutLocal(addrs.LocalProviderConfig{

internal/configs/config_test.go

@@ -35,8 +35,10 @@ func TestConfigProviderTypes(t *testing.T) {
 	}
 
 	got = cfg.ProviderTypes()
+
 	want := []addrs.Provider{
 		addrs.NewDefaultProvider("aws"),
+		addrs.NewDefaultProvider("external"),
 		addrs.NewDefaultProvider("local"),
 		addrs.NewDefaultProvider("null"),
 		addrs.NewDefaultProvider("template"),

internal/configs/parser_test.go

@@ -45,6 +45,7 @@ func testParser(files map[string]string) *Parser {
 // module and returns its configuration. This is a helper for use in unit tests.
 func testModuleConfigFromFile(filename string) (*Config, hcl.Diagnostics) {
 	parser := NewParser(nil)
+	parser.allowExperiments = true // Allow language experiments in tests so we can verify actions.
 	f, diags := parser.LoadConfigFile(filename)
 	mod, modDiags := NewModule([]*File{f}, nil)
 	diags = append(diags, modDiags...)

internal/configs/testdata/valid-files/providers-explicit-implied.tf

@@ -14,6 +14,9 @@ resource "null_resource" "foo" {
 
 }
 
+action "external_test" "foo" {
+}
+
 import {
   id = "directory/filename"
   to = local_file.foo