已关闭
Add abilityms_src source_set for test targets, drop dead abilityms_static #20363
Add abilityms_src source_set for test targets, drop dead abilityms_static #20363
已关闭
m0_65533854创建于 9月2日关闭于 9月3日
m0_65533854
m0_65533854
9月2日

Close #16134

问题

libabilityms.map 为让测试链接通过,导出了约 100 个类模式;生产消费方(appkit ×2、tools/cc ×1)经审计全部零符号引用(纯 include 路径依赖)。

方案(步骤 1/2,map 收敛随后单独提交)

  1. 僵尸目标 abilityms_static(从未编译,deps 与 so 漂移:缺 modular_object / agent_extension_connection / agent_manager / cli_tool_client / ability_observer_ipc 等)原位替换为 ohos_source_set("abilityms_src")
  2. source_set 与 so 完全对齐:同 sources/configs/include_dirs/deps/external_deps,14 个 feature 条件块逐一比对一致;补 public_configs(cli_tool include)保证测试编译 insight 源码可达;去掉 tdd_config(-flto=full 位码进 host/mac 测试链接有兼容风险)
  3. 750 个测试 BUILD 切换(801 处 dep,test/ + tools/ohos-aa/tests + tools/test/,残留 0)
  4. 生产消费方保持 dep so 不动

影响面

751 文件 +810/-806(绝大部分为单行 dep 切换);so 与 map 本步零改动,生产行为不变。

likedislike
当前Pull Request已关闭, 关闭人@m0_65533854
afwk_helper成员
9月2日 评论:

开始进行AI检视!

AI review has been started, please wait...

likedislike
afwk_helper成员
9月2日 评论:
check type result report
start ai_review pass -
likedislike
m0_65533854m0_65533854
9月2日 修改了pull request 的描述
openharmony_ciopenharmony_ci成员
9月2日 添加了label:waiting_on_author
afwk_helper成员
9月2日 评论:

⚠️ 🤖 AI 代码检视报告 ⚠️

总体评估: NEEDS_ATTENTION

问题统计:

  • 总问题数: 0
  • 严重问题: 0
  • 高危问题: 0

摘要:
PR将僵尸目标abilityms_static替换为abilityms_src source_set并切换750个测试BUILD文件依赖,架构方向合理,但由于未提供实际代码diff和完整文件内容,无法进行逐行深入审查,需补充diff内容后重新评审。

📊 详细报告
查看完整的审查详情,包括具体的问题描述、建议和代码位置:
🔗 查看详细报告


此评论由 OpenHarmony Insight 代码审查系统自动生成

likedislike
此处折叠了139条消息 查看更多
openharmony_ciopenharmony_ci成员
9月2日 删除了label:waiting_on_author
openharmony_ciopenharmony_ci成员
9月2日 添加了label:waiting_for_review
openharmony_ci
openharmony_ci成员
9月2日 评论:

您好,Committer @jayleehw @zhangyafei-echo @littlejerry1 @jsjzju @wkljy @yzkp @ccllee1 @yangxuguang-huawei @li-weifeng2024 @wendel ,请分配检视人员检视该PR,可以通过命令"assign [@someone_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @jayleehw @zhangyafei-echo @littlejerry1 @jsjzju @wkljy @yzkp @ccllee1 @yangxuguang-huawei @li-weifeng2024 @wendel . Please assign someone to review the PR. You can assign a reviewer by using the command "assign [@someone_id]", or you can comment "assign" to review the PR by yourself.

likedislike
m0_65533854m0_65533854
9月3日 关闭了 pull request
openharmony_dcp
openharmony_dcp成员
24 天前 评论:

你因提交次数过多被限制提交,为保障上库效率、避免资源浪费,请先观看学习视频:https://dcp.openharmony.cn/workbench/video/videoDisplay

likedislike