feat(slack): add remove reaction tool#3414
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview Updates the Slack block to support a new Written by Cursor Bugbot for commit f5da9bc. Configure here. |
Greptile SummaryThis PR cleanly adds a Key changes:
Safe to merge — the implementation is a well-structured addition with no logic errors or security concerns. Confidence Score: 5/5
Sequence DiagramsequenceDiagram
participant U as User/LLM
participant B as Slack Block (slack.ts)
participant T as slackRemoveReactionTool
participant R as /api/tools/slack/remove-reaction
participant S as Slack API (reactions.remove)
U->>B: operation = "unreact", channel, reactionTimestamp, emojiName
B->>T: resolve tool → slack_remove_reaction<br/>params: { accessToken|botToken, channel, timestamp, name }
T->>R: POST { accessToken, channel, timestamp, name }
R->>R: checkInternalAuth()
R->>R: ZodSchema.parse(body)
R->>S: POST reactions.remove { channel, timestamp, name }
S-->>R: { ok: true } or { ok: false, error: "..." }
alt Success
R-->>T: { success: true, output: { content, metadata } }
T-->>B: { content: "Successfully removed :name: reaction", metadata }
else Failure
R-->>T: { success: false, error: "..." }
T-->>B: { success: false, output: { content: error, metadata: empty } }
end
B-->>U: result
Last reviewed commit: f5da9bc |
Summary
slack_remove_reactiontool using Slack'sreactions.removeAPIunreactoperation to Slack block, sharing subBlocks with add reaction/api/tools/slack/remove-reactionType of Change
Testing
Tested manually
Checklist