@metago-ai/dev-kit 1.1.0 → 1.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.
- package/LICENSE +21 -21
- package/README.md +103 -103
- package/index.js +56 -56
- package/package.json +59 -59
- package/skills/metago-architecture-design/SKILL.md +114 -114
- package/skills/metago-code-review-deep/SKILL.md +103 -103
- package/skills/metago-refactor-suggest/SKILL.md +116 -116
- package/skills/metago-security-audit/SKILL.md +178 -178
|
@@ -1,114 +1,114 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: "metago-architecture-design"
|
|
3
|
-
description: "架构设计技能。根据业务需求生成完整系统架构方案,含架构模式选择、模块分解、接口契约、数据流设计、技术选型依据。遵循高内聚低耦合、可扩展、可维护原则,输出含决策依据的架构文档。"
|
|
4
|
-
version: "1.0.0"
|
|
5
|
-
author: "MetaGO"
|
|
6
|
-
category: "科技开发"
|
|
7
|
-
platforms: ["cursor","claude-code","codex","trae","codebuddy","qoder","zcode","workbuddy","qclaw","ima"]
|
|
8
|
-
trigger:
|
|
9
|
-
- "帮我设计架构"
|
|
10
|
-
- "系统架构设计"
|
|
11
|
-
- "技术选型方案"
|
|
12
|
-
- "模块分解设计"
|
|
13
|
-
- "生成ADR"
|
|
14
|
-
---
|
|
15
|
-
|
|
16
|
-
# 架构设计(metago-architecture-design)
|
|
17
|
-
|
|
18
|
-
## 描述
|
|
19
|
-
根据业务需求生成完整系统架构方案,覆盖架构模式选择、模块分解、接口契约设计、数据流规划、技术选型。输出含决策依据和权衡分析的架构文档。
|
|
20
|
-
|
|
21
|
-
## 触发条件
|
|
22
|
-
- 用户请求架构设计、系统设计、技术选型时激活
|
|
23
|
-
- 新项目启动阶段
|
|
24
|
-
- 系统重构/迁移规划阶段
|
|
25
|
-
- 与 `metago-action-plan` 协同:重大技术决策时强制触发
|
|
26
|
-
|
|
27
|
-
## 核心流程
|
|
28
|
-
|
|
29
|
-
### 1. 需求分解
|
|
30
|
-
- 功能性需求拆解(核心用例、边缘场景)
|
|
31
|
-
- 非功能性需求量化(QPS、延迟、可用性、数据规模)
|
|
32
|
-
- 约束条件识别(预算、团队规模、技术栈、合规要求)
|
|
33
|
-
- 演进路径规划(MVP → 完整版 → 规模化)
|
|
34
|
-
|
|
35
|
-
### 2. 架构模式选择
|
|
36
|
-
- 评估候选架构模式(单体/微服务/事件驱动/分层/CQRS)
|
|
37
|
-
- 基于需求约束的权衡分析
|
|
38
|
-
- 架构决策记录(ADR - Architecture Decision Record)
|
|
39
|
-
- 替代方案与降级策略
|
|
40
|
-
|
|
41
|
-
### 3. 模块分解
|
|
42
|
-
- 领域驱动设计(DDD)边界划分
|
|
43
|
-
- 模块职责定义(单一职责原则)
|
|
44
|
-
- 模块间依赖关系(无循环依赖)
|
|
45
|
-
- 接口契约定义(输入/输出/异常)
|
|
46
|
-
|
|
47
|
-
### 4. 数据流设计
|
|
48
|
-
- 数据模型设计(ER 图、字段约束)
|
|
49
|
-
- 数据流向(同步/异步、推/拉)
|
|
50
|
-
- 一致性策略(强一致/最终一致)
|
|
51
|
-
- 缓存策略与失效机制
|
|
52
|
-
|
|
53
|
-
### 5. 技术选型
|
|
54
|
-
- 技术栈评估(成熟度、社区、性能、人才市场)
|
|
55
|
-
- 选型依据记录(为什么选 A 不选 B)
|
|
56
|
-
- 风险评估(供应商锁定、废弃风险)
|
|
57
|
-
- 替代方案储备
|
|
58
|
-
|
|
59
|
-
## 架构原则检查清单
|
|
60
|
-
|
|
61
|
-
- [ ] 高内聚低耦合(模块内聚度 ≥ 0.8,耦合度 ≤ 0.3)
|
|
62
|
-
- [ ] 单一职责(每个模块只做一件事)
|
|
63
|
-
- [ ] 开闭原则(对扩展开放,对修改关闭)
|
|
64
|
-
- [ ] 可独立部署(模块可独立部署和回滚)
|
|
65
|
-
- [ ] 可水平扩展(无状态服务可水平扩展)
|
|
66
|
-
- [ ] 故障隔离(单点故障不影响全局)
|
|
67
|
-
- [ ] 可观测性(日志、指标、追踪完备)
|
|
68
|
-
|
|
69
|
-
## 输出格式
|
|
70
|
-
|
|
71
|
-
```
|
|
72
|
-
【架构设计文档】
|
|
73
|
-
|
|
74
|
-
## 1. 需求摘要
|
|
75
|
-
- 核心用例:...
|
|
76
|
-
- 性能指标:QPS=X, 延迟≤Xms, 可用性≥X%
|
|
77
|
-
- 约束条件:...
|
|
78
|
-
|
|
79
|
-
## 2. 架构概览
|
|
80
|
-
架构模式:[模式名]
|
|
81
|
-
选择依据:[权衡分析]
|
|
82
|
-
替代方案:[方案 B, 方案 C]
|
|
83
|
-
|
|
84
|
-
## 3. 模块设计
|
|
85
|
-
| 模块 | 职责 | 接口 | 依赖 |
|
|
86
|
-
|------|------|------|------|
|
|
87
|
-
| ... | ... | ... | ... |
|
|
88
|
-
|
|
89
|
-
## 4. 数据流
|
|
90
|
-
[数据流向图描述]
|
|
91
|
-
一致性策略:...
|
|
92
|
-
|
|
93
|
-
## 5. 技术选型
|
|
94
|
-
| 层次 | 技术 | 选型依据 | 替代方案 |
|
|
95
|
-
|------|------|----------|----------|
|
|
96
|
-
| ... | ... | ... | ... |
|
|
97
|
-
|
|
98
|
-
## 6. 演进路径
|
|
99
|
-
- MVP:...
|
|
100
|
-
- V1:...
|
|
101
|
-
- 规模化:...
|
|
102
|
-
|
|
103
|
-
## 7. 风险与缓解
|
|
104
|
-
| 风险 | 概率 | 影响 | 缓解措施 |
|
|
105
|
-
|------|------|------|----------|
|
|
106
|
-
| ... | ... | ... | ... |
|
|
107
|
-
```
|
|
108
|
-
|
|
109
|
-
## 与其他技能的协同
|
|
110
|
-
- 与 `metago-action-plan` 协同:架构设计输出联动行动计划生成
|
|
111
|
-
- 与 `metago-decision-eval` 协同:架构决策需通过决策评估
|
|
112
|
-
- 与 `metago-critique` 协同:架构方案需通过 L1-L5 批判性分析
|
|
113
|
-
- 与 `metago-whatif` 协同:反事实推演验证架构韧性
|
|
114
|
-
- 与 `metago-coupling-optimize` 协同:模块耦生度量化优化
|
|
1
|
+
---
|
|
2
|
+
name: "metago-architecture-design"
|
|
3
|
+
description: "架构设计技能。根据业务需求生成完整系统架构方案,含架构模式选择、模块分解、接口契约、数据流设计、技术选型依据。遵循高内聚低耦合、可扩展、可维护原则,输出含决策依据的架构文档。"
|
|
4
|
+
version: "1.0.0"
|
|
5
|
+
author: "MetaGO"
|
|
6
|
+
category: "科技开发"
|
|
7
|
+
platforms: ["cursor","claude-code","codex","trae","codebuddy","qoder","zcode","workbuddy","qclaw","ima"]
|
|
8
|
+
trigger:
|
|
9
|
+
- "帮我设计架构"
|
|
10
|
+
- "系统架构设计"
|
|
11
|
+
- "技术选型方案"
|
|
12
|
+
- "模块分解设计"
|
|
13
|
+
- "生成ADR"
|
|
14
|
+
---
|
|
15
|
+
|
|
16
|
+
# 架构设计(metago-architecture-design)
|
|
17
|
+
|
|
18
|
+
## 描述
|
|
19
|
+
根据业务需求生成完整系统架构方案,覆盖架构模式选择、模块分解、接口契约设计、数据流规划、技术选型。输出含决策依据和权衡分析的架构文档。
|
|
20
|
+
|
|
21
|
+
## 触发条件
|
|
22
|
+
- 用户请求架构设计、系统设计、技术选型时激活
|
|
23
|
+
- 新项目启动阶段
|
|
24
|
+
- 系统重构/迁移规划阶段
|
|
25
|
+
- 与 `metago-action-plan` 协同:重大技术决策时强制触发
|
|
26
|
+
|
|
27
|
+
## 核心流程
|
|
28
|
+
|
|
29
|
+
### 1. 需求分解
|
|
30
|
+
- 功能性需求拆解(核心用例、边缘场景)
|
|
31
|
+
- 非功能性需求量化(QPS、延迟、可用性、数据规模)
|
|
32
|
+
- 约束条件识别(预算、团队规模、技术栈、合规要求)
|
|
33
|
+
- 演进路径规划(MVP → 完整版 → 规模化)
|
|
34
|
+
|
|
35
|
+
### 2. 架构模式选择
|
|
36
|
+
- 评估候选架构模式(单体/微服务/事件驱动/分层/CQRS)
|
|
37
|
+
- 基于需求约束的权衡分析
|
|
38
|
+
- 架构决策记录(ADR - Architecture Decision Record)
|
|
39
|
+
- 替代方案与降级策略
|
|
40
|
+
|
|
41
|
+
### 3. 模块分解
|
|
42
|
+
- 领域驱动设计(DDD)边界划分
|
|
43
|
+
- 模块职责定义(单一职责原则)
|
|
44
|
+
- 模块间依赖关系(无循环依赖)
|
|
45
|
+
- 接口契约定义(输入/输出/异常)
|
|
46
|
+
|
|
47
|
+
### 4. 数据流设计
|
|
48
|
+
- 数据模型设计(ER 图、字段约束)
|
|
49
|
+
- 数据流向(同步/异步、推/拉)
|
|
50
|
+
- 一致性策略(强一致/最终一致)
|
|
51
|
+
- 缓存策略与失效机制
|
|
52
|
+
|
|
53
|
+
### 5. 技术选型
|
|
54
|
+
- 技术栈评估(成熟度、社区、性能、人才市场)
|
|
55
|
+
- 选型依据记录(为什么选 A 不选 B)
|
|
56
|
+
- 风险评估(供应商锁定、废弃风险)
|
|
57
|
+
- 替代方案储备
|
|
58
|
+
|
|
59
|
+
## 架构原则检查清单
|
|
60
|
+
|
|
61
|
+
- [ ] 高内聚低耦合(模块内聚度 ≥ 0.8,耦合度 ≤ 0.3)
|
|
62
|
+
- [ ] 单一职责(每个模块只做一件事)
|
|
63
|
+
- [ ] 开闭原则(对扩展开放,对修改关闭)
|
|
64
|
+
- [ ] 可独立部署(模块可独立部署和回滚)
|
|
65
|
+
- [ ] 可水平扩展(无状态服务可水平扩展)
|
|
66
|
+
- [ ] 故障隔离(单点故障不影响全局)
|
|
67
|
+
- [ ] 可观测性(日志、指标、追踪完备)
|
|
68
|
+
|
|
69
|
+
## 输出格式
|
|
70
|
+
|
|
71
|
+
```
|
|
72
|
+
【架构设计文档】
|
|
73
|
+
|
|
74
|
+
## 1. 需求摘要
|
|
75
|
+
- 核心用例:...
|
|
76
|
+
- 性能指标:QPS=X, 延迟≤Xms, 可用性≥X%
|
|
77
|
+
- 约束条件:...
|
|
78
|
+
|
|
79
|
+
## 2. 架构概览
|
|
80
|
+
架构模式:[模式名]
|
|
81
|
+
选择依据:[权衡分析]
|
|
82
|
+
替代方案:[方案 B, 方案 C]
|
|
83
|
+
|
|
84
|
+
## 3. 模块设计
|
|
85
|
+
| 模块 | 职责 | 接口 | 依赖 |
|
|
86
|
+
|------|------|------|------|
|
|
87
|
+
| ... | ... | ... | ... |
|
|
88
|
+
|
|
89
|
+
## 4. 数据流
|
|
90
|
+
[数据流向图描述]
|
|
91
|
+
一致性策略:...
|
|
92
|
+
|
|
93
|
+
## 5. 技术选型
|
|
94
|
+
| 层次 | 技术 | 选型依据 | 替代方案 |
|
|
95
|
+
|------|------|----------|----------|
|
|
96
|
+
| ... | ... | ... | ... |
|
|
97
|
+
|
|
98
|
+
## 6. 演进路径
|
|
99
|
+
- MVP:...
|
|
100
|
+
- V1:...
|
|
101
|
+
- 规模化:...
|
|
102
|
+
|
|
103
|
+
## 7. 风险与缓解
|
|
104
|
+
| 风险 | 概率 | 影响 | 缓解措施 |
|
|
105
|
+
|------|------|------|----------|
|
|
106
|
+
| ... | ... | ... | ... |
|
|
107
|
+
```
|
|
108
|
+
|
|
109
|
+
## 与其他技能的协同
|
|
110
|
+
- 与 `metago-action-plan` 协同:架构设计输出联动行动计划生成
|
|
111
|
+
- 与 `metago-decision-eval` 协同:架构决策需通过决策评估
|
|
112
|
+
- 与 `metago-critique` 协同:架构方案需通过 L1-L5 批判性分析
|
|
113
|
+
- 与 `metago-whatif` 协同:反事实推演验证架构韧性
|
|
114
|
+
- 与 `metago-coupling-optimize` 协同:模块耦生度量化优化
|
|
@@ -1,103 +1,103 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: "metago-code-review-deep"
|
|
3
|
-
description: "深度代码审查技能。对代码进行多维度分级审查(Critical/Major/Minor/Info),覆盖逻辑正确性、安全漏洞、性能瓶颈、可维护性、风格一致性。基于元构批判性分析(metago-critique)和事实核查(metago-fact-check)能力,输出含修复建议的结构化审查报告。"
|
|
4
|
-
version: "1.0.0"
|
|
5
|
-
author: "MetaGO"
|
|
6
|
-
category: "科技开发"
|
|
7
|
-
platforms: ["cursor","claude-code","codex","trae","codebuddy","qoder","zcode","workbuddy","qclaw","ima"]
|
|
8
|
-
trigger:
|
|
9
|
-
- "帮我做代码审查"
|
|
10
|
-
- "审查这段代码"
|
|
11
|
-
- "Code Review"
|
|
12
|
-
- "检查代码质量"
|
|
13
|
-
- "PR Review"
|
|
14
|
-
---
|
|
15
|
-
|
|
16
|
-
# 深度代码审查(metago-code-review-deep)
|
|
17
|
-
|
|
18
|
-
## 描述
|
|
19
|
-
对代码进行多维度分级审查,检测逻辑错误、安全漏洞、性能问题、可维护性缺陷、风格不一致。输出含修复建议和优先级的结构化报告。
|
|
20
|
-
|
|
21
|
-
## 触发条件
|
|
22
|
-
- 用户请求代码审查、PR Review、Code Review 时激活
|
|
23
|
-
- 代码提交前审查(pre-commit hook 场景)
|
|
24
|
-
- 与 `metago-decision-lock` 协同:开发者任务输出代码时强制触发
|
|
25
|
-
|
|
26
|
-
## 核心流程
|
|
27
|
-
|
|
28
|
-
### 1. 静态结构分析
|
|
29
|
-
- 文件组织与模块划分是否合理
|
|
30
|
-
- 函数/类职责是否单一
|
|
31
|
-
- 依赖关系是否清晰(无循环依赖)
|
|
32
|
-
- 代码重复度检测(DRY 原则)
|
|
33
|
-
|
|
34
|
-
### 2. 逻辑正确性审查
|
|
35
|
-
- 边界条件处理(空值、极值、越界)
|
|
36
|
-
- 异常路径覆盖(错误处理、资源释放)
|
|
37
|
-
- 并发安全(竞态条件、死锁)
|
|
38
|
-
- 算法正确性(时间复杂度、空间复杂度)
|
|
39
|
-
|
|
40
|
-
### 3. 安全漏洞审查
|
|
41
|
-
- 注入漏洞(SQL/命令/XSS)
|
|
42
|
-
- 认证授权缺陷
|
|
43
|
-
- 敏感信息泄露(硬编码密钥、日志泄密)
|
|
44
|
-
- 依赖漏洞(已知 CVE)
|
|
45
|
-
|
|
46
|
-
### 4. 性能审查
|
|
47
|
-
- 数据库查询效率(N+1、全表扫描)
|
|
48
|
-
- 内存泄漏风险
|
|
49
|
-
- 不必要的同步阻塞
|
|
50
|
-
- 资源未释放(文件句柄、连接池)
|
|
51
|
-
|
|
52
|
-
### 5. 可维护性审查
|
|
53
|
-
- 命名清晰度(变量、函数、类)
|
|
54
|
-
- 注释完整度(复杂逻辑、公共 API)
|
|
55
|
-
- 单一职责原则遵循度
|
|
56
|
-
- 开闭原则遵循度
|
|
57
|
-
|
|
58
|
-
## 分级标准
|
|
59
|
-
|
|
60
|
-
| 级别 | 含义 | 处理时机 |
|
|
61
|
-
|------|------|----------|
|
|
62
|
-
| 🔴 Critical | 必须立即修复(安全漏洞、数据丢失、崩溃) | 阻断合并 |
|
|
63
|
-
| 🟠 Major | 应在合并前修复(逻辑错误、性能严重问题) | 阻断合并 |
|
|
64
|
-
| 🟡 Minor | 建议修复(可维护性、风格) | 可后续修复 |
|
|
65
|
-
| 🔵 Info | 提示性建议(优化、最佳实践) | 仅记录 |
|
|
66
|
-
|
|
67
|
-
## 输出格式
|
|
68
|
-
|
|
69
|
-
```
|
|
70
|
-
【深度代码审查报告】
|
|
71
|
-
|
|
72
|
-
文件:path/to/file.ext
|
|
73
|
-
审查范围:函数/类/模块
|
|
74
|
-
总体评分:XX/100
|
|
75
|
-
|
|
76
|
-
🔴 Critical(N 项)
|
|
77
|
-
[C-001] 行 XX: 问题描述
|
|
78
|
-
影响:...
|
|
79
|
-
修复建议:...
|
|
80
|
-
依据:...
|
|
81
|
-
|
|
82
|
-
🟠 Major(N 项)
|
|
83
|
-
[M-001] 行 XX: 问题描述
|
|
84
|
-
影响:...
|
|
85
|
-
修复建议:...
|
|
86
|
-
|
|
87
|
-
🟡 Minor(N 项)
|
|
88
|
-
[m-001] 行 XX: 问题描述
|
|
89
|
-
建议改进:...
|
|
90
|
-
|
|
91
|
-
🔵 Info(N 项)
|
|
92
|
-
[i-001] 建议优化:...
|
|
93
|
-
|
|
94
|
-
审查结论:✅ 通过 / ❌ 需修复后重新审查
|
|
95
|
-
```
|
|
96
|
-
|
|
97
|
-
## 与其他技能的协同
|
|
98
|
-
- 与 `metago-critique` 协同:审查过程应用 L1-L5 批判性分析
|
|
99
|
-
- 与 `metago-fact-check` 协同:检测虚构 API、不存在的函数(幻觉代码)
|
|
100
|
-
- 与 `metago-decision-lock` 协同:审查报告需通过决策锁校验
|
|
101
|
-
- 与 `metago-problem-trace` 协同:问题无限溯源至根本原因
|
|
102
|
-
- 与 `metago-security-audit` 协同:安全维度联动深度安全审计
|
|
103
|
-
- 与 `metago-refactor-suggest` 协同:可维护性问题联动重构建议
|
|
1
|
+
---
|
|
2
|
+
name: "metago-code-review-deep"
|
|
3
|
+
description: "深度代码审查技能。对代码进行多维度分级审查(Critical/Major/Minor/Info),覆盖逻辑正确性、安全漏洞、性能瓶颈、可维护性、风格一致性。基于元构批判性分析(metago-critique)和事实核查(metago-fact-check)能力,输出含修复建议的结构化审查报告。"
|
|
4
|
+
version: "1.0.0"
|
|
5
|
+
author: "MetaGO"
|
|
6
|
+
category: "科技开发"
|
|
7
|
+
platforms: ["cursor","claude-code","codex","trae","codebuddy","qoder","zcode","workbuddy","qclaw","ima"]
|
|
8
|
+
trigger:
|
|
9
|
+
- "帮我做代码审查"
|
|
10
|
+
- "审查这段代码"
|
|
11
|
+
- "Code Review"
|
|
12
|
+
- "检查代码质量"
|
|
13
|
+
- "PR Review"
|
|
14
|
+
---
|
|
15
|
+
|
|
16
|
+
# 深度代码审查(metago-code-review-deep)
|
|
17
|
+
|
|
18
|
+
## 描述
|
|
19
|
+
对代码进行多维度分级审查,检测逻辑错误、安全漏洞、性能问题、可维护性缺陷、风格不一致。输出含修复建议和优先级的结构化报告。
|
|
20
|
+
|
|
21
|
+
## 触发条件
|
|
22
|
+
- 用户请求代码审查、PR Review、Code Review 时激活
|
|
23
|
+
- 代码提交前审查(pre-commit hook 场景)
|
|
24
|
+
- 与 `metago-decision-lock` 协同:开发者任务输出代码时强制触发
|
|
25
|
+
|
|
26
|
+
## 核心流程
|
|
27
|
+
|
|
28
|
+
### 1. 静态结构分析
|
|
29
|
+
- 文件组织与模块划分是否合理
|
|
30
|
+
- 函数/类职责是否单一
|
|
31
|
+
- 依赖关系是否清晰(无循环依赖)
|
|
32
|
+
- 代码重复度检测(DRY 原则)
|
|
33
|
+
|
|
34
|
+
### 2. 逻辑正确性审查
|
|
35
|
+
- 边界条件处理(空值、极值、越界)
|
|
36
|
+
- 异常路径覆盖(错误处理、资源释放)
|
|
37
|
+
- 并发安全(竞态条件、死锁)
|
|
38
|
+
- 算法正确性(时间复杂度、空间复杂度)
|
|
39
|
+
|
|
40
|
+
### 3. 安全漏洞审查
|
|
41
|
+
- 注入漏洞(SQL/命令/XSS)
|
|
42
|
+
- 认证授权缺陷
|
|
43
|
+
- 敏感信息泄露(硬编码密钥、日志泄密)
|
|
44
|
+
- 依赖漏洞(已知 CVE)
|
|
45
|
+
|
|
46
|
+
### 4. 性能审查
|
|
47
|
+
- 数据库查询效率(N+1、全表扫描)
|
|
48
|
+
- 内存泄漏风险
|
|
49
|
+
- 不必要的同步阻塞
|
|
50
|
+
- 资源未释放(文件句柄、连接池)
|
|
51
|
+
|
|
52
|
+
### 5. 可维护性审查
|
|
53
|
+
- 命名清晰度(变量、函数、类)
|
|
54
|
+
- 注释完整度(复杂逻辑、公共 API)
|
|
55
|
+
- 单一职责原则遵循度
|
|
56
|
+
- 开闭原则遵循度
|
|
57
|
+
|
|
58
|
+
## 分级标准
|
|
59
|
+
|
|
60
|
+
| 级别 | 含义 | 处理时机 |
|
|
61
|
+
|------|------|----------|
|
|
62
|
+
| 🔴 Critical | 必须立即修复(安全漏洞、数据丢失、崩溃) | 阻断合并 |
|
|
63
|
+
| 🟠 Major | 应在合并前修复(逻辑错误、性能严重问题) | 阻断合并 |
|
|
64
|
+
| 🟡 Minor | 建议修复(可维护性、风格) | 可后续修复 |
|
|
65
|
+
| 🔵 Info | 提示性建议(优化、最佳实践) | 仅记录 |
|
|
66
|
+
|
|
67
|
+
## 输出格式
|
|
68
|
+
|
|
69
|
+
```
|
|
70
|
+
【深度代码审查报告】
|
|
71
|
+
|
|
72
|
+
文件:path/to/file.ext
|
|
73
|
+
审查范围:函数/类/模块
|
|
74
|
+
总体评分:XX/100
|
|
75
|
+
|
|
76
|
+
🔴 Critical(N 项)
|
|
77
|
+
[C-001] 行 XX: 问题描述
|
|
78
|
+
影响:...
|
|
79
|
+
修复建议:...
|
|
80
|
+
依据:...
|
|
81
|
+
|
|
82
|
+
🟠 Major(N 项)
|
|
83
|
+
[M-001] 行 XX: 问题描述
|
|
84
|
+
影响:...
|
|
85
|
+
修复建议:...
|
|
86
|
+
|
|
87
|
+
🟡 Minor(N 项)
|
|
88
|
+
[m-001] 行 XX: 问题描述
|
|
89
|
+
建议改进:...
|
|
90
|
+
|
|
91
|
+
🔵 Info(N 项)
|
|
92
|
+
[i-001] 建议优化:...
|
|
93
|
+
|
|
94
|
+
审查结论:✅ 通过 / ❌ 需修复后重新审查
|
|
95
|
+
```
|
|
96
|
+
|
|
97
|
+
## 与其他技能的协同
|
|
98
|
+
- 与 `metago-critique` 协同:审查过程应用 L1-L5 批判性分析
|
|
99
|
+
- 与 `metago-fact-check` 协同:检测虚构 API、不存在的函数(幻觉代码)
|
|
100
|
+
- 与 `metago-decision-lock` 协同:审查报告需通过决策锁校验
|
|
101
|
+
- 与 `metago-problem-trace` 协同:问题无限溯源至根本原因
|
|
102
|
+
- 与 `metago-security-audit` 协同:安全维度联动深度安全审计
|
|
103
|
+
- 与 `metago-refactor-suggest` 协同:可维护性问题联动重构建议
|
|
@@ -1,116 +1,116 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: "metago-refactor-suggest"
|
|
3
|
-
description: "重构建议技能。检测代码异味(Code Smell)、量化技术债务、识别重构机会,生成含优先级和影响范围的重构方案。遵循行为保持变换原则,确保重构不改变外部行为。输出含前后对比的重构报告。"
|
|
4
|
-
version: "1.0.0"
|
|
5
|
-
author: "MetaGO"
|
|
6
|
-
category: "科技开发"
|
|
7
|
-
platforms: ["cursor","claude-code","codex","trae","codebuddy","qoder","zcode","workbuddy","qclaw","ima"]
|
|
8
|
-
trigger:
|
|
9
|
-
- "帮我重构代码"
|
|
10
|
-
- "检测代码异味"
|
|
11
|
-
- "技术债务评估"
|
|
12
|
-
- "代码优化建议"
|
|
13
|
-
- "重构方案生成"
|
|
14
|
-
---
|
|
15
|
-
|
|
16
|
-
# 重构建议(metago-refactor-suggest)
|
|
17
|
-
|
|
18
|
-
## 描述
|
|
19
|
-
检测代码异味、量化技术债务、识别重构机会,生成含优先级和影响范围的重构方案。遵循行为保持变换(Behavior Preservation)原则,确保重构不改变外部行为。
|
|
20
|
-
|
|
21
|
-
## 触发条件
|
|
22
|
-
- 用户请求重构、代码优化、技术债务评估时激活
|
|
23
|
-
- 代码审查发现可维护性问题时联动触发
|
|
24
|
-
- 定期技术债务盘点
|
|
25
|
-
- 与 `metago-code-review-deep` 协同:审查发现 Minor/Major 问题时触发
|
|
26
|
-
|
|
27
|
-
## 核心流程
|
|
28
|
-
|
|
29
|
-
### 1. 代码异味检测
|
|
30
|
-
- 长函数(行数 > 阈值,默认 50 行)
|
|
31
|
-
- 大类(字段/方法数 > 阈值)
|
|
32
|
-
- 重复代码(相似度 > 80%)
|
|
33
|
-
- 过长参数列表(参数数 > 5)
|
|
34
|
-
- 发散式修改(一个类因多个原因修改)
|
|
35
|
-
- 霰弹式修改(一个修改引发多处变更)
|
|
36
|
-
- 特性依恋(方法调用大量其他类)
|
|
37
|
-
- 数据泥团(多个变量总是一起出现)
|
|
38
|
-
|
|
39
|
-
### 2. 复杂度量分析
|
|
40
|
-
- 圈复杂度(Cyclomatic Complexity)> 10 需重构
|
|
41
|
-
- 认知复杂度(Cognitive Complexity)> 15 需重构
|
|
42
|
-
- 继承深度(DIT)> 4 需关注
|
|
43
|
-
- 耦合度(CBO)> 10 需优化
|
|
44
|
-
|
|
45
|
-
### 3. 依赖分析
|
|
46
|
-
- 循环依赖检测
|
|
47
|
-
- 不稳定依赖(依赖不稳定模块)
|
|
48
|
-
- 上层依赖下层(违反依赖倒置)
|
|
49
|
-
- 过度耦合(一个模块依赖过多模块)
|
|
50
|
-
|
|
51
|
-
### 4. 重构机会识别
|
|
52
|
-
| 异味类型 | 推荐重构手法 | 优先级 | 影响范围 |
|
|
53
|
-
|----------|-------------|--------|----------|
|
|
54
|
-
| 长函数 | Extract Method | 高 | 函数级 |
|
|
55
|
-
| 重复代码 | Extract Method / Pull Up Method | 高 | 类级 |
|
|
56
|
-
| 大类 | Extract Class | 中 | 模块级 |
|
|
57
|
-
| 过长参数 | Introduce Parameter Object | 低 | 函数级 |
|
|
58
|
-
| 特性依恋 | Move Method | 中 | 类级 |
|
|
59
|
-
| 循环依赖 | Extract Class / Move Method | 高 | 模块级 |
|
|
60
|
-
|
|
61
|
-
### 5. 重构方案生成
|
|
62
|
-
- 重构步骤分解(原子化操作)
|
|
63
|
-
- 行为保持验证(测试覆盖要求)
|
|
64
|
-
- 风险评估(回归概率)
|
|
65
|
-
- 前后对比(复杂度、可读性、可维护性指标)
|
|
66
|
-
|
|
67
|
-
## 输出格式
|
|
68
|
-
|
|
69
|
-
```
|
|
70
|
-
【重构建议报告】
|
|
71
|
-
|
|
72
|
-
## 1. 技术债务概览
|
|
73
|
-
债务等级:🟢 低 / 🟡 中 / 🔴 高
|
|
74
|
-
债务量化:预估 N 人天
|
|
75
|
-
热点模块:path/to/module
|
|
76
|
-
|
|
77
|
-
## 2. 代码异味清单
|
|
78
|
-
| # | 异味 | 位置 | 严重度 | 推荐手法 |
|
|
79
|
-
|---|------|------|--------|----------|
|
|
80
|
-
| 1 | 长函数(85行) | file.ext:L10-L95 | 🟡 Major | Extract Method |
|
|
81
|
-
| 2 | 重复代码(92%相似) | file.ext:L20 / file2.ext:L15 | 🔴 Critical | Extract Method |
|
|
82
|
-
| ... | ... | ... | ... | ... |
|
|
83
|
-
|
|
84
|
-
## 3. 重构方案(按优先级排序)
|
|
85
|
-
|
|
86
|
-
### 方案1:[重构名称]
|
|
87
|
-
- 目标:...
|
|
88
|
-
- 步骤:
|
|
89
|
-
1. ...
|
|
90
|
-
2. ...
|
|
91
|
-
3. ...
|
|
92
|
-
- 测试要求:覆盖率达 X%
|
|
93
|
-
- 风险评估:低 / 中 / 高
|
|
94
|
-
- 预期收益:复杂度 15→8,可读性 +40%
|
|
95
|
-
|
|
96
|
-
### 方案2:...
|
|
97
|
-
|
|
98
|
-
## 4. 重构前后对比
|
|
99
|
-
| 指标 | 重构前 | 重构后 | 改善 |
|
|
100
|
-
|------|--------|--------|------|
|
|
101
|
-
| 圈复杂度 | 18 | 6 | -67% |
|
|
102
|
-
| 重复率 | 35% | 5% | -86% |
|
|
103
|
-
| 可维护性 | 4.2/10 | 8.1/10 | +93% |
|
|
104
|
-
|
|
105
|
-
## 5. 建议执行顺序
|
|
106
|
-
1. 先补充测试(当前覆盖率 X%)
|
|
107
|
-
2. 执行方案1(低风险高收益)
|
|
108
|
-
3. 执行方案2(...)
|
|
109
|
-
```
|
|
110
|
-
|
|
111
|
-
## 与其他技能的协同
|
|
112
|
-
- 与 `metago-code-review-deep` 协同:审查发现可维护性问题时触发重构建议
|
|
113
|
-
- 与 `metago-coupling-optimize` 协同:耦生度量化指导重构优先级
|
|
114
|
-
- 与 `metago-critique` 协同:重构方案需通过批判性分析
|
|
115
|
-
- 与 `metago-decision-lock` 协同:重构执行前需通过决策锁校验
|
|
116
|
-
- 与 `metago-fact-check` 协同:验证重构前后行为一致性
|
|
1
|
+
---
|
|
2
|
+
name: "metago-refactor-suggest"
|
|
3
|
+
description: "重构建议技能。检测代码异味(Code Smell)、量化技术债务、识别重构机会,生成含优先级和影响范围的重构方案。遵循行为保持变换原则,确保重构不改变外部行为。输出含前后对比的重构报告。"
|
|
4
|
+
version: "1.0.0"
|
|
5
|
+
author: "MetaGO"
|
|
6
|
+
category: "科技开发"
|
|
7
|
+
platforms: ["cursor","claude-code","codex","trae","codebuddy","qoder","zcode","workbuddy","qclaw","ima"]
|
|
8
|
+
trigger:
|
|
9
|
+
- "帮我重构代码"
|
|
10
|
+
- "检测代码异味"
|
|
11
|
+
- "技术债务评估"
|
|
12
|
+
- "代码优化建议"
|
|
13
|
+
- "重构方案生成"
|
|
14
|
+
---
|
|
15
|
+
|
|
16
|
+
# 重构建议(metago-refactor-suggest)
|
|
17
|
+
|
|
18
|
+
## 描述
|
|
19
|
+
检测代码异味、量化技术债务、识别重构机会,生成含优先级和影响范围的重构方案。遵循行为保持变换(Behavior Preservation)原则,确保重构不改变外部行为。
|
|
20
|
+
|
|
21
|
+
## 触发条件
|
|
22
|
+
- 用户请求重构、代码优化、技术债务评估时激活
|
|
23
|
+
- 代码审查发现可维护性问题时联动触发
|
|
24
|
+
- 定期技术债务盘点
|
|
25
|
+
- 与 `metago-code-review-deep` 协同:审查发现 Minor/Major 问题时触发
|
|
26
|
+
|
|
27
|
+
## 核心流程
|
|
28
|
+
|
|
29
|
+
### 1. 代码异味检测
|
|
30
|
+
- 长函数(行数 > 阈值,默认 50 行)
|
|
31
|
+
- 大类(字段/方法数 > 阈值)
|
|
32
|
+
- 重复代码(相似度 > 80%)
|
|
33
|
+
- 过长参数列表(参数数 > 5)
|
|
34
|
+
- 发散式修改(一个类因多个原因修改)
|
|
35
|
+
- 霰弹式修改(一个修改引发多处变更)
|
|
36
|
+
- 特性依恋(方法调用大量其他类)
|
|
37
|
+
- 数据泥团(多个变量总是一起出现)
|
|
38
|
+
|
|
39
|
+
### 2. 复杂度量分析
|
|
40
|
+
- 圈复杂度(Cyclomatic Complexity)> 10 需重构
|
|
41
|
+
- 认知复杂度(Cognitive Complexity)> 15 需重构
|
|
42
|
+
- 继承深度(DIT)> 4 需关注
|
|
43
|
+
- 耦合度(CBO)> 10 需优化
|
|
44
|
+
|
|
45
|
+
### 3. 依赖分析
|
|
46
|
+
- 循环依赖检测
|
|
47
|
+
- 不稳定依赖(依赖不稳定模块)
|
|
48
|
+
- 上层依赖下层(违反依赖倒置)
|
|
49
|
+
- 过度耦合(一个模块依赖过多模块)
|
|
50
|
+
|
|
51
|
+
### 4. 重构机会识别
|
|
52
|
+
| 异味类型 | 推荐重构手法 | 优先级 | 影响范围 |
|
|
53
|
+
|----------|-------------|--------|----------|
|
|
54
|
+
| 长函数 | Extract Method | 高 | 函数级 |
|
|
55
|
+
| 重复代码 | Extract Method / Pull Up Method | 高 | 类级 |
|
|
56
|
+
| 大类 | Extract Class | 中 | 模块级 |
|
|
57
|
+
| 过长参数 | Introduce Parameter Object | 低 | 函数级 |
|
|
58
|
+
| 特性依恋 | Move Method | 中 | 类级 |
|
|
59
|
+
| 循环依赖 | Extract Class / Move Method | 高 | 模块级 |
|
|
60
|
+
|
|
61
|
+
### 5. 重构方案生成
|
|
62
|
+
- 重构步骤分解(原子化操作)
|
|
63
|
+
- 行为保持验证(测试覆盖要求)
|
|
64
|
+
- 风险评估(回归概率)
|
|
65
|
+
- 前后对比(复杂度、可读性、可维护性指标)
|
|
66
|
+
|
|
67
|
+
## 输出格式
|
|
68
|
+
|
|
69
|
+
```
|
|
70
|
+
【重构建议报告】
|
|
71
|
+
|
|
72
|
+
## 1. 技术债务概览
|
|
73
|
+
债务等级:🟢 低 / 🟡 中 / 🔴 高
|
|
74
|
+
债务量化:预估 N 人天
|
|
75
|
+
热点模块:path/to/module
|
|
76
|
+
|
|
77
|
+
## 2. 代码异味清单
|
|
78
|
+
| # | 异味 | 位置 | 严重度 | 推荐手法 |
|
|
79
|
+
|---|------|------|--------|----------|
|
|
80
|
+
| 1 | 长函数(85行) | file.ext:L10-L95 | 🟡 Major | Extract Method |
|
|
81
|
+
| 2 | 重复代码(92%相似) | file.ext:L20 / file2.ext:L15 | 🔴 Critical | Extract Method |
|
|
82
|
+
| ... | ... | ... | ... | ... |
|
|
83
|
+
|
|
84
|
+
## 3. 重构方案(按优先级排序)
|
|
85
|
+
|
|
86
|
+
### 方案1:[重构名称]
|
|
87
|
+
- 目标:...
|
|
88
|
+
- 步骤:
|
|
89
|
+
1. ...
|
|
90
|
+
2. ...
|
|
91
|
+
3. ...
|
|
92
|
+
- 测试要求:覆盖率达 X%
|
|
93
|
+
- 风险评估:低 / 中 / 高
|
|
94
|
+
- 预期收益:复杂度 15→8,可读性 +40%
|
|
95
|
+
|
|
96
|
+
### 方案2:...
|
|
97
|
+
|
|
98
|
+
## 4. 重构前后对比
|
|
99
|
+
| 指标 | 重构前 | 重构后 | 改善 |
|
|
100
|
+
|------|--------|--------|------|
|
|
101
|
+
| 圈复杂度 | 18 | 6 | -67% |
|
|
102
|
+
| 重复率 | 35% | 5% | -86% |
|
|
103
|
+
| 可维护性 | 4.2/10 | 8.1/10 | +93% |
|
|
104
|
+
|
|
105
|
+
## 5. 建议执行顺序
|
|
106
|
+
1. 先补充测试(当前覆盖率 X%)
|
|
107
|
+
2. 执行方案1(低风险高收益)
|
|
108
|
+
3. 执行方案2(...)
|
|
109
|
+
```
|
|
110
|
+
|
|
111
|
+
## 与其他技能的协同
|
|
112
|
+
- 与 `metago-code-review-deep` 协同:审查发现可维护性问题时触发重构建议
|
|
113
|
+
- 与 `metago-coupling-optimize` 协同:耦生度量化指导重构优先级
|
|
114
|
+
- 与 `metago-critique` 协同:重构方案需通过批判性分析
|
|
115
|
+
- 与 `metago-decision-lock` 协同:重构执行前需通过决策锁校验
|
|
116
|
+
- 与 `metago-fact-check` 协同:验证重构前后行为一致性
|