[codex] redesign Windows Desktop client

.gitignore

@@ -6,8 +6,6 @@ config.json
 CMakeUserPresets.json
 .merak/
 .claude/
-/docs/
-.superpowers/
 
 # Dependencies
 node_modules/
@@ -34,5 +32,9 @@ Thumbs.db
 # Build artifacts
 dist/
 
+# Documentation (generated)
+/docs/
+.superpowers/
+
 # Worktrees
 .worktrees/

.superpowers/sdd/task-7-report.md

@@ -0,0 +1,113 @@
+# Task 7 Report: Sessions Workbench
+
+## Status
+
+Implemented the lazy `sessions` route as a three-column sessions workbench using the existing session history, conversation, composer, SSE, and inspector components.
+
+## RED
+
+1. Baseline command:
+   `npm test -- --run src/__tests__/UserFlow.test.tsx src/__tests__/components.test.tsx`
+2. Baseline result: 48 passed, 2 failed.
+3. Preserved baseline failures:
+   - `World time control` / `Current: Day 4, dusk` did not match the approved rendered labels `世界时间` / `当前：Day 4, dusk`.
+   - The removed `Foreshadowing` sub-tab did not exist; `Create` is the approved accessible top-level tab.
+4. New page test RED:
+   `npm test -- --run src/__tests__/UserFlow.test.tsx`
+5. New page test result: failed because `../pages/SessionsPage` did not exist.
+
+## GREEN
+
+1. Focused command:
+   `npm test -- --run src/__tests__/UserFlow.test.tsx src/__tests__/components.test.tsx`
+2. Result: 51 passed, 0 failed across 2 test files.
+3. Build command: `npm run build`
+4. Result: TypeScript and Vite production build passed; Vite emitted a pre-existing large main-chunk advisory.
+5. Lint command: `npm run lint`
+6. Result: 0 errors and 12 pre-existing unused-type warnings in `src/api/client.ts`.
+
+## Preserved Workflows
+
+- `SessionList` remains the sole owner of create, select, rename, generated-title, archive, and restore behavior.
+- `ChatTimeline` and `Composer` remain mounted once through `MainPanel`, preserving messages, send, cancel, approvals, SSE status, and tool/run rendering.
+- `InspectorPanel` remains the sole owner of Story, Files, Agents, Create, and Run content, preserving generated files, editor entry points, creation dialogs, run replay, and audit behavior.
+- `App` still owns SSE setup and keeps help, setup, chapter review, ask, creation request, and export overlays outside the page route.
+- Editor routing and close protection remain unchanged.
+
+## Structure
+
+- Left: current-world session history using real `state.sessions` data.
+- Center: selected session title, connection status, chat timeline, and composer.
+- Right: existing inspector with approved `Story`, `Files`, `Agents`, `Create`, and `Run` labels.
+- Responsive behavior retains the history and inspector toggles without duplicating stateful workbench components.
+
+## Self-review
+
+- Confirmed `SessionsPage` is emitted as a separate lazy build chunk.
+- Confirmed the legacy workbench remains available for routes not migrated in this task.
+- Confirmed overlays remain siblings of the lazy page and were not moved or duplicated.
+- Confirmed no fixture content was added to production UI.
+- No task-specific lint errors remain.
+
+## Review Fixes
+
+### RED
+
+Command:
+`npm test -- --run src/__tests__/UserFlow.test.tsx src/__tests__/components.test.tsx src/__tests__/AppState.test.ts`
+
+The new regressions demonstrated that:
+
+- `SET_SESSION` retained the previous agent when the selected session had no `agent_id`.
+- The sessions page exposed no `aria-hidden` state and desktop/tablet toggles did not control panel visibility.
+- A typed composer prompt could enable the send button without a selected session.
+- The App phase policy did not admit `sessions` during `no_agent`.
+
+### GREEN
+
+Focused command:
+`npm test -- --run src/__tests__/UserFlow.test.tsx src/__tests__/components.test.tsx src/__tests__/AppState.test.ts`
+
+Result: 98 passed, 0 failed across 3 test files.
+
+Build command: `npm run build`
+
+Result: TypeScript and Vite production build passed. The existing main-chunk size advisory remains; `SessionsPage` remains a separate lazy chunk.
+
+Lint command: `npm run lint`
+
+Result: 0 errors and the same 12 pre-existing unused-type warnings in `src/api/client.ts`.
+
+### Preserved constraints
+
+- `no_world` still renders onboarding before any sessions route can mount.
+- `no_agent` permits the sessions route so a user can select or create a session.
+- Selecting or creating a session atomically sets its agent, clears a missing agent, and moves the app to ready.
+- The composer send action stays disabled until both prompt text and a session ID exist; its handler guard remains intact.
+- Desktop and tablet toggles now collapse grid columns, remove hidden panels from interaction, and expose matching `aria-hidden` / `aria-expanded` state.
+
+## Final Review Fixes
+
+### RED
+
+Command:
+`npm test -- --run src/__tests__/components.test.tsx src/__tests__/UserFlow.test.tsx`
+
+Result: 2 new failures, confirming that the base sessions grid required a 420px center track and legacy `MainPanel` sidebar mode referenced the sessions-only `session-history-panel` ID.
+
+### GREEN
+
+Command:
+`npm test -- --run src/__tests__/components.test.tsx src/__tests__/UserFlow.test.tsx`
+
+Result: 57 passed, 0 failed across 2 test files.
+
+Build command: `npm run build`
+
+Result: TypeScript and Vite production build passed. The existing main-chunk size advisory remains.
+
+### Preserved constraints
+
+- The base three-column grid now uses `minmax(0, 1fr)`, so a 1180-1293px viewport can retain the inspector without exceeding the shell content width.
+- The existing `1179px` inspector-overlay breakpoint remains, preserving the 960px minimum-window layout.
+- Sessions mode retains `aria-controls="session-history-panel"`; legacy sidebar mode emits no reference to that sessions-only ID.