Skip to content

devex: Enable Electron MCP servers with DevTools debug port#25795

Merged
Hona merged 1 commit intoanomalyco:devfrom
Hona:fix/electron-debug-port
May 5, 2026
Merged

devex: Enable Electron MCP servers with DevTools debug port#25795
Hona merged 1 commit intoanomalyco:devfrom
Hona:fix/electron-debug-port

Conversation

@Hona
Copy link
Copy Markdown
Member

@Hona Hona commented May 5, 2026

Summary

  • expose Electron's remote debugging port in non-packaged desktop builds
  • keeps packaged builds unchanged

Tests

  • bun typecheck in packages/desktop-electron
  • push hook: bun turbo typecheck

Copilot AI review requested due to automatic review settings May 5, 2026 01:09
@Hona Hona enabled auto-merge (squash) May 5, 2026 01:10
@Hona Hona disabled auto-merge May 5, 2026 01:10
@Hona Hona changed the title Enable Electron MCP servers with DevTools debug port devex: Enable Electron MCP servers with DevTools debug port May 5, 2026
@Hona Hona enabled auto-merge (squash) May 5, 2026 01:11
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR exposes Electron/Chromium’s remote debugging port for non-packaged desktop builds, enabling tooling that relies on the DevTools protocol while leaving packaged (production) builds unchanged.

Changes:

  • Add --remote-debugging-port=9222 to Electron command-line switches when app.isPackaged === false.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

function setupApp() {
ensureLoopbackNoProxy()
app.commandLine.appendSwitch("proxy-bypass-list", "<-loopback>")
if (!app.isPackaged) app.commandLine.appendSwitch("remote-debugging-port", "9222")
@Hona Hona disabled auto-merge May 5, 2026 01:12
@Hona Hona enabled auto-merge (squash) May 5, 2026 01:15
@Hona Hona disabled auto-merge May 5, 2026 01:37
@Hona Hona merged commit 2740d39 into anomalyco:dev May 5, 2026
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants