Keep block code plain when copying from rendered markdown

[yashranaway]

What Changed

The markdown clipboard serializer now recognizes block code that arrives without its pre wrapper (highlighter line spans or embedded newlines) and copies it verbatim instead of wrapping it like inline code. Real inline code keeps its backticks.

Why

Selecting the contents of a rendered code block can produce a copy fragment holding the code element but not its pre parent, most visibly on Firefox. The serializer fell into the inline-code branch and wrapped the copied text in single backticks. Pasting that into a shell runs the command inside command substitution:

λ `git show-ref --verify refs/remotes/origin/opt/deploy/dev`
zsh: 97649d91...: command not found

Copies that legitimately span prose plus a whole code block still include the pre element and keep their fenced form, so the markdown-preserving copy feature is unchanged.

Fixes #4368

Testing

• vp test run apps/web/src/markdown-clipboard.test.ts passes (3/3): inline code still wrapped, highlighted block selection stays plain, multi-line selection stays plain
• pnpm typecheck in apps/web clean
• vp lint on changed files clean

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes
• I included a video for animation/interaction changes

---

Note

Low Risk
Small, localized change to markdown copy serialization with new unit tests; no auth, data, or API impact.

Overview
Copying from rendered chat markdown no longer wraps block code in single backticks when the selection only includes the inner code node (common in Firefox) instead of the surrounding pre.

serializeRenderedMarkdownFragment now uses isBlockCodeElement to treat CODE as block when the text has newlines or Shiki-style .line child spans, and returns plain text in those cases. True inline code still gets backtick wrapping.

Unit tests cover inline code, highlighted block fragments without pre, and multi-line selections.

^{Reviewed by Cursor Bugbot for commit 3f7588a. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Keep block code plain when copying from rendered markdown

When copying from rendered markdown, CODE elements that represent block code were incorrectly wrapped in backticks. The fix introduces isBlockCodeElement in markdown-clipboard.ts, which checks for newlines in content or child elements with class line (used by syntax-highlighted blocks). Block code is now returned as plain text; only inline code gets backtick wrapping.

^{Macroscope summarized 3f7588a.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. 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: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 496d76c0-caf7-49cd-8156-7bb38de08afa

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

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Approvability

Verdict: Approved

Small bug fix that prevents block code from being incorrectly wrapped in backticks when copying from rendered markdown. The change is self-contained with comprehensive unit tests covering the new behavior.

^{You can customize Macroscope's approvability policy. Learn more.}