feat(dora): exclude bot/automation accounts from PR Pickup Time calcu… - #9000
Merged
Conversation
Contributor
Author
|
Hi @klesh could you please review and merge pull request. |
Contributor
|
Hi. Please fix the errors first. |
bramhanandlingala
force-pushed
the
fix/#8974
branch
from
July 28, 2026 05:33
4351a5c to
b3e1fb3
Compare
Contributor
Author
|
Hi @klesh, I've addressed the previous issues and pushed the requested fixes. The PR is now waiting for workflow approval before the QC checks can run. When you have a chance, could you please approve the workflows and take another look? Thank you! |
Contributor
|
Hi. Thanks for the quick response. I reran the test and it was still failing. Please take another look. |
Contributor
Author
|
Hi @klesh The PR is now waiting for workflow approval before the QC checks can run. When you have a chance, could you please approve the workflows and take another look? Thank you! |
klesh
approved these changes
Aug 14, 2026
klesh
left a comment
Contributor
There was a problem hiding this comment.
LGTM
Thanks for your contribution.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #8974.
AI-powered code review bots (such as GitHub Copilot, Dependabot, and GitHub Actions) often post comments immediately after a pull request is created. DevLake currently considers the first PR comment as the start of the review process, regardless of whether the comment is from a human or an automated account. As a result, the
PR Pickup Timemetric can become artificially close to zero and no longer represent the actual time taken for a human reviewer to respond.This PR introduces bot account detection and excludes bot-generated comments when calculating the first review event.
Changes
IsBotfield to the cross-domainAccountmodel.is_botcolumn in theaccountstable with a default value offalse.IsBotusing:type == "Bot"when available.[bot]login suffix as a fallback.batchFetchFirstReviewslogic to ignore comments from bot accounts when determining the first review, ensuringPR Pickup TimeandPR Review Timereflect the first human review.Scope
This change currently supports GitHub accounts only.
For other integrations (GitLab, Bitbucket, Azure DevOps, etc.), the
is_botfield defaults tofalse, resulting in no behavioral changes until bot detection is implemented for those providers.Fixes #8974