chore: adopt Playwright CLI over MCP for browser automation#46
Merged
Conversation
- Disable markdownlint MD034 so URLs need not be wrapped in angle brackets - Unwrap the one existing `<...>` URL in agentmd-over-skills.md to comply Bare URLs render identically on GitHub and remove friction when pasting links into research notes; the single existing offender is corrected so the lint baseline stays clean. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Captures the preliminary research informing the upcoming switch from Playwright MCP to the Playwright CLI for browser-driven Claude Code workflows on this project. Contents: - Side-by-side comparison of capability surfaces, calling conventions, and token economics - Decision matrix mapping common Next.js iteration tasks to the recommended surface - Migration steps, plus notes on when MCP still wins (chat clients, headed-first ergonomics, existing investment) Filed under .xdocs/own/ as a reference document; no code or config changes accompany it. Subsequent commits on this branch will action the migration itself. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…g pin Dependency updates: - next 16.2.4 → 16.2.6 - react / react-dom 19.2.5 → 19.2.6 - @biomejs/biome 2.4.13 → 2.4.15 (biome.json schema ref updated) - @playwright/test 1.59.1 → 1.60.0 - @tailwindcss/postcss 4.2.4 → 4.3.0; tailwindcss 4.2.2 → 4.3.0 - @types/node 25.6.0 → 25.7.0 - jsdom 29.0.2 → 29.1.1 - vitest 4.1.5 → 4.1.6 Removed devDependency: - baseline-browser-mapping was pinned without documented reason in PR #12 (Dec 2025); next@16.2.6 and browserslist already pull it in as a transitive dependency, making the explicit pin redundant. Clean install verified: `npm run check` and `npm run test` both pass against the regenerated lockfile. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Browser tooling: - Remove @playwright/mcp server from .mcp.json - Drop mcp__playwright__browser_* permissions from .claude/settings.json - Add Bash(playwright-cli *) permission to .claude/settings.json - Add .playwright/cli.config.json (Chromium/Chrome, mobile viewport 390×844, headless, TypeScript codegen) - Track cli.config.json via .gitignore negation whilst ignoring artefacts Playwright CLI skill: - Add .claude/skills/playwright-cli/SKILL.md — core skill definition with quick-start and full command reference - Add 10 reference docs covering element attributes, playwright-tests integration, request mocking, running code, session management, spec-driven testing, storage state, test generation, tracing, and video recording README: - Expand setup steps to include global @playwright/cli install and playwright-cli install --skills bootstrap - Update .mcp.json table entry to reflect Playwright MCP removal Playwright CLI avoids a persistent MCP process and the npx cold-start on first use. Browser config is version-controlled in cli.config.json rather than passed as MCP server flags. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Bump Tailwind 4.2 → 4.3, Playwright 1.59 → 1.60 - Add centralised theme location (app/globals.css) to Styling entry - Remove known-tool parentheticals from Quality and Deployment entries - Add playwright-cli example to Key Commands Adds signal where it's useful (theme token location) and removes prose that restates what tool names already imply. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Vendor skill content lives in .claude/skills/ but is not maintained in this repo, so linting it generates noise without value. Adding it to the markdownlint-cli2 ignore list keeps lint output focused on files we actually own. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (22)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@playwright/mcpfrom.mcp.json, drop its permissions, addBash(playwright-cli *), and version-control browser config in.playwright/cli.config.json(Chromium, mobile 390×844, headless).playwright-cliskill (SKILL.md+ 10 reference docs covering element attributes, playwright-tests integration, request mocking, running code, session/storage state, spec-driven testing, test generation, tracing, video recording) and a research note (.xdocs/own/playwright-cli-over-mcp.md) explaining the trade-off.baseline-browser-mappingpin, allow bare URLs in markdown (MD034), exclude vendor skill content from markdownlint, and tightenCLAUDE.mdtech-stack entries.Test plan
npm installsucceeds against the regenerated lockfilenpm run checkpasses (Biome lint + tsc typecheck, 26 files, 0 errors)npm run testpasses (4 Vitest unit + 2 Playwright e2e, Chromium + Mobile Safari)npm run lint:mdpasses (after excluding.claude/skills/**as vendor content).mcp.jsonno longer lists Playwright MCP (Ref server remains)playwright-cli open http://localhost:3000against a running dev server🤖 Generated with Claude Code