fix(desktop): restore workspace logo to rail top

Revert the iteration-8 move that placed the workspace switcher above
Settings; match Nexus sidebar-top pattern with Settings alone in rail-bottom.

Author: jravenel

libs/naas-abi/naas_abi/apps/desktop/AGENTS.md

@@ -125,7 +125,7 @@ desktop/
 
 A workspace is a **folder on disk** (VS Code / Cursor semantics), not a Nexus tenant.
 
-- **UI**: icon rail bottom (above Settings): **logo button only** opens a glass portal dropdown to the right (square corners). Hover on the logo shows the full path; the menu lists recent workspaces with a checkmark on the active entry and **Open Folder…**. The status bar left shows the current workspace basename and git branch as read-only context (no switch action). Top bar and main body stay clean (panel toggle + section title only). Rail top is an empty spacer.
+- **UI**: icon rail top (Nexus sidebar-top pattern): **logo button only** opens a glass portal dropdown to the right (square corners). Hover on the logo shows the full path; the menu lists recent workspaces with a checkmark on the active entry and **Open Folder…**. The status bar left shows the current workspace basename and git branch as read-only context (no switch action). Top bar and main body stay clean (panel toggle + section title only). Rail bottom is Settings only.
 - **Switch / open**: `POST /api/workspaces/open` or `PUT /api/settings` with a new `workspace_root`. Triggers `ensure_workspace`, harness restart, terminal reconnect, file index refresh, org/model context reload, and graph rescaffold.
 - **Recent list**: `recent_workspaces` setting (JSON array, max 10 paths). Updated on every open/switch.
 - **First run**: `maybe_upgrade_workspace_setting()` auto-detects `~/abi` (git + `.env`) when still on the factory default.
@@ -200,7 +200,7 @@ Scaffolded `instances.ttl` seeds concrete routing individuals:
 **Iteration 8** (current):
 
 - **Events rail section**: dedicated `#events` main view with full-height process events table (`GET /api/processes`); row click opens detail panel, double-click or **View in Graph** navigates to Graph Overview and focuses the process node
-- Workspace logo switcher moved to rail bottom (above Settings); rail top is spacer only
+- Workspace logo switcher at rail top (Nexus sidebar-top pattern); rail bottom is Settings only
 - Graph Overview tab: vis-network canvas only (process events table removed from split view above graph)
 - Tables tab still shows raw SQLite dumps (`processes`, `process_aspects`) for debugging
 

libs/naas-abi/naas_abi/apps/desktop/gui/web/index.html

@@ -12,14 +12,7 @@
     <div id="app-shell">
     <!-- Icon rail (Nexus Sidebar, collapsed w-14) -->
     <nav id="rail" class="glass">
-      <div class="rail-top" aria-hidden="true"></div>
-      <div class="rail-nav">
-        <button class="rail-btn active" data-section="chat" title="Chat" data-icon="message-square"></button>
-        <button class="rail-btn" data-section="code" title="Code" data-icon="code-xml"></button>
-        <button class="rail-btn" data-section="graph" title="Knowledge Graph" data-icon="waypoints"></button>
-        <button class="rail-btn" data-section="events" title="Events" data-icon="scroll-text"></button>
-      </div>
-      <div class="rail-bottom">
+      <div class="rail-top">
         <button
           id="workspace-switcher"
           class="workspace-switcher"
@@ -30,6 +23,14 @@
         >
           <img id="workspace-logo" src="/static/assets/abi-logo.png" alt="ABI" width="28" height="28" />
         </button>
+      </div>
+      <div class="rail-nav">
+        <button class="rail-btn active" data-section="chat" title="Chat" data-icon="message-square"></button>
+        <button class="rail-btn" data-section="code" title="Code" data-icon="code-xml"></button>
+        <button class="rail-btn" data-section="graph" title="Knowledge Graph" data-icon="waypoints"></button>
+        <button class="rail-btn" data-section="events" title="Events" data-icon="scroll-text"></button>
+      </div>
+      <div class="rail-bottom">
         <button class="rail-btn" data-section="settings" title="Settings" data-icon="settings"></button>
       </div>
     </nav>

libs/naas-abi/naas_abi/apps/desktop/gui/web/shell_ui_test.py

@@ -37,24 +37,22 @@ def _read(name: str) -> str:
     return (WEB_DIR / name).read_text(encoding="utf-8")
 
 
-def test_rail_logo_workspace_switcher_in_bottom() -> None:
+def test_rail_logo_workspace_switcher_in_top() -> None:
     html = _read("index.html")
     assert 'id="workspace-switcher"' in html
     assert 'id="workspace-logo"' in html
     assert 'id="workspace-name"' not in html
     assert "workspace-switcher-label" not in html
     assert "workspace-chevron" not in html
-    rail_bottom_start = html.index('class="rail-bottom"')
+    rail_top_start = html.index('class="rail-top"')
     rail_nav_start = html.index('class="rail-nav"')
+    rail_bottom_start = html.index('class="rail-bottom"')
     workspace_idx = html.index('id="workspace-switcher"')
     settings_idx = html.index('data-section="settings"')
-    assert rail_nav_start < workspace_idx
-    assert workspace_idx < settings_idx
-    assert html.index('class="rail-bottom"', rail_bottom_start) == rail_bottom_start
-    assert workspace_idx > rail_bottom_start
-    assert 'class="rail-top"' in html
-    rail_top_end = html.index("</div>", html.index('class="rail-top"'))
-    assert html.index('id="workspace-switcher"') > rail_top_end
+    assert rail_top_start < workspace_idx < rail_nav_start
+    assert rail_nav_start < settings_idx
+    assert workspace_idx < rail_bottom_start
+    assert 'aria-hidden="true"' not in html[html.index('class="rail-top"'):rail_nav_start]
 
 
 def test_status_bar_shows_workspace_name() -> None:
@@ -93,7 +91,7 @@ def test_workspace_switcher_css_logo_button() -> None:
     css = _read("style.css")
     assert ".glass-card" in css
     assert ".workspace-switcher" in css
-    assert ".rail-bottom .workspace-switcher" in css
+    assert ".rail-top" in css
     assert "workspace-switcher-label" not in css
     assert "#workspace-name" not in css
 

libs/naas-abi/naas_abi/apps/desktop/gui/web/style.css

@@ -111,8 +111,11 @@ svg.icon { flex-shrink: 0; }
 }
 
 .rail-top {
-  height: 12px;
+  height: 56px;
   flex-shrink: 0;
+  display: flex;
+  align-items: center;
+  justify-content: center;
 }
 
 .rail-bottom {
@@ -125,10 +128,6 @@ svg.icon { flex-shrink: 0; }
   border-top: 1px solid color-mix(in srgb, var(--border) 50%, transparent);
 }
 
-.rail-bottom .workspace-switcher {
-  margin-bottom: 4px;
-}
-
 #workspace-logo {
   width: 100%;
   height: 100%;