jiebi/CodeConvo huggingface dataset:
Change:
if (name === 'RANDOM') { return Math.random().toString().slice(-6); <del> } else if (name === 'RANDOM_HEX') { </del> <ins> } else if (name === 'RANDOM_HEX') { </ins> return Math.random().toString(16).slice(-6); <ins> } else if (name === 'UUID') { return generateUuid(); </ins> } return undefined;

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. --Vscode snippets support - e.g. variables related to filepaths, timestamps etc. Could you please add a new variable called something like that will auto-generate a version 4 UUID? (random) e.g. When executing the snippet, the placeholder would be replaced with a value like:
, that's a wrap! Now put it on the backlog, and in the software :D
could the generateUuid function be used with a snippet variable? I don't know how different the uuid and uuidv4 are, or if that matters here. I can submit a PR if that function works, or if you have any input on how to go about this.
Yeah, that function should be OK to be used
To verify make sure that you can have a snippet like this:
Change:
return; } <del> if (this.userDataProfileService.currentProfile.icon && this.userDataProfileService.currentProfile.icon !== DEFAULT_ICON.id) { this.profileBadgeContent.classList.toggle('profile-icon-overlay', true); this.profileBadgeContent.classList.toggle('profile-text-overlay', false); append(this.profileBadgeContent, $(ThemeIcon.asCSSSelector(DEFAULT_ICON))); } else { </del> <ins> if (!this.userDataProfileService.currentProfile.icon || this.userDataProfileService.currentProfile.icon === DEFAULT_ICON.id) { </ins> this.profileBadgeContent.classList.toggle('profile-text-overlay', true); this.profileBadgeContent.classList.toggle('profile-icon-overlay', false); this.profileBadgeContent.textContent = this.userDataProfileService.currentProfile.name.substring(0, 2).toUpperCase();

Discussion:
I wonder if we should not overlay here: ! My understanding is that this is the only place where we show the profile icon but then its always overlayed with the gear which is quite large. I think people can make the connection that the location of the button is for settings.
I tend to agree that the overlay is superfluous, but is an underlay possible as an alternative to be tried?
We are trying to come up with a smaller gear icon for overlay ! ! I am preferring to have gear overlay icon to indicate it is for settings, this will be helpful when there are duplicates like extensions profile icon.
I am thinking to undo the overlay until we have better overlay icon.
At present we do not overlay and show the profile icon as is. Hence closing this.
Change:
const progressAmount = assertIsDefined(progress.querySelector('.progress-bar-inner') as HTMLDivElement).style.width; if (!toFocus && progressAmount !== '100%') { toFocus = assertIsDefined(progress.parentElement?.parentElement); } }); <del> (toFocus ?? assertIsDefined(this.container.querySelector('button.skip')) as HTMLButtonElement).focus(); </del> <ins> (toFocus ?? assertIsDefined(this.container.querySelector('button.getting-started-category')) as HTMLButtonElement)?.focus(); </ins> } private setSlide(toEnable: 'details' | 'categories') {

Discussion:
<!-- 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. --- VS Code Version: 1.55.0-insider Steps to Reproduce: all the actions in the 2 getting started tiles all items are complete, go back and try opening them again. Unable to reopen them once complete. ! <!-- 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 direct to the extension publisher. The 'Help Report Issue' dialog can assist with this. --
Good catch thanks!
Change:
license.style.display = 'none'; license.tabIndex = 0; <ins> const version = append(subtitle, $('span.version')); version.textContent = localize('version', 'Version'); version.style.display = 'none'; version.tabIndex = 0; </ins> const description = append(details, $('.description')); const extensionActions = append(details, $('.actions'));

Discussion:
Version: 1.44.0-insider Commit: Date: 2020-03-31T08:28:37.335Z The extensions view header has all the important information about an extension, but not the installed or latest version number. I often need this when filing issues to a extension. !
Change:
import * as vscode from 'vscode'; <del> const knownSchemes = ['http:', 'https:', 'file:', 'mailto:', 'data:', 'vscode-resource:']; </del> <ins> const knownSchemes = ['http:', 'https:', 'file:', 'mailto:', 'data:', 'vscode:', 'vscode-insiders:', 'vscode-resource:']; </ins> export function getUriForLinkWithKnownExternalScheme( link: string,

Discussion:
Issue Type: <bBug</bCreate a Markdown file that contains a link with the URI scheme: Expected link location to be , and displayed as . Actual link location , and displayed as . Add to array in . VS Code version: Code 1.32.3 (, 2019-03-14T22:46:48.547Z) OS version: Darwin x64 18.0.0 <details<summarySystem Info</summary </details<details<summaryExtensions (34)</summaryExtensionVersion ------ rtf2.0.1 solargraph0.19.3 vscode-modelines2.0.2 npm-intellisense1.3.0 vscode-npm-script0.3.5 vscode-diff1.4.0 vscode-git-history2.0.5 beautify1.4.11 vscode-duplicate1.2.1 python2019.3.6139 cpptools0.22.1 csharp1.18.0 mono-debug0.15.8 incrementor0.1.0 printcode3.0.0 openhab0.4.1 vscode-docker0.6.1 platformio-ide1.6.0 diff0.0.1 ruby0.22.3 java0.41.0 LiveServer5.5.1 partial-diff1.4.0 code-settings-sync3.2.7 code-spell-checker1.6.10 unity-debug2.7.2 vscodeintellicode1.1.5 vscode-arduino0.2.25 vscode-java-debug0.17.0 vscode-java-dependency0.4.0 vscode-java-pack0.6.0 vscode-java-test0.15.1 vscode-maven0.15.2 markdown-all-in-one2.2.0 (1 theme extensions excluded) </details<!-- generated by issue reporter --
Change:
const { uri, isDirectory } = result; const linkToOpen = { // Use the absolute URI's path here so the optional line/col get detected <del> text: result.uri.fsPath + (text.match(/:d+(:d+)?$/)?.[0] || ''), </del> <ins> text: result.uri.path + (text.match(/:d+(:d+)?$/)?.[0] || ''), </ins> uri, bufferRange: link.bufferRange, type: link.type

Discussion:
My project is under https://user- Reproduce steps: vscode open a folder which file path contains space, for example: an integrated terminal the path link Versions:
nice I can repro with the space in the name, thanks for following up
I can still repro with Windows.
I looked into this, and it will be tricky because we currently use space as one of the characters which indicates the end of a path. One idea would be to check if such a link exists (using our original regex) and if not, use regex on the subsequent sequence - then concatenating the two results for First result would be doesn't exist process exists
https://user- Look the above video, I have several questions. path is not a existed path in the disk, but also clickable. I click the , it open a file which is not why the whole string is recogonized to a path? the real path is:
this is actually about when the parents of the link have a space, it's expected currently that an absolute path with a space won't work.
is it expected that a local link with a space somewhere within the path is not detected? that is the case currently - verified via adding a test to
absolute paths (eg. ) won't work because of Relative paths without spaces with absolute ancestors that have spaces (eg. inside ) should work but don't in some cases.
When verifying, make sure to test both on a Windows machine and a non-Windows machine.
Doesn't work for me on a Mac, I guess thats probably because I still haven't got the fix in the latest insider build.
are you on ?
Worked for me on macOS. Don't have a Windows machine to try this out so someone else will need to verify on Windows.
Yes, I'm on that same version. Here's what it looks like for me on a Mac with spaces !
still not work in latest version: https://user-
For absolute paths, see I wasn't able to get to work, though. It'd open the file, but place the cursor at the beginning.
please provide some more details, it works for me: !
This is the behaviour with version 1.70.0 After ctrl+clik a popup appears at the top of the screen and I have to hit enter to go to the suggested file:line.
I can repro that on Windows, Remote-WSL and macOS.

The file must not be in the root directory for this to happen. !
let's track that problem in and as this issue is becoming quite unwieldy/unfocused.
created to track the problem related to quotes you're seeing. Could you re-verify this if all good outside of these 3 issues I created? It's difficult to manage this issue as it's so general.
Change:
this.input.dispose(); } super.clearInput(); <ins> this.resourceContext.reset(); </ins> } protected override createEditor(parent: HTMLElement): void {

Discussion:
! Observed by
Do you have special steps for this? Do you see this with all output channels or only the MySQL one? Can you do the following when this happens: F1 Inspect Context Keys and then share what's printed on dev tools console
No specific repro steps - this happened immediately after upgrading in a test session - will try to repo on desktop doing it and report back!
Hi there, I think I received this email erroneously? I'm Nick Carver, but it looks like the original msg was from a Nick Craver. Cheers, Nick.
any new insights here?
This issue has been closed automatically because it needs more information and has not had recent activity. See also our guidelines. Happy Coding!
Apologies I was on vacation for a bit - haven't been able to repro this since the latest update landed (but also have to unfortunately rebuild the machine we saw it on for terribad Windows reasons). Recommend we close this and if I can repro, will poke with more info :)
Leaving this open since saw this once but forgot to inspect context keys ()
This issue has been closed automatically because it needs more information and has not had recent activity. See also our guidelines. Happy Coding!
steps: empty profile have a file under merge-conflict open the output view open the merge editor for the conflict file :bug: output view shows the merge overlay button
Change:
private dimension?: dom.Dimension; private splitView!: SplitView; private contentProviders!: IPeekOutputRenderer[]; <ins> private contentProvidersUpdateLimiter = this._register(new Limiter(1)); </ins> public current?: InspectSubject;

Discussion:
Type: Bug Version: 1.81.0 Commit: Date: 2023-08-02T12:36:11.334Z Electron: 22.3.18 ElectronBuildId: Chromium: 108.0.5359.215 16.17.1 V8: 10.8.168.25-electron.0 OS: Linux x64 5.15.0-76-generic Steps to Reproduce: the "Test Results" Panel command "Test: Run Tests in Current File" very quickly for several times "Test Results" Panel now displays two terminals, with the top one acting like phantom, and the bottom one displaying the real messages. After some investigation, I found the culprit to be that is not concurrently safe. Under certain cases, multiple calls to may happen concurrently within a short time period, which further leads to concurrent calls to , in which multiple instances of terminal are created and attached to the panel. This bug was largely improved by commit introduced by , which changed into This change reduces the possibility of concurrently calling to some extent and thus this bug may be harder to reproduce in the latest main branch. That being said, there's still a chance for this to happen. To thoroughly eliminate the risk, I fix it in PR .
Change:
assert.strictEqual((first.range as any).replace.endColumn, 6); model.dispose(); }); <ins> test('Leading whitespace in snippet prefix #123860', async function () { snippetService = new SimpleSnippetService([new Snippet( ['fooLang'], 'cite-name', ' cite', '', '~cite{$CLIPBOARD}', '', SnippetSource.User )]); const provider = new SnippetCompletionProvider(modeService, snippetService); let model = createTextModel(' ci', undefined, 'fooLang'); let result = await provider.provideCompletionItems(model, new Position(1, 4), context)!; assert.strictEqual(result.suggestions.length, 1); let [first] = result.suggestions; assert.strictEqual((<CompletionItemLabel>first.label).label, ' cite'); assert.strictEqual((<CompletionItemRanges>first.range).insert.startColumn, 1); model.dispose(); }); </ins> });

