@ohos-cpf/3rdloop 0.0.2 → 0.0.4

package/vendor/Server/Skills/rn-code-check/references/REPORT-TEMPLATE.md

@@ -1,247 +1,247 @@
-# React Native 代码检视报告模板
-
-本文档提供标准化的检视报告格式模板。
-
----
-
-## 一、基本信息
-
-| 字段 | 内容 |
-|------|------|
-| **PR 地址** | {prUrl} |
-| **代码仓库** | {owner}/{repo} |
-| **目标分支** | {targetBranch} |
-| **PR 标题** | {prTitle（commit Subject）} |
-| **提交作者** | {author} <{email}> |
-| **提交时间** | {date} |
-| **Commit Hash** | {commitHash} |
-| **库类型** | React Native |
-| **报告生成时间** | {YYYY-MM-DD HH:mm} |
-
----
-
-## 二、改动概述
-
-### 2.1 改动规模
-
-| 指标 | 数值 |
-|------|------|
-| 涉及文件数 | N 个 |
-| 新增行数 | +X 行 |
-| 删除行数 | -Y 行 |
-| 净变更 | +Z 行 |
-
-### 2.2 改动文件清单
-
-| 文件路径 | 类型 | 改动性质 | 改动摘要 |
-|----------|------|----------|----------|
-| `harmony/src/main/cpp/...` | C++ CAPI | 功能修复 | 修复跨线程回调未使用 napi_threadsafe_function |
-| `src/...` | JS/TS | 接口更新 | 新增平台判断逻辑 |
-
-### 2.3 整体改动意图
-
-{根据 commit message 和 diff 内容描述改动意图}
-
----
-
-## 三、检视结果汇总
-
-| 等级 | 问题数 |
-|------|--------|
-| 🔴 严重 (CRITICAL) | N |
-| 🟠 主要 (MAJOR)    | N |
-| 🟡 次要 (MINOR)    | N |
-| 🔵 建议 (SUGGEST)  | N |
-
----
-
-## 四、详细检视意见
-
-### [文件路径]
-
-#### 🔴 严重问题
-
-**[问题标题]**
-- **位置**：第 X 行
-- **描述**：...
-- **建议**：...
-
-```diff
-- 问题代码
-+ 建议改法
-```
-
----
-
-## 五、影响分析
-
-- **功能影响**：本次改动是否引入新功能或修改现有行为
-- **兼容性影响**：对外暴露的 TurboModule API 是否向后兼容
-- **性能影响**：是否优化了跨线程调用性能或内存占用
-- **安全影响**：是否修复或引入了内存安全或线程安全问题
-
----
-
-## 六、跨平台一致性对比
-
-> 对比 PR diff 涉及的 harmony 实现与 Android/iOS 原生模块的方法路由、参数类型、返回值格式一致性。
-
-| 对比维度 | harmony 实现 | Android 实现 | 一致性 | 问题等级 |
-|----------|-------------|-------------|--------|----------|
-| TurboModule 方法路由 | `methodName1`, `methodName2`, ... | `methodName1`, `methodName2`, ... | ✅一致 / ❌缺失X个方法 | 🔴/🟠/✅ |
-| TurboModule 名称 | `XxxModule` | `XxxModule` | ✅一致 / ❌不一致 | 🔴/✅ |
-| 参数类型映射 | `call.argument('key')` 类型 | `call.argument('key')` 类型 | ✅一致 / ❌类型不匹配 | 🔴/✅ |
-| 返回值格式 | `{code: 0, data: [...]}` | `{code: 0, data: [...]}` | ✅一致 / ❌格式不同 | 🟠/✅ |
-| 错误码/消息 | code: `"E_XXX"` | code: `"E_XXX"` | ✅一致 / ❌不匹配 | 🟠/✅ |
-| 功能完整性 | 全部实现 / Stub: X个 | 全部实现 | ✅完整 / ❌缺失X个方法 | 🔴/✅ |
-| EventEmitter 事件名 | `eventName1`, `eventName2` | `eventName1`, `eventName2` | ✅一致 / ❌不一致 | 🟠/✅ |
-
-### 不一致详情
-
-#### [方法名]
-- **Android 实现**：{描述 Android 的方法签名、参数类型、返回值、错误处理}
-- **harmony 实现**：{描述 harmony 的对应实现}
-- **差异**：{具体差异说明}
-- **风险**：{JS 侧调用时可能的行为差异}
-- **建议**：{修复建议}
-
----
-
-## 七、总体评估
-
-**结论**：[APPROVED / NEEDS_REVISION / REJECTED]
-
-**说明**：...
-
----
-
-## 八、合入建议分级
-
-> 根据问题严重程度统计，按照分级标准判定合入建议等级。
-
-### 8.1 问题统计
-
-| 等级 | 数量 | 说明 |
-|------|------|------|
-| 🔴 **Critical** | N 个 | 必须修复，会导致崩溃、数据损坏、安全漏洞 |
-| 🟠 **Major** | N 个 | 强烈建议修复，影响功能正确性/性能/兼容性 |
-| 🟡 **Minor** | N 个 | 建议修复，代码质量/规范/可读性问题 |
-| 🔵 **Suggest** | N 个 | 可选改进，不强制要求修复 |
-
-### 8.2 合入建议
-
-**等级标识**：✅ PASS / ⚠️ CONDITIONAL / 🔧 NEED_FIX / ❌ REJECT
-
-**判定结果**：{等级标识}
-
-**分级理由**：
-- {根据分级标准说明判定理由}
-- {例如：存在 1 个 Critical 级别问题（子线程直接调用 NAPI），判定为 NEED_FIX，需修复后合入}
-- {线程安全问题示例：回调函数在子线程执行但未使用 napi_threadsafe_function，会导致 JS 崩溃}
-
-**合入条件**（仅 CONDITIONAL/NEED_FIX 时填写）：
-- {列出必须满足的条件，如：修复线程安全问题、使用 napi_threadsafe_function、补充测试用例等}
-
----
-
-## 九、修改点详细分级清单
-
-> 每个修改点的具体分析，明确标注所属的合入建议级别。
-
-### 9.1 建议合入级修改点 ✅
-
-> 无风险或低风险修改，可直接合入。
-
-| # | 文件 | 修改内容 | 风险评估 | 说明 |
-|---|------|----------|----------|------|
-| 1 | `src/...` | 删除无用导入 | 低风险 | 代码清理，不影响功能 |
-
-### 9.2 条件合入级修改点 ⚠️
-
-> 需满足特定条件后方可合入。
-
-| # | 文件 | 修改内容 | 潜在风险 | 合入条件 |
-|---|------|----------|----------|----------|
-| 1 | `harmony/src/main/ets/...` | 新增 TurboModule 方法 | 需验证跨平台一致性 | 补充 JS 端测试用例 |
-
-### 9.3 修改后合入级修改点 🔧
-
-> 存在明确问题，需修复后方可合入。
-
-| # | 文件 | 位置 | 问题描述 | 修复建议 | 严重级别 |
-|---|------|------|----------|----------|----------|
-| 1 | `harmony/src/main/cpp/...` | `callback()` | 子线程直接调用 NAPI 函数 | 使用 napi_threadsafe_function | 🔴 Critical |
-
-### 9.4 不允许合入级修改点 ❌
-
-> 严重问题，不建议合入。
-
-| # | 文件 | 位置 | 问题描述 | 影响 | 严重级别 |
-|---|------|------|----------|------|----------|
-| 1 | `harmony/src/main/cpp/...` | NAPI 模块 | napi_ref 未正确释放导致内存泄漏 | 长期运行内存增长 | 🔴 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 问题，每个占一行}
-
-━━━ 合入建议 ━━━
-{最终合入建议和条件说明}
-
-=== 分级结束 ===
-```
-
----
-
-*本报告由 RNCodeCheck 自动生成，仅供参考，最终合入决策由人工评审确认。*
-
----
-
-## 报告生成说明
-
-1. **文件命名**：必须为 `check-report.md`
-2. **保存路径**：必须在 `{outputDir}` 下
-3. **文件列表**：同时生成 `changed-files.txt`（每行一个文件路径）
-4. **无鸿蒙变更**：若 PR 只修改 `android/` 或 `ios/`，在报告中注明"本次 PR 无鸿蒙平台代码变更"并给出 APPROVED
-5. **线程安全优先**：对 C++ CAPI 的内存安全和线程安全问题给予最高优先级标记为 CRITICAL
-
----
-
-**问题等级定义**：
-
-- 🔴 **CRITICAL（严重）**：会导致崩溃、数据损坏、安全漏洞的严重问题，必须修复
-- 🟠 **MAJOR（主要）**：影响功能正确性、性能、兼容性的重要问题，强烈建议修复
-- 🟡 **MINOR（次要）**：代码质量问题、规范问题、可读性问题，建议修复
+# React Native 代码检视报告模板
+
+本文档提供标准化的检视报告格式模板。
+
+---
+
+## 一、基本信息
+
+| 字段 | 内容 |
+|------|------|
+| **PR 地址** | {prUrl} |
+| **代码仓库** | {owner}/{repo} |
+| **目标分支** | {targetBranch} |
+| **PR 标题** | {prTitle（commit Subject）} |
+| **提交作者** | {author} <{email}> |
+| **提交时间** | {date} |
+| **Commit Hash** | {commitHash} |
+| **库类型** | React Native |
+| **报告生成时间** | {YYYY-MM-DD HH:mm} |
+
+---
+
+## 二、改动概述
+
+### 2.1 改动规模
+
+| 指标 | 数值 |
+|------|------|
+| 涉及文件数 | N 个 |
+| 新增行数 | +X 行 |
+| 删除行数 | -Y 行 |
+| 净变更 | +Z 行 |
+
+### 2.2 改动文件清单
+
+| 文件路径 | 类型 | 改动性质 | 改动摘要 |
+|----------|------|----------|----------|
+| `harmony/src/main/cpp/...` | C++ CAPI | 功能修复 | 修复跨线程回调未使用 napi_threadsafe_function |
+| `src/...` | JS/TS | 接口更新 | 新增平台判断逻辑 |
+
+### 2.3 整体改动意图
+
+{根据 commit message 和 diff 内容描述改动意图}
+
+---
+
+## 三、检视结果汇总
+
+| 等级 | 问题数 |
+|------|--------|
+| 🔴 严重 (CRITICAL) | N |
+| 🟠 主要 (MAJOR)    | N |
+| 🟡 次要 (MINOR)    | N |
+| 🔵 建议 (SUGGEST)  | N |
+
+---
+
+## 四、详细检视意见
+
+### [文件路径]
+
+#### 🔴 严重问题
+
+**[问题标题]**
+- **位置**：第 X 行
+- **描述**：...
+- **建议**：...
+
+```diff
+- 问题代码
++ 建议改法
+```
+
+---
+
+## 五、影响分析
+
+- **功能影响**：本次改动是否引入新功能或修改现有行为
+- **兼容性影响**：对外暴露的 TurboModule API 是否向后兼容
+- **性能影响**：是否优化了跨线程调用性能或内存占用
+- **安全影响**：是否修复或引入了内存安全或线程安全问题
+
+---
+
+## 六、跨平台一致性对比
+
+> 对比 PR diff 涉及的 harmony 实现与 Android/iOS 原生模块的方法路由、参数类型、返回值格式一致性。
+
+| 对比维度 | harmony 实现 | Android 实现 | 一致性 | 问题等级 |
+|----------|-------------|-------------|--------|----------|
+| TurboModule 方法路由 | `methodName1`, `methodName2`, ... | `methodName1`, `methodName2`, ... | ✅一致 / ❌缺失X个方法 | 🔴/🟠/✅ |
+| TurboModule 名称 | `XxxModule` | `XxxModule` | ✅一致 / ❌不一致 | 🔴/✅ |
+| 参数类型映射 | `call.argument('key')` 类型 | `call.argument('key')` 类型 | ✅一致 / ❌类型不匹配 | 🔴/✅ |
+| 返回值格式 | `{code: 0, data: [...]}` | `{code: 0, data: [...]}` | ✅一致 / ❌格式不同 | 🟠/✅ |
+| 错误码/消息 | code: `"E_XXX"` | code: `"E_XXX"` | ✅一致 / ❌不匹配 | 🟠/✅ |
+| 功能完整性 | 全部实现 / Stub: X个 | 全部实现 | ✅完整 / ❌缺失X个方法 | 🔴/✅ |
+| EventEmitter 事件名 | `eventName1`, `eventName2` | `eventName1`, `eventName2` | ✅一致 / ❌不一致 | 🟠/✅ |
+
+### 不一致详情
+
+#### [方法名]
+- **Android 实现**：{描述 Android 的方法签名、参数类型、返回值、错误处理}
+- **harmony 实现**：{描述 harmony 的对应实现}
+- **差异**：{具体差异说明}
+- **风险**：{JS 侧调用时可能的行为差异}
+- **建议**：{修复建议}
+
+---
+
+## 七、总体评估
+
+**结论**：[APPROVED / NEEDS_REVISION / REJECTED]
+
+**说明**：...
+
+---
+
+## 八、合入建议分级
+
+> 根据问题严重程度统计，按照分级标准判定合入建议等级。
+
+### 8.1 问题统计
+
+| 等级 | 数量 | 说明 |
+|------|------|------|
+| 🔴 **Critical** | N 个 | 必须修复，会导致崩溃、数据损坏、安全漏洞 |
+| 🟠 **Major** | N 个 | 强烈建议修复，影响功能正确性/性能/兼容性 |
+| 🟡 **Minor** | N 个 | 建议修复，代码质量/规范/可读性问题 |
+| 🔵 **Suggest** | N 个 | 可选改进，不强制要求修复 |
+
+### 8.2 合入建议
+
+**等级标识**：✅ PASS / ⚠️ CONDITIONAL / 🔧 NEED_FIX / ❌ REJECT
+
+**判定结果**：{等级标识}
+
+**分级理由**：
+- {根据分级标准说明判定理由}
+- {例如：存在 1 个 Critical 级别问题（子线程直接调用 NAPI），判定为 NEED_FIX，需修复后合入}
+- {线程安全问题示例：回调函数在子线程执行但未使用 napi_threadsafe_function，会导致 JS 崩溃}
+
+**合入条件**（仅 CONDITIONAL/NEED_FIX 时填写）：
+- {列出必须满足的条件，如：修复线程安全问题、使用 napi_threadsafe_function、补充测试用例等}
+
+---
+
+## 九、修改点详细分级清单
+
+> 每个修改点的具体分析，明确标注所属的合入建议级别。
+
+### 9.1 建议合入级修改点 ✅
+
+> 无风险或低风险修改，可直接合入。
+
+| # | 文件 | 修改内容 | 风险评估 | 说明 |
+|---|------|----------|----------|------|
+| 1 | `src/...` | 删除无用导入 | 低风险 | 代码清理，不影响功能 |
+
+### 9.2 条件合入级修改点 ⚠️
+
+> 需满足特定条件后方可合入。
+
+| # | 文件 | 修改内容 | 潜在风险 | 合入条件 |
+|---|------|----------|----------|----------|
+| 1 | `harmony/src/main/ets/...` | 新增 TurboModule 方法 | 需验证跨平台一致性 | 补充 JS 端测试用例 |
+
+### 9.3 修改后合入级修改点 🔧
+
+> 存在明确问题，需修复后方可合入。
+
+| # | 文件 | 位置 | 问题描述 | 修复建议 | 严重级别 |
+|---|------|------|----------|----------|----------|
+| 1 | `harmony/src/main/cpp/...` | `callback()` | 子线程直接调用 NAPI 函数 | 使用 napi_threadsafe_function | 🔴 Critical |
+
+### 9.4 不允许合入级修改点 ❌
+
+> 严重问题，不建议合入。
+
+| # | 文件 | 位置 | 问题描述 | 影响 | 严重级别 |
+|---|------|------|----------|------|----------|
+| 1 | `harmony/src/main/cpp/...` | NAPI 模块 | napi_ref 未正确释放导致内存泄漏 | 长期运行内存增长 | 🔴 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 问题，每个占一行}
+
+━━━ 合入建议 ━━━
+{最终合入建议和条件说明}
+
+=== 分级结束 ===
+```
+
+---
+
+*本报告由 RNCodeCheck 自动生成，仅供参考，最终合入决策由人工评审确认。*
+
+---
+
+## 报告生成说明
+
+1. **文件命名**：必须为 `check-report.md`
+2. **保存路径**：必须在 `{outputDir}` 下
+3. **文件列表**：同时生成 `changed-files.txt`（每行一个文件路径）
+4. **无鸿蒙变更**：若 PR 只修改 `android/` 或 `ios/`，在报告中注明"本次 PR 无鸿蒙平台代码变更"并给出 APPROVED
+5. **线程安全优先**：对 C++ CAPI 的内存安全和线程安全问题给予最高优先级标记为 CRITICAL
+
+---
+
+**问题等级定义**：
+
+- 🔴 **CRITICAL（严重）**：会导致崩溃、数据损坏、安全漏洞的严重问题，必须修复
+- 🟠 **MAJOR（主要）**：影响功能正确性、性能、兼容性的重要问题，强烈建议修复
+- 🟡 **MINOR（次要）**：代码质量问题、规范问题、可读性问题，建议修复
 - 🔵 **SUGGEST（建议）**：可选的改进建议，不强制要求修复