refactor: rename action to actions in template context

core/src/config/template-contexts/actions.ts

@@ -203,13 +203,12 @@ export interface ActionSpecContextParams {
  * Used to resolve action spec and variables.
  */
 export class ActionSpecContext extends OutputConfigContext {
-  // TODO-0.13.0: rename to actions (to allow using action singular in certain contexts + match the modules field)
   @schema(
     ActionReferencesContext.getSchema().description(
       "Runtime outputs and information from other actions (only resolved at runtime when executing actions)."
     )
   )
-  public action: ActionReferencesContext
+  public actions: ActionReferencesContext
 
   @schema(ActionReferencesContext.getSchema().description("Alias for `action`."))
   public runtime: ActionReferencesContext
@@ -239,8 +238,15 @@ export class ActionSpecContext extends OutputConfigContext {
   public this: ActionReferenceContext
 
   constructor(params: ActionSpecContextParams) {
-    const { action, garden, partialRuntimeResolution, variables, inputs, resolvedDependencies, executedDependencies } =
-      params
+    const {
+      action,
+      garden,
+      partialRuntimeResolution,
+      variables,
+      inputs,
+      resolvedDependencies,
+      executedDependencies,
+    } = params
 
     const internal = action.getInternal()
 
@@ -260,13 +266,13 @@ export class ActionSpecContext extends OutputConfigContext {
     const parentName = internal?.parentName
     const templateName = internal?.templateName
 
-    this.action = new ActionReferencesContext(this, partialRuntimeResolution, [
+    this.actions = new ActionReferencesContext(this, partialRuntimeResolution, [
       ...resolvedDependencies,
       ...executedDependencies,
     ])
 
     // Throw specific error when attempting to resolve self
-    this.action[action.kind.toLowerCase()].set(
+    this.actions[action.kind.toLowerCase()].set(
       name,
       new ErrorContext(`Action ${chalk.white.bold(action.key())} cannot reference itself.`)
     )
@@ -277,7 +283,7 @@ export class ActionSpecContext extends OutputConfigContext {
     }
     this.inputs = inputs
 
-    this.runtime = this.action
+    this.runtime = this.actions
 
     this.this = new ActionReferenceContext({
       root: this,

core/src/outputs.ts

@@ -51,7 +51,7 @@ export async function resolveProjectOutputs(garden: Garden, log: Log): Promise<O
       } else if (ref[1] === "tasks") {
         needActions.push({ kind: "Run", name: ref[2] as string })
       }
-    } else if (ref[0] === "action" && ref[1] && ref[2]) {
+    } else if (ref[0] === "actions" && ref[1] && ref[2]) {
       needActions.push({ kind: <ActionKind>ref[1], name: ref[2] as string })
     }
   }

core/src/plugins/kubernetes/sync.ts

@@ -128,7 +128,7 @@ export interface KubernetesDeployDevModeSyncSpec extends DevModeSyncOptions {
   containerName?: string
 }
 
-const exampleActionRef = templateStringLiteral("action.build.my-container-image.sourcePath")
+const exampleActionRef = templateStringLiteral("actions.build.my-container-image.sourcePath")
 
 export const kubernetesDeploySyncPathSchema = () =>
   syncDefaultsSchema()

core/src/template-string/template-string.ts

@@ -494,7 +494,7 @@ export function getActionTemplateReferences<T extends object>(config: T): Action
 
   // ${action.*}
   const refs: ActionTemplateReference[] = rawRefs
-    .filter((ref) => ref[0] === "action")
+    .filter((ref) => ref[0] === "actions")
     .map((ref) => {
       if (!ref[1]) {
         throw new ConfigurationError("Found invalid action reference (missing kind)", { config, ref })

core/test/data/test-projects/helm/api/garden.yml

@@ -15,7 +15,7 @@ spec:
   values:
     image:
       repository: api-image
-      tag: ${action.build.api-image.version}
+      tag: ${actions.build.api-image.version}
     ingress:
       enabled: true
       paths: [/]

core/test/data/test-projects/kubernetes-module/with-source-module/garden.yml

@@ -40,7 +40,7 @@ spec:
           spec:
             containers:
               - name: api
-                image: ${action.build.api-image.outputs.deployment-image-id}
+                image: ${actions.build.api-image.outputs.deployment-image-id}
                 args: [python, app.py]
                 ports:
                   - containerPort: 80

core/test/unit/src/actions/action-configs-to-graph.ts

@@ -296,7 +296,7 @@ describe("actionConfigsToGraph", () => {
             basePath: tmpDir.path,
           },
           spec: {
-            command: ["echo", "${action.build.foo.version}"],
+            command: ["echo", "${actions.build.foo.version}"],
           },
         },
       ],
@@ -313,7 +313,7 @@ describe("actionConfigsToGraph", () => {
         explicit: false,
         kind: "Build",
         name: "foo",
-        fullRef: ["action", "build", "foo", "version"],
+        fullRef: ["actions", "build", "foo", "version"],
         needsExecutedOutputs: false,
         needsStaticOutputs: true,
       },
@@ -343,7 +343,7 @@ describe("actionConfigsToGraph", () => {
             basePath: tmpDir.path,
           },
           spec: {
-            command: ["echo", "${action.build.foo.outputs.bar}"],
+            command: ["echo", "${actions.build.foo.outputs.bar}"],
           },
         },
       ],
@@ -360,7 +360,7 @@ describe("actionConfigsToGraph", () => {
         explicit: false,
         kind: "Build",
         name: "foo",
-        fullRef: ["action", "build", "foo", "outputs", "bar"],
+        fullRef: ["actions", "build", "foo", "outputs", "bar"],
         needsExecutedOutputs: true,
         needsStaticOutputs: false,
       },

core/test/unit/src/tasks/resolve-action.ts

@@ -91,7 +91,7 @@ describe("ResolveActionTask", () => {
           type: "test",
           name: "foo",
           spec: {
-            foo: "${action.build.foo.outputs.something}",
+            foo: "${actions.build.foo.outputs.something}",
           },
         },
       ])
@@ -116,7 +116,7 @@ describe("ResolveActionTask", () => {
           type: "test",
           name: "foo",
           spec: {
-            foo: "${action.build.foo.version}",
+            foo: "${actions.build.foo.version}",
           },
         },
       ])
@@ -258,7 +258,7 @@ describe("ResolveActionTask", () => {
           type: "test",
           name: "foo",
           spec: {
-            deployCommand: ["echo", "${action.build.foo.version}"],
+            deployCommand: ["echo", "${actions.build.foo.version}"],
           },
         },
       ])
@@ -290,7 +290,7 @@ describe("ResolveActionTask", () => {
           type: "test",
           name: "foo",
           spec: {
-            foo: "${action.build.foo.version}",
+            foo: "${actions.build.foo.version}",
           },
         },
       ])
@@ -419,7 +419,7 @@ describe("ResolveActionTask", () => {
           type: "test",
           name: "foo",
           spec: {
-            deployCommand: ["echo", "${action.run.foo.outputs.log}"],
+            deployCommand: ["echo", "${actions.run.foo.outputs.log}"],
           },
         },
       ])

core/test/unit/src/template-string.ts

@@ -1683,42 +1683,42 @@ describe("collectTemplateReferences", () => {
 })
 
 describe("getActionTemplateReferences", () => {
-  context("action.*", () => {
+  context("actions.*", () => {
     it("returns valid action references", () => {
       const config = {
-        build: '${action["build"].build-a}',
-        deploy: '${action["deploy"].deploy-a}',
-        run: '${action["run"].run-a}',
-        test: '${action["test"].test-a}',
+        build: '${actions["build"].build-a}',
+        deploy: '${actions["deploy"].deploy-a}',
+        run: '${actions["run"].run-a}',
+        test: '${actions["test"].test-a}',
       }
       const actionTemplateReferences = getActionTemplateReferences(config)
       expect(actionTemplateReferences).to.eql([
         {
           kind: "Build",
           name: "build-a",
-          fullRef: ["action", "build", "build-a"],
+          fullRef: ["actions", "build", "build-a"],
         },
         {
           kind: "Deploy",
           name: "deploy-a",
-          fullRef: ["action", "deploy", "deploy-a"],
+          fullRef: ["actions", "deploy", "deploy-a"],
         },
         {
           kind: "Run",
           name: "run-a",
-          fullRef: ["action", "run", "run-a"],
+          fullRef: ["actions", "run", "run-a"],
         },
         {
           kind: "Test",
           name: "test-a",
-          fullRef: ["action", "test", "test-a"],
+          fullRef: ["actions", "test", "test-a"],
         },
       ])
     })
 
     it("throws if action ref has no kind", () => {
       const config = {
-        foo: "${action}",
+        foo: "${actions}",
       }
       void expectError(() => getActionTemplateReferences(config), {
         contains: "Found invalid action reference (missing kind)",
@@ -1727,7 +1727,7 @@ describe("getActionTemplateReferences", () => {
 
     it("throws if action ref has invalid kind", () => {
       const config = {
-        foo: '${action["badkind"].some-name}',
+        foo: '${actions["badkind"].some-name}',
       }
       void expectError(() => getActionTemplateReferences(config), {
         contains: "Found invalid action reference (invalid kind 'badkind')",
@@ -1736,7 +1736,7 @@ describe("getActionTemplateReferences", () => {
 
     it("throws if action kind is not a string", () => {
       const config = {
-        foo: "${action[123]}",
+        foo: "${actions[123]}",
       }
       void expectError(() => getActionTemplateReferences(config), {
         contains: "Found invalid action reference (kind is not a string)",
@@ -1745,7 +1745,7 @@ describe("getActionTemplateReferences", () => {
 
     it("throws if action kind is not resolvable", () => {
       const config = {
-        foo: "${action[foo.bar].some-name}",
+        foo: "${actions[foo.bar].some-name}",
       }
       void expectError(() => getActionTemplateReferences(config), {
         contains: "Found invalid action reference (invalid kind '${foo.bar}')",
@@ -1754,7 +1754,7 @@ describe("getActionTemplateReferences", () => {
 
     it("throws if action ref has no name", () => {
       const config = {
-        foo: '${action["build"]}',
+        foo: '${actions["build"]}',
       }
       void expectError(() => getActionTemplateReferences(config), {
         contains: "Found invalid action reference (missing name)",
@@ -1763,7 +1763,7 @@ describe("getActionTemplateReferences", () => {
 
     it("throws if action name is not a string", () => {
       const config = {
-        foo: '${action["build"].123}',
+        foo: '${actions["build"].123}',
       }
       void expectError(() => getActionTemplateReferences(config), {
         contains: "Found invalid action reference (name is not a string)",

docs/k8s-plugins/action-types/helm.md

@@ -84,7 +84,7 @@ type: helm-pod
 dependencies:
   - deploy.api
 variables:
-  hostname: vote.${var.baseHostname}  
+  hostname: vote.${var.baseHostname}
 timeout: 60
 spec:
   resource:
@@ -167,8 +167,8 @@ dependencies: [build.worker-image]
 spec:
   values:
     image:
-      repository: ${action.build.worker-image.outputs.deployment-image-name}
-      tag: ${action.build.worker-image.version}
+      repository: ${actions.build.worker-image.outputs.deployment-image-name}
+      tag: ${actions.build.worker-image.version}
 
 ```
 
@@ -195,7 +195,7 @@ spec:
   sync:
     paths:
       - containerPath: /app/src
-        sourcePath: ${action.build.vote-image.sourcePath}/src
+        sourcePath: ${actions.build.vote-image.sourcePath}/src
         mode: two-way
 
 ```
@@ -216,7 +216,7 @@ You can define a remote environment as a `production` environment by setting the
 
 ## Next steps
 
-Check out the full [action reference](../../reference/action-types/README.md) for more details 
+Check out the full [action reference](../../reference/action-types/README.md) for more details
 and the [vote-helm](../../../examples/vote-helm/README.md) example project for a full project
 that showcases Garden's Helm support.
 

docs/k8s-plugins/action-types/kubernetes.md

@@ -14,7 +14,7 @@ you'll find a lot common between the two guides.
 
 See the full spec for the `kubernetes` deploy action in our [reference docs](../../reference/action-types/Deploy/kubernetes.md).
 
-[`kubernetes-pod` run](../../reference/action-types/Run/kubernetes-pod.md), [`kubernetes-pod` test](../../reference/action-types/Test/kubernetes-pod.md) and `kubernetes-exec` 
+[`kubernetes-pod` run](../../reference/action-types/Run/kubernetes-pod.md), [`kubernetes-pod` test](../../reference/action-types/Test/kubernetes-pod.md) and `kubernetes-exec`
 actions can be used for testing and task porposes.
 
 ## Referencing manifests
@@ -169,7 +169,7 @@ type: kubernetes-pod
 dependencies:
   - deploy.api
 variables:
-  hostname: vote.${var.baseHostname}  
+  hostname: vote.${var.baseHostname}
 timeout: 60
 spec:
   resource:
@@ -208,7 +208,7 @@ spec:
       spec:
         containers:
         - name: worker
-          image: ${action.build.worker-image.outputs.deployment-image-id} # <--- Here we're referencing the output from the api-image module. This will also work in manifest files.
+          image: ${actions.build.worker-image.outputs.deployment-image-id} # <--- Here we're referencing the output from the api-image module. This will also work in manifest files.
           # ...
 ```
 
@@ -232,7 +232,7 @@ spec:
   sync:
     paths:
       - containerPath: /app/src
-        sourcePath: ${action.build.vote-image.sourcePath}/src
+        sourcePath: ${actions.build.vote-image.sourcePath}/src
         mode: two-way
 
 ```

docs/reference/action-types/Deploy/helm.md

@@ -336,8 +336,8 @@ spec:
         # defined.
         #
         # This should generally be a templated path to another action's source path (e.g.
-        # `${action.build.my-container-image.sourcePath}`), or a relative path. If a path is hard-coded, you must make
-        # sure the path exists, and that it is reliably the correct path for every user.
+        # `${actions.build.my-container-image.sourcePath}`), or a relative path. If a path is hard-coded, you must
+        # make sure the path exists, and that it is reliably the correct path for every user.
         sourcePath: .
 
         # POSIX-style absolute path to sync to inside the container. The root path (i.e. "/") is not allowed.
@@ -1126,7 +1126,7 @@ The name of a container in the target. Specify this if the target contains more
 
 The local path to sync from, either absolute or relative to the source directory where the Deploy action is defined.
 
-This should generally be a templated path to another action's source path (e.g. `${action.build.my-container-image.sourcePath}`), or a relative path. If a path is hard-coded, you must make sure the path exists, and that it is reliably the correct path for every user.
+This should generally be a templated path to another action's source path (e.g. `${actions.build.my-container-image.sourcePath}`), or a relative path. If a path is hard-coded, you must make sure the path exists, and that it is reliably the correct path for every user.
 
 | Type        | Default | Required |
 | ----------- | ------- | -------- |

docs/reference/action-types/Deploy/kubernetes.md

@@ -318,8 +318,8 @@ spec:
         # defined.
         #
         # This should generally be a templated path to another action's source path (e.g.
-        # `${action.build.my-container-image.sourcePath}`), or a relative path. If a path is hard-coded, you must make
-        # sure the path exists, and that it is reliably the correct path for every user.
+        # `${actions.build.my-container-image.sourcePath}`), or a relative path. If a path is hard-coded, you must
+        # make sure the path exists, and that it is reliably the correct path for every user.
         sourcePath: .
 
         # POSIX-style absolute path to sync to inside the container. The root path (i.e. "/") is not allowed.
@@ -1070,7 +1070,7 @@ The name of a container in the target. Specify this if the target contains more
 
 The local path to sync from, either absolute or relative to the source directory where the Deploy action is defined.
 
-This should generally be a templated path to another action's source path (e.g. `${action.build.my-container-image.sourcePath}`), or a relative path. If a path is hard-coded, you must make sure the path exists, and that it is reliably the correct path for every user.
+This should generally be a templated path to another action's source path (e.g. `${actions.build.my-container-image.sourcePath}`), or a relative path. If a path is hard-coded, you must make sure the path exists, and that it is reliably the correct path for every user.
 
 | Type        | Default | Required |
 | ----------- | ------- | -------- |