feat: add file readiness summaries to sidebar

Author: haasonsaas

TODO.md

@@ -91,7 +91,7 @@ This roadmap is derived from deep research into Greptile's public docs, blog, MC
 52. [ ] Add comment grouping by unresolved, fixed, stale, and informational sections in `ReviewView`.
 53. [x] Add a "show only blockers" mode for large reviews.
 54. [ ] Add keyboard actions for thumbs, resolve, and jump-to-next-finding workflows.
-55. [ ] Add file-level readiness summaries in the diff sidebar.
+55. [x] Add file-level readiness summaries in the diff sidebar.
 56. [x] Add lifecycle-aware PR summaries that explain what still blocks merge.
 57. [ ] Add a "train the reviewer" callout when thumbs coverage on a review is low.
 58. [ ] Add review-change comparisons so users can diff one review run against the next on the same PR.
@@ -163,4 +163,5 @@ This roadmap is derived from deep research into Greptile's public docs, blog, MC
 - [x] Surface lifecycle-aware PR readiness summaries in the GitHub PR detail workflow.
 - [x] Surface unresolved blocker counts in repo and PR GitHub discovery views.
 - [x] Add a blocker-only review mode that narrows large reviews to open Error and Warning findings.
+- [x] Add file-level readiness summaries to the review diff sidebar.
 - [ ] Commit and push each validated checkpoint before moving to the next epic.

web/src/components/FileSidebar.tsx

@@ -10,6 +10,15 @@ interface Props {
   onSelectFile: (file: string | null) => void
 }
 
