Validate params in interactive mode

packages/ethernaut-common/src/type-address.js

@@ -16,10 +16,13 @@ module.exports = {
         `"${argValue}" is not an address`,
         `Invalid ${argName}`,
       )
-
       if (typeof describe === 'function') {
         throw err
       }
+
+      return false
     }
+
+    return true
   },
 }

packages/ethernaut-common/src/type-bytes32.js

@@ -20,6 +20,10 @@ module.exports = {
       if (typeof describe === 'function') {
         throw err
       }
+
+      return false
     }
+
+    return true
   },
 }

packages/ethernaut-common/src/type-ens.js

@@ -18,6 +18,10 @@ module.exports = {
       if (typeof describe === 'function') {
         throw err
       }
+
+      return false
     }
+
+    return true
   },
 }

packages/ethernaut-common/src/type-int.js

@@ -15,6 +15,10 @@ module.exports = {
       if (typeof describe === 'function') {
         throw err
       }
+
+      return false
     }
+
+    return true
   },
 }

packages/ethernaut-common/src/type-string.js

@@ -13,6 +13,10 @@ module.exports = {
       if (typeof describe === 'function') {
         throw err
       }
+
+      return false
     }
+
+    return true
   },
 }

packages/ethernaut-interact-ui/src/index.js

@@ -22,14 +22,15 @@ extendEnvironment((hre) => {
   contract.paramDefinitions.params.prompt = paramsPrompt
 
   const token = hre.scopes.interact.tasks.token
-  token.paramDefinitions.address.suggest = addressSuggest
+  token.positionalParamDefinitions.find((p) => p.name === 'address').suggest =
+    addressSuggest
   token.paramDefinitions.fn.prompt = fnERC20Prompt
   token.paramDefinitions.params.prompt = paramsERC20Prompt
 
   const logs = hre.scopes.interact.tasks.logs
+  logs.paramDefinitions.address.suggest = addressSuggest
   logs.paramDefinitions.abi.prompt = abiPrompt
   logs.paramDefinitions.abi.suggest = abiSuggest
-  logs.paramDefinitions.address.suggest = addressSuggest
-  logs.paramDefinitions.event.prompt = eventPrompt
   logs.paramDefinitions.params.prompt = paramsLogPrompt
+  logs.paramDefinitions.event.prompt = eventPrompt
 })

packages/ethernaut-interact/src/tasks/token.js

@@ -4,7 +4,12 @@ const interact = require('../internal/interact')
 
 require('../scopes/interact')
   .task('token', 'Interacts with any ERC20 token')
-  .addParam('address', 'The address of the token', undefined, types.string)
+  .addPositionalParam(
+    'address',
+    'The address of the token',
+    undefined,
+    types.address,
+  )
   .addParam('fn', 'The function of the token to call', undefined, types.string)
   .addOptionalParam(
     'value',

packages/ethernaut-ui/src/internal/collect-args.js

@@ -15,7 +15,9 @@ module.exports = async function collectArguments(providedArgs, task, hre) {
   )
 
   const collectedArgs = {}
-  for (let paramDef of paramDefinitions) {
+  for (let i = 0; i < paramDefinitions.length; i++) {
+    const paramDef = paramDefinitions[i]
+
     if (paramDef.originallyOptional) continue
 
     const providedArg = providedArgs[paramDef.name]
@@ -29,7 +31,14 @@ module.exports = async function collectArguments(providedArgs, task, hre) {
       argsSoFar,
     )
     if (collectedArg !== undefined) {
-      debug.log(`Collected ${paramDef.name}" with "${collectedArg}"`, 'ui')
+      debug.log(`Collected "${paramDef.name}" with "${collectedArg}"`, 'ui')
+
+      const isValid = paramDef.type.validate(paramDef.name, collectedArg)
+      debug.log(`Validation result for "${paramDef.name}": ${isValid}`, 'ui')
+      if (!isValid) {
+        i--
+        continue
+      }
 
       collectedArgs[paramDef.name] = collectedArg
     }
@@ -129,5 +138,5 @@ async function rawPrompt(paramDef, suggested) {
   if (result === 'false') return false
   if (result === 'true') return true
 
-  return result
+  return result === '' ? undefined : result
 }

packages/ethernaut-ui/src/internal/make-interactive.js

@@ -73,7 +73,7 @@ function makeInteractive(task) {
       return parsedValue
     },
     validate: (argName, argValue) => {
-      originalType.validate(argName, argValue)
+      return originalType.validate(argName, argValue)
     },
   })
   for (let paramDef of paramDefinitions) {