From 03b5e4ca40de1337392588a19b1c360865694117 Mon Sep 17 00:00:00 2001
From: Ahmad Ragab <ahmad.ragab@joinhandshake.com>
Date: Mon, 30 Mar 2026 16:51:52 -0400
Subject: [PATCH] fix(ui): improve diff preview for added and deleted files

Added files no longer trigger broken GapView/TrailingGap context fetches.
Deleted files show a "This file was deleted" banner instead of raw hunks.
Status badges for Added/Deleted now render with tinted backgrounds.
Footer line totals now include uncommitted/untracked files.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 .prettierignore                      |  1 +
 src/components/ChangedFilesList.tsx  |  8 ++------
 src/components/ScrollingDiffView.tsx | 29 ++++++++++++++++++++++------
 3 files changed, 26 insertions(+), 12 deletions(-)

diff --git a/.prettierignore b/.prettierignore
index dd601b5e..97d7bbb8 100644
--- a/.prettierignore
+++ b/.prettierignore
@@ -5,6 +5,7 @@ release/
 node_modules/
 .worktrees/
 .claude/
+.omc/
 .planning/
 .letta/
 package-lock.json
diff --git a/src/components/ChangedFilesList.tsx b/src/components/ChangedFilesList.tsx
index e26de1fe..31b8bb59 100644
--- a/src/components/ChangedFilesList.tsx
+++ b/src/components/ChangedFilesList.tsx
@@ -115,12 +115,8 @@ export function ChangedFilesList(props: ChangedFilesListProps) {
     });
   });
 
-  const totalAdded = createMemo(() =>
-    files().reduce((s, f) => s + (f.committed ? f.lines_added : 0), 0),
-  );
-  const totalRemoved = createMemo(() =>
-    files().reduce((s, f) => s + (f.committed ? f.lines_removed : 0), 0),
-  );
+  const totalAdded = createMemo(() => files().reduce((s, f) => s + f.lines_added, 0));
+  const totalRemoved = createMemo(() => files().reduce((s, f) => s + f.lines_removed, 0));
   const uncommittedCount = createMemo(() => files().filter((f) => !f.committed).length);
 
   /** For each file, compute the display filename and an optional disambiguating directory. */
diff --git a/src/components/ScrollingDiffView.tsx b/src/components/ScrollingDiffView.tsx
index fc8081f0..16de3eb8 100644
--- a/src/components/ScrollingDiffView.tsx
+++ b/src/components/ScrollingDiffView.tsx
@@ -1,6 +1,6 @@
 import { For, Show, createSignal, createEffect, onMount, onCleanup } from 'solid-js';
 import type { JSX } from 'solid-js';
-import { theme } from '../lib/theme';
+import { theme, bannerStyle } from '../lib/theme';
 import { sf } from '../lib/fontScale';
 import { getStatusColor } from '../lib/status-colors';
 import { openFileInEditor } from '../lib/shell';
@@ -591,7 +591,10 @@ function FileSection(props: {
             padding: '2px 8px',
             'border-radius': '4px',
             color: getStatusColor(props.file.status),
-            background: 'rgba(255,255,255,0.06)',
+            background:
+              props.file.status === 'M'
+                ? 'rgba(255,255,255,0.06)'
+                : `color-mix(in srgb, ${getStatusColor(props.file.status)} 15%, transparent)`,
           }}
         >
           {STATUS_LABELS[props.file.status] ?? props.file.status}
@@ -672,9 +675,23 @@ function FileSection(props: {
           </div>
         </Show>
 
-        <Show when={!props.file.binary}>
+        <Show when={!props.file.binary && props.file.status === 'D'}>
+          <div
+            style={{
+              ...bannerStyle(theme.error),
+              margin: '12px',
+              'font-size': sf(12),
+              'text-align': 'center',
+              'font-weight': '600',
+            }}
+          >
+            This file was deleted
+          </div>
+        </Show>
+
+        <Show when={!props.file.binary && props.file.status !== 'D'}>
           <div style={{ 'padding-bottom': '8px', background: 'rgba(0, 0, 0, 0.15)' }}>
-            <Show when={props.file.hunks.length > 0 && props.file.status !== 'D'}>
+            <Show when={props.file.hunks.length > 0 && props.file.status === 'M'}>
               <GapView
                 startLine={1}
                 endLine={props.file.hunks[0].newStart}
@@ -691,7 +708,7 @@ function FileSection(props: {
             <For each={props.file.hunks}>
               {(hunk, hunkIdx) => (
                 <>
-                  <Show when={hunkIdx() > 0}>
+                  <Show when={hunkIdx() > 0 && props.file.status === 'M'}>
                     <GapView
                       startLine={
                         props.file.hunks[hunkIdx() - 1].newStart +
@@ -777,7 +794,7 @@ function FileSection(props: {
                 </>
               )}
             </For>
-            <Show when={props.file.hunks.length > 0 && props.file.status !== 'D'}>
+            <Show when={props.file.hunks.length > 0 && props.file.status === 'M'}>
               <TrailingGap
                 lastHunk={props.file.hunks[props.file.hunks.length - 1]}
                 lang={lang()}