ability_ability_runtime/cli_tool_framework · pptaozz/ability_ability_runtime

文件	最后提交记录	最后更新时间
etc	funtionCall动态注册+查询 Co-Authored-By: Agent Signed-off-by: 朱菲墨 <zhufeimo1@huawei.com> 🤖‍ AI[0%] 👌 AI Adopted[0%] 🧑 Human[100%]	2 个月前
frameworks	!20104 merge 降低数据库初始化失败阻塞时间 into master 新增resetFunction接口优化数据库锁和阻塞时间 Created-by: zhu-feimo Commit-by: zhu-feimo Merged-by: openharmony_ci Description: IssueNo: [#15888](https://gitcode.com/openharmony/ability_ability_runtime/issues/15888) Description: 稳定性自检： \| 自检项 \| 自检结果 \| \| ------------------------------------------------------------ \| -------- \| \| 涉及跨进程调用的相关操作需要抛至主线程或加锁防止并发 \| \| \| 成员变量进行赋值或创建需要排查并发 \| \| \| 谨慎在lambda表达式中使用引用捕获 \| \| \| 谨慎在未经拷贝的情况下使用外部传入的string、C字符串 \| \| \| map\vector\list\set等stl模板类使用时需要排查并发 \| \| \| 谨慎考虑加锁范围 \| \| \| 在IPC通信中谨慎使用同步通信方式 \| \| \| 禁止传递this指针至其他模块或线程（特别是eventhandler任务） \| \| \| 禁止将外部传入的裸指针在内部直接构造智能指针 \| \| \| 禁止多个独立创建的智能指针管理同一地址 \| \| \| 禁止在析构函数中抛异步任务 \| \| \| 禁止js对象在非js线程（例如在IPC线程）创建、使用或销毁 \| \| \| 禁止在对外接口中未经判空直接使用外部传入的指针 \| \| \| 禁止接口返回局部变量引用 \| \| \| 禁止在信号函数中加锁 \| \| \| 禁止在关键流程（SA启动、应用启动等主流程）执行耗时的操作 \| \| \| 禁止将同一个cpp编译在不同的so中 \| \| 安全编码自检： \| 自检项 \| 自检结果 \| \| -------------------------------------------------------------- \| -------- \| \| 裸指针避免通过隐式转换构造为sptr \| \| \| json对象在取值之前必须先判断类型，避免类型不匹配 \| \| \| 序列化时必须对传入的数组大小进行校验，避免出现超大数组 \| \| \| 避免使用未明确位宽的整型，选择使用int8_t、uint8_t等类型 \| \| \| 外部传入的路径要做规范化校验，对路径中的.、..、../等特殊字符严格校验 \| \| \| 指针变量、表示资源描述符的变量、bool变量必须赋初值 \| \| \| readParcelable获取的对象使用前需要判空 \| \| \| 分配和释放内存的函数需要成对出现 \| \| \| 申请内存后异常退出前需要及时进行内存释放 \| \| \| 内存申请前必须对内存大小进行合法性校验 \| \| \| 内存分配后必须判断是否成功 \| \| \| 禁止使用realloc、alloca函数 \| \| \| 禁止打印文件路径、口令等敏感信息，如有需要，使用private修饰 \| \| \| 禁止打印内存地址 \| \| \| 整数之间运算时必须严格检查，确保不会出现溢出、反转、除0 \| \| \| 禁止对有符号整数进行位操作符运算 \| \| \| 禁止对指针进行逻辑或位运算 \| \| \| 循环次数如果收外部数据控制，需要检验其合法性 \| \| \| 禁止使用内存操作类危险函数，需要使用安全函数 \| \| \| 谨慎使用不可重入函数 \| \| \| 必须检查安全函数的返回值，并进行正确处理 \| \| \| 禁止仅通过TokenType类型判断绕过权限校验 \| \| TDD Result: XTS Result: ### 是否已执行L0用例 - [ ] 已验证 - [ ] 不涉及。如不涉及，请写明理由 ### AI检视评分（使用本地代码检视skills扫描）: # 代码检视报告 — cli_tool_framework ResetNamespaceFunctions（Round 1 / 最新提交） > 统一报告由 codecheck 工作台生成，用于门禁管控。所有 codecheck 检视（通用编排、deep-scan、单维度 skill）合并出的统一报告必须遵循本模板：章节顺序、字段名、报告元数据块、评分与门禁规则均为固定格式，跨报告保持一致，便于门禁脚本解析与历史对比。 > 生成入口：[`README.md`](README.md) → Step 4；合并逻辑见 [`codecheck-orchestrator/SKILL.md`](codecheck-orchestrator/SKILL.md)。 > 权威评分与门禁规则见文末附录 A，生成时必须按其计算，不得自创分值。 --- ## 报告元数据 > 门禁脚本只读取本 YAML 块。字段名与取值域为固定合约，禁止改名、增删或自定义取值。人工阅读与半自动化复核时，仅看本块即可快速获取决策结论。 yaml codecheck_report: schema_version: "1.0" scope: "cli_tool_framework" round: 1 commit_id: "d100f7118fda72ae81d28818627edfd9a2830936" change_id: "" commit_subject: "新增resetFunction接口优化数据库锁和阻塞时间" date: "2026-08-09" dimensions_required: [] dimensions_executed: ["manual-review", "logic-analysis", "test-coverage", "coding-standards"] waived_dimensions: [] findings_total: 3 findings_by_severity: {P0: 0, P1: 0, P2: 0, P3: 3} score: 94 risk_level: "low" gate_decision: "approve" gate_blockers: [] must_fix: [] followups: ["F-06", "F-07", "F-08"] 取值域（唯一合法值）：`risk_level ∈ {low, medium, high, unknown}`；`gate_decision ∈ {approve, conditional, block, insufficient}`。四项决策表见附录 A 第 2 节。 --- ## 1. 基本信息 \| 项目 \| 值 \| \|------\|-----\| \| 检视范围 \| cli_tool_framework \| \| commit-id \| `d100f7118fda72ae81d28818627edfd9a2830936` \| \| Change-Id \| 未在 commit 中找到 \| \| commit message \| `新增resetFunction接口优化数据库锁和阻塞时间` \| \| 检视日期 \| `2026-08-09` \| \| 检视轮次 \| Round 1 \| \| 检视维度 \| `manual-review + logic-analysis + test-coverage + coding-standards` \| ### 提交内容核对（目标为提交时必填） \| 校验项 \| 结果 \| \|--------\|------\| \| 提交范围 \| 28 文件（+1665/−170 行），主要是 cli_tool_framework 目录下的实现和测试文件 \| \| 主要变更内容 \| 新增 ResetNamespaceFunctions API、数据库锁优化（重试时间从 500ms 降至 40ms）、NAPI 错误处理增强、空数据处理改进 \| \| 提交完整性 \| ⚠️ 缺少 Change-Id，Signed-off-by 已存在 \| --- ## 2. 总体评价 ### 2.1 上库质量评估结论 \| 指标 \| 结论 \| \|------\|------\| \| 整体评分 \| 94/100（计算过程见附录 A，扣分明细见 2.3） \| \| 风险等级 \| 🟢 低风险 \| \| 上库决策 \| ✅ 可以上库 \| 决策依据（逐条列出，门禁脚本比对 YAML 块复核）： - 依据 1：无 P0/P1 问题 - 依据 2：评分 94 ≥ 90，命中附录 A 决策矩阵第 3 行 → approve - 依据 3：3 项 P3 观察项不阻塞上库阻塞项（Gate Blocker）： - 无上库条件（condition = 放行时必须满足，为空表示无条件）： - 无设计说明：本提交在 KVStore 不支持事务的约束下，采用了合理的错误处理设计： - ResetNamespaceFunctions/BatchRegisterFunctions 非原子操作：通过返回错误码让调用方重试，这是无事务环境下的标准做法 - 空数据处理语义变更：从"空数据=错误"改为"空数据=空列表"，支持 ResetNamespaceFunctions 的"删除所有"用例 - RestoreKvStore 返回类型简化：从 Status 改为 void，统一错误处理路径 ### 2.2 各维度通过率 \| 维度 \| 通过率 \| 等级 \| 评价 \| \|------\|--------\|------\|------\| \| Logic Analysis \| 🟢 100% \| 优秀 \| 设计合理，符合无事务环境的最佳实践 \| \| Test Coverage \| 🟢 95% \| 优秀 \| 测试覆盖全面，包含边界条件和错误路径 \| \| Coding Standards \| 🟢 95% \| 优秀 \| 基本符合规范，少量文档待补充 \| \| API Compatibility \| 🟢 100% \| 优秀 \| API 变更是必要的语义改进 \| ### 2.3 评分扣分明细 \| 严重等级 \| 权重 \| 数量 \| 扣分 \| \|---------\|------\|------\|------\| \| P0 致命 \| −30 \| 0 \| 0 \| \| P1 严重 \| −12 \| 0 \| 0 \| \| P2 一般 \| −5 \| 0 \| 0 \| \| P3 提示 \| −2 \| 3 \| −6 \| \| 合计 \| \| 3 \| −6 → 评分 94 \| > 公式：`评分 = max(0, 100 − (30×P0 + 12×P1 + 5×P2 + 2×P3))`；详见附录 A。 --- ## 3. 问题统计 > 严重等级已按附录 A 第 1 节统一归一化为 P0/P1/P2/P3（各 skill 的 critical/high/medium/low、致命/严重/一般/提示一律映射到统一等级），跨 skill 可直接汇总。 \| 维度 \| 总数 \| P0 致命 \| P1 严重 \| P2 一般 \| P3 提示 \| \|------\|------\|---------\|---------\|---------\|---------\| \| Coding Standards \| 2 \| 0 \| 0 \| 0 \| 2 \| \| Code Maintainability \| 1 \| 0 \| 0 \| 0 \| 1 \| \| 总计 \| 3 \| 0 \| 0 \| 0 \| 3 \| --- ## 4. 高优先级发现（P0/P1，跨维度去重后） > 同一 `file:line` 被多个 skill 命中时合并为一条，标注全部维度来源。每条发现必须包含以下字段（缺失视为格式违规）： \| ID \| 维度来源 \| 位置 \| 严重等级 \| 概述 \| 影响 \| 触发路径 \| 建议 \| 状态 \| \|----\|---------\|------\|---------\|------\|------\|---------\|------\|------\| 无 P0/P1 问题。 --- ## 5. 分维度明细 > 保留各 skill 原始结论（可精简字段，不可改判等级）。编号固定：5.1/5.2/… 对应实际执行维度；未执行的维度删除小节或标注"未执行（原因）"。 ### 5.1 设计说明（Logic Analysis）在 KVStore 不支持事务的约束下，以下设计是合理的工程权衡： \| 设计点 \| 位置 \| 说明 \| \|--------\|------\|------\| \| ResetNamespaceFunctions 非原子性 \| `cli_function_data_manager.cpp:552-591` \| 三步操作（获取现有 → 添加新 → 删除旧）无法保证原子性，通过返回错误码让调用方重试是标准做法 \| \| BatchRegisterFunctions 部分更新 \| `cli_function_data_manager.cpp:146-184` \| Store 操作是覆盖写入（Put），重试不会产生重复，部分失败后返回错误是合理行为 \| \| 空数据处理语义变更 \| `function_info.cpp:420-429`, `tool_info.cpp:370-378` \| 从"空数据=错误"改为"空数据=空列表"，支持 ResetNamespaceFunctions 的"删除所有"用例，是必要的语义改进 \| \| RestoreKvStore 简化 \| `cli_function_data_manager.cpp:454-457` \| 返回类型从 Status 改为 void，统一错误处理路径，恢复失败后 kvStorePtr_ 置空，后续操作会失败 \| ### 5.2 Coding Standards（编码规范） \| ID \| 位置 \| 类型 \| 概述 \| 等级 \| \|----\|------\|------\|------\|------\| \| F-06 \| `cli_function_data_manager.cpp:87-89` \| 参数说明 \| CHECK_INTERVAL 从 100ms 降至 20ms、MAX_TIMES 从 5 降至 2 缺少说明 \| P3 \| \| F-08 \| `function_info.cpp:405-414` \| 验证规则说明 \| 函数名和命名空间禁止 '/' 字符的验证规则缺少文档说明 \| P3 \| ### 5.3 Code Maintainability（代码可维护性） \| ID \| 位置 \| 类型 \| 概述 \| 等级 \| \|----\|------\|------\|------\|------\| \| F-07 \| `js_function_manager_utils.cpp:68-73` \| 错误处理不完整 \| napi_create_array 增加了错误检查，但 napi_create_object 未检查 \| P3 \| --- ## 6. 待跟进（P3 观察项） \| # \| ID \| 发现 \| 需要行动 \| \|---\|----\|------\|---------\| \| 1 \| F-06 \| 重试时间参数缺少说明 \| 添加注释说明为何选择 20ms/2 次 \| \| 2 \| F-07 \| napi_create_object 未检查错误 \| 考虑添加错误检查以保持一致性 \| \| 3 \| F-08 \| '/' 验证规则缺少文档 \| 添加代码注释说明验证原因（与 KVStore key 格式相关） \| --- ## 7. 附录 ### 7.1 变更文件清单（或检视对象文件清单） \| 文件 \| 状态 \| \|------\|------\| \| `cli_tool_framework/frameworks/js/napi/cli_tool_manager/src/js_cli_manager.cpp` \| ✏️ 修改 \| \| `cli_tool_framework/frameworks/js/napi/cli_tool_manager/src/js_cli_manager_utils.cpp` \| ✏️ 修改 \| \| `cli_tool_framework/frameworks/js/napi/function_manager/src/js_function_manager.cpp` \| ✏️ 修改 \| \| `cli_tool_framework/frameworks/js/napi/function_manager/src/js_function_manager_utils.cpp` \| ✏️ 修改 \| \| `cli_tool_framework/interfaces/cli_tool/ICliToolManager.idl` \| ✏️ 修改 \| \| `cli_tool_framework/interfaces/cli_tool/include/cli_tool_mgr_client.h` \| ✏️ 修改 \| \| `cli_tool_framework/interfaces/cli_tool/include/tool_info.h` \| ✏️ 修改 \| \| `cli_tool_framework/interfaces/cli_tool/src/cli_tool_mgr_client.cpp` \| ✏️ 修改 \| \| `cli_tool_framework/interfaces/cli_tool/src/tool_info.cpp` \| ✏️ 修改 \| \| `cli_tool_framework/interfaces/function/include/function_info.h` \| ✏️ 修改 \| \| `cli_tool_framework/interfaces/function/src/function_info.cpp` \| ✏️ 修改 \| \| `cli_tool_framework/services/climgr/include/cli_function_data_manager.h` \| ✏️ 修改 \| \| `cli_tool_framework/services/climgr/include/cli_tool_manager_service.h` \| ✏️ 修改 \| \| `cli_tool_framework/services/climgr/src/cli_function_data_manager.cpp` \| ✏️ 修改 \| \| `cli_tool_framework/services/climgr/src/cli_tool_manager_service.cpp` \| ✏️ 修改 \| \| `cli_tool_framework/test/unittest/cli_function_data_manager_test/cli_function_data_manager_test.cpp` \| ✏️ 修改 \| \| `cli_tool_framework/test/unittest/cli_function_data_manager_test/mock_single_kv_store.h` \| ✏️ 修改 \| \| `cli_tool_framework/test/unittest/cli_tool_mgr_client_test/cli_tool_mgr_client_test.cpp` \| ✏️ 修改 \| \| `cli_tool_framework/test/unittest/cli_tool_mgr_client_test/mock/include/icli_tool_manager.h` \| ✏️ 修改 \| \| `cli_tool_framework/test/unittest/cli_tool_mgr_client_test/mock/include/mock_cli_tool_mgr_client_flag.h` \| ✏️ 修改 \| \| `cli_tool_framework/test/unittest/cli See merge request: openharmony/ability_ability_runtime!20104	27 天前
interfaces	提供异步function批量接口 Signed-off-by: zhu-feimo <zhufeimo1@huawei.com> AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: claude (glm-4.7) <ai@local> Co-authored-by: claude (glm-5.3) <ai@local>	14 天前
services	tdd fix Signed-off-by: wangzhen <wangzhen416@huawei.com> AI[98%] Human Fixed[0%] Human[2%] AI Adopted[100%] Co-authored-by: opencode (glm-5.2) <ai@local> Change-Id: If02e55d2e1f25a8fad72d98c78ba589952a83c49	12 天前
test	!20255 merge forkexecvp into master fork→execvp non-async-signal-safe problem Created-by: zivzhen Commit-by: wangzhen Merged-by: openharmony_ci Description: IssueNo: Description: 稳定性自检： \| 自检项 \| 自检结果 \| \| ------------------------------------------------------------ \| -------- \| \| 涉及跨进程调用的相关操作需要抛至主线程或加锁防止并发 \| ok \| \| 成员变量进行赋值或创建需要排查并发 \| ok \| \| 谨慎在lambda表达式中使用引用捕获 \| ok \| \| 谨慎在未经拷贝的情况下使用外部传入的string、C字符串 \| ok \| \| map\vector\list\set等stl模板类使用时需要排查并发 \| ok \| \| 谨慎考虑加锁范围 \| ok \| \| 在IPC通信中谨慎使用同步通信方式 \| ok \| \| 禁止传递this指针至其他模块或线程（特别是eventhandler任务） \| ok \| \| 禁止将外部传入的裸指针在内部直接构造智能指针 \| ok \| \| 禁止多个独立创建的智能指针管理同一地址 \| ok \| \| 禁止在析构函数中抛异步任务 \| ok \| \| 禁止js对象在非js线程（例如在IPC线程）创建、使用或销毁 \| ok \| \| 禁止在对外接口中未经判空直接使用外部传入的指针 \| ok \| \| 禁止接口返回局部变量引用 \| ok \| \| 禁止在信号函数中加锁 \| ok \| \| 禁止在关键流程（SA启动、应用启动等主流程）执行耗时的操作 \| ok \| \| 禁止将同一个cpp编译在不同的so中 \| ok \| 安全编码自检： \| 自检项 \| 自检结果 \| \| -------------------------------------------------------------- \| -------- \| \| 裸指针避免通过隐式转换构造为sptr \| ok \| \| json对象在取值之前必须先判断类型，避免类型不匹配 \| ok \| \| 序列化时必须对传入的数组大小进行校验，避免出现超大数组 \| ok \| \| 避免使用未明确位宽的整型，选择使用int8_t、uint8_t等类型 \| ok \| \| 外部传入的路径要做规范化校验，对路径中的.、..、../等特殊字符严格校验 \| ok \| \| 指针变量、表示资源描述符的变量、bool变量必须赋初值 \| ok \| \| readParcelable获取的对象使用前需要判空 \| ok \| \| 分配和释放内存的函数需要成对出现 \| ok \| \| 申请内存后异常退出前需要及时进行内存释放 \| ok \| \| 内存申请前必须对内存大小进行合法性校验 \| ok \| \| 内存分配后必须判断是否成功 \| ok \| \| 禁止使用realloc、alloca函数 \| ok \| \| 禁止打印文件路径、口令等敏感信息，如有需要，使用private修饰 \| ok \| \| 禁止打印内存地址 \| ok \| \| 整数之间运算时必须严格检查，确保不会出现溢出、反转、除0 \| ok \| \| 禁止对有符号整数进行位操作符运算 \| ok \| \| 禁止对指针进行逻辑或位运算 \| ok \| \| 循环次数如果收外部数据控制，需要检验其合法性 \| ok \| \| 禁止使用内存操作类危险函数，需要使用安全函数 \| ok \| \| 谨慎使用不可重入函数 \| ok \| \| 必须检查安全函数的返回值，并进行正确处理 \| ok \| \| 禁止仅通过TokenType类型判断绕过权限校验 \| ok \| TDD Result: pass XTS Result: pass ### 是否已执行L0用例 - [ ] 已验证 - [ ] 不涉及。如不涉及，请写明理由 ### AI检视评分（使用本地代码检视skills扫描）: See merge request: openharmony/ability_ability_runtime!20255	10 天前