jiebi/CodeConvo huggingface dataset:
Change:
Issue Type: <bBug</bUsually when editing JavaScript files, one can get useful support from TypeScript. However, when editing JavaScript codes in HTML files, this does not work. Repro steps: VSCode like . (<kbd⌘ N</kbd) and set to HTML mode (<kbd⌘ K</kbd→ <kbdM</kbd). the following text: Result: The following problems are detected. of type 'number' is not assignable to parameter of type 'string'. javascript [Ln 4, Col 19] Cannot find module 'lodash'. Did you mean to set the 'moduleResolution' option to 'node', or to add aliases to the 'paths' option? javascript [Ln 6, Col 24] Compare this with (full) JavaScript mode result: of type 'number' is not assignable to parameter of type 'string'. ts(2345) [Ln 3, Col 17] Type 'number' is not assignable to type 'string'. ts(2322) [Ln 7, Col 25] <img width="1392" alt="image" src="https://user-"VS Code version: Code 1.68.1 (, 2022-06-14T12:52:13.188Z) OS version: Darwin x64 20.5.0 Restricted Mode: No <details<summarySystem Info</summary </detailsExtensions disabled<details<summaryA/B Experiments</summary</details<!-- generated by issue reporter --
Yes, we do not resolve import (). But there should not be an error.

Discussion:
}; } <ins> const ignoredErrors = [ 1108,  /* A_return_statement_can_only_be_used_within_a_function_body_1108 */ 2792, /* Cannot_find_module_0_Did_you_mean_to_set_the_moduleResolution_option_to_node_or_to_add_aliases_to_the_paths_option */ ]; </ins> export function getJavaScriptMode(documentRegions: LanguageModelCache<HTMLDocumentRegions>, languageId: 'javascript' | 'typescript', workspace: Workspace): LanguageMode { const jsDocuments = getLanguageModelCache<TextDocument>(10, 60, document => documentRegions.get(document).getEmbeddedDocument(languageId)); const host = getLanguageServiceHost(languageId === 'javascript' ? ts.ScriptKind.JS : ts.ScriptKind.TS); const globalSettings: Settings = {}; return { getId() { return languageId;

const languageService = await host.getLanguageService(jsDocument); const syntaxDiagnostics: ts.Diagnostic[] = languageService.getSyntacticDiagnostics(jsDocument.uri); const semanticDiagnostics = languageService.getSemanticDiagnostics(jsDocument.uri); <del> return syntaxDiagnostics.concat(semanticDiagnostics).filter(d => d.code !== 1108).map((diag: ts.Diagnostic): Diagnostic => { </del> <ins> return syntaxDiagnostics.concat(semanticDiagnostics).filter(d => !ignoredErrors.includes(d.code)).map((diag: ts.Diagnostic): Diagnostic => { </ins> return { range: convertRange(jsDocument, diag), severity: DiagnosticSeverity.Error,
Change:
Issue Type: <bBug</b<kbdTab</kbdto move focus to status bar. right-arrow to move focus to Settings Profiles item. spacebar to click the item (instead of Enter). :bug: quickpick opens pre-seeded with a single space character, causing the placeholder text not to appear. VS Code version: Code - Insiders 1.69.0-insider (, 2022-07-01T05:17:04.343Z) OS version: Windows_NT x64 10.0. Restricted Mode: No <!-- generated by issue reporter --
/assign
This seems to be coming from status bar as the status bar triggers the command.
<!-- AUTHORVERIFICATIONREQUEST --This bug has been fixed in the latest release of ! you can help us out by commenting if things are now working as expected. If things still don't seem right, please ensure you're on version of Insiders (today's or later - you can use in the command palette to check), and leave a comment letting us know what isn't working as expected. Happy Coding!
/verified

Discussion:
import { WorkbenchActionExecutedEvent, WorkbenchActionExecutedClassification } from 'vs/base/common/actions'; import { IThemeService, ThemeColor } from 'vs/platform/theme/common/themeService'; import { isThemeColor } from 'vs/editor/common/editorCommon'; <del> import { addDisposableListener, EventType, hide, show, append } from 'vs/base/browser/dom'; </del> <ins> import { addDisposableListener, EventType, hide, show, append, EventHelper } from 'vs/base/browser/dom'; </ins> import { INotificationService } from 'vs/platform/notification/common/notification'; import { assertIsDefined } from 'vs/base/common/types'; import { Command } from 'vs/editor/common/languages';

this.commandKeyboardListener.value = addDisposableListener(this.labelContainer, EventType.KEY_DOWN, e => { const event = new StandardKeyboardEvent(e); if (event.equals(KeyCode.Space) || event.equals(KeyCode.Enter)) { <ins> EventHelper.stop(e); </ins> this.executeCommand(command); } });
Change:
Another instance:
I have logging to see with what values the test fails - the test input is randomly generated. I ran it times locally and didn't see a failure... I you come across this then please point to the failing build
Something is weird: <img width="1000" alt="image" src="https://user-"This looks somewhat like Why is there no stack trace on what the assertion is that failed? And why does the task timeout? This is also Windows, so maybe its platform specific and an issue with the unit test runner?
link to CI run?
oh I was just clicking on the first one from this issue:
Oh, super old one without my logging yet
This didn't happen again

Discussion:
for (const item of keys) { tst.set(item, true); <del> assert.ok(tst._isBalanced()); </del> <ins> assert.ok(tst._isBalanced(), `SET${item}|${keys.map(String).join()}`); </ins> } for (const item of keys) { tst.delete(item); <del> assert.ok(tst._isBalanced()); </del> <ins> assert.ok(tst._isBalanced(), `DEL${item}|${keys.map(String).join()}`); </ins> } } });
Change:
<!-- Do Not Delete This! bugreporttemplate --<!-- Please read our Rules of Conduct: --<!-- Read our guide about submitting issues: --<!-- Search existing issues to avoid creating duplicates. --<!-- Test using the latest Insiders build to see if your issue has already been fixed: --<!-- Instead of creating your report here, use 'Report Issue' from the 'Help' menu in VS Code to pre-fill useful information. --<!-- Launch with to check. --Does this issue occur when all extensions are disabled?: Yes/No <!-- If you answered No above, use 'Help: Start Extension Bisect' from Command Palette to try to identify the cause. --<!-- Issues caused by an extension need to be reported directly to the extension publisher. The 'Help Report Issue' dialog can assist with this. --- VS Code Version: from commit to the latest version OS Version: win 10 Steps to Reproduce: Actually I'm not sure if it's been designed to be so but the logic is unresonable. In file srcvsplatforminstantiationcommon there's a code block as below ! It seems will always output but not detailed message. Not sure if you guys have already noticed this and if it's designed to be so.
Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.68.1. Please try upgrading to the latest version and checking whether this issue remains. Happy Coding!

Discussion:
private _throwIfStrict(msg: string, printWarning: boolean): void { if (printWarning) { <del> console.warn(printWarning); </del> <ins> console.warn(msg); </ins> } if (this._strict) { throw new Error(msg);
Change:
Please could someone provide a summary of what this bug is and why the color white cannot be used? (I'm wondering if using it will cause problems in our project) I'm unable to view the link in the comment as I don't have access to the project: Screenshot: ! Many thanks
Do you remember more about
I believe that was some IE limitation from back in the days and not needed anymore. Adding for a trip down memory-lane. do answer you question: No, using white shouldn't cause any issue
Yes, this was a workaround for an old IE version. We can remove this safely.

Discussion:
/** * Editor background color. <del> * Because of bug https://monacotools.visualstudio.com/DefaultCollection/Monaco/_workitems/edit/13254 * we are *not* using the color white (or #ffffff, rgba(255,255,255)) but something very close to white. </del> */ <del> export const editorBackground = registerColor('editor.background', { light: '#fffffe', dark: '#1E1E1E', hcDark: Color.black, hcLight: Color.white }, nls.localize('editorBackground', "Editor background color.")); </del> <ins> export const editorBackground = registerColor('editor.background', { light: '#ffffff', dark: '#1E1E1E', hcDark: Color.black, hcLight: Color.white }, nls.localize('editorBackground', "Editor background color.")); </ins> /** * Editor foreground color.
Change:
Type: <bBug</bDev Container with default or the prefered profile a new profile from Current Profile Settings After reloading the window you get a message to install the remote containers extension and when opening the profile in a local workspace you can see that no extensions and no settings where copied from the old profile. VS Code version: Code - Insiders 1.70.0-insider (, 2022-07-08T05:20:13.606Z) OS version: WindowsNT x64 10.0. Modes: Restricted Connection to 'vscode-remote' could not be established No remote extension installed to resolve dev-container. <details<summarySystem Info</summary Connection to 'vscode-remote' could not be established No remote extension installed to resolve dev-container. </detailsExtensions: none <!-- generated by issue reporter --

Discussion:
} private async enterProfile(profile: IUserDataProfile, preserveData: boolean, reloadMessage?: string): Promise<void> { <del> if (this.environmentService.remoteAuthority) { </del> <ins> const isRemoteWindow = !!this.environmentService.remoteAuthority; if (!isRemoteWindow) { this.extensionService.stopExtensionHosts(); } // In a remote window update current profile before reloading so that data is preserved from current profile if asked to preserve await this.userDataProfileService.updateCurrentProfile(profile, preserveData); if (isRemoteWindow) { </ins> const result = await this.dialogService.confirm({ type: 'info', message: reloadMessage ?? localize('reload message', "Switching a settings profile requires reloading VS Code."),

if (result.confirmed) { await this.hostService.reload(); } <del> return; </del> <ins> } else { await this.extensionService.startExtensionHosts(); </ins> } <del> this.extensionService.stopExtensionHosts(); await this.userDataProfileService.updateCurrentProfile(profile, preserveData); await this.extensionService.startExtensionHosts(); </del> } }
Change:
<!-- 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. --

Discussion:
import { AuthenticationSession, AuthenticationSessionsChangeEvent, IAuthenticationService } from 'vs/workbench/services/authentication/common/authentication'; import { IExtensionService } from 'vs/workbench/services/extensions/common/extensions'; import { EDIT_SESSIONS_SIGNED_IN, EditSession, EDIT_SESSION_SYNC_CATEGORY, IEditSessionsWorkbenchService, EDIT_SESSIONS_SIGNED_IN_KEY, IEditSessionsLogService } from 'vs/workbench/contrib/editSessions/common/editSessions'; <ins> import { IDialogService } from 'vs/platform/dialogs/common/dialogs'; </ins> import { generateUuid } from 'vs/base/common/uuid'; type ExistingSession = IQuickPickItem & { session: AuthenticationSession & { providerId: string } };

@IProductService private readonly productService: IProductService, @IContextKeyService private readonly contextKeyService: IContextKeyService, @IRequestService private readonly requestService: IRequestService, <ins> @IDialogService private readonly dialogService: IDialogService, </ins> ) { super();

}); } <del> run() { that.clearAuthenticationPreference(); </del> <ins> async run() { const result = await that.dialogService.confirm({ type: 'info', message: localize('sign out of edit sessions clear data prompt', 'Do you want to sign out of edit sessions?'), checkbox: { label: localize('delete all edit sessions', 'Delete all stored edit sessions from the cloud.') }, primaryButton: localize('clear data confirm', 'Yes'), }); if (result.confirmed) { if (result.checkboxChecked) { that.storeClient?.delete('editSessions', null); } that.clearAuthenticationPreference(); } </ins> } })); }
Change:
Pre-req: Settings Sync on brand new empty profile a theme Settings sync asks this strange question: ! Additionally: this dialog Theme is still applied but not installed
I tried and can't repro. Are there any specific steps you followed?
were there specific steps to repro this?
I can verify that cancelling the dialog no longer leaves the theme applied, thanks I can still reproduce the main issue though. Why would I not want to synchronize the theme I just picked for my brand new profile, given the profile is being synced?
I think is using the general extension install command that will ask user to sync or not before installing. If it is in core, one can use the API directly to install.
I think we still want an install confirmation dialog, but I agree that there should be no question about syncing.

Discussion:
const success = await this.installExtension(themeItem.galleryExtension); if (success) { selectTheme(themeItem.theme, true); <ins> } else { selectTheme(currentTheme, true); </ins> } } });

} } }); <del> quickpick.onDidChangeActive(themes => selectTheme(themes[0]?.theme, false)); </del> <ins> quickpick.onDidChangeActive(themes => { if (result === undefined) { selectTheme(themes[0]?.theme, false); } }); </ins> quickpick.onDidHide(() => { if (result === undefined) {
Change:
From : Error: TreeError [FileExplorer] Data tree node not found: ExplorerItem:
Any idea here because there's no explorer stack involved this might be a tree issue
You can try this. Enable fileNesting and select a file with nesting, control+c, the error would appear. The chosen file should be in preview.
third line in the stack frame:
This also happens when you try to open a file with enter or space keys. ! Pressing delete instead of enter will display the same error. (related issue: ) Once the nested files are expanded, this error does not occur until the next window reload, perhaps because the node is to the tree.
Thanks for the repro

Discussion:
const items = new Set<ExplorerItem>(this.view.getContext(respectMultiSelection)); items.forEach(item => { <del> if (respectMultiSelection && this.view?.isItemCollapsed(item) && item.nestedChildren) { for (const child of item.nestedChildren) { items.add(child); </del> <ins> try { if (respectMultiSelection && this.view?.isItemCollapsed(item) && item.nestedChildren) { for (const child of item.nestedChildren) { items.add(child); } </ins> } <ins> } catch { // We will error out trying to resolve collapsed nodes that have not yet been resolved. // So we catch and ignore them in the multiSelect context return; </ins> } });
Change:
Separator background is incorrect Separator cursor is incorrect <img width="620" alt="image" src="https://user-"

Discussion:
cursor: pointer; } <ins> .monaco-button-dropdown.disabled { cursor: default; } </ins> .monaco-button-dropdown > .monaco-button:focus { outline-offset: -1px !important; } <ins> .monaco-button-dropdown.disabled .monaco-button-dropdown-separator { opacity: 0.4; } </ins> .monaco-button-dropdown .monaco-button-dropdown-separator { padding: 4px 0; }

set enabled(enabled: boolean) { this.button.enabled = enabled; this.dropdownButton.enabled = enabled; <ins> this.element.classList.toggle('disabled', !enabled); </ins> } get enabled(): boolean {
Change:
<!-- Do Not Delete This! bugreporttemplate --<!-- Please read our Rules of Conduct: --<!-- Read our guide about submitting issues: --<!-- Search existing issues to avoid creating duplicates. --<!-- Test using the latest Insiders build to see if your issue has already been fixed: --<!-- Instead of creating your report here, use 'Report Issue' from the 'Help' menu in VS Code to pre-fill useful information. --<!-- Launch with to check. --Does this issue occur when all extensions are disabled?: Yes <!-- If you answered No above, use 'Help: Start Extension Bisect' from Command Palette to try to identify the cause. --<!-- Issues caused by an extension need to be reported directly to the extension publisher. The 'Help Report Issue' dialog can assist with this. --- VS Code Version: 1.70.0-insider (, 2022-07-11T12:20:21.684Z) OS Version: Windows_NT x64 10.0. Suppose we have this text: It can't be matched with original text using regex:  But if we join the first line and second line with a , it can be matched: 
For some background info, see and

Discussion:
for (let i = 0, len = searchString.length; i < len; i++) { const chCode = searchString.charCodeAt(i); <ins> if (chCode === CharCode.LineFeed) { return true; } </ins> if (chCode === CharCode.Backslash) { // move to next char

assert(isMultilineRegexSource('foorn')); assert(isMultilineRegexSource('n')); assert(isMultilineRegexSource('fooW')); <ins> assert(isMultilineRegexSource('foon')); assert(isMultilineRegexSource('foorn')); </ins> }); test('issue #74715. d* finds empty string and stops searching.', () => {
Change:
<!-- Do Not Delete This! bugreporttemplate --<!-- Please read our Rules of Conduct: --<!-- Read our guide about submitting issues: --<!-- Search existing issues to avoid creating duplicates. --<!-- Test using the latest Insiders build to see if your issue has already been fixed: --<!-- Instead of creating your report here, use 'Report Issue' from the 'Help' menu in VS Code to pre-fill useful information. --<!-- Launch with to check. --Does this issue occur when all extensions are disabled?: Yes/No <!-- If you answered No above, use 'Help: Start Extension Bisect' from Command Palette to try to identify the cause. --<!-- Issues caused by an extension need to be reported directly to the extension publisher. The 'Help Report Issue' dialog can assist with this. --- VS Code Version: 1.69.0 (Universal) OS Version: Darwin arm64 21.5.0 (macOS 12.4) Steps to Reproduce: a ".md" file command "Markdown: Open Preview" an blank window I've tried following methods, but the issues persist: VSC via "code --no-sandbox" VSC via "code --disable-extensions" a "Visual Studio Code - " (use it for the first time) Below is my VSC <img width="1800" alt="image" src="https://user-"Below is my newly downloaded VSC-insiders <img width="1800" alt="image" src="https://user-"
Do the release notes and other webview still work?
Release notes, as well as other webview, do not work. They just show a blank window. <img width="1444" alt="image" src="https://user-"
Any further suggestions on solving this problem?
If you run , you see any errors in the developer console?
This issue has been closed automatically because it needs more information and has not had recent activity. See also our guidelines. Happy Coding!
It seems that there isn't any helpful information. <img width="1444" alt="image" src="https://user-"
It seems that I have found the cause of this bug: something to do with the "%" in the app path. When path is "/Applications/Visual Studio ": <img width="1105" alt="Snipaste2022-08-2911-53-27" src="https://user-"- When path is "/Applications/%IDEs/Visual Studio ": <img width="1146" alt="Snipaste2022-08-2911-54-56" src="https://user-"
<!-- AUTHORVERIFICATIONREQUEST --This bug has been fixed in the latest release of ! you can help us out by commenting if things are now working as expected. If things still don't seem right, please ensure you're on version of Insiders (today's or later - you can use in the command palette to check), and leave a comment letting us know what isn't working as expected. Happy Coding!
/verified

Discussion:
const relativeResourcePath: AppResourcePath = `vs/workbench/contrib/webview/browser/pre/${entry}`; const url = FileAccess.asFileUri(relativeResourcePath); return callback({ <del> path: decodeURIComponent(url.fsPath), </del> <ins> path: url.fsPath, </ins> headers: { ...COI.getHeadersFromQuery(request.url), 'Cross-Origin-Resource-Policy': 'cross-origin'
Change:
I've noticed that the notebook toolbar has a lot of actions that can make it hard to scan. I also noticed that some actions appear to be disabled when not available (like the interrupt action). Here are some minor proposals. <img width="985" alt="CleanShot 2022-07-12 at 14 04 05 src="https://user-"<img width="987" alt="CleanShot 2022-07-12 at 14 05 11 src="https://user-""Clear Outputs of All Cells" to "Clear All Outputs" the "Interrupt" action when not available
both makes sense to me. "Clear All Outputs" is much easier to read. A disabled action on the toolbar is not great, since users can't really do anything about it. Not sure if we hide it or merge it with "Restart" (making it a dropdown like what we did for the Run button on cells)
Related to this is sticking to the principle of only showing actions when they are available instead of showing them as enabled/disabled

Discussion:
constructor() { super({ id: CLEAR_ALL_CELLS_OUTPUTS_COMMAND_ID, <del> title: localize('clearAllCellsOutputs', 'Clear Outputs of All Cells'), </del> <ins> title: localize('clearAllCellsOutputs', 'Clear All Outputs'), </ins> precondition: NOTEBOOK_HAS_OUTPUTS, menu: [ {
Change:
Ctrl clicking an error from our selfhost build no longer sends me to the right location of the file. It simply opens the file and highlights position (0,0). !
Pretty sure i've seen this too

Discussion:
'((S*)['"], line ((d+)( column (d+))?))', // "(file path)", line 45 [see #40468] '((S*)['"],((d+)(:(d+))?))', // "(file path)",45 [see #78205] '((S*) on line ((d+)(, column (d+))?))', // (file path) on line 8, column 13 <del> '((S*):s?line ((d+)(, col(umn)? (d+))?))', // (file path):line 8, column 13, (file path): line 8, col 13 </del> <ins> '((S*):s?line ((d+)(, col(?:umn)? (d+))?))', // (file path):line 8, column 13, (file path): line 8, col 13 </ins> '(([^s()]*)(s?[([](d+)(,s?(d+))?)[)]])', // (file path)(45), (file path) (45), (file path)(45,18), (file path) (45,18), (file path)(45, 18), (file path) (45, 18), also with [] '(([^:s()<>'"[]]*)(:(d+))?(:(d+))?)' // (file path):336, (file path):336:9 ].join('|').replace(/ /g, `[${'u00A0'} ]`);
Change:
deferring from as I spent more than an hour on it
This is a little different to how we do it in the terminal because the command palette entry: is registered separately to the command: also doesn't show up at all as a command anymore: Insiders: ! Stable: !
Commenting out the schema fixes the problem for runTask and the commands below.
here needs workspace trust:
removed because the change was reverted so it doesn't work

Discussion:
properties: { type: { type: 'string', <del> description: nls.localize('runTask.type', "The contributed task type"), enum: Array.from(this._providerTypes.values()).map(provider => provider) </del> <ins> description: nls.localize('runTask.type', "The contributed task type") </ins> }, taskName: { type: 'string', <del> description: nls.localize('runTask.taskName', "The task's label or a term to filter by"), enum: await this.tasks().then((tasks) => tasks.map(t => t._label)) </del> <ins> description: nls.localize('runTask.taskName', "The task's label or a term to filter by") </ins> } } }
Change:
Issue Type: <bBug</b#Steps Go to preferences Enter a zoom value too small (I entered -140) #Result Everything dissapears and clicking on the window minimizes it. CTRL + "+" doesn't work #Workaround Go to under AppData/Roaming/Code and change the zoom value in the file to normal levels VS Code version: Code 1.69.1 (, 2022-07-12T08:21:24.514Z) OS version: WindowsNT x64 10.0. Restricted Mode: No Remote OS version: Linux x64 5.10.16.3-microsoft-standard-WSL2 <details<summarySystem Info</summary </details<details<summaryExtensions (11)</summaryExtensionVersion ------ better-cpp-syntax1.15.19 jupyter-keymap1.0.0 remote-containers0.241.3 remote-ssh0.84.0 remote-ssh-edit0.80.0 remote-wsl0.66.3 vscode-intelephense-client1.8.2 prettier-vscode9.5.0 cpptools1.10.8 save-as-root1.4.2 markdown-all-in-one3.4.3 (2 theme extensions excluded) </details<details<summaryA/B Experiments</summary</details<!-- generated by issue reporter --

Discussion:
'window.zoomLevel': { 'type': 'number', 'default': 0, <ins> 'minimum': -5, </ins> 'description': localize('zoomLevel', "Adjust the zoom level of the window. The original size is 0 and each increment above (e.g. 1) or below (e.g. -1) represents zooming 20% larger or smaller. You can also enter decimals to adjust the zoom level with a finer granularity."), ignoreSync: true, tags: ['accessibility']