fix: cancel stale completion frames

frontend/src/lib/completion-motion.ts

@@ -1,6 +1,7 @@
 type CompletionId = string
 
 type CompletionTimer = number
+type CompletionFrame = number
 
 export function createCompletionPulse(
   activate: (id: CompletionId) => void,
@@ -8,19 +9,23 @@ export function createCompletionPulse(
   duration = 420,
 ) {
   const timers = new Map<CompletionId, CompletionTimer>()
+  const frames = new Map<CompletionId, CompletionFrame>()
 
   return (id: CompletionId) => {
-    const previous = timers.get(id)
+    const previousTimer = timers.get(id)
-    if (previous !== undefined) {
+    if (previousTimer !== undefined) window.clearTimeout(previousTimer)
-      window.clearTimeout(previous)
+    const previousFrame = frames.get(id)
-      deactivate(id)
+    if (previousFrame !== undefined) window.cancelAnimationFrame(previousFrame)
-    }
+    deactivate(id)
-    window.requestAnimationFrame(() => {
+
+    const frame = window.requestAnimationFrame(() => {
+      frames.delete(id)
       activate(id)
       timers.set(id, window.setTimeout(() => {
         deactivate(id)
         timers.delete(id)
       }, duration))
     })
+    frames.set(id, frame)
   }
 }

frontend/src/style.test.ts

@@ -64,6 +64,7 @@ describe('completion feedback motion', () => {
     const active = new Set<string>()
     const pulse = createCompletionPulse((id) => active.add(id), (id) => active.delete(id), 420)
     pulse('item-1')
+    pulse('item-1')
     vi.advanceTimersByTime(16)
     expect(active.has('item-1')).toBe(true)
     vi.advanceTimersByTime(200)