Skip to content

Disable issue commenting to focus on pull request review only#55

Merged
slavingia merged 1 commit intomainfrom
devin/1723779245-disable-issue-commenting
Aug 16, 2025
Merged

Disable issue commenting to focus on pull request review only#55
slavingia merged 1 commit intomainfrom
devin/1723779245-disable-issue-commenting

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

Disable issue commenting to focus on pull request review only

Summary

This PR disables Jacquez from commenting on issues and issue comments while preserving pull request review functionality. The change involves commenting out two webhook event listeners in app/api/webhook/route.ts:

  • app.webhooks.on("issues.opened", handleIssueOpened)
  • app.webhooks.on("issue_comment.created", handleIssueCommentCreated)

The pull_request.opened handler remains active, so Jacquez will continue to moderate pull requests according to contributing guidelines.

Review & Testing Checklist for Human

  • Verify requirement alignment: Confirm this change matches the intended behavior (stop issue commenting, keep PR review)
  • Test PR webhook functionality: Create a test PR to ensure pull request moderation still works correctly
  • Consider code cleanup: Evaluate whether unused handleIssueOpened and handleIssueCommentCreated functions should be removed
  • Check for side effects: Review if any other parts of the codebase depend on issue webhook handling

Recommended test plan: Deploy to a test environment and create both an issue and a pull request to verify that only the PR receives moderation comments.


Diagram

%%{ init : { "theme" : "default" }}%%
graph TB
    Webhook["GitHub Webhook<br/>POST /api/webhook"]
    Router["webhook/route.ts"]:::major-edit
    PRHandler["handlePullRequestOpened()"]:::context
    IssueHandler["handleIssueOpened()"]:::context
    CommentHandler["handleIssueCommentCreated()"]:::context
    
    Webhook --> Router
    Router -.-> PRHandler
    Router -.->|"DISABLED"| IssueHandler
    Router -.->|"DISABLED"| CommentHandler
    
    PRHandler --> AI["Anthropic AI<br/>Analysis"]:::context
    IssueHandler -.->|"NOT CALLED"| AI
    CommentHandler -.->|"NOT CALLED"| AI
    
    AI --> Comment["Post GitHub<br/>Comment"]:::context
    
    subgraph Legend
        L1["Major Edit"]:::major-edit
        L2["Minor Edit"]:::minor-edit  
        L3["Context/No Edit"]:::context
    end
    
    classDef major-edit fill:#90EE90
    classDef minor-edit fill:#87CEEB
    classDef context fill:#F5F5F5
Loading

Notes

  • The disabled functions are still defined in the codebase but no longer receive webhook events
  • All existing tests pass, confirming no breaking changes to current functionality
  • This is a simple but impactful change that fundamentally alters Jacquez's behavior scope

Link to Devin run: https://app.devin.ai/sessions/4a7961e41b0e4bb2a02b98b085cedf4c
Requested by: Sahil Lavingia (@slavingia)

- Comment out webhook handlers for issues.opened and issue_comment.created
- Keep pull_request.opened handler active for PR review functionality
- This change makes Jacquez focus exclusively on pull request moderation

Co-Authored-By: Sahil Lavingia <sahil@gumroad.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Aug 16, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
jacquez Ready Ready Preview Comment Aug 16, 2025 3:54am

@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@slavingia slavingia merged commit a9c68d6 into main Aug 16, 2025
4 checks passed
@slavingia slavingia deleted the devin/1723779245-disable-issue-commenting branch August 16, 2025 03:57
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