refactor(mobile): one inspector column, declared with <Inspector>#27085

External image Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: t3/workspace-split-1
• File: .coderabbit.config.ts
• Commit: 3bf1ebc
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Team
• Run ID: 2e1f00e9-b9e0-4ec0-80ba-f38bfe06a6ab

External image Commits

Reviewing files that changed from the base of the PR and between 10935b014e5e6f6ce9a82743f1699e3439e9e869 and 3bf1ebcbd2be2f544b979cac736c32b935b617bd.

External image Files selected for processing: 7

• apps/mobile/src/features/files/ThreadFilesRouteScreen.tsx
• apps/mobile/src/features/layout/AdaptiveWorkspaceLayout.tsx
• apps/mobile/src/features/review/ReviewSheet.tsx
• apps/mobile/src/features/threads/ThreadRouteScreen.tsx
• apps/mobile/src/features/threads/thread-inspector-navigation.test.tsx
• apps/mobile/src/lib/layout.test.ts
• apps/mobile/src/lib/layout.ts

External image Files with no reviewable changes: 1

• apps/mobile/src/lib/layout.test.ts

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

The mobile adaptive workspace layout replaces role-based auxiliary panes with a file inspector pane. Screens render inspector content through Inspector, and thread actions use showInspector to reveal the pane.

Changes: Adaptive workspace inspector

Layer / File(s)	Summary
File inspector pane layout apps/mobile/src/lib/layout.ts, apps/mobile/src/lib/layout.test.ts	Auxiliary-pane support, visibility, and width now use file-inspector layout calculations. The role-based pane type and input are removed, and existing visibility expectations remain unchanged.
Inspector registration and reveal apps/mobile/src/Stack.tsx, apps/mobile/src/features/layout/AdaptiveWorkspaceLayout.tsx, apps/mobile/src/features/layout/adaptive-workspace-context.ts	The layout registers focused inspector content and tracks reveal keys by route. It exposes showInspector and uses file-inspector preferences for pane visibility and width. Workspace route data now provides the active route key and name instead of keys for all routes.
Screen inspector integration apps/mobile/src/features/files/ThreadFilesRouteScreen.tsx, apps/mobile/src/features/threads/ThreadRouteScreen.tsx, apps/mobile/src/features/review/ReviewSheet.tsx, apps/mobile/src/features/threads/thread-inspector-navigation.test.tsx	File, thread, and review screens render inspector content through Inspector. Thread actions reveal the file inspector, file-screen branches retain the inspector wrapper, and the navigation test exercises registration through rendered content.

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Refactor

Sequence Diagram(s)

mermaid

1	sequenceDiagram
2	participant ThreadRouteScreen
3	participant Inspector
4	participant AdaptiveWorkspaceContext
5	participant AdaptiveWorkspaceLayout
6	ThreadRouteScreen->>Inspector: Render inspector content with revealKey
7	Inspector->>AdaptiveWorkspaceContext: Register focused content and reveal data
8	AdaptiveWorkspaceContext->>AdaptiveWorkspaceLayout: Provide registered content and reveal data
9	AdaptiveWorkspaceLayout->>AdaptiveWorkspaceLayout: Record reveal key and show file inspector

Suggested reviewers: pixpmusic, maria-rcks, t3dotgg

No actionable merge-blocking issue was identified in this change. Normal checks remain appropriate before merging.

^Failed Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Description check	Warning	The description explains the problem and the implementation, but it does not follow the required section structure. It lacks explicit Scope and approval details and provides no focused verification re…	Add the required Problem, Change, Scope and approval, and Verification sections. Link the triaged issue or maintainer approval, and document the tests or manual checks performed with their observed results. Include before/after UI evidence …

^Passed Passed checks: 3 passed

Check name	Status	Explanation
Title check	Passed	The title is concise, specific, and accurately describes the main change: replacing separate workspace-pane roles with a declared Inspector component.
Linked Issues check	Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	Passed	Check skipped because no linked issues were found for this pull request.

Full details: Description check

Explanation

The description explains the problem and the implementation, but it does not follow the required section structure. It lacks explicit Scope and approval details and provides no focused verification results or stated limitations.

Resolution

Add the required Problem, Change, Scope and approval, and Verification sections. Link the triaged issue or maintainer approval, and document the tests or manual checks performed with their observed results. Include before/after UI evidence if required.

Generate docstrings

• Commit to this branch
• Create a new PR

Generate unit tests (beta)

• Commit to this branch
• Create a new PR

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 108.5 KiB
• Claude decoded thread snapshot: 108.8 KiB