fix: name and remove unused param

Author: vvava

src/background/providers/ChainAgnosticProvider.test.ts

@@ -1,6 +1,6 @@
 import { ethErrors } from 'eth-rpc-errors';
 import AutoPairingPostMessageConnection from '../utils/messaging/AutoPairingPostMessageConnection';
-import { ChainAgnostinProvider } from './ChainAgnosticProvider';
+import { ChainAgnosticProvider } from './ChainAgnosticProvider';
 
 jest.mock('./utils/onDomReady');
 jest.mock('../utils/messaging/AutoPairingPostMessageConnection', () => {
@@ -16,15 +16,15 @@ describe('src/background/providers/ChainAgnosticProvider', () => {
 
   describe('initialization', () => {
     it('should connect to the backgroundscript', async () => {
-      new ChainAgnostinProvider(channelMock);
+      new ChainAgnosticProvider(channelMock);
 
       expect(channelMock.connect).toHaveBeenCalled();
       expect(channelMock.request).not.toHaveBeenCalled();
     });
     it('waits for message channel to be connected', async () => {
       const mockedChannel = new AutoPairingPostMessageConnection(false);
 
-      const provider = new ChainAgnostinProvider(channelMock);
+      const provider = new ChainAgnosticProvider(channelMock);
       expect(mockedChannel.connect).toHaveBeenCalled();
       expect(mockedChannel.request).not.toHaveBeenCalled();
 
@@ -39,7 +39,7 @@ describe('src/background/providers/ChainAgnosticProvider', () => {
 
   describe('request', () => {
     it('should use the rate limits on `eth_requestAccounts` requests', async () => {
-      const provider = new ChainAgnostinProvider(channelMock);
+      const provider = new ChainAgnosticProvider(channelMock);
       (channelMock.request as jest.Mock).mockResolvedValue('success');
 
       const firstCallCallback = jest.fn();
@@ -66,7 +66,7 @@ describe('src/background/providers/ChainAgnosticProvider', () => {
       );
     });
     it('shoud not use the rate limits on `random_method` requests', async () => {
-      const provider = new ChainAgnostinProvider(channelMock);
+      const provider = new ChainAgnosticProvider(channelMock);
       (channelMock.request as jest.Mock).mockResolvedValue('success');
 
       const firstCallCallback = jest.fn();
@@ -90,7 +90,7 @@ describe('src/background/providers/ChainAgnosticProvider', () => {
     });
 
     it('should call the request of the connection', async () => {
-      const provider = new ChainAgnostinProvider(channelMock);
+      const provider = new ChainAgnosticProvider(channelMock);
       (channelMock.request as jest.Mock).mockResolvedValueOnce('success');
 
       await provider.request({
@@ -102,7 +102,7 @@ describe('src/background/providers/ChainAgnosticProvider', () => {
     });
     describe('CAIP-27', () => {
       it('should wrap the incoming request into CAIP-27 envelope and reuses the provided ID', async () => {
-        const provider = new ChainAgnostinProvider(channelMock);
+        const provider = new ChainAgnosticProvider(channelMock);
         // response for the actual call
         (channelMock.request as jest.Mock).mockResolvedValueOnce('success');
 

src/background/providers/ChainAgnosticProvider.ts

@@ -9,7 +9,7 @@ import AbstractConnection from '../utils/messaging/AbstractConnection';
 import { ChainId } from '@avalabs/core-chains-sdk';
 import RequestRatelimiter from './utils/RequestRatelimiter';
 
-export class ChainAgnostinProvider extends EventEmitter {
+export class ChainAgnosticProvider extends EventEmitter {
   #contentScriptConnection: AbstractConnection;
 
   #requestRateLimiter = new RequestRatelimiter([
@@ -68,19 +68,14 @@ export class ChainAgnostinProvider extends EventEmitter {
   };
 
   request = async ({
-    internal,
     data,
     sessionId,
     chainId,
   }: {
-    internal?: boolean;
     data: PartialBy<JsonRpcRequestPayload, 'id' | 'params'>;
     sessionId: string;
     chainId: string | null;
   }) => {
-    if (internal) {
-      return this.#request({ data, chainId, sessionId });
-    }
     return this.#requestRateLimiter.call(data.method, () =>
       this.#request({ data, chainId, sessionId })
     );

src/background/providers/CoreProvider.ts

@@ -17,7 +17,7 @@ import {
   type JsonRpcRequestPayload,
 } from '../connections/dAppConnection/models';
 import type { PartialBy, ProviderInfo } from '../models';
-import { ChainAgnostinProvider } from './ChainAgnosticProvider';
+import { ChainAgnosticProvider } from './ChainAgnosticProvider';
 import AbstractConnection from '../utils/messaging/AbstractConnection';
 
 interface ProviderState {
@@ -30,7 +30,7 @@ interface ProviderState {
 export class CoreProvider extends EventEmitter {
   #providerReadyPromise = new ProviderReadyPromise();
   #contentScriptConnection: AbstractConnection;
-  #chainagnosticProvider?: ChainAgnostinProvider;
+  #chainagnosticProvider?: ChainAgnosticProvider;
 
   readonly info: ProviderInfo = {
     name: EVM_PROVIDER_INFO_NAME,

src/background/providers/initializeInpageProvider.ts

@@ -1,5 +1,5 @@
 import type AbstractConnection from '../utils/messaging/AbstractConnection';
-import { ChainAgnostinProvider } from './ChainAgnosticProvider';
+import { ChainAgnosticProvider } from './ChainAgnosticProvider';
 import { CoreProvider } from './CoreProvider';
 import { createMultiWalletProxy } from './MultiWalletProviderProxy';
 import { EventNames, type EIP6963ProviderDetail } from './models';
@@ -18,7 +18,7 @@ export function initializeProvider(
   globalObject = window
 ): CoreProvider {
   const chainAgnosticProvider = new Proxy(
-    new ChainAgnostinProvider(connection),
+    new ChainAgnosticProvider(connection),
     {
       deleteProperty: () => true,
     }
@@ -149,10 +149,10 @@ function announceWalletProvider(
 }
 
 function announceChainAgnosticProvider(
-  providerInstance: ChainAgnostinProvider,
+  providerInstance: ChainAgnosticProvider,
   globalObject = window
 ): void {
-  const announceEvent = new CustomEvent<{ provider: ChainAgnostinProvider }>(
+  const announceEvent = new CustomEvent<{ provider: ChainAgnosticProvider }>(
     EventNames.CORE_WALLET_ANNOUNCE_PROVIDER,
     {
       detail: Object.freeze({