Skip to content

fix(webhook): remove dismissed reviews from scoring#215

Open
Kostyuk1998 wants to merge 1 commit into
entrius:testfrom
Kostyuk1998:fix/issue-204-dismissed-reviews
Open

fix(webhook): remove dismissed reviews from scoring#215
Kostyuk1998 wants to merge 1 commit into
entrius:testfrom
Kostyuk1998:fix/issue-204-dismissed-reviews

Conversation

@Kostyuk1998

@Kostyuk1998 Kostyuk1998 commented Jul 14, 2026

Copy link
Copy Markdown

Summary

  • handle pull_request_review.dismissed webhooks by deleting the matching stored review
  • refresh the repository event timestamp after dismissal, matching other delete handlers
  • add regression coverage for dismissed, submitted, and ignored review actions

Related Issues

Fixes #204

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other (describe below)

Testing

  • npm test -- --runInBand
  • npm test -- --runInBand --coverage --collectCoverageFrom='src/webhook/handlers/review.handler.ts' src/webhook/handlers/review.handler.spec.ts (100% statements, branches, functions, and lines)
  • npm run lint
  • npm run format:check
  • npm run build

Checklist

  • I have read the Contributing Guide
  • Code builds without errors
  • New and existing tests pass (if applicable)
  • Documentation updated (not applicable)
  • No unnecessary dependencies added

@xiao-xiao-mao xiao-xiao-mao Bot added the bug Something isn't working label Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] Dismissed PR reviews remain in review_summary scoring counts

1 participant