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
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
65k46d 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
64k18d 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
64k46d 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
No other project uses it yet
63k55d agoReviewMountCreate a free account to mount
slides untouched also use [`edit-native-pptx`](skills/ppt-master/workflows/edit-native-pptx.md); export must report `rebuilt=0`.
- [`visual-review`](skills/ppt-master/workflows/stages/visual-review.md), [`customize-animations`](skills/ppt-master/workflows/stages/customize-animations.md), and [`generate-audio`](skills/ppt-master/workflows/stages/generate-audio.md) are supporting stages; their trigger rules remain
Steady13 mentions in 2 weeks
No other project uses it yet
63k6mo 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
63k43d 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
62k2d 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
62k51d 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
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
58k2mo agoReviewMountCreate a free account to mount
symlinks expose the
same skills to Claude Code. Current skills cover localization, UI work, core/platform changes, and pre-commit
structural quality review
Steady11 mentions in 2 weeks
No other project uses it yet
58k9d 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
58k17d 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
Steady14 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
instructions for exotic cases — the model usually handles them from context, and the reviewing human can correct it when it does not.
- Write template conditionals as block-level
Coolinginterest down 30% this month
No other project uses it yet
55k42d agoReviewMountCreate a free account to mount