[maria-rcks]

mobile's react test project fails to collect the branch occupancy test with Unexpected token 'typeof' on main (ci run).

mock the worktree hook's unused entities and threads dependencies to keep native sources out of collection while preserving its real cwd calculation and both checkout occupancy assertions.

Written by gpt-6.1-sol on behalf of Maria

[macroscopeapp]

Approvability

Verdict: Approved at 0df9bbb

Macroscope's review found this PR approvable — The change is confined to ignored paths and is described as isolating a mobile test from native dependencies. No material product or operational risk is apparent within that assessment scope.

Notes:

• No code objects were reviewed. Approvability was decided on eligibility alone.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: maria/fix-mobile-occupancy-test
• File: .coderabbit.config.ts
• Commit: 0df9bbb
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 4f6ca907-55cb-4111-b780-493b7869b177

Commits

Reviewing files that changed from the base of the PR and between fa4f3a2 and 0df9bbb.

Files selected for processing: 1

• apps/mobile/src/features/threads/git/GitBranchesSheet.occupancy.test.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 7 remain after this review.

Walkthrough

The occupancy test now mocks the entities and threads state modules as empty modules.

Changes: Occupancy test setup

Layer / File(s)	Summary
State module mocks apps/mobile/src/features/threads/git/GitBranchesSheet.occupancy.test.tsx	The test adds empty-module mocks for the entities and threads state modules.

Priority: Normal

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Possibly related PRs

• pingdotgg/t3code#23764: Adds the occupancy test and changes GitBranchesSheet to classify the main checkout by its effective cwd. This PR updates that test's setup.

Suggested labels: size:XS

Suggested reviewers: only21mil

Merge Risk: Minimal · up to 0df9b

The test setup change preserves the existing occupancy coverage and is safe to merge.

Pre-merge checks: 3 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Description check		The description clearly states the collection failure and the mocking change. It omits the required Scope and approval and Verification sections, including test results and any unavailable checks.	Add Scope and approval with the issue or approval reference, or explain why this focused fix is exempt. Add Verification with the focused test command, observed result, and any checks not run.

Passed checks: 3 passed

Check name	Status	Explanation
Title check		The title is concise, specific, and accurately summarizes the main change: loading the mobile branch occupancy test without native sources.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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