From a0e1aa250731d0557393385bf95d2b955fa13a87 Mon Sep 17 00:00:00 2001 From: SoftFever Date: Sat, 16 May 2026 22:22:06 +0800 Subject: [PATCH] update agents.md --- AGENTS.md | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/AGENTS.md b/AGENTS.md index e1a230c3e3..3ebfccd0c0 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -46,3 +46,12 @@ ctest --test-dir ./tests/fff_print - **Cross-platform** — all changes must work on Windows, macOS, and Linux - Profile/format changes need version migration handling - Dependencies built separately in `deps/build/`, then linked to main app + +## Code review focus areas + +- Changes must not cause regressions in existing functionality, defaults, profiles, or project compatibility. +- Features gated by options must not affect existing behavior when those options are disabled. +- Changes should follow the existing code style and architecture. Architectural changes should be justified in code comments and the PR description. +- Add helper functions or utilities only when existing code cannot reasonably be reused. Avoid duplication. +- Keep code concise and clear. Manually simplify AI generated bloated codes before review. +- Include targeted tests or documented verification for behavior changes, especially in slicing logic, profiles, formats, and GUI defaults.