Skip to content

优化小程序订阅消息字符串处理#4049

Merged
binarywang merged 2 commits into
binarywang:developfrom
buaazyl:develop
Jun 8, 2026
Merged

优化小程序订阅消息字符串处理#4049
binarywang merged 2 commits into
binarywang:developfrom
buaazyl:develop

Conversation

@buaazyl
Copy link
Copy Markdown
Contributor

@buaazyl buaazyl commented Jun 8, 2026

简单处理长度和格式,覆盖常见场景

简单处理长度和格式,覆盖常见场景
@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented Jun 8, 2026

🤖 Augment PR Summary

Summary: 本 PR 在小程序订阅消息 WxMaSubscribeMessage 中新增模板字段值的预处理逻辑,尝试减少因长度/格式不符导致的发送失败。

Changes:

  • 新增多组预编译 Pattern,对 number/letter/symbol/phone_number/name/phrase 等字段进行过滤
  • addData 时统一调用 resetValue,对空值做兜底、对超长值做截断
  • 对手机号字段做 + 号数量限制(仅保留一个)
Technical Notes: 通过静态正则缓存避免重复编译;该处理会直接修改传入的 MsgData 内容。

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

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

Review completed. 3 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2c0a44d9cb

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@binarywang binarywang merged commit 1037dfd into binarywang:develop Jun 8, 2026
@binarywang binarywang added this to the 4.8.4 milestone Jun 8, 2026
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