refactor: clean wallet files

src/bearbyWallet/BearbyWallet.ts

@@ -27,17 +27,11 @@ export class BearbyWallet implements Wallet {
     return [new BearbyAccount(await web3.wallet.account.base58)];
   }
 
-  public async importAccount(
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
-    publicKey: string,
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
-    privateKey: string,
-  ): Promise<void> {
+  public async importAccount(): Promise<void> {
     throw new Error('Method not implemented.');
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  public async deleteAccount(address: string): Promise<void> {
+  public async deleteAccount(): Promise<void> {
     throw new Error('Method not implemented.');
   }
 
@@ -48,8 +42,7 @@ export class BearbyWallet implements Wallet {
     return networkInfos();
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  public async setRpcUrl(url: string): Promise<void> {
+  public async setRpcUrl(): Promise<void> {
     throw new Error(
       'setRpcUrl is not yet implemented for the current provider.',
     );

src/metamaskSnap/MetamaskWallet.ts

@@ -54,8 +54,7 @@ export class MetamaskWallet implements Wallet {
     throw new Error('Method not implemented.');
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  public async deleteAccount(address: string): Promise<void> {
+  public async deleteAccount(): Promise<void> {
     throw new Error('Method not implemented.');
   }
 
@@ -70,6 +69,12 @@ export class MetamaskWallet implements Wallet {
     };
   }
 
+  /**
+   * Sets the RPC URL for the MetaMask provider.
+   *
+   * @param url - The new RPC URL.
+   * @returns A promise that resolves when the RPC URL is updated.
+   */
   public async setRpcUrl(url: string): Promise<void> {
     await setRpcUrl(this.metamaskProvider, { network: url });
   }
@@ -87,21 +92,15 @@ export class MetamaskWallet implements Wallet {
   /**
    * Subscribes to network changes.
    *
-   * @param callback - Callback to be called when the network changes.
-   *
-   * @returns A promise that resolves to a function that can be called to unsubscribe.
-   *
-   * @remarks
-   * Don't forget to unsubscribe to avoid memory leaks.
+   * @param callback - Callback function called when the network changes.
+   * @returns An object with an `unsubscribe` method to stop listening.
+   * @remarks Periodically checks for network changes every 500ms.
    *
    * @example
    * ```typescript
-   * // Subscribe
    * const observer = await provider.listenNetworkChanges((network) => {
-   *  console.log(network);
+   *   console.log(network);
    * });
-   *
-   * // Unsubscribe
    * observer.unsubscribe();
    * ```
    */
@@ -110,7 +109,6 @@ export class MetamaskWallet implements Wallet {
   ): { unsubscribe: () => void } | undefined {
     this.eventsListener.on(METAMASK_NETWORK_CHANGED, (evt) => callback(evt));
 
-    // check periodically if network changed
     const intervalId = setInterval(async () => {
       const network = await this.networkInfos();
       if (!this.currentNetwork) {
@@ -135,6 +133,11 @@ export class MetamaskWallet implements Wallet {
     };
   }
 
+  /**
+   * Connects to MetaMask and ensures it is unlocked and ready.
+   *
+   * @returns A promise that resolves to `true` if connected successfully, otherwise `false`.
+   */
   public async connect() {
     try {
       const isUnlocked = await isMetaMaskUnlocked();

src/metamaskSnap/services/buyRolls.ts

@@ -10,12 +10,6 @@ export type BuyRollsResponse = {
   operationId: string;
 };
 
-/**
- * Function that calls the MetaMask provider to buy rolls
- * @param provider - The MetaMask provider
- * @param params - The buy rolls parameters (fee and amount)
- * @returns The operationId of the operation
- */
 export const buyRolls = async (
   provider: MetaMaskInpageProvider,
   params: BuyRollsParams,

src/metamaskSnap/services/callSC.ts

@@ -14,12 +14,6 @@ export type CallSCResponse = {
   operationId: string;
 };
 
-/**
- * Function that calls the MetaMask provider to call a smart contract
- * @param provider - The MetaMask provider
- * @param params - The call smart contract parameters (see massa standard)
- * @returns The response of the operation
- */
 export const callSC = async (
   provider: MetaMaskInpageProvider,
   params: CallSCParams,

src/metamaskSnap/services/getActiveAccount.ts

@@ -5,11 +5,6 @@ export type ActiveAccountResponse = {
   address: string;
 };
 
-/**
- * Function that calls the MetaMask provider to get the active account
- * @param provider - The MetaMask provider
- * @returns The active account address
- */
 export const getActiveAccount = async (
   provider: MetaMaskInpageProvider,
 ): Promise<ActiveAccountResponse | undefined> => {

src/metamaskSnap/services/getMinimalFees.ts

@@ -2,11 +2,6 @@ import { MetaMaskInpageProvider } from '@metamask/providers';
 import { getNetwork } from './getNetwork';
 import { Mas } from '@massalabs/massa-web3';
 
-/**
- * Function that calls the MetaMask provider to get the current network
- * @param provider - The MetaMask provider
- * @returns The current network details
- */
 export const getMinimalFees = async (
   provider: MetaMaskInpageProvider,
 ): Promise<bigint> => {

src/metamaskSnap/services/getNetwork.ts

@@ -7,11 +7,6 @@ export type NetworkResponse = {
   minimalFees: string;
 };
 
-/**
- * Function that calls the MetaMask provider to get the current network
- * @param provider - The MetaMask provider
- * @returns The current network details
- */
 export const getNetwork = async (
   provider: MetaMaskInpageProvider,
 ): Promise<NetworkResponse | undefined> => {

src/metamaskSnap/services/readSC.ts

@@ -18,12 +18,6 @@ export type ReadSCResponse = {
   };
 };
 
-/**
- * Function that calls the MetaMask provider to read a smart contract
- * @param provider - The MetaMask provider
- * @param params - The read smart contract parameters (see massa standard)
- * @returns The response of the operation
- */
 export const readSC = async (
   provider: MetaMaskInpageProvider,
   params: ReadSCParameters,

src/metamaskSnap/services/sellRolls.ts

@@ -10,12 +10,6 @@ export type SellRollsResponse = {
   operationId: string;
 };
 
-/**
- * Function that calls the MetaMask provider to sell rolls
- * @param provider - The MetaMask provider
- * @param params - The sell rolls parameters (amount and fee)
- * @returns The operation id of the sell rolls operation
- */
 export const sellRolls = async (
   provider: MetaMaskInpageProvider,
   params: SellRollsParams,

src/metamaskSnap/services/setRpcUrl.ts

@@ -9,12 +9,6 @@ export type SetRpcUrlResponse = {
   network: string; // url
 };
 
-/**
- * Function that calls the MetaMask provider to set a new rpc url
- * @param provider - The MetaMask provider
- * @param params - The set network parameters (network id to set as a string)
- * @returns The response of the operation
- */
 export const setRpcUrl = async (
   provider: MetaMaskInpageProvider,
   params: SetRpcUrlParams,

src/metamaskSnap/services/signMessage.ts

@@ -10,12 +10,6 @@ export type SignMessageResponse = {
   publicKey: string;
 };
 
-/**
- * Function that calls the MetaMask provider to sign a message
- * @param provider - The MetaMask provider
- * @param params - The sign message parameters (serialized data message as bytes)
- * @returns The signature and public key used to sign the message
- */
 export const signMessage = async (
   provider: MetaMaskInpageProvider,
   params: SignMessageParams,

src/metamaskSnap/services/transfer.ts

@@ -11,13 +11,6 @@ export type TransferResponse = {
   operationId: string;
 };
 
-/**
- * Function that calls the MetaMask provider to transfer funds
- * @param provider - The MetaMask provider
- * @param params - The transfer parameters (recipient address, amount, and fee)
- * @returns The operation id of the transfer operation
- * @throws If the user denies the transaction
- */
 export const transfer = async (
   provider: MetaMaskInpageProvider,
   params: TransferParams,