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
49k3mo agoReviewMountCreate a free account to mount
config.json`, for this TV. Do not edit the repo's docs, tests or other code, and do not open issues or pull requests unless the person asks you to. |
| Change
48k5d 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
No other project uses it yet
48k50d 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
Steady4 mentions in 2 weeks
No other project uses it yet
48k2mo 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
47k3d 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
47k3d 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
47k46d 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
47k19d 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 authorized maintainer debt maintenance, use [tech-debt-burndown](.github/skills/tech-debt-burndown/SKILL.md).
For terminal demonstration evidence
No other project uses it yet
47k40d 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
Steady4 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
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
46k2mo 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
Steady5 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
46k2d agoReviewMountCreate a free account to mount
third person** in descriptions (e.g., "Validates code changes" not "I validate code changes")
- **Keep file references one level deep** from SKILL.md
- **Use sentence casing** for titles and headers - Capitalize only
46k2d agoReviewMountCreate a free account to mount