[codex] Prevent Telegram market symbol auto-links#109
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a12706938d
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
|
|
||
| DEFAULT_TELEGRAM_BOT_API_BASE_URL = "https://api.telegram.org" | ||
| _TELEGRAM_MARKET_SYMBOL_LINK_RE = re.compile(r"(?<![A-Za-z0-9_])([A-Z0-9]{1,12})\.([A-Z]{2,4})(?![A-Za-z0-9_])") |
There was a problem hiding this comment.
Preserve uppercase URL hosts before breaking symbols
When a notification contains an uppercase host/domain, this pattern still matches the domain token because it runs over the whole message and only excludes adjacent alphanumerics; for example https://EXAMPLE.COM/report is rewritten with U+2060 inside the host. That breaks preservation of real links even though the added test only covers lowercase URLs, so guard URL spans or otherwise avoid substituting domain matches before sending.
Useful? React with 👍 / 👎.
Summary
Validation