jiebi/CodeConvo huggingface dataset:
Change:
/* ----- server cli ----- */ help: boolean; version: boolean; <ins> 'shell-integration'?: string; </ins> compatibility: string;

Discussion:
Currently install is a little cumbersome, for example: We should also make sure these cases are covered: Remote development extensions - injection works currently but manual install doesn't talk about the server paths Regular SSH - it's not so easy to get the shell integration script to the remote machine
Hi, I’m thinking about another way, in addition to the CLI mentioned above: how about creating a package (deb/rpm) with only directory? This way I only need to install this on the server if I want to SSH to it without remote development (assuming that for some reason I don’t want remote development), since I don’t need the other part of VS Code. This can even go into Debian/Fedora because it is pure MIT, and the package doesn’t have any dependency. I understand that this won’t serve all users (e.g. Windows Server users), so the CLI-way still works as a catch-all.
The deb/rpm approach isn't great as that's a bunch of overhead building and maintaining the packages, additionally if we keep the script bundled with VS Code then we're able to make breaking changes without many consequences, even new features could break old version of VS Code so it's best to keep them in sync using the same package. I'll think about how we can support SSH better when not using the Remote - SSH extension. Now that you remote, I realize the remote development case isn't covered by manual install currently as it uses desktop paths, we'll need to make sure that's covered as well.
The installation instructions at the link below are broken for what I have tested (Linux local and Remote): ,LINUX%20AND%20MACOS,-The%20script%20below When inside Remote - WSL or Remote - SSH (probably Containers too), the Resources Dir can be found with: vs : required to step back a folder from a file ( fails)calling is better than since it at least warns the user if the path can't be 't required In such case, the latter snippet also needs to be adjusted: And when not running through Remote, I wasn't able to find such a file in a Linux .deb installation. Even by following the original instructions, which would probably resolve to a path like: which isn't valid too. The correct path would be instead. And, in such case, the latter snippet would be correct.

thanks for the info, this is what we're planning for the improved experience in : This will also work on the server so the remote extensions will be covered the same.
I'm actually worried about perf here. Running will be slow in many systems, slow enough to notice lag when launching a shell. Given that this always returns the same value every time we execute it... could we perhaps do something similar to ?
installing it directly like the command there isn't portable across system though. We are looking into having the CLI redone in rust which will make it plenty fast enough as we won't need to launch node. I also considered having it in the .sh file itself, but that wouldn't work unless the cli is sourced, not run, and it also wouldn't work at all in pwsh since our CLI is a batch script.
Change:
), ); if (networkPath !== undefined) { <ins> // If the repository is at the root of the mapped drive then we // have to append `` (ex: D:) otherwise the path is not valid. const isDriveRoot = pathEquals(repoUri.fsPath, networkPath); </ins> return path.normalize( repoUri.fsPath.replace( networkPath, <del> `${letter.toLowerCase()}:${networkPath.endsWith('') ? '' : ''}` </del> <ins> `${letter.toLowerCase()}:${isDriveRoot || networkPath.endsWith('') ? '' : ''}` </ins> ), ); }

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. --<!-- 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: Version: 1.73.1 (system setup) Commit: Date: 2022-11-09T04:27:29.066Z Electron: 19.0.17 Chromium: 102.0.5005.167 16.14.2 V8: 10.2.154.15-electron.0 OS: Windows_NT x64 10.0. Sandboxed: No OS Version: Windows 11 Steps to Reproduce: a volume in Window that contains a git repo directly in the root of the volume (in my case it is an NFS share hosted on my NAS, but this also happens if it is simply an empty disk drive too) the root directory in VS Code and go to the Source Control window that it shows the diffs, but all the folders contain the volume letter at the beginning of the folders/files and the diff previews don't work !
<!-- 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!
I do see this message even though I created the volumes: !
I think you can capture that in a new issue, but you should be able to open files from the diff if you trust the repo
Yes. You can open files.

But since this is I removed my
author-verification-requested's can still be verified by team members (and often are if we don't get responses in time for endgame). It's just a helpful label so the bot can post a message and start the process asking the author to do the verification.
Looks good! It works for me! Thanks so much for fixing this! :) /verified
Change:
return !definition || !definition.when || this._contextKeyService.contextMatchesRules(definition.when); } <del> private async _getGroupedTasks(filter?: ITaskFilter, waitToActivate?: boolean): Promise<TaskMap> { </del> <ins> private async _getGroupedTasks(filter?: ITaskFilter, waitToActivate?: boolean, knownOnlyOrTrusted?: boolean): Promise<TaskMap> { </ins> await this._waitForAllSupportedExecutions; const type = filter?.type; const needsRecentTasksMigration = this._needsRecentTasksMigration();

Discussion:
a repo in :bug: see this !
This is expected if the repo that you open has tasks that run on . Could you pls provide me with a repo for which this happens that doesn't have those?
seeing this dialog when I just open a folder, so wondering if something with workspace trust has changed recently. No changes with tasks recently could explain this. !
Doesn't repro in , repros in : Bad: Good:
found these task PRs in that time frame: ,
found it also happens in an empty window on desktop
Still seeing this in (and also RemoteHub on desktop) with the changes in !
This is coming from -here. perhaps we don't want to do that for the or only do that in trusted workspaces? !
Change:
return Promise.reject(new Error('Parent folder is readonly.')); } <del> const stat = new ExplorerItem("_URI, folder, false); </del> <ins> const stat = new NewExplorerItem(folder, false); </ins> return folder.fetchChildren(this.fileService, this.explorerService).then(() => { folder.addChild(stat);

Discussion:
Issue Type: <bBug</bExplorer asks to input file name for a new file on the wrong level after pressing <kbdEscape</kbdbutton. On the gif file below we have focus on directory, but input field created on the root level of the project. ! asks to create this issue. Related issue is Keyboard navigation: simple. Extension to switching keyboard context is installed. VS Code version: Code - Insiders 1.32.0-insider (, 2019-02-22T08:34:22.116Z) OS version: Darwin x64 18.2.0 <details<summarySystem Info</summary </details<details<summaryExtensions (1)</summaryExtensionVersion ------ atom-keybindings3.0.6 </details<!-- generated by issue reporter --
My understanding is that this is because keyboard navigation still attempts to select the nearest file beginning with when you press , and that the “create file” action is only triggered secondary to that, so the focus has already shifted when the file is created. Super annoying.
Change:
} focusList(); list.focusNext(); <ins> alertFocusChange(notificationIndex, length, 'next'); </ins> renderAccessibleView(); }, previous(): void {

Discussion:
Testing Press alt+F2, VoiceOver starts reading the response Press alt+], the view shows the next response but VoiceOver is still reading that response and it never reads the next response Maybe related, but sometimes, it also doesn't read the response when I press alt+F2. It will say "you are now on a text area" and just sit there. Then, it only wants to read the text a single character at a time when I move the cursor.
Actually, I believe this might be working as expected - I have to move the cursor for the screen reader to move on to the next content. and could you pls let us know if this is a bug or working as expected?
Unfortunately, this is a voiceover issue, but I believe last time we something like this it was resolved my slightly increasing the wait time for certain focus event. I don't remember the full context and cannot find the issue right now. does it sound familiar? It was related to the opening and closing of the contextual help I believe.
Great recall the issue you are talking about is
Correct. I believe it could be the same underlying cause. By the way, having to move the cursor to read content is totally fine as you described. the real issue is that Voiceover is not announcing the change of context.
Makes sense - because without that announcement, you wouldn't know to move your cursor
I wonder if we have a way to calculate the total number of responses. if we do, we could issue an alert announcing position information when we move back and forth. This would resolve the lack of context for macOs and improve the experience in the other platforms. Thoughts?
Yes that makes sense to me - I linked an issue where this is a request for the notification a11y view and we could also support this for the responses
It might have to be an August thing - I'll investigate.
this is also an issue for notifications
I think we have to go with what you suggested i tried a bunch of hacks and setTimeouts and it didn't work
I think it could make the feature more helpful for all platforms. Moving between suggestions is right now a bit confusing because I can't tell for sure if the one on focus is the last one, or if it truly moved to a different one after pressing the command.

