controls that keep reviewers from flagging intentional behavior. After a merge-ready or other codereview verifies a finding, add to that file only when the finding exposes a recurring
Steady16 mentions in 2 weeks
1 project uses a version of this
28k3mo agoReviewMountCreate a free account to mount
with 100% certainty, you can start by adding log statements to the relevant code and reviewing the logs to identify the problem. The log output should contain sufficient information
28k7d agoReviewMountCreate a free account to mount
tests and reviews enforce them.
- `bin/eslint.js` → `lib/cli.js` → `lib/eslint/eslint.js` → `lib/linter/linter.js` → `lib/rules/*.js`
- **`lib/cli.js`** is the only place that reads argv, writes to the console, and sets exit codes
28k11d agoReviewMountCreate a free account to mount
code you edited.
## Never present output you did not run
Every command, hash, exit code and line of output in a commit message, a pull request or an issue
No other project uses it yet
27k14d agoReviewMountCreate a free account to mount
local
secrets and the file exists.
Review the output file for `success`, `score`, and `error` fields. With the default
pass-rate threshold, exit code 0 means the eval
Steady22 mentions in 2 weeks
No other project uses it yet
27k7mo agoReviewMountCreate a free account to mount
Debug Steps pane), so run it from a local build, not the installed tool.
## Codereview
- **A PR review and exploratory testing are different jobs.** A review is scoped
Steady2 mentions in 2 weeks
No other project uses it yet
27k6y agoReviewMountCreate a free account to mount
titles.
## Building
- Make a local build of all packages:
```shell
pnpm build --local
```
## Coding Practices
- **Imports:**
- Always use the `node:` protocol for Node.js built-in imports (e.g., `node:fs`, `node
27k2y agoReviewMountCreate a free account to mount
tell them instead of opening another.
In addition, there are rules inferred from previous codereviews. Follow them like a
reviewer's note: they encode reasons, not ceremony, so when
27k6mo agoReviewMountCreate a free account to mount
open a pull request or link a branch when code is available.
## Pull requests
Before opening a pull request:
1. Review `CONTRIBUTING.md` and follow the repository PR template.
2. Keep
27k23d agoReviewMountCreate a free account to mount
tests that do not exercise the change, padded commit messages) will be closed without review
## CodeReview Guidelines
When reviewing a PR or diff against this repo:
- Look for opportunities
26k3mo agoReviewMountCreate a free account to mount
that covers the change. Use broad wrappers
when touching shared surfaces or before handoff:
- Code tests: `scripts/test/test.sh` (JS, API, Python, then policy).
- Focused runners: `scripts/test/test-js.sh`, `scripts/test/test-api.sh`, `scripts/test/test-docs.sh`,
`scripts/test/test-python.sh`, `scripts/test/test-global.sh`.
`test-python.sh
26k4d agoReviewMountCreate a free account to mount
tightly as possible for a true one-off: ` ` in Markdown/MDX (`// spellchecker:ignore ...` in code), or ` ` for a single line.
Never disable the checker for a whole page.
## Deployment
Deployment
26k3mo agoReviewMountCreate a free account to mount
manually write the pull request in their own words to show they care.
## CodeReview Focus
- **Ignore style/linting issues** in test sample files (`test/*/samples/`) except for `_config.js` files
- Test
26k25d agoReviewMountCreate a free account to mount
tools (Copilot, Claude, Cursor, etc.) to help write or refine code is fine, but a human must understand, review, and take responsibility for every change in the PR.
- **Fully automated
26k9d agoReviewMountCreate a free account to mount
open-source
community, with maintainers who have consistently emphasized **high
standards for codebase readability, codereview, commit discipline,
debuggability, automated testing, tooling, documentation, and all the
other subtle details that
26k2mo agoReviewMountCreate a free account to mount
exception — they review the files, they don't re-read them for context. `ce-code-review`'s project-standards reviewer globs `CODING_STANDARDS.md`, the designated criteria source, and reads `CLAUDE.md`/`AGENTS.md
26k4mo agoReviewMountCreate a free account to mount
After implementation, code generation, formatting, linting, and tests, review the complete change with [biome-code-review](./.agents/skills/biome-code-review/SKILL.md).
- Use a fresh subagent when one is available.
- Provide only the review scope
exception — they review the files, they don't re-read them for context. `ce-code-review`'s project-standards reviewer globs `CODING_STANDARDS.md`, the designated criteria source, and reads `CLAUDE.md`/`AGENTS.md
26k4mo agoReviewMountCreate a free account to mount
strictly a manual step for the human author to confirm they have reviewed the changes.
### Coding and attribution
Only human beings can ever be credited in commit messages. This means
26k17d agoReviewMountCreate a free account to mount
code follows the style doc or is moving toward it. Read the doc in full before your first edit here, and follow it.
- No marker: Core. A person reviews every
26k3mo agoReviewMountCreate a free account to mount