Discussion:
Issue Type: <bBug</bI would like the prefix to trigger only when there is a leading whitespace in front of the prefix (it is supposed to be substituted with an explicit non-breaking space character ). The example of the snippet: --The prefix behaves as if there is no leading space, meaning the remains one extra whitespace before the expanded body, resulting in a double space: . VS Code version: Code 1.55.2 (, 2021-04-13T09:35:57.887Z) OS version: WindowsNT x64 10.0. <details<summarySystem Info</summary </details<details<summaryExtensions (37)</summaryExtensionVersion ------ html-snippets0.2.1 Handlebars0.4.1 rc-beautify1.2.0 git-temporal-vscode1.0.0 insert-unicode0.10.0 i30.0.1 vscode-generate-getter-setter0.5.0 gitlens11.4.1 vscode-html-css1.10.2 vscode-npm-script0.3.21 carbon-now-sh1.2.0 vscode-angular-html1.7.0 beautify1.5.0 latex-workshop8.18.0 polacode-.5.2 svg1.4.6 Angular211.0.0 solidity0.0.113 Kotlin1.7.1 rainbow-csv1.8.1 vscode-docker1.12.1 python2021.5. vscode-pylance2021.5.2 jupyter2021.5. remote-containers0.166.1 postscript1.0.1 java0.79.1 LiveServer5.6.1 bash-debug0.3.9 scala0.5.3 bash-beautify0.1.1 code-settings-sync3.4.3 code-spell-checker1.10.2 sass-indented1.8.16 html-preview-vscode0.2.5 vscode-ruby0.28.0 markdown-all-in-one3.4.0 (2 theme extensions excluded) </details<details<summaryA/B Experiments</summary</details<!-- generated by issue reporter --
I have a fix for this but it makes bigger changes in how snippet completion are computed. Pushing out to November to get more Insider usage for it, changes are in
Change:
this.updateDynamicCSSRules(themeData); this.applyTheme(themeData, null, true); <ins> let iconData: FileIconThemeData = null; let persistedIconThemeData = this.storageService.get(PERSISTED_ICON_THEME_STORAGE_KEY); if (persistedIconThemeData) { iconData = FileIconThemeData.fromStorageData(persistedIconThemeData); if (iconData) { _applyIconTheme(iconData, () => { this.doSetFileIconTheme(iconData); return TPromise.wrap(iconData); }); } } </ins> this.initialize().then(null, errors.onUnexpectedError).then(_ => { this.installConfigurationListener(); });