In the copilot chat panel, for example, I find much more practical and easier to simply move to the list of responses and go on from there. In addition, hacking assistive technology rarely justifies the effort, and often leads to undesirable outcomes that are not easy to anticipate.
it seems focus still gets trapped initially - even alert isn't taking precedent. i'm hoping it's just what I'm experiencing. about to make a PR with a change that does this for notifications and chat responses
What do you mean it gets trapped? Do you want me to review something. I have half an hour until next meeting.
yes, the linked PR please sorry I have been doing a lot atm
Verified by with the follow up item of to be addressed in August
Change:
this._onDidAttachViewModel.fire(); } <del> private async _warmupViewport(viewModel: NotebookViewModel, viewState: INotebookEditorViewState | undefined) { </del> <ins> private async _warmupViewportMarkdownCells(viewModel: NotebookViewModel, viewState: INotebookEditorViewState | undefined) { </ins> if (viewState && viewState.cellTotalHeights) { const totalHeightCache = viewState.cellTotalHeights; const scrollTop = viewState.scrollPosition?.top ?? 0;

Discussion:
Type: <bBug</ba jupyter notebook, create a python cell the cell the notebook two earch results the notebook bug One of the search results goes away <img width="1179" alt="Screenshot 2023-04-25 at 11 09 04 AM" src="https://user-"Making any changes to the search term makes both search results show again VS Code version: Code - Insiders 1.78.0-insider (Universal) (, 2023-04-25T05:41:22.331Z) OS version: Darwin x64 22.4.0 Modes: Sandboxed: Yes <details<summarySystem Info</summary </details<!-- generated by issue reporter --
Change:
for (( i=0; i < "${#str}"; ++i )); do byte="${str:$i:1}" <del> # Backslashes must be doubled. </del> <ins> # Escape backslashes and semi-colons </ins> if [ "$byte" = "" ]; then token="" <del> # Conservatively pass alphanumerics through. elif [[ "$byte" == [0-9A-Za-z] ]]; then token="$byte" # Hex-encode anything else. # (Importantly including: semicolon, newline, and control chars). </del> <ins> elif [ "$byte" = ";" ]; then token="x3b" </ins> else <del> # The printf '0x%02X' "'$byte'" converts the character to a hex integer. # See printf's specification: # > If the leading character is a single-quote or double-quote, the value shall be the numeric value in the # > underlying codeset of the character following the single-quote or double-quote. # However, the result is a sign-extended int, so a high bit like 0xD7 becomes 0xFFF…FD7 # We mask that word with 0xFF to get lowest 8 bits, and then encode that byte as "xD7" per our escaping scheme. builtin printf -v token 'x%02X' "$(( $(builtin printf '0x%X' "'$byte'") & 0xFF ))" #             |________| ^^^ ^^^                        ^^^^^^  ^^^^^^^  |______| #                   |     |  |                            |        |         | # store in `token` -+     |  |   the hex value -----------+        |         | # the 'x…'-prefixed -----+  |   of the byte as an integer --------+         | # 0-padded, two hex digits --+   masked to one byte (due to sign extension) -+ </del> <ins> token="$byte" </ins> fi out+="$token"

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. --<!-- Launch with to check. --Does this issue occur when all extensions are disabled?: Yes/No Splitting terminal in folder with ë character or any subdirectories errors and doesn't split. The terminal process failed to launch: Starting directory (cwd) "/Users/liam-nt/Documents/vscodebugtest/eÃ«e" does not exist. <!-- 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.74.0 (Universal) OS Version: macOS Monterey 12.5 (21G72) Steps to Reproduce: a folder containing 'ë' in name integrated VSCode terminal and split terminal by keyboard shortcut or interface.
related
!
Regression from is encoding all multibyte utf8 characters
Change:
} private async _activate(link: string) { <del> // Normalize the link and remove any leading ./ or ../ since quick access doesn't understand // that format </del> <ins> // Remove file:/// and any leading ./ or ../ since quick access doesn't understand that format link = link.replace(/^file:///?/, ''); </ins> link = normalize(link).replace(/^(.+[/])+/, ''); // Remove `:in` from the end which is how Ruby outputs stack traces

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. --<!-- 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.58.2 OS Version: Linux x64 5.4.0-80-generic Steps to Reproduce: open a vscode window open the embedded vscode terminal any file path to a path that exists, but prepended with the uri 'file:///' prefix (example: file:////home/somedir) ctrl+click on that path error message appears: "Unable to open 'somedir': File is a directory The expected behavior (opening the directory in vscode) works for regular paths (/home/somedir)
+1
duplicate of
Can we get a little explanation of how this is a duplicate? Seems completely unrelated. This is incorrect handling of RFC 8089, RFC 3986 links while the other is failure to handle softlinks while using remote ssh. The issue here is file:// schema is treated as if it can only be a file while on the posix path it detects file vs directory.
Can reproduce the problem, it should say folder not file: ! !
Repros on Windows too. Same thing happens in the editor: ! Above should use 3 slashes:
So the link does work (tried it with multiple folders)... but I guess I expected the underline to include the bit. It's a nit and I don't feel too strongly about it, but thought I'd mention. !
let's wait until is back for that. I'm considering this verified for now
Problem is back, looks a little different though: !
I think the new problem is Windows only
Change:
// using the most up to date dimensions (eg. when terminal is created in the background // using cached dimensions of a split terminal). this._resize(); <del> // Trigger a forced refresh of the viewport to sync the viewport and scroll bar. This is // necessary if the number of rows in the terminal has decreased while it was in the // background since scrollTop changes take no effect but the terminal's position does // change since the number of visible rows decreases. </del> <ins> // HACK: Trigger a forced refresh of the viewport to sync the viewport and scroll bar. // This is necessary if the number of rows in the terminal has decreased while it was in // the background since scrollTop changes take no effect but the terminal's position // does change since the number of visible rows decreases. </ins> // This can likely be removed after https://github.com/xtermjs/xterm.js/issues/291 is // fixed upstream. <del> this.xterm.forceRefresh(); </del> <ins> setTimeout(() => this.xterm!.forceRefresh(), 0); </ins> } }

Discussion:
Type: <bBug</bintegrated terminal either through button top right or key shortcut the key shortcut for "toggle panel visibility" -Opens integrated terminal from before is still there, looks normal (BUT scrollbar is on TOP not where it should be!) to scroll up on mouse wheel -Doesnt react to scroll down on mouse wheel -Terminal scrolls up to top (where scrollbar is) Key binding for "toggle panel visibility": ! Demo: Opening through View Terminal doesnt produce this bug. VS Code version: Code 1.75.1 (, 2023-02-08T21:35:30.018Z) OS version: Linux x64 5.15.0-60-generic snap Modes: Sandboxed: Yes <details<summarySystem Info</summary </details<details<summaryExtensions (10)</summaryExtensionVersion ------ url-encode1.1.0 vscode-pull-request-github0.58.1 beautify1.5.0 .8.0 isort2022.8.0 python2023.2.0 vscode-pylance2023.2.20 notepadplusplus-keybindings1.0.7 tyranosyntax0.15.0 markdown-pdf1.4.4 (1 theme extensions excluded) </details<details<summaryA/B Experiments</summary</details<!-- generated by issue reporter --
To add, it doesn't scroll all the way down to the input line. It skips a whole page. !
It also only seems to happen on the second opening. On the first opening it is scrolled down correctly for some reason...
I am encountering the same issue. All terminals are affected by this problem. The issue arose after hiding and reopening the panel. As you can see in the video, the scroll bar becomes shorter after reopening the terminal. Could it be that this only happens with ZSH? https://user-
Same issue with mine above. I'm using cmd, so the problem is not specific to a terminal type.
Related:
Change:
expires_in: Option<i64>, } <ins> #[derive(Deserialize)] struct AuthenticationError { error: String, error_description: Option<String>, } </ins> #[derive(clap::ArgEnum, Serialize, Deserialize, Debug, Clone, Copy)] pub enum AuthProvider { Microsoft,

Discussion:
Windows 11 Latest Insiders WSL with Ubuntu 20.04.5 LTS I am trying to get and run the insiders code cli on a WSL Ubuntu 20.04 instance. the linux x64 insiders cli from here: and save the file to someplace in your WSL Ubuntu 20.04 folder : the file is the same name as the stable x64 version, so i'm not sure if am really getting insiders. , note the time stamp in the future error disc i see a executable, why is there a code-insiders`? see 2 above the GitHub login dance Result - it hangs in the terminal, no URL is displayed, here's the output from (which repeats):
can you try again with today's Insiders build of the CLI? It should print more detailed information about the error, if any, that's happening.
works great today! i was able to log in, get the URL, and hit it from another machine. still, why is the executable and not ? I see I can use the option but that breaks my flow (always use from the terminal)
That's a to fix, tracked in I'll go ahead and close this issue for now. I have a feeling it was something on GH's end, but at least next time we see this, we should be able to get more info.
Change:
import { PartFingerprint, PartFingerprints, ViewPart } from 'vs/editor/browser/view/viewPart'; import { LineNumbersOverlay } from 'vs/editor/browser/viewParts/lineNumbers/lineNumbers'; import { Margin } from 'vs/editor/browser/viewParts/margin/margin'; <del> import { RenderLineNumbersType, EditorOption, IComputedEditorOptions } from 'vs/editor/common/config/editorOptions'; </del> <ins> import { RenderLineNumbersType, EditorOption, IComputedEditorOptions, EditorOptions } from 'vs/editor/common/config/editorOptions'; </ins> import { BareFontInfo } from 'vs/editor/common/config/fontInfo'; import { WordCharacterClass, getMapForWordSeparators } from 'vs/editor/common/controller/wordCharacterClassifier'; import { Position } from 'vs/editor/common/core/position';

Discussion:
default shoudl be 100. As pointed out we should measure if this has a perf impact on Edge, Chromium and FF We can also increase the default value only if we see that a screen reader is detected