new_audit(webmcp-registered-tools): add audit to list registered webmcp tools#16959
Conversation
…cp tool on website
paulirish
left a comment
There was a problem hiding this comment.
gatherer looks good!
audit feels cramped but i got some ideas.
| async getArtifact(context) { | ||
| const session = context.driver.defaultSession; | ||
|
|
||
| // Remove duplicates based on name, keeping the latest occurrence. |
There was a problem hiding this comment.
Did you witness this happening in the wild already?
There was a problem hiding this comment.
No! Just being cautious. If there is multiple registrations happening for the same name, we just keep the latest.
| * @param {{tools: WebMCPTool[]}} event | ||
| */ | ||
| onToolsAdded(event) { | ||
| // Note there is a bug right now in WebMCP.enable CDP where on newly registered tools |
There was a problem hiding this comment.
I wonder if you're hitting this line: https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/html/forms/html_form_element.cc;l=271;drc=9a98849abd453b7472fb87b971c19162dfe3ea19
But I don't know why ;(
There was a problem hiding this comment.
There was a problem hiding this comment.
Yea I tried computing the schema at registration time instead of the hard coded {} and that still didn't give me anything. Ultimately I think the children are not done processing by the time the CDP event fires. Talked to WebMCP CDP team about it and they're taking a look.
There was a problem hiding this comment.
Hopefully https://chromium-review.googlesource.com/c/chromium/src/+/7758503 will fix this
|
Latest report UI: |
add webmcp audit to custom agentic category.
off by default for now.
note there is an issue with the CDP domain right now, where it's not showing the schema for newly registered declarative tools while WebMCP domain is enabled.