refactor: encapsulate provider logic in utils/provider.ts, eliminate ethQuery from transaction-controller

Replace all ethQuery/EthQuery usage with a new rpcRequest() utility that
accepts messenger + chainId/networkClientId + method + params. Any file
can now send RPC requests without passing around ethQuery instances.

- Add utils/provider.ts with rpcRequest() and getProvider() functions
- Migrate all utils, gas-flows, helpers, hooks, and TransactionController
- Update GasFeeFlowRequest type: ethQuery -> networkClientId (BREAKING)
- Update determineTransactionType signature to options object (BREAKING)
- Remove @metamask/eth-query dependency
- Update all test files to match new signatures

Author: matthewwalsh0

.gitignore

@@ -34,4 +34,7 @@ scripts/coverage
 !.yarn/versions
 
 # typescript
-packages/*/*.tsbuildinfo
+packages/*/*.tsbuildinfo
+
+# AI
+.sisyphus/

packages/transaction-controller/CHANGELOG.md

@@ -55,12 +55,20 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Add optional `sourceHash` field to `MetamaskPayMetadata` for tracking source chain transaction hashes when no local transaction exists ([#8133](https://github.com/MetaMask/core/pull/8133))
 - Add `predictDepositAndOrder` to `TransactionType` ([#8135](https://github.com/MetaMask/core/pull/8135))
+- Add `rpcRequest` and `getProvider` utility functions in `utils/provider.ts` for simplified RPC access using messenger + chainId/networkClientId
 
 ### Changed
 
+- **BREAKING:** `GasFeeFlowRequest` type now uses `networkClientId` instead of `ethQuery`
+- **BREAKING:** `determineTransactionType` second parameter changed from `ethQuery` to an options object with `messenger` and `networkClientId`
+- **BREAKING:** Removed `ethQuery`/`EthQuery` from all internal types and utility function signatures, replaced with `rpcRequest` from `utils/provider`
 - Use distinct error message when failing incomplete transactions at startup whose required transactions are all confirmed ([#8189](https://github.com/MetaMask/core/pull/8189))
 - Bump `@metamask/core-backend` from `^6.1.0` to `^6.1.1` ([#8162](https://github.com/MetaMask/core/pull/8162))
 
+### Removed
+
+- **BREAKING:** Removed `@metamask/eth-query` as a dependency
+
 ### Fixed
 
 - Use effective recipient (decoded from tx data for token transfers) when checking for existing transactions in first-time interaction logic ([#8130](https://github.com/MetaMask/core/pull/8130))

packages/transaction-controller/package.json

@@ -60,7 +60,6 @@
     "@metamask/base-controller": "^9.0.0",
     "@metamask/controller-utils": "^11.19.0",
     "@metamask/core-backend": "^6.2.0",
-    "@metamask/eth-query": "^4.0.0",
     "@metamask/gas-fee-controller": "^26.1.0",
     "@metamask/messenger": "^0.3.0",
     "@metamask/metamask-eth-abis": "^3.1.1",