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
Steady2 mentions in 2 weeks
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
47k8d 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
47k8d 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
47k59d 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
47k17d 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
47k32d 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
47k53d 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
Steady5 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
Steady6 mentions in 2 weeks
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
46k29d 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
Steady25 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
Steady4 mentions in 2 weeks
No other project uses it yet
46k12d 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
Steady4 mentions in 2 weeks
No other project uses it yet
46k12d agoReviewMountCreate a free account to mount
template
3. Create PR titled `[spec] Feature name` (keep in Draft until ready for review)
## Spec Guidelines
### Problem First, Solution Second
Never start with what you want to build. Start
Steady4 mentions in 2 weeks
No other project uses it yet
46k12d agoReviewMountCreate a free account to mount
user requests like PR review, branch audit, and Warden follow-up.
- If a downstream review harness controls the final response shape, do not hardcode a competing output format
No other project uses it yet
46k5mo agoReviewMountCreate a free account to mount