Skip to content

Commit 0b3cb69

Browse files
Add learnings from code review analysis (#2280)
Co-authored-by: openhands <openhands@all-hands.dev>
1 parent 8a6a230 commit 0b3cb69

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

AGENTS.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -337,6 +337,7 @@ These are enforced by `check_sdk_api_breakage.py` (runs on release PRs). Depreca
337337
- DON'T write TEST CLASSES unless absolutely necessary!
338338
- If you find yourself duplicating logics in preparing mocks, loading data etc, these logic should be fixtures in conftest.py!
339339
- Please test only the logic implemented in the current codebase. Do not test functionality (e.g., BaseModel.model_dumps()) that is not implemented in this repository.
340+
- For changes to prompt templates, tool descriptions, or agent decision logic, add the `integration-test` label to trigger integration tests and verify no unexpected impact on benchmark performance.
340341

341342
# Behavior Tests
342343

@@ -423,4 +424,5 @@ For examples that use the critic model (e.g., `34_critic_example.py`), the criti
423424
- Ruff ignores `ARG` (unused arguments) under `tests/**/*.py` to allow pytest fixtures.
424425
- Repository guidance lives in `AGENTS.md` (loaded as a third-party skill file).
425426
</REPO_CONFIG_NOTES>
427+
426428
</REPO>

0 commit comments

Comments
 (0)