From 671aecd4ff8e21533424dd9265c97e458d641c26 Mon Sep 17 00:00:00 2001 From: Jinwoo-H Date: Mon, 7 Sep 2026 15:08:50 -0400 Subject: [PATCH] test(mobile): retire the shell echo checks the host lane no longer performs The page synthesizes its own workspace handle for activate/update/remove now, so there is no shell echo left to mismatch. Claude-Session: https://claude.ai/code/session_01JNnE9qzUZMMnqpZWCqM3nb --- ...web-operation-response-correlation.test.ts | 26 ------------------- .../bridge-operation-echo-census.test.ts | 9 +++---- .../bridge-operation-registry-census.test.ts | 4 +-- 3 files changed, 5 insertions(+), 34 deletions(-) diff --git a/src/mobile-web/src/mobile-web-operation-response-correlation.test.ts b/src/mobile-web/src/mobile-web-operation-response-correlation.test.ts index 38a6213b6ed..3b28a9d8a1f 100644 --- a/src/mobile-web/src/mobile-web-operation-response-correlation.test.ts +++ b/src/mobile-web/src/mobile-web-operation-response-correlation.test.ts @@ -42,32 +42,6 @@ const CORRELATION_CASES: CorrelationCase[] = [ limit: 2 } }, - { - name: 'workspace activation workspace', - capability: 'workspace', - operation: 'activate', - invoke: (client) => client.workspaceActivate({ workspaceId: WORKSPACE_ID }), - result: { - workspaceId: OTHER_WORKSPACE_ID, - activated: true, - sleepingAgentWake: 'not-applicable' - } - }, - { - name: 'workspace mutation workspace', - capability: 'workspace', - operation: 'update', - invoke: (client) => - client.workspaceUpdate({ mutation: 'pin', workspaceId: WORKSPACE_ID, pinned: true }), - result: { workspaceId: OTHER_WORKSPACE_ID, updated: true } - }, - { - name: 'workspace removal workspace', - capability: 'workspace', - operation: 'remove', - invoke: (client) => client.workspaceRemove({ workspaceId: WORKSPACE_ID }), - result: { workspaceId: OTHER_WORKSPACE_ID, removed: true } - }, { name: 'session snapshot workspace', capability: 'workspace', diff --git a/src/shared/mobile-web/bridge-operation-echo-census.test.ts b/src/shared/mobile-web/bridge-operation-echo-census.test.ts index 85452ec551b..4882005ab38 100644 --- a/src/shared/mobile-web/bridge-operation-echo-census.test.ts +++ b/src/shared/mobile-web/bridge-operation-echo-census.test.ts @@ -67,13 +67,10 @@ const EXPECTED_ECHO_FIELDS: Record = { 'task.loadLinearIssue': ['issue.targetId'], 'task.projectTable': ['project', 'selectedView.id'], 'task.resolveProjectRef': ['host'], - 'workspace.activate': ['workspaceId'], 'workspace.creationSaveSparsePreset': ['directories', 'id', 'name', 'repoId'], 'workspace.creationSparsePresets': ['repoId'], 'workspace.creationSshConnect': ['targetId'], - 'workspace.creationSshState': ['targetId'], - 'workspace.remove': ['workspaceId'], - 'workspace.update': ['workspaceId'] + 'workspace.creationSshState': ['targetId'] } /** Echo helpers shared across request clients, and the result fields each one compares. Resolved @@ -296,7 +293,7 @@ describe('mobile web bridge operation echo census', () => { ) expect(Object.keys(EXPECTED_ECHO_FIELDS).filter((key) => !registered.has(key))).toEqual([]) - expect(Object.keys(EXPECTED_ECHO_FIELDS).length).toBeGreaterThanOrEqual(49) + expect(Object.keys(EXPECTED_ECHO_FIELDS).length).toBeGreaterThanOrEqual(46) }) it('guards the page workspace handle on every workspace-scoped echo it records', () => { @@ -304,6 +301,6 @@ describe('mobile web bridge operation echo census', () => { fields.some((field) => field === 'workspaceId') ) - expect(workspaceScoped.length).toBeGreaterThanOrEqual(40) + expect(workspaceScoped.length).toBeGreaterThanOrEqual(37) }) }) diff --git a/src/shared/mobile-web/bridge-operation-registry-census.test.ts b/src/shared/mobile-web/bridge-operation-registry-census.test.ts index 8fc802d9ec8..9bc81803199 100644 --- a/src/shared/mobile-web/bridge-operation-registry-census.test.ts +++ b/src/shared/mobile-web/bridge-operation-registry-census.test.ts @@ -30,7 +30,7 @@ describe('mobile web bridge operation registry census', () => { } expect(files.length).toBeGreaterThanOrEqual(40) - expect(named.size).toBeGreaterThanOrEqual(126) + expect(named.size).toBeGreaterThanOrEqual(125) expect([...named].filter((pair) => !registered.has(pair))).toEqual([]) }) @@ -61,7 +61,7 @@ describe('mobile web bridge operation registry census', () => { } } - expect(pairs.size).toBeGreaterThanOrEqual(126) + expect(pairs.size).toBeGreaterThanOrEqual(125) expect([...pairs].filter(([, schemas]) => schemas.size !== 1).map(([key]) => key)).toEqual([]) expect([...pairs.keys()].filter((key) => !registered.has(key))).toEqual([]) expect(