') + ')', 'gi'); if (regex.test(text)) { found = true; var frag = document.createDocumentFragment(); var parts = text.split(regex); parts.forEach(function(part, i) { if (i % 2 === 0) { frag.appendChild(document.createTextNode(part)); } else { var span = document.createElement('span'); span.className = 'userscript-highlight'; span.textContent = part; frag.appendChild(span); } }); node.parentNode.replaceChild(frag, node); } }); } else if (node.nodeType === 1 && node.childNodes) { // element var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT']; if (!skipTags.includes(node.tagName)) { Array.from(node.childNodes).forEach(highlight); } } } highlight(document.body); // Re-highlight on dynamic content var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1 || node.nodeType === 3) highlight(node); }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ', 'i'); if (__m === '*' || __re.test(location.href)) { // Strip utm_, fbclid, gclid, etc. from all links on page (function() { var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content', 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid', 'ref', 'ref_src', 'source', 'medium', 'campaign']; function cleanUrl(url) { try { var u = new URL(url, window.location.origin); var changed = false; trackingParams.forEach(function(p) { if (u.searchParams.has(p)) { u.searchParams.delete(p); changed = true; } }); return changed ? u.toString() : url; } catch (e) { return url; } } function cleanLinks() { document.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } cleanLinks(); var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1) { if (node.tagName === 'A') cleanLinks(); node.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + ', 'i'); if (__m === '*' || __re.test(location.href)) { // Auto-enable theater mode on YouTube (function() { function tryTheater() { var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]'); if (btn && !btn.classList.contains('activated')) { btn.click(); } } // Try immediately tryTheater(); // Try after navigation (SPA) var lastUrl = location.href; setInterval(function() { if (location.href !== lastUrl) { lastUrl = location.href; setTimeout(tryTheater, 500); } }, 1000); // Also try on player load var observer = new MutationObserver(tryTheater); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ', 'i'); if (__m === '*' || __re.test(location.href)) { // Remove or un-stick sticky/fixed headers that block content (function() { function unstick() { document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) { if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') { el.style.position = 'static'; el.style.top = 'auto'; el.style.zIndex = 'auto'; } }); } unstick(); var observer = new MutationObserver(unstick); observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] }); })(); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); })(); Add 'terminalQuoteCharacter' to the debug configuration we send to debugpy by rchiodo · Pull Request #886 · microsoft/vscode-python-debugger · GitHub
Skip to content

Add 'terminalQuoteCharacter' to the debug configuration we send to debugpy - #886

Merged
Rich Chiodo (rchiodo) merged 5 commits into
microsoft:mainfrom
rchiodo:rchiodo/fixSpaces
Dec 9, 2025
Merged

Add 'terminalQuoteCharacter' to the debug configuration we send to debugpy#886
Rich Chiodo (rchiodo) merged 5 commits into
microsoft:mainfrom
rchiodo:rchiodo/fixSpaces

Conversation

@rchiodo

@rchiodoRich Chiodo (rchiodo) commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

In order to fixmicrosoft/debugpy#1980, the debugger needs to quote the values it sends in the 'runInTerminal' request. However the quote being used depends upon the terminal that will eventually be used.

For example, quote a string this way:

"C:\My Dir With Spaces\python.exe"

will fail on powershell, but succeed on cmd.exe

Powershell requires single quotes like so:

'C:\My Dir With Spaces\python.exe'

This PR adds a new terminalQuoteCharacter to the debug configuration we send to debugpy. A follow up PR in debugpy will use this value to properly quote arguments in the 'runInTerminal' reverse request.

@rchiodoRich Chiodo (rchiodo) added the bug Issue identified by VS Code Team member as probable bug label Dec 9, 2025
@rchiodoRich Chiodo (rchiodo) mentioned this pull request Dec 9, 2025
Merged
eleanorjboyd
Eleanor Boyd (eleanorjboyd) reviewed Dec 9, 2025
View reviewed changes
Comment threadcontributing.md
@vs-code-engineeringvs-code-engineeringBot added this to the December / January 2026 milestone Dec 9, 2025
@rchiodo
Rich Chiodo (rchiodo) merged commit 5a633be into microsoft:mainDec 9, 2025
18 checks passed
@eleanorjboydEleanor Boyd (eleanorjboyd) mentioned this pull request Dec 12, 2025
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eleanorjboydEleanor Boyd (eleanorjboyd)eleanorjboyd approved these changes
@heejaechangHeejae Chang (heejaechang)Awaiting requested review from heejaechang
@bschnurrBill Schnurr (bschnurr)Awaiting requested review from bschnurr
@StellaHuang95Stella Huang (StellaHuang95)Awaiting requested review from StellaHuang95

Assignees

No one assigned

Labels

bugIssue identified by VS Code Team member as probable bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Cannot debug with cmdline args if python.exe in 'c:\Program Files'

2 participants

@rchiodo@eleanorjboyd