feat(mcp): add tools request details #37322
Conversation
|
@microsoft-github-policy-service agree |
|
|
||
| try { | ||
| const contentType = resHeaders['content-type'] || ''; | ||
| if (contentType.includes('application/json') || contentType.includes('text/')) { |
There was a problem hiding this comment.
So you mostly want it for fetch requests? I think LLM will try to call it for other types of resources, and that is fine. When calling it for JS it'll hit the context limit though. If we are doing it, we should make body optional and introduce pagination. It'll grow into a large change and require tests. I'd suggest that you file it at https://github.com/microsoft/playwright-mcp and let us do consider doing it. We rarely succeed at accepting the third party patches of this size / complexity.
There was a problem hiding this comment.
playwright-mcp the project has been moved to this project. Should I pull it to the MCP project?
There was a problem hiding this comment.
You should file an issue at https://github.com/microsoft/playwright-mcp, don't create PRs yet.
Add tools to retrieve details of a request, such as request header, request body, response header, and response body