Discussion:
It would be nice if icons would show up directly after starting the workbench instead of having to wait for the extension host to be ready. Similar to color themes where the data is stored in local storage, the icons data should also be stored so that the overall experience is smooth.
can we consider this for this milestone? I want to mark this "perf-startup" because the fact that the icons only "kick in" later after the UI is already done makes the entire editor appear like a website that needs to load icons later separately which is ugly. It also causes some ugly jumping for opened tabs. ! /cc
PR:
Change:
&& this.hideCursor === other.hideCursor && this.cursorColor === other.cursorColor && this.themeType === other.themeType <del> && this.backgroundColor === other.backgroundColor </del> <ins> && Color.equals(this.backgroundColor, other.backgroundColor) </ins> && this.top === other.top && this.right === other.right && this.domWidth === other.domWidth

Discussion:
Ruler background is set to black with 50% transparency, but vscode shows solid black! ! ! Description for editorOverviewRuler.background also says: , but vscode use this background if minimap is enabled or not!
The color description is out of date. We changed the behavior of via so we now always respect . However, it never supported transparency, so we could add support for that.
Change:
position: absolute; width: 600px; z-index: 2550; <del> padding: 0 1px 1px 1px; </del> left: 50%; margin-left: -300px; -webkit-app-region: no-drag;

