Description

The TypeScript language server (vtsls/tsserver) crashes or runs out of memory in editors when working on this repo. Root cause: two identical 741KB webpack bundles (scripts/inpage-bridge/dist/index.js and app/core/InpageBridgeWeb3.js) are included in the TypeScript project via include globs. These are minified single-line files that provide zero value to type-checking but force tsserver to parse ~759K characters and infer types for the entire bundled module graph.

This PR:

1. Excludes the two 741KB bundles from type-checking by adding them to exclude — saves ~345MB peak RSS
2. Enables incremental: true — allows tsc/tsserver to cache type information between runs. Warm runs drop from ~63s/7.4GB to ~10s/3.5GB
3. Fixes .gitignore glob for .tsbuildinfo files — *.tsbuildinfo catches any filename generated by incremental builds

Benchmark results (10 iterations, measured with tsc --extendedDiagnostics + /usr/bin/time -l):

Note: The CI lint:tsc script already uses --max-old-space-size=8192 (8GB heap). Most editor LSPs default to 3-4GB, which is insufficient for this project's 17K+ file type graph. These changes reduce the memory footprint and dramatically improve warm startup times.

Changelog

CHANGELOG entry: null

Related issues

Fixes: N/A

Manual testing steps

Feature: TypeScript type-checking

  Scenario: tsc passes with no regressions
    Given the developer is at the repo root

    When developer runs `yarn lint:tsc`
    Then type-checking completes with zero errors

  Scenario: incremental compilation produces cache
    Given the developer is at the repo root
    And no tsconfig.tsbuildinfo file exists

    When developer runs `yarn lint:tsc` twice
    Then a tsconfig.tsbuildinfo file is created after the first run
    And the second run completes significantly faster (~10s vs ~60s)

  Scenario: excluded bundles are not type-checked
    Given the developer is at the repo root

    When developer runs `yarn tsc --listFiles --noEmit`
    Then the output does not contain scripts/inpage-bridge/dist/index.js
    And the output does not contain app/core/InpageBridgeWeb3.js

Screenshots/Recordings

N/A — configuration-only change with no UI impact.

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I've included tests if applicable
• I've documented my code using JSDoc format if applicable
• I've applied the right labels on the PR (see labeling guidelines). 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.