Conversation
* Refactor hook execution * Fix compilation: add IExtHostHooks import, remove unused IHookResult, inline ChatRequestHooks type * Move hooks property to chatHooks proposal, sync DTS * cleanup * Remove dead hook execution code: proxy, RPC, output channel, progress events All hook execution now happens in the extension via NodeHookExecutor. HooksExecutionService is now a pure registry (registerHooks/getHooksForSession). Removed: - executeHook, setProxy, onDidHookProgress from service - IHooksExecutionProxy, IHookProgressEvent, HookAbortError, formatHookErrorMessage - hooksCommandTypes.ts, hooksTypes.ts (dead type files) - mainThreadHooks proxy setup - extHostHooksNode., extHostHooksWorker. - ExtHostHooksShape. protocol - IExtHostHooks DI registrations - ChatHooksProgressContribution - All associated test files * Remove HooksExecutionService entirely The service was only a registry for session hooks, but hooks are already passed directly on the chat request DTO. The registerHooks/getHooksForSession pattern was redundant. * Restore modelName support in chatSubagentContentPart that was accidentally removed during merge * Revert unrelated tabIndex change on chatSubagentContentPart * Remove empty hooks ext host infrastructure Delete IExtHostHooks, NodeExtHostHooks, WorkerExtHostHooks, MainThreadHooks, ExtHostHooksShape, MainThreadHooksShape - all were empty stubs after hook execution moved to extension. * Remove mainThreadHooks import from extensionHost.contribution * Fix DTS comments: env and timeoutSec are values, not implementation promises
* Tweak tips * add steering
Allow contributed configurations to define additional "search terms"
For #274403 Should let us support non-local sessions. Keeping around id support for backcompat
* experiment background agent display name * update session type picker to handle background agent display name changes
Store sessionResource as view state instead of id
For #292934 Testing this with a self contained area first: code lenses. Will keep support for the old setting values too to avoid breaking existing settings
* PROTOTYPE: Add growth agent (microsoft/vscode-copilot-chat#3460) * support vscode.ChatSessionStatus.NeedsInput in chatSessions ext api ref #292430 * Add isReadOnly flag to chat sessions contributions Read-only session types (e.g., Growth) are passive/informational and should not be registered as agents, appear in session target pickers, or be delegation targets. Commands are still registered to support openSessionWithPrompt. * Collapse isReadOnly and canDelegate branches in _enableContribution Both need agent and command registration; picker filtering handles keeping isReadOnly sessions out of the UI separately. the alternative (and probably ideal) UI is to 'grey out'/'disable' the chat input for isReadOnly sessions. That way we don't have this problem at all of a non-functional chatInput * fix description * redundant doc * update test
Start adopting unified js/ts config for code lenses
Reduce amount of data we're passing over to ext host each time an update happens
Add optimized way to update a single chat session item
Add vertical spacing to rendered markdown list items in chat
Use font-weight 600 for bold text in rendered markdown
… toolbars in chat (#294275) Improve rendered markdown table styling in chat
Tweak code backgrounds in markdown
This is just more clear and returns search results for "agent"
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
10 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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )