fix(desktop): restore gateway footer after workflow removal

Author: Pyiner

AGENTS.md

@@ -77,6 +77,9 @@ Detailed data and runtime contracts: @docs/agents/repository-contracts.md and
   top-level concepts.
 - Use native platform patterns: Electron/shadcn-style desktop surfaces where
   appropriate, and native grouped iOS management surfaces on mobile.
+- Desktop app-shell chrome must keep its complete geometry and state recipe in
+  an always-loaded owner stylesheet. Never place global shell component rules
+  in a removable feature stylesheet; pin the owner import with a contract test.
 - Mobile route state, presentation mapping, formatting, and business-rule
   transformations should live in `GaryxMobileCore` with SwiftPM tests.
 - Message, transcript, and tool-row display is server-render-state first:

CLAUDE.md

@@ -77,6 +77,9 @@ Detailed data and runtime contracts: @docs/agents/repository-contracts.md and
   top-level concepts.
 - Use native platform patterns: Electron/shadcn-style desktop surfaces where
   appropriate, and native grouped iOS management surfaces on mobile.
+- Desktop app-shell chrome must keep its complete geometry and state recipe in
+  an always-loaded owner stylesheet. Never place global shell component rules
+  in a removable feature stylesheet; pin the owner import with a contract test.
 - Mobile route state, presentation mapping, formatting, and business-rule
   transformations should live in `GaryxMobileCore` with SwiftPM tests.
 - Message, transcript, and tool-row display is server-render-state first:

desktop/garyx-desktop/src/renderer/src/sidebar-footer-design.test.mjs

@@ -1,6 +1,6 @@
 import test from 'node:test';
 import assert from 'node:assert/strict';
-import { readFileSync } from 'node:fs';
+import { readFileSync, readdirSync } from 'node:fs';
 import path from 'node:path';
 import { fileURLToPath } from 'node:url';
 
@@ -9,6 +9,7 @@ import { fileURLToPath } from 'node:url';
 // insets and gap, a 29px identity row, and a 32px trailing utility button.
 
 const rendererDir = path.dirname(fileURLToPath(import.meta.url));
+const stylesDir = path.join(rendererDir, 'styles');
 const read = (relativePath) =>
   readFileSync(path.join(rendererDir, relativePath), 'utf8');
 
