From b2427fd07b3aba46e4b24438190c5f1582305833 Mon Sep 17 00:00:00 2001
From: janpeter visser <janpetervisser2@gmail.com>
Date: Sat, 2 May 2026 18:12:35 +0200
Subject: [PATCH] fix(insights): skip sprints with null completed_at in
 velocity

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 lib/insights/velocity.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/insights/velocity.ts b/lib/insights/velocity.ts
index 528b321..7cf123e 100644
--- a/lib/insights/velocity.ts
+++ b/lib/insights/velocity.ts
@@ -33,7 +33,7 @@ export async function getVelocity(userId: string, sprintsBack = 5): Promise<Velo
   })
 
   // Reverse to chronological order (oldest first, for x-axis)
-  const chronological = [...sprints].reverse()
+  const chronological = [...sprints].filter(s => s.completed_at != null).reverse()
 
   const result: VelocitySprint[] = chronological.map(sprint => ({
     sprintId: sprint.id,
@@ -41,7 +41,7 @@ export async function getVelocity(userId: string, sprintsBack = 5): Promise<Velo
     productId: sprint.product.id,
     productName: sprint.product.name,
     doneCount: sprint.tasks.filter(t => t.status === 'DONE').length,
-    completedAt: sprint.completed_at!.toISOString(),
+    completedAt: sprint.completed_at.toISOString(),
   }))
 
   const seen = new Set<string>()