Skip to content

chore: enhance account summary table with links to issues and PRs#15177

Merged
mnkiefer merged 1 commit intomainfrom
table-with-link
Feb 12, 2026
Merged

chore: enhance account summary table with links to issues and PRs#15177
mnkiefer merged 1 commit intomainfrom
table-with-link

Conversation

@mnkiefer
Copy link
Contributor

  • Improves account summary table and ensures review submissions are properly included in user activity tracking.

@mnkiefer mnkiefer self-assigned this Feb 12, 2026
Copilot AI review requested due to automatic review settings February 12, 2026 14:47
@mnkiefer mnkiefer merged commit 030fbf7 into main Feb 12, 2026
35 checks passed
@mnkiefer mnkiefer deleted the table-with-link branch February 12, 2026 14:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Enhances the bot-detection workflow’s account summary output by making counts clickable (linking to relevant GitHub searches) and by including PR review submissions in the “user activity” data used to identify high-risk new accounts.

Changes:

  • Add linked counts in the Step Summary account table (issues/PRs/comments).
  • Include PR review submissions via pulls.listReviews so non-comment reviews are tracked as activity.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +51 to +56
const openIssuesLink = linkCount(openIssues, `${baseRepoURL}/issues?q=is%3Aissue+is%3Aopen+author%3A${encodedLogin}`);
const closedIssuesLink = linkCount(closedIssues, `${baseRepoURL}/issues?q=is%3Aissue+is%3Aclosed+author%3A${encodedLogin}`);
const openPRsLink = linkCount(openPRs, `${baseRepoURL}/pulls?q=is%3Apr+is%3Aopen+author%3A${encodedLogin}`);
const closedPRsLink = linkCount(closedPRs, `${baseRepoURL}/pulls?q=is%3Apr+is%3Aclosed+author%3A${encodedLogin}`);
const commentsLink = linkCount(comments, `${baseRepoURL}/issues?q=commenter%3A${encodedLogin}`);

Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The table counts issues/PRs/comments from data.*, which are filtered to created_at >= cutoff elsewhere in this script, but the new links here don’t include any time filter. Clicking the linked counts can therefore show more results than the count displayed. Consider passing the cutoff (or at least the cutoff date) into buildAccountSummaryTable and adding a created:>=... qualifier (or otherwise making the linked query match the same window as the counts).

Suggested change
const openIssuesLink = linkCount(openIssues, `${baseRepoURL}/issues?q=is%3Aissue+is%3Aopen+author%3A${encodedLogin}`);
const closedIssuesLink = linkCount(closedIssues, `${baseRepoURL}/issues?q=is%3Aissue+is%3Aclosed+author%3A${encodedLogin}`);
const openPRsLink = linkCount(openPRs, `${baseRepoURL}/pulls?q=is%3Apr+is%3Aopen+author%3A${encodedLogin}`);
const closedPRsLink = linkCount(closedPRs, `${baseRepoURL}/pulls?q=is%3Apr+is%3Aclosed+author%3A${encodedLogin}`);
const commentsLink = linkCount(comments, `${baseRepoURL}/issues?q=commenter%3A${encodedLogin}`);
const cutoffIso = cutoff.toISOString();
const cutoffQuery = `+created%3A%3E%3D${encodeURIComponent(cutoffIso)}`;
const openIssuesLink = linkCount(
openIssues,
`${baseRepoURL}/issues?q=is%3Aissue+is%3Aopen+author%3A${encodedLogin}${cutoffQuery}`,
);
const closedIssuesLink = linkCount(
closedIssues,
`${baseRepoURL}/issues?q=is%3Aissue+is%3Aclosed+author%3A${encodedLogin}${cutoffQuery}`,
);
const openPRsLink = linkCount(
openPRs,
`${baseRepoURL}/pulls?q=is%3Apr+is%3Aopen+author%3A${encodedLogin}${cutoffQuery}`,
);
const closedPRsLink = linkCount(
closedPRs,
`${baseRepoURL}/pulls?q=is%3Apr+is%3Aclosed+author%3A${encodedLogin}${cutoffQuery}`,
);
const commentsLink = linkCount(
comments,
`${baseRepoURL}/issues?q=commenter%3A${encodedLogin}${cutoffQuery}`,
);

Copilot uses AI. Check for mistakes.
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.

2 participants