-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Search Agent with GPT mini model #2736
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 our terms of service and privacy 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?
Conversation
…ent, rather than hardcoding gpt4.1
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.
@bhavyaus let me know when you have a chance to take a look at the updates/questions
…ha/vscode-copilot-chat into anisha/flight_searchagent
…re no nested loops with codebase
| import { IBuildPromptContext } from '../common/intents'; | ||
| import { IBuildPromptResult } from './intents'; | ||
|
|
||
| export interface ISubagentToolCallingLoopOptions extends IToolCallingLoopOptions { |
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 this file still needed?
|
|
||
| const request = this._inputContext!.request!; | ||
|
|
||
| const loop = this.instantiationService.createInstance(SubagentToolCallingLoop, { |
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.
this should be using the SearchSubagentToolCallingLoop?
|
|
||
| constructor( | ||
| @IInstantiationService instantiationService: IInstantiationService, | ||
| @ILogService logService: ILogService |
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.
nit: please undo these changes since no code changes are going in.
| "github.copilot.config.githubMcpServer.lockdown": "Enable lockdown mode for the GitHub MCP Server. When enabled, hides public issue details created by users without push access. [Learn more](https://aka.ms/vscode-gh-mcp-lockdown)." | ||
| "github.copilot.config.githubMcpServer.toolsets": "Specify toolsets to use from the GitHub MCP Server.", | ||
| "github.copilot.config.githubMcpServer.readonly": "Enable read-only mode for the GitHub MCP Server. When enabled, only read tools are available.", | ||
| "github.copilot.config.githubMcpServer.lockdown": "Enable lockdown mode for the GitHub MCP Server. When enabled, hides public issue details created by users without push access.", |
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.
Please undo these changes to string that are not subagent related
|
Do we need to update the grok and gemini agent prompts also? |
Flight the search subagent powered by a non fine-tuned model. Use of search subagent is also conditional on whether the user is in Auto mode in the model picker.