Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
There was a problem hiding this comment.
Thanks, @qgallouedec, having clear guidelines for automated coding agents is definitely valuable.
However, I have a few concerns about this approach.
Cursor already supports AGENTS.md: From my understanding, Cursor already recognizes AGENTS.md as a source of repository guidance. If that is the case, duplicating its contents into .cursor/rules/agents.mdc may not be necessary.
Prefer repository-agnostic conventions: Introducing tool-specific configuration files can make the repository gradually accumulate rules tailored to individual frameworks or editors. While supporting developer tooling is helpful, it may be better to rely on agnostic and broadly supported conventions (such as AGENTS.md) rather than adding files tied to a specific tool like Cursor.
This keeps the repository cleaner and avoids implicitly favoring one development environment over others.
Do you have any pointer for this? I felt like the guidelines were ignored. Let me open a pr with a clear violation of to rule and see. |
|
See: #5287 agents.md explicitly asks for replacing arXiv with HF papers, and cursor outputs: Note Low Risk Overview Written by Cursor Bugbot for commit dca45db. This will update automatically on new commits. Configure here. |
Just googling it: https://cursor.com/docs/rules#agentsmd
|
Oh, I see you are talking about Cursor Bugbot, not local Cursor. My understanding is that Cursor Bugbot operates somewhat separately from the normal editor agent, so it may not ingest repository guidance files the same way the interactive agent does. In other words, even if Cursor itself reads AGENTS.md (or .cursor/rules/*.mdc) during coding, Bugbot’s automated PR summaries may be generated with a different prompt/configuration, which could explain why the rule was ignored. So the behavior you observed in #5287 might indicate that Bugbot does not read AGENTS.md. It could be worth checking Cursor’s docs or configuration options to see whether Bugbot supports:
If Bugbot indeed ignores repo-level rules entirely, then this PR may not change its behavior. It would mostly affect interactive Cursor agents, which (as far as I know) already have some support for AGENTS.md. |
Adds
.cursor/rules/agents.mdcso Cursor users get the same repository guidelines as Claude Code / Codex agents.Note: this duplicates
AGENTS.md, which isn't ideal, a cleaner solution is being tracked in #5268Note
Low Risk
Low risk: adds a documentation-only Cursor rule file with no runtime or library behavior changes.
Overview
Adds
.cursor/rules/agents.mdcto mirror the repository’s contribution/engineering guidelines (currently inAGENTS.md) so Cursor users automatically see the same rules (duplication/consistency expectations, docstring format, and paper-link conventions).Written by Cursor Bugbot for commit 2759d06. This will update automatically on new commits. Configure here.