feat(reactions): wire up Linear reaction for Comment.create webhook events#1106
feat(reactions): wire up Linear reaction for Comment.create webhook events#1106
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
nhopeatall
left a comment
There was a problem hiding this comment.
Summary
LGTM — Clean implementation that correctly wires up Linear comment reactions following established patterns. The payload parsing matches the Linear webhook format used throughout the codebase, error handling follows the fire-and-forget contract, and tests are comprehensive.
One minor observation (not blocking): sendLinearReaction resolves credentials inline via getIntegrationCredential(projectId, 'pm', 'api_key') rather than reusing the existing resolveLinearCredentials() helper from platformClients/credentials.ts (which does the exact same thing). The Trello reaction sender uses the shared resolveTrelloCredentials() helper, so there's a small pattern inconsistency. The behavior is identical either way.
🕵️ claude-code · claude-opus-4-6 · run details
Summary
sendLinearReactioninsrc/router/reactions.ts— previously a no-op placeholder, now adds a 👀 reaction to Linear comments onComment.createwebhook eventsgetIntegrationCredential(projectId, 'pm', 'api_key')following the same credential resolution pattern as Trello/JIRAlinearClient.createReaction(commentId, '👀')wrapped inwithLinearCredentials— consistent with existing client patternsTest plan
sendLinearReactionsends 👀 reaction forComment.createeventsCard: https://trello.com/c/69deb0cb7779e42640f5647a
🤖 Generated with Claude Code
🕵️ claude-code · claude-sonnet-4-6 · run details