chore(idea): translate the Chinese comments in services/
Author: lizhengfeng101Created Sep 18, 2026Updated Sep 18, 2026
Labelsgood first issuehelp wanted
Part of #1419. Ready to start — no open pull request touches extensions/idea/.
Scope
Translate the Chinese comments under
extensions/idea/src/main/kotlin/com/alibaba/opencodereview/idea/services/
(252 flagged lines across 11 files). Leave allowedPrefixes alone; #1424 removes the extensions/idea/ entry once every idea sub-issue is merged.
Files
| Lines | File (under .../idea/services/) |
|---|---|
| 81 | GitService.kt |
| 45 | CliService.kt |
| 29 | GitMap.kt |
| 25 | ConfigService.kt |
| 22 | ShellEnv.kt |
| 18 | ConfigDraft.kt |
| 7 | ConfigParse.kt |
| 7 | ConfigUtils.kt |
| 6 | CliParse.kt |
| 6 | Providers.kt |
| 6 | ReviewSession.kt |
ShellEnv.kt and CliService.kt carry the reasoning about PATH resolution under nvm and
about process-tree termination. Translate that reasoning, do not summarise it away.
Done when
- Temporarily dropping the
extensions/idea/exemption no longer reports any file underservices/. ./gradlew test -PskipFrontend=truepasses inextensions/idea.
Source: alibaba/open-code-review