Skip to content

3.3.0 自定义快捷触发配置#10

Merged
handy-git merged 1 commit intomainfrom
3.3.0
Feb 26, 2026
Merged

3.3.0 自定义快捷触发配置#10
handy-git merged 1 commit intomainfrom
3.3.0

Conversation

@handy-git
Copy link
Owner

No description provided.

@handy-git handy-git self-assigned this Feb 25, 2026
Copy link

@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: 748e6cbc69

ℹ️ 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".

// 原消息内容
chatParam.setMessage(message);
// 快捷键节点替换
ShortcutUtil.convert(player, channel, chatParam);

Choose a reason for hiding this comment

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

P2 Badge Evaluate shortcuts before mention rewriting

This call runs after ChatUtil.at(...) has already rewritten the user’s input (it can prepend @ and inject color formatting when a token matches an online player), but ShortcutUtil.convert matches against chatParam.getMessage() rather than the raw chat text. In practice, a shortcut pattern that should match the original input can stop matching whenever the same word is also a player name, so trigger behavior becomes dependent on who is online.

Useful? React with 👍 / 👎.

@handy-git handy-git merged commit 39174f0 into main Feb 26, 2026
1 check passed
@handy-git handy-git deleted the 3.3.0 branch February 26, 2026 02:22
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.

1 participant