Discussion:
! Not sure if this is recent, but shouldn't the title background go all the way?
It should. I didn't change anything recently but I'll take a look
verification (Using tomorrow's build): Using GHPR extension open a PR webview Select the milestone using the default theme you should be able to not see any border on the title bar verification that this didn't break Turn off settings sync if you have it on Turn on settings sync to see: ! Tab to select a Quick pick item row Click with your mouse on the "Please sign in to synchronize..." text which will cause the focus to be entirely around the qp (like in the above screenshot) verify there is still a solid border around the entire qp.
Change:
} return undefined; <del> } </del>  No newline at end of file <ins> } </ins>

Discussion:
Issue Type: <bBug</bCreate a Markdown file that contains a link with the URI scheme: Expected link location to be , and displayed as . Actual link location , and displayed as . Add to array in . VS Code version: Code 1.32.3 (, 2019-03-14T22:46:48.547Z) OS version: Darwin x64 18.0.0 <details<summarySystem Info</summary </details<details<summaryExtensions (34)</summaryExtensionVersion ------ rtf2.0.1 solargraph0.19.3 vscode-modelines2.0.2 npm-intellisense1.3.0 vscode-npm-script0.3.5 vscode-diff1.4.0 vscode-git-history2.0.5 beautify1.4.11 vscode-duplicate1.2.1 python2019.3.6139 cpptools0.22.1 csharp1.18.0 mono-debug0.15.8 incrementor0.1.0 printcode3.0.0 openhab0.4.1 vscode-docker0.6.1 platformio-ide1.6.0 diff0.0.1 ruby0.22.3 java0.41.0 LiveServer5.5.1 partial-diff1.4.0 code-settings-sync3.2.7 code-spell-checker1.6.10 unity-debug2.7.2 vscodeintellicode1.1.5 vscode-arduino0.2.25 vscode-java-debug0.17.0 vscode-java-dependency0.4.0 vscode-java-pack0.6.0 vscode-java-test0.15.1 vscode-maven0.15.2 markdown-all-in-one2.2.0 (1 theme extensions excluded) </details<!-- generated by issue reporter --
Change:
resolved "https://registry.yarnpkg.com/xterm-addon-unicode11/-/xterm-addon-unicode11-0.4.0-beta.5.tgz#3900e66f10d2e506133b61d7421aab6878d32665" integrity sha512-+g+fuxAd/tkCEJ/jhdnebXKtdPrhsu4VKWNnB/3qM35GbuGQOasmYFYnKL+HYZMpbQ6YqeZcXTVg/wnCTttz0g== <del> xterm-addon-webgl@0.13.0-beta.35: version "0.13.0-beta.35" resolved "https://registry.yarnpkg.com/xterm-addon-webgl/-/xterm-addon-webgl-0.13.0-beta.35.tgz#98b5dc102583120de25c7c6e0f35cd34ab6bef98" integrity sha512-powgeb3ifEZK7zWwLJuE8YRz/Z0UzDrrVA9NTx9mH7+vWxrGt9ZgroychoeCqZBvKMofiUM5vlf1oWdSmsMfzw== </del> <ins> xterm-addon-webgl@0.13.0-beta.36: version "0.13.0-beta.36" resolved "https://registry.yarnpkg.com/xterm-addon-webgl/-/xterm-addon-webgl-0.13.0-beta.36.tgz#94c0f05a1be7fd027e7d5367a623969a656fb97a" integrity sha512-HG86+KCYHNf7Pi9/ksRU9LLrMcfex1e2D6rU1NCK/p2hEiQqTALX2QnLf7BoO0Yl97671GllZvVSZvhA7/bwrA== </ins> xterm@5.0.0-beta.35: version "5.0.0-beta.35"

Discussion:
I've been seeing a lot of this since last week. I assume there's another issue for it but I don't see it !
! Is this related? It keeps happening too
that's because of our improved underlining, it doesn't look at good on your font because it's drawn a bit lower in the cell though. What font is it?
Cascadia Code
With the default, still a bit off <img width="179" alt="image" src="https://user-"
thanks, that's macOS with zoom level 0?
Yeah
Default font on macOS (Menlo): <img width="228" alt="Screen Shot 2022-08-19 at 1 04 24 pm" src="https://user-"
Font size 12: <img width="148" alt="Screen Shot 2022-08-19 at 1 06 40 pm" src="https://user-"Font size 14: <img width="167" alt="Screen Shot 2022-08-19 at 1 07 06 pm" src="https://user-"Font size 16: <img width="191" alt="Screen Shot 2022-08-19 at 1 07 52 pm" src="https://user-"
Can also happen on dpr 1 (Windows 11, Hack): !
Change:
.forEach(({ id, arrow, label, command, arrowLast }) => { const div = dom.append(this._overlay, $(`.key.${id}`)); if (arrow && !arrowLast) { <del> dom.append(div, $('span.arrow')).innerHTML = arrow; </del> <ins> dom.append(div, $('span.arrow', undefined, arrow)); </ins> } dom.append(div, $('span.label')).textContent = label; if (command) {

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. --The next step in resolving . The Trusted Types violation here is setting an HTMLElement’s text content by assigning it to . Because in all of the cases the HTMLElement is already created with function, the idea is to pass the text as its argument. That is to replace: with .
Change:
resolved "https://registry.yarnpkg.com/xterm-addon-unicode11/-/xterm-addon-unicode11-0.4.0-beta.3.tgz#f350184155fafd5ad0d6fbf31d13e6ca7dea1efa" integrity sha512-FryZAVwbUjKTmwXnm1trch/2XO60F5JsDvOkZhzobV1hm10sFLVuZpFyHXiUx7TFeeFsvNP+S77LAtWoeT5z+Q== <del> xterm-addon-webgl@0.12.0-beta.37: version "0.12.0-beta.37" resolved "https://registry.yarnpkg.com/xterm-addon-webgl/-/xterm-addon-webgl-0.12.0-beta.37.tgz#e465100041a7e0b1d32b01cd0b0ba5a516ac13c2" integrity sha512-d4GfKlMrWZGzsMfMHd2siG+QiOvwikhwiu+JQlIhlnAShT/wU5BOGbNfDDA1tHmkW7G1UCFgucBuVxrs5wjuBQ== xterm@4.19.0-beta.58: version "4.19.0-beta.58" resolved "https://registry.yarnpkg.com/xterm/-/xterm-4.19.0-beta.58.tgz#cbe535b0b85bc7f8ae3cc242df006bea04cdf541" integrity sha512-fr3QC2qS2NBab8X7kwA/3QZELzQqhAKsngvK8NkhsrFN1DAuyEETkzqXuKbO7OKpCB6VqGDN6GPkNlUB9VrIDA== </del> <ins> xterm-addon-webgl@0.12.0-beta.38: version "0.12.0-beta.38" resolved "https://registry.yarnpkg.com/xterm-addon-webgl/-/xterm-addon-webgl-0.12.0-beta.38.tgz#59fc8f3711e38b9a238fd460cd070007b7904e7a" integrity sha512-fqRhfwUx/WZlDLFsZX26irLYTdRGMf1xuRwoqfr4KiFoQvHye55U8VJ4WLHdWglN2Kzh8PbINk5HfMhIwYHe9Q== xterm@4.19.0-beta.60: version "4.19.0-beta.60" resolved "https://registry.yarnpkg.com/xterm/-/xterm-4.19.0-beta.60.tgz#de89d93153ae3ec17b53f3e5d03b8ea859126081" integrity sha512-YQjvp4xtSAyNm9+gF4OA5+QzkSWdKMCy4WABETpe7CsrEGx+mJVoXCZ9wgopmvwRXz7DeJvKM5dWz1HPjVLDtA== </ins>

Discussion:
Upstream issue: Repro: Alt+click and drag in the terminal Moving right on the same line does not make a selection Moving to top-right or bottom-left doesn't make a selection (this works in canvas renderer only?) Cursor isn't correct ! Correct behavior in 1.67: !
When verifying make sure that copy works as expected for column selection in each quadrant as well.
Change:
if (label.icon && ThemeIcon.isThemeIcon(label.icon) && labels.includes(label.title)) { const icon = append(templateData.labelContainer, $('div.label')); icon.classList.add(...ThemeIcon.asClassNameArray(label.icon)); <del> const hover = this.hoverService.setupManagedHover(instantHoverDelegate, icon, label.title); templateData.elementDisposables.add(hover); </del> } } }

Discussion:
Testing I think we have another issue discussing a similar issue. This hover and the main item hover can conflict with each other: <img width="579" alt="Screenshot 2024-08-27 at 10 37 49 AM" src=""Instead of having a special hover just for the badges I think the main hover accomplishes the goal just fine as the badge and its description shows up at the bottom: <img width="698" alt="Screenshot 2024-08-27 at 10 39 05 AM" src=""