feedback; defer checks and tests until after implementation where possible.
2. Once the code and self-review are complete, run the relevant tests and fix the code until they pass
Steady5 mentions in 2 weeks
2 projects use a version of this
66k8d agoReviewMountCreate a free account to mount
answer *why*, not *what*.** Comments that restate what the code already says are noise: they go stale and waste review time. Only add a comment when it records
Steady2 mentions in 2 weeks
No other project uses it yet
65k4mo agoReviewMountCreate a free account to mount
only on what it has as well.
- Review your own diff. Remove whatever the change does not need: unused
parameters, dead code, leftover debugging, comments that restate the code.
## Code
No other project uses it yet
65k57d agoReviewMountCreate a free account to mount
response redaction.
Black-box service tests live in `server/api/test/`, unit tests next to the code.
## Change Routing
| Change | Update | Verify |
| --- | --- | --- |
| Go service or router behavior | Handlers under `server/`, rules under
No other project uses it yet
64k29d agoReviewMountCreate a free account to mount
Reviewing translation quality
## Background knowledge for localization workflows
Essential background for the workflows below; understand these concepts before
performing any housekeeping tasks in this document.
### Language code and notation
3 projects use a version of this
64k8d agoReviewMountCreate a free account to mount
narrate change history ("now", "previously", "no longer") and never address the reviewer ("this correctly handles..."). State how the code works, not how it came to be or why the change
No other project uses it yet
64k3mo agoReviewMountCreate a free account to mount
intentionally removes them with evidence.
**Review gate:** for every addition, the reviewer decides whether deleting or changing existing code would have fixed the problem instead — if it would, that
Steady3 mentions in 2 weeks
No other project uses it yet
63k2mo agoReviewMountCreate a free account to mount
requesting maintainer review, verify the current PR tip passes required CI and code coverage and has acceptable reviews from the review bots on the PR, such as Veria and Bugbot
No other project uses it yet
63k12d agoReviewMountCreate a free account to mount
same source of truth. Local-only overrides (`CLAUDE.local.md`, `AGENTS.override.md`, `.claude/settings.local.json`) stay ignored.
## Code Conventions
- No Chinese comments in any source (Rust / TypeScript / any file). Comments and identifiers in English; follow
No other project uses it yet
63k54d agoReviewMountCreate a free account to mount
Writing rules, from Orwell, 1946. These govern prose: docs, PR text, messages. Never touch code or technical terms; swap in everyday words only where precision survives.
1. Never
Steady5 mentions in 2 weeks
No other project uses it yet
62k2mo agoReviewMountCreate a free account to mount
expose the
same skills to Claude Code. Current skills cover localization, UI work, UI performance, core/platform changes, and
pre-commit structural quality review
No other project uses it yet
62k9d agoReviewMountCreate a free account to mount
elegant, smallest change you can. Do not refactor or re-order code unless necessary as this makes review more challenging. Re-use existing test scaffolding, existing or library routines
Steady4 mentions in 2 weeks
No other project uses it yet
61k5mo agoReviewMountCreate a free account to mount
usage.
## Docs
- Reference docs are AsciiDoc in `framework-docs/modules/ROOT`; code snippets for `include-code::` live in `framework-docs/src/main/{java,kotlin}`.
## Commits and pull requests
- Pull requests target `main
No other project uses it yet
60k2y agoReviewMountCreate a free account to mount
intentionally removes them with evidence.
**Review gate:** for every addition, the reviewer decides whether deleting or changing existing code would have fixed the problem instead — if it would, that
No other project uses it yet
58k3mo agoReviewMountCreate a free account to mount
autoloading. Avoid dependencies outside the `utopia-php` ecosystem. Never hardcode credentials — use env vars. Code changes may require a container restart; logs live on the relevant container
No other project uses it yet
58k27d agoReviewMountCreate a free account to mount
Authorship, labels, safety, review, and PR requirements. |
## Runtime entry points
When debugging or changing one integration, begin with its entry point:
| Runtime | Read first |
| --- | --- |
| Claude Code | `.claude-plugin/plugin.json`, `hooks/hooks.json
Steady9 mentions in 2 weeks
1 project uses a version of this
57k5mo agoReviewMountCreate a free account to mount
discussion instead.
- Read the agreed design, constraints, non-goals, and verification plan before changing code.
- Keep the implementation within the issue's agreed scope.
- If implementation reveals a material design
1 project uses a version of this
56k4mo agoReviewMountCreate a free account to mount