chore(cli): self-documenting cli.config.json#40081
chore(cli): self-documenting cli.config.json#40081Skn0tt wants to merge 2 commits intomicrosoft:mainfrom
cli.config.json#40081Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
|
||
| import type * as playwright from '../../..'; | ||
|
|
||
| // This file contains type declarations for the MCP and CLI config. Please keep the following files in sync: |
There was a problem hiding this comment.
I think this file is copied verbatim to microsoft/playwright-mcp repo, so let's not have internal comments inside.
There was a problem hiding this comment.
I think we need the comment so that agents know to keep the other one updated. Why are you concerned about adding it to the other repo?
| }, | ||
|
|
||
| // List of enabled tool capabilities. | ||
| // Possible values: "config", "core", "core-navigation", "core-tabs", |
There was a problem hiding this comment.
It's hard to imagine this list of possible values will be kept in sync.
There was a problem hiding this comment.
while we have the comment, I'd hope the agent keeps it in sync 🤷
Test results for "tests 1"1 failed 8 flaky39138 passed, 846 skipped Merge workflow run. |
This comment has been minimized.
This comment has been minimized.
|
I'll wait to for the bundles to settle before continuing this. |
This comment has been minimized.
This comment has been minimized.
Test results for "MCP"6533 passed, 383 skipped Merge workflow run. |
No description provided.