Skip to content

fix: dynamically calculate Avg Review Time instead of hardcoded value #263

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Soumya-codr merged 2 commits into from
Jun 6, 2026
Merged

fix: dynamically calculate Avg Review Time instead of hardcoded value #263

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
bd13e49
fix: dynamically calculate avg review time from merged PRs (#261)
yush-1018 Jun 5, 2026
a902892
refactor: address review feedback - fix metric label, floating point,…
yush-1018 Jun 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 21 additions & 4 deletions src/app/(app)/my-prs/page.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -287,7 +287,17 @@ export default async function MyPRsPage() {
const prsMerged = enrichedPRs.filter((pr) => pr.state === 'merged').length;
const prsTotal = enrichedPRs.length;
const successRate = prsTotal > 0 ? Math.round((prsMerged / prsTotal) * 100) : 0;
const avgReviewDays = 2.3;
const mergedWithDates = enrichedPRs.filter(
(pr) => pr.state === 'merged' && pr.github_created_at && pr.merged_at,
);
const avgMergeDays =
mergedWithDates.length > 0
? mergedWithDates.reduce((sum, pr) => {
const created = new Date(pr.github_created_at).getTime();
const merged = new Date(pr.merged_at!).getTime();
return sum + (merged - created) / (1000 * 60 * 60 * 24);
}, 0) / mergedWithDates.length
: null;

const levelFloor = xpForLevel(level);
const levelCeiling = xpForLevel(level + 1);
Expand Down Expand Up @@ -348,10 +358,17 @@ export default async function MyPRsPage() {

<div className="mb-5 border-t border-[#2d333b] pt-5">
<div className="mb-1 text-[10px] uppercase tracking-widest text-zinc-500">
Avg Review Time
Avg Time to Merge
</div>
<div className="font-sans text-[24px] font-black leading-none text-white">
{avgReviewDays} days
<div className="flex items-baseline gap-1 font-sans text-[24px] font-black leading-none text-white">
{avgMergeDays !== null ? (
<>
<span>{avgMergeDays.toFixed(1)}</span>
<span className="text-[14px] font-medium text-zinc-400">days</span>
</>
) : (
'N/A'
)}
</div>
</div>

Expand Down
Loading