[flcrom]

Fixes #7416.

The commits list wasn't invalidated when an agent or terminal moved HEAD, so it could stay stale until the window refocused. Refresh it on broad git:changed events, alongside the existing branch and diff invalidations. Path-only edits still skip this refresh.

Tests:

• bun test src/renderer/hooks/host-service/useGitStatus/useGitStatus.test.tsx src/renderer/hooks/host-service/useGitStatus/createTrailingRefreshScheduler.test.ts
• bun x biome check apps/desktop/src/renderer/hooks/host-service/useGitStatus/useGitStatus.ts apps/desktop/src/renderer/hooks/host-service/useGitStatus/useGitStatus.test.tsx
• desktop tsc --noEmit

---

Summary by cubic

Fixes #7416 by refreshing the commits list when a broad git:changed event occurs, so external HEAD moves (for example, an agent committing outside the app) no longer leave the list stale. Path-only edits still skip this refresh.

^{Written for commit b891cee. Summary will update on new commits.}

Summary by CodeRabbit

• Bug Fixes

  • Commit lists now refresh when Git metadata changes move the current branch position, ensuring displayed commit history stays up to date.
  • Existing behavior is preserved for changes limited to specific file paths; these do not trigger unnecessary commit-list refreshes.

• Tests

  • Added coverage for Git status events and commit-list refresh behavior.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 61f8f12c-6b3f-4a2b-8343-339f3d00004a

📥 Commits

Reviewing files that changed from the base of the PR and between 0b0d79f and b891cee.

📒 Files selected for processing (2)

• apps/desktop/src/renderer/hooks/host-service/useGitStatus/useGitStatus.test.tsx
• apps/desktop/src/renderer/hooks/host-service/useGitStatus/useGitStatus.ts

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

useGitStatus now invalidates the workspace commit-list query after broad Git changes without paths. Tests verify invalidation for broad events and no invalidation for path-scoped events.

Changes

Git status invalidation

Layer / File(s)	Summary
Commit list invalidation and coverage apps/desktop/src/renderer/hooks/host-service/useGitStatus/useGitStatus.ts, apps/desktop/src/renderer/hooks/host-service/useGitStatus/useGitStatus.test.tsx	The no-paths branch now invalidates git.listCommits. Tests cover broad and path-scoped git:changed events.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix · Severity of issue fixed: Medium

Suggested reviewers: kitenite

Merge Risk: ⚪ Minimal · up to b891c

Broad Git changes that can move HEAD refresh the commits list, while path-scoped changes retain targeted invalidation. No actionable merge risk remains.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main change: refreshing commits after external Git changes. It uses the required conventional commit format.
Description check	✅ Passed	The description explains the problem, the fix, the path-only event behavior, the linked issue, and the tests that were run. It does not include the template checklist, but the required technical infor…
Linked Issues check	✅ Passed	The change addresses issue #7416. When git:changed has no paths, useGitStatus invalidates git.listCommits for the workspace. This refreshes the commits view after an agent or terminal moves `HEA…
Out of Scope Changes check	✅ Passed	The changes are limited to useGitStatus.ts and its test file. The implementation supports commit-list refresh, and the tests verify the required behavior. No unrelated change is shown.

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}