@d5render/cli 0.0.90 → 0.1.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
@@ -0,0 +1,43 @@
1
+ ---
2
+ applyTo: "**"
3
+ excludeAgent: ["code-review"]
4
+ ---
5
+
6
+ 1. **MUST** Take the file as the review target:
7
+ - The focus of this review is on the content involved in the diff but the evaluation dimensions need to be based on the file content.
8
+ - When the diff in the discrepancy report differs from the document content, the diff is only used as a line number reference.
9
+ - The actual content that needs to be reviewed should be **based on the document content**.
10
+ 2. Establish context by **reading relevant files**, files that are **imported/used by** the diff files or are **structurally neighboring** them (e.g., related configuration or test files).
11
+ 3. **Ensure the relationship diagram of the diff is complete:**
12
+ - After establishing the context, analyze the complete context of the difference code to ensure a relatively complete code relationship diagram can be built.
13
+ - If the content is incomplete, supplement the context and rebuild the relationship diagram.
14
+ 4. Read the commit message as the basic understanding of the current change
15
+ 5. Determine technology the repository belongs, draw upon your understanding of that technology, provide recommendations on cutting-edge technologies and best practices:
16
+ - **Consult the workspace introduction** yourself.
17
+ - Consult cutting-edge industry documentation yourself.
18
+ 6. **Execute as many relevant commands as possible to enrich your context.**
19
+ 7. **Prioritize Analysis Focus**:
20
+ - For all collected code, meticulously trace the logic to uncover functional bugs and correctness issues.
21
+ - Pay attention to the **correctness** of the analytical logic, the **efficiency** of the code, and its **long-term maintainability**.
22
+ - Concentrate your deepest analysis on the application code (non-test files).
23
+ - Actively consider edge cases, off-by-one errors, race conditions, and improper null/error handling.
24
+ - **focusing on major errors**, identify potential bugs, architectural impact, security vulnerabilities, performance bottlenecks, and clarity issues.
25
+ - focusing on last commit when the same functionality rolling commits occur.
26
+ 8. **Maintain skepticism but verify carefully**:
27
+ - Gather as much context as possible for the code you suspect, raise questions only after **ensuring you have a sufficient understanding** of the business logic.
28
+ - If doubts persist even after establishing a complete context, lower the severity rating of the issue.
29
+ 9. **Strictly** classify the severity according to the content of **file: [severity.instructions.md](./severity.instructions.md)**.
30
+ 10. **Further summary report:**
31
+ - **Tone/Content:** **DO NOT** add comments that:
32
+ - Tell the user to "check," "confirm," "verify," or "ensure" something.
33
+ - Explain what the code change does or validate its purpose.
34
+ - Explain the code to the author (they are assumed to know their own code).
35
+ - Comment on missing trailing newlines or other purely stylistic issues that do not affect code execution or readability in a meaningful way.
36
+ - **Technical Detail:**
37
+ - Pay **meticulous attention to line numbers and indentation** in code suggestions; they **must** be correct and match the surrounding code.
38
+ - **NEVER** comment on license headers, copyright headers, or anything related to future dates/versions (e.g., "this date is in the future").
39
+ - **NEVER** comment on the presence or absence of comments in the code.
40
+ - **Formatting/Structure:**
41
+ - Keep the **change summary** concise (aim for a single sentence).
42
+ - Keep **comment bodies concise** and focused on a single issue.
43
+ - When similar issues occur frequently, please review the code to determine if it meets the change objectives.
@@ -0,0 +1,34 @@
1
+ ---
2
+ applyTo: "**"
3
+ excludeAgent: ["code-review"]
4
+ ---
5
+
6
+ # **Severity Guidelines (for consistent classification):**
7
+
8
+ - **Functional correctness bugs that lead to behavior contrary to the change's intent should generally be classified as HIGH or CRITICAL.**
9
+ - **Issues where input/output validation cannot be performed even after searching all contexts do not require reporting.**
10
+ - **CRITICAL:**
11
+ - security vulnerabilities
12
+ - system-breaking bugs
13
+ - complete logic failure
14
+ - the features already in repository or suspected highly duplicated code
15
+ - **HIGH:**
16
+ - performance bottlenecks (e.g., N+1 queries)
17
+ - resource leaks
18
+ - major architectural violations
19
+ - code style mismatch best practices
20
+ - severe code smell that significantly impairs maintainability.
21
+ - **MEDIUM:**
22
+ - typographical errors in code (not comments)
23
+ - complex logic that could be simplified
24
+ - non-compliant style guide issues (e.g., wrong naming convention).
25
+ - **LOW:**
26
+ - confirmation items for multi-device interaction
27
+ - Refactoring hardcoded values to constants
28
+ - minor log message enhancements
29
+ - comments on docstring/Javadoc expansion
30
+ - typos in documentation (.md files)
31
+ - comments on tests or test quality
32
+ - suppressing unchecked warnings/TODOs.
33
+
34
+ # **Focus on understanding the content of the comment itself, and ignore any inappropriate wording.**
@@ -0,0 +1,9 @@
1
+ ---
2
+ name: code-review
3
+ description: code review suggestions
4
+ ---
5
+
6
+ 1. read file [review.instructions.md](./review.instructions.md).
7
+ 2. find all Markdown resumes within the project and their basic understanding of the project.
8
+ 3. based on the understanding of review to supplemented.
9
+ 4. run...
@@ -0,0 +1 @@
1
+ 0.0.0
package/CHANGELOG.md ADDED
@@ -0,0 +1,9 @@
1
+ ## 0.1.1 (2026-01-08)
2
+ skills:review 0.0.0
3
+
4
+ 评审流程转移至skills内
5
+ 增加去重功能
6
+
7
+
8
+ ## 0.1.0 (2026-01-07)
9
+ 基础评审能力
package/README.md ADDED
@@ -0,0 +1,44 @@
1
+ # install
2
+
3
+ 请运行 npm run setup 进行项目初始化
4
+
5
+ 整体使用 ts 进行开发,会进行一轮打包,最终 CI 运行的是 bin/d5cli 文件
6
+
7
+ # dev
8
+
9
+ CI MCP 的入口是 [server.ts](.\copilot\server\index.ts) 会经历一轮打包进入 dist (只需要启动 server)
10
+
11
+ 开发的一些注意点:
12
+
13
+ 1. [copilot.config.js](.\copilot\server\config.ts) 属于公共配置,有需要可以放这里
14
+ 2. [vscode](.\vscode\index.ts) 插件的 client 需要单独开发
15
+ 3. 本地功能调试 gpt-5-mini 食用最佳🤣,效果测试建议切回当前模型(开发环境已内置)
16
+ 4. 返回 Promise 之后会丢失开发过程中的类型支持,registerTool 调用时可先不写 Promise 看 MCP Server 支持的返回参数
17
+ 5. 其他用例参考 https://github.com/modelcontextprotocol/typescript-sdk/tree/main/src/examples
18
+
19
+ copilot\CHANGELOG.md:有功能变更的时候写这里,将会
20
+ 1. 跟随发包时写进 根CHANGELOG.md
21
+ 2. 包第一次安装后会进行通知
22
+
23
+ # debug
24
+
25
+ 项目初始化会多出来 Standalone 的调试配置。
26
+
27
+ 可点其进行开发过程中的调试
28
+ win下linux调试建议:
29
+ 1. 安装 wsl
30
+ - 系统:`wsl --install`
31
+ -
32
+ ```
33
+ # wsl内安装nvm
34
+ curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.40.3/install.sh | bash
35
+
36
+ # 代替重启 shell
37
+ \. "$HOME/.nvm/nvm.sh"
38
+
39
+ # 安装 Node
40
+ # 安装 pnpm
41
+ ```
42
+ - 插件 WSL (已在插件建议中)
43
+ 2. git worktree 创建linux分支(或者直接靠代码)
44
+ 3. ide 基于wsl启动(mnt是默认磁盘位置)
@@ -0,0 +1,4 @@
1
+ skills:review 0.0.0
2
+
3
+ 评审流程转移至skills内
4
+ 增加去重功能