selimaj-dev/claude-hook
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make this for PRs as well #2

New Issue
Closed
opened 2026-09-28 15:06:22 +00:00 by selimaj-dev · 1 comment
selimaj-dev commented 2026-09-28 15:06:22 +00:00
Owner
Copy Link
Copy Source

@claude currently this is only for issues or issue comments, make it support pull requests and pr comments

@claude currently this is only for issues or issue comments, make it support pull requests and pr comments
claude commented 2026-09-28 15:09:17 +00:00
Collaborator
Copy Link
Copy Source

Summary: extended classifyEvent in src/events.js to also handle three new Gitea webhook event types — pull_request (opened/edited, triggers off the PR description like issues does for issue bodies), pull_request_review_comment (inline diff comments), and pull_request_review (overall review body on submit) — all marked isPullRequest: true and flowing through the existing PR-aware job logic in job.js unchanged. Added corresponding tests in test/events.test.js, and updated README.md to document the new webhook triggers to enable in Gitea and removed the now-stale "Known limitations" note about PR review comments not being wired up.

Pull request: http://gitea:3000/selimaj-dev/claude-hook/pulls/3

Summary: extended `classifyEvent` in `src/events.js` to also handle three new Gitea webhook event types — `pull_request` (opened/edited, triggers off the PR description like `issues` does for issue bodies), `pull_request_review_comment` (inline diff comments), and `pull_request_review` (overall review body on submit) — all marked `isPullRequest: true` and flowing through the existing PR-aware job logic in `job.js` unchanged. Added corresponding tests in `test/events.test.js`, and updated `README.md` to document the new webhook triggers to enable in Gitea and removed the now-stale "Known limitations" note about PR review comments not being wired up. Pull request: http://gitea:3000/selimaj-dev/claude-hook/pulls/3
claude referenced this issue from a commit 2026-09-28 15:09:17 +00:00
Claude: address #2
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
claude (Claude) selimaj-dev (Klesti Selimaj)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: selimaj-dev/claude-hook#2