fix: don't accept --networkId in commands where node is required

aeternity · Apr 9, 2024 · cfd7ac5 · cfd7ac5
1 parent df67561
commit cfd7ac5
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 10 deletions.
diff --git a/src/commands/account.js b/src/commands/account.js
@@ -46,7 +46,6 @@ addCommonOptions(program
       return { amount: coinAmountParser(amount) };
     },
   )
-  .addOption(networkIdOption)
   .option('--payload [payload]', 'Transaction payload.', '')
   .addOption(feeOption)
   .addOption(ttlOption(true))

diff --git a/src/commands/contract.js b/src/commands/contract.js
@@ -15,7 +15,6 @@ import {
   forceOption,
   passwordOption,
   ttlOption,
-  networkIdOption,
 } from '../arguments.js';
 
 const callArgs = new Argument('[args]', 'JSON-encoded arguments array of contract call')
@@ -105,7 +104,6 @@ addCommonOptions(program
   .addOption(contractSourceFilenameOption)
   .addOption(contractAciFilenameOption)
   .option('-W, --no-waitMined', 'Force waiting until transaction will be mined')
-  .addOption(networkIdOption)
   .addOption(passwordOption)
   .addOption(gasOption)
   .option('-s, --callStatic', 'Call static')
@@ -133,7 +131,6 @@ addCommonOptions(program
   .addOption(contractSourceFilenameOption)
   .option('--contractBytecode [contractBytecode]', 'Contract bytecode file name')
   .addOption(contractAciFilenameOption)
-  .addOption(networkIdOption)
   .option('-W, --no-waitMined', 'Force waiting until transaction will be mined')
   .addOption(passwordOption)
   .addOption(gasOption)

diff --git a/src/commands/name.js b/src/commands/name.js
@@ -8,7 +8,7 @@ import { Command } from 'commander';
 import { NAME_TTL, CLIENT_TTL } from '@aeternity/aepp-sdk';
 import * as AENS from '../actions/aens.js';
 import {
-  nodeOption, jsonOption, feeOption, forceOption, passwordOption, ttlOption, networkIdOption,
+  nodeOption, jsonOption, feeOption, forceOption, passwordOption, ttlOption,
 } from '../arguments.js';
 
 const program = new Command().name('aecli name');
@@ -20,7 +20,6 @@ const addCommonOptions = (p) => p
   .addOption(feeOption)
   .option('--nonce [nonce]', 'Override the nonce that the transaction is going to be sent with')
   .addOption(passwordOption)
-  .addOption(networkIdOption)
   .addOption(forceOption)
   .addOption(jsonOption);
 
@@ -121,7 +120,7 @@ addCommonOptions(program
 //
 // Example: `aecli name revoke ./myWalletKeyFile --password testpass testname.chain`
 addCommonOptions(program
-  .command('revoke  <wallet_path> <name>')
+  .command('revoke <wallet_path> <name>')
   .option('-M, --no-waitMined', 'Do not wait until transaction will be mined')
   .description('Revoke an AENS name')
   .action(AENS.revokeName));

diff --git a/src/commands/oracle.js b/src/commands/oracle.js
@@ -6,7 +6,7 @@ import { Command } from 'commander';
 import { ORACLE_TTL, QUERY_TTL, RESPONSE_TTL } from '@aeternity/aepp-sdk';
 import * as Oracle from '../actions/oracle.js';
 import {
-  nodeOption, jsonOption, feeOption, forceOption, passwordOption, ttlOption, networkIdOption,
+  nodeOption, jsonOption, feeOption, forceOption, passwordOption, ttlOption,
 } from '../arguments.js';
 
 const program = new Command().name('aecli oracle');
@@ -18,7 +18,6 @@ const addCommonOptions = (p) => p
   .addOption(feeOption)
   .option('--nonce [nonce]', 'Override the nonce that the transaction is going to be sent with')
   .addOption(passwordOption)
-  .addOption(networkIdOption)
   .addOption(forceOption)
   .addOption(jsonOption);
 
@@ -85,7 +84,7 @@ addCommonOptions(program
   .command('respond-query <wallet_path> <oracleId> <queryId> <response>')
   .option('-M, --no-waitMined', 'Do not wait until transaction will be mined')
   .option('--responseTtl [responseTtl]', 'Query response time to leave', RESPONSE_TTL.value)
-  .description('Respond to  Oracle Query')
+  .description('Respond to Oracle Query')
   .action(Oracle.respondToQuery));
 
 // ## Initialize `get oracle` command