GitHub device auth is a two-command split: `setup --github-start` returns the code fast (foreground, copies to clipboard), `setup --github-poll` waits for authorization (`setup --github` chains both
68k5mo agoReviewMountCreate a free account to mount
supporting analysis (Google Doc, issue, etc.) for complex changes.
---
## Code Style
- Follow [Go CodeReview Comments](https://go.dev/wiki/CodeReviewComments)
and the formatting/style section of
[Go: Best Practices for Production Environments
No other project uses it yet
68k13d agoReviewMountCreate a free account to mount
check its presence to know which rules to apply.
### General rules
- Humans review all the code: always write code that is easy to read and review, following the [`code-style
No other project uses it yet
67k9d agoReviewMountCreate a free account to mount
real before/after (code, output, size, memory or CPU) under **Summary**.
- **Subscribe to every PR you open**; unsubscribe once CI is green and every automated review is handled, or the requester
No other project uses it yet
66k27d agoReviewMountCreate a free account to mount
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
Steady9 mentions in 2 weeks
2 projects use a version of this
66k12d 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
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
65k2mo 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
64k32d 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
64k11d 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
suite minus the multi-arch real build |
| `pnpm test` | Full suite including release workflow |
## Code Conventions
- No Chinese comments in any source (Rust / TypeScript / any file). Comments and identifiers
64k57d 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
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
63k16d 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
Steady6 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
62k12d agoReviewMountCreate a free account to mount
same paths. That only works when it is a service method, not handler code.
- **Contracts.** Anything crossing the wire is typed in `packages/contracts`. Change the schema and the server
61k2d 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
Steady2 mentions in 2 weeks
No other project uses it yet
60k2y agoReviewMountCreate a free account to mount