Agent Guidelines for open-code-review
This file provides instructions for AI coding assistants working on this project.
## Development Assistance
Below are rules that a contributor must follow during development
Steady6 mentions in 2 weeks
No other project uses it yet
50k3mo agoReviewMountCreate a free account to mount
package (`gitnexus/skills/`, installed to editor targets
by `gitnexus setup`) and the Claude Code plugin; review also has a standalone Cursor
mirror. `gitnexus/test/unit/shipped-skills-sync.test.ts` guards the copies. Token savings of the workflow
Steady29 mentions in 2 weeks
No other project uses it yet
48k2mo agoReviewMountCreate a free account to mount
incompetent you are. Do it right on your first try, fact-check and review after changes. If you are not sure, ask for help.
- When you see changes made outside
No other project uses it yet
48k5mo agoReviewMountCreate a free account to mount
standalone
runtime smoke test must remain executable on Node 8, and generated library code
must continue to pass the runtime compatibility workflow.
Moment has no runtime dependencies
No other project uses it yet
48k3mo agoReviewMountCreate a free account to mount
selected project in the monorepo with only dependencies that it has
## Security Model
When reviewingcode, writing security documentation, or performing security research, keep in
mind the following aspects
No other project uses it yet
47k6d agoReviewMountCreate a free account to mount
When reviewing provider PRs
Flag any of these patterns:
- `**conn.extra_dejson` or `**self.extra_dejson` spread into a constructor
or call.
- Looping over `conn.extra_dejson.items()` and forwarding every key.
- New code paths
No other project uses it yet
47k6d agoReviewMountCreate a free account to mount
already split into state, actions, and input. Keep it that way.
- **Platform code is isolated.** OS-specific behavior lives in the matching `src/platform/ .rs` file, with only shared traits, types
1 project uses a version of this
47k57d agoReviewMountCreate a free account to mount
cli/src/color.rs`. This module respects the `NO_COLOR` environment variable. Never use hardcoded ANSI color codes.
- CLI flags must always use kebab-case (e.g., `--auto-connect`, `--allow-file-access`). Never
Steady24 mentions in 2 weeks
No other project uses it yet
47k15d agoReviewMountCreate a free account to mount
major features, review applicable compliance requirements, licensing terms, privacy risks, and costs before technical design
2. **Check the premise and existing capabilities:** Inspect the relevant implementation, settings, code snippets, shortcuts
No other project uses it yet
47k29d agoReviewMountCreate a free account to mount
call sites.
Keep `esbuild` listed in `external` in `packages/sf-core/esbuild.js` — bundling esbuild's own code breaks the worker it spawns (see the comment there).
`packages/framework-dist` is an empty shell
No other project uses it yet
47k2mo agoReviewMountCreate a free account to mount
commit, push, or publish merely because implementation is complete.
For PR review, use [code-review](.github/skills/code-review/SKILL.md).
For sandboxed reproductions with `ghmock`, use [ghmock](.github/skills/ghmock/SKILL.md).
For authorized maintainer debt maintenance
No other project uses it yet
47k51d agoReviewMountCreate a free account to mount
Review Comments**: Before requesting re-review, ensure all previous review comments are addressed, not just some of them.
### Code Organization
45. **Move Functions Up**: Move function definitions to the highest
Steady3 mentions in 2 weeks
No other project uses it yet
47k6mo agoReviewMountCreate a free account to mount
Keep only what the change carries, and describe
the code as it stands rather than narrating earlier commits or review rounds. Naming the merged
pull request that caused
1 project uses a version of this
47k3mo agoReviewMountCreate a free account to mount
being implemented, or where direct package-manager behavior is required.
# Background Work
- Long-running code (the shell, daemons, watchers) reacts to events instead of polling with a timer. Polling
Steady6 mentions in 2 weeks
1 project uses a version of this
47k2mo agoReviewMountCreate a free account to mount
changes, run the focused build that can prove it and report anything left unverified.
## CodeReview Rules
### Native interop
- Reject new uses of `DllImport`. Prefer existing interop wrappers or CsWin32
No other project uses it yet
47k2mo agoReviewMountCreate a free account to mount
After finishing, use `ruff format .` and `ruff check .` to format and check the code.
4. When committing, ensure to use conventional commits messages, such as `feat: add new agent
No other project uses it yet
46k26d agoReviewMountCreate a free account to mount
Before PRs, run `npm run validate && npm run test && npm run security:docs`.
## Coding Style & Naming Conventions
Use Markdown for skills and docs, JavaScript/Node for most tooling, and Python
Steady31 mentions in 2 weeks
No other project uses it yet
46k4mo agoReviewMountCreate a free account to mount
review comments |
| `qa-testing-feature` | QA test features using Playwright |
| `reviewing-local-changes` | Codereview for quality, security, best practices, and product/API alignment |
| `simplifying-local-changes` | Simplify and refine code
46k9d agoReviewMountCreate a free account to mount