Add Slack Share Update Trigger

[stevysmith]

Slack Share Update, three triggers:

• When joining a room, ping a message out on Slack setting the topic of conversation (or asking to Pair)
• When leaving a room, ping a message out on Slack with a summary of what you talked about and any follow-up
• When finishing a call, ping a message out on Slack with a summary of what you talked about and any follow-up

All messages are optional, messages can be previewed before sending

[github-actions]

Thanks for your contribution, @stevysmith! 🎉

We've done some automated sense checks on your trigger, slack-share-update. All looks great! Someone from Tuple will now manually review your trigger's code and get back to you.

[stevysmith]

@mcmillan was this a bit too OTT for a single trigger do you think 😁? I can make them individual maybe?

[mcmillan]

@stevysmith Not necessarily OTT at all! The thing that makes me a bit wary of approving this is the JSON escaping logic – right now we're doing that using sed:

ESCAPED_MESSAGE=$(echo "$FULL_MESSAGE" | sed 's/"/\\"/g; s/\\/\\\\/g; s/\//\\\//g; s/`/\\`/g')

This feels a little precarious to me and could perhaps be misused by a bad actor. I think there's a few options for getting around this:

1. Shell out to something like jq:
   • You could build a "safe" JSON string via jq -n --arg full_message $FULL_MESSAGE '{"message": $full_message}'
   • This would require users of the script to install jq, e.g. via brew install jq on macOS – you should include this in the hook's README
2. Use a higher-level language (e.g. Ruby / Python)
   • Bash is inherently a bit of a pig – using something like Ruby or Python would make it a lot easier to reuse functionality between your various hooks and solve the JSON escaping problem
   • I'm happy to help port your hooks if needs be!

More generally, I reckon this trigger is very specific to your workflow – if we could separate it out into a few different triggers to make it more generally useful, that'd be ace, but if not let's get it it in as-is!

[stevysmith]

Thanks! After some use it does get a bit intense with all three being active. It might be better with them separated out and then people can pick and choose.

I'll rework it..