Skip to content

Conversation

@FrederikBolding
Copy link
Member

@FrederikBolding FrederikBolding commented Jan 29, 2026

Description

Replace metamask_sendDomainMetadata with a no-op as we no longer need the provider to report metadata for sites. The SubjectMetadataController is populated already using the tabs API. The RPC method is kept around for backwards compatibility.

Open in GitHub Codespaces

Changelog

CHANGELOG entry: Calling the RPC method metamask_sendDomainMetadata no longer has any effect.

Manual testing steps

  1. Connect to any dapp
  2. See that the name and icon for a site is populated in the connection screen

Note

Medium Risk
Disables domain-metadata reporting from the inpage provider and removes the backend hook that persisted this metadata, which could affect any flows still relying on metamask_sendDomainMetadata for site name/icon display.

Overview
Stops the inpage provider from sending site metadata by setting shouldSendMetadata: false during initializeProvider.

Replaces the metamask_sendDomainMetadata (MESSAGE_TYPE.SEND_METADATA) RPC handler with a no-op that always returns true, removing subject-metadata persistence hooks (addSubjectMetadata/subjectType) from middleware wiring and updating the unit test to match.

Written by Cursor Bugbot for commit cf06db4. This will update automatically on new commits. Configure here.

@metamaskbot metamaskbot added team-core-platform Core Platform team INVALID-PR-TEMPLATE PR's body doesn't match template labels Jan 29, 2026
@metamaskbotv2
Copy link
Contributor

metamaskbotv2 bot commented Jan 29, 2026

✨ Files requiring CODEOWNER review ✨

👨‍🔧 @MetaMask/wallet-integrations (2 files, +9 -59)
  • 📁 app/
    • 📁 scripts/
      • 📁 lib/
        • 📁 rpc-method-middleware/
          • 📁 handlers/
            • 📄 send-metadata.test.ts +4 -18
            • 📄 send-metadata.ts +5 -41

@FrederikBolding FrederikBolding force-pushed the fb/remove-send-domain-metadata branch from d5ac62f to 4507ee6 Compare January 29, 2026 16:17
@FrederikBolding FrederikBolding changed the title fix: Remove metamask_sendDomainMetadata fix: Replace metamask_sendDomainMetadata with a no-op Jan 29, 2026
@github-actions github-actions bot added size-S and removed size-M labels Jan 29, 2026
@metamaskbotv2
Copy link
Contributor

metamaskbotv2 bot commented Jan 29, 2026

Builds ready [b4906b8]
UI Startup Metrics (1332 ± 117 ms)
PlatformBuildTypePageMetricTest Title (ms)Persona (ms)Mean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--13321062168811713931530
load--1126904148411711961302
domContentLoaded--1118897147611611851297
domInteractive--281696192483
firstPaint--1946511432192031057
backgroundConnect--23621527612240265
firstReactRender--17113441826
initialActions--106113
loadScripts--89869212601159661086
setupStore--1362961626
numNetworkReqs--221582171574
19--------
BrowserifyPower User Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--282115168181128942294866
load--1130963169713511891422
domContentLoaded--1111950166013111761415
domInteractive--3719291343399
firstPaint--212781505187261433
backgroundConnect--865278317786813883122
firstReactRender--23165362436
initialActions--104113
loadScripts--87673412331169341172
setupStore--1465071623
numNetworkReqs--1345433752162236
19--------
WebpackStandard Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--8256711145938691030
load--68258692278724888
domContentLoaded--67658291677719881
domInteractive--251497182274
firstPaint--1065834556116225
backgroundConnect--3718114204480
firstReactRender--15103851727
initialActions--107112
loadScripts--67458091476717879
setupStore--1266371324
numNetworkReqs--221581181576
19--------
WebpackPower User Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--1285869248528614611716
load--70661612211186881007
domContentLoaded--69661012151186811000
domInteractive--35181792932110
firstPaint--15566514102192467
backgroundConnect--15913157466148349
firstReactRender--23173842431
initialActions--103111
loadScripts--6936081207116679992
setupStore--1243851320
numNetworkReqs--1275331353142257
19--------
FirefoxBrowserifyStandard Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--15021179213818516181817
load--1194988149210812651394
domContentLoaded--1193988149210812651393
domInteractive--813429351112163
firstPaint--------
backgroundConnect--64232144385162
firstReactRender--14104761425
initialActions--103122
loadScripts--115997214229312331315
setupStore--174220271369
numNetworkReqs--231283181876
19--------
BrowserifyPower User Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--28641972725373130763956
load--13451094501643914001739
domContentLoaded--13451093501643914001739
domInteractive--15554998127177355
firstPaint--------
backgroundConnect--45213218913516631182
firstReactRender--22146272330
initialActions--203122
loadScripts--12861035496842513301610
setupStore--12615753151134529
numNetworkReqs--91422354592221
19--------
WebpackStandard Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--16351359200515317561912
load--13981181170711714631661
domContentLoaded--13971181170711714631661
domInteractive--872823242125141
firstPaint--------
backgroundConnect--69262024487167
firstReactRender--14112921519
initialActions--102122
loadScripts--1357116816379714331548
setupStore--165145231243
numNetworkReqs--231285191983
19--------
WebpackPower User Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--30492272480750533084178
load--15771271261325817122220
domContentLoaded--15771271261225817112219
domInteractive--17443922156239546
firstPaint--------
backgroundConnect--50813016203327351246
firstReactRender--22167172431
initialActions--203122
loadScripts--15371235246525216592178
setupStore--14314784169160535
numNetworkReqs--94452814694196
19--------
📊 Page Load Benchmark Results

