@ohos-cpf/3rdloop 0.0.2 → 0.0.3

package/vendor/Server/Skills/flutter-code-check/assets/OUTPUT_TEMPLATE.md

@@ -1,312 +1,312 @@
-# 检视报告模板
-
-使用此模板生成 Flutter PR 代码检视报告，保存至 `{outputDir}/check-report.md`。
-
----
-
-```markdown
-# Flutter 代码检视报告
-
-## 一、基本信息
-
-| 字段 | 内容 |
-|------|------|
-| **PR 地址** | {prUrl} |
-| **代码仓库** | {owner}/{repo} |
-| **目标分支** | {targetBranch} |
-| **PR 标题** | {prTitle} |
-| **提交作者** | {author} <{email}> |
-| **提交时间** | {date} |
-| **Commit Hash** | {commitHash} |
-| **库类型** | Flutter |
-| **报告生成时间** | {YYYY-MM-DD HH:mm} |
-
----
-
-## 二、改动概述
-
-### 2.1 改动规模
-
-| 指标 | 数值 |
-|------|------|
-| 涉及文件数 | N 个 |
-| 新增行数 | +X 行 |
-| 删除行数 | -Y 行 |
-| 净变更 | +Z 行 |
-
-### 2.2 改动文件清单
-
-| 文件路径 | 类型 | 改动性质 | 改动摘要 |
-|----------|------|----------|----------|
-| `ohos/src/main/ets/...` | ArkTS | 功能修复 | 修复 Plugin 生命周期释放逻辑 |
-| `lib/...` | Dart | 接口更新 | 新增平台判断逻辑 |
-
-### 2.3 整体改动意图
-
-{根据 commit message 和 diff 内容描述改动意图}
-
----
-
-## 三、检视结果汇总
-
-| 等级 | 问题数 |
-|------|--------|
-| 🔴 严重 (CRITICAL) | N |
-| 🟠 主要 (MAJOR)    | N |
-| 🟡 次要 (MINOR)    | N |
-| 🔵 建议 (SUGGEST)  | N |
-
----
-
-## 四、详细检视意见
-
-### [文件路径]
-
-#### 🟠 主要问题
-
-**[问题标题]**
-- **位置**：第 X 行
-- **描述**：...
-- **建议**：...
-
-```diff
-- 问题代码
-+ 建议改法
-```
-
----
-
-## 五、跨平台一致性分析
-
-> PR 中仅包含鸿蒙代码提交，需从克隆的仓库中读取 `android/` 和 `ios/` 的已有实现进行对比。
-
-### 5.1 对比基准
-
-| 字段 | 内容 |
-|------|------|
-| **Android 实现文件** | `{android/.../XxxPlugin.java}` |
-| **iOS 实现文件** | `{ios/Classes/XxxPlugin.m}` |
-| **OHOS 实现文件** | `{ohos/src/main/ets/.../XxxPlugin.ets}` |
-
-### 5.2 方法路由一致性
-
-| # | 方法名 | Android 文件:行号 | OHOS 文件:行号 | 状态 | 说明 |
-|---|--------|------------------|---------------|------|------|
-| 1 | `methodName1` | `Plugin.java:56` | `Plugin.ets:89` | ✅ 一致 | 两端均已实现 |
-| 2 | `methodName2` | `Plugin.java:78` | — | ❌ 缺失 | OHOS 未实现此方法，Dart 侧将收到 MissingPluginException |
-
-### 5.3 参数类型映射一致性
-
-| # | 方法名 | 参数名 | Android 类型 | OHOS 类型 | 状态 | 说明 |
-|---|--------|--------|-------------|----------|------|------|
-| 1 | `methodName` | `param1` | `boolean` | `boolean` | ✅ 一致 | |
-| 2 | `methodName` | `param2` | `int` | `string` ❌ | ❌ 致命 | 类型映射错误导致功能失效 |
-
-### 5.4 返回值与错误码一致性
-
-| # | 方法名 | Android 返回 | OHOS 返回 | 错误码对齐 | 状态 | 说明 |
-|---|--------|-------------|----------|-----------|------|------|
-| 1 | `methodName` | `Map<String, Object>` | `Record<string, Object>` | ✅ 一致 | ✅ 一致 | |
-| 2 | `methodName` | `result.success(data)` | `result.success(fixedValue)` | ❌ 不一致 | ❌ Stub | OHOS 返回固定值，未实际执行逻辑 |
-
-### 5.5 跨平台一致性总结
-
-| 检查项 | 结果 |
-|--------|------|
-| 方法路由完整性 | {缺失 N 个方法} |
-| 参数类型正确性 | {N 个致命类型错误} |
-| 返回值格式一致性 | {N 个不一致} |
-| 错误码对齐 | {N 个不一致} |
-| Stub 方法 | {N 个 Stub 实现} |
-
----
-
-## 六、Dart 改动必要性分析
-
-> **核心原则**：鸿蒙适配 PR 中 Dart 代码（`lib/` 目录）原则上不应修改。若 PR 包含 Dart 变更，必须逐一论证必要性和跨平台影响。
-
-### 6.1 Dart 变更总览
-
-| 指标 | 数值 |
-|------|------|
-| Dart 变更文件数 | N 个 |
-| Dart 变更总行数 | +X / -Y |
-| 必要改动数 | N 处 |
-| 不应修改数 | N 处 |
-| 需评估数 | N 处 |
-
-### 6.2 逐项必要性分析
-
-| # | 文件 | 行号 | 变更类型 | 变更摘要 | 必要性判定 | 问题等级 | 影响平台 | 说明 |
-|---|------|------|----------|----------|------------|----------|----------|------|
-| 1 | `lib/xxx.dart` | L42 | 新增 | 新增 `Platform.isOHOS` 条件分支 | ✅ 必要 | — | 无 | 鸿蒙平台注册逻辑，仅 ohos 分支 |
-| 2 | `lib/xxx.dart` | L78 | 修改 | 修改 `invokeMethod` 参数结构 | ❌ 不应修改 | 🔴 Critical | android/iOS | 应在 ohos 侧适配参数解析 |
-| 3 | `lib/xxx.dart` | L105 | 新增 | 新增 `import 'dart:io'` | ⚠️ 需评估 | 🟡 Minor | Web | 可能影响 Web 平台编译 |
-
-### 6.3 跨平台影响评估
-
-| # | Dart 变更 | 影响平台 | 影响类型 | 影响描述 | 严重程度 |
-|---|-----------|----------|----------|----------|----------|
-| 1 | 修改 `invokeMethod` 参数 | android/iOS | 运行时 | 参数结构变更导致 android 侧解析失败 | 🔴 Critical |
-| 2 | 新增 `dart:io` import | Web | 编译时 | Web 平台不支持 `dart:io`，编译报错 | 🟠 Major |
-
-### 6.4 替代方案建议
-
-| # | 原始变更 | 问题描述 | 替代方案 | 优势 |
-|---|----------|----------|----------|------|
-| 1 | 修改 `MethodChannel` 名称 | android 侧无法识别新 Channel 名 | 在 ohos 侧注册与 android 相同的 Channel 名 | 不影响 Dart 和其他平台 |
-| 2 | 修改 `invokeMethod` 参数 | 破坏 android 侧参数解析 | 在 ohos 侧 `onMethodCall` 中适配参数格式 | 仅 ohos 侧改动 |
-
-### 6.5 Dart 改动结论
-
-| 结论 | 说明 |
-|------|------|
-| Dart 变更整体评估 | {✅ 全部必要 / ⚠️ 部分需修改 / ❌ 存在不必要变更} |
-| 需回退的变更数 | N 处 |
-| 建议保留的变更数 | N 处 |
-
----
-
-## 七、影响分析
-
-- **功能影响**：是否引入新功能或修改行为
-- **兼容性影响**：Plugin API 是否向后兼容
-- **跨平台一致性影响**：ohos 实现与 android/iOS 是否存在行为差异
-- **Dart 变更对其他平台的影响**：Dart 侧修改是否影响 android/iOS/Web 的编译或运行
-- **性能影响**：异步处理是否改善
-- **安全影响**：是否修复或引入安全问题
-
----
-
-## 八、合入建议分级
-
-> 根据问题严重程度统计，按照分级标准判定合入建议等级。
-
-### 8.1 问题统计
-
-| 等级 | 数量 | 说明 |
-|------|------|------|
-| 🔴 **Critical** | N 个 | 必须修复，会导致崩溃或安全漏洞 |
-| 🟠 **Major** | N 个 | 强烈建议修复，影响功能/兼容性 |
-| 🟡 **Minor** | N 个 | 建议改进，代码质量/规范问题 |
-| 🔵 **Suggest** | N 个 | 可选改进，可读性/文档优化 |
-
-### 8.2 合入建议
-
-**等级标识**：✅ PASS / ⚠️ CONDITIONAL / 🔧 NEED_FIX / ❌ REJECT
-
-**判定结果**：{等级标识}
-
-**分级理由**：
-- {根据分级标准说明判定理由}
-- {例如：存在 1 个 Critical 级别问题（result.success() 后未 return），判定为 NEED_FIX，需修复后合入}
-
-**合入条件**（仅 CONDITIONAL/NEED_FIX 时填写）：
-- {列出必须满足的条件，如：修复 Critical 问题、补充测试用例等}
-
----
-
-## 九、修改点详细分级清单
-
-> 每个修改点的具体分析，明确标注所属的合入建议级别。
-
-### 9.1 建议合入级修改点 ✅
-
-> 无风险或低风险修改，可直接合入。
-
-| # | 文件 | 修改内容 | 风险评估 | 说明 |
-|---|------|----------|----------|------|
-| 1 | `lib/...` | 删除无用导入 | 低风险 | 代码清理，不影响功能 |
-
-### 9.2 条件合入级修改点 ⚠️
-
-> 需满足特定条件后方可合入。
-
-| # | 文件 | 修改内容 | 潜在风险 | 合入条件 |
-|---|------|----------|----------|----------|
-| 1 | `ohos/...` | 新增异步调用 | 需验证异常场景 | 补充异常测试用例 |
-
-### 9.3 修改后合入级修改点 🔧
-
-> 存在明确问题，需修复后方可合入。
-
-| # | 文件 | 位置 | 问题描述 | 修复建议 | 严重级别 |
-|---|------|------|----------|----------|----------|
-| 1 | `ohos/src/main/ets/...` | `method()` | `result.success()` 后未 return | 添加 return 语句 | 🔴 Critical |
-
-### 9.4 不允许合入级修改点 ❌
-
-> 严重问题，不建议合入。
-
-| # | 文件 | 位置 | 问题描述 | 影响 | 严重级别 |
-|---|------|------|----------|------|----------|
-| 1 | `ohos/src/main/cpp/...` | NAPI 模块 | NAPI 对象未正确释放 | 内存泄漏 | 🔴 Critical |
-
----
-
-## 十、result.txt 文件生成规范
-
-> 检视完成后，生成 `result.txt` 文件供前端展示和数据库存储。
-
-```txt
-=== 合入建议分级报告 ===
-PR 地址：{prUrl}
-仓库：{owner}/{repo}
-PR 编号：#{prNumber}
-生成时间：{YYYY-MM-DD HH:mm:ss}
-
-【合入建议】：{等级标识}
-
-【问题统计】
-- Critical 级别：{criticalCount} 个
-- Major 级别：{majorCount} 个  
-- Minor 级别：{minorCount} 个
-- Suggest 级别：{suggestCount} 个
-
-【分级理由】
-{分级理由详细说明}
-
-【详细分析】
-━━━ Critical 级别问题 ━━━
-{列出所有 Critical 问题，每个占一行}
-
-━━━ Major 级别问题 ━━━
-{列出所有 Major 问题，每个占一行}
-
-━━━ Minor 级别问题 ━━━
-{列出所有 Minor 问题，每个占一行}
-
-━━━ Suggest 级别问题 ━━━
-{列出所有 Suggest 问题，每个占一行}
-
-━━━ 合入建议 ━━━
-{最终合入建议和条件说明}
-
-=== 分级结束 ===
-```
-
----
-
-*本报告由 flutter-code-check 自动生成，仅供参考，最终合入决策由人工评审确认。*
-```
-
----
-
-## 问题等级定义
-
-| 等级 | 图标 | 说明 |
-|------|------|------|
-| 严重 (CRITICAL) | 🔴 | 必须修复，否则会导致功能缺陷或安全问题 |
-| 主要 (MAJOR) | 🟠 | 强烈建议修复，影响代码质量或可维护性 |
-| 次要 (MINOR) | 🟡 | 建议改进，优化代码风格或实现方式 |
-| 建议 (SUGGEST) | 🔵 | 可选改进，提升代码可读性或文档完整性 |
-
----
-
-## 评估结论定义
-
-| 结论 | 条件 |
-|------|------|
-| APPROVED | 无严重/主要问题，代码质量良好 |
-| NEEDS_REVISION | 存在严重或主要问题，需修订后重新检视 |
+# 检视报告模板
+
+使用此模板生成 Flutter PR 代码检视报告，保存至 `{outputDir}/check-report.md`。
+
+---
+
+```markdown
+# Flutter 代码检视报告
+
+## 一、基本信息
+
+| 字段 | 内容 |
+|------|------|
+| **PR 地址** | {prUrl} |
+| **代码仓库** | {owner}/{repo} |
+| **目标分支** | {targetBranch} |
+| **PR 标题** | {prTitle} |
+| **提交作者** | {author} <{email}> |
+| **提交时间** | {date} |
+| **Commit Hash** | {commitHash} |
+| **库类型** | Flutter |
+| **报告生成时间** | {YYYY-MM-DD HH:mm} |
+
+---
+
+## 二、改动概述
+
+### 2.1 改动规模
+
+| 指标 | 数值 |
+|------|------|
+| 涉及文件数 | N 个 |
+| 新增行数 | +X 行 |
+| 删除行数 | -Y 行 |
+| 净变更 | +Z 行 |
+
+### 2.2 改动文件清单
+
+| 文件路径 | 类型 | 改动性质 | 改动摘要 |
+|----------|------|----------|----------|
+| `ohos/src/main/ets/...` | ArkTS | 功能修复 | 修复 Plugin 生命周期释放逻辑 |
+| `lib/...` | Dart | 接口更新 | 新增平台判断逻辑 |
+
+### 2.3 整体改动意图
+
+{根据 commit message 和 diff 内容描述改动意图}
+
+---
+
+## 三、检视结果汇总
+
+| 等级 | 问题数 |
+|------|--------|
+| 🔴 严重 (CRITICAL) | N |
+| 🟠 主要 (MAJOR)    | N |
+| 🟡 次要 (MINOR)    | N |
+| 🔵 建议 (SUGGEST)  | N |
+
+---
+
+## 四、详细检视意见
+
+### [文件路径]
+
+#### 🟠 主要问题
+
+**[问题标题]**
+- **位置**：第 X 行
+- **描述**：...
+- **建议**：...
+
+```diff
+- 问题代码
++ 建议改法
+```
+
+---
+
+## 五、跨平台一致性分析
+
+> PR 中仅包含鸿蒙代码提交，需从克隆的仓库中读取 `android/` 和 `ios/` 的已有实现进行对比。
+
+### 5.1 对比基准
+
+| 字段 | 内容 |
+|------|------|
+| **Android 实现文件** | `{android/.../XxxPlugin.java}` |
+| **iOS 实现文件** | `{ios/Classes/XxxPlugin.m}` |
+| **OHOS 实现文件** | `{ohos/src/main/ets/.../XxxPlugin.ets}` |
+
+### 5.2 方法路由一致性
+
+| # | 方法名 | Android 文件:行号 | OHOS 文件:行号 | 状态 | 说明 |
+|---|--------|------------------|---------------|------|------|
+| 1 | `methodName1` | `Plugin.java:56` | `Plugin.ets:89` | ✅ 一致 | 两端均已实现 |
+| 2 | `methodName2` | `Plugin.java:78` | — | ❌ 缺失 | OHOS 未实现此方法，Dart 侧将收到 MissingPluginException |
+
+### 5.3 参数类型映射一致性
+
+| # | 方法名 | 参数名 | Android 类型 | OHOS 类型 | 状态 | 说明 |
+|---|--------|--------|-------------|----------|------|------|
+| 1 | `methodName` | `param1` | `boolean` | `boolean` | ✅ 一致 | |
+| 2 | `methodName` | `param2` | `int` | `string` ❌ | ❌ 致命 | 类型映射错误导致功能失效 |
+
+### 5.4 返回值与错误码一致性
+
+| # | 方法名 | Android 返回 | OHOS 返回 | 错误码对齐 | 状态 | 说明 |
+|---|--------|-------------|----------|-----------|------|------|
+| 1 | `methodName` | `Map<String, Object>` | `Record<string, Object>` | ✅ 一致 | ✅ 一致 | |
+| 2 | `methodName` | `result.success(data)` | `result.success(fixedValue)` | ❌ 不一致 | ❌ Stub | OHOS 返回固定值，未实际执行逻辑 |
+
+### 5.5 跨平台一致性总结
+
+| 检查项 | 结果 |
+|--------|------|
+| 方法路由完整性 | {缺失 N 个方法} |
+| 参数类型正确性 | {N 个致命类型错误} |
+| 返回值格式一致性 | {N 个不一致} |
+| 错误码对齐 | {N 个不一致} |
+| Stub 方法 | {N 个 Stub 实现} |
+
+---
+
+## 六、Dart 改动必要性分析
+
+> **核心原则**：鸿蒙适配 PR 中 Dart 代码（`lib/` 目录）原则上不应修改。若 PR 包含 Dart 变更，必须逐一论证必要性和跨平台影响。
+
+### 6.1 Dart 变更总览
+
+| 指标 | 数值 |
+|------|------|
+| Dart 变更文件数 | N 个 |
+| Dart 变更总行数 | +X / -Y |
+| 必要改动数 | N 处 |
+| 不应修改数 | N 处 |
+| 需评估数 | N 处 |
+
+### 6.2 逐项必要性分析
+
+| # | 文件 | 行号 | 变更类型 | 变更摘要 | 必要性判定 | 问题等级 | 影响平台 | 说明 |
+|---|------|------|----------|----------|------------|----------|----------|------|
+| 1 | `lib/xxx.dart` | L42 | 新增 | 新增 `Platform.isOHOS` 条件分支 | ✅ 必要 | — | 无 | 鸿蒙平台注册逻辑，仅 ohos 分支 |
+| 2 | `lib/xxx.dart` | L78 | 修改 | 修改 `invokeMethod` 参数结构 | ❌ 不应修改 | 🔴 Critical | android/iOS | 应在 ohos 侧适配参数解析 |
+| 3 | `lib/xxx.dart` | L105 | 新增 | 新增 `import 'dart:io'` | ⚠️ 需评估 | 🟡 Minor | Web | 可能影响 Web 平台编译 |
+
+### 6.3 跨平台影响评估
+
+| # | Dart 变更 | 影响平台 | 影响类型 | 影响描述 | 严重程度 |
+|---|-----------|----------|----------|----------|----------|
+| 1 | 修改 `invokeMethod` 参数 | android/iOS | 运行时 | 参数结构变更导致 android 侧解析失败 | 🔴 Critical |
+| 2 | 新增 `dart:io` import | Web | 编译时 | Web 平台不支持 `dart:io`，编译报错 | 🟠 Major |
+
+### 6.4 替代方案建议
+
+| # | 原始变更 | 问题描述 | 替代方案 | 优势 |
+|---|----------|----------|----------|------|
+| 1 | 修改 `MethodChannel` 名称 | android 侧无法识别新 Channel 名 | 在 ohos 侧注册与 android 相同的 Channel 名 | 不影响 Dart 和其他平台 |
+| 2 | 修改 `invokeMethod` 参数 | 破坏 android 侧参数解析 | 在 ohos 侧 `onMethodCall` 中适配参数格式 | 仅 ohos 侧改动 |
+
+### 6.5 Dart 改动结论
+
+| 结论 | 说明 |
+|------|------|
+| Dart 变更整体评估 | {✅ 全部必要 / ⚠️ 部分需修改 / ❌ 存在不必要变更} |
+| 需回退的变更数 | N 处 |
+| 建议保留的变更数 | N 处 |
+
+---
+
+## 七、影响分析
+
+- **功能影响**：是否引入新功能或修改行为
+- **兼容性影响**：Plugin API 是否向后兼容
+- **跨平台一致性影响**：ohos 实现与 android/iOS 是否存在行为差异
+- **Dart 变更对其他平台的影响**：Dart 侧修改是否影响 android/iOS/Web 的编译或运行
+- **性能影响**：异步处理是否改善
+- **安全影响**：是否修复或引入安全问题
+
+---
+
+## 八、合入建议分级
+
+> 根据问题严重程度统计，按照分级标准判定合入建议等级。
+
+### 8.1 问题统计
+
+| 等级 | 数量 | 说明 |
+|------|------|------|
+| 🔴 **Critical** | N 个 | 必须修复，会导致崩溃或安全漏洞 |
+| 🟠 **Major** | N 个 | 强烈建议修复，影响功能/兼容性 |
+| 🟡 **Minor** | N 个 | 建议改进，代码质量/规范问题 |
+| 🔵 **Suggest** | N 个 | 可选改进，可读性/文档优化 |
+
+### 8.2 合入建议
+
+**等级标识**：✅ PASS / ⚠️ CONDITIONAL / 🔧 NEED_FIX / ❌ REJECT
+
+**判定结果**：{等级标识}
+
+**分级理由**：
+- {根据分级标准说明判定理由}
+- {例如：存在 1 个 Critical 级别问题（result.success() 后未 return），判定为 NEED_FIX，需修复后合入}
+
+**合入条件**（仅 CONDITIONAL/NEED_FIX 时填写）：
+- {列出必须满足的条件，如：修复 Critical 问题、补充测试用例等}
+
+---
+
+## 九、修改点详细分级清单
+
+> 每个修改点的具体分析，明确标注所属的合入建议级别。
+
+### 9.1 建议合入级修改点 ✅
+
+> 无风险或低风险修改，可直接合入。
+
+| # | 文件 | 修改内容 | 风险评估 | 说明 |
+|---|------|----------|----------|------|
+| 1 | `lib/...` | 删除无用导入 | 低风险 | 代码清理，不影响功能 |
+
+### 9.2 条件合入级修改点 ⚠️
+
+> 需满足特定条件后方可合入。
+
+| # | 文件 | 修改内容 | 潜在风险 | 合入条件 |
+|---|------|----------|----------|----------|
+| 1 | `ohos/...` | 新增异步调用 | 需验证异常场景 | 补充异常测试用例 |
+
+### 9.3 修改后合入级修改点 🔧
+
+> 存在明确问题，需修复后方可合入。
+
+| # | 文件 | 位置 | 问题描述 | 修复建议 | 严重级别 |
+|---|------|------|----------|----------|----------|
+| 1 | `ohos/src/main/ets/...` | `method()` | `result.success()` 后未 return | 添加 return 语句 | 🔴 Critical |
+
+### 9.4 不允许合入级修改点 ❌
+
+> 严重问题，不建议合入。
+
+| # | 文件 | 位置 | 问题描述 | 影响 | 严重级别 |
+|---|------|------|----------|------|----------|
+| 1 | `ohos/src/main/cpp/...` | NAPI 模块 | NAPI 对象未正确释放 | 内存泄漏 | 🔴 Critical |
+
+---
+
+## 十、result.txt 文件生成规范
+
+> 检视完成后，生成 `result.txt` 文件供前端展示和数据库存储。
+
+```txt
+=== 合入建议分级报告 ===
+PR 地址：{prUrl}
+仓库：{owner}/{repo}
+PR 编号：#{prNumber}
+生成时间：{YYYY-MM-DD HH:mm:ss}
+
+【合入建议】：{等级标识}
+
+【问题统计】
+- Critical 级别：{criticalCount} 个
+- Major 级别：{majorCount} 个  
+- Minor 级别：{minorCount} 个
+- Suggest 级别：{suggestCount} 个
+
+【分级理由】
+{分级理由详细说明}
+
+【详细分析】
+━━━ Critical 级别问题 ━━━
+{列出所有 Critical 问题，每个占一行}
+
+━━━ Major 级别问题 ━━━
+{列出所有 Major 问题，每个占一行}
+
+━━━ Minor 级别问题 ━━━
+{列出所有 Minor 问题，每个占一行}
+
+━━━ Suggest 级别问题 ━━━
+{列出所有 Suggest 问题，每个占一行}
+
+━━━ 合入建议 ━━━
+{最终合入建议和条件说明}
+
+=== 分级结束 ===
+```
+
+---
+
+*本报告由 flutter-code-check 自动生成，仅供参考，最终合入决策由人工评审确认。*
+```
+
+---
+
+## 问题等级定义
+
+| 等级 | 图标 | 说明 |
+|------|------|------|
+| 严重 (CRITICAL) | 🔴 | 必须修复，否则会导致功能缺陷或安全问题 |
+| 主要 (MAJOR) | 🟠 | 强烈建议修复，影响代码质量或可维护性 |
+| 次要 (MINOR) | 🟡 | 建议改进，优化代码风格或实现方式 |
+| 建议 (SUGGEST) | 🔵 | 可选改进，提升代码可读性或文档完整性 |
+
+---
+
+## 评估结论定义
+
+| 结论 | 条件 |
+|------|------|
+| APPROVED | 无严重/主要问题，代码质量良好 |
+| NEEDS_REVISION | 存在严重或主要问题，需修订后重新检视 |
 | REJECTED | 存在多个严重问题或安全缺陷，不建议合并 |