Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 25 additions & 4 deletions .agent/tools/browser/playwriter.md
Original file line number Diff line number Diff line change
Expand Up @@ -59,12 +59,14 @@ Add to your MCP client configuration:
"mcp": {
"playwriter": {
"type": "local",
"command": ["npx", "playwriter@latest"],
"command": ["/opt/homebrew/bin/npx", "-y", "playwriter@latest"],
Copy link

Choose a reason for hiding this comment

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

This hard-codes the Homebrew Apple Silicon npx path, which will differ for Intel Homebrew (and for nvm/asdf installs), so copy/paste may fail for many readers. Consider clarifying that the npx path is environment-specific and should be verified locally before use.

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎

"enabled": true
}
}
}
```text
```

Choose a reason for hiding this comment

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

medium

To enable syntax highlighting for this JSON example and improve readability, please specify the language in the markdown code block. Using json is appropriate here.

Suggested change
```
```json


> **Note**: Use full path to `npx` (e.g., `/opt/homebrew/bin/npx` on macOS with Homebrew) for reliability. The `-y` flag auto-confirms package installation.
Copy link

Choose a reason for hiding this comment

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

This note calls out full-path npx for OpenCode, but Claude Desktop (GUI-launched) often doesn’t inherit the user’s shell PATH, so command: "npx" can also fail. It may be worth mentioning that Claude Desktop users might need a full npx path for the same reliability reason.

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎


**Claude Desktop** (`claude_desktop_config.json`):

Expand All @@ -73,11 +75,30 @@ Add to your MCP client configuration:
"mcpServers": {
"playwriter": {
"command": "npx",
"args": ["playwriter@latest"]
"args": ["-y", "playwriter@latest"]
Comment on lines 77 to +78

Choose a reason for hiding this comment

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

medium

There's an inconsistency in the documentation. The note for the OpenCode configuration recommends using the full path to npx for reliability, but this claude_desktop example uses just npx. This could be confusing for users. For consistency and to ensure reliability, please consider either updating this example to also use a full path for npx or adding a clarification on why it's not necessary in this case.

}
}
}
```text
```

Choose a reason for hiding this comment

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

medium

Similar to the other configuration block, specifying the language as json here will enable syntax highlighting, making the example easier to read and understand.

Suggested change
```
```json


**Enable per-agent** (OpenCode tools section):

```json
{
"tools": {
"playwriter_*": false
},
"agent": {
"Build+": {
"tools": {
"playwriter_*": true
}
}
}
}
```

> **Tip**: Disable globally with `"playwriter_*": false` in `tools`, then enable per-agent to reduce context token usage.

## Usage

Expand Down
Loading