Create TDD tests for behavioral changes. Keep tests focused on user behavior and real code paths:
- Use Arrange, Act, Assert structure and clear test data.
- Avoid duplicate cases and duplicate
1 project uses a version of this
93k6d agoReviewMountCreate a free account to mount
programming language), even though it is working.
- You use AI to reply to our codereview comments.
You may use AI tools to understand the codebase and as an assistant
Steady7 mentions in 2 weeks
No other project uses it yet
93k10mo agoReviewMountCreate a free account to mount
home-assistantPipes a download into a shellAGENTS.md
unchecked checkboxes in place so reviewers can see which options were not selected.
## Codereview
- Start review comments with a short, one-sentence summary of the suggested
No other project uses it yet
92k8d agoReviewMountCreate a free account to mount
each skill has a symlink in `.claude/skills/`; add one when you add a skill.
## CodeReview
Before reviewing a pull request or a diff, read the [`code-review` skill](.agents/skills/code-review/SKILL.md
No other project uses it yet
91k11d agoReviewMountCreate a free account to mount
that PR after relevant review or CI failures.
- Keep explicit diagnosis-only requests, standalone codereviews, and pure explanations read-only. Interpret the whole request and the accepted task scope
Steady18 mentions in 2 weeks
No other project uses it yet
90k8d agoReviewMountCreate a free account to mount
follow-up commit. This gives reviewers (and `git bisect`) a
clear before/after and proves the test actually exercises the broken code path.
This applies only to defects that existed before
Steady4 mentions in 2 weeks
No other project uses it yet
84k6mo agoReviewMountCreate a free account to mount
finds the user's files, and it keeps your real pipeline out of it.
## Reviews and scope
`web/` is maintained as first-party: functional and correctness fixes are very
welcome
1 project uses a version of this
81k8d agoReviewMountCreate a free account to mount
Never infer product behavior from page titles or old docs. Verify it in current code.
- Do not preserve a page merely because it already exists. Merge, rewrite, redirect, or remove
Steady34 mentions in 2 weeks
No other project uses it yet
81k24d agoReviewMountCreate a free account to mount
breaker reservations, pooled memory allocations, etc) must be released eventually.
- When writing or reviewing asynchronous code, consult `docs/internal/Liveness.md` for guidance.
## Logging & Error Handling
- Elasticsearch should prefer its own logger `org.elasticsearch.logging.LogManager
No other project uses it yet
79k7mo agoReviewMountCreate a free account to mount
agent answers for what it opens and for its review comments.
2. **Check, then claim.** Before writing code, read the issue comments, search pull requests for `#N` and the keywords
Steady31 mentions in 2 weeks
No other project uses it yet
78k8d agoReviewMountCreate a free account to mount
security issue reporting page](https://grafana.com/legal/report-a-security-issue/) and not directly in this repository.
## CodeReview Rules
### Metrics
- Review metric changes against `contribute/backend/instrumentation.md`, especially registration, duplicate registration, label cardinality
No other project uses it yet
78k17d agoReviewMountCreate a free account to mount
slides untouched also use [`edit-native-pptx`](skills/ppt-master/workflows/edit-native-pptx.md); export must report `rebuilt=0`.
- [`visual-review`](skills/ppt-master/workflows/stages/visual-review.md), [`customize-animations`](skills/ppt-master/workflows/stages/customize-animations.md), and [`generate-audio`](skills/ppt-master/workflows/stages/generate-audio.md) are supporting stages; their trigger rules remain
Steady10 mentions in 2 weeks
No other project uses it yet
75k7mo agoReviewMountCreate a free account to mount
pull requests.
It is the source of truth for contribution scope, coding style, testing, commits, DCO sign-off, review, AI usage, and release notes.
# Pull requests
When opening a pull
No other project uses it yet
72k4mo agoReviewMountCreate a free account to mount
found in `packages/core/src/version.ts`. The next version should increment the patch version by 1.
## Coding style
- Keep things in one function unless they are composable or reusable.
- Do not extract single
Steady8 mentions in 2 weeks
No other project uses it yet
72k7d agoReviewMountCreate a free account to mount
negotiable -- licensing violations can create serious legal issues for the project.
## General Principles
When reviewingcode, don't flag issues that `ansible-test sanity` already catches.
Focus review effort
No other project uses it yet
71k2mo agoReviewMountCreate a free account to mount