[mirror] microsoft/vscode#309670 [cherry-pick] Don't show onboarding on web#1695
Open
austindyoung wants to merge 7706 commits intomainfrom
Open
[mirror] microsoft/vscode#309670 [cherry-pick] Don't show onboarding on web#1695austindyoung wants to merge 7706 commits intomainfrom
austindyoung wants to merge 7706 commits intomainfrom
Conversation
…-property build: fix private-to-property rewrite bugs, add syntax check
…hat do not have any commits (microsoft#308752) * Sessions - isolation/branch picker should handle empty repositories that do not have any commits * Fix the tests
…rosoft#308608) - Add isClientBYOKEnabled() method to CopilotToken class - Update isBYOKEnabled to allow users with client_byok=1 token flag - Add github.copilot.clientByokEnabled context key for VS Code core model picker Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Without this, `this._controller.items.get(uri)` returns undefined... but the metadata is stored on the item so that is needed to resolve. FYI @mjbvz as an interesting quirk of the API.
* ghost: debt: migrate to CompletionsFetchService and remove dead code * ghost: propagate copilotAnnotations in convertStreamToApiChoices The v2 stream processor (convertStreamToApiChoices) was not setting copilotAnnotations on the returned APIChoice objects, despite tracking them in the CompletionAccumulator. This caused annotations (e.g., code citations) to be lost. Add accumulator.annotations.current to the createAPIChoice helper. * ghost: fix tests to mock ICompletionsFetchService for v2 After migrating to CompletionsFetchService, LiveOpenAIFetcher no longer calls ICompletionsFetcherService (via postRequest). Tests that mocked the old low-level fetcher with StaticFetcher were silently bypassed. Add StaticCompletionsFetchService adapter that bridges StaticFetcher responses into ICompletionsFetchService by parsing SSE bodies through jsonlStreamToCompletions. Wire it up in ghostText.test.ts and inlineCompletion.test.ts setupCompletion helpers.
…icrosoft#308726) * inline chat: bypass zone widget in hover mode, use direct sendRequest * address review: notebook config gate, faked timers in tests
* proper insider build increments * Update extensions/copilot/.esbuild.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ewrite In minified code, `async#method()` has no space between `async` and `#method`. The `#` naturally starts a new token, but when replaced with `$a`, the `$` is a valid identifier character — `async$a` fuses into a single identifier. This makes any `await` inside the method body a syntax error. Fix: when the character immediately before the `#` is an identifier character, prepend a space to the replacement text. Add 3 tests covering async, static async, and the readable variant.
…election and update related tests
…308600) Bumps [basic-ftp](https://github.com/patrickjuchli/basic-ftp) from 5.2.0 to 5.2.1. - [Release notes](https://github.com/patrickjuchli/basic-ftp/releases) - [Changelog](https://github.com/patrickjuchli/basic-ftp/blob/master/CHANGELOG.md) - [Commits](patrickjuchli/basic-ftp@v5.2.0...v5.2.1) --- updated-dependencies: - dependency-name: basic-ftp dependency-version: 5.2.1 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 5 to 7. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v5...v7) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/cache](https://github.com/actions/cache) from 4 to 5. - [Release notes](https://github.com/actions/cache/releases) - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md) - [Commits](actions/cache@v4...v5) --- updated-dependencies: - dependency-name: actions/cache dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/github-script](https://github.com/actions/github-script) from 7 to 8. - [Release notes](https://github.com/actions/github-script/releases) - [Commits](actions/github-script@v7...v8) --- updated-dependencies: - dependency-name: actions/github-script dependency-version: '8' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…session-types agentHost: Support multi-agent agent host session types, fix model picker
Chat queue/steer: fall back to normal send when no request is active When the queue or steer action is invoked but no request is in progress, send the message as a normal chat message instead of silently doing nothing. Relaxes the precondition and Alt+Enter keybinding so the actions remain reachable even after the session finishes, while keeping the Enter keybinding gated behind queuingActionsPresent so it doesn't conflict with the regular submit action. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
display this model may be experimental or in evaluation
…ith code block (microsoft#309336) * chat: fix body tag leaking into rendered markdown when content ends with code block When markdown content ends with a fenced code block, the closing backticks were immediately followed by </body> on the same line, so CommonMark didn't recognize the fence as closed. Add \n\n before </body> to ensure proper parsing. Add regression test. (Written by Copilot) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * test: delete stale snapshots for regeneration The extra \n\n before </body> changes rendered HTML output slightly, so snapshot baselines need to be regenerated by CI. (Written by Copilot) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * test: regenerate snapshot baselines for body tag fix Update snapshot files to reflect the extra \n\n before </body> in the markdown template, which adds trailing whitespace before the closing wrapper div in rendered HTML output. (Written by Copilot) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…309308) * Chat - handle invalid URIs in extractCodeblockUrisFromText When an LLM explains code that uses vscode_codeblock_uri tags, the literal tag content may not be a valid URI, causing URI.parse to throw and crash the chat renderer. Add try-catch around URI.parse so invalid content gracefully returns undefined, which callers already handle by falling back to normal code block rendering. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix test: use content with backticks and colon to trigger URI scheme validation error Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…and branch updates (microsoft#309338)
[cherry-pick] inlineChat: fix model picker overflow when selecting Auto
…n log tree (microsoft#309477) Co-authored-by: vs-code-engineering[bot] <vs-code-engineering[bot]@users.noreply.github.com>
… terminal notifications (microsoft#309551) Fix microsoft#309523
Fix focus keybindings in sessions window (microsoft#309472, microsoft#309475, microsoft#309476)
Remove the _session = undefined line that breaks the back-reference from requests to the ChatModel during dispose. This causes a crash in chatListRenderer when the _onDidUpdateViewModel event fires and accesses template.currentElement.sessionResource on a stale item whose session has been nulled out. The _requests array is already emptied right after, so the back-references are broken when the requests are GC'd with the model. Fixes microsoft#309568 (Written by Copilot)
…the in-flight request before reusing it (microsoft#309488) Co-authored-by: vs-code-engineering[bot] <vs-code-engineering[bot]@users.noreply.github.com>
- Make 'Continue with GitHub' a true primary button (accent bg, white text/icon) - Change 'Skip' to secondary style on the sign-in step - Replace 'Continue' with tertiary 'Continue without Signing In' on sign-in step - Other steps retain their original button styles
…izing - Swap Skip/Continue without Signing In button styles (Skip → ghost, Continue → secondary) - Fix focus rings on primary-colored buttons using double-ring box-shadow pattern - Add focus-visible styling for doc links (Agents tutorial) - Fix focus ring clipping on Agents tutorial link - Bump theme card label font-size from 10px to 12px Fixes microsoft#309377 Fixes microsoft#309380 Fixes microsoft#309381 Fixes microsoft#309386
…pport - Make theme cards individually tabbable (tabindex=0) instead of roving tabindex - Separate focus (outline ring) from selected (border+box-shadow) for clear visual distinction - Add high contrast overrides using contrastBorder/contrastActiveBorder - Fix focus clipping by changing step-content overflow to visible and adding grid padding Fixes microsoft#309517
…-release VS Code Onboarding TPI Fixes
…microsoft#309646) Co-authored-by: vs-code-engineering[bot] <vs-code-engineering[bot]@users.noreply.github.com>
…ender (microsoft#309649) Co-authored-by: vs-code-engineering[bot] <vs-code-engineering[bot]@users.noreply.github.com>
…ages (microsoft#309653) Co-authored-by: vs-code-engineering[bot] <vs-code-engineering[bot]@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated mirror PR for DriftFence counterfactual telemetry.
Source PR: microsoft#309670
Source URL: microsoft#309670
Mirror branch: driftfence/mirror/pr-309670/latest
This PR is maintained by automation for telemetry and review links.