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
Steady6 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
92k9d agoReviewMountCreate a free account to mount
used, not general-purpose
products.
**This file holds the _rules_: the conventions a reviewer cites against a diff.**
It is loaded in full at the start of every session
Steady2 mentions in 2 weeks
No other project uses it yet
92k17d 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
90k9d agoReviewMountCreate a free account to mount
broken. Decision tree: `docs/ops/HARD_RULES_DETAIL.md` → Rule 18.
**Copilot coverage policy**: When a PR changes production code and coverage is below 60% (statements/lines/functions/branches), do not just report — add or update tests, rerun
Steady35 mentions in 2 weeks
1 project uses a version of this
86k24d agoReviewMountCreate a free account to mount
uses `[...value.trim()].length`, matching Pydantic;
do not use HTML `maxLength`, which counts UTF-16 code units instead.
Custom Agent tool access is edited from the configured names returned
Steady14 mentions in 2 weeks
No other project uses it yet
85k5mo agoReviewMountCreate a free account to mount
later changes after
it is part of the trusted base. Preapproved digests must be code-reviewed in
that first change; after the corresponding file revision lands, promote the
consumed digest
Steady14 mentions in 2 weeks
No other project uses it yet
85k5mo agoReviewMountCreate a free account to mount
skill for placement, naming, test location, and how a large file becomes a directory.
### CodeReview
Before reviewing a PR / diff / branch change, read the **deep-review** skill. Ordinary review
1 project uses a version of this
84k10d 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
Review findings are leads until verified against the relevant design or shipped code and its contracts.
- A review request alone is read-only, as defined under "When
Steady4 mentions in 2 weeks
No other project uses it yet
81k11d agoReviewMountCreate a free account to mount
career-ops project (parent/sibling repos, other codebases on the machine), knowledge from other Claude Code projects on the same machine, and cross-session inferences not written into an in-scope
1 project uses a version of this
81k9d 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
81k9d 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
Notebooks should execute without errors
- README files should be clear and accurate
- Follow existing code patterns in the repository
- Maintain consistency with other lessons
## Additional Notes
### Common Gotchas
1. **Python
Steady33 mentions in 2 weeks
No other project uses it yet
79k4mo agoReviewMountCreate a free account to mount
Notebooks should execute without errors
- README files should be clear and accurate
- Follow existing code patterns in the repository
- Maintain consistency with other lessons
## Additional Notes
### Common Gotchas
1. **Python
Steady33 mentions in 2 weeks
No other project uses it yet
79k4mo agoReviewMountCreate a free account to mount
example for `
- `[Fix] Correct typo in lesson-XX README`
- `[Update] Improve code sample in lesson-XX`
- `[Docs] Update setup instructions`
### Pemeriksaan Wajib
- Notebook harus dijalankan tanpa error
- Berkas README harus
Steady33 mentions in 2 weeks
No other project uses it yet
79k4mo agoReviewMountCreate a free account to mount