welcome.
### Accountability
- Pure code-agent PRs are **not allowed**. A human submitter must understand and
defend the change end-to-end.
- The submitting human must review every changed line
No other project uses it yet
44k7mo agoReviewMountCreate a free account to mount
doc/BUILD.bazel` and the example `BUILD.bazel` files emit it, because they define test targets. Library code tags select the same steps.
- **`doc_example`** is what doc assets emit: `.py`, `.ipynb`, `.yaml
No other project uses it yet
44k7mo agoReviewMountCreate a free account to mount
Understand what you commit.** AI assistance is fine; shipping code you couldn't have written, debugged, or defended in review is not.
- **A limit change is not done until
44k1d agoReviewMountCreate a free account to mount
template. Trivial autocomplete does not need to be disclosed. All AI-generated code must be self-reviewed.
4. **Format your code before pushing.**
- Backend: `bash scripts/format_backend.sh`
- Frontend: `bash scripts/format_frontend.sh
No other project uses it yet
44k2mo agoReviewMountCreate a free account to mount
runs on every non-draft pull request. The contribution workflow is in [CONTRIBUTING.md](CONTRIBUTING.md).
## CodeReview Rules
### Severity
- P1 is only for problems that produce wrong behavior at runtime: user
Steady7 mentions in 2 weeks
No other project uses it yet
44k42d agoReviewMountCreate a free account to mount
This limit applies only to creating issues; investigate or fix issues fully when asked.
## CodeReview Rules
Check new and modified tests for the common-header and include-group conventions
43k5d agoReviewMountCreate a free account to mount
child crate is not a new workspace dependency. Evaluate new crates by the maintained code and correctness obligations they remove versus their integration, compatibility, security, and upkeep costs; add them
No other project uses it yet
43k6mo agoReviewMountCreate a free account to mount
enforced by the required CI check; do not report commit-signoff findings in codereviews.
- Formatting: yapf (column_limit=119, `.style.yapf`) + flake8 (`.flake8`).
- Always verify changed files pass pre-commit
No other project uses it yet
43k11d agoReviewMountCreate a free account to mount
comments, and do what it asks first (an audit, a discussion) before changing code. A review checks the change against it.
- One concern per pull request.
- A fix comes with
No other project uses it yet
42k41d agoReviewMountCreate a free account to mount
Wikipedia article]: https://en.wikipedia.org/wiki/BitTorrent
## Communication
Use clear, appropriate English for non-native readers.
## CodeReview Guidelines
* Respect project configuration. \
Avoid reading excluded files unless it is strictly necessary
42k2y agoReviewMountCreate a free account to mount
runtime/syntax/testdir`
- Tests for Indent runtime are in `runtime/indent/testdir`
## Common gotchas
- **Distinguish what code enforces from what docs claim.** If a patch
changes documented behavior, say so in the Problem/Solution.
- **Generated
No other project uses it yet
41k9mo agoReviewMountCreate a free account to mount
changes. Keep evidence links current, preserve explicit inference labels, and review map changes with the same owners as the code. Stale context is worse than no context.
## Agent skills
### Issue
Steady7 mentions in 2 weeks
No other project uses it yet
41k4d agoReviewMountCreate a free account to mount
that still passes against broken production code certifies nothing.
**Full reference: `src/test/README.md`** (all frontend code). Review rules: `.github/instructions/frontend-tests.instructions.md`.
- **Stack:** Jest + Testing Library + MSW.
- **Layout:**
- specs are `*.spec.*` in a colocated
Steady7 mentions in 2 weeks
No other project uses it yet
41k4d agoReviewMountCreate a free account to mount
lane is obsolete, preserve its intent and evidence rather than merging stale
code mechanically.
- Agent-owned work lands on current `main`. Each writing agent works in its
own short-lived
41k6d agoReviewMountCreate a free account to mount
dictionary/content spine; do not add page-local language booleans, `isZh`
copy forks, or hard-coded translated page clones. A compatibility exception
must name the external contract that still requires
41k6d agoReviewMountCreate a free account to mount
your contribution works with GitHub Copilot
- [ ] Checked that file names follow the naming convention
### CodeReview Checklist
For instruction files (\*.instructions.md):
- [ ] Has markdown front matter
- [ ] Has non-empty `description` field
41k21d agoReviewMountCreate a free account to mount
voice, formatting rules | `communicationStyle` |
| Working preferences, hard rules | `preferencesAndConstraints` |
| "Tell me about myself" / broad review | All six |
Cross-cutting questions (e.g., "what do I do for team X?") may require
No other project uses it yet
41k21d agoReviewMountCreate a free account to mount
lane is obsolete, preserve its intent and evidence rather than merging stale
code mechanically.
- Agent-owned work uses the canonical checkout on current `main`. Do not
create branches, worktrees, nested
2 projects use a version of this
41k42d agoReviewMountCreate a free account to mount