bug(jat-q8r49): surface accept/reject events in activity timeline

- history endpoint now includes status_change comments (e.g. from feedback
  widget respond) in the timeline as task_accepted / task_rejected events
- TaskDetailDrawer renders them on the left side with green/red check/x icons
- Tasks tab filter includes status_change events alongside jat_events
- count.status_changes added to the history response

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: joewinke

ide/src/lib/components/TaskDetailDrawer.svelte

@@ -191,7 +191,7 @@
 
 	// Task history state
 	interface TimelineEvent {
-		type: 'jat_event' | 'agent_mail';
+		type: 'jat_event' | 'agent_mail' | 'status_change';
 		event: string;
 		timestamp: string;
 		description: string;
@@ -205,6 +205,7 @@
 		count: {
 			total: number;
 			jat_events: number;
+			status_changes: number;
 			agent_mail: number;
 		};
 		timestamp: string;
@@ -863,7 +864,7 @@
 	const filteredTimeline = $derived((): TimelineEvent[] => {
 		if (!taskHistory?.timeline) return [];
 		if (timelineFilter === 'all') return taskHistory.timeline;
-		if (timelineFilter === 'tasks') return taskHistory.timeline.filter(e => e.type === 'jat_event');
+		if (timelineFilter === 'tasks') return taskHistory.timeline.filter(e => e.type === 'jat_event' || e.type === 'status_change');
 		return taskHistory.timeline.filter(e => e.type === 'agent_mail');
 	});
 
@@ -4969,7 +4970,7 @@
 										class="tab {timelineFilter === 'tasks' ? 'tab-active' : ''}"
 										onclick={() => timelineFilter = 'tasks'}
 									>
-										Tasks ({taskHistory?.count?.jat_events || 0})
+										Tasks ({(taskHistory?.count?.jat_events || 0) + (taskHistory?.count?.status_changes || 0)})
 									</button>
 									<button
 										class="tab {timelineFilter === 'messages' ? 'tab-active' : ''}"
@@ -5011,10 +5012,34 @@
 										<li style="grid-template-columns: 30% min-content 1fr;">
 											<!-- Top connector (skip for first item) -->
 											{#if i > 0}
-												<hr class="{event.type === 'jat_event' ? 'bg-info' : 'bg-warning'}" />
+												<hr class="{event.type === 'jat_event' || event.type === 'status_change' ? (event.event === 'task_accepted' ? 'bg-success' : event.type === 'status_change' ? 'bg-error' : 'bg-info') : 'bg-warning'}" />
 											{/if}
 
-											{#if event.type === 'jat_event'}
+											{#if event.type === 'status_change'}
+												<!-- Accept/reject decision on LEFT side -->
+												{@const isAccepted = event.event === 'task_accepted'}
+												<div class="timeline-start timeline-box text-xs mb-4 {isAccepted ? 'border-success/30 bg-success/5' : 'border-error/30 bg-error/5'}">
+													<div class="font-semibold text-base-content">
+														{event.description}
+													</div>
+													<div class="text-base-content/60 mt-1">
+														{formatDate(event.timestamp)}
+													</div>
+													{#if event.metadata.author}
+														<div class="text-base-content/50 mt-1 text-xs">by {event.metadata.author}</div>
+													{/if}
+												</div>
+												<div class="timeline-middle">
+													<svg xmlns="http://www.w3.org/2000/svg" class="h-4 w-4 {isAccepted ? 'text-success' : 'text-error'}" fill="none" viewBox="0 0 24 24" stroke="currentColor">
+														{#if isAccepted}
+															<path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M9 12l2 2 4-4m6 2a9 9 0 11-18 0 9 9 0 0118 0z" />
+														{:else}
+															<path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M10 14l2-2m0 0l2-2m-2 2l-2-2m2 2l2 2m7-2a9 9 0 11-18 0 9 9 0 0118 0z" />
+														{/if}
+													</svg>
+												</div>
+												<div class="timeline-end"></div>
+											{:else if event.type === 'jat_event'}
 												<!-- Task events on LEFT side (timeline-start) -->
 												<div class="timeline-start timeline-box text-xs mb-4 border-info/30 bg-info/5">
 													<!-- Event description -->
@@ -5134,7 +5159,8 @@
 
 											<!-- Bottom connector (skip for last item) -->
 											{#if i < filteredTimeline().length - 1}
-												<hr class="{filteredTimeline()[i + 1]?.type === 'jat_event' ? 'bg-info' : 'bg-warning'}" />
+												{@const nextEvent = filteredTimeline()[i + 1]}
+												<hr class="{nextEvent?.type === 'status_change' ? (nextEvent.event === 'task_accepted' ? 'bg-success' : 'bg-error') : nextEvent?.type === 'jat_event' ? 'bg-info' : 'bg-warning'}" />
 											{/if}
 										</li>
 									{/each}

ide/src/routes/api/tasks/[id]/history/+server.js

@@ -6,6 +6,7 @@
  * Combines:
  * - JAT task state changes (created, status updates, assignee changes, etc.)
  * - Agent Mail coordination messages (filtered by thread_id = task.id)
+ * - status_change comments (accept/reject decisions from feedback widget)
  *
  * Returns chronological timeline with visual distinction between event types.
  */
@@ -117,7 +118,28 @@ export async function GET({ params }) {
 			});
 		}
 
-		// 2. Add Agent Mail messages
+		// 2. Add status_change comments (e.g. accept/reject from feedback widget)
+		const comments = /** @type {Array<any>} */ (task.comments ?? []);
+		comments
+			.filter(c => c.comment_type === 'status_change')
+			.forEach(c => {
+				const meta = typeof c.metadata === 'string' ? safeParse(c.metadata) : (c.metadata ?? {});
+				const isAccepted = meta?.response === 'accepted';
+				timeline.push({
+					type: 'status_change',
+					event: isAccepted ? 'task_accepted' : 'task_rejected',
+					timestamp: c.created_at,
+					description: c.text || (isAccepted ? 'Accepted' : 'Rejected'),
+					metadata: {
+						author: c.author,
+						response: meta?.response,
+						reason: meta?.reason,
+						source: meta?.source
+					}
+				});
+			});
+
+		// 3. Add Agent Mail messages
 		mailMessages.forEach(msg => {
 			timeline.push({
 				type: 'agent_mail',
@@ -135,7 +157,7 @@ export async function GET({ params }) {
 			});
 		});
 
-		// 3. Sort timeline by timestamp (newest first)
+		// 4. Sort timeline by timestamp (newest first)
 		timeline.sort((a, b) => {
 			const timeA = new Date(a.timestamp).getTime();
 			const timeB = new Date(b.timestamp).getTime();
@@ -149,6 +171,7 @@ export async function GET({ params }) {
 			count: {
 				total: timeline.length,
 				jat_events: timeline.filter(e => e.type === 'jat_event').length,
+				status_changes: timeline.filter(e => e.type === 'status_change').length,
 				agent_mail: timeline.filter(e => e.type === 'agent_mail').length
 			},
 			timestamp: new Date().toISOString()
@@ -166,3 +189,9 @@ export async function GET({ params }) {
 		}, { status: 500 });
 	}
 }
+
+/** @param {unknown} s */
+function safeParse(s) {
+	if (typeof s !== 'string') return s;
+	try { return JSON.parse(s); } catch { return {}; }
+}