Current Commit: b4906b8 | Date: 1/29/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.05s (±39ms) 🟡 | historical mean value: 1.04s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 730ms (±37ms) 🟢 | historical mean value: 724ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 79ms (±10ms) 🟢 | historical mean value: 77ms ⬆️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.05s 39ms 1.02s 1.33s 1.08s 1.33s
domContentLoaded 730ms 37ms 704ms 1.00s 755ms 1.00s
firstPaint 79ms 10ms 64ms 172ms 88ms 172ms
firstContentfulPaint 79ms 10ms 64ms 172ms 88ms 172ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 4.55 KiB (0.08%)
  • ui: 3 Bytes (0%)
  • common: 3.69 KiB (0.04%)

@FrederikBolding FrederikBolding marked this pull request as ready for review January 29, 2026 19:57
@metamaskbotv2
Copy link
Contributor

metamaskbotv2 bot commented Jan 29, 2026

Builds ready [cf06db4]
UI Startup Metrics (1349 ± 121 ms)
PlatformBuildTypePageMetricTest Title (ms)Persona (ms)Mean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--13491060162412114191566
load--1155899143211812281378
domContentLoaded--1148894142511812151370
domInteractive--271599192384
firstPaint--169651155162190324
backgroundConnect--23921630014243270
firstReactRender--18113952127
initialActions--106112
loadScripts--92468112061189891147
setupStore--1463861826
numNetworkReqs--221582181578
19--------
BrowserifyPower User Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--340816628133153643797448
load--12351048185213913051497
domContentLoaded--12141019184113812691485
domInteractive--40201492841114
firstPaint--252891456214281446
backgroundConnect--11343094663100617692823
firstReactRender--26186172740
initialActions--108112
loadScripts--96179015471279971203
setupStore--1666481831
numNetworkReqs--1095822335117190
19--------
WebpackStandard Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--83866913381039041007
load--695585121893752848
domContentLoaded--690580121292746839
domInteractive--2516123192176
firstPaint--1116061982134278
backgroundConnect--3618132224099
firstReactRender--1610118111827
initialActions--104111
loadScripts--687578120491739836
setupStore--1154351120
numNetworkReqs--221591181571
19--------
WebpackPower User Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--1310867259328615061695
load--72663613481197251017
domContentLoaded--71562513421207121007
domInteractive--39171562738112
firstPaint--15978968117197351
backgroundConnect--15512942444155238
firstReactRender--22173132428
initialActions--103111
loadScripts--71262313341197101002
setupStore--1153141418
numNetworkReqs--1668933254176309
19--------
FirefoxBrowserifyStandard Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--14061110191217115321720
load--1136949160611912361336
domContentLoaded--1136949160611912321336
domInteractive--65302173988144
firstPaint--------
backgroundConnect--64242264677167
firstReactRender--1394051319
initialActions--103122
loadScripts--1102934148310111691261
setupStore--145221261036
numNetworkReqs--231286191881
19--------
BrowserifyPower User Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--31172112936289532524758
load--14601128293431215582069
domContentLoaded--14591128293431215542064
domInteractive--193521073171221498
firstPaint--------
backgroundConnect--72513972937859161434
firstReactRender--22155252432
initialActions--208123
loadScripts--13801096291128314921994
setupStore--10522824125116292
numNetworkReqs--94402504597222
19--------
WebpackStandard Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--15251280202814315841833
load--13201104180111213861534
domContentLoaded--13191104180011213831534
domInteractive--742716238107140
firstPaint--------
backgroundConnect--68242194196152
firstReactRender--13103831318
initialActions--102012
loadScripts--1280108916969513351457
setupStore--144266281133
numNetworkReqs--221282161970
19--------
WebpackPower User Home0--------
1--------
2--------
3--------
4--------
5--------
6--------
7--------
8--------
9--------
10--------
11--------
12--------
13--------
14--------
15--------
16--------
17--------
18--------
uiStartup--31312093794470633663789
load--16571289621559217512118
domContentLoaded--16571289621459217472118
domInteractive--16046703129182511
firstPaint--------
backgroundConnect--52913913783307981148
firstReactRender--22153242430
initialActions--213122
loadScripts--16181265617959116822086
setupStore--143151013183147597
numNetworkReqs--95362804993223
19--------
📊 Page Load Benchmark Results

