code-assist
Signed-off-by: Your Name <your.email@example.com>
```
### Resolving agent reviewsReview comments from agent bots (e.g., gemini-code-assist) can be outdated or wrong. Always verify their suggestions against
24k6mo agoReviewMountCreate a free account to mount
When adding changes, do NOT break existing behavior.
- Make changes small and easy to review.
- Code and comments must be written in English.
- Prefer root-cause fixes over band-aids
Steady2 mentions in 2 weeks
No other project uses it yet
24k12mo agoReviewMountCreate a free account to mount
local variable properties that lint cannot reliably detect, so they must be caught in codereview.
For full details on the browser platform APIs involved, see
[Shadow DOM and iframes
24k17d agoReviewMountCreate a free account to mount
install` once, then `npm test` or `npm run build`; repeat for `research/` and `index/`.
## Coding Style & Naming Conventions
- Python: 4-space indentation, Black line length 88, Ruff
24k13mo agoReviewMountCreate a free account to mount
post_migrate` signal used to trigger data-migration tasks — see Data Migrations below.)
# Code style
- Use global import statements. Place all imports at the top of the file rather than
24k3mo agoReviewMountCreate a free account to mount
users
Being an open source library, the public API, abstractions, documentation, and the code itself are the product and deserve careful consideration.
When working in this repository, you should consider
23k4mo agoReviewMountCreate a free account to mount
messages in production builds. Use the error codes from `src/rx-error.ts` and add new error codes if needed like `PL1`, `PL2`. Example: `throw newRxError('PL1', { plugin });`
## Documentation Style
- SHOULD use clear
23k5mo agoReviewMountCreate a free account to mount
quality or unreviewed AI content will be closed immediately
All AI-generated code must be thoroughly reviewed, tested, and understood by the contributor before submission. Code should meet
23k13d agoReviewMountCreate a free account to mount
work, including .NET 12 fixes | `main` |
| .NET 11 servicing fixes | Literal `net11`, for review |
| Older servicing, release integration, infrastructure, or documentation | The branch designated for that work; do not guess
23k15d agoReviewMountCreate a free account to mount
cold-build comparison, or maintainer-requested cleanup.
- Run focused tests during development; the final code needs focused tests plus `make check`, with the full suite from
Steady3 mentions in 2 weeks
No other project uses it yet
23k5mo agoReviewMountCreate a free account to mount
rerun affected checks without asking at each step. Continue into PR
publication and review follow-up when requested. A first patch or lifecycle handoff is not completion.
Keep local verification
Steady2 mentions in 2 weeks
No other project uses it yet
23k12d agoReviewMountCreate a free account to mount
review documentation.
This file owns agent-specific documentation routing and workflow.
- Write clear, accurate, task-oriented documentation for developers who run NemoClaw with OpenClaw, Hermes, LangChain Deep Agents Code
Steady2 mentions in 2 weeks
No other project uses it yet
23k12d agoReviewMountCreate a free account to mount
they matter.
- Prefer a direct, focused tool over overlapping projections or orchestration layers. Ordinary codereview, representative harness exercises, and repository hooks are the validation boundary. Do not add repository
Steady2 mentions in 2 weeks
No other project uses it yet
23k12d agoReviewMountCreate a free account to mount
messaging egress opt-in unless accepted product policy states otherwise. Load `nemoclaw-maintainer-security-code-review` when the change affects credentials, public ingress, sender authorization, network policy, or another trust
Steady2 mentions in 2 weeks
No other project uses it yet
23k12d agoReviewMountCreate a free account to mount
node runtime behavior, include smoke + targeted node suite.
### CJS to ESM migration conventions (runtime code)
- Scope: prioritize runtime modules under `lib/`; do not include test fixtures unless explicitly requested
23k5mo agoReviewMountCreate a free account to mount
avoid "Co-Authored-By: Claude" attribution)
- **Pull requests:** Do NOT add "Generated with Claude Code" or similar footers — keep PR descriptions focused on the technical changes
### Preserve Open Pull Request
No other project uses it yet
23k8mo agoReviewMountCreate a free account to mount
repetitive events.
## Testing Standards
- **Regression Tests are Mandatory**: specific regression tests to verify your code works are required for almost all PRs.
- **Follow Test Patterns**: Use `create(:factory)` syntax (FactoryBot
No other project uses it yet
23k4mo agoReviewMountCreate a free account to mount