[mirror] microsoft/vscode#309240 Add Agent Host session configuration flow#1589
Open
austindyoung wants to merge 7681 commits intomainfrom
Open
[mirror] microsoft/vscode#309240 Add Agent Host session configuration flow#1589austindyoung wants to merge 7681 commits intomainfrom
austindyoung wants to merge 7681 commits intomainfrom
Conversation
…rimate Update build ts version
…mit checks directly in hygiene function (microsoft#308698)
* chat confirmation carousel rehaul v2 * address some comments on duplication * address some more comments
microsoft#308708) Refactor applyPackageJsonPatch to remove isPreRelease parameter and derive it from VSCODE_QUALITY environment variable; add getDateBasedPatch function for versioning Co-authored-by: Copilot <copilot@github.com>
…ax check - Fix heritage clause scope resolution: walk extends expressions before pushing the inner class scope so #field references resolve to the enclosing class (matching JS lexical scoping) - Fix name collision: skip generated names that collide with existing public member names on the same class - Add syntax validation for post-processed JS bundles using esbuild - Add 5 tests covering both bugs and the brand-check variant
…deterministic errors
…oft#244597) Check user-configured editor associations as a fallback when the editor resolver cache is empty, ensuring custom editors are available on first startup.
Co-authored-by: Andrea Mah <andreamah@microsoft.com>
…erive it from VSCODE_QUALITY environment variable; add getDateBasedPatch function for versioning (microsoft#308737) lock versions up
…arWorkspaceChanges to handle both session IDs and folder URIs (microsoft#308644) * refactor: remove getSessionIdForWorkspaceFolder method and update clearWorkspaceChanges to handle both session IDs and folder URIs * Address comments
…#308749) * nes: replace implicit _icon state with explicit outcome state machine (microsoft#308719) Replace the implicit \`_icon\` field with an explicit \`_outcome: LogContextOutcome\` discriminated union in InlineEditRequestLogContext. - Define LogContextOutcome type with 8 states: pending, succeeded, noSuggestions, cached, cachedFromGhostText, skipped, cancelled, errored - Derive icon from outcome via switch in _resolveIcon() - Keep _isCompleted orthogonal (lifecycle flag, not an outcome) - Add _setOutcome() with debug warning on invalid re-transitions - Refactor setIsCachedResult to use direct field copy (avoids triggering outcome transitions during bulk data inheritance) - Add double-completion warning in markCompleted() - No caller changes required — public method signatures unchanged * nes: unify live and static log entry paths (microsoft#308720, microsoft#308722) Remove the static \`add()\` path from InlineEditLogger — all log entries now use the live pattern with callbacks for icon/content resolution. - Remove InlineEditLogger.add() and _liveRequestIds tracking - Ghost text provider now uses addLive() + markCompleted() in finally block, ensuring lifecycle is always closed - Remove redundant add() call in NES inlineCompletionProvider (the live entry was already registered via addLive()) This eliminates the static snapshot path that was the source of frozen spinner bugs, and ensures ghost text entries have proper lifecycle management matching the NES pattern. * nes: fix stale icon caching in ChatPromptItem tree nodes (microsoft#308721) Store the main entry reference in ChatPromptItem and always resolve the icon unconditionally from it, preventing stale icon snapshots. - Add _mainEntryRef field to ChatPromptItem - setMainEntry() always sets iconPath (even to undefined when icon resolves to undefined), clearing any previous stale value - withFilteredChildren() re-resolves from the entry ref via setMainEntry() instead of copying a potentially stale iconPath This makes stale icons structurally impossible — the icon is always derived from the live entry state on each tree refresh. * nes: map cancelled outcome to skipped icon instead of loading Cancelled requests should show a terminal icon, not a spinner. Map 'cancelled' to Icon.skipped alongside 'skipped' to avoid frozen-spinner appearance in the log tree.
…of XtabProvider (microsoft#308753) * nes: fix: check chatResponseFailure consistently before yielding edits * nes: xtab: cleanup: split response format parsing implementation out of XtabProvider
…-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
…epositories using GVFS (microsoft#308740)" (microsoft#309162) This reverts commit f6d4cad.
Update distro commit to 21295cb9 Co-authored-by: vs-code-engineering[bot] <122617954+vs-code-engineering[bot]@users.noreply.github.com>
(Written by Copilot)
(Written by Copilot)
(Written by Copilot)
(Written by Copilot)
(Written by Copilot)
…08655) * Show unsandboxed retry in chat terminal * Update src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/tools/runInTerminalTool.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fixing failed tests * fixing tests * Remove automatic unsandbox retry note --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Add advanced sandbox runtime config setting
…crosoft#309239) Chat - add null checks for editingSession across await boundaries (microsoft#309129) awaitStatsForSession reads model.editingSession across multiple await points. If the model is disposed during one of these awaits, the editingSession property is nulled and the next access crashes. Re-check model.editingSession after each await and bail out with undefined when the session has gone away. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…lock.json (microsoft#309257) * Update Copilot SDK to version 1.0.24 * Updates * Fix tets * Fixes
Tell the heap-snapshot-analysis skill to start fresh and not be influenced by prior findings in /memories/ or the scratchpad. Previous investigations can bias analysis toward suspects that are no longer relevant. (Written by Copilot)
…session-cwd Fix agent host session working directories
…sion-types Co-authored-by: Copilot <copilot@github.com>
… into roblou/agent-host-session-config # Conflicts: # src/vs/sessions/contrib/localAgentHost/browser/localAgentHostSessionsProvider.ts # src/vs/sessions/contrib/remoteAgentHost/browser/remoteAgentHostSessionsProvider.ts
Written by Copilot
Written by Copilot
Written by Copilot
(Written by Copilot)
(Written by Copilot)
(Written by Copilot)
(Written by Copilot)
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
20 participants
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#309240
Source URL: microsoft#309240
Mirror branch: driftfence/mirror/pr-309240/latest
This PR is maintained by automation for telemetry and review links.