Skip to content
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

Merged
merged 9 commits into from
Apr 2, 2025

Conversation

go-wyvern
Copy link
Member

No description provided.

@go-wyvern go-wyvern changed the title feat(copilot): add tool support and send tools to llms feat(copilot): add tool support and send tools to LLMs Mar 25, 2025

### xbuilder-action

#### Available Tools
Copy link
Collaborator

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`.
Copy link
Collaborator

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}}
Copy link
Collaborator

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

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@qiniu-ci
Copy link

qiniu-ci commented Apr 2, 2025

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.

@nighca nighca merged commit e8994cb into goplus:dev Apr 2, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants