已关注词汇中数字匹配的规则

zogstrip · 2026年二月21日 10:54

这次应该能把问题彻底解决。我们之前在 Ruby 版本和 JS 版本使用的正则表达式之间存在一些不一致，但现在不再需要了。

github.com/discourse/discourse

FIX: unify watched word boundary regex across Ruby and JS engines (#37965)

main ← fix/unify-word-boundary-regex

opened 11:54PM - 20 Feb 26 UTC

ZogStriP

+122 -251

The CJK fix (d7a53ada16) introduced separate boundary patterns for Ruby and JS e…ngines in `match_word_regexp`. The Ruby engine used `[:word:]` (which includes digits), while the JS engine used `\P{L}` (non-Letter). Since digits are not letters, the JS pattern treated them as valid word boundaries — causing "123Test" to match as "3Test" and standalone number watched words like "123" to match inside "abc123". Replace both engine-specific patterns with a single unified pattern using Unicode property classes (`\p{L}`, `\p{M}`, `\p{N}`, `\p{Pc}`) that work identically in Ruby and JavaScript. This treats letters, marks, numbers, and connector punctuation as word characters in boundary checks, which fixes the number-matching bug for JS consumers while preserving the existing correct behavior on the Ruby side. Since `match_word_regexp` no longer branches on engine, remove the now-dead `engine:` parameter from all 5 method signatures that threaded it through (`match_word_regexp`, `word_to_regexp`, `regexps_for_action`, `compiled_regexps_for_action`, `serialized_regexps_for_action`) and all call sites passing `engine: :js` (serializers, pretty_text). https://meta.discourse.org/t/396110 https://meta.discourse.org/t/396109 Follow-up to d7a53ada16 (#37844)

話題		回覆	觀看
Hope Watched words adds support for non-English characters Bug	1	119	2026年二月16日
Russian characters in Watched Words list are failing to be properly identified Bug watched-words	1	580	2021年二月10日
Can't enter watched words regex to catch phone numbers Support regex , watched-words	2	181	2025年五月17日
Watched word regular expression crash Bug watched-words	6	971	2023年十一月29日
How to use Discourse regexes with watched words? Support	6	2527	2019年五月30日

已关注词汇中数字匹配的规则

相關主題