已合并
fix: fix a crash caused by removing a NAPI reference on JS thread #1801
gyange创建于 2025年11月11日
fix: fix a crash caused by removing a NAPI reference on JS thread #1801
已合并
gyange创建于 2025年11月11日
gyange成员
2025年11月11日

描述

变更日志

测试步骤

合入前自检

    • 如需合入77分支,PR地址:
likedislike
Pull Request已成功合入, 合并人@huangyouhua
(感谢 gyange 的贡献)
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
openharmony_ciopenharmony_ci成员
2025年11月11日 添加了label:dco检查成功
gyange成员
2025年11月11日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

该提交没有关联任何Issue,请先关联一个Issue后再触发构建
The PR is not associated with any Issue, Please associate the PR with one Issue before trigger build.

likedislike
Ggyange成员
2025年11月11日 修改了pull request 的描述
Ggyange成员
2025年11月11日 关联了issue:[Bug]: fix a crash caused by removing a NAPI reference on JS thread
gyange成员
2025年11月11日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

首次触发
门禁构建开始,包含静态检查、代码编译【ohos_react_native_master编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/69129ac264650f998b60279d/runlist

likedislike
openharmony_ciopenharmony_ci成员
2025年11月11日 添加了label:编译失败
Ggyange成员
2025年11月11日 强制推送  1 个提交:e87a198c-[Fix]: fix a crash caused by removing a NAPI reference on JS thread
openharmony_ciopenharmony_ci成员
2025年11月11日 删除了label:编译失败
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

代码有更新,重置PR验证状态

likedislike
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

监测到PR: https://gitcode.com/openharmony-sig/ohos_react_native/merge_requests/1801 代码有更新,当前正在执行的门禁任务已自动中止,如有需要,可重新触发

likedislike
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

感谢提交 Pull Requests !此PR未通过DCO校验。
校验失败可能原因:

1. 未签署“DCO协议”(开发者原创声明协议),在线签署、查看签署状态

2. Commits 中未包含 Signed-off-by信息,参考FAQ处理

修复上述问题后,在PR的评论框输入“check dco” ,单击”评论”,系统将再次进行DCO校验。

当前检测到如下Commits 未包含Signed-off-by信息:


Thanks for submitting a pull request. This pull request has not passed the DCO check.
Possible causes:

1. You have not signed the Developer Certificate of Origin (DCO). Sign the DCO and check DCO status.

2. The commits do not contain the Signed-off-by information. To resolve this issue, see FAQs.

After resolving the preceding issues, enter check dco in the comment box of this pull request and click Comment. The system will check DCO status again.

The following commits do not contain the Signed-off-by information:

likedislike
openharmony_ciopenharmony_ci成员
2025年11月11日 删除了label:dco检查成功
openharmony_ciopenharmony_ci成员
2025年11月11日 添加了label:dco检查失败
Ggyange成员
2025年11月11日 强制推送  1 个提交:11a47f29-[Fix]: fix a crash caused by removing a NAPI reference on JS thread
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
openharmony_ciopenharmony_ci成员
2025年11月11日 删除了label:dco检查失败
openharmony_ciopenharmony_ci成员
2025年11月11日 添加了label:dco检查成功
gyange成员
2025年11月11日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译【ohos_react_native_master编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6912a91464650f998b6d3ccf/runlist

likedislike
openharmony_ciopenharmony_ci成员
2025年11月11日 添加了label:编译失败
openharmony_ciopenharmony_ci成员
2025年11月11日 添加了label:静态检查成功
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

代码门禁未通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6912a91464650f998b6d3ccf/runlist

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result package
1 ohos_react_native_master failed(compile failed) >>>

likedislike
Ggyange成员
2025年11月11日 推送  1 个提交:4fe58841-add package
openharmony_ciopenharmony_ci成员
2025年11月11日 删除了label:静态检查成功
openharmony_ciopenharmony_ci成员
2025年11月11日 删除了label:编译失败
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

代码有更新,重置PR验证状态

likedislike
gyange成员
2025年11月11日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

感谢提交 Pull Requests!
Thanks for submitting a pull request.

likedislike
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译【ohos_react_native_master编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6912db9064650f998b9b5d66/runlist

likedislike
openharmony_ciopenharmony_ci成员
2025年11月11日 添加了label:编译成功
openharmony_ciopenharmony_ci成员
2025年11月11日 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
2025年11月11日 通过测试
openharmony_ci
openharmony_ci成员
2025年11月11日 评论:

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6912db9064650f998b9b5d66/runlist

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result package
1 ohos_react_native_master success >>>

likedislike
openharmony_dcp
openharmony_dcp成员
2025年11月11日 评论:

提交次数已达到或超过3次,为了保证上库效率和减少构建资源浪费,请点击链接观看学习视频,谢谢!https://dcp.openharmony.cn/workbench/video/videoDisplay

likedislike
zichenyang21
zichenyang212025年11月11日进行代码检视1
tester/harmony/react_native_openharmony/src/main/cpp/RNOH/MountingManager.h
@@ -27,3 +27,3 @@
2727 public:
2828 using Shared = std::shared_ptr<MountingManager>;
29- using Weak = std::weak_ptr<MountingManager>;
29+ class Weak {
zichenyang21
zichenyang212025年11月11日评论:

这里的修改是否会产生非兼容变更?

likedislike
Pancake_Su
Pancake_Su成员2025年11月11日进行代码检视1
react-native-harmony/react-native.config.js
@@ -18,0 +8,4 @@
8+try {
9+ const { config: harmonyConfig } = require('@rnoh/react-native-harmony-cli');
10+} catch (err) {
11+ console.warn(
Pancake_Su
Pancake_Su2025年11月11日评论:

【建议】【基本代码】【日志输出问题】异常日志等级是否使用error

likedislike
zichenyang21zichenyang21
2025年11月11日 解决了最后一个问题
zichenyang21zichenyang21
2025年11月11日 将zichenyang21设为评审人
zichenyang21zichenyang21
2025年11月11日 通过了评审
huangyouhuahuangyouhua
2025年11月11日 通过审查
huangyouhuahuangyouhua
2025年11月11日 添加了 squash commit 备注:fix: fix a crash caused by removing a NAPI reference on JS thread
huangyouhuahuangyouhua
2025年11月11日 合入了pull request