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
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
Steady13 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
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
Steady19 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
indentation, UTF-8, LF endings, and final newlines as defined in `.editorconfig`. Follow neighboring code and repository ESLint/Stylelint rules. Prettier is configured with single quotes, semicolons, and a 100-character
Steady2 mentions in 2 weeks
No other project uses it yet
46k6y agoReviewMountCreate a free account to mount
bodies, trailers, branch names, PR titles, or PR descriptions.
- PR titles must describe the code change only.
- Before creating or updating commits or PRs, verify that the generated metadata does
No other project uses it yet
46k35d agoReviewMountCreate a free account to mount
package name and export map, never through a
`src` path or path alias. Runtime-specific code sits behind explicit subpaths (`@reactive-resume/pdf/browser`,
`@reactive-resume/pdf/server`, `@reactive-resume/env/server`); root exports
2 projects use a version of this
45k6d agoReviewMountCreate a free account to mount
scripts/`, or `assets/` only when they earn their place.
4. Run the commands and code you document. Scope claims to the release you actually tested
("targets stable GeoPandas
Steady4 mentions in 2 weeks
1 project uses a version of this
45k4mo agoReviewMountCreate a free account to mount
Apache Spark
## Pre-flight Checks
Before the first code edit or running test in a session, ensure a clean working environment. DO NOT skip these checks:
1. Run `git remote
No other project uses it yet
44k46d agoReviewMountCreate a free account to mount
review the maintainer's changes — they are final, not a proposal. Don't thank them for "letting" the maintainer rewrite their work.
- When posting comments with code samples
Steady6 mentions in 2 weeks
No other project uses it yet
44k2mo agoReviewMountCreate a free account to mount