Conversation
Reviewer's GuideAdds Simplified Chinese contributing and security guides under docs/, translates the original project guidance without changing runtime code, and adds bidirectional language links between English and Chinese versions. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path="docs/CONTRIBUTING.zh-Hans.md" line_range="80-81" />
<code_context>
+pnpm eslint "src/**/*.{js,jsx,ts,tsx}" --fix # 修复
+
+# 后端部分(Linux、macOS 或 Windows 上的 Git Bash)
+rustfmt --check src-tauri/src/**/*.rs # 检查
+rustfmt src-tauri/src/**/*.rs # 修复
+
+# 后端部分(Windows PowerShell)
</code_context>
<issue_to_address>
**nitpick:** The manual Rust formatting commands only target `src-tauri/src/**/*.rs` and omit the repository's backend files under `src-tauri/crates/**/*.rs` and `src-tauri/build.rs`, so contributors following this guide can report a clean format check while those Rust files remain unformatted.
**Triggers:** When contributors use the manual formatting commands instead of `pnpm lint-staged`.
**Suggested fix:** Use the repository-wide paths from `package.json` (including `src-tauri/crates/**/*.rs` and `src-tauri/build.rs`), or instruct contributors to run `cargo fmt --all -- --check` from `src-tauri`.
```suggestion
(cd src-tauri && cargo fmt --all -- --check) # 检查
(cd src-tauri && cargo fmt --all) # 修复
```
</issue_to_address>Sourcery assessment
Approved.
There was a problem hiding this comment.
🟡 Changes recommended
Correct the documented Bash glob and PowerShell formatting commands before approval.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Adds Simplified Chinese contributing and security guides with English/Chinese navigation links.
Changes:
- Adds translated contributing and security documentation.
- Preserves project requirements, commands, support versions, and reporting procedures.
- Adds bidirectional language links.
File summaries
| File | Summary |
|---|---|
SECURITY.md |
Adds a link to the Chinese security policy. |
docs/SECURITY.zh-Hans.md |
Adds the Simplified Chinese security policy. |
docs/CONTRIBUTING.zh-Hans.md |
Adds the translated contributing guide; formatting commands need correction. |
CONTRIBUTING.md |
Adds a link to the Chinese contributing guide. |
Review details
Suppressed comments (1)
docs/CONTRIBUTING.zh-Hans.md:81
- These Bash/Git Bash globs are not recursive unless
globstaris enabled, so the commands only cover Rust files one directory belowsrc-tauri/srcand skip the many nested modules (for examplesrc-tauri/src/instance/helpers/...). That can report a clean format check while leaving most of the backend unchecked; use a recursivefind/xargsinvocation or otherwise enable recursive globbing here. The same correction should also be reflected in the English guide for parity.
rustfmt --check src-tauri/src/**/*.rs # 检查
rustfmt src-tauri/src/**/*.rs # 修复
- Files reviewed: 4/4 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
b43aa24 to
c852449
Compare
Checklist
This PR is a ..
Related Issues
N/A — no linked issue.
Description
Add Simplified Chinese versions of the contributing guide and security policy under
docs/, following the localization layout established by the Code of Conduct.The English and Simplified Chinese documents now provide bidirectional language links. The translations keep the development commands, version requirements, contribution rules, supported-version table, vulnerability reporting channel, and response-time expectations aligned with the English originals.
Additional Context
No runtime code is changed. The new documents pass Prettier validation, and their headings, code blocks, tables, relative links, and technical literals were checked against the English originals.
Summary by Sourcery
Provide localized Simplified Chinese contribution and security guidance while keeping the English documentation aligned.
Documentation: