packages/ui/src/i18n/session-review-title.test.ts (1)

6-9: Optionally assert the new branch title key in the same suite.

Since ui.sessionReview.title.branch was introduced alongside git, adding assertions here would keep this regression test complete.

Suggested test expansion

 describe("session review title translations", () => {
   test("localizes Git changes for Chinese users", () => {
     expect(zh["ui.sessionReview.title.git"]).toBe("文件变更")
     expect(zht["ui.sessionReview.title.git"]).toBe("檔案變更")
+    expect(zh["ui.sessionReview.title.branch"]).toBe("分支变更")
+    expect(zht["ui.sessionReview.title.branch"]).toBe("分支變更")
   })
 })

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@packages/ui/src/i18n/session-review-title.test.ts` around lines 6 - 9, The
test "localizes Git changes for Chinese users" is missing assertions for the new
key ui.sessionReview.title.branch; update that test to also assert
zh["ui.sessionReview.title.branch"] and zht["ui.sessionReview.title.branch"]
equal the expected Chinese strings (use the same locale fixtures zh and zht used
for the git key) so both branch and git keys are checked in the same suite.

packages/app/src/pages/session/review-tab.test.tsx (1)

37-42: Consider cleanup of global React mock.

The React.createElement mock attached to globalThis in beforeEach is not cleaned up in afterEach. While beforeEach resets it each time, consider adding cleanup in afterAll to avoid polluting the global scope for subsequent test files:

 afterAll(() => {
   mock.restore()
+  delete (globalThis as any).React
 })

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@packages/app/src/pages/session/review-tab.test.tsx` around lines 37 - 42, The
test attaches a mock to globalThis.React.createElement in the beforeEach block
and never restores it; capture the original globalThis.React (e.g., const
_originalReact = globalThis.React) before mocking, then add an afterAll that
restores globalThis.React to _originalReact (or deletes globalThis.React if it
was undefined) to clean up the global mock; update the same test file’s
beforeEach/afterAll so React.createElement is mocked for each test but the
original global is restored afterwards.

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes