Skip to content

chore: add conservative testing guidelines to AGENTS.md#2330

Merged
samsja merged 1 commit intomainfrom
chore/update-agents-testing-guidelines
Apr 20, 2026
Merged

chore: add conservative testing guidelines to AGENTS.md#2330
samsja merged 1 commit intomainfrom
chore/update-agents-testing-guidelines

Conversation

@samsja
Copy link
Copy Markdown
Member

@samsja samsja commented Apr 19, 2026

Summary

  • Add guidelines to AGENTS.md discouraging adding new tests unless explicitly requested or clearly necessary
  • Editing existing tests is fine; adding new ones should be the exception
  • Only test code with clear, isolated logic (pure functions, abstract classes, data transformations) — not runtime-level code that requires heavy mocking

Note

Low Risk
Low risk: documentation-only changes with no impact on runtime behavior or production systems.

Overview
Adds more conservative testing guidelines to AGENTS.md, emphasizing that new tests should only be added when explicitly requested or clearly necessary.

Clarifies what to test by encouraging tests for isolated logic (e.g., pure functions and data transforms) and discouraging tests that require heavy mocking or exercise framework/runtime glue.

Reviewed by Cursor Bugbot for commit f493187. Bugbot is set up for automated code reviews on this repo. Configure here.

@samsja samsja marked this pull request as ready for review April 19, 2026 21:49
@samsja samsja force-pushed the chore/update-agents-testing-guidelines branch from 4b50171 to f493187 Compare April 19, 2026 21:49
@samsja samsja merged commit e8abfa2 into main Apr 20, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant