From ece01196509ed8fefbc578f116c0d7e1c205497d Mon Sep 17 00:00:00 2001
From: Vladimir Vagaytsev <vladimir.vagaitsev@gmail.com>
Date: Tue, 25 Jun 2024 16:44:50 +0200
Subject: [PATCH] fix(pulumi): fix dependency resolution when
 `--skip-dependencies` is on (#6229)

* fix(pulumi): fix process deps resolution

A follow-up fix for #6226.
It returns the current task itself if the deps as skipped.

* refactor: extract local var
---
 plugins/pulumi/src/commands.ts | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/plugins/pulumi/src/commands.ts b/plugins/pulumi/src/commands.ts
index d1d3b42075..76c07aa6d5 100644
--- a/plugins/pulumi/src/commands.ts
+++ b/plugins/pulumi/src/commands.ts
@@ -272,8 +272,9 @@ class PulumiPluginCommandTask extends PluginActionTask<PulumiDeploy, PulumiComma
    * Override the base method to be sure that `garden plugins pulumi preview` happens in dependency order.
    */
   override resolveProcessDependencies() {
+    const currentTask = this.getResolveTask(this.action)
     if (this.skipRuntimeDependencies) {
-      return []
+      return [currentTask]
     }
 
     const pulumiDeployNames = this.graph
@@ -290,7 +291,7 @@ class PulumiPluginCommandTask extends PluginActionTask<PulumiDeploy, PulumiComma
       })
       .filter(isDeployAction)
 
-    const tasks = deps.map((action) => {
+    const depTasks = deps.map((action) => {
       return new PulumiPluginCommandTask({
         garden: this.garden,
         graph: this.graph,
@@ -305,7 +306,7 @@ class PulumiPluginCommandTask extends PluginActionTask<PulumiDeploy, PulumiComma
       })
     })
 
-    return [this.getResolveTask(this.action), ...tasks]
+    return [currentTask, ...depTasks]
   }
 
   async getStatus() {