Skip to content

feat: add option to enable pptr logs to the log file. - #2706

Closed
yulunz wants to merge 1 commit into
mainfrom
feat/disable-pptr-log-to-file
Closed

yulunz wants to merge 1 commit into
mainfrom
feat/disable-pptr-log-to-file

Conversation

@yulunz

@yulunz yulunz commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Noisy pptr logs clutter the log file. Make this disabled by default.

@yulunz
yulunz requested a review from OrKoN September 9, 2026 13:47
Comment thread docs/configuration.md

- **`--logFile`/ `--log-file`**
Path to a file to write debug logs to. Set the env variable `DEBUG` to `*` to enable verbose logs. Useful for submitting bug reports.
Path to a file to write debug logs to. Set the env variable `DEBUG` to `*` to enable verbose logs. To include verbose Puppeteer protocol logs (`puppeteer:protocol:*`), set the env variable `CHROME_DEVTOOLS_MCP_PPTR_LOGGING_ENABLED` to `true`. Useful for submitting bug reports.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

is this really needed? You can set DEBUG to the prefix you are interested in. Usually Puppeteer CDP logs are relevant for debugging.

zjdCander pushed a commit to zjdCander/chrome-devtools-mcp that referenced this pull request Sep 14, 2026
…s#2743)

Currently we unconditionally log all MCP and Puppeteer logs to the
files.
This adds regression test using mocks to simplify testings. 

Closes ChromeDevTools#2734
Closes ChromeDevTools#2706
Fixes ChromeDevTools#2731
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