From a4253a80f580227a1cbc97773869f5feabf8aa0f Mon Sep 17 00:00:00 2001
From: weeihan <whyap79@gmail.com>
Date: Thu, 30 Jul 2026 08:41:41 +0800
Subject: [PATCH] feat(capa): allow overdue CAPAs to be acted on, fix owner
 page stats
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- CapaOwnerActions: add 'overdue' to open/reopened branch so DB-overdue
  CAPAs show 'Mark In Progress' button (cron transitions open→overdue)
- capa-owner page: compute overdue at runtime (dueDate < today) so stats
  and red highlight are correct even before cron runs
- openCount now excludes overdue items to avoid double-counting
- Add CapaOwnerActions test suite covering all status transitions

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HEYxFQiCyxJvnBCoZeYzB9
---
 app/(protected)/capa-owner/page.tsx           | 12 ++++--
 components/capa/capa-owner-actions.tsx        |  2 +-
 .../capa/capa-owner-actions.test.tsx          | 40 +++++++++++++++++++
 3 files changed, 50 insertions(+), 4 deletions(-)
 create mode 100644 tests/components/capa/capa-owner-actions.test.tsx

diff --git a/app/(protected)/capa-owner/page.tsx b/app/(protected)/capa-owner/page.tsx
index aa113b9..86ae460 100644
--- a/app/(protected)/capa-owner/page.tsx
+++ b/app/(protected)/capa-owner/page.tsx
@@ -52,8 +52,14 @@ export default async function CapaOwnerPage() {
     .orderBy(asc(capaActions.dueDate))
   )
 
-  const openCount = rows.filter(c => ['open', 'in_progress', 'reopened'].includes(c.status)).length
-  const overdueCount = rows.filter(c => c.status === 'overdue').length
+  const today = new Date().toISOString().slice(0, 10)
+  function isCapaOverdue(c: { status: string; dueDate: string | null }): boolean {
+    if (c.status === 'overdue') return true
+    return (c.status === 'open' || c.status === 'in_progress') && !!c.dueDate && c.dueDate < today
+  }
+
+  const openCount = rows.filter(c => ['open', 'in_progress', 'reopened'].includes(c.status) && !isCapaOverdue(c)).length
+  const overdueCount = rows.filter(c => isCapaOverdue(c)).length
   const doneCount = rows.filter(c => ['verified', 'closed'].includes(c.status)).length
 
   const PRIORITY_COLORS: Record<string, string> = {
@@ -82,7 +88,7 @@ export default async function CapaOwnerPage() {
       ) : (
         <div className="bg-white rounded-xl shadow-sm divide-y divide-gray-50">
           {activeRows.map(capa => {
-            const isOverdue = capa.status === 'overdue'
+            const isOverdue = isCapaOverdue(capa)
             return (
               <div key={capa.id} className={`p-4 ${isOverdue ? 'bg-red-50' : ''}`}>
                 <div className="flex items-start justify-between gap-4">
diff --git a/components/capa/capa-owner-actions.tsx b/components/capa/capa-owner-actions.tsx
index d63eb05..0f2f334 100644
--- a/components/capa/capa-owner-actions.tsx
+++ b/components/capa/capa-owner-actions.tsx
@@ -31,7 +31,7 @@ export function CapaOwnerActions({ capaId, currentStatus }: Props) {
     }
   }
 
-  if (currentStatus === 'open' || currentStatus === 'reopened') {
+  if (currentStatus === 'open' || currentStatus === 'reopened' || currentStatus === 'overdue') {
     return (
       <button
         onClick={() => updateStatus('in_progress')}
diff --git a/tests/components/capa/capa-owner-actions.test.tsx b/tests/components/capa/capa-owner-actions.test.tsx
new file mode 100644
index 0000000..ec23b52
--- /dev/null
+++ b/tests/components/capa/capa-owner-actions.test.tsx
@@ -0,0 +1,40 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { render, screen } from '@testing-library/react'
+import { CapaOwnerActions } from '@/components/capa/capa-owner-actions'
+
+// Mock Next.js router
+vi.mock('next/navigation', () => ({
+  useRouter: () => ({ refresh: vi.fn() }),
+}))
+
+describe('CapaOwnerActions', () => {
+  it('shows Mark In Progress for open status', () => {
+    render(<CapaOwnerActions capaId="abc" currentStatus="open" />)
+    expect(screen.getByText('Mark In Progress')).toBeTruthy()
+  })
+
+  it('shows Mark In Progress for reopened status', () => {
+    render(<CapaOwnerActions capaId="abc" currentStatus="reopened" />)
+    expect(screen.getByText('Mark In Progress')).toBeTruthy()
+  })
+
+  it('shows Mark In Progress for overdue status', () => {
+    render(<CapaOwnerActions capaId="abc" currentStatus="overdue" />)
+    expect(screen.getByText('Mark In Progress')).toBeTruthy()
+  })
+
+  it('shows Submit for Verification for in_progress status', () => {
+    render(<CapaOwnerActions capaId="abc" currentStatus="in_progress" />)
+    expect(screen.getByText('Submit for Verification')).toBeTruthy()
+  })
+
+  it('renders nothing for pending_verification status', () => {
+    const { container } = render(<CapaOwnerActions capaId="abc" currentStatus="pending_verification" />)
+    expect(container.firstChild).toBeNull()
+  })
+
+  it('renders nothing for verified status', () => {
+    const { container } = render(<CapaOwnerActions capaId="abc" currentStatus="verified" />)
+    expect(container.firstChild).toBeNull()
+  })
+})