Skip to content

[BEEEP] [PM-38497] Dev-ex: Add note for Add attach VSCode debugger to desktop, web configs#813

Merged
quexten merged 4 commits into
mainfrom
km/update-debuggers
Jun 4, 2026
Merged

[BEEEP] [PM-38497] Dev-ex: Add note for Add attach VSCode debugger to desktop, web configs#813
quexten merged 4 commits into
mainfrom
km/update-debuggers

Conversation

@quexten
Copy link
Copy Markdown
Contributor

@quexten quexten commented Jun 3, 2026

🎟️ Tracking

bitwarden/clients#21016
https://bitwarden.atlassian.net/browse/PM-38497

📔 Objective

Adds notes on how to attach debuggers to desktop / web clients from vscode.

📸 Screenshots

@quexten quexten changed the title Add notes about attaching debuggers [BEEEP] [PM-38497] Dev-ex: Add note for Add attach VSCode debugger to desktop, web configs Jun 3, 2026
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Jun 3, 2026

Deploying contributing-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9c165b7
Status:⚡️  Build in progress...

View logs

@quexten quexten marked this pull request as ready for review June 3, 2026 11:03
@quexten quexten requested a review from a team as a code owner June 3, 2026 11:03
@quexten quexten requested a review from Thomas-Avery June 3, 2026 11:03
@quexten quexten enabled auto-merge (squash) June 3, 2026 11:03
@withinfocus withinfocus added the ai-review Request a Claude code review label Jun 3, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR adds a new docs/getting-started/clients/debugging.mdx page documenting how to attach the VS Code debugger to the Desktop (Electron main + renderer) and Web Vault (via Chrome remote debugging) clients, and updates desktop/index.mdx and web-vault/index.mdx to link to the new page. The new page follows existing conventions for frontmatter, Tabs/TabItem/CodeBlock imports, and anchor links. Sidebar positions do not collide with sibling files.

Code Review Details
  • 🎨 : "VSCode" used once where the rest of the file/codebase uses "VS Code"; may also fail npm run spellcheck
    • docs/getting-started/clients/debugging.mdx:21

Copy link
Copy Markdown
Contributor

@Thomas-Avery Thomas-Avery left a comment

Choose a reason for hiding this comment

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

Nice 🚀

@quexten quexten merged commit 446bf06 into main Jun 4, 2026
11 of 12 checks passed
@quexten quexten deleted the km/update-debuggers branch June 4, 2026 21:55
Comment on lines +20 to +21
Select **`Desktop: Attach Debugger (main + renderer)`** from the Run and Debug dropdown. This
attaches VSCode's debugger to a running desktop client. You can then set breakpoints directly in the
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🎨 SUGGESTED: Use "VS Code" (with space) for consistency with the rest of the file and codebase.

Details and fix

This line uses "VSCode" without a space, but every other occurrence in this file (lines 9, 11, 52) and across the docs (e.g. tools/index.md, commit-signing.mdx, desktop/index.mdx, web-vault/index.mdx) uses "VS Code". The custom dictionary in custom-words.txt also does not include "VSCode", so this may trigger a npm run spellcheck failure.

Suggested change
Select **`Desktop: Attach Debugger (main + renderer)`** from the Run and Debug dropdown. This
attaches VSCode's debugger to a running desktop client. You can then set breakpoints directly in the
Select **`Desktop: Attach Debugger (main + renderer)`** from the Run and Debug dropdown. This
attaches VS Code's debugger to a running desktop client. You can then set breakpoints directly in the

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants