email or GitHub advisory reporting, and do not submit unverified AI-generated security reports.
## Codereview
- Review the entire diff, not just the first files/issues found.
- Flag every occurrence
No other project uses it yet
20k2y agoReviewMountCreate a free account to mount
prompt templates.
- Keep placeholder names, include aliases, and template assumptions synchronized with prompt-loading code and extensions.
- `agent.system.main.communication.native.md` owns native function-call formatting; profiles may override it alongside the legacy
20k8mo agoReviewMountCreate a free account to mount
local/community choice; ask once when it is unknown before creating a new plugin.
- Keep review and contribution discoverable through this entrypoint; do not recreate separate router/debug/review/contribution skills.
- Keep plugin manifest
No other project uses it yet
20k8mo agoReviewMountCreate a free account to mount
useful effect, unnecessary tests,
defensive code, validation, or truncation. Preserve required safety boundaries.
- Validate real behavior end to end and attach results for the reviewed commit.
Use existing tests. Avoid
No other project uses it yet
20k39d agoReviewMountCreate a free account to mount
must also add a corresponding subsection to `paper_index.md`.
When reviewing such PRs, ensure that `paper_index.md` was updated.
### Code duplication and consistency
Trainers in this repository are **self-contained by design
20k2mo agoReviewMountCreate a free account to mount
domains, binary folders, OS allow-list) change only in review. The CI `cargo tree` isolation steps
remain the transitive check.
## Coding standards
- **400 lines at most per hand-written
Surging1 mention in 2 weeks
1 project uses a version of this
19k15d agoReviewMountCreate a free account to mount
involving design decisions) requires a Wasp maintainer's agreement on the approach _before_ the code is written, in a [GitHub issue](https://github.com/wasp-lang/wasp/issues) or in the [`#wasp-dev` channel
19k3mo agoReviewMountCreate a free account to mount
Coding Style
- **Indentation:** 2 spaces for all files (JS/TS/CSS/HTML). No tabs.
- **TypeScript:** All new code should be TypeScript. Strict mode is enabled.
- **Comments:** Provide clear comments for complex logic only
1 project uses a version of this
19k6mo agoReviewMountCreate a free account to mount
pages).
**What this project is NOT:** an Elasticsearch clone or fork. It shares no code and no architecture with Elasticsearch or Lucene. It *also* speaks the ES 8.x wire
Steady49 mentions in 2 weeks
No other project uses it yet
19k4mo agoReviewMountCreate a free account to mount
open a PR for an issue labeled with `plotly-internal`.
2. A human reviews the code before it reaches a pull request. If you cannot see the rendered plot
Steady2 mentions in 2 weeks
No other project uses it yet
18k5mo agoReviewMountCreate a free account to mount
apply it to
`megatron/training` or other training-loop code unless the PR explicitly
opts into that migration.
- In reviews, flag new direct `parallel_state.get_*_group()` usage in
`megatron/core` unless
18k22d agoReviewMountCreate a free account to mount
again until you get it.
4. Implement the tests first. Ask for confirmation and codereview.
5. Once tests are approved, once again suggest your implementation plan for making them
18k22mo agoReviewMountCreate a free account to mount
This file
summarizes; it never overrides it.
## Review guidelines
These rules apply to all codereviews on this repository, including automated
reviewers (Codex, CodeRabbit).
- **Language:** always review in English, regardless
18k53d agoReviewMountCreate a free account to mount
boundary
Every workflow, ownership, branch-enforcement, release, or repository-automation
change requires explicit security review under `MAINTAINERS.md`.
## Workflow rules
- Grant the minimum required `permissions`.
- Pin third-party actions to immutable
No other project uses it yet
18k53d agoReviewMountCreate a free account to mount
APIs. Introduce a Node-only runtime dependency only when the task explicitly requires compatibility code and the owning module already has that role.
- Preserve existing public exports and configuration compatibility
No other project uses it yet
18k53d agoReviewMountCreate a free account to mount
that followed it. Reasoning written as ordinary
prose is not detectable and stays a review judgement. The check is a literal match, which is why
this line describes the marker
No other project uses it yet
18k53d agoReviewMountCreate a free account to mount
payloads out of commits, fixtures, diagnostic logs, and development or review artifacts.
## Read for the task
- For setup, coding conventions, contribution workflow, or release preparation, use the relevant section
18k3mo agoReviewMountCreate a free account to mount