chore(vscode): translate the extension's Chinese comments and test names (blocked)

Author: lizhengfeng101Created Sep 18, 2026Updated Sep 18, 2026
Labelsgood first issuehelp wanted

Part of #1419.

Blocked until these open pull requests land, all of which touch the same files: #1379, #1396, #1392, #1381, #1298, #1212. Please feel free to claim this issue now so the work is reserved, but rebase after they merge to avoid a second round of conflicts for their authors.

Scope

Translate the Chinese comments and test names under extensions/vscode/src/extension/ (141 flagged lines across 19 files), then delete the extensions/vscode/ entry from allowedPrefixes in scripts/verify-english-only.go and replace it with a narrow one for the NLS bundle:

go
{"extensions/vscode/package.nls.zh-cn.json", "translated UI copy for the extension manifest"},

That narrow entry is currently redundant (the broad one covers it), which is why it is added here rather than earlier.

Files

Lines File (under extensions/vscode/)
33 src/extension/services/__tests__/gitMap.test.ts
18 src/extension/services/__tests__/cliParse.test.ts
15 src/extension/services/GitService.ts
15 src/extension/services/gitMap.ts
9 src/extension/services/__tests__/CliService.test.ts
8 src/extension/services/shellEnv.ts
6 src/extension/providers/__tests__/lineOffset.test.ts
5 src/extension/providers/commentAnchor.ts
5 src/extension/services/__tests__/ReviewSession.test.ts
5 src/extension/services/__tests__/configParse.test.ts
4 src/extension/services/CliService.ts
4 src/extension/services/__tests__/shellEnv.test.ts
3 src/extension/providers/CommentProvider.ts
3 src/extension/services/__tests__/configDraft.test.ts
3 src/extension/services/cliParse.ts
2 src/extension/services/__tests__/CliService.cancel.test.ts
1 src/extension/commands.ts
1 src/extension/services/ConfigService.ts
1 src/extension/services/configDraft.ts

Done when

  • make check passes with the broad extensions/vscode/ exemption removed.
  • yarn test and yarn lint still pass in extensions/vscode (note: this package is yarn-managed — do not run npm there).
  • package.nls.zh-cn.json is untouched and covered by the narrow exemption.

Source: alibaba/open-code-review