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
Steady28 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
78k18d agoReviewMountCreate a free account to mount
interview loop for a role, or a review of interview gaps, read
and follow `skills/interview-prep/SKILL.md` before asking anything. Claude
Code also discovers the copy under `.claude/skills/`. The skill asks
Steady2 mentions in 2 weeks
No other project uses it yet
78k12d agoReviewMountCreate a free account to mount
summing per-turn assistant usage**, not `result.usage` (last-turn only in current Claude Code). See `docs/benchmarks/call-sequence-analysis.md`.) The mechanism that drives everything here: **an agent falls back to Read/Grep the instant
Steady5 mentions in 2 weeks
No other project uses it yet
77k16d 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
files, 25 MiB/file, 250 MiB expanded, 50 MiB compressed. Source-only infers `code_review`; source plus a domain infers `live_test`.
- The temporary local archive is always removed. A staged
Steady5 mentions in 2 weeks
No other project uses it yet
74k9mo 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
72k8d 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
REAL `senpi` binary through the drivers in `packages/omo-senpi/scripts/qa/`, which build their own isolated `SENPI_CODING_AGENT_DIR` and IGNORE a caller-provided one, so the real `~/.senpi/agent` is NEVER written
1 project uses a version of this
71k10d agoReviewMountCreate a free account to mount
communicate with the user.
Use short sentences, active voice, and consistent terms. Keep code identifiers,
API names, and quoted source text exact. State the result and any limits clearly.
## Project
No other project uses it yet
70k40d agoReviewMountCreate a free account to mount
your new functionality. Refactor existing code as necessary to make this happen.
Likewise, when reviewingcode, do not hesitate to push back on PRs that would unnecessarily add code
Steady4 mentions in 2 weeks
1 project uses a version of this
69k4mo agoReviewMountCreate a free account to mount