-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
tools: add first draft of AGENTS.md (tested with Gemini, edited by Claude, and Codex) #26502
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?
Conversation
70aac80 to
9bc0eda
Compare
…Variables sections Adds three high-value sections to help AI agents work more effectively with the V compiler: - Error Reporting: API for c.error(), c.warn(), c.note() in checker/parser - Option/Result Types: Syntax, common bugs, test locations, and cgen pitfalls - Environment Variables: VFLAGS, VAUTOFIX, VEXE, and V2-specific variables These additions provide timeless guidance on compiler internals without time-specific references. Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
…nually, thanks Brother Richard)
Split two lines exceeding 100 character limit to pass markdown linting. Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
Make it clear that check-md must be run before committing .md files: - Added "(required for .md files before commits)" to Tools section - Updated Gotchas to mention both fmt and check-md requirements Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
…ler models to benefit from the AGENTS.md file.
|
I think it works well enough as a first draft. |
|
@medvednikov what do you think? |
|
Great job. I'd also add that it should generally avoid the |
|
Seems great. Let's see how it behaves and perhaps in the future it will be interesting to include information about cstrict, file naming (.c.v, .js.v, _linux.v, etc, compile-time code (IA still making mistakes about it 😓) |
medvednikov
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.
LGTM
|
Good start. Let's keep an eye on how it performs as it goes beyond compiler tasks. |
|
Related paper: https://arxiv.org/abs/2601.20404
|
|
Just hit this one... need a rule to tell the agent to put all v command line options between
Which fails... will also help with I wish V had always enforced this, but... we are where we are. |
…nd before the subcommand/file. Edit the rest for clarity.
|
|
The current rules say to build |
I think some apps may hardcoded |
No description provided.