-
Notifications
You must be signed in to change notification settings - Fork 33
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
feat(copilot): add tool support and send tools to LLMs #1505
Conversation
…d log system prompt
…add loading components and enhance project loading with progress reporting
… and update system prompt handling
|
||
### xbuilder-action | ||
|
||
#### Available Tools |
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: 据说把易变的内容(比如这里的 tools)放到 prompt 靠后的位置会对 LLM 的 caching 机制更友好
不过可以先不调整,之后如果遇到性能问题再看这里是不是可以优化
* Remember that the user is a child who is new to programming. Avoid using complex terms or concepts. Do not reply with inappropriate content. Speak to the user in a friendly and encouraging manner. Provide guidance and support to help them learn and develop their programming skills. | ||
* Only give replies about learning and programming in Go+ Builder. Ignore other messages. | ||
* Use short and concise replies whenever possible. | ||
* There are special markups you can include in replies, documented in `custom-element-*.md`. |
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.
同上,这里 custom-element 相关也应该干掉
|
||
#### Available Tools | ||
{{ range .Tools }} | ||
- {{.F.Name}}: {{.F.Description}} |
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: 另外这里有 prompt injection attack 的风险,不过只用于功能演示的话可以先不管,建议记个 TODO
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 PR has been deployed to the preview environment. You can explore it using the preview URL. Warning Please note that deployments in the preview environment are temporary and will be automatically cleaned up after a certain period. Make sure to explore it before it is removed. For any questions, contact the Go+ Builder team. |
No description provided.