Skip to content

fix: console formatter hides frames from ignored scripts#927

Merged
OrKoN merged 1 commit intomainfrom
honor-ignore-listing
Feb 10, 2026
Merged

fix: console formatter hides frames from ignored scripts#927
OrKoN merged 1 commit intomainfrom
honor-ignore-listing

Conversation

@szuend
Copy link
Copy Markdown
Contributor

@szuend szuend commented Feb 9, 2026

This PR wires up the default DevTools ignore listing to hide frames from third party scripts in stack traces. By default, we ignore:

  • Scripts with /node_modules/ in their URL or where the URL starts with node:/
  • Scripts that are marked as ignore listed in the source map
  • Content scripts from extensions

The PR adds both, unit tests and an e2e test. This is because we allow the unit test to pass a mocked ignore checker, where-as the e2e uses the real DevTools one.

@szuend szuend requested a review from OrKoN February 9, 2026 16:03
@OrKoN OrKoN added this pull request to the merge queue Feb 10, 2026
Merged via the queue into main with commit 8e2380b Feb 10, 2026
31 of 33 checks passed
@OrKoN OrKoN deleted the honor-ignore-listing branch February 10, 2026 07:28
github-merge-queue Bot pushed a commit that referenced this pull request Feb 10, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.17.0](chrome-devtools-mcp-v0.16.0...chrome-devtools-mcp-v0.17.0)
(2026-02-10)


### 🎉 Features

* include Error.cause chain for uncaught errors and logged Errors
([#906](#906))
([05b01ec](05b01ec))
* Integrate CrUX data into performance trace summaries
([#733](#733))
([b747f9d](b747f9d))
* show message and stack trace in details when console.log'ging Error
objects
([#902](#902))
([ffa00da](ffa00da))


### 🛠️ Fixes

* console formatter hides frames from ignored scripts
([#927](#927))
([8e2380b](8e2380b))
* limit stack traces to 50 lines
([#923](#923))
([caea23a](caea23a))


### 📄 Documentation

* add macOS Web Bluetooth troubleshooting note
([#930](#930))
([3c9528b](3c9528b)),
closes
[#917](#917)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
wolfib pushed a commit to wolfib/chrome-devtools-mcp that referenced this pull request Mar 10, 2026
…ols#927)

This PR wires up the default DevTools ignore listing to hide frames from
third party scripts in stack traces. By default, we ignore:

* Scripts with `/node_modules/` in their URL or where the URL starts
with `node:/`
* Scripts that are marked as ignore listed in the source map
* Content scripts from extensions

The PR adds both, unit tests and an e2e test. This is because we allow
the unit test to pass a mocked ignore checker, where-as the e2e uses the
real DevTools one.
wolfib pushed a commit to wolfib/chrome-devtools-mcp that referenced this pull request Mar 10, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.17.0](ChromeDevTools/chrome-devtools-mcp@chrome-devtools-mcp-v0.16.0...chrome-devtools-mcp-v0.17.0)
(2026-02-10)


### 🎉 Features

* include Error.cause chain for uncaught errors and logged Errors
([ChromeDevTools#906](ChromeDevTools#906))
([05b01ec](ChromeDevTools@05b01ec))
* Integrate CrUX data into performance trace summaries
([ChromeDevTools#733](ChromeDevTools#733))
([b747f9d](ChromeDevTools@b747f9d))
* show message and stack trace in details when console.log'ging Error
objects
([ChromeDevTools#902](ChromeDevTools#902))
([ffa00da](ChromeDevTools@ffa00da))


### 🛠️ Fixes

* console formatter hides frames from ignored scripts
([ChromeDevTools#927](ChromeDevTools#927))
([8e2380b](ChromeDevTools@8e2380b))
* limit stack traces to 50 lines
([ChromeDevTools#923](ChromeDevTools#923))
([caea23a](ChromeDevTools@caea23a))


### 📄 Documentation

* add macOS Web Bluetooth troubleshooting note
([ChromeDevTools#930](ChromeDevTools#930))
([3c9528b](ChromeDevTools@3c9528b)),
closes
[ChromeDevTools#917](ChromeDevTools#917)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants