fix(diff): improve sidebar deleted-file presentation

[jschwxrz]

summary:

• this PR updates the right sidebar file list in the diff panel to better distinguish deleted files from modified files.

changes:

• seleted files are displayed with a crossed-out filename
• deleted files no longer show +N/-N pills
• for non-deleted files, diff pills are only shown when the value is non-zero

This improves scanability and avoids noisy/ambiguous counters in the sidebar.

fixes #1444

[vercel]

@jschwxrz is attempting to deploy a commit to the General Action Team on Vercel.

A member of the Team first needs to authorize it.

[greptile-apps]

Greptile Summary

This PR improves the diff sidebar's file list by visually distinguishing deleted files: their filenames now render with line-through styling and their +N/-N diff pills are suppressed. For non-deleted files, pills are also hidden when the count is zero, reducing noise.

Key changes and observations:

• renderPath now accepts an optional status parameter and applies line-through to the filename <span> when status === 'deleted'.
• A new shouldShowDiffPill helper hides pills when the value is exactly 0, but still shows them when the value is null or undefined (rendering +?/-?). This means binary files or other entries with unknown counts will continue to display pills — which may or may not match the intent stated in the PR description ("non-zero only").
• The dir portion of the path (e.g. src/utils/) does not receive line-through when a file is deleted; only the base filename is struck through.
• Both render paths (PR review mode and local changes mode) are updated symmetrically.
• Minor redundancy: both branches of the isDeleted ternary in renderPath include text-foreground, making the difference solely the presence of line-through.

Confidence Score: 4/5

• Safe to merge with minor style/UX considerations worth addressing
• The change is small, focused, and correctly implements its stated goals. Both render paths (PR review mode and local changes mode) are updated symmetrically with no regression risk. The only concerns are: (1) a subtle semantic ambiguity in shouldShowDiffPill around null values vs. the "non-zero" intent, (2) redundant CSS class branches, and (3) the directory part of a deleted path not being struck through — all of which are non-breaking and presentational in nature.
• No files require special attention beyond src/renderer/components/FileChangesPanel.tsx for the minor issues noted.

Important Files Changed

Filename	Overview
src/renderer/components/FileChangesPanel.tsx	Adds strikethrough styling for deleted file names in the sidebar and conditionally suppresses diff pills for deleted files and zero-count non-deleted files. Minor issues: redundant CSS class in ternary, shouldShowDiffPill returns true for null which may contradict the stated "non-zero only" intent, and the directory segment of a deleted path is not struck through.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Render sidebar file entry] --> B{change.status === 'deleted'?}

    B -- Yes --> C[Apply line-through to filename]
    B -- No --> D[Normal filename styling]

    C --> E[Skip diff pills entirely]
    D --> F{shouldShowDiffPill\nadditions}
    F -- true --> G[Render additions pill\n+N or +?]
    F -- false\nvalue === 0 --> H[Hide additions pill]

    D --> I{shouldShowDiffPill\ndeletions}
    I -- true --> J[Render deletions pill\n-N or -?]
    I -- false\nvalue === 0 --> K[Hide deletions pill]

    subgraph shouldShowDiffPill logic
        L[value] --> M{typeof value\n!== 'number'?}
        M -- Yes\nnull or undefined --> N[return true\nshow pill with '?']
        M -- No --> O{value !== 0?}
        O -- Yes --> P[return true\nshow pill]
        O -- No --> Q[return false\nhide pill]
    end

Loading

_{Last reviewed commit: 0b614e8}