From 2905d9d8289628226c764b387d044e7cefde11f3 Mon Sep 17 00:00:00 2001
From: Jeromy Cannon <jeromy@swirldslabs.com>
Date: Tue, 27 Aug 2024 18:47:09 +0100
Subject: [PATCH] fix: node add call triggers no prompt found for
 local-build-path error and exits (#515)

Signed-off-by: Jeromy Cannon <jeromy@swirldslabs.com>
---
 src/commands/flags.mjs              | 11 +++++++++++
 src/commands/node.mjs               |  4 +++-
 src/commands/prompts.mjs            |  2 ++
 test/e2e/commands/node-add.test.mjs |  2 +-
 4 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/src/commands/flags.mjs b/src/commands/flags.mjs
index 847eb9150..d445afaea 100644
--- a/src/commands/flags.mjs
+++ b/src/commands/flags.mjs
@@ -725,6 +725,17 @@ export const allFlags = [
   valuesFile
 ]
 
+/**
+ * Resets the definition.disablePrompt for all flags
+ */
+export function resetDisabledPrompts () {
+  allFlags.forEach(f => {
+    if (f.definition.disablePrompt) {
+      delete f.definition.disablePrompt
+    }
+  })
+}
+
 export const allFlagsMap = new Map(allFlags.map(f => [f.name, f]))
 
 export const nodeConfigFileFlags = new Map([
diff --git a/src/commands/node.mjs b/src/commands/node.mjs
index 166a48df8..95694624c 100644
--- a/src/commands/node.mjs
+++ b/src/commands/node.mjs
@@ -1194,7 +1194,8 @@ export class NodeCommand extends BaseCommand {
           // disable the prompts that we don't want to prompt the user for
           prompts.disablePrompts([
             flags.devMode,
-            flags.force
+            flags.force,
+            flags.localBuildPath
           ])
 
           await prompts.execute(task, self.configManager, NodeCommand.REFRESH_FLAGS_LIST)
@@ -1452,6 +1453,7 @@ export class NodeCommand extends BaseCommand {
             flags.endpointType,
             flags.force,
             flags.fstChartVersion,
+            flags.localBuildPath,
             flags.gossipEndpoints,
             flags.grpcEndpoints
           ])
diff --git a/src/commands/prompts.mjs b/src/commands/prompts.mjs
index 0e5b2f9f9..6a5b32ccc 100644
--- a/src/commands/prompts.mjs
+++ b/src/commands/prompts.mjs
@@ -20,6 +20,7 @@ import { FullstackTestingError, IllegalArgumentError } from '../core/errors.mjs'
 import { ConfigManager, constants } from '../core/index.mjs'
 import * as flags from './flags.mjs'
 import * as helpers from '../core/helpers.mjs'
+import { resetDisabledPrompts } from './flags.mjs'
 
 async function prompt (type, task, input, defaultValue, promptMessage, emptyCheckMessage, flagName) {
   try {
@@ -531,6 +532,7 @@ export async function execute (task, configManager, flagList = []) {
  * @param {CommandFlag[]} flags list of flags to disable prompts for
  */
 export function disablePrompts (flags) {
+  resetDisabledPrompts()
   for (const flag of flags) {
     if (flag.definition) {
       flag.definition.disablePrompt = true
diff --git a/test/e2e/commands/node-add.test.mjs b/test/e2e/commands/node-add.test.mjs
index 2765de628..3ba5904f5 100644
--- a/test/e2e/commands/node-add.test.mjs
+++ b/test/e2e/commands/node-add.test.mjs
@@ -75,7 +75,7 @@ describe('Node add', () => {
       flags.devMode.constName
     ])
     await nodeCmd.accountManager.close()
-  }, 600000)
+  }, 800000)
 
   balanceQueryShouldSucceed(nodeCmd.accountManager, nodeCmd, namespace)