From a8c8af21e5ba708002de1e12cb6b828038f60e08 Mon Sep 17 00:00:00 2001
From: Jason Woltje <jason.woltje@uscllc.com>
Date: Tue, 3 Feb 2026 14:24:24 -0600
Subject: [PATCH] fix(#92): use PDA-friendly language (Target instead of Due)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Critical PDA-friendly design compliance fix.

Changed forbidden "Due:" to approved "Target:" throughout FederatedTaskCard
component and tests, per DESIGN-PRINCIPLES.md requirements.

Changes:
- FederatedTaskCard.tsx: Changed "Due: {dueDate}" to "Target: {dueDate}"
- FederatedTaskCard.test.tsx: Updated all test expectations from "Due:" to "Target:"
- Updated test names to reflect "target date" terminology

All 11 tests passing.

This ensures full compliance with PDA-friendly language guidelines:
| ❌ NEVER | ✅ ALWAYS   |
| DUE      | Target date |

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
---
 .../federation/FederatedTaskCard.test.tsx          | 14 +++++++-------
 .../components/federation/FederatedTaskCard.tsx    |  4 ++--
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/apps/web/src/components/federation/FederatedTaskCard.test.tsx b/apps/web/src/components/federation/FederatedTaskCard.test.tsx
index a4246ab..3f87998 100644
--- a/apps/web/src/components/federation/FederatedTaskCard.test.tsx
+++ b/apps/web/src/components/federation/FederatedTaskCard.test.tsx
@@ -66,11 +66,11 @@ describe("FederatedTaskCard", () => {
     expect(screen.getByText("High")).toBeInTheDocument();
   });
 
-  it("should render due date", () => {
+  it("should render target date", () => {
     render(<FederatedTaskCard federatedTask={mockTask} />);
 
-    // Check for "Due:" text followed by a date
-    expect(screen.getByText(/Due:/)).toBeInTheDocument();
+    // Check for "Target:" text followed by a date
+    expect(screen.getByText(/Target:/)).toBeInTheDocument();
     expect(screen.getByText(/2026/)).toBeInTheDocument();
   });
 
@@ -113,8 +113,8 @@ describe("FederatedTaskCard", () => {
     ).not.toBeInTheDocument();
   });
 
-  it("should handle task without due date", () => {
-    const taskNoDue: FederatedTask = {
+  it("should handle task without target date", () => {
+    const taskNoTarget: FederatedTask = {
       ...mockTask,
       task: {
         ...mockTask.task,
@@ -122,10 +122,10 @@ describe("FederatedTaskCard", () => {
       },
     };
 
-    render(<FederatedTaskCard federatedTask={taskNoDue} />);
+    render(<FederatedTaskCard federatedTask={taskNoTarget} />);
 
     expect(screen.getByText("Review pull request")).toBeInTheDocument();
-    expect(screen.queryByText(/Due:/)).not.toBeInTheDocument();
+    expect(screen.queryByText(/Target:/)).not.toBeInTheDocument();
   });
 
   it("should use PDA-friendly language for status", () => {
diff --git a/apps/web/src/components/federation/FederatedTaskCard.tsx b/apps/web/src/components/federation/FederatedTaskCard.tsx
index 3511a0f..d60619d 100644
--- a/apps/web/src/components/federation/FederatedTaskCard.tsx
+++ b/apps/web/src/components/federation/FederatedTaskCard.tsx
@@ -105,8 +105,8 @@ export function FederatedTaskCard({
         {/* Priority */}
         <span className={`text-xs font-medium ${priority.colorClass}`}>{priority.text}</span>
 
-        {/* Due date */}
-        {dueDate && <span className="text-xs text-gray-600">Due: {dueDate}</span>}
+        {/* Target date */}
+        {dueDate && <span className="text-xs text-gray-600">Target: {dueDate}</span>}
       </div>
     </div>
   );