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

Inline comments:
In `@packages/app/src/context/layout.tsx`:
- Around line 223-227: The current migratedRightPanel falls through to undefined
for older layouts because it returns non-record rightPanel values as-is; change
the logic so when rightPanel is null/undefined you return an object that
preserves the legacy open state by calling legacyRightPanelOpened(review,
fileTree) (i.e., set opened to legacyRightPanelOpened(review, fileTree) and
merge with the default/rightPanel shape), while still keeping the existing
branches: if isRecord(rightPanel) migrate its opened field, and if rightPanel is
a boolean return it unchanged.

In `@packages/app/src/pages/session/migrate-session-view.ts`:
- Around line 90-95: The nested ternary that computes activeCandidate is hard to
read; extract this logic into a small helper (e.g., computeActiveTab or
resolveActiveCandidate) that takes the entry (or entry.sessionTabsRaw,
entry.migratedSidePanelTab, entry.reviewInSessionTabs) and returns "context" |
"review" | undefined using clear if/return branches. Replace the inline ternary
with a call to that helper (e.g., const activeCandidate =
resolveActiveCandidate(entry)) and keep the same precedence: prefer "context" if
sessionTabsRaw.active === "context", then map "changes" or "review" to "review",
otherwise fall back to migratedSidePanelTab or reviewInSessionTabs -> "review".

In `@packages/app/src/pages/session/right-panel-tabs.ts`:
- Around line 134-142: The moveShellTab function doesn't validate the
destination index and can move tabs before the pinned "status" tab; update
moveShellTab (and its use of ShellTabState.openShellTabs and normalizeShellTabs)
to clamp the incoming to index into the valid range so the tab never lands
before the pinned tab: compute a clampedTo = Math.min(Math.max(to, 1),
state.openShellTabs.length - 1) (or equivalent) before performing the splice and
use clampedTo in place of to; leave the early returns for target === "status"
and from === -1 unchanged.

In `@packages/app/src/pages/session/session-side-panel.tsx`:
- Around line 119-121: openReviewPanel currently only opens the side panel but
doesn't activate the "review" tab, so newly opened files are hidden under the
Tabs.Content value="review" branch; modify openReviewPanel to first select the
"review" panel/tab via the sidePanel selector (e.g. call
view().sidePanel.select("review") or the equivalent select method on sidePanel)
and then ensure the panel is opened (open it only if not already opened) so the
review shell is active before file tabs are opened.

In `@packages/app/src/pages/session/terminal-shell-tab.ts`:
- Around line 16-21: The toggleDesktopTerminal function currently calls
view.sidePanel.toggleTab("terminal") and then directly calls
terminal.open()/terminal.close(), which duplicates the state update performed by
the createEffect in session-side-panel (which also syncs terminal state based on
the side panel tab); if the immediate-call is intentional for UI responsiveness,
add a brief clarifying comment above toggleDesktopTerminal explaining that
terminal.open()/close() are called immediately to provide instant feedback while
the createEffect in session-side-panel.tsx ensures global synchronization and is
idempotent, otherwise remove the direct terminal.open()/close() calls and rely
solely on the createEffect to manage terminal state.

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

Inline comments:
In `@packages/app/e2e/files/file-tree.spec.ts`:
- Around line 16-18: The test currently uses page.getByRole("tablist").first()
(stored in shellTabList) which can grab the wrong tablist; instead scope the
lookup to the right utility panel before selecting the tablist: locate the
right-utility panel via a stable semantic selector (e.g., a
data-component/data-action attribute or a named landmark/role) and call
getByRole("tablist") or locator("role=tablist") on that panel element, then
click the last button and select the "Review" menu item; update references to
shellTabList to use this scoped panel.getByRole("tablist") to avoid ordering
flakiness.

In `@packages/app/src/components/prompt-input.tsx`:
- Around line 206-216: The side-panel openTab("review") call is duplicated;
hoist view().sidePanel.openTab("review") before the wantsReview branch so it's
executed once, then inside the branch only call
view().sidePanel.explorer.setTab("changes"), tabs().setActive("review"), and
queueCommentFocus(), and in the else branch only call
view().sidePanel.explorer.setTab("all") to avoid repeating openTab; use the
existing wantsReview variable to control which explorer tab and follow-up
actions run.

In `@packages/app/src/context/layout.tsx`:
- Around line 209-216: The code currently returns a bare boolean for legacy
rightPanel values which loses the stored open state; change the boolean branch
in migratedRightPanel to return an object instead (e.g., return { width:
DEFAULT_RIGHT_PANEL_WIDTH, opened: rightPanel }) so consumers expecting { width,
opened } see the correct opened value; keep the other branches using
legacyRightPanelOpened(rightPanel, review, fileTree), isRecord, and the existing
opened merge logic intact.

In `@packages/app/src/pages/session/migrate-session-view.ts`:
- Around line 99-114: The current validity check (activeIsValidInnerTab) allows
any file:// active value even if that file isn't present in rawAll, preventing
cleanup; update activeIsValidInnerTab to require that a file:// active also
exists in the rawAll list (i.e., rawActive === undefined || rawActive ===
"review" || (rawActive.startsWith("file://") && rawAll.includes(rawActive))).
Keep the later normalization using fileTabsOnly(rawAll) and the computed active
assignment (which already guards with all.includes(rawActive)), but change the
initial guard so entries with orphaned rawActive values fall into the cleanup
branch that sets sessionTabs[key] and toggles tabsShapeChanged.

In `@packages/app/src/pages/session/session-side-panel.tsx`:
- Around line 217-227: The handler handleShellDragOver computes the drop index
incorrectly for shell tabs: get fromIndex =
view().sidePanel.openTabs().indexOf(from) and toIndex =
view().sidePanel.openTabs().indexOf(to), then if fromIndex < toIndex decrement
toIndex by 1 (i.e. toIndex = toIndex - 1) before calling
view().sidePanel.moveTab(from, toIndex) so the splice-based moveTab inserts at
the hovered position the same way file tabs do; keep the existing guards
(isRightPanelTab checks and -1 checks).

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

Inline comments:
In `@packages/app/src/pages/session/close-session-tab.ts`:
- Around line 11-29: Replace the hardcoded "status" checks with the canonical
closability metadata from RIGHT_PANEL_TAB_META: import RIGHT_PANEL_TAB_META and
use RIGHT_PANEL_TAB_META[tab].closable (or equivalent) in both
canCloseSessionTab and closeSessionTab instead of comparing sidePanelTab() ===
"status"; when closing a shell tab call input.closeShellTab(shellTab) only if
RIGHT_PANEL_TAB_META[shellTab].closable is true, and return true after closing;
ensure you still prefer closing file tabs via input.closeFileTab(fileTab) first
and reference the same RIGHT_PANEL_TAB_META lookup in both functions so
enablement and behavior stay in sync.

In `@packages/app/src/pages/session/helpers.ts`:
- Around line 60-66: The normalized active tab (from input.normalizeTab(active))
is returned as long as it looks like a file tab, but it may be stale; update the
logic in the helper that computes the active/closable tab so that after
computing normalizedActive and verifying input.pathFromTab(normalizedActive),
you also check openedTabs().includes(normalizedActive) (or equivalent
membership) before returning it; this ensures input.tabs().active() entries are
guarded against orphaned/stale tabs the same way activeTab()/openedTabs() are
used elsewhere.