Match on raw text instead of formatted text on incoming messages · slackapi/hubot-slack#241

(3 评论) (0 反应) (0 负责人)CoffeeScript (2,339 star) (637 fork)batch import

enhancementgood first issue

描述

This was originally filed at https://github.com/github/hubot/issues/1053 but I believe this to be an adapter specific problem. From @callumacrae:

We have a script which starts like this:
robot.hear /some sentence here/i, (msg) ->
However, it doesn't hear it if the words are formatted at all, e.g. "some sentence here".

Would it be possible / desired to remove formatting from messages before parsing them?

I suggested this:

Slack already has a place it strips formatting for links. That'd be a good place to do it for other things: https://github.com/slackhq/hubot-slack/blob/master/src/slack.coffee#L153-L192

I think it is going to be up to the adapters to strip formatting like this. They probably should preserve the raw message in case an adapter-specific script wants to do something about, like hubot-slack does here: https://github.com/slackhq/hubot-slack/blob/master/src/slack.coffee#L112-L120

贡献者指南

技术栈: javascriptnodejs
领域: backend
议题类型: feature
难度: 2
预计时间: under 1 hour
活动状态: stale
清晰度: clear
前置要求: Familiarity with CoffeeScript
新手友好度: 50
研究方向: Focus on the slack.coffee file, specifically the message formatting logic around lines 153-192 where link formatting is stripped. Extend that logic to strip other formatting (bold, italic, etc.) before passing the text to the robot.hear function. Ensure the original formatted message is preserved via the rawMessage property as shown in lines 112-120. Test with messages containing various formatting to confirm the regex matches work on raw text.