genlayer 0.18.0 → 0.18.2-beta.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "genlayer",
-  "version": "0.18.0",
+  "version": "0.18.2-beta.0",
   "description": "GenLayer Command Line Tool",
   "main": "src/index.ts",
   "type": "module",
@@ -64,7 +64,7 @@
     "dotenv": "^16.4.5",
     "ethers": "^6.13.4",
     "fs-extra": "^11.3.0",
-    "genlayer-js": "^0.9.0",
+    "genlayer-js": "^0.11.0",
     "inquirer": "^12.0.0",
     "node-fetch": "^3.0.0",
     "open": "^10.1.0",

package/src/commands/contracts/call.ts

@@ -1,23 +1,21 @@
-import { simulator } from "genlayer-js/chains";
-import type { GenLayerClient } from "genlayer-js/types";
-import { BaseAction } from "../../lib/actions/BaseAction";
+import {BaseAction} from "../../lib/actions/BaseAction";
 
 export interface CallOptions {
   args: any[];
   rpc?: string;
 }
 
-export class CallAction extends BaseAction{
+export class CallAction extends BaseAction {
   constructor() {
     super();
   }
 
   async call({
-               contractAddress,
-               method,
-               args,
-               rpc,
-             }: {
+    contractAddress,
+    method,
+    args,
+    rpc,
+  }: {
     contractAddress: string;
     method: string;
     args: any[];
@@ -27,26 +25,7 @@ export class CallAction extends BaseAction{
     await client.initializeConsensusSmartContract();
     this.startSpinner(`Calling method ${method} on contract at ${contractAddress}...`);
 
-    const contractSchema = await client.getContractSchema(contractAddress);
-
-    if(!contractSchema.methods.hasOwnProperty(method)){
-      this.failSpinner(`method ${method} not found.`);
-      return
-    }
-
-    const readonly = contractSchema.methods[method as any].readonly;
-
-    if (readonly) {
-      await this.executeRead(contractAddress, method, args);
-      return
-    }
-
-    await this.executeWrite(contractAddress, method, args);
-  }
-
-  private async executeRead(contractAddress: string, method: string, args: any[]): Promise<void> {
     try {
-      const client = await this.getClient();
       const result = await client.readContract({
         address: contractAddress as any,
         functionName: method,
@@ -57,25 +36,4 @@ export class CallAction extends BaseAction{
       this.failSpinner("Error during read operation", error);
     }
   }
-
-  private async executeWrite(contractAddress: string, method: string, args: any[]): Promise<void> {
-    try {
-      const client = await this.getClient();
-      const hash = await client.writeContract({
-        address: contractAddress as any,
-        functionName: method,
-        args,
-        value: 0n,
-      });
-      const result = await client.waitForTransactionReceipt({
-        hash,
-        retries: 15,
-        interval: 2000,
-      });
-      this.log("Write transaction hash:", hash);
-      this.succeedSpinner("Write operation successfully executed", result);
-    } catch (error) {
-      this.failSpinner("Error during write operation", error);
-    }
-  }
 }

package/src/commands/contracts/deploy.ts

@@ -1,11 +1,9 @@
 import fs from "fs";
 import path from "path";
-import { simulator } from "genlayer-js/chains";
-import type { GenLayerClient } from "genlayer-js/types";
-import { BaseAction } from "../../lib/actions/BaseAction";
-import { pathToFileURL } from "url";
-import { TransactionStatus } from "genlayer-js/types";
-import { buildSync } from "esbuild";
+import {BaseAction} from "../../lib/actions/BaseAction";
+import {pathToFileURL} from "url";
+import {TransactionStatus} from "genlayer-js/types";
+import {buildSync} from "esbuild";
 
 export interface DeployOptions {
   contract?: string;
@@ -44,7 +42,7 @@ export class DeployAction extends BaseAction {
         target: "es2020",
         sourcemap: false,
       });
-     await this.executeJsScript(filePath, outFile, rpcUrl);
+      await this.executeJsScript(filePath, outFile, rpcUrl);
     } catch (error) {
       this.failSpinner(`Error executing: ${filePath}`, error);
     } finally {
@@ -52,13 +50,17 @@ export class DeployAction extends BaseAction {
     }
   }
 
-  private async executeJsScript(filePath: string, transpiledFilePath?: string, rpcUrl?: string): Promise<void> {
+  private async executeJsScript(
+    filePath: string,
+    transpiledFilePath?: string,
+    rpcUrl?: string,
+  ): Promise<void> {
     this.startSpinner(`Executing file: ${filePath}`);
     try {
       const module = await import(pathToFileURL(transpiledFilePath || filePath).href);
       if (!module.default || typeof module.default !== "function") {
         this.failSpinner(`No "default" function found in: ${filePath}`);
-        return
+        return;
       }
       const client = await this.getClient(rpcUrl);
       await module.default(client);
@@ -74,7 +76,8 @@ export class DeployAction extends BaseAction {
       this.failSpinner("No deploy folder found.");
       return;
     }
-    const files = fs.readdirSync(this.deployDir)
+    const files = fs
+      .readdirSync(this.deployDir)
       .filter(file => file.endsWith(".ts") || file.endsWith(".js"))
       .sort((a, b) => {
         const numA = parseInt(a.split("_")[0]);
@@ -127,16 +130,17 @@ export class DeployAction extends BaseAction {
       }
 
       const leaderOnly = false;
-      const deployParams: any = { code: contractCode, args: options.args, leaderOnly };
+      const deployParams: any = {code: contractCode, args: options.args, leaderOnly};
 
       this.setSpinnerText("Starting contract deployment...");
       this.log("Deployment Parameters:", deployParams);
 
       const hash = (await client.deployContract(deployParams)) as any;
+      this.log("Deployment Transaction Hash:", hash);
       const result = await client.waitForTransactionReceipt({
         hash,
-        retries: 15,
-        interval: 2000,
+        retries: 50,
+        interval: 5000,
         status: TransactionStatus.ACCEPTED,
       });
 

package/src/commands/contracts/index.ts

@@ -1,6 +1,7 @@
-import { Command } from "commander";
-import { DeployAction, DeployOptions, DeployScriptsOptions } from "./deploy";
-import { CallAction, CallOptions } from "./call";
+import {Command} from "commander";
+import {DeployAction, DeployOptions, DeployScriptsOptions} from "./deploy";
+import {CallAction, CallOptions} from "./call";
+import {WriteAction, WriteOptions} from "./write";
 
 function parseArg(value: string, previous: any[] = []): any[] {
   if (value === "true") return [...previous, true];
@@ -10,31 +11,55 @@ function parseArg(value: string, previous: any[] = []): any[] {
 }
 
 export function initializeContractsCommands(program: Command) {
-
   program
     .command("deploy")
     .description("Deploy intelligent contracts")
     .option("--contract <contractPath>", "Path to the smart contract to deploy")
     .option("--rpc <rpcUrl>", "RPC URL for the network")
-    .option("--args <args...>", "Positional arguments for the contract (space-separated, use quotes for multi-word arguments)", parseArg, [])
+    .option(
+      "--args <args...>",
+      "Positional arguments for the contract (space-separated, use quotes for multi-word arguments)",
+      parseArg,
+      [],
+    )
     .action(async (options: DeployOptions) => {
       const deployer = new DeployAction();
-      if(options.contract){
+      if (options.contract) {
         await deployer.deploy(options);
-      }else {
-        const deployScriptsOptions: DeployScriptsOptions = { rpc: options.rpc };
+      } else {
+        const deployScriptsOptions: DeployScriptsOptions = {rpc: options.rpc};
         await deployer.deployScripts(deployScriptsOptions);
       }
     });
 
   program
     .command("call <contractAddress> <method>")
-    .description("Call a contract method")
+    .description("Call a contract method without sending a transaction or changing the state")
     .option("--rpc <rpcUrl>", "RPC URL for the network")
-    .option("--args <args...>", "Positional arguments for the method (space-separated, use quotes for multi-word arguments)", parseArg, [])
+    .option(
+      "--args <args...>",
+      "Positional arguments for the method (space-separated, use quotes for multi-word arguments)",
+      parseArg,
+      [],
+    )
     .action(async (contractAddress: string, method: string, options: CallOptions) => {
       const caller = new CallAction();
-      await caller.call({ contractAddress, method, ...options });
+      await caller.call({contractAddress, method, ...options});
+    });
+
+  program
+    .command("write <contractAddress> <method>")
+    .description("Sends a transaction to a contract method that modifies the state")
+    .option("--rpc <rpcUrl>", "RPC URL for the network")
+    .option(
+      "--args <args...>",
+      "Positional arguments for the method (space-separated, use quotes for multi-word arguments)",
+      parseArg,
+      [],
+    )
+    .action(async (contractAddress: string, method: string, options: WriteOptions) => {
+      const writer = new WriteAction();
+      await writer.write({contractAddress, method, ...options});
     });
 
   return program;

package/src/commands/contracts/write.ts

@@ -0,0 +1,49 @@
+// import {simulator} from "genlayer-js/chains";
+// import type {GenLayerClient} from "genlayer-js/types";
+import {BaseAction} from "../../lib/actions/BaseAction";
+
+export interface WriteOptions {
+  args: any[];
+  rpc?: string;
+}
+
+export class WriteAction extends BaseAction {
+  constructor() {
+    super();
+  }
+
+  async write({
+    contractAddress,
+    method,
+    args,
+    rpc,
+  }: {
+    contractAddress: string;
+    method: string;
+    args: any[];
+    rpc?: string;
+  }): Promise<void> {
+    const client = await this.getClient(rpc);
+    await client.initializeConsensusSmartContract();
+    this.startSpinner(`Calling write method ${method} on contract at ${contractAddress}...`);
+
+    try {
+      const hash = await client.writeContract({
+        address: contractAddress as any,
+        functionName: method,
+        args,
+        value: 0n,
+      });
+      this.log("Write Transaction Hash:", hash);
+
+      const result = await client.waitForTransactionReceipt({
+        hash,
+        retries: 100,
+        interval: 5000,
+      });
+      this.succeedSpinner("Write operation successfully executed", result);
+    } catch (error) {
+      this.failSpinner("Error during write operation", error);
+    }
+  }
+}

package/src/commands/general/init.ts

@@ -1,10 +1,10 @@
 import inquirer from "inquirer";
-import { DistinctQuestion } from "inquirer";
-import { ISimulatorService } from "../../lib/interfaces/ISimulatorService";
-import { AI_PROVIDERS_CONFIG, AiProviders } from "../../lib/config/simulator";
-import { OllamaAction } from "../update/ollama";
-import { BaseAction } from "../../lib/actions/BaseAction";
-import { SimulatorService } from "../../lib/services/simulator";
+import {DistinctQuestion} from "inquirer";
+import {ISimulatorService} from "../../lib/interfaces/ISimulatorService";
+import {AI_PROVIDERS_CONFIG, AiProviders} from "../../lib/config/simulator";
+import {OllamaAction} from "../update/ollama";
+import {BaseAction} from "../../lib/actions/BaseAction";
+import {SimulatorService} from "../../lib/services/simulator";
 
 export interface InitActionOptions {
   numValidators: number;
@@ -14,14 +14,14 @@ export interface InitActionOptions {
   ollama: boolean;
 }
 
-function getRequirementsErrorMessage({ docker }: Record<string, boolean>): string {
+function getRequirementsErrorMessage({docker}: Record<string, boolean>): string {
   if (!docker) {
     return "Docker is not installed. Please install Docker and try again.\n";
   }
   return "";
 }
 
-function getVersionErrorMessage({ docker, node }: Record<string, string>): string {
+function getVersionErrorMessage({docker, node}: Record<string, string>): string {
   let message = "";
 
   if (docker) {
@@ -73,7 +73,7 @@ export class InitAction extends BaseAction {
       const confirmMessage = isRunning
         ? `GenLayer Localnet is already running and this command is going to reset GenLayer docker images and containers, providers API Keys, and GenLayer database (accounts, transactions, validators and logs). Contract code (gpy files) will be kept. Do you want to continue?`
         : `This command is going to reset GenLayer docker images and containers, providers API Keys, and GenLayer database (accounts, transactions, validators and logs). Contract code (gpy files) will be kept. Do you want to continue?`;
-      
+
       await this.confirmPrompt(confirmMessage);
 
       this.logInfo(`Initializing GenLayer CLI with ${options.numValidators} validators`);
@@ -90,8 +90,7 @@ export class InitAction extends BaseAction {
           name: "selectedLlmProviders",
           message: "Select which LLM providers do you want to use:",
           choices: this.simulatorService.getAiProvidersOptions(true, options.ollama ? [] : ["ollama"]),
-          validate: (answer) =>
-            answer.length < 1 ? "You must choose at least one option." : true,
+          validate: answer => (answer.length < 1 ? "You must choose at least one option." : true),
         },
       ];
       const llmProvidersAnswer = await inquirer.prompt(llmQuestions);
@@ -100,7 +99,7 @@ export class InitAction extends BaseAction {
       let defaultOllamaModel = this.getConfig().defaultOllamaModel;
       const aiProvidersEnvVars: Record<string, string> = {};
       const configurableAiProviders = selectedLlmProviders.filter(
-        (provider: AiProviders) => AI_PROVIDERS_CONFIG[provider].envVar
+        (provider: AiProviders) => AI_PROVIDERS_CONFIG[provider].envVar,
       );
       for (const provider of configurableAiProviders) {
         const providerConfig = AI_PROVIDERS_CONFIG[provider];
@@ -119,14 +118,13 @@ export class InitAction extends BaseAction {
 
       this.startSpinner("Configuring GenLayer Localnet environment...");
       this.simulatorService.addConfigToEnvFile(aiProvidersEnvVars);
-      this.simulatorService.addConfigToEnvFile({ LOCALNETVERSION: localnetVersion });
+      this.simulatorService.addConfigToEnvFile({LOCALNETVERSION: localnetVersion});
 
       this.setSpinnerText("Running GenLayer Localnet...");
       await this.simulatorService.runSimulator();
 
       this.setSpinnerText("Waiting for localnet to be ready...");
-      const { initialized, errorCode, errorMessage } =
-        await this.simulatorService.waitForSimulatorToBeReady();
+      const {initialized, errorCode, errorMessage} = await this.simulatorService.waitForSimulatorToBeReady();
       if (!initialized) {
         if (errorCode === "ERROR") {
           this.failSpinner(`Unable to initialize the GenLayer Localnet: ${errorMessage}`);
@@ -134,7 +132,7 @@ export class InitAction extends BaseAction {
         }
         if (errorCode === "TIMEOUT") {
           this.failSpinner(
-            "The localnet is taking too long to initialize. Please try again after the localnet is ready."
+            "The localnet is taking too long to initialize. Please try again after the localnet is ready.",
           );
           return;
         }
@@ -153,10 +151,7 @@ export class InitAction extends BaseAction {
 
       this.startSpinner("Initializing validators...");
       await this.simulatorService.deleteAllValidators();
-      await this.simulatorService.createRandomValidators(
-        Number(options.numValidators),
-        selectedLlmProviders
-      );
+      await this.simulatorService.createRandomValidators(Number(options.numValidators), selectedLlmProviders);
 
       if (options.resetDb) {
         this.setSpinnerText("Cleaning database...");

package/src/commands/general/start.ts

@@ -1,8 +1,8 @@
 import inquirer from "inquirer";
-import { ISimulatorService } from "../../lib/interfaces/ISimulatorService";
-import { DistinctQuestion } from "inquirer";
-import { BaseAction } from "../../lib/actions/BaseAction";
-import { SimulatorService } from "../../lib/services/simulator";
+import {ISimulatorService} from "../../lib/interfaces/ISimulatorService";
+import {DistinctQuestion} from "inquirer";
+import {BaseAction} from "../../lib/actions/BaseAction";
+import {SimulatorService} from "../../lib/services/simulator";
 
 export interface StartActionOptions {
   resetValidators: boolean;
@@ -21,7 +21,7 @@ export class StartAction extends BaseAction {
   }
 
   async execute(options: StartActionOptions) {
-    const { resetValidators, numValidators, headless, resetDb, ollama } = options;
+    const {resetValidators, numValidators, headless, resetDb, ollama} = options;
 
     this.simulatorService.setComposeOptions(headless, ollama);
     this.startSpinner("Checking CLI version...");
@@ -49,7 +49,7 @@ export class StartAction extends BaseAction {
 
     try {
       this.setSpinnerText("Waiting for the simulator to be ready...");
-      const { initialized, errorCode, errorMessage } = await this.simulatorService.waitForSimulatorToBeReady();
+      const {initialized, errorCode, errorMessage} = await this.simulatorService.waitForSimulatorToBeReady();
 
       if (!initialized) {
         if (errorCode === "ERROR") {
@@ -61,7 +61,6 @@ export class StartAction extends BaseAction {
           return;
         }
       }
-
     } catch (error) {
       this.failSpinner("Error waiting for the simulator to be ready", error);
       return;
@@ -83,12 +82,15 @@ export class StartAction extends BaseAction {
             name: "selectedLlmProviders",
             message: "Select which LLM providers do you want to use:",
             choices: this.simulatorService.getAiProvidersOptions(false, ollama ? [] : ["ollama"]),
-            validate: (answer) => (answer.length < 1 ? "You must choose at least one option." : true),
+            validate: answer => (answer.length < 1 ? "You must choose at least one option." : true),
           },
         ];
 
         const llmProvidersAnswer = await inquirer.prompt(questions);
-        await this.simulatorService.createRandomValidators(numValidators, llmProvidersAnswer.selectedLlmProviders);
+        await this.simulatorService.createRandomValidators(
+          numValidators,
+          llmProvidersAnswer.selectedLlmProviders,
+        );
       } catch (error) {
         this.failSpinner("Unable to initialize the validators", error);
         return;
@@ -96,7 +98,9 @@ export class StartAction extends BaseAction {
     }
 
     let successMessage = "GenLayer simulator initialized successfully! ";
-    successMessage += headless ? "" : `Go to ${this.simulatorService.getFrontendUrl()} in your browser to access it.`;
+    successMessage += headless
+      ? ""
+      : `Go to ${this.simulatorService.getFrontendUrl()} in your browser to access it.`;
     this.succeedSpinner(successMessage);
 
     if (!headless) {

package/src/commands/keygen/create.ts

@@ -1,4 +1,4 @@
-import { BaseAction } from "../../lib/actions/BaseAction";
+import {BaseAction} from "../../lib/actions/BaseAction";
 
 export interface CreateKeypairOptions {
   output: string;
@@ -19,4 +19,4 @@ export class KeypairCreator extends BaseAction {
       this.failSpinner("Failed to generate keypair", error);
     }
   }
-}
+}

package/src/commands/network/index.ts

@@ -0,0 +1,14 @@
+import {Command} from "commander";
+import {NetworkActions} from "./setNetwork";
+
+export function initializeNetworkCommands(program: Command) {
+  const networkActions = new NetworkActions();
+
+  program
+    .command("network")
+    .description("Set the network to use")
+    .argument("[network]", "The network to use")
+    .action((networkName?: string) => networkActions.setNetwork(networkName));
+
+  return program;
+}

package/src/commands/network/setNetwork.ts

@@ -0,0 +1,60 @@
+import {AiProviders} from "@/lib/config/simulator";
+import {BaseAction} from "../../lib/actions/BaseAction";
+import inquirer, {DistinctQuestion} from "inquirer";
+import {localnet, studionet, testnetAsimov} from "genlayer-js/chains";
+import {} from "genlayer-js/chains";
+
+const networks = [
+  {
+    name: localnet.name,
+    alias: "localnet",
+    value: localnet,
+  },
+  {
+    name: studionet.name,
+    alias: "studionet",
+    value: studionet,
+  },
+  {
+    name: testnetAsimov.name,
+    alias: "testnet-asimov",
+    value: testnetAsimov,
+  },
+];
+
+export class NetworkActions extends BaseAction {
+  constructor() {
+    super();
+  }
+
+  async setNetwork(networkName?: string): Promise<void> {
+    if (networkName || networkName === "") {
+      if (!networks.some(n => n.name === networkName || n.alias === networkName)) {
+        this.failSpinner(`Network ${networkName} not found`);
+        return;
+      }
+      const selectedNetwork = networks.find(n => n.name === networkName || n.alias === networkName);
+      if (!selectedNetwork) {
+        this.failSpinner(`Network ${networkName} not found`);
+        return;
+      }
+      this.writeConfig("network", JSON.stringify(selectedNetwork.value));
+      this.succeedSpinner(`Network successfully set to ${selectedNetwork.name}`);
+      return;
+    }
+
+    const networkQuestions: DistinctQuestion[] = [
+      {
+        type: "list",
+        name: "selectedNetwork",
+        message: "Select which network do you want to use:",
+        choices: networks,
+      },
+    ];
+    const networkAnswer = await inquirer.prompt(networkQuestions);
+    const selectedNetwork = networkAnswer.selectedNetwork;
+
+    this.writeConfig("network", JSON.stringify(selectedNetwork));
+    this.succeedSpinner(`Network successfully set to ${selectedNetwork.name}`);
+  }
+}

package/src/index.ts

@@ -2,13 +2,14 @@
 import {program} from "commander";
 import {version} from "../package.json";
 import {CLI_DESCRIPTION} from "../src/lib/config/text";
-import { initializeGeneralCommands } from "../src/commands/general";
-import { initializeKeygenCommands } from "../src/commands/keygen";
-import { initializeContractsCommands } from "../src/commands/contracts";
-import { initializeConfigCommands } from "../src/commands/config";
+import {initializeGeneralCommands} from "../src/commands/general";
+import {initializeKeygenCommands} from "../src/commands/keygen";
+import {initializeContractsCommands} from "../src/commands/contracts";
+import {initializeConfigCommands} from "../src/commands/config";
 import {initializeValidatorCommands} from "../src/commands/validators";
-import { initializeUpdateCommands } from "../src/commands/update";
-import { initializeScaffoldCommands } from "../src/commands/scaffold";
+import {initializeUpdateCommands} from "../src/commands/update";
+import {initializeScaffoldCommands} from "../src/commands/scaffold";
+import {initializeNetworkCommands} from "../src/commands/network";
 
 export function initializeCLI() {
   program.version(version).description(CLI_DESCRIPTION);
@@ -16,9 +17,10 @@ export function initializeCLI() {
   initializeKeygenCommands(program);
   initializeContractsCommands(program);
   initializeConfigCommands(program);
-  initializeUpdateCommands(program)
+  initializeUpdateCommands(program);
   initializeValidatorCommands(program);
   initializeScaffoldCommands(program);
+  initializeNetworkCommands(program);
   program.parse(process.argv);
 }
 

package/src/lib/actions/BaseAction.ts

@@ -1,29 +1,31 @@
-import { ConfigFileManager } from "../../lib/config/ConfigFileManager";
-import ora, { Ora } from "ora";
+import {ConfigFileManager} from "../../lib/config/ConfigFileManager";
+import ora, {Ora} from "ora";
 import chalk from "chalk";
 import inquirer from "inquirer";
-import { KeypairManager } from "../accounts/KeypairManager";
-import { createClient, createAccount } from "genlayer-js";
-import { localnet } from "genlayer-js/chains";
-import type { GenLayerClient } from "genlayer-js/types";
+import {KeypairManager} from "../accounts/KeypairManager";
+import {createClient, createAccount} from "genlayer-js";
+import {localnet} from "genlayer-js/chains";
+import type {GenLayerClient, GenLayerChain} from "genlayer-js/types";
 
 export class BaseAction extends ConfigFileManager {
   protected keypairManager: KeypairManager;
   private spinner: Ora;
-  private _genlayerClient: GenLayerClient<typeof localnet> | null = null;
+  private _genlayerClient: GenLayerClient<GenLayerChain> | null = null;
 
   constructor() {
-    super()
-    this.spinner = ora({ text: "", spinner: "dots" });
+    super();
+    this.spinner = ora({text: "", spinner: "dots"});
     this.keypairManager = new KeypairManager();
   }
 
-  protected async getClient(rpcUrl?: string): Promise<GenLayerClient<typeof localnet>> {
+  protected async getClient(rpcUrl?: string): Promise<GenLayerClient<GenLayerChain>> {
     if (!this._genlayerClient) {
+      const networkConfig = this.getConfig().network;
+      const network = networkConfig ? JSON.parse(networkConfig) : localnet;
       this._genlayerClient = createClient({
-        chain: localnet,
+        chain: network,
         endpoint: rpcUrl,
-        account: createAccount(await this.getPrivateKey() as any),
+        account: createAccount((await this.getPrivateKey()) as any),
       });
     }
     return this._genlayerClient;
@@ -64,11 +66,11 @@ export class BaseAction extends ConfigFileManager {
       };
       return JSON.stringify(errorDetails, null, 2);
     }
-    
+
     if (data instanceof Map) {
       data = Object.fromEntries(data);
     }
-    
+
     return typeof data === "object" ? JSON.stringify(data, null, 2) : String(data);
   }
 
@@ -103,12 +105,12 @@ export class BaseAction extends ConfigFileManager {
   }
 
   protected succeedSpinner(message: string, data?: any): void {
-    if (data !== undefined) this.log('Result:', data);
+    if (data !== undefined) this.log("Result:", data);
     this.spinner.succeed(chalk.green(message));
   }
 
-  protected failSpinner(message: string, error?:any): void {
-    if (error) this.log('Error:', error);
+  protected failSpinner(message: string, error?: any): void {
+    if (error) this.log("Error:", error);
     this.spinner.fail(chalk.red(message));
   }
 
@@ -119,4 +121,4 @@ export class BaseAction extends ConfigFileManager {
   protected setSpinnerText(message: string): void {
     this.spinner.text = chalk.blue(message);
   }
-}
+}