-
Notifications
You must be signed in to change notification settings - Fork 200
Add initial copilot prompt for e2e flow #20686
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
|
The latest updates on your projects. Learn more about Vercel for GitHub. 2 Skipped Deployments
|
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.
Coding Rules LGTM \o/
| sessionInitialization: `<session_initialization> | ||
| When a session begins, you MUST call \`get_agent_config\` to retrieve the current agent configuration and any pending suggestions. | ||
| This tool must be called every time you are invoked to ensure you have the latest state. | ||
| The response includes: | ||
| - Agent settings (name, description, scope, model, tools, skills) | ||
| - Instructions: The committed instructions text (without pending suggestions) | ||
| - pendingSuggestions: Array of suggestions that have been made but not yet accepted/rejected by the user | ||
| Based on the context, follow the appropriate initialization flow: | ||
| <new_agent_flow> | ||
| For a new agent with a blank template: | ||
| 1. Based on the user's metadata (job domain, preferences) and the conversation context, provide a list of suggested use cases | ||
| 2. Guide the user to select a use case or describe their custom needs | ||
| </new_agent_flow> | ||
| <existing_agent_flow> | ||
| For an existing agent: | ||
| 1. Call \`list_suggestions\` filtered by \`pending\` state and the current agent version | ||
| 2. Call \`get_agent_feedback\` to retrieve feedback for the current version | ||
| 3. Summarize key insights and pending suggestions to the user | ||
| IMPORTANT: Avoid calling \`list_suggestions\` again mid-session unless explicitly asked. This prevents race conditions with concurrent agent builder sessions where another builder might accept/reject reinforced suggestions. | ||
| </existing_agent_flow> | ||
| Balance context gathering with latency - the first copilot message should be fast but helpful in driving builder actions. | ||
| </session_initialization>`, |
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.
I think the session initialization belongs to the first message sent in CopilotPanelContext, as it brings value only for the first messages but not for the rest of the 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.
Agreed 👍
| ## WORKFLOW VISUALIZATION | ||
| 4. On each new agent message, suggestions for the current agent are refreshed to reflect the latest data. | ||
| 5. Use \`update_suggestions_state\` to programmatically reject or mark suggestions as outdated when needed. |
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.
do we have this tool yet ?
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.
I was playing with it locally, but can remove, no need to include for now
Description
Tests
Risk
Deploy Plan