-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Introduce agents md #42955
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: master
Are you sure you want to change the base?
Introduce agents md #42955
Conversation
🤖 AI-Powered Code Review AvailableYou can leverage AI-powered code review to assist with this PR! Available Commands:
|
* Add `log` function to write messages to stderr * Fix deletion of `.github/workflows/validate-agents-md.yml` file
|
@content-bot start review |
|
🤖 Content AI Reviewer: Analysis started. Please wait for results... |
|
@content-bot start review |
|
🤖 Content AI Reviewer: Analysis started. Please wait for results... |
1 similar comment
|
🤖 Content AI Reviewer: Analysis started. Please wait for results... |
🤖 Content-bot Review DisclaimerThis review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause. |
content-bot
left a comment
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.
Thanks for the contribution! I've reviewed the changes and have a few suggestions to improve the script's robustness and the documentation's accuracy.
Please update the invisible characters script to handle potential UTF-8 decoding errors and expand the covered character ranges, while also verifying that the logging format functions correctly in local environments. Regarding AGENTS.md, I recommend modernizing the guidance to favor CommandResults over return_outputs and clarifying the injection behavior of common server files.
Great work on this!
@yuvalbenshalom, @ShahafBenYakir, @YuvHayun, @DeanArbel, @yedidyacohenpalo please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.
|
Validate summary Verdict: PR can be force merged from validate perspective? ✅ |
Description
Introducing
AGENTS.mdas a standardized guidance file for AI coding agents working with thedemisto/contentrepository.