+type FileStats = {
+  total: number
+  worst: number
+  openBlockers: number
+  openInformational: number
+  resolved: number
+  dismissed: number
+}
+
 const statusIcon = {
   added: FilePlus,
   deleted: FileX,
@@ -26,14 +35,71 @@ const statusDot = {
 
 const sevRank: Record<Severity, number> = { Error: 0, Warning: 1, Info: 2, Suggestion: 3 }
 
+function buildReadinessLabel(stats: FileStats): { label: string; tone: string } {
+  if (stats.openBlockers > 0) {
+    return {
+      label: `${stats.openBlockers} blocker${stats.openBlockers === 1 ? '' : 's'}`,
+      tone: 'text-sev-warning',
+    }
+  }
+
+  if (stats.openInformational > 0) {
+    return {
+      label: 'Info only',
+      tone: 'text-accent',
+    }
+  }
+
+  return {
+    label: 'Clear',
+    tone: 'text-sev-suggestion',
+  }
+}
+
+function buildReadinessDetails(stats: FileStats): string | null {
+  const details: string[] = []
+
+  if (stats.openBlockers > 0 && stats.openInformational > 0) {
+    details.push(`${stats.openInformational} info`)
+  }
+  if (stats.resolved > 0) {
+    details.push(`${stats.resolved} resolved`)
+  }
+  if (stats.dismissed > 0) {
+    details.push(`${stats.dismissed} dismissed`)
+  }
+
+  return details.length > 0 ? details.join(' • ') : null
+}
+
 export function FileSidebar({ files, comments, selectedFile, onSelectFile }: Props) {
   const fileStats = useMemo(() => {
-    const stats = new Map<string, { count: number; worst: number }>()
+    const stats = new Map<string, FileStats>()
     for (const c of comments) {
       const path = c.file_path.replace(/^\.\//, '')
-      const existing = stats.get(path) || { count: 0, worst: 4 }
-      existing.count++
+      const existing = stats.get(path) || {
+        total: 0,
+        worst: 4,
+        openBlockers: 0,
+        openInformational: 0,
+        resolved: 0,
+        dismissed: 0,
+      }
+      const lifecycle = c.status ?? 'Open'
+
+      existing.total++
       existing.worst = Math.min(existing.worst, sevRank[c.severity])
+
+      if (lifecycle === 'Resolved') {
+        existing.resolved++
+      } else if (lifecycle === 'Dismissed') {
+        existing.dismissed++
+      } else if (c.severity === 'Error' || c.severity === 'Warning') {
+        existing.openBlockers++
+      } else {
+        existing.openInformational++
+      }
+
       stats.set(path, existing)
     }
     return stats
@@ -95,23 +161,35 @@ export function FileSidebar({ files, comments, selectedFile, onSelectFile }: Pro
               const Icon = statusIcon[file.status]
               const stats = fileStats.get(file.path)
               const isSelected = selectedFile === file.path
+              const readiness = stats ? buildReadinessLabel(stats) : null
+              const readinessDetails = stats ? buildReadinessDetails(stats) : null
 
               return (
                 <button
                   key={file.path}
                   onClick={() => onSelectFile(file.path)}
-                  className={`w-full text-left px-3 py-1 flex items-center gap-1.5 text-[12px] transition-colors ${
+                  className={`w-full text-left px-3 py-1.5 flex items-start gap-1.5 text-[12px] transition-colors ${
                     isSelected
                       ? 'bg-accent/10 text-accent'
                       : 'text-text-secondary hover:bg-surface-2 hover:text-text-primary'
                   }`}
                 >
                   <span className={`w-1.5 h-1.5 rounded-full shrink-0 ${statusDot[file.status]}`} />
                   <Icon size={12} className="shrink-0 text-text-muted" />
-                  <span className="truncate font-code">{getFileName(file.path)}</span>
+                  <div className="min-w-0 flex-1">
+                    <div className="truncate font-code">{getFileName(file.path)}</div>
+                    {readiness && (
+                      <div className="mt-0.5 flex flex-wrap items-center gap-x-1.5 gap-y-0.5 text-[10px]">
+                        <span className={`font-medium ${readiness.tone}`}>{readiness.label}</span>
+                        {readinessDetails && (
+                          <span className="text-text-muted">{readinessDetails}</span>
+                        )}
+                      </div>
+                    )}
+                  </div>
                   {stats && (
                     <span className={`ml-auto shrink-0 text-[10px] text-white px-1 py-0 rounded ${countColor(stats.worst)}`}>
-                      {stats.count}
+                      {stats.total}
                     </span>
                   )}
                 </button>

web/src/components/__tests__/FileSidebar.test.tsx

@@ -0,0 +1,73 @@
+import { render, screen } from '@testing-library/react'
+import userEvent from '@testing-library/user-event'
+import { describe, expect, it, vi } from 'vitest'
+
+import { FileSidebar } from '../FileSidebar'
+import type { Comment, DiffFile } from '../../api/types'
+
+function makeFile(path: string): DiffFile {
+  return {
+    path,
+    status: 'modified',
+    hunks: [],
+  }
+}
+
+function makeComment(overrides: Partial<Comment> = {}): Comment {
+  return {
+    id: 'comment-1',
+    file_path: 'src/a.ts',
+    line_number: 1,
+    content: 'A finding',
+    severity: 'Error',
+    category: 'Bug',
+    confidence: 0.9,
+    tags: [],
+    fix_effort: 'Medium',
+    status: 'Open',
+    ...overrides,
+  }
+}
+
+describe('FileSidebar', () => {
+  it('renders file-level readiness summaries from comment lifecycle state', () => {
+    render(
+      <FileSidebar
+        files={[makeFile('src/a.ts'), makeFile('src/b.ts'), makeFile('src/c.ts')]}
+        comments={[
+          makeComment(),
+          makeComment({ id: 'comment-2', severity: 'Info' }),
+          makeComment({ id: 'comment-3', severity: 'Warning', status: 'Resolved' }),
+          makeComment({ id: 'comment-4', file_path: 'src/b.ts', severity: 'Info' }),
+          makeComment({ id: 'comment-5', file_path: 'src/b.ts', severity: 'Suggestion', status: 'Dismissed' }),
+          makeComment({ id: 'comment-6', file_path: 'src/c.ts', severity: 'Warning', status: 'Resolved' }),
+        ]}
+        selectedFile={null}
+        onSelectFile={() => {}}
+      />,
+    )
+
+    expect(screen.getByText('1 blocker')).toBeInTheDocument()
+    expect(screen.getByText('1 info • 1 resolved')).toBeInTheDocument()
+    expect(screen.getByText('Info only')).toBeInTheDocument()
+    expect(screen.getByText('1 dismissed')).toBeInTheDocument()
+    expect(screen.getByText('Clear')).toBeInTheDocument()
+  })
+
+  it('keeps file selection working', async () => {
+    const user = userEvent.setup()
+    const onSelectFile = vi.fn()
+
+    render(
+      <FileSidebar
+        files={[makeFile('src/a.ts')]}
+        comments={[makeComment()]}
+        selectedFile={null}
+        onSelectFile={onSelectFile}
+      />,
+    )
+
+    await user.click(screen.getByRole('button', { name: /a.ts/i }))
+    expect(onSelectFile).toHaveBeenCalledWith('src/a.ts')
+  })
+})