Current Commit: cf06db4 | Date: 1/29/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.05s (±38ms) 🟡 | historical mean value: 1.04s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 729ms (±36ms) 🟢 | historical mean value: 724ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 78ms (±10ms) 🟢 | historical mean value: 79ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.05s 38ms 1.02s 1.33s 1.07s 1.33s
domContentLoaded 729ms 36ms 709ms 1.00s 753ms 1.00s
firstPaint 78ms 10ms 60ms 160ms 84ms 160ms
firstContentfulPaint 78ms 10ms 60ms 160ms 84ms 160ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 4.77 KiB (0.09%)
  • ui: 4.71 KiB (0.06%)
  • common: 3.73 KiB (0.04%)

Copy link
Contributor

@adonesky1 adonesky1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@FrederikBolding FrederikBolding added this pull request to the merge queue Jan 30, 2026
github-merge-queue bot pushed a commit that referenced this pull request Jan 30, 2026
<!--
Please submit this PR as a draft initially.
Do not mark it as "Ready for review" until the template has been
completely filled out, and PR status checks have passed at least once.
-->

## **Description**

Replace `metamask_sendDomainMetadata` with a no-op as we no longer need
the provider to report metadata for sites. The
`SubjectMetadataController` is populated already using the tabs API. The
RPC method is kept around for backwards compatibility.

[![Open in GitHub
Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/MetaMask/metamask-extension/pull/39642?quickstart=1)

## **Changelog**

CHANGELOG entry: Calling the RPC method `metamask_sendDomainMetadata` no
longer has any effect.

## **Manual testing steps**

1. Connect to any dapp
2. See that the name and icon for a site is populated in the connection
screen

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Medium Risk**
> Disables domain-metadata reporting from the inpage provider and
removes the backend hook that persisted this metadata, which could
affect any flows still relying on `metamask_sendDomainMetadata` for site
name/icon display.
> 
> **Overview**
> Stops the inpage provider from sending site metadata by setting
`shouldSendMetadata: false` during `initializeProvider`.
> 
> Replaces the `metamask_sendDomainMetadata`
(`MESSAGE_TYPE.SEND_METADATA`) RPC handler with a **no-op** that always
returns `true`, removing subject-metadata persistence hooks
(`addSubjectMetadata`/`subjectType`) from middleware wiring and updating
the unit test to match.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
cf06db4. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 30, 2026
@FrederikBolding FrederikBolding added this pull request to the merge queue Jan 30, 2026
Merged via the queue into main with commit 29f9591 Jan 30, 2026
179 checks passed
@FrederikBolding FrederikBolding deleted the fb/remove-send-domain-metadata branch January 30, 2026 12:02
@github-actions github-actions bot locked and limited conversation to collaborators Jan 30, 2026
@metamaskbot metamaskbot added the release-13.18.0 Issue or pull request that will be included in release 13.18.0 label Jan 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

INVALID-PR-TEMPLATE PR's body doesn't match template release-13.18.0 Issue or pull request that will be included in release 13.18.0 size-S team-core-platform Core Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants