已开启
fix: 修改react-native-restart三方库82分支AI文档质量扫描整改 #29
fix: 修改react-native-restart三方库82分支AI文档质量扫描整改 #29
已开启
fanwangah创建于 25 天前
fanwangah
25 天前

fix: 修改react-native-restart三方库82分支AI文档质量扫描整改

likedislike
合并受阻
Ffanwangah
25 天前 关联了issue:fix: 修改react-native-restart三方库82分支AI文档质量扫描整改
openharmony_ci
openharmony_ci成员
25 天前 评论:

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

likedislike
openharmony_ciopenharmony_ci成员
25 天前 添加了label:dco检查成功
wupingyuan
wupingyuan
25 天前 评论:

已审视,无问题

likedislike
cpf-manager
cpf-manager25 天前进行代码检视1
README.md
@@ -244,3 +267,2 @@
244267| restart | 重启RN工程 | Function | no | All | yes |
245-| Restart | 重启RN工程 | Function | no | All | yes |
246-| getReason | 获取重启原因 | Function | no | All | yes |
268+| getReason | 获取重启原因 | Function | no | HarmonyOS | yes |
cpf-manager
cpf-manager25 天前评论:

【AI-Review】【一般】【基础代码问题】【代码逻辑错误】API 表格删除 Restart 但代码仍保留该方法

● 问题:本次 PR 将 API 表格中的 Restart 行删除,但实际代码中 Restart 方法仍然存在并对外暴露:

  • src/NativeRestart.ts:30 的 Spec 接口声明了 Restart(reason?: string): void;
  • harmony/restart/src/main/ets/RNRestartTurboModule.ts:76public Restart = this.restart;Restart 作为 restart 的别名实现

用户通过 import RNRestart from "react-native-restart" 拿到的模块对象上 RNRestart.Restart() 仍然可调用。

● 影响:文档与代码不一致。开发者阅读 README 时会认为 Restart 不可用,但 TypeScript IntelliSense 和运行时仍会暴露该方法,导致使用者对 API 可用性产生困惑,也不利于后续维护时判断 Restart 是否可以安全删除。

● 建议:二选一:

  1. Restart 仍需保留(作为兼容别名),在 API 表格中补回 Restart 行并标注其为 restart 的别名(或标记为 deprecated)。
  2. Restart 已废弃,应在代码中同步移除 NativeRestart.ts Spec 接口的 Restart 声明与 RNRestartTurboModule.ts 中的 public Restart = this.restart; 实现,使文档与代码一致。
likedislike
cpf-manager
cpf-manager25 天前进行代码检视1
README_en.md
@@ -245,3 +268,2 @@
245268| restart | restart your react native project | Function | no | All | yes |
246-| Restart | restart your react native project | Function | no | All | yes |
247-| getReason | get the cause of the last restart | Function | no | All | yes |
269+| getReason | get the cause of the last restart | Function | no | HarmonyOS | yes |
cpf-manager
cpf-manager25 天前评论:

【AI-Review】【一般】【基础代码问题】【代码逻辑错误】API 表格删除 Restart 但代码仍保留该方法

● 问题:本次 PR 将 API 表格中的 Restart 行删除,但实际代码中 Restart 方法仍然存在并对外暴露:

  • src/NativeRestart.ts:30 的 Spec 接口声明了 Restart(reason?: string): void;
  • harmony/restart/src/main/ets/RNRestartTurboModule.ts:76public Restart = this.restart;Restart 作为 restart 的别名实现

用户通过 import RNRestart from "react-native-restart" 拿到的模块对象上 RNRestart.Restart() 仍然可调用。

● 影响:文档与代码不一致。开发者阅读 README 时会认为 Restart 不可用,但 TypeScript IntelliSense 和运行时仍会暴露该方法,导致使用者对 API 可用性产生困惑,也不利于后续维护时判断 Restart 是否可以安全删除。

● 建议:二选一:

  1. Restart 仍需保留(作为兼容别名),在 API 表格中补回 Restart 行并标注其为 restart 的别名(或标记为 deprecated)。
  2. Restart 已废弃,应在代码中同步移除 NativeRestart.ts Spec 接口的 Restart 声明与 RNRestartTurboModule.ts 中的 public Restart = this.restart; 实现,使文档与代码一致。
likedislike
yangweiping
yangweiping25 天前进行代码检视2
README.md
已过期
@@ -1,0 +1,0 @@
11> 文档模板:v0.4.1
yangweiping
yangweiping25 天前评论:

模板做了一些变更,后续的文档需要按照新的去更改

变更点:

  1. 版本号由0.4.1更新为0.4.2
  2. 顶部所属关系映射表,三方库版本修改为三方库版本(npm地址),链接修改为原先npm地址的链接,同时删除npm地址字段
  3. 增加源码地址字段,值为对应分支,并添加对应分支跳转链接

另外这些也一并修改下:
1.openahrmony-sig的链接修改为CPF-RN。
2.

不再支持居中,修改为p align="center"方式居中。
3.部分库链接到usage-docs,迁移后相对路径不存在,应该使用完整路径。
例如:对于未发布到npm的旧版本,请参考安装指南安装tgz包。链接应该修改为 https://gitcode.com/CPF-RN/usage-docs/blob/master/zh-cn/tgz-usage.md
例如:> [!TIP] 如需使用直接链接源码,请参考直接链接源码说明。链接应该修改为 https://gitcode.com/CPF-RN/usage-docs/blob/master/zh-cn/link-source-code.md

likedislike
System
系统消息系统
25 天前 评论:

changed this line on 2570855f view diff detail

Ffanwangah
25 天前 强制推送  1 个提交:2570855f-fix: 修改react-native-restart三方库82分支AI文档质量扫描整改
openharmony_ci
openharmony_ci成员
25 天前 评论:

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

likedislike
Ffanwangah
25 天前 强制推送  1 个提交:49f57c9f-fix: 修改react-native-restart三方库82分支AI文档质量扫描整改
openharmony_ci
openharmony_ci成员
25 天前 评论:

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

likedislike
Ffanwangah
25 天前 强制推送  1 个提交:d9f15421-fix: 修改react-native-restart三方库82分支AI文档质量扫描整改
openharmony_ci
openharmony_ci成员
25 天前 评论:

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

likedislike
cpf-manager
cpf-manager25 天前进行代码检视1
README.md
@@ -244,3 +274,2 @@
244274| restart | 重启RN工程 | Function | no | All | yes |
245-| Restart | 重启RN工程 | Function | no | All | yes |
246-| getReason | 获取重启原因 | Function | no | All | yes |
275+| getReason | 获取重启原因 | Function | no | HarmonyOS | yes |
cpf-manager
cpf-manager25 天前评论:

【AI-Review】【一般】【基础代码问题】【文档问题】getReason的Platform列值与列定义不一致

● 问题:README.md第266行定义"Platform"列表示"该属性在原三方库上支持的平台"。第275行将getReason的Platform设为"HarmonyOS",但getReason是HarmonyOS适配新增的API,原社区库react-native-restart(https://github.com/avishayil/react-native-restart)的API仅有restart和Restart(已废弃),不存在getReason接口,且原社区库仅支持iOS/Android平台、不支持HarmonyOS。因此"HarmonyOS"与列定义矛盾——原三方库既没有getReason接口,也不支持HarmonyOS平台。

触发路径:用户阅读API表 → 看到"Platform"列定义提示"原三方库上支持的平台" → 查看getReason行Platform为"HarmonyOS" → 误判getReason是原社区库在HarmonyOS上就已支持的接口 → 混淆了"原库能力"与"HarmonyOS扩展能力"的边界。

● 影响:一般。用户会误解getReason的来源,误以为它是原社区库的跨平台能力,而实际上它是HarmonyOS专属扩展。这会影响开发者对该API可移植性的判断(例如误以为在其他平台的react-native-restart中也能使用getReason)。

● 建议:方案一:将getReason的Platform改为"N/A"(表示原库不存在该接口),通过"HarmonyOS Support"列的"yes"表明鸿蒙支持,这样与列定义保持一致;方案二:更新"Platform"列定义,补充说明"HarmonyOS扩展API的Platform标记为HarmonyOS"。README_en.md第276行存在同样问题,需同步修改。

likedislike
fanwangah
24 天前 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
24 天前 评论:

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

likedislike
openharmony_ciopenharmony_ci成员
24 天前 添加了label:静态检查失败
openharmony_ci
openharmony_ci成员
24 天前 评论:

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

静态检查:

# check type result report
1 codeCheck noPass >>>

likedislike