temp+chore: cp-7.61.2 update @metamask/network-controller to latest version (#24166)

## **Description**

The package `@metamask/network-controller` has been updated from v27.0
to v27.1, which is the latest version.

The primary motivation for the update is this bug fix:
MetaMask/core#7532 (included in v27.1.0).


## **Changelog**

CHANGELOG entry: Fix bug where EIP-1559 compatibility check was skipped
in some circumstances

## **Related issues**

Related core issue: MetaMask/core#7533

## **Manual testing steps**

- do a fresh install of 7.61.0
- submit a transaction (see that it was a type 0.. no gas fee properties
included)
- try to submit a type 4 transaction (upgrade/downgrade doesn't matter..
see that the transaction fails locally)
- update to the build from your PR
- re-do the previous actions and they all should succeed
  - type 2 transaction submitted
  - type 4 transaction submitted

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [x] I’ve followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [ ] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Update `@metamask/network-controller` to `^27.1.0` and refresh
lockfile with related dependency bumps.
> 
> - **Dependencies**:
> - Bump `@metamask/network-controller` from `^27.0.0` to `^27.1.0` in
`package.json` and `yarn.lock`.
>   - Lockfile updates for transitive deps:
>     - `@metamask/controller-utils` → `11.17.0`.
>     - `@metamask/eth-json-rpc-middleware` → `22.0.1`.
>   - Corresponding `yarn.lock` resolutions and checksums updated.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
aea2716. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: runway-github[bot]

.cursor/rules/feature-flag-guidelines.mdc

@@ -0,0 +1,91 @@
+---
+globs: "**/*"
+alwaysApply: true
+---
+
+# Feature Flag Guidelines
+
+## Core Principle
+
+**ALWAYS** use the `useFeatureFlag` hook instead of creating new feature flag selectors.
+
+## Forbidden Patterns
+
+### ❌ NEVER Create New Feature Flag Selectors
+
+**DO NOT** create new selectors using `createSelector` for feature flags:
+
+```typescript
+// ❌ FORBIDDEN - Do not create new feature flag selectors
+export const selectMyFeatureEnabledFlag = createSelector(
+  selectRemoteFeatureFlags,
+  (remoteFeatureFlags) => {
+    // ... selector logic
+  },
+);
+```
+
+**DO NOT** add new feature flag selectors in:
+- `app/selectors/featureFlagController/**/*.ts`
+- `app/components/**/selectors/featureFlags/**/*.ts`
+- Any other location that creates feature flag selectors
+
+## Required Pattern
+
+### ✅ ALWAYS Use the `useFeatureFlag` Hook
+
+**MUST** use the `useFeatureFlag` hook from `app/components/hooks/FeatureFlags/useFeatureFlag.ts`:
+
+```typescript
+// ✅ REQUIRED - Use the hook instead
+import { useFeatureFlag, FeatureFlagNames } from '../../../hooks/FeatureFlags/useFeatureFlag';
+
+const MyComponent = () => {
+  const isFeatureEnabled = useFeatureFlag(FeatureFlagNames.rewardsEnabled);
+  
+  // Use the flag value
+  if (isFeatureEnabled) {
+    // ... feature logic
+  }
+};
+```
+
+## Steps to Use Feature Flags
+
+1. **Add the flag name** to the `FeatureFlagNames` enum in `app/components/hooks/FeatureFlags/useFeatureFlag.ts`:
+   ```typescript
+   export enum FeatureFlagNames {
+     rewardsEnabled = 'rewardsEnabled',
+     myNewFeature = 'myNewFeature', // Add your new flag here
+   }
+   ```
+
+2. **Use the hook** in your component:
+   ```typescript
+   const isMyFeatureEnabled = useFeatureFlag(FeatureFlagNames.myNewFeature);
+   ```
+
+3. **Do NOT** create a selector for the feature flag
+
+## Migration Pattern
+
+If you encounter existing feature flag selectors, prefer migrating to the hook:
+
+```typescript
+// ❌ Old pattern (existing code - do not replicate)
+const isFeatureEnabled = useSelector(selectMyFeatureEnabledFlag);
+
+// ✅ New pattern (use this instead)
+const isFeatureEnabled = useFeatureFlag(FeatureFlagNames.myNewFeature);
+```
+
+## Enforcement
+
+- **REJECT** any code that creates new `createSelector` instances for feature flags
+- **REJECT** any new files in `app/selectors/featureFlagController/` directories
+- **REQUIRE** use of `useFeatureFlag` hook for all feature flag access
+- **REQUIRE** adding flag names to `FeatureFlagNames` enum before use
+
+## Exception
+
+The only exception is the base selector `selectRemoteFeatureFlags` in `app/selectors/featureFlagController/index.ts`, which is used internally by the `useFeatureFlag` hook infrastructure.

.depcheckrc.yml

@@ -1,5 +1,4 @@
 # List things here that *are - 'used, that depcheck is wrong about'
-
 ignores:
   - '@metamask/oss-attribution-generator'
   - '@metamask/test-dapp-multichain'
@@ -35,8 +34,6 @@ ignores:
   # ESBuild is used for AI E2E script compilation
   - 'esbuild'
   - 'esbuild-register'
-  # xml2js is used in .github/scripts/ for E2E test report processing
-  - 'xml2js'
 
   # Used in scripts/repack for CI optimization
   - '@expo/repack-app'
@@ -46,11 +43,19 @@ ignores:
   ## Unused dependencies to investigate
   - '@babel/preset-env'
   - '@babel/runtime'
+  - '@cucumber/message-streams'
+  - '@cucumber/messages'
   - '@metamask/mobile-provider'
+  - '@rpii/wdio-html-reporter'
   - '@testing-library/react'
   - '@testing-library/react-hooks'
   - '@types/jest'
   - '@types/react-native-video'
+  - '@wdio/appium-service'
+  - '@wdio/browserstack-service'
+  - '@wdio/junit-reporter'
+  - '@wdio/local-runner'
+  - '@wdio/spec-reporter'
   - 'appium'
   - 'assert'
   - 'babel-core'
@@ -62,6 +67,7 @@ ignores:
   - 'execa'
   - 'jetifier'
   - 'metro-react-native-babel-preset'
+  - 'prettier-plugin-gherkin'
   - 'react-native-svg-asset-plugin'
   - 'regenerator-runtime'
   - 'prettier-2'
@@ -108,7 +114,6 @@ ignores:
   - 'cross-spawn'
   - 'expo-build-properties'
   - 'expo-dev-client'
-  - 'expo-font'
 
   ## react native
   - '@react-native-community/cli'

.eslintignore

@@ -3,7 +3,6 @@
 /app/util/blockies.js
 __snapshots__
 android
-build
 coverage
 ios
 jest.preprocessor.js

.eslintrc.js

@@ -1,4 +1,4 @@
-/* eslint-disable import/no-commonjs */
+// eslint-disable-next-line import/no-commonjs
 module.exports = {
   root: true,
   parser: '@typescript-eslint/parser',
@@ -34,11 +34,15 @@ module.exports = {
         'react/no-unused-prop-types': 'off',
         'react/prop-types': 'off',
         'react/self-closing-comp': 'off',
-        // Temporarily overriding this rule to postpone this breaking change: https://github.com/MetaMask/eslint-config/pull/216
-        // TODO: Remove this override and align on prefering type over interface.
-        '@typescript-eslint/consistent-type-definitions': [
+        // This change is included in `@metamask/[email protected]
+        '@typescript-eslint/no-unused-vars': [
           'error',
-          'interface',
+          {
+            vars: 'all',
+            args: 'all',
+            argsIgnorePattern: '[_]+',
+            ignoreRestSiblings: true, // this line is what has changed
+          },
         ],
         '@typescript-eslint/no-explicit-any': 'error',
         // Under discussion
@@ -75,9 +79,9 @@ module.exports = {
     {
       files: ['scripts/**/*.js', 'e2e/tools/**/*.{js,ts}', 'app.config.js'],
       rules: {
-        'no-console': 'off',
-        'import/no-commonjs': 'off',
-        'import/no-nodejs-modules': 'off',
+        'no-console': 0,
+        'import/no-commonjs': 0,
+        'import/no-nodejs-modules': 0,
       },
     },
     {
@@ -188,7 +192,7 @@ module.exports = {
   rules: {
     // Set to error once all warnings reported by React Compiler are resolved
     'react-compiler/react-compiler': 'warn',
-    'no-catch-shadow': 'off',
+    'no-catch-shadow': 0,
     'no-console': ['error', { allow: ['warn', 'error'] }],
     quotes: [
       'error',
@@ -198,105 +202,105 @@ module.exports = {
         allowTemplateLiterals: true,
       },
     ],
-    'comma-dangle': 'off',
-    curly: 'off',
-    'no-shadow': 'off',
-    'no-bitwise': 'off',
-    'class-methods-use-this': 'off',
-    'eol-last': 'warn',
-    'import/no-named-as-default': 'off',
-    'no-invalid-this': 'off',
-    'no-new': 'off',
-    'react/jsx-handler-names': 'off',
-    'react/no-did-mount-set-state': 'off',
-    'react/prefer-stateless-function': 'off',
-    'require-atomic-updates': 'off',
-    'array-callback-return': 'error',
-    'arrow-body-style': 'error',
-    'dot-notation': 'error',
-    eqeqeq: 'error',
-    'import/no-amd': 'error',
-    'import/no-commonjs': 'error',
-    'import/no-duplicates': 'error',
+    'comma-dangle': 0,
+    curly: 0,
+    'no-shadow': 0,
+    'no-bitwise': 0,
+    'class-methods-use-this': 0,
+    'eol-last': 1,
+    'import/no-named-as-default': 0,
+    'no-invalid-this': 0,
+    'no-new': 0,
+    'react/jsx-handler-names': 0,
+    'react/no-did-mount-set-state': 0,
+    'react/prefer-stateless-function': 0,
+    'require-atomic-updates': 0,
+    'array-callback-return': 2,
+    'arrow-body-style': 2,
+    'dot-notation': 2,
+    eqeqeq: 2,
+    'import/no-amd': 2,
+    'import/no-commonjs': 2,
+    'import/no-duplicates': 2,
     'import/no-extraneous-dependencies': ['error', { packageDir: ['./'] }],
-    'import/no-mutable-exports': 'error',
-    'import/no-namespace': 'error',
-    'import/no-nodejs-modules': 'error',
-    'import/prefer-default-export': 'off',
-    'no-alert': 'error',
+    'import/no-mutable-exports': 2,
+    'import/no-namespace': 2,
+    'import/no-nodejs-modules': 2,
+    'import/prefer-default-export': 0,
+    'no-alert': 2,
     'no-constant-condition': [
-      'error',
+      2,
       {
         checkLoops: false,
       },
     ],
-    'no-duplicate-imports': 'error',
-    'no-empty-function': 'error',
-    'no-else-return': 'error',
-    'no-eval': 'error',
-    'no-extend-native': 'error',
-    'no-extra-bind': 'error',
-    'no-global-assign': 'error',
-    'no-implicit-globals': 'error',
-    'no-implied-eval': 'error',
-    'no-lonely-if': 'error',
-    'no-loop-func': 'error',
-    'no-new-func': 'error',
-    'no-new-wrappers': 'error',
-    'no-proto': 'error',
-    'no-script-url': 'error',
-    'no-self-compare': 'error',
-    'no-throw-literal': 'error',
-    'no-unmodified-loop-condition': 'error',
+    'no-duplicate-imports': 2,
+    'no-empty-function': 2,
+    'no-else-return': 2,
+    'no-eval': 2,
+    'no-extend-native': 2,
+    'no-extra-bind': 2,
+    'no-global-assign': 2,
+    'no-implicit-globals': 2,
+    'no-implied-eval': 2,
+    'no-lonely-if': 2,
+    'no-loop-func': 2,
+    'no-new-func': 2,
+    'no-new-wrappers': 2,
+    'no-proto': 2,
+    'no-script-url': 2,
+    'no-self-compare': 2,
+    'no-throw-literal': 2,
+    'no-unmodified-loop-condition': 2,
     'no-unneeded-ternary': [
-      'error',
+      2,
       {
         defaultAssignment: false,
       },
     ],
-    'no-unsafe-negation': 'error',
+    'no-unsafe-negation': 2,
     'no-unused-expressions': 'off',
-    'no-use-before-define': ['error', 'nofunc'],
-    'no-useless-call': 'error',
-    'no-useless-computed-key': 'error',
-    'no-useless-concat': 'error',
-    'no-useless-constructor': 'error',
-    'no-useless-rename': 'error',
-    'no-var': 'error',
-    'no-with': 'error',
-    'object-shorthand': 'error',
-    'operator-assignment': 'error',
-    'prefer-arrow-callback': 'error',
-    'prefer-const': 'error',
-    'prefer-rest-params': 'error',
-    'prefer-spread': 'error',
-    'import/no-unresolved': 'error',
-    'eslint-comments/no-unlimited-disable': 'off',
-    'eslint-comments/no-unused-disable': 'off',
-    'react-native/no-color-literals': 'error',
-    'react-native/no-inline-styles': 'error',
-    'react-native/no-unused-styles': 'off',
-    'react-native/split-platform-components': 'error',
-    'react/jsx-boolean-value': 'error',
-    'react/jsx-key': 'warn',
-    'react/jsx-no-bind': 'off',
-    'react/jsx-pascal-case': 'error',
-    'react/jsx-wrap-multilines': 'error',
-    'react/no-danger': 'error',
-    'react/no-did-update-set-state': 'error',
-    'react/no-find-dom-node': 'error',
+    'no-use-before-define': [2, 'nofunc'],
+    'no-useless-call': 2,
+    'no-useless-computed-key': 2,
+    'no-useless-concat': 2,
+    'no-useless-constructor': 2,
+    'no-useless-rename': 2,
+    'no-var': 2,
+    'no-with': 2,
+    'object-shorthand': 2,
+    'operator-assignment': 2,
+    'prefer-arrow-callback': 2,
+    'prefer-const': 2,
+    'prefer-rest-params': 2,
+    'prefer-spread': 2,
+    'import/no-unresolved': 2,
+    'eslint-comments/no-unlimited-disable': 0,
+    'eslint-comments/no-unused-disable': 0,
+    'react-native/no-color-literals': 2,
+    'react-native/no-inline-styles': 2,
+    'react-native/no-unused-styles': 0,
+    'react-native/split-platform-components': 2,
+    'react/jsx-boolean-value': 2,
+    'react/jsx-key': 1,
+    'react/jsx-no-bind': 0,
+    'react/jsx-pascal-case': 2,
+    'react/jsx-wrap-multilines': 2,
+    'react/no-danger': 2,
+    'react/no-did-update-set-state': 2,
+    'react/no-find-dom-node': 2,
     'react/no-multi-comp': [
-      'error',
+      2,
       {
         ignoreStateless: true,
       },
     ],
-    'react/no-render-return-value': 'error',
-    'react/no-string-refs': 'error',
-    'react/no-unused-prop-types': 'error',
-    'react/prefer-es6-class': 'error',
+    'react/no-render-return-value': 2,
+    'react/no-string-refs': 2,
+    'react/no-unused-prop-types': 2,
+    'react/prefer-es6-class': 2,
     '@metamask/design-tokens/color-no-hex': 'warn',
-    radix: 'off',
+    radix: 0,
   },
 
   ignorePatterns: ['wdio.conf.js', 'app/util/termsOfUse/termsOfUseContent.ts'],