Codereview instructions
Repo context lives in [CLAUDE.md](../CLAUDE.md) — read it first.
When reviewing a PR, analyze the changes against these criteria:
| Area | What to check |
| ------------------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| Async error handling | uncaught
2k47d agoReviewMountCreate a free account to mount
dune fmt # Auto-format code (run before committing)
dune promote # Accept test output changes (ask user first)
make dev # Full build (bootstraps if necessary)
```
## Reviewing PRs
Follow the repository review
1.9k5mo agoReviewMountCreate a free account to mount
AppSec Mode)
- **Context**: Cacti Takes Security Seriously consider the security implications when reviewing current and new code base
- **Vulnerability Checks**:
- **SQL Injection**: STRICTLY enforce prepared statements (`db_*_prepared`). Flag
1.9k2mo agoReviewMountCreate a free account to mount
CodeReview
- Focus on bugs, security issues, and correctness. Do not flag style nits.
- Do not suggest adding comments, javadocs, or type annotations to unchanged code.
- Do not flag naming
Steady6 mentions in 2 weeks
1 project uses a version of this
1.9k5mo agoReviewMountCreate a free account to mount
entire module name.
## Generated Code — DO NOT suggest changes
When performing a codereview, if a file contains the header `// Code generated by Microsoft (R) Go Code Generator
1.9k7mo agoReviewMountCreate a free account to mount
merge. Please retarget this PR to `main`.
---
## Out of Scope
Do not review or comment on:
- Code formatting that is already enforced by `black` — these are caught by the lint
No other project uses it yet
1.8k3y agoReviewMountCreate a free account to mount
Relaticle CodeReview Guidelines
## Repository Context
Multi-tenant SaaS CRM with paying customers. Tenant isolation is the highest-priority concern. A single cross-tenant data leak is a critical security
Steady2 mentions in 2 weeks
1 project uses a version of this
1.8k14d agoReviewMountCreate a free account to mount
technically valid but inconsistent with the project's convention and suppresses `-Wundef` unnecessarily.
## CodeReview Instructions
When reviewingcode, focus on the following aspects:
- Adherence to coding standards defined
1.8k4mo agoReviewMountCreate a free account to mount
Checks
- ❌ **Flag framework-testing code** - tests should verify component behavior, not framework features
- ✅ Ensure tests cover the actual changes made
- ✅ Verify mobile testing when applicable
## Review Checklist
Before approving, verify
Steady4 mentions in 2 weeks
No other project uses it yet
1.8k4mo agoReviewMountCreate a free account to mount
integration in GraphicsMemory |
> `_M_ARM` / `__arm__` is legacy 32-bit ARM which is deprecated.
## CodeReview Instructions
When reviewingcode, focus on the following aspects:
- Adherence to coding standards defined
1.8k4mo agoReviewMountCreate a free account to mount
rewrite of GNU Emacs. Observable GNU compatibility is a
correctness requirement; read `AGENTS.md` before reviewing.
## Compatibility claims
- The behavioral oracle is GNU Emacs 31.1 at commit
a360712c9d272d950d8d8255ef74570f7e90b7d9
(https://github.com/emacs-mirror/emacs).
1.8k56d agoReviewMountCreate a free account to mount
lives in `AGENTS.md`; the
checks below are the diff-visible rules to apply when reviewing a PR. Everything here
can be judged from the diff text alone.
## Check on every
1.7k3y agoReviewMountCreate a free account to mount
scoped namespaces, system usings first).
- Prefer async/await with `ConfigureAwait(false)` when awaiting inside library code.
- Keep public APIs nullable-enabled and document exceptions and edge cases.
- Treat Spectre.Console renderables
1.7k9d agoReviewMountCreate a free account to mount
concrete review
questions.
## Review Language
Respond in **English**. Be terse. Use code blocks for every actionable diff.
## Review Priorities
### 🔴 CRITICAL (Block merge)
- **Correctness**: logic errors, data corruption risks, race conditions
1.7k53d agoReviewMountCreate a free account to mount
behavior
### PR Requirements
- Include a small unit test demonstrating the change
- Conform to LLVM Coding Standards
- No unrelated changes
- Be an isolated change (split independent changes into separate PRs)
### Commit
1.7k7d agoReviewMountCreate a free account to mount
instead of `# this is a comment`. Ignore
requirement for full sentences and punctuation when reviewingcode.
Wrap comments at 88 characters when writing them. Ignore comment line length when
reviewing
1.7k19mo agoReviewMountCreate a free account to mount
GitHub Copilot Instructions: Agent Skills Expert
Reviewcode changes as an **Agent Skills expert**, focusing on the [Agent Skills open format](https://agentskills.io/home) specification and best practices.
## Core Agent
Steady6 mentions in 2 weeks
No other project uses it yet
1.7k3mo agoReviewMountCreate a free account to mount
scalityCopilot instructions
