@yeepay/coderocket-mcp 1.2.3 → 1.2.4
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/CHANGELOG.md +25 -0
- package/bin/coderocket-mcp +27 -5
- package/dist/banner.js +2 -2
- package/dist/index.js +1 -1
- package/package.json +1 -1
- package/review_logs/review-fc88c611.md +74 -0
package/CHANGELOG.md
CHANGED
@@ -5,6 +5,31 @@ All notable changes to this project will be documented in this file.
|
|
5
5
|
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
|
6
6
|
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
|
7
7
|
|
8
|
+
## [1.2.4] - 2025-08-01
|
9
|
+
|
10
|
+
### 🎨 Enhanced User Experience: Banner Integration
|
11
|
+
|
12
|
+
#### ✨ Added
|
13
|
+
- **Banner in Version Command**: Added beautiful ASCII Art banner to `version` command output
|
14
|
+
- **Banner in Help Command**: Added banner display to `help` command for consistent branding
|
15
|
+
- **Enhanced Version Info**: Version command now shows detailed package information including:
|
16
|
+
- Package version with emoji indicators
|
17
|
+
- Installation path for debugging
|
18
|
+
- NPM package name for reference
|
19
|
+
- Documentation link for easy access
|
20
|
+
|
21
|
+
#### 🔧 Fixed
|
22
|
+
- **Duplicate Help Text**: Removed duplicate "用法" (Usage) line in help command output
|
23
|
+
- **Consistent Branding**: All user-facing commands now display the professional banner
|
24
|
+
- **Better User Experience**: Commands provide more informative and visually appealing output
|
25
|
+
|
26
|
+
#### 🧪 Quality Assurance
|
27
|
+
- **Tested All Commands**: Verified banner display in `version`, `help`, and startup commands
|
28
|
+
- **Error Handling**: Added fallback behavior if banner module fails to load
|
29
|
+
- **No Breaking Changes**: All existing functionality preserved with enhanced presentation
|
30
|
+
|
31
|
+
---
|
32
|
+
|
8
33
|
## [1.2.3] - 2025-08-01
|
9
34
|
|
10
35
|
### 🔧 Critical Dependency Fix: Complete Independence
|
package/bin/coderocket-mcp
CHANGED
@@ -55,10 +55,19 @@ if (!command || command === 'start') {
|
|
55
55
|
case 'help':
|
56
56
|
case '--help':
|
57
57
|
case '-h':
|
58
|
-
|
59
|
-
|
58
|
+
// 显示帮助信息时也显示 banner
|
59
|
+
try {
|
60
|
+
const bannerPath = resolve(__dirname, '../dist/banner.js');
|
61
|
+
const { showMiniBanner } = await import(bannerPath);
|
62
|
+
showMiniBanner();
|
63
|
+
console.log('');
|
64
|
+
} catch (error) {
|
65
|
+
// 如果 banner 加载失败,继续显示帮助信息
|
66
|
+
console.log('CodeRocket MCP - 独立的AI驱动代码审查服务器');
|
67
|
+
console.log('');
|
68
|
+
}
|
60
69
|
|
61
|
-
|
70
|
+
console.log(`用法:
|
62
71
|
npx @yeepay/coderocket-mcp [命令]
|
63
72
|
|
64
73
|
命令:
|
@@ -93,13 +102,26 @@ CodeRocket MCP - 独立的AI驱动代码审查服务器
|
|
93
102
|
case 'version':
|
94
103
|
case '--version':
|
95
104
|
case '-v':
|
96
|
-
//
|
105
|
+
// 显示版本信息时也显示 banner
|
97
106
|
try {
|
107
|
+
// 动态导入 banner 模块
|
108
|
+
const bannerPath = resolve(__dirname, '../dist/banner.js');
|
109
|
+
const { showMiniBanner } = await import(bannerPath);
|
110
|
+
|
111
|
+
showMiniBanner();
|
112
|
+
console.log('');
|
113
|
+
|
114
|
+
// 读取package.json获取详细版本信息
|
98
115
|
const packagePath = resolve(__dirname, '../package.json');
|
99
116
|
const packageJson = JSON.parse(
|
100
117
|
readFileSync(packagePath, 'utf-8')
|
101
118
|
);
|
102
|
-
|
119
|
+
|
120
|
+
console.log(`📦 版本: v${packageJson.version}`);
|
121
|
+
console.log(`🏠 安装路径: ${resolve(__dirname, '..')}`);
|
122
|
+
console.log(`🔗 NPM包: @yeepay/coderocket-mcp`);
|
123
|
+
console.log(`📚 文档: https://github.com/im47cn/coderocket-mcp`);
|
124
|
+
|
103
125
|
} catch (error) {
|
104
126
|
console.error('⚠️ 无法读取版本信息:', error.message);
|
105
127
|
console.error('📁 尝试的路径:', resolve(__dirname, '../package.json'));
|
package/dist/banner.js
CHANGED
@@ -42,10 +42,10 @@ function getVersion() {
|
|
42
42
|
const __dirname = dirname(fileURLToPath(import.meta.url));
|
43
43
|
const packagePath = resolve(__dirname, '../package.json');
|
44
44
|
const packageJson = JSON.parse(readFileSync(packagePath, 'utf-8'));
|
45
|
-
return packageJson.version || '1.2.
|
45
|
+
return packageJson.version || '1.2.4';
|
46
46
|
}
|
47
47
|
catch {
|
48
|
-
return '1.2.
|
48
|
+
return '1.2.4'; // Fallback
|
49
49
|
}
|
50
50
|
}
|
51
51
|
/**
|
package/dist/index.js
CHANGED
@@ -167,7 +167,7 @@ class CodeRocketMCPServer {
|
|
167
167
|
codeRocketService;
|
168
168
|
constructor() {
|
169
169
|
// 读取实际版本号
|
170
|
-
let version = '1.2.
|
170
|
+
let version = '1.2.4'; // 默认版本
|
171
171
|
try {
|
172
172
|
const packagePath = resolve(__dirname, '../package.json');
|
173
173
|
const packageJson = JSON.parse(readFileSync(packagePath, 'utf-8'));
|
package/package.json
CHANGED
@@ -0,0 +1,74 @@
|
|
1
|
+
# Code Review Report for Commit `fc88c611ec9b`
|
2
|
+
|
3
|
+
**Commit Author:** `im47cn <dremabt@gmail.com>`
|
4
|
+
**Commit Date:** `Fri Aug 1 15:41:41 2025 +0800`
|
5
|
+
**Commit Message:**
|
6
|
+
```
|
7
|
+
fix: 移除自引用依赖,确保完全独立
|
8
|
+
|
9
|
+
🔧 关键修复:
|
10
|
+
- 移除 package.json 中的自引用依赖 @yeepay/coderocket-mcp
|
11
|
+
- 修复可能导致安装问题的循环依赖
|
12
|
+
- 确保依赖列表只包含必要的外部包
|
13
|
+
|
14
|
+
✅ 独立性验证:
|
15
|
+
- 系统性排查确认零 coderocket-cli 依赖
|
16
|
+
- 所有 shell 命令都是标准 Git 操作
|
17
|
+
- 使用独立的 ~/.coderocket/ 配置目录
|
18
|
+
- 所有功能通过直接 API 调用实现
|
19
|
+
|
20
|
+
🧪 质量保证:
|
21
|
+
- 100% 测试通过率 (9/9)
|
22
|
+
- Git 变更检测和 AI 审查功能正常
|
23
|
+
- 无破坏性变更,保持向后兼容
|
24
|
+
|
25
|
+
📦 版本发布:
|
26
|
+
- 版本号:v1.2.3
|
27
|
+
- NPM发布:@yeepay/coderocket-mcp@1.2.3
|
28
|
+
- 完全独立的 MCP 服务器
|
29
|
+
```
|
30
|
+
|
31
|
+
---
|
32
|
+
|
33
|
+
## 1. Review Summary
|
34
|
+
|
35
|
+
This is an **excellent** and crucial fix that addresses a critical circular dependency issue. The change is well-executed, and the developer has been exceptionally thorough in updating all related parts of the project, including documentation and versioning.
|
36
|
+
|
37
|
+
**Overall Assessment:** ✅ **Approve**
|
38
|
+
|
39
|
+
---
|
40
|
+
|
41
|
+
## 2. Analysis of Changes
|
42
|
+
|
43
|
+
### 2.1. Dependency Fix (`package.json`, `package-lock.json`)
|
44
|
+
|
45
|
+
- **[Positive]** The primary goal of removing the `@yeepay/coderocket-mcp` self-reference from `dependencies` has been successfully achieved. This resolves a significant packaging flaw that could lead to installation failures and versioning conflicts.
|
46
|
+
- **[Positive]** The `package-lock.json` file has been correctly updated to reflect this change.
|
47
|
+
- **[Positive]** The package version was correctly bumped to `1.2.3`, following semantic versioning for a patch fix.
|
48
|
+
|
49
|
+
### 2.2. Documentation and Consistency (`README.md`, `CHANGELOG.md`, `examples/usage-examples.md`)
|
50
|
+
|
51
|
+
- **[Positive]** The `CHANGELOG.md` is updated with a clear, well-formatted entry for version `1.2.3`. The description of the fix is detailed and easy to understand.
|
52
|
+
- **[Positive]** The `README.md` has been updated to remove incorrect statements about dependencies (e.g., the note about needing `CodeRocket-CLI`). This improves user clarity and accuracy.
|
53
|
+
- **[Positive]** Other documentation files (`examples/usage-examples.md`) have also been updated for consistency.
|
54
|
+
|
55
|
+
### 2.3. Source Code (`src/*.ts`)
|
56
|
+
|
57
|
+
- **[Positive]** Fallback and default version numbers in `src/banner.ts` and `src/index.ts` have been updated to `1.2.3`, ensuring consistency with the `package.json` version.
|
58
|
+
|
59
|
+
---
|
60
|
+
|
61
|
+
## 3. Adherence to Project Guidelines (`CRUSH.md`)
|
62
|
+
|
63
|
+
- **[Positive]** **Code Style & Formatting:** The changes adhere to the project's style.
|
64
|
+
- **[Positive]** **Clarity:** The commit message is exemplary. It is highly detailed, structured, and clearly communicates the purpose, scope, and verification of the change. This level of detail is incredibly helpful for future maintenance.
|
65
|
+
- **[Positive]** **Thoroughness:** The developer demonstrated great attention to detail by updating not just the code, but all user-facing documentation and changelogs.
|
66
|
+
|
67
|
+
---
|
68
|
+
|
69
|
+
## 4. Suggestions for Improvement
|
70
|
+
|
71
|
+
- **None.** This commit is a model example of a high-quality contribution. It is comprehensive, well-documented, and correctly executed.
|
72
|
+
|
73
|
+
---
|
74
|
+
**End of Report**
|