(MOT-4260) feat(memory,console): ship the memory page as injectable console UI

console/web/src/App.tsx

@@ -30,7 +30,6 @@ import { Browser } from '@/pages/Browser'
 import { Configuration } from '@/pages/Configuration'
 import { ExtPage } from '@/pages/Ext'
 import { Github } from '@/pages/Github'
-import { Memory } from '@/pages/Memory'
 import { TracesV2 } from '@/pages/TracesV2'
 import { Workers } from '@/pages/Workers'
 import { Worktrees } from '@/pages/Worktrees'
@@ -114,8 +113,6 @@ export function App() {
               <Worktrees />
             ) : view === 'browser' ? (
               <Browser />
-            ) : view === 'memory' ? (
-              <Memory />
             ) : view === 'github' ? (
               <Github />
             ) : view === 'ext' ? (
@@ -158,23 +155,17 @@ function Header({
   // Optional-worker entries appear only while their worker is present; a
   // direct #/worktrees or #/browser hit still lands on that page's install
   // notice.
-  const {
-    worktreeAvailable,
-    browserAvailable,
-    memoryAvailable,
-    githubAvailable,
-  } = useConversationsCtx()
+  // `memoryAvailable` stays on the context for the chat composer's bank
+  // selector; the memory page is now injected UI (nav entry via `extPages`),
+  // so it is no longer a first-party nav option here.
+  const { worktreeAvailable, browserAvailable, githubAvailable } =
+    useConversationsCtx()
   // Injected pages: the runtime analogue of worker-presence gating —
   // presence is the script being loaded, which already tracks worker
   // connectedness via trigger GC.
   const extPages = useExtPages()
   const viewOptions: { value: string; label: string }[] = [
-    ...buildViewOptions(
-      worktreeAvailable,
-      browserAvailable,
-      memoryAvailable,
-      githubAvailable,
-    ),
+    ...buildViewOptions(worktreeAvailable, browserAvailable, githubAvailable),
     ...extPages.map((page) => ({
       value: extNavValue(page),
       label: page.title,

console/web/src/hooks/use-hash-route.ts

@@ -14,7 +14,6 @@ export type View =
   | 'workers'
   | 'worktrees'
   | 'browser'
-  | 'memory'
   | 'github'
   | 'ext'
 
@@ -99,9 +98,6 @@ function routeFromHash(hash: string): View | null {
   if (hash === '#/worktrees') {
     return 'worktrees'
   }
-  if (hash === '#/memory') {
-    return 'memory'
-  }
   if (hash === '#/github') {
     return 'github'
   }
@@ -139,8 +135,6 @@ function hashFor(view: View): string {
       return '#/worktrees'
     case 'browser':
       return '#/browser'
-    case 'memory':
-      return '#/memory'
     case 'github':
       return '#/github'
     case 'configuration':

console/web/src/lib/nav-options.test.ts

@@ -3,38 +3,43 @@ import { buildViewOptions } from './nav-options'
 
 describe('buildViewOptions', () => {
   it('hides the optional-worker entries while their workers are absent', () => {
-    expect(
-      buildViewOptions(false, false, false, false).map((o) => o.value),
-    ).toEqual(['traces', 'workers'])
+    expect(buildViewOptions(false, false, false).map((o) => o.value)).toEqual([
+      'traces',
+      'workers',
+    ])
   })
 
   it('appends the worktrees entry when the worker is present', () => {
-    expect(
-      buildViewOptions(true, false, false, false).map((o) => o.value),
-    ).toEqual(['traces', 'workers', 'worktrees'])
+    expect(buildViewOptions(true, false, false).map((o) => o.value)).toEqual([
+      'traces',
+      'workers',
+      'worktrees',
+    ])
   })
 
   it('appends the browser entry when the worker is present', () => {
-    expect(
-      buildViewOptions(false, true, false, false).map((o) => o.value),
-    ).toEqual(['traces', 'workers', 'browser'])
-  })
-
-  it('appends the memory entry when the worker is present', () => {
-    expect(
-      buildViewOptions(false, false, true, false).map((o) => o.value),
-    ).toEqual(['traces', 'workers', 'memory'])
+    expect(buildViewOptions(false, true, false).map((o) => o.value)).toEqual([
+      'traces',
+      'workers',
+      'browser',
+    ])
   })
 
   it('appends the github entry when the worker is present', () => {
-    expect(
-      buildViewOptions(false, false, false, true).map((o) => o.value),
-    ).toEqual(['traces', 'workers', 'github'])
+    expect(buildViewOptions(false, false, true).map((o) => o.value)).toEqual([
+      'traces',
+      'workers',
+      'github',
+    ])
   })
 
   it('appends every entry when all optional workers are present', () => {
-    expect(
-      buildViewOptions(true, true, true, true).map((o) => o.value),
-    ).toEqual(['traces', 'workers', 'worktrees', 'browser', 'memory', 'github'])
+    expect(buildViewOptions(true, true, true).map((o) => o.value)).toEqual([
+      'traces',
+      'workers',
+      'worktrees',
+      'browser',
+      'github',
+    ])
   })
 })

console/web/src/lib/nav-options.ts

@@ -9,7 +9,6 @@ import type { View } from '@/hooks/use-hash-route'
 export function buildViewOptions(
   worktreeAvailable: boolean,
   browserAvailable: boolean,
-  memoryAvailable: boolean,
   githubAvailable: boolean,
 ): { value: View; label: string }[] {
   const options: { value: View; label: string }[] = [
@@ -22,9 +21,6 @@ export function buildViewOptions(
   if (browserAvailable) {
     options.push({ value: 'browser', label: 'browser' })
   }
-  if (memoryAvailable) {
-    options.push({ value: 'memory', label: 'memory' })
-  }
   if (githubAvailable) {
     options.push({ value: 'github', label: 'github' })
   }