@linxin666/dsh-remote-web-ui 0.1.12 → 0.1.13

package/src/mobile/views/WorkspaceView.test.tsx

@@ -0,0 +1,66 @@
+// @vitest-environment jsdom
+/** Mobile workspace landing: roster rendering and QR deep-link selection. */
+import { afterEach, describe, expect, it, vi } from 'vitest'
+import { cleanup, render, screen, waitFor } from '@testing-library/react'
+import type { WorkspaceView as WorkspaceRow } from '@deepseek-ai/dsh-host-apiproxy/api/workspace'
+import { mobileWorkspaceTarget } from './App.tsx'
+import { WorkspaceView } from './WorkspaceView.tsx'
+
+vi.mock('../api.ts', () => ({
+  listWorkspaces: vi.fn(),
+}))
+import { listWorkspaces } from '../api.ts'
+
+const listWorkspacesMock = vi.mocked(listWorkspaces)
+const workspaces: WorkspaceRow[] = [
+  {
+    workspaceId: 'ws-1' as never,
+    path: '/tmp/first',
+    title: 'First',
+    sessionIds: [] as never,
+    createdAt: '2026-01-01T00:00:00.000Z',
+    updatedAt: '2026-01-01T00:00:00.000Z',
+  },
+  {
+    workspaceId: 'ws-2' as never,
+    path: '/tmp/second',
+    title: 'Second',
+    sessionIds: [] as never,
+    createdAt: '2026-01-01T00:00:00.000Z',
+    updatedAt: '2026-01-01T00:00:00.000Z',
+  },
+]
+
+afterEach(() => {
+  cleanup()
+  vi.clearAllMocks()
+})
+
+describe('mobile workspace deep link', () => {
+  it('reads a non-empty workspace target from the query', () => {
+    expect(mobileWorkspaceTarget('?workspace=ws-2')).toBe('ws-2')
+    expect(mobileWorkspaceTarget('?workspace=')).toBeUndefined()
+    expect(mobileWorkspaceTarget('')).toBeUndefined()
+  })
+
+  it('opens the targeted workspace as soon as the roster loads', async () => {
+    listWorkspacesMock.mockResolvedValue(workspaces)
+    const onPick = vi.fn()
+
+    render(<WorkspaceView initialWorkspaceId="ws-2" onPick={onPick} />)
+
+    await waitFor(() => expect(onPick).toHaveBeenCalledWith(workspaces[1]))
+    expect(onPick).toHaveBeenCalledTimes(1)
+  })
+
+  it('falls back to the roster when the target no longer exists', async () => {
+    listWorkspacesMock.mockResolvedValue(workspaces)
+    const onPick = vi.fn()
+
+    render(<WorkspaceView initialWorkspaceId="missing" onPick={onPick} />)
+
+    expect(await screen.findByText('First')).toBeTruthy()
+    expect(await screen.findByText('Second')).toBeTruthy()
+    expect(onPick).not.toHaveBeenCalled()
+  })
+})

package/src/mobile/views/WorkspaceView.tsx

@@ -13,6 +13,8 @@ import { ThemeToggle } from '../theme-toggle.tsx'
 
 /** Props for the workspace roster. */
 export interface WorkspaceViewProps {
+  /** Workspace carried by the pairing link; opened after the roster loads. */
+  initialWorkspaceId?: string
   /** Open one workspace's session list. */
   onPick(workspace: WorkspaceRow): void
 }
@@ -22,7 +24,7 @@ export interface WorkspaceViewProps {
  * @param props - the pick action.
  * @returns the roster.
  */
-export function WorkspaceView({ onPick }: WorkspaceViewProps) {
+export function WorkspaceView({ initialWorkspaceId, onPick }: WorkspaceViewProps) {
   const [items, setItems] = useState<WorkspaceRow[] | undefined>(undefined)
   const [error, setError] = useState<string | undefined>(undefined)
 
@@ -31,6 +33,13 @@ export function WorkspaceView({ onPick }: WorkspaceViewProps) {
     void listWorkspaces().then(
       (rows) => {
         if (cancelled) return
+        const target = initialWorkspaceId === undefined
+          ? undefined
+          : rows.find(workspace => workspace.workspaceId === initialWorkspaceId)
+        if (target !== undefined) {
+          onPick(target)
+          return
+        }
         setItems(rows)
       },
       (reason: unknown) => {
@@ -39,7 +48,7 @@ export function WorkspaceView({ onPick }: WorkspaceViewProps) {
       },
     )
     return () => { cancelled = true }
-  }, [])
+  }, [initialWorkspaceId, onPick])
 
   if (error !== undefined) {
     return (

package/src/mobile-api.ts

@@ -40,6 +40,13 @@ const MOBILE_ALLOWLIST = new Set([
   'session.rename',
 ])
 
+/**
+ * Locally answered display-preference method (the phone's read-only
+ * surface preferences; never proxied to the host ApiProxy and never a
+ * settings-domain write).
+ */
+const MOBILE_PREFERENCES_METHOD = 'mobile.preferences'
+
 /** One session.list page (thin phones load incrementally). */
 const SESSION_PAGE_SIZE = 20
 
@@ -69,6 +76,8 @@ export interface MobileApiDeps {
   service: PairingService
   /** The host ApiProxy service (injected by the plugin). */
   apiProxy: ApiProxy
+  /** The resolved mobile composer preference (live per request). */
+  mobileEnterToSend: () => boolean
 }
 
 /** Mobile API route paths. */
@@ -87,7 +96,7 @@ const MOBILE_API_METHOD_PREFIX = `${MOBILE_API_PREFIX}/`
  * @returns the routes to register on webServer.
  */
 export function makeMobileApiRoutes(deps: MobileApiDeps): WebRoute[] {
-  const { service, apiProxy } = deps
+  const { service, apiProxy, mobileEnterToSend } = deps
 
   /** The phone gate: a live paired-device cookie, or nothing else proceeds. */
   const gateOk = (req: IncomingMessage): boolean => {
@@ -116,7 +125,8 @@ export function makeMobileApiRoutes(deps: MobileApiDeps): WebRoute[] {
       return
     }
     const method = pathname.slice(MOBILE_API_METHOD_PREFIX.length)
-    if (!MOBILE_ALLOWLIST.has(method)) {
+    const local = method === MOBILE_PREFERENCES_METHOD
+    if (!MOBILE_ALLOWLIST.has(method) && !local) {
       writeJson(res, 403, { ok: false, error: { code: 'forbidden', message: `method ${method} is not exposed to the mobile surface` } })
       return
     }
@@ -133,6 +143,14 @@ export function makeMobileApiRoutes(deps: MobileApiDeps): WebRoute[] {
       writeJson(res, 400, { ok: false, error: { code: 'bad-request', message: 'missing rpcId' } })
       return
     }
+    if (local) {
+      writeJson(res, 200, {
+        type: 'server-response',
+        rpcId,
+        result: { ok: true, value: { mobileEnterToSend: mobileEnterToSend() } },
+      })
+      return
+    }
     try {
       const response = await dispatch(apiProxy, method, parsed?.payload, rpcId)
       writeJson(res, 200, response)