@ohos-cpf/3rdloop 0.0.2 → 0.0.3

package/vendor/Server/Skills/arkts-code-check/references/REPORT_TEMPLATE.md

@@ -1,220 +1,220 @@
-# 检视报告模板
-
-本模板用于生成 ArkTS 代码检视报告，输出至 `{outputDir}/check-report.md`。
-
----
-
-```markdown
-# ArkTS 代码检视报告
-
-## 一、基本信息
-
-| 字段 | 内容 |
-|------|------|
-| **PR 地址** | {prUrl} |
-| **代码仓库** | {owner}/{repo} |
-| **目标分支** | {targetBranch} |
-| **PR 标题** | {prTitle} |
-| **提交作者** | {author} <{email}> |
-| **提交时间** | {date} |
-| **Commit Hash** | {commitHash} |
-| **库类型** | ArkTS |
-| **报告生成时间** | {YYYY-MM-DD HH:mm} |
-
----
-
-## 二、改动概述
-
-### 2.1 改动规模
-
-| 指标 | 数值 |
-|------|------|
-| 涉及文件数 | N 个 |
-| 新增行数 | +X 行 |
-| 删除行数 | -Y 行 |
-| 净变更 | +Z 行 |
-
-### 2.2 改动文件清单
-
-| 文件路径 | 类型 | 改动性质 | 改动摘要 |
-|----------|------|----------|----------|
-| `path/to/File.ets` | ArkTS | 功能修复 | 修复连接逻辑，避免复用旧设备对象 |
-| `oh-package.json5` | 配置 | 依赖更新 | 新增 @ohos/xxx 依赖 |
-
-### 2.3 整体改动意图
-
-> 根据 commit message 和 diff 内容综合描述本次提交的目的。
-
----
-
-## 三、详细检视结果
-
-> 问题按严重程度分级：🔴 Blocker / 🟡 Major / 🔵 Minor
-
-### 3.1 非兼容变更（Breaking Changes）
-
-> 若无，填写：**无**
-
-| # | 严重级 | 文件 | 位置 | 问题描述 |
-|---|--------|------|------|----------|
-| 1 | 🔴 | `File.ets` | `connect()` 方法 | 删除了公开接口参数，外部调用方受影响 |
-
-### 3.2 兼容性问题
-
-> 若无，填写：**无**
-
-| # | 严重级 | 类型 | 文件 | 问题描述 |
-|---|--------|------|------|----------|
-| 1 | 🟡 | API Level | `File.ets` | 使用了 API 12 接口，工程 minAPIVersion 为 10 |
-
-### 3.3 接口变化
-
-- **新增接口**：`接口名` — 说明
-- **修改接口**：`接口名` — 变化说明
-- **废弃接口**：`接口名` — 处理方式
-
-### 3.4 功能逻辑改变
-
-| # | 严重级 | 文件 | 改动内容 | 影响评估 |
-|---|--------|------|----------|----------|
-| 1 | 🔴 | `File.ets` | 嵌套循环索引变量误用（`includeServices[i]` → `includeServices[j]`） | Bug 修复，原代码多 include service 场景下越界或匹配错误 |
-| 2 | 🟡 | `File.ets` | 将 `BLECharacteristicChange` 监听移至 `connect` 时注册 | 修复了监听器堆积问题，但需确认 close 路径同步更新 |
-
-### 3.5 内存泄露与资源管理
-
-| # | 严重级 | 文件 | 问题描述 | 建议 |
-|---|--------|------|----------|------|
-| 1 | 🟡 | `File.ets` | 连接失败 catch 块新增 off，但 close 路径是否同步更新需确认 | 核查 disconnect 函数中的 off 逻辑 |
-
-### 3.6 其他问题
-
-| # | 严重级 | 类型 | 描述 |
-|---|--------|------|------|
-| 1 | 🔵 | 代码质量 | 删除了无用变量 `pairFlag`、`key1`，代码更简洁 |
-| 2 | 🔵 | 可观测性 | 错误信息从模糊描述改为包含 `e.code` 和 `e.message` 的格式 |
-
----
-
-## 四、影响分析
-
-### 4.1 功能影响
-
-- **正向影响**：（示例）修复了蓝牙多设备并发场景下的连接状态混乱问题
-- **潜在风险**：（示例）监听器注册时机变更，需验证非 notify 状态下回调行为
-
-### 4.2 接口兼容性影响
-
-> 对外暴露的接口（Plugin API 等）是否有变化，是否向后兼容。
-
-### 4.3 性能影响
-
-> 本次改动对性能的正向/负向影响说明。
-
----
-
-## 五、合入建议分级
-
-> 根据问题严重程度统计，按照分级标准判定合入建议等级。
-
-### 5.1 问题统计
-
-| 等级 | 数量 | 说明 |
-|------|------|------|
-| 🔴 **Blocker** | N 个 | 必须修复，影响功能/安全/兼容 |
-| 🟡 **Major** | N 个 | 建议修复，存在风险 |
-| 🔵 **Minor** | N 个 | 优化建议 |
-
-### 5.2 合入建议
-
-**等级标识**：✅ PASS / ⚠️ CONDITIONAL / 🔧 NEED_FIX / ❌ REJECT
-
-**判定结果**：{等级标识}
-
-**分级理由**：
-- {根据分级标准说明判定理由}
-- {例如：存在 1 个 Blocker 级别问题，判定为 NEED_FIX，需修复后合入}
-
-**合入条件**（仅 CONDITIONAL/NEED_FIX 时填写）：
-- {列出必须满足的条件，如：修复 Blocker 问题、补充测试用例等}
-
----
-
-## 六、修改点详细分级清单
-
-> 每个修改点的具体分析，明确标注所属的合入建议级别。
-
-### 6.1 建议合入级修改点 ✅
-
-> 无风险或低风险修改，可直接合入。
-
-| # | 文件 | 修改内容 | 风险评估 | 说明 |
-|---|------|----------|----------|------|
-| 1 | `path/to/File.ets` | 删除无用变量 | 低风险 | 代码清理，不影响功能 |
-
-### 6.2 条件合入级修改点 ⚠️
-
-> 需满足特定条件后方可合入。
-
-| # | 文件 | 修改内容 | 潜在风险 | 合入条件 |
-|---|------|----------|----------|----------|
-| 1 | `path/to/File.ets` | 新增异步调用 | 需验证异常场景 | 补充异常测试用例 |
-
-### 6.3 修改后合入级修改点 🔧
-
-> 存在明确问题，需修复后方可合入。
-
-| # | 文件 | 位置 | 问题描述 | 修复建议 | 严重级别 |
-|---|------|------|----------|----------|----------|
-| 1 | `path/to/File.ets` | `connect()` | API 兼容性破坏 | 恢复参数签名或迁移文档 | 🔴 Blocker |
-
-### 6.4 不允许合入级修改点 ❌
-
-> 严重问题，不建议合入。
-
-| # | 文件 | 位置 | 问题描述 | 影响 | 严重级别 |
-|---|------|------|----------|------|----------|
-| 1 | `path/to/File.ets` | 安全模块 | 硬编码密钥泄露 | 安全漏洞 | 🔴 Blocker |
-
----
-
-## 七、result.txt 文件生成规范
-
-> 检视完成后，生成 `result.txt` 文件供前端展示和数据库存储。
-
-```txt
-=== 合入建议分级报告 ===
-PR 地址：{prUrl}
-仓库：{owner}/{repo}
-PR 编号：#{prNumber}
-生成时间：{YYYY-MM-DD HH:mm:ss}
-
-【合入建议】：{等级标识}
-
-【问题统计】
-- Blocker 级别：{blockerCount} 个
-- Major 级别：{majorCount} 个  
-- Minor 级别：{minorCount} 个
-
-【分级理由】
-{分级理由详细说明}
-
-【详细分析】
-━━━ Blocker 级别问题 ━━━
-{列出所有 Blocker 问题，每个占一行}
-
-━━━ Major 级别问题 ━━━
-{列出所有 Major 问题，每个占一行}
-
-━━━ Minor 级别问题 ━━━
-{列出所有 Minor 问题，每个占一行}
-
-━━━ 合入建议 ━━━
-{最终合入建议和条件说明}
-
-=== 分级结束 ===
-```
-
----
-
-*本报告由 ArkTsCodeCheckSkill 自动生成，仅供参考，最终合入决策由人工评审确认。*
+# 检视报告模板
+
+本模板用于生成 ArkTS 代码检视报告，输出至 `{outputDir}/check-report.md`。
+
+---
+
+```markdown
+# ArkTS 代码检视报告
+
+## 一、基本信息
+
+| 字段 | 内容 |
+|------|------|
+| **PR 地址** | {prUrl} |
+| **代码仓库** | {owner}/{repo} |
+| **目标分支** | {targetBranch} |
+| **PR 标题** | {prTitle} |
+| **提交作者** | {author} <{email}> |
+| **提交时间** | {date} |
+| **Commit Hash** | {commitHash} |
+| **库类型** | ArkTS |
+| **报告生成时间** | {YYYY-MM-DD HH:mm} |
+
+---
+
+## 二、改动概述
+
+### 2.1 改动规模
+
+| 指标 | 数值 |
+|------|------|
+| 涉及文件数 | N 个 |
+| 新增行数 | +X 行 |
+| 删除行数 | -Y 行 |
+| 净变更 | +Z 行 |
+
+### 2.2 改动文件清单
+
+| 文件路径 | 类型 | 改动性质 | 改动摘要 |
+|----------|------|----------|----------|
+| `path/to/File.ets` | ArkTS | 功能修复 | 修复连接逻辑，避免复用旧设备对象 |
+| `oh-package.json5` | 配置 | 依赖更新 | 新增 @ohos/xxx 依赖 |
+
+### 2.3 整体改动意图
+
+> 根据 commit message 和 diff 内容综合描述本次提交的目的。
+
+---
+
+## 三、详细检视结果
+
+> 问题按严重程度分级：🔴 Blocker / 🟡 Major / 🔵 Minor
+
+### 3.1 非兼容变更（Breaking Changes）
+
+> 若无，填写：**无**
+
+| # | 严重级 | 文件 | 位置 | 问题描述 |
+|---|--------|------|------|----------|
+| 1 | 🔴 | `File.ets` | `connect()` 方法 | 删除了公开接口参数，外部调用方受影响 |
+
+### 3.2 兼容性问题
+
+> 若无，填写：**无**
+
+| # | 严重级 | 类型 | 文件 | 问题描述 |
+|---|--------|------|------|----------|
+| 1 | 🟡 | API Level | `File.ets` | 使用了 API 12 接口，工程 minAPIVersion 为 10 |
+
+### 3.3 接口变化
+
+- **新增接口**：`接口名` — 说明
+- **修改接口**：`接口名` — 变化说明
+- **废弃接口**：`接口名` — 处理方式
+
+### 3.4 功能逻辑改变
+
+| # | 严重级 | 文件 | 改动内容 | 影响评估 |
+|---|--------|------|----------|----------|
+| 1 | 🔴 | `File.ets` | 嵌套循环索引变量误用（`includeServices[i]` → `includeServices[j]`） | Bug 修复，原代码多 include service 场景下越界或匹配错误 |
+| 2 | 🟡 | `File.ets` | 将 `BLECharacteristicChange` 监听移至 `connect` 时注册 | 修复了监听器堆积问题，但需确认 close 路径同步更新 |
+
+### 3.5 内存泄露与资源管理
+
+| # | 严重级 | 文件 | 问题描述 | 建议 |
+|---|--------|------|----------|------|
+| 1 | 🟡 | `File.ets` | 连接失败 catch 块新增 off，但 close 路径是否同步更新需确认 | 核查 disconnect 函数中的 off 逻辑 |
+
+### 3.6 其他问题
+
+| # | 严重级 | 类型 | 描述 |
+|---|--------|------|------|
+| 1 | 🔵 | 代码质量 | 删除了无用变量 `pairFlag`、`key1`，代码更简洁 |
+| 2 | 🔵 | 可观测性 | 错误信息从模糊描述改为包含 `e.code` 和 `e.message` 的格式 |
+
+---
+
+## 四、影响分析
+
+### 4.1 功能影响
+
+- **正向影响**：（示例）修复了蓝牙多设备并发场景下的连接状态混乱问题
+- **潜在风险**：（示例）监听器注册时机变更，需验证非 notify 状态下回调行为
+
+### 4.2 接口兼容性影响
+
+> 对外暴露的接口（Plugin API 等）是否有变化，是否向后兼容。
+
+### 4.3 性能影响
+
+> 本次改动对性能的正向/负向影响说明。
+
+---
+
+## 五、合入建议分级
+
+> 根据问题严重程度统计，按照分级标准判定合入建议等级。
+
+### 5.1 问题统计
+
+| 等级 | 数量 | 说明 |
+|------|------|------|
+| 🔴 **Blocker** | N 个 | 必须修复，影响功能/安全/兼容 |
+| 🟡 **Major** | N 个 | 建议修复，存在风险 |
+| 🔵 **Minor** | N 个 | 优化建议 |
+
+### 5.2 合入建议
+
+**等级标识**：✅ PASS / ⚠️ CONDITIONAL / 🔧 NEED_FIX / ❌ REJECT
+
+**判定结果**：{等级标识}
+
+**分级理由**：
+- {根据分级标准说明判定理由}
+- {例如：存在 1 个 Blocker 级别问题，判定为 NEED_FIX，需修复后合入}
+
+**合入条件**（仅 CONDITIONAL/NEED_FIX 时填写）：
+- {列出必须满足的条件，如：修复 Blocker 问题、补充测试用例等}
+
+---
+
+## 六、修改点详细分级清单
+
+> 每个修改点的具体分析，明确标注所属的合入建议级别。
+
+### 6.1 建议合入级修改点 ✅
+
+> 无风险或低风险修改，可直接合入。
+
+| # | 文件 | 修改内容 | 风险评估 | 说明 |
+|---|------|----------|----------|------|
+| 1 | `path/to/File.ets` | 删除无用变量 | 低风险 | 代码清理，不影响功能 |
+
+### 6.2 条件合入级修改点 ⚠️
+
+> 需满足特定条件后方可合入。
+
+| # | 文件 | 修改内容 | 潜在风险 | 合入条件 |
+|---|------|----------|----------|----------|
+| 1 | `path/to/File.ets` | 新增异步调用 | 需验证异常场景 | 补充异常测试用例 |
+
+### 6.3 修改后合入级修改点 🔧
+
+> 存在明确问题，需修复后方可合入。
+
+| # | 文件 | 位置 | 问题描述 | 修复建议 | 严重级别 |
+|---|------|------|----------|----------|----------|
+| 1 | `path/to/File.ets` | `connect()` | API 兼容性破坏 | 恢复参数签名或迁移文档 | 🔴 Blocker |
+
+### 6.4 不允许合入级修改点 ❌
+
+> 严重问题，不建议合入。
+
+| # | 文件 | 位置 | 问题描述 | 影响 | 严重级别 |
+|---|------|------|----------|------|----------|
+| 1 | `path/to/File.ets` | 安全模块 | 硬编码密钥泄露 | 安全漏洞 | 🔴 Blocker |
+
+---
+
+## 七、result.txt 文件生成规范
+
+> 检视完成后，生成 `result.txt` 文件供前端展示和数据库存储。
+
+```txt
+=== 合入建议分级报告 ===
+PR 地址：{prUrl}
+仓库：{owner}/{repo}
+PR 编号：#{prNumber}
+生成时间：{YYYY-MM-DD HH:mm:ss}
+
+【合入建议】：{等级标识}
+
+【问题统计】
+- Blocker 级别：{blockerCount} 个
+- Major 级别：{majorCount} 个  
+- Minor 级别：{minorCount} 个
+
+【分级理由】
+{分级理由详细说明}
+
+【详细分析】
+━━━ Blocker 级别问题 ━━━
+{列出所有 Blocker 问题，每个占一行}
+
+━━━ Major 级别问题 ━━━
+{列出所有 Major 问题，每个占一行}
+
+━━━ Minor 级别问题 ━━━
+{列出所有 Minor 问题，每个占一行}
+
+━━━ 合入建议 ━━━
+{最终合入建议和条件说明}
+
+=== 分级结束 ===
+```
+
+---
+
+*本报告由 ArkTsCodeCheckSkill 自动生成，仅供参考，最终合入决策由人工评审确认。*
 ```