chore: Migrate @metamask/eth-json-rpc-middleware to /packages (#6866)

## Explanation

Finalizes the migration of `@metamask/eth-json-rpc-middleware` from its
own repository to the core monorepo per "Phase C" of [the migration
process](https://github.com/MetaMask/core/blob/1bb289e3d9a76291ba66defe4117792600231751/docs/package-migration-process-guide.md#pr6-1-migrate-the-source-repos-git-history-into-the-merged-packages-temporary-directory-in-core).

## References

N/A

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've communicated my changes to consumers by [updating changelogs
for packages I've
changed](https://github.com/MetaMask/core/tree/main/docs/contributing.md#updating-changelogs),
highlighting breaking changes as necessary
- [x] I've prepared draft pull requests for clients and consumer
packages to resolve any breaking changes

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Migrates @metamask/eth-json-rpc-middleware into /packages, bumps Node
to ^18.18 and eth-block-tracker to ^13, updates network-controller to
middleware v20, and adjusts ESLint/Jest/ownership/TS configs.
> 
> - **Packages**:
> - **@metamask/eth-json-rpc-middleware**: Added to `packages/` with
build/test scripts, changelog, keywords/homepage/bugs updates, Node
engine `^18.18`, and coverage thresholds; minor test/doc tweaks.
>   - Bump `@metamask/eth-block-tracker` to `^13.0.0`.
> - **Consumers**:
> - **@metamask/network-controller**: Update dependency to
`@metamask/eth-json-rpc-middleware@^20.0.0`; changelog and TS project
references updated.
> - **Repo config**:
> - **ESLint**: Add package-specific overrides; broaden test globs;
update warning thresholds.
> - **Ownership**: Update `.github/CODEOWNERS` and `teams.json` to
include `eth-json-rpc-middleware` and related paths; adjust some owners.
> - **TypeScript**: Add `eth-json-rpc-middleware` to root and consumer
`tsconfig` references; exclude tests from its build.
> - **Yarn/constraints**: Update constraints (license/export checks) and
lockfile for new/updated deps.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
efd2d67. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: rekmarks

.github/CODEOWNERS

@@ -67,6 +67,7 @@
 /packages/composable-controller        @MetaMask/core-platform
 /packages/controller-utils             @MetaMask/core-platform
 /packages/error-reporting-service      @MetaMask/core-platform
+/packages/eth-json-rpc-middleware      @MetaMask/core-platform
 /packages/messenger                    @MetaMask/core-platform
 /packages/sample-controllers           @MetaMask/core-platform
 /packages/polling-controller           @MetaMask/core-platform
@@ -82,18 +83,20 @@
 /packages/subscription-controller        @MetaMask/web3auth
 
 ## Joint team ownership
+/packages/core-backend                          @MetaMask/core-platform @MetaMask/metamask-assets
 /packages/eth-block-tracker            @MetaMask/wallet-integrations @MetaMask/core-platform
-/packages/eth-json-rpc-provider        @MetaMask/wallet-integrations @MetaMask/core-platform
-/packages/json-rpc-engine              @MetaMask/wallet-integrations @MetaMask/core-platform
-/packages/json-rpc-middleware-stream   @MetaMask/wallet-integrations @MetaMask/core-platform
-/packages/keyring-controller           @MetaMask/accounts-engineers @MetaMask/core-platform
-/packages/multichain-network-controller  @MetaMask/core-platform @MetaMask/accounts-engineers @MetaMask/metamask-assets
-/packages/network-controller           @MetaMask/core-platform @MetaMask/metamask-assets
-/packages/permission-controller        @MetaMask/wallet-integrations @MetaMask/core-platform
-/packages/permission-log-controller    @MetaMask/wallet-integrations @MetaMask/core-platform
-/packages/remote-feature-flag-controller @MetaMask/extension-platform @MetaMask/mobile-platform
-/packages/foundryup                    @MetaMask/mobile-platform @MetaMask/extension-platform
-/packages/core-backend                 @MetaMask/core-platform @MetaMask/metamask-assets
+/packages/eth-json-rpc-middleware/src/methods   @MetaMask/confirmations @MetaMask/wallet-api-platform-engineers
+/packages/eth-json-rpc-middleware/src/wallet.*  @MetaMask/confirmations @MetaMask/wallet-api-platform-engineers
+/packages/eth-json-rpc-provider                 @MetaMask/wallet-integrations @MetaMask/core-platform
+/packages/foundryup                             @MetaMask/mobile-platform @MetaMask/extension-platform
+/packages/json-rpc-engine                       @MetaMask/wallet-integrations @MetaMask/core-platform
+/packages/json-rpc-middleware-stream            @MetaMask/wallet-integrations @MetaMask/core-platform
+/packages/keyring-controller                    @MetaMask/accounts-engineers @MetaMask/core-platform
+/packages/multichain-network-controller         @MetaMask/core-platform @MetaMask/accounts-engineers @MetaMask/metamask-assets
+/packages/network-controller                    @MetaMask/core-platform @MetaMask/metamask-assets
+/packages/permission-controller                 @MetaMask/wallet-integrations @MetaMask/core-platform
+/packages/permission-log-controller             @MetaMask/wallet-integrations @MetaMask/core-platform
+/packages/remote-feature-flag-controller        @MetaMask/extension-platform @MetaMask/mobile-platform @MetaMask/core-platform
 
 ## Package Release related
 /packages/account-tree-controller/package.json  @MetaMask/accounts-engineers @MetaMask/core-platform

eslint-warning-thresholds.json

@@ -133,14 +133,99 @@
     "@typescript-eslint/prefer-promise-reject-errors": 1,
     "promise/param-names": 3
   },
+  "packages/eip-5792-middleware/src/hooks/processSendCalls.ts": {
+    "@typescript-eslint/no-misused-promises": 1
+  },
   "packages/ens-controller/src/EnsController.test.ts": {
     "import-x/order": 2
   },
   "packages/ens-controller/src/EnsController.ts": {
     "jsdoc/check-tag-names": 6
   },
-  "packages/eip-5792-middleware/src/hooks/processSendCalls.ts": {
-    "@typescript-eslint/no-misused-promises": 1
+  "packages/eth-json-rpc-middleware/src/block-cache.test.ts": {
+    "@typescript-eslint/explicit-function-return-type": 1
+  },
+  "packages/eth-json-rpc-middleware/src/block-cache.ts": {
+    "@typescript-eslint/no-explicit-any": 1,
+    "no-restricted-syntax": 1
+  },
+  "packages/eth-json-rpc-middleware/src/block-ref.test.ts": {
+    "@typescript-eslint/explicit-function-return-type": 1,
+    "@typescript-eslint/no-explicit-any": 4,
+    "jest/expect-expect": 2,
+    "jsdoc/match-description": 1
+  },
+  "packages/eth-json-rpc-middleware/src/block-tracker-inspector.ts": {
+    "@typescript-eslint/no-explicit-any": 1,
+    "jsdoc/match-description": 1
+  },
+  "packages/eth-json-rpc-middleware/src/fetch.test.ts": {
+    "@typescript-eslint/explicit-function-return-type": 1,
+    "jsdoc/match-description": 1
+  },
+  "packages/eth-json-rpc-middleware/src/fetch.ts": {
+    "jsdoc/match-description": 1
+  },
+  "packages/eth-json-rpc-middleware/src/inflight-cache.test.ts": {
+    "@typescript-eslint/explicit-function-return-type": 1
+  },
+  "packages/eth-json-rpc-middleware/src/inflight-cache.ts": {
+    "@typescript-eslint/explicit-function-return-type": 1,
+    "@typescript-eslint/no-explicit-any": 1
+  },
+  "packages/eth-json-rpc-middleware/src/methods/wallet-request-execution-permissions.test.ts": {
+    "@typescript-eslint/explicit-function-return-type": 1
+  },
+  "packages/eth-json-rpc-middleware/src/methods/wallet-revoke-execution-permission.test.ts": {
+    "@typescript-eslint/explicit-function-return-type": 1
+  },
+  "packages/eth-json-rpc-middleware/src/retryOnEmpty.test.ts": {
+    "@typescript-eslint/explicit-function-return-type": 4,
+    "@typescript-eslint/no-explicit-any": 4,
+    "jest/expect-expect": 5,
+    "jsdoc/match-description": 4
+  },
+  "packages/eth-json-rpc-middleware/src/utils/cache.ts": {
+    "jsdoc/match-description": 6
+  },
+  "packages/eth-json-rpc-middleware/src/utils/common.ts": {
+    "@typescript-eslint/explicit-function-return-type": 1,
+    "jsdoc/match-description": 1
+  },
+  "packages/eth-json-rpc-middleware/src/utils/normalize.test.ts": {
+    "@typescript-eslint/explicit-function-return-type": 1
+  },
+  "packages/eth-json-rpc-middleware/src/utils/normalize.ts": {
+    "@typescript-eslint/explicit-function-return-type": 2,
+    "jsdoc/match-description": 3
+  },
+  "packages/eth-json-rpc-middleware/src/utils/timeout.test.ts": {
+    "@typescript-eslint/explicit-function-return-type": 4
+  },
+  "packages/eth-json-rpc-middleware/src/utils/timeout.ts": {
+    "jsdoc/match-description": 1
+  },
+  "packages/eth-json-rpc-middleware/src/wallet.test.ts": {
+    "@typescript-eslint/explicit-function-return-type": 53
+  },
+  "packages/eth-json-rpc-middleware/src/wallet.ts": {
+    "@typescript-eslint/explicit-function-return-type": 2,
+    "@typescript-eslint/no-explicit-any": 2,
+    "@typescript-eslint/prefer-nullish-coalescing": 5,
+    "jsdoc/match-description": 3
+  },
+  "packages/eth-json-rpc-middleware/test/setupAfterEnv.ts": {
+    "@typescript-eslint/explicit-function-return-type": 2,
+    "@typescript-eslint/no-explicit-any": 3,
+    "jsdoc/match-description": 2
+  },
+  "packages/eth-json-rpc-middleware/test/util/createHitTrackerMiddleware.ts": {
+    "@typescript-eslint/explicit-function-return-type": 2
+  },
+  "packages/eth-json-rpc-middleware/test/util/helpers.ts": {
+    "@typescript-eslint/explicit-function-return-type": 4,
+    "@typescript-eslint/no-explicit-any": 1,
+    "jsdoc/match-description": 9
   },
   "packages/eth-json-rpc-provider/src/safe-event-emitter-provider.test.ts": {
     "import-x/namespace": 1
@@ -256,15 +341,6 @@
     "@typescript-eslint/no-unused-vars": 1,
     "@typescript-eslint/prefer-promise-reject-errors": 1
   },
-  "packages/network-controller/tests/create-network-client.test.ts": {
-    "import-x/order": 1
-  },
-  "packages/network-controller/tests/provider-api-tests/helpers.ts": {
-    "@typescript-eslint/prefer-promise-reject-errors": 1,
-    "import-x/namespace": 1,
-    "import-x/no-named-as-default-member": 1,
-    "promise/catch-or-return": 1
-  },
   "packages/permission-controller/src/Permission.ts": {
     "prettier/prettier": 11
   },

eslint.config.mjs

@@ -62,6 +62,7 @@ const config = createConfig([
     files: [
       '**/*.{js,cjs,mjs}',
       '**/*.test.{js,ts}',
+      '**/test/**/*.{js,ts}',
       '**/tests/**/*.{js,ts}',
       'scripts/*.ts',
       'scripts/create-package/**/*.ts',
@@ -218,6 +219,28 @@ const config = createConfig([
       sourceType: 'module',
     },
   },
+  {
+    files: ['packages/eth-json-rpc-middleware/**/*.ts'],
+    rules: {
+      // TODO: Review and re-enable these rules
+      '@typescript-eslint/consistent-type-definitions': 'off',
+      '@typescript-eslint/explicit-function-return-type': 'warn',
+      '@typescript-eslint/naming-convention': 'off',
+      '@typescript-eslint/no-explicit-any': 'warn',
+      '@typescript-eslint/prefer-nullish-coalescing': 'warn',
+      '@typescript-eslint/prefer-optional-chain': 'warn',
+      '@typescript-eslint/restrict-template-expressions': 'off',
+      'id-denylist': 'off',
+      'id-length': 'off',
+      'jsdoc/match-description': 'warn',
+      'jsdoc/no-types': 'warn',
+      'jsdoc/require-hyphen-before-param-description': 'warn',
+      'jsdoc/require-jsdoc': 'off',
+      'jsdoc/require-param-description': 'warn',
+      'no-restricted-globals': 'off',
+      'no-restricted-syntax': 'warn',
+    },
+  },
   {
     files: ['packages/foundryup/**/*.{js,ts}'],
     rules: {

merged-packages/eth-json-rpc-middleware/tsconfig.build.json

@@ -20,10 +20,10 @@ module.exports = merge(baseConfig, {
   // An object that configures minimum threshold enforcement for coverage results
   coverageThreshold: {
     global: {
-      branches: 77.19,
-      functions: 82.45,
-      lines: 83.89,
-      statements: 83.89,
+      branches: 67.25,
+      functions: 81.57,
+      lines: 79.71,
+      statements: 79.83,
     },
   },
 });

merged-packages/eth-json-rpc-middleware/CHANGELOG.md renamed to packages/eth-json-rpc-middleware/CHANGELOG.md

@@ -2,9 +2,13 @@
   "name": "@metamask/eth-json-rpc-middleware",
   "version": "20.0.0",
   "description": "Ethereum-related json-rpc-engine middleware",
-  "homepage": "https://github.com/MetaMask/eth-json-rpc-middleware#readme",
+  "keywords": [
+    "MetaMask",
+    "Ethereum"
+  ],
+  "homepage": "https://github.com/MetaMask/core/tree/main/packages/eth-json-rpc-middleware#readme",
   "bugs": {
-    "url": "https://github.com/MetaMask/eth-json-rpc-middleware/issues"
+    "url": "https://github.com/MetaMask/core/issues"
   },
   "repository": {
     "type": "git",
@@ -29,24 +33,25 @@
   "module": "./dist/index.mjs",
   "types": "./dist/index.d.cts",
   "files": [
-    "dist"
+    "dist/"
   ],
   "scripts": {
-    "build": "ts-bridge --project tsconfig.build.json --clean",
-    "build:clean": "rimraf dist && yarn build",
+    "build": "ts-bridge --project tsconfig.build.json --verbose --clean --no-references",
     "build:docs": "typedoc",
-    "lint": "yarn lint:eslint && yarn lint:constraints && yarn lint:misc --check && yarn lint:dependencies --check && yarn lint:changelog",
-    "lint:changelog": "auto-changelog validate --prettier",
-    "lint:constraints": "yarn constraints",
-    "lint:dependencies": "depcheck && yarn dedupe --check",
-    "lint:dependencies:fix": "depcheck && yarn dedupe",
-    "lint:eslint": "eslint . --cache",
-    "lint:fix": "yarn lint:eslint --fix && yarn lint:constraints --fix && yarn lint:misc --write && yarn lint:dependencies:fix && yarn lint:changelog",
-    "lint:misc": "prettier '**/*.json' '**/*.md' '**/*.yml' '!.yarnrc.yml' --ignore-path .gitignore --no-error-on-unmatched-pattern",
-    "prepack": "./scripts/prepack.sh",
-    "test": "jest && yarn build:clean && yarn test:types",
-    "test:types": "tsd --files 'src/**/*.test-d.ts' && attw --pack",
-    "test:watch": "jest --watch"
+    "changelog:update": "../../scripts/update-changelog.sh @metamask/eth-json-rpc-middleware --tag-prefix-before-package-rename eth-json-rpc-middleware@ --version-before-package-rename 6.1.0",
+    "changelog:validate": "../../scripts/validate-changelog.sh @metamask/eth-json-rpc-middleware --tag-prefix-before-package-rename eth-json-rpc-middleware@ --version-before-package-rename 6.1.0",
+    "lint": "yarn lint:eslint && yarn lint:misc --check && yarn lint:dependencies && yarn lint:changelog",
+    "lint:changelog": "auto-changelog validate",
+    "lint:dependencies": "depcheck",
+    "lint:eslint": "eslint . --cache --ext js,ts",
+    "lint:fix": "yarn lint:eslint --fix && yarn lint:misc --write && yarn lint:dependencies && yarn lint:changelog",
+    "lint:misc": "prettier '**/*.json' '**/*.md' '!CHANGELOG.md' '**/*.yml' '!.yarnrc.yml' --ignore-path .gitignore --no-error-on-unmatched-pattern",
+    "publish:preview": "yarn npm publish --tag preview",
+    "since-latest-release": "../../scripts/since-latest-release.sh",
+    "test": "NODE_OPTIONS=--experimental-vm-modules jest --reporters=jest-silent-reporter",
+    "test:clean": "NODE_OPTIONS=--experimental-vm-modules jest --clearCache",
+    "test:verbose": "NODE_OPTIONS=--experimental-vm-modules jest --verbose",
+    "test:watch": "NODE_OPTIONS=--experimental-vm-modules jest --watch"
   },
   "dependencies": {
     "@metamask/eth-block-tracker": "^13.0.0",
@@ -72,9 +77,8 @@
     "typedoc": "^0.24.8",
     "typescript": "~5.2.2"
   },
-  "packageManager": "yarn@4.1.1",
   "engines": {
-    "node": "^18.16 || ^20 || >=22"
+    "node": "^18.18 || >=20"
   },
   "publishConfig": {
     "access": "public",