github-starred/actual
2
0
Fork 0
mirror of https://github.com/actualbudget/actual.git synced 2026-03-11 20:44:32 -05:00
Code Issues 250 Packages Projects Releases 53 Wiki Activity

🐛 Paginate the count points list files call (#5448)

Browse Source 
* paginate the count points list files call

* fix tier assignment

* paginate the count points list files call

* fix tier assignment

* ensure utc dates and fix issue filtering to include issues updated in other months
This commit is contained in:
Michael Clark
2025-08-02 17:33:18 +00:00
committed by GitHub
parent 895d69f875
commit 239a087542
1 changed files with 46 additions and 40 deletions
Download Patch File Download Diff File Expand all files Collapse all files

86
.github/scripts/count-points.mjs vendored
View File

@@ -46,24 +46,25 @@ const REPOSITORY_CONFIG = new Map([
function getLastMonthDates() {
// Get data relating to the last month
const now = new Date();
// Always use UTC for calculations
const firstDayOfLastMonth = new Date(
now.getFullYear(),
now.getMonth() - 1,
1,
Date.UTC(now.getUTCFullYear(), now.getUTCMonth() - 1, 1, 0, 0, 0, 0),
);
const since = process.env.START_DATE
? new Date(process.env.START_DATE)
? new Date(Date.parse(process.env.START_DATE))
: firstDayOfLastMonth;
// Calculate the end of the month for the since date
// Calculate the end of the month for the since date in UTC
const until = new Date(
since.getFullYear(),
since.getMonth() + 1,
0,
23,
59,
59,
999,
Date.UTC(
since.getUTCFullYear(),
since.getUTCMonth() + 1,
0,
23,
59,
59,
999,
),
);
return { since, until };
@@ -76,7 +77,9 @@ function getLastMonthDates() {
* @returns {number} The total points earned for the repository
*/
async function countContributorPoints(repo) {
const octokit = new Octokit({ auth: process.env.GITHUB_TOKEN });
const octokit = new Octokit({
auth: process.env.GITHUB_TOKEN,
});
const owner = 'actualbudget';
const config = REPOSITORY_CONFIG.get(repo);
@@ -140,12 +143,17 @@ async function countContributorPoints(repo) {
pull_number: pr.number,
});
// Get list of modified files
const { data: modifiedFiles } = await octokit.pulls.listFiles({
owner,
repo,
pull_number: pr.number,
});
// Get list of modified files with pagination
const modifiedFiles = await octokit.paginate(
octokit.pulls.listFiles,
{
owner,
repo,
pull_number: pr.number,
per_page: 100, // Maximum allowed by GitHub API
},
response => response.data,
);
// Calculate points based on PR size, excluding specified files
const totalChanges = modifiedFiles
@@ -162,7 +170,7 @@ async function countContributorPoints(repo) {
// Calculate points for reviewers based on PR size
const prPoints =
config.PR_REVIEW_POINT_TIERS.find(tier => totalChanges > tier.minChanges)
config.PR_REVIEW_POINT_TIERS.find(tier => totalChanges >= tier.minChanges)
?.points ?? 0;
// Award points to the PR creator if it's a release PR
@@ -197,20 +205,15 @@ async function countContributorPoints(repo) {
}
// Get all issues with label events in the last month
const issues = await octokit.paginate(
octokit.issues.listForRepo,
{
owner,
repo,
state: 'all',
sort: 'updated',
direction: 'desc',
per_page: 100,
since: since.toISOString(),
},
(response, done) =>
response.data.filter(issue => new Date(issue.updated_at) <= until),
);
const issues = await octokit.paginate(octokit.issues.listForRepo, {
owner,
repo,
state: 'all',
sort: 'updated',
direction: 'desc',
per_page: 100,
since: since.toISOString(),
});
// Get label events for each issue
for (const issue of issues) {
@@ -222,12 +225,15 @@ async function countContributorPoints(repo) {
// Process events
events
.filter(
event =>
new Date(event.created_at) > since &&
new Date(event.created_at) <= until &&
stats.has(event.actor?.login),
)
.filter(event => {
// Always compare UTC times
const createdAt = new Date(event.created_at);
return (
createdAt.getTime() > since.getTime() &&
createdAt.getTime() <= until.getTime() &&
stats.has(event.actor?.login)
);
})
.forEach(event => {
if (
event.event === 'unlabeled' &&