fix: terminal/browser panels open behind agent panel in single mode

[liyakhar]

Summary

Terminal and browser panels now open correctly in single-panel view mode instead of being hidden behind the agent panel. A secondary fix removes the group-hover fade effect on project header action icons so each icon highlights independently.

What changed

Panel store: terminal/browser as first-class top-level panels (panel-store.svelte.ts)

Terminal and browser panels are stored outside workspacePanels, but 3 key mechanisms only checked workspacePanels when resolving targets:

1. isTopLevelFullscreenTarget — didn't recognize terminal/browser panel IDs, routing them through the wrong fullscreen path.
2. focusOpenedTopLevelPanel — couldn't resolve project paths for terminal/browser panels, leaving focusedViewProjectPath unset and the panel invisible.
3. topLevelPanelsWithProject in panels-container.svelte — missed terminal/browser panels entirely, so single-mode view state fell back to the agent panel as fullscreen target.

All 3 sites now include terminal and browser panels. A new resolveTopLevelPanelProjectPath helper unifies the project-path lookup across workspace, terminal, and browser panels.

Project header icon hover (session-list-ui.svelte)

projectHeaderHoverActionButtonClass previously used opacity-50 group-hover:opacity-100, causing all 3 folder/terminal/browser icons to fade in together on row hover. Now each icon uses only per-icon hover:bg-accent hover:text-foreground, matching the rest of the app's icon hover behavior. transition-all narrowed to transition-colors.

CI note

The Frontend CI job fails on main (pre-existing runed/bun-test ESM resolution issue). Unrelated to this PR.

---

[cubic-dev-ai]

No issues found across 1 file

_{Re-trigger cubic}

[cubic-dev-ai]

1 issue found across 33 files (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="packages/desktop/tabler-icons-mock.ts">

<violation number="1" location="packages/desktop/tabler-icons-mock.ts:85">
P2: Duplicate mock.module registration for the same worker module creates unnecessary maintenance risk</violation>
</file>

_{Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic}

[cubic-dev-ai]

P2: Duplicate mock.module registration for the same worker module creates unnecessary maintenance risk

Prompt for AI agents

Check if this issue is valid — if so, understand the root cause and fix it. At packages/desktop/tabler-icons-mock.ts, line 85:

<comment>Duplicate mock.module registration for the same worker module creates unnecessary maintenance risk</comment>

<file context>
@@ -82,51 +82,8 @@ mock.module("vaul-svelte", () => ({
-	box: runedState,
-	watcher: runedNoop,
-	AnimationFrames: class {},
+mock.module("@pierre/diffs/worker/worker.js?worker&url", () => ({
+	default: "/mock-pierre-worker.js",
 }));
</file context>

_{Tip: Review your code locally with the cubic CLI to iterate faster.}

[cubic-dev-ai]

You're iterating quickly on this pull request. To help protect your rate limits, cubic has paused automatic reviews on new pushes for now—when you're ready for another review, comment @cubic-dev-ai review.