From d655976ccf33fb5ea17bdc1bd38dce95a7b4f0e6 Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 24 Aug 2026 18:40:19 +0000 Subject: [PATCH] docs(pw): propagate 'MCP servers not auto-registered' caveat to remaining docs (#978) Addresses review feedback on #981: README.md, skills/pw/SKILL.md, and the testrail/browserstack SKILL.md files still described the integrations as working after just exporting env vars. Add a caveat to each that pw-testrail/pw-browserstack are no longer auto-registered and that /pw:testrail // /pw:browserstack (and the testrail_*/browserstack_* MCP tools) fail with 'tool not found' until the server is enabled manually, pointing to the CLAUDE.md Integrations section. --- engineering-team/playwright-pro/README.md | 8 ++++++++ .../playwright-pro/skills/browserstack/SKILL.md | 9 +++++++++ engineering-team/playwright-pro/skills/pw/SKILL.md | 7 +++++++ engineering-team/playwright-pro/skills/testrail/SKILL.md | 8 ++++++++ 4 files changed, 32 insertions(+) diff --git a/engineering-team/playwright-pro/README.md b/engineering-team/playwright-pro/README.md index 3e31f2d3..aa489652 100644 --- a/engineering-team/playwright-pro/README.md +++ b/engineering-team/playwright-pro/README.md @@ -80,6 +80,14 @@ Ready-to-use, parametrizable templates covering: ## Integrations Setup +> **Note (issue #978):** the TestRail and BrowserStack MCP servers are **not +> auto-registered** — they were removed from the plugin's `.mcp.json` because they +> failed to connect for every user (no bundled `node_modules`). Setting the env +> vars below is **not** enough on its own; `/pw:testrail` and `/pw:browserstack` +> will fail with "tool not found" until you enable the server manually +> (`cd integrations/-mcp && npm install`, then register it in your own +> user/project MCP config). See the **Integrations** section of `CLAUDE.md`. + ### TestRail (Optional) Set environment variables: diff --git a/engineering-team/playwright-pro/skills/browserstack/SKILL.md b/engineering-team/playwright-pro/skills/browserstack/SKILL.md index 17def4f4..24bdf990 100644 --- a/engineering-team/playwright-pro/skills/browserstack/SKILL.md +++ b/engineering-team/playwright-pro/skills/browserstack/SKILL.md @@ -18,6 +18,15 @@ Environment variables must be set: If not set, inform the user how to get them from [browserstack.com/accounts/settings](https://www.browserstack.com/accounts/settings) and stop. +> **The BrowserStack MCP server is not auto-registered (issue #978).** +> `pw-browserstack` was removed from the plugin's `.mcp.json` because it failed to +> connect for every user (the plugin ships no `node_modules`). The `browserstack_*` +> MCP tools used below, and the `/pw:browserstack` command, will fail with "tool +> not found" until it is enabled manually — see the **Integrations** section of the +> plugin's `CLAUDE.md` (`cd integrations/browserstack-mcp && npm install`, then +> register the server in your own user/project MCP config). Setting the env vars +> alone is not sufficient. + ## Capabilities ### 1. Configure for BrowserStack diff --git a/engineering-team/playwright-pro/skills/pw/SKILL.md b/engineering-team/playwright-pro/skills/pw/SKILL.md index 3647e875..3f572ec2 100644 --- a/engineering-team/playwright-pro/skills/pw/SKILL.md +++ b/engineering-team/playwright-pro/skills/pw/SKILL.md @@ -98,6 +98,13 @@ npx playwright test tests/auth/login.spec.ts --headed ## Integration Setup +> **Not auto-registered (issue #978).** The TestRail and BrowserStack MCP servers +> are no longer declared in the plugin's `.mcp.json` (they failed to connect for +> every user — no bundled `node_modules`). Exporting the env vars below is not +> enough: `/pw:testrail` / `/pw:browserstack` fail with "tool not found" until you +> enable the server manually (`cd integrations/-mcp && npm install`, then +> register it in your own user/project MCP config). See `CLAUDE.md` → Integrations. + ### TestRail (Optional) ```bash export TESTRAIL_URL="https://your-instance.testrail.io" diff --git a/engineering-team/playwright-pro/skills/testrail/SKILL.md b/engineering-team/playwright-pro/skills/testrail/SKILL.md index 60183bce..1684c4dc 100644 --- a/engineering-team/playwright-pro/skills/testrail/SKILL.md +++ b/engineering-team/playwright-pro/skills/testrail/SKILL.md @@ -19,6 +19,14 @@ Environment variables must be set: If not set, inform the user how to configure them and stop. +> **The TestRail MCP server is not auto-registered (issue #978).** `pw-testrail` +> was removed from the plugin's `.mcp.json` because it failed to connect for every +> user (the plugin ships no `node_modules`). The `testrail_*` MCP tools used below, +> and the `/pw:testrail` command, will fail with "tool not found" until it is +> enabled manually — see the **Integrations** section of the plugin's `CLAUDE.md` +> (`cd integrations/testrail-mcp && npm install`, then register the server in your +> own user/project MCP config). Setting the env vars alone is not sufficient. + ## Capabilities ### 1. Import Test Cases → Generate Playwright Tests