diff --git a/.agents/docs/code-review.md b/.agents/docs/code-review.md index 3092e419a..fe7e63689 100644 --- a/.agents/docs/code-review.md +++ b/.agents/docs/code-review.md @@ -26,10 +26,10 @@ line, not as background reading. call leaves a null result, not just what enables it. - Check the change is still needed against current `master`: the surrounding code may have moved, or another change may have landed the same fix. -- On an existing PR, walk every discussion item one by one: what was raised, whether it was - answered, and whether it still applies to the current head. Never skip one because it looks - resolved, old, or minor; this walk overrides any read-comments-lightly default of the reviewing - skill. `gh pr view` misses review bodies and inline threads; pull all three: +- On an existing PR, walk every discussion item one by one, bot reviews included: what was raised, + whether it was answered, and whether it still applies to the current head. Never skip one + because it looks resolved, old, or minor; this walk overrides any read-comments-lightly default + of the reviewing skill. `gh pr view` misses review bodies and inline threads; pull all three: ``` gh api repos/azerothcore/azerothcore-wotlk/issues//comments --paginate # conversation comments diff --git a/AGENTS.md b/AGENTS.md index 54745bec6..fb1b06047 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -19,7 +19,7 @@ Read the matching doc(s) BEFORE starting the task: - Creating or modifying SQL → `.agents/docs/sql-guidelines.md` - SmartAI work (`smart_scripts` data) → also `.agents/docs/cpp-scripts.md` - Reviewing a changeset or PR → `.agents/docs/code-review.md` -- Self-reviewing a changeset before submission → also `.agents/docs/self-review-rules.md` +- Self-reviewing, or opening or updating a PR → also `.agents/docs/self-review-rules.md` - Touching a subsystem that has a doc in `.agents/docs/systems/` → read that doc too - Capturing a lesson or adding/updating agent docs → `.agents/docs/README.md`