Skip to content

GH#3746: Fix broken code block fences and add PATH note in Playwriter docs#4047

Merged
marcusquinn merged 1 commit intomainfrom
chore/fix-playwright-docs-quality-debt
Mar 10, 2026
Merged

GH#3746: Fix broken code block fences and add PATH note in Playwriter docs#4047
marcusquinn merged 1 commit intomainfrom
chore/fix-playwright-docs-quality-debt

Conversation

@marcusquinn
Copy link
Copy Markdown
Owner

Summary

  • Fix 8 code blocks in playwriter.md that used ```text as closing fence instead of ```, breaking syntax highlighting and rendering subsequent content as plain text blocks
  • Add PATH guidance note to Claude Desktop MCP config example per CodeRabbit review feedback

Details

The Playwriter documentation had a systematic bug where JavaScript code blocks were closed with ```text instead of ```. This caused:

  1. The closing fence to be interpreted as an opening fence for a new text code block
  2. The content between the broken closing and the next code block opening to render as plain text inside a code block
  3. Loss of syntax highlighting for the affected JavaScript examples

Also addresses CodeRabbit feedback from PR #153 about the Claude Desktop config example not mentioning the full npx path, which is inconsistent with the OpenCode example that recommends it for reliability.

Verification

  • markdownlint-cli2: 0 errors
  • Only remaining ```text is the legitimate opening fence for the ASCII architecture diagram (line 228)

Closes #3746

… docs

Fix 8 code blocks using ```text as closing fence instead of ```,
which broke syntax highlighting and rendered subsequent content as
plain text blocks. Add PATH guidance note to Claude Desktop MCP
config example per CodeRabbit review feedback on PR #153.

Closes #3746
@gemini-code-assist
Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions bot added the bug Auto-created from TODO.md tag label Mar 10, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 10, 2026

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Review skipped — only excluded labels are configured. (1)
  • no-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 02feb221-8c45-4075-b6b5-abcf94139dd3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/fix-playwright-docs-quality-debt

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown
Contributor

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

�[0;34m[INFO]�[0m Latest Quality Status:
SonarCloud: 0 bugs, 0 vulnerabilities, 399 code smells

�[0;34m[INFO]�[0m Recent monitoring activity:
Tue Mar 10 04:24:21 UTC 2026: Code review monitoring started
Tue Mar 10 04:24:22 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 399

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 399
  • VULNERABILITIES: 0

Generated on: Tue Mar 10 04:24:24 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

@sonarqubecloud
Copy link
Copy Markdown

@marcusquinn marcusquinn merged commit 86143c0 into main Mar 10, 2026
19 checks passed
@marcusquinn marcusquinn deleted the chore/fix-playwright-docs-quality-debt branch March 10, 2026 04:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Auto-created from TODO.md tag

Projects

None yet

Development

Successfully merging this pull request may close these issues.

quality-debt: PR #153 review feedback (high)

1 participant