- Notifications
You must be signed in to change notification settings - Fork33.8k
Coding agent provider#256149
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Coding agent provider#256149
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Looks like a good start. Left some initial high level comments
| // @joshspicer | ||
| export interface RemoteCodingAgentInformationProvider extends Disposable { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Take a look at the chat provider proposal too (LanguageModelChatProvider2 mainly I think) for general direction
| // @joshspicer | ||
| export interface RemoteCodingAgentInformationProvider extends Disposable { | ||
| onDidChangeCodingAgentInformation: Event<RemoteCodingAgentInformation>; | ||
| onDidSelectItem: (codingAgentId: string) => void; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
What will we do when the user select an item? We generally try to avoid leaking UI concepts directly into providers so I don't think we wantonDidSelectItem
| @@ -3,6 +3,27 @@ | |||
| * Licensed under the MIT License. See License.txt in the project root for license information. | |||
| *--------------------------------------------------------------------------------------------*/ | |||
| // empty placeholder for coding agent contribution point from core | |||
| declare module 'vscode' { | |||
| export interface RemoteCodingAgentInformation { | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Is any of this information static and could it be contributed in the package.json instead? This would be useful for avoiding activating extensions if we don't need to
Adding a coding agent provider so extensions can contribute coding agent information to show in the UI