Skip to content

Adjust AGENTS.md to include testing scope#107

Merged
imnasnainaec merged 1 commit into
mainfrom
agent-testing-scope
Jun 15, 2026
Merged

Adjust AGENTS.md to include testing scope#107
imnasnainaec merged 1 commit into
mainfrom
agent-testing-scope

Conversation

@alex-rawlings-yyc

@alex-rawlings-yyc alex-rawlings-yyc commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

This change is Reviewable

Summary by CodeRabbit

  • Documentation
    • Updated branch coverage guidelines with new subsection clarifying scenarios for excluding specific branches from 100% coverage requirements, including defensive guards, trivial pass-throughs, wiring glue, and logging-only branches.

@coderabbitai

coderabbitai Bot commented Jun 15, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d47c6841-d496-47bd-8969-fc95c25ac782

📥 Commits

Reviewing files that changed from the base of the PR and between b9dec18 and 477a218.

📒 Files selected for processing (1)
  • AGENTS.md

📝 Walkthrough

Walkthrough

AGENTS.md gains a new subsection titled "Branches not worth testing" that enumerates categories of branches eligible for explicit coverage exclusions—such as defensive/unreachable guards, trivial pass-throughs, wiring glue, and logging-only branches—and states the guidance does not justify consolidating tests for branches with real behavior.

Changes

Coverage exclusion documentation

Layer / File(s) Summary
Branches not worth testing section
AGENTS.md
Adds 11 lines defining categories of branches that may be explicitly excluded from the 100% coverage requirement, and clarifies the scope of the guidance.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • sillsdev/interlinearizer-extension#55: Both PRs modify AGENTS.md to add or refine AI agent testing conventions; this PR directly extends that file with the new coverage-exclusion subsection.

Suggested reviewers

  • imnasnainaec

Poem

🐇 Hop, hop, through the branches I go,
Some paths are guarded—no tests need to flow!
Defensive and trivial, logging-only too,
A note in the docs tells the AI what to do.
The warren stays tidy, the coverage stays bright,
Only branches worth testing need coverage tonight! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding content about testing scope to AGENTS.md documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agent-testing-scope

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@alex-rawlings-yyc alex-rawlings-yyc linked an issue Jun 15, 2026 that may be closed by this pull request
@alex-rawlings-yyc alex-rawlings-yyc marked this pull request as ready for review June 15, 2026 16:59

@imnasnainaec imnasnainaec left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@imnasnainaec reviewed 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on alex-rawlings-yyc).

@imnasnainaec imnasnainaec merged commit e25c94b into main Jun 15, 2026
11 checks passed
@imnasnainaec imnasnainaec deleted the agent-testing-scope branch June 15, 2026 18:10
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.

Adjust AGENTS.md to include testing scope

2 participants