@@ -57,13 +58,13 @@ test('sidebar footer pins the Codex frame and divider geometry', () => {
 });
 
 test('gateway and settings controls pin the Codex footer alignment', () => {
-  const layoutCss = read('styles/workflows.css');
-  expectRule(layoutCss, '.gateway-identity-bar', [
+  const ownerCss = read('styles/gateway-status.css');
+  expectRule(ownerCss, '.gateway-identity-bar', [
     'gap: 8px',
     'height: 46px',
     'padding: 0 8px',
   ]);
-  expectRule(layoutCss, '.gateway-identity-main', [
+  expectRule(ownerCss, '.gateway-identity-main', [
     'gap: 8px',
     'height: 29px',
     'padding: 0 8px',
@@ -73,17 +74,16 @@ test('gateway and settings controls pin the Codex footer alignment', () => {
     'font-weight: 445',
     'line-height: 21px',
   ]);
-  expectRule(layoutCss, '.gateway-identity-main:hover', [
+  expectRule(ownerCss, '.gateway-identity-main:hover', [
     'background: var(--color-token-row-hover)',
   ]);
 
-  const controlsCss = read('styles/task-forest.css');
-  expectRule(controlsCss, '.gateway-identity-name', [
+  expectRule(ownerCss, '.gateway-identity-name', [
     'font-size: inherit',
     'font-weight: inherit',
     'line-height: inherit',
   ]);
-  expectRule(controlsCss, '.gateway-identity-gear', [
+  expectRule(ownerCss, '.gateway-identity-gear', [
     'gap: 4px',
     'width: 32px',
     'height: 32px',
@@ -93,7 +93,7 @@ test('gateway and settings controls pin the Codex footer alignment', () => {
     'color: var(--color-token-description-foreground)',
     'font: 445 16px/24px -apple-system',
   ]);
-  expectRule(controlsCss, '.gateway-identity-gear:hover', [
+  expectRule(ownerCss, '.gateway-identity-gear:hover', [
     'background: var(--color-token-row-hover)',
   ]);
 });
@@ -122,3 +122,26 @@ test('gateway identity keeps Codex icon scale while preserving status semantics'
   );
   assert.ok(!source.includes('gateway-identity-status'));
 });
+
+test('gateway switcher recipe is always loaded and has one stylesheet owner', () => {
+  const entryCss = read('styles.css');
+  assert.equal(
+    entryCss.match(/@import "\.\/styles\/gateway-status\.css";/g)?.length,
+    1,
+    'the renderer entrypoint must import the gateway owner stylesheet exactly once',
+  );
+  assert.ok(
+    !entryCss.includes('workflows.css'),
+    'app-shell gateway chrome must not depend on the removed workflow feature',
+  );
+
+  const gatewaySelector = /\.(?:gateway-identity|gateway-switcher|gateway-row-glyph|gateway-glyph-badge)(?:[-\w]*)/;
+  const offenders = readdirSync(stylesDir)
+    .filter((fileName) => fileName.endsWith('.css') && fileName !== 'gateway-status.css')
+    .filter((fileName) => gatewaySelector.test(read(`styles/${fileName}`)));
+  assert.deepEqual(
+    offenders,
+    [],
+    `gateway switcher selectors escaped their owner stylesheet: ${offenders.join(', ')}`,
+  );
+});

desktop/garyx-desktop/src/renderer/src/styles/gateway-status.css

@@ -1,4 +1,45 @@
-/* Gateway presence glyph badge. */
+/* Bottom-left gateway identity / switcher and shared presence glyphs.
+ *
+ * This is app-shell chrome, so its complete layout recipe must stay in this
+ * always-loaded owner stylesheet. Do not move pieces into removable feature
+ * stylesheets: deleting the old workflow feature once left the trigger with
+ * only its child styles and collapsed it into a vertical, content-width stack.
+ */
+
+.gateway-identity-bar {
+  display: flex;
+  align-items: center;
+  gap: 8px;
+  width: 100%;
+  height: 46px;
+  min-width: 0;
+  padding: 0 8px;
+}
+
+.gateway-identity-main {
+  display: flex;
+  align-items: center;
+  gap: 8px;
+  flex: 1;
+  height: 29px;
+  min-width: 0;
+  padding: 0 8px;
+  border: none;
+  border-radius: 10px;
+  background: transparent;
+  color: var(--color-token-text-primary);
+  font-family: -apple-system, BlinkMacSystemFont, "Segoe UI", sans-serif;
+  font-size: 14px;
+  font-weight: 445;
+  line-height: 21px;
+  text-align: left;
+  cursor: default;
+}
+
+.gateway-identity-main:hover,
+.gateway-identity-main[data-state='open'] {
+  background: var(--color-token-row-hover);
+}
 
 /* Avatar-like gateway glyph with a presence badge on its corner. */
 .gateway-identity-glyph,
@@ -68,3 +109,75 @@
 .gateway-glyph-badge.is-offline {
   background: var(--color-token-error-foreground);
 }
+
+.gateway-identity-copy {
+  flex: 1;
+  min-width: 0;
+}
+
+.gateway-identity-name {
+  display: block;
+  min-width: 0;
+  overflow: hidden;
+  color: var(--color-token-text-primary);
+  font-size: inherit;
+  font-weight: inherit;
+  line-height: inherit;
+  text-overflow: ellipsis;
+  white-space: nowrap;
+}
+
+.gateway-identity-gear {
+  flex: none;
+  display: inline-flex;
+  align-items: center;
+  justify-content: center;
+  gap: 4px;
+  width: 32px;
+  height: 32px;
+  padding: 4px 0;
+  border: 1px solid transparent;
+  border-radius: 10px;
+  background: transparent;
+  color: var(--color-token-description-foreground);
+  font: 445 16px/24px -apple-system, BlinkMacSystemFont, "Segoe UI", sans-serif;
+  cursor: default;
+}
+
+.gateway-identity-gear:hover {
+  background: var(--color-token-row-hover);
+  color: var(--color-token-description-foreground);
+}
+
+.gateway-switcher-popover {
+  /* Hug the identity bar below. Surface identity comes from the shared
+     .menu-popover-surface recipe in styles/menus.css. */
+  width: var(--radix-popover-trigger-width);
+  min-width: 208px;
+  padding: 8px;
+}
+
+.gateway-switcher-list {
+  display: grid;
+  gap: 2px;
+}
+
+/* Row visuals come from the shared two-line menu variant in styles/menus.css;
+   only gateway-specific chrome lives here. */
+.gateway-switcher-item-state {
+  flex: none;
+  color: var(--color-token-description-foreground);
+  font-size: var(--text-xs-plus);
+}
+
+.gateway-switcher-check {
+  flex: none;
+  color: var(--color-token-text-primary);
+}
+
+.gateway-switcher-error {
+  margin: 4px 0 0;
+  padding: 0 10px;
+  color: #9b3d3d;
+  font-size: 11.5px;
+}

desktop/garyx-desktop/src/renderer/src/styles/task-forest.css

@@ -594,76 +594,3 @@
     transition: none;
   }
 }
-
-.gateway-identity-copy {
-  flex: 1;
-  min-width: 0;
-}
-
-.gateway-identity-name {
-  display: block;
-  min-width: 0;
-  overflow: hidden;
-  color: var(--color-token-text-primary);
-  font-size: inherit;
-  font-weight: inherit;
-  line-height: inherit;
-  text-overflow: ellipsis;
-  white-space: nowrap;
-}
-
-.gateway-identity-gear {
-  flex: none;
-  display: inline-flex;
-  align-items: center;
-  justify-content: center;
-  gap: 4px;
-  width: 32px;
-  height: 32px;
-  padding: 4px 0;
-  border: 1px solid transparent;
-  border-radius: 10px;
-  background: transparent;
-  color: var(--color-token-description-foreground);
-  font: 445 16px/24px -apple-system, BlinkMacSystemFont, "Segoe UI", sans-serif;
-  cursor: default;
-}
-
-.gateway-identity-gear:hover {
-  background: var(--color-token-row-hover);
-  color: var(--color-token-description-foreground);
-}
-
-.gateway-switcher-popover {
-  /* Hug the identity bar below: same width as the trigger, so the popover
-     and the bar read as one column even when the sidebar is resized.
-     Surface identity comes from .menu-popover-surface (styles/menus.css). */
-  width: var(--radix-popover-trigger-width);
-  min-width: 208px;
-  padding: 8px;
-}
-
-.gateway-switcher-list {
-  display: grid;
-  gap: 2px;
-}
-
-/* Row visuals come from the shared two-line menu variant in styles/menus.css
-   (.menu-item-two-line and friends); only gateway-specific chrome is local. */
-.gateway-switcher-item-state {
-  flex: none;
-  color: var(--color-token-description-foreground);
-  font-size: var(--text-xs-plus);
-}
-
-.gateway-switcher-check {
-  flex: none;
-  color: var(--color-token-text-primary);
-}
-
-.gateway-switcher-error {
-  margin: 4px 0 0;
-  padding: 0 10px;
-  color: #9b3d3d;
-  font-size: 11.5px;
-}

docs/agents/desktop-ui.md

@@ -32,6 +32,16 @@
 - Menu shortcut hints use the shared `DropdownMenuShortcut` component, not
   local spans.
 
+## App-Shell Chrome Ownership
+
+- Global shell components must own one complete, always-loaded CSS recipe.
+  Do not scatter required layout declarations across unrelated feature files.
+- The bottom-left gateway identity, settings control, presence glyph, and
+  switcher popover are owned by `styles/gateway-status.css`. A contract test
+  must fail if those selectors escape that file or its root import disappears.
+- Deleting an optional feature must not remove styles required by navigation,
+  the conversation header, the sidebar footer, or other persistent chrome.
+
 ## Responsive Conversation Layout
 
 - Keep the desktop app shell horizontal at every supported window width.