@yeepay/coderocket-mcp 1.2.5 → 1.2.6
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.
package/dist/toolDefinitions.js
CHANGED
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
import { z } from 'zod';
|
|
2
|
-
import { ReviewCodeRequestSchema, ReviewChangesRequestSchema, ReviewCommitRequestSchema, ReviewFilesRequestSchema, ConfigureAIServiceRequestSchema, } from './types';
|
|
2
|
+
import { ReviewCodeRequestSchema, ReviewChangesRequestSchema, ReviewCommitRequestSchema, ReviewFilesRequestSchema, ConfigureAIServiceRequestSchema, } from './types.js';
|
|
3
3
|
export const toolDefinitions = [
|
|
4
4
|
{
|
|
5
5
|
name: 'review_code',
|
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"toolDefinitions.js","sourceRoot":"","sources":["../src/toolDefinitions.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AACxB,OAAO,EACL,uBAAuB,EACvB,0BAA0B,EAC1B,yBAAyB,EACzB,wBAAwB,EACxB,+BAA+B,GAChC,MAAM,
|
|
1
|
+
{"version":3,"file":"toolDefinitions.js","sourceRoot":"","sources":["../src/toolDefinitions.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AACxB,OAAO,EACL,uBAAuB,EACvB,0BAA0B,EAC1B,yBAAyB,EACzB,wBAAwB,EACxB,+BAA+B,GAChC,MAAM,YAAY,CAAC;AAQpB,MAAM,CAAC,MAAM,eAAe,GAAc;IACxC;QACE,IAAI,EAAE,aAAa;QACnB,WAAW,EAAE;;;;;;;;;;;;;;;;;kCAiBiB;QAC9B,MAAM,EAAE,uBAAuB;KAChC;IACD;QACE,IAAI,EAAE,gBAAgB;QACtB,WAAW,EAAE;;;;;;;;;;;;;;;;;;;;;;;gCAuBe;QAC5B,MAAM,EAAE,0BAA0B;KACnC;IACD;QACE,IAAI,EAAE,eAAe;QACrB,WAAW,EAAE;;;;;;;;;;;;;;;;;gCAiBe;QAC5B,MAAM,EAAE,yBAAyB;KAClC;IACD;QACE,IAAI,EAAE,cAAc;QACpB,WAAW,EAAE;;;;;;;;;;;;;;;;;kCAiBiB;QAC9B,MAAM,EAAE,wBAAwB;KACjC;IACD;QACE,IAAI,EAAE,sBAAsB;QAC5B,WAAW,EAAE;;;;;;;;;;;;;;;;wBAgBO;QACpB,MAAM,EAAE,+BAA+B;KACxC;IACD;QACE,IAAI,EAAE,uBAAuB;QAC7B,WAAW,EAAE;;;;;;;;;;;;;;;;gCAgBe;QAC5B,MAAM,EAAE,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,EAAE,QAAQ;KAC/B;CACF,CAAC"}
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@yeepay/coderocket-mcp",
|
|
3
|
-
"version": "1.2.
|
|
3
|
+
"version": "1.2.6",
|
|
4
4
|
"description": "CodeRocket MCP - Independent AI-powered code review server for Model Context Protocol",
|
|
5
5
|
"main": "dist/index.js",
|
|
6
6
|
"type": "module",
|
|
@@ -45,6 +45,7 @@
|
|
|
45
45
|
"@anthropic-ai/sdk": "^0.27.0",
|
|
46
46
|
"@google/generative-ai": "^0.21.0",
|
|
47
47
|
"@modelcontextprotocol/sdk": "^0.5.0",
|
|
48
|
+
"@yeepay/coderocket-mcp": "^1.2.5",
|
|
48
49
|
"axios": "^1.6.0",
|
|
49
50
|
"zod": "^3.22.4",
|
|
50
51
|
"zod-to-json-schema": "^3.24.6"
|
|
@@ -0,0 +1,33 @@
|
|
|
1
|
+
# Code Review Report for Commit 5767eab8
|
|
2
|
+
|
|
3
|
+
**Commit Author:** im47cn <dremabt@gmail.com>
|
|
4
|
+
**Commit Date:** Fri Aug 1 19:32:16 2025 +0800
|
|
5
|
+
**Commit Message:** fix: 修复ES模块导入路径问题
|
|
6
|
+
|
|
7
|
+
---
|
|
8
|
+
|
|
9
|
+
## 1. Review Summary
|
|
10
|
+
|
|
11
|
+
The commit primarily addresses an ES module import path issue, which is a valid and necessary fix for compatibility. The change in `src/toolDefinitions.ts` aligns with the project's coding standards.
|
|
12
|
+
|
|
13
|
+
However, the changes in `package.json` and `package-lock.json` introduce a self-dependency, which is highly unusual and likely an error. This could lead to problems with package resolution and management.
|
|
14
|
+
|
|
15
|
+
## 2. Changes Analysis
|
|
16
|
+
|
|
17
|
+
### `src/toolDefinitions.ts`
|
|
18
|
+
|
|
19
|
+
* **Change:** The import path for `./types` was changed to `./types.js`.
|
|
20
|
+
* **Assessment:** **Positive**. This change correctly follows the guidelines in `CRUSH.md` which state: "Add a `.js` extension to relative imports". This resolves a runtime module loading error in an ES module context.
|
|
21
|
+
|
|
22
|
+
### `package.json` and `package-lock.json`
|
|
23
|
+
|
|
24
|
+
* **Change:** The package `@yeepay/coderocket-mcp` was added as a dependency to itself.
|
|
25
|
+
* **Assessment:** **Potential Issue**. This is not a standard practice and should be investigated. It might be a mistake made during dependency updates. Self-dependencies can create circular resolution problems and are generally unnecessary.
|
|
26
|
+
|
|
27
|
+
## 3. Actionable Recommendations
|
|
28
|
+
|
|
29
|
+
* **Recommendation:** Revert the changes in `package.json` and `package-lock.json` that introduce the self-dependency. The version bump is acceptable, but the dependency itself should be removed.
|
|
30
|
+
* **Action:** The change in `src/toolDefinitions.ts` is correct and should be kept.
|
|
31
|
+
|
|
32
|
+
---
|
|
33
|
+
*This report was generated automatically.*
|