fix(ui): hide empty changes badge while loading

Author: fy0

ui/src/components/changes/__tests__/gitChangesSummary.test.ts

@@ -223,7 +223,7 @@ describe('gitChangesSummary', () => {
     ).toBe('');
   });
 
-  it('formats pending badge deltas and keeps loading badges visible', () => {
+  it('formats pending badge deltas and hides zero-valued loading badges', () => {
     expect(formatGitChangesBadgeDelta('+', null)).toBe('+?');
     expect(formatGitChangesBadgeDelta('-', 4)).toBe('-4');
 
@@ -234,6 +234,14 @@ describe('gitChangesSummary', () => {
         deletions: 0,
         pending: true,
       })
+    ).toBe(false);
+    expect(
+      shouldShowGitChangesBadge({
+        count: 1,
+        additions: null,
+        deletions: null,
+        pending: true,
+      })
     ).toBe(true);
     expect(
       shouldShowGitChangesBadge({

ui/src/components/changes/gitChangesSummary.ts

@@ -135,5 +135,5 @@ export function shouldShowGitChangesBadge(summary: GitChangesBadgeSummary | null
   if (!summary) {
     return false;
   }
-  return summary.pending || summary.count > 0;
+  return summary.count > 0;
 }