Skip to content

Comments

Script Manager Click Crash Fix & API Enhancement#8

Merged
cwuom merged 7 commits intocwuom:masterfrom
huajiqaq:patch3
Feb 21, 2026
Merged

Script Manager Click Crash Fix & API Enhancement#8
cwuom merged 7 commits intocwuom:masterfrom
huajiqaq:patch3

Conversation

@huajiqaq
Copy link
Contributor

描述 / Description

修复了点击脚本管理器崩溃的问题 新增了更多脚本API 并补充了Release变体的测试方法

类型 / Type

  • Bug 修复 / Bug Fix
  • 新功能 / New Feature
  • 文档更新 / Documentation Update
  • 其他(请描述)/ Other (please describe):

修复或解决的问题 / Issues Fixed or Closed by This PR

清单 / Checklist

  • 我已阅读并遵循贡献指南 / I have read and followed the contribution guidelines
  • 我已在本地测试这些更改 / I have tested these changes locally
  • 我已更新相关文档或注释(如适用) / I have updated relevant documentation or comments (if applicable)
  • 我确认此更改不会破坏任何原有功能 / I confirm this change does not break any existing features
  • 我已进行多版本适配(如适用) / I have used MMVersion for version compatibility (if applicable)
  • 我已在多个微信版本上测试此更改(如适用) / I have tested this change on multiple WeChat versions (if applicable)

其他信息 / Additional Information

@cwuom
Copy link
Owner

cwuom commented Feb 14, 2026

WePkgDispatcher 在之前的测试中并不会导致无限递归问题,解释一下为什么要修改它?

}

/**
* 获取当前用户ID
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里的注释有误导性,容易将它联想成wxid,实际上它获取的是微信号

@huajiqaq
Copy link
Contributor Author

WePkgDispatcher 在之前的测试中并不会导致无限递归问题,解释一下为什么要修改它?

我认为在脚本中 sendCgi 以及发送信息可能会出现无限递归的情况

@huajiqaq
Copy link
Contributor Author

我再修改修改吧

@cwuom
Copy link
Owner

cwuom commented Feb 16, 2026

WePkgDispatcher 在之前的测试中并不会导致无限递归问题,解释一下为什么要修改它?

我认为在脚本中 sendCgi 以及发送信息可能会出现无限递归的情况

WePkgDispatcher 是始终运行(ApiHookItem)的,你可以在它初始化完成以后手动调用一次 sendCgi 进行测试,经过我之前的测试,它并没有无限递归,一般无限递归出现的话多半是 WePkgDispatcher 中调用了sendCgi 或者间接触发了 dispatch

最好还是测试一下

@huajiqaq
Copy link
Contributor Author

huajiqaq commented Feb 20, 2026

因为我之前为脚本提供了主动sendCgi和发送信息的接口 开启脚本功能后如果调用了相关api就会造成无限递归 现在由2秒改为了500毫秒延迟

@cwuom cwuom merged commit 23ccf50 into cwuom:master Feb 21, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants