diff --git a/.cursor/rules/code-review-agent.md b/.cursor/rules/code-review-agent.md index 8380f32..e4fbb1b 100644 --- a/.cursor/rules/code-review-agent.md +++ b/.cursor/rules/code-review-agent.md @@ -83,6 +83,7 @@ In **chat only**, you may use Cursor line-number **code citations** when referen ## Boundaries - Do **not** skip writing `docs/reviews/…`; the document is mandatory output of this agent. +- When suggesting **PR description** text for the user to paste, follow [commit-and-review](commit-and-review.md) — **no** “Made-with: Cursor” / tool-attribution footers unless the user asks. - Do **not** rewrite the whole change unless asked; review is the default. - Do **not** request large refactors unrelated to the diff without strong justification labeled as non-blocking. - If context is insufficient (missing diff, wrong branch), still create the document with a **“Blocked”** verdict section explaining what is missing, then ask in chat for the missing context. diff --git a/.cursor/rules/commit-and-review.md b/.cursor/rules/commit-and-review.md index 70ca3c2..86b3226 100644 --- a/.cursor/rules/commit-and-review.md +++ b/.cursor/rules/commit-and-review.md @@ -16,6 +16,11 @@ alwaysApply: true - The user should **review** the diff before anything is committed. The agent’s job is to make the changes visible (uncommitted) and explain them; the user decides when to commit. - If the user asks to commit, use a message that matches repo conventions; still follow [git workflow](git-workflow.md) (branch vs `main`, doc-only vs code). +## Pull request and push descriptions + +- Do **not** add **“Made-with: Cursor”**, **“Generated with Cursor”**, tool co-author lines, or similar AI/IDE boilerplate to **PR descriptions**, **GitHub merge/squash commit bodies** you draft, or other **remote-facing** narrative unless the user explicitly requests it. +- Keep PR text to scope, verification, and project-required contract snippets (e.g. from `docs/plans/`). + ## Scope - Applies to **all** commits the agent might make, including **documentation-only** changes (e.g. `docs/plans/`, README), not only application source. diff --git a/AGENTS.md b/AGENTS.md index 44fabc4..8db068c 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -6,4 +6,4 @@ Optional **personas** for Cursor. Enable by **@ mentioning** the rule in chat or |--------|-----------|-------------| | **Code review** | [`.cursor/rules/code-review-agent.md`](.cursor/rules/code-review-agent.md) | PR / diff / pre-merge review; **always writes** `docs/reviews/YYYY-MM-DD-{slug}.md` with **Documentation checked** vs `docs/plans/` and `docs/decomposition/modules/`; short chat pointer | -Project-wide conventions live under [`.cursor/rules/`](.cursor/rules/) (many are `alwaysApply`). **Godot client layout** (thin `main.gd`, split by concern): [`.cursor/rules/godot-client-script-organization.md`](.cursor/rules/godot-client-script-organization.md). +Project-wide conventions live under [`.cursor/rules/`](.cursor/rules/) (many are `alwaysApply`). **Godot client layout** (thin `main.gd`, split by concern): [`.cursor/rules/godot-client-script-organization.md`](.cursor/rules/godot-client-script-organization.md). **PR / push text:** no “Made-with: Cursor” boilerplate — [`.cursor/rules/commit-and-review.md`](.cursor/rules/commit-and-review.md).