jiebi/CodeConvo huggingface dataset:
Change:
return result as IssueReporterData | undefined; } <ins> // Listens to data from the issue reporter model, which is updated regularly async getIssueData(): Promise<IssuePassData | undefined> { const sendChannel = `vscode:triggerIssueData`; mainWindow.postMessage({ sendChannel }, '*'); const result = await new Promise((resolve, reject) => { const timeout = setTimeout(() => { mainWindow.removeEventListener('message', listener); reject(new Error('Timeout exceeded')); }, 5000); // Set the timeout value in milliseconds (e.g., 5000 for 5 seconds) const listener = (event: MessageEvent) => { const replyChannel = `vscode:triggerIssueDataResponse`; if (event.data && event.data.replyChannel === replyChannel) { clearTimeout(timeout); mainWindow.removeEventListener('message', listener); resolve(event.data.data); } }; mainWindow.addEventListener('message', listener); }); return result as IssuePassData | undefined; } </ins> async $closeReporter(): Promise<void> { this.issueReporterWindow?.close(); }

Discussion:
Testing to open the issue reporter for an extension issue reporter open, return to the main window step 1 for another extension =issue reporter opens but doesn't update the issue reporter input fields.
Change:
"windows-mutex": "0.3.0", "windows-process-tree": "0.2.4" } <del> } </del> <ins> } </ins>  No newline at end of file

Discussion:
For details pls see I've updated the of insider/stable to define the extension as a UI extension Candidate since the bracket pair colorizer is a popular extension and we should also enable users that use the V2 version.
I also previously pinged the extension about fixing this:
Change:
const labelAbout = localize("terminal.learnShellIntegration", 'Learn About Shell Integration'); actions.push({ class: undefined, tooltip: labelAbout, dispose: () => { }, id: 'terminal.learnShellIntegration', label: labelAbout, enabled: true, <del> run: () => this._openerService.open('https://code.visualstudio.com/docs/editor/integrated-terminal#_shell-integration') </del> <ins> run: () => this._openerService.open('https://code.visualstudio.com/docs/terminal/shell-integration') </ins> }); return actions; }

Discussion:
It goes here
should go here
or perhaps better
We should fix this one for July, small change with big impact
Change:
const options: ITextQueryBuilderOptions = { _reason: 'searchView', extraFileResources: this.instantiationService.invokeFunction(getOutOfWorkspaceEditorResources), <del> maxResults: SearchView.MAX_TEXT_RESULTS, </del> <ins> maxResults: withNullAsUndefined(this.searchConfig.maxResults), </ins> disregardIgnoreFiles: !useExcludesAndIgnoreFiles || undefined, disregardExcludeSettings: !useExcludesAndIgnoreFiles || undefined, onlyOpenEditors: onlySearchInOpenEditors,

Discussion:
Investigate removing Max results limitation for Search and replace results Currently we set max limit to . We need to investigate on lifting this.
Assigning this to you. I see that you are working on improving the performance for Search results. Currently we have an hard limitation of 2048. But we never measure what is the threshold. Since you are already exercising Search performance this would be a good one to look at. Thanks
I noticed . An improvement may be to not open all files in which terms were found and replaced. As a software engineer myself I sometimes have to make many changes (just today some 33k machine names in 280 XML files). Version 1.3.1 crashed on me, but even if it had worked, I would have stared at many open files. And there's no shortcut to "save all" and "close all" :( I had to revert to Notepad++, which simply makes the changes in-place on disk, without opening all files. Maybe doing something similar is an option?
This would be nice to fix/improve. I have to open Atom just to do find/replace.
There should at least be a warning telling the user that not all the occurrences have been replaced. It is very misleading when "Replace All" only replaces the first items.
Please increase this number or make it configurable
Same here. I'd like you to make the number configurable. I'd like to set this to "No Limit" also, hopefully.
The current upper bound for search in VSCode is occurrences. Is there a way to change it?
Please make this maximum / upper boundary configurable.
I wonder what is measuring? ! Also, when I set it to 2000, I see 4000 results show up when I search a single letter.
You will also get results in open files beyond the limit
This is a bit of a foot gun... Using some ridicules high value freeses VS Code eventually. Will create a separate issue for that
Change:
@IMenuService private readonly menuService: IMenuService ) { } <del> getResourceActions(elements: ITreeItem[], disposableStore: DisposableStore): { actions: IAction[] } { const actions = this.getActions(MenuId.ViewItemContext, elements, disposableStore); </del> <ins> getResourceActions(elements: ITreeItem[]): { actions: IAction[] } { const actions = this.getActions(MenuId.ViewItemContext, elements); </ins> return { actions: actions.primary }; }

Discussion:
It looks like everything is getting disposed properly and I haven't been able to find a way to repro. It looks like this has only been seen on builds that are 12 days old. Let's see if we see it again.
I can repro if I have a tree with ~145 elements visible, as each one has a listener for whether the inline menu items change base on context key value changes. This listener is also apparently listening for whether a menu item's "hidden state" also changes, which is where the problem occurs. It seems that every menu in the workbench listens for the same "hidden state" event.
The error telemetry should be fixed, but we're still going to listen on tree menus.
Change:
private _terminalInstances = new Map<vscode.Terminal, { lastUsedAt: number; config: string }>(); <del> public async checkout(config: string, name: string) { </del> <ins> public async checkout(config: string, name: string, cleanupOthersByName = false) { </ins> const entries = [...this._terminalInstances.entries()]; const promises = entries.map(([terminal, termInfo]) => createCancelablePromise(async ct => {

Discussion:
<!-- Do Not Delete This! bugreporttemplate --<!-- Please read our Rules of Conduct: --<!-- Please search existing issues to avoid creating duplicates. --<!-- Also please test using the latest insiders build to make sure your issue has not already been fixed: --It's the second time it happened. First time was a fresh project so as I didn't investigate further. I thought I messed up somehow. Second time I lost some datasets and got close to lose some code. I found the culprit so I hope it won't to me a third time. <!-- Use Help Report Issue to prefill these. --Version: 1.48.2 (system setup) Electron: 7.3.2 V8: 7.8.279.23-electron.0 OS: WindowsNT x64 10.0. Steps to Reproduce: a debugging session, that open a fresh python debug console terminal debugging session terminate in python debug console terminal: pressing enter, so the command don't execute but sit there a new debugging session VS code will then reuse the 'python debug console terminal' previously open, with my un-executed command still sitting there. It do its business as usually and inputting: resulting the command: and Poof my whole project is gone <!-- Launch with to check. --Does this issue occur when all extensions are disabled?: Yes/No
We should clear any input before executing a command.
I agree, I'm quite suprise it was not the case already. Waiting for the fix. There is the possibility of setting the debug console to be spawn in the debug output tab. In ’’ ’"console": "internalConsole"’
I'm now cancelling the input with Control-C. This should work on all platforms and for all shells.
Perfect, thanks for your work
The fix for this causes Reverting for the release.
Same thing happens with the C++/CMake extension when pressing F5.
Change:
let description: string; let detail: string; let picked: boolean; <ins> let alwaysShow: boolean; </ins> if (typeof item === 'string') { label = item;

Discussion:
Problem The time you search for the branch and not find any, there are no other options to proceed. <img width="682" alt="screen shot 2018-09-15 at 11 06 09" src="https://user-"I as a developer would need to delete the search and then click on "create a new branch". Solution Keep "create a new branch" option always. <img width="656" alt="screen shot 2018-09-15 at 11 22 56" src="https://user-"
(Experimental duplicate detection) Thanks for submitting this issue. Please also check if it is already covered by an existing one, like: <!-- score: 0.617 --- <!-- score: 0.587 --- <!-- score: 0.552 --- <!-- score: 0.547 --- <!-- score: 0.529 --<!-- potentialduplicatescomment --
Is this supported from an extension? Eg: keep an option always visible.
Dang, just noticed that you already submitted a PR to address this. Closed my PR, feel free to grab the integration test I if you want
since my PR is merged already, feel free to create a PR with tests :)
I a PR to add tests and also fix a small bug (when explicitly settings items on a the field was not properly propagated). Additional thoughts I had (should open another issue for discussion?): In my opinion, items that should always be shown shouldn't display search highlights. The highlights are possibly redundant and IMO it would look more elegant without I feel like after accepting the option, the input text should be preserved. Would a PR to address this be accepted? Also, possibly should this behavior be exposed for user control? (i.e. similarly to a field, a field.)
:+1: Nah, sounds like a very good default value.
Change:
return; } <ins> this._hasProvidedAriaStatus = false; </ins> this.localToDispose.add(this.editor.onMouseDown(e => this.onEditorMouseDown(e))); this.localToDispose.add(this.editor.onMouseUp(e => this.onEditorMouseUp(e))); if (this._editorDisposables.length) {

Discussion:
[x] Indicate when a document has commentable ranges Aria status when a document is opened [x] Add keyboard shortcut for adding a comment [x] Provide commands to go to next and previous commentable range [x] Add an accessible help menu to the comment widget Esc dismisses widget Lists go to next/previous commentable range commands + keyboard shortcuts Lists command and keyboard shortcut to add a comment Lists keyboard shortcut to execute the comment primary action and what the primary action is [x] The Comments view should respect the enter/space = reveal+focus/reveal [x] Make the comment widget toolbar always visible when in screen reader mode
Related to
is the last part of this, and it's almost done!
Mr.KITSANAPAN MEEKEAW
Change:
}); })); <del> const procChildren = fs.readdirSync('/proc'); const processes: { pid: number, cwd: string, cmd: string }[] = []; </del> <ins> const procChildren = await pfs.readdir('/proc'); const processes: { pid: number, cwd: string, cmd: string }[] = []; </ins> for (let childName of procChildren) { try { const pid: number = Number(childName); const childUri = resources.joinPath(URI.file('/proc'), childName); <del> const childStat = fs.statSync(childUri.fsPath); </del> <ins> const childStat = await pfs.stat(childUri.fsPath); </ins> if (childStat.isDirectory() && !isNaN(pid)) { <del> const cwd = fs.readlinkSync(resources.joinPath(childUri, 'cwd').fsPath); const cmd = fs.readFileSync(resources.joinPath(childUri, 'cmdline').fsPath, 'utf8'); </del> <ins> const cwd = await promisify(fs.readlink)(resources.joinPath(childUri, 'cwd').fsPath); const cmd = await pfs.readFile(resources.joinPath(childUri, 'cmdline').fsPath, 'utf8'); </ins> processes.push({ pid, cwd, cmd }); } } catch (e) {

Discussion:
We have been seeing a bunch of remote issues where the remote extension host burns a lot of CPU. I think it is caused by our port scanning code, which executes even if the setting is false. See Possible issues: / Other: ? ? ? *
Looks like we need to work on our indentation and ensuring that we break lines once they go past the edge of the screen for the sake of readability
I could reproduce some increased CPU usage of the remote extension host using the project linked to i.e.
In insiders, you can now temporarily disable the port scanning code by adding this line to your .bashrc file, then restarting the VS Code server on your remote:
Once we have an insiders build I'll start commenting in each of the issues asking folks to try out the workaround in insiders.
Keeping open for now so that the issue can be found more easily.
Closing since the fix is available in insiders.
Verified that cpu usage was reasonable when running the petclinic app
Change:
removed.push(...this._extensions.splice(index, 1)); } } <del> if (removed.length) { this._onDidChangeExtensions.fire({ added: [], removed }); </del> <ins> if (added.length || removed.length) { this._onDidChangeExtensions.fire({ added, removed, isProfileSwitch }); </ins> } } }

Discussion:
VS Code Version: stable 1.74.0 OS Version: Windows 11 Build .900 Feature: Settings Profiles (Profiles) When certain Extensions that I had disabled for the active profile automatically update, they get re-inserted (re-enabled) in my Extensions list for that profile, and will remain there from that point onwards. I've noticed this with all the "C++" extensions from Microsoft, as well as the "Remote Explorer", and "Remote - SSH" also from Microsoft, but this might be happening with other extensions as well. Even if I disable them again for that profile, they'll come back the next time they update Steps to Reproduce: Profiles feature by setting the to true. the C++ from Microsoft on the default profile. a new profile "A" other than default, disable the C++ extension. a second profile "B" but enable the C++ extension. a third profile "C" following step 2. to either A or C profile. vscode, reopen it on the same folder, and wait for the C++ extension to update (I'm not aware of any means to force an extension update besides creating one myself). You'll notice that once the C++ extension update, it will re-enable automatically on both profiles A and C, and will remain there even after re-starting vscode, until you disable it again. In which case it will reappear again once it updates.
How did you force the extension update? Can you please share your profiles (you can use profile export feature) so that I can reproduce?
Hi as mentioned in the issue, I did not force the update since I'm not aware of any means to do that, I just waited for extensions to update. By the way I also noticed this happening with a non-Microsoft extension called "Better C++ Syntax" How can I share the profiles with you in a non-public manner?
You can export the profile to a file or github gist and share it with me via email -
Done. Hope that can solve it since it keeps happening. Thank you
Why is this still labelled as info-needed?
is there a way to simulate the extension update that is needed to reproduce the issue?

Yeah there is, but its a lengthy one: Please open extensions installation folder and the folder where profiles are stored that is user data folder/User/profiles Quit VS Code Change the name of the extension in the extensions installation folder and also the version in the Update the name and version in file in all profiles include the file in extension installation folder.
Change:
get alwaysShow(): boolean { return true; } } <ins> class AddRemoteItem implements QuickPickItem { constructor(private cc: CommandCenter) { } get label(): string { return localize('add remote', '$(plus) Add a new remote...'); } get description(): string { return ''; } get alwaysShow(): boolean { return true; } async run(repository: Repository): Promise<void> { await this.cc.addRemote(repository); } } </ins> interface CommandOptions { repository?: boolean; diff?: boolean;

Discussion:
To address the original issue in more directly, it would be great if this worked: Create a new branch Run the "Publish branch" command See the "Pick a remote..." prompt Select a "Add New Remote" option The remote is and the branch is pushed to that remote Is that what you originally had in mind
I would like to take this.
yep that was it.
Change:
import { INotebookActionContext, INotebookCellActionContext, NotebookAction, NotebookCellAction, NOTEBOOK_EDITOR_WIDGET_ACTION_WEIGHT } from 'vs/workbench/contrib/notebook/browser/controller/coreActions'; import { CellEditState, NOTEBOOK_CELL_HAS_OUTPUTS, NOTEBOOK_EDITOR_FOCUSED, NOTEBOOK_OUTPUT_FOCUSED } from 'vs/workbench/contrib/notebook/browser/notebookBrowser'; import { CellKind, NOTEBOOK_EDITOR_CURSOR_BOUNDARY } from 'vs/workbench/contrib/notebook/common/notebookCommon'; <ins> import { EditorExtensionsRegistry } from 'vs/editor/browser/editorExtensions'; </ins> const NOTEBOOK_FOCUS_TOP = 'notebook.focusTop'; const NOTEBOOK_FOCUS_BOTTOM = 'notebook.focusBottom';

Discussion:
Issue Type: <bBug</ba notebook file. a cell which is large enough to overflow screen. <kbdPgDown</kbdand <kbdPgUp</kbdInstead of going one page down in PgDown, cursor goes to end of cell ! Does this issue occur when all extensions are disabled?: Yes VS Code version: Code 1.63.2 (, 2021-12-15T09:39:46.686Z) OS version: Linux x64 5.13.0-22-generic snap Restricted Mode: No <details<summarySystem Info</summary </details<details<summaryA/B Experiments</summary</details<!-- generated by issue reporter --
Change:
const vscode = acquireVsCodeApi(); delete (globalThis as any).acquireVsCodeApi; <del> const tokenizationStyleElement = document.querySelector('style#vscode-tokenization-styles'); </del> <ins> const tokenizationStyle = new CSSStyleSheet(); tokenizationStyle.replaceSync(ctx.style.tokenizationCss); </ins> const runWhenIdle: (callback: (idle: IdleDeadline) => void, timeout?: number) => IDisposable = (typeof requestIdleCallback !== 'function' || typeof cancelIdleCallback !== 'function') ? (runner) => { setTimeout(() => {

Discussion:
<!-- Please search existing issues to avoid creating duplicates. --- VS Code version: 1.79.1 Jupyter Extension version: 2023.5. Python Extension version: 2023.10.1 OS and version: Windows 11 Home Version 22H2, build .1778 If I change the theme after entering code in a Markdown cell, the code highlighting should change accordingly. Light theme: <img width="51" alt="image" src=""Dark theme: <img width="52" alt="image" src=""The code highlighting in Markdown cells does not change according to the theme. Light theme: <img width="44" alt="image" src=""Dark theme: <img width="48" alt="image" src=""an .ipynb file. a Markdown cell. some text formatted as code, e.g. python a = 5 ` Shift + Enter to render cell. theme (for example, if code was entered when using Light Modern, switch to Dark Modern). The code highlighting in the Markdown cell does not change and is now difficult to read.
Change:
args.push('--pcre2'); } <ins> // Allow $ to match /r/n args.push('--crlf'); </ins> if (query.isRegExp) { query.pattern = unicodeEscapesToPCRE2(query.pattern); <ins> args.push('--auto-hybrid-regex'); </ins> } <del> // Allow $ to match /r/n args.push('--crlf'); </del> let searchPatternAfterDoubleDashes: Maybe<string>; if (query.isWordMatch) { const regexp = createRegExp(query.pattern, !!query.isRegExp, { wholeWord: query.isWordMatch });

Discussion:
Issue Type: <bBug</bFind in Files Regexp with lookbehind Use Regular Expression and Match Whole Word "Regex parse error" message appear This works fine in Find widget and if manually to regexp. Root cause: is not if is used with . /cc VS Code version: Code - Insiders 1.39.0-insider (, 2019-10-04T09:24:45.690Z) OS version: WindowsNT x64 10.0. <details<summarySystem Info</summary </detailsExtensions: none <!-- generated by issue reporter --
(Experimental duplicate detection) Thanks for submitting this issue. Please also check if it is already covered by an existing one, like: <!-- score: 0.598 --- <!-- score: 0.549 --- <!-- score: 0.5 --<!-- potentialduplicatescomment --
Change:
*/ ignoreCase?: boolean; /** <ins> * is the language unicode-aware? (i.e., /u{1D306}/) */ unicode?: boolean; /** </ins> * if no match in the tokenizer assign this token class (default 'source') */ defaultToken?: string;

Discussion:
<!-- Do Not Delete This! featurerequesttemplate --<!-- Please read our Rules of Conduct: --<!-- Please search existing issues to avoid creating duplicates. --<!-- Describe the feature you'd like. --Feature request for the Monaco Editor. Create an optional flag for Monaco Monarch language definitions to indicate the usage of unicode-aware regular expressions. An example of one such regular expression is , matching the '' character.
What exactly does not work? Since you can pass in regular expressions directly e.g.
Hi, What's currently not working is the unicode-aware regular expressions supported by ES2015. Even though you're able to write a RegExp including unicode-aware characters (in the format ), and even set the unicode flag for that RegExp, they don't work in Monaco Monarch at the moment. Below is the monaco-language file for ABAP, edited to support the character in identifiers. ! This is how it shows up in the editor: ! That happens precisely because the unicode flag is stripped away by compileRegExp, causing the match to fail for that character: If we change that line to, say it works as expected: ! The proposed solution is a way for the developer to tell Monaco Monarch that, instead of stripping away the unicode flag from all the regular expressions, it should instead add it to all the regular expressions if a flag is set in the language definition, just like it's the current behavior for the option. Another option I've considered was to instead use the unicode flag if, and only if, it is set in the regular expression itself, but from what I could gather, that would force us to compile the RegExp an extra time just for testing the flag, or to parse the RegExp manually. Let me know if you need more information.
:+1: Thank you, this makes good sense!
Change:
builtin printf "033]633;D;%s007" "$__vsc_status" __vsc_last_history_id=$__vsc_history_id fi <ins> else builtin printf "033]633;D007" </ins> fi __vsc_update_cwd }

Discussion:
!
This happens because command executed happens but not command finished
And that's because of this:
nice, the fix works https://user-