starred/orca
1
0
Fork 0
mirror of https://github.com/stablyai/orca.git synced 2026-09-30 00:03:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(windows): teach the mobile RPC double the status probe the lane added

Browse Source 
CI's first-ever run on this lane caught a pre-existing lane defect. The lane
changed status.get to resolve through
runtime.getStatusAfterWindowsProcessStartTimeProbe(), but never taught the
mobile-surface runtime double about it, so status.get failed for mobile
clients with "not a function". The lane's own test list did not include this
file and the lane had zero CI, so nothing ever ran it.

The real runtime always implements the method; the double omitted it.
This commit is contained in:
Merge Sim
2026-09-03 16:27:52 -07:00
parent aec6fbcfbd
commit 42702a680f
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/main/runtime/runtime-rpc-mobile-method-allowlist-fixtures.ts
+6
View File
@@ -8,6 +8,11 @@ export type MobileRpcMock = Mock<(...args: unknown[]) => unknown>
// Full mobile-surface runtime double: every RPC the mobile allowlist may reach.
export function createMobileRpcSurfaceRuntime() {
const getStatus: MobileRpcMock = vi.fn().mockResolvedValue({ graphStatus: 'ok' })
// status.get resolves through the Windows start-time probe, so the double
// must answer it too -- the real runtime always does.
const getStatusAfterWindowsProcessStartTimeProbe: MobileRpcMock = vi
.fn()
.mockResolvedValue({ graphStatus: 'ok' })
const pushRuntimeGit: MobileRpcMock = vi.fn().mockResolvedValue({ ok: true })
const selectClaudeAccount: MobileRpcMock = vi.fn().mockResolvedValue({ ok: true })
const selectCodexAccount: MobileRpcMock = vi.fn().mockResolvedValue({ ok: true })
@@ -119,6 +124,7 @@ export function createMobileRpcSurfaceRuntime() {
const runtime = {
getRuntimeId: () => 'test-runtime',
getStatus,
getStatusAfterWindowsProcessStartTimeProbe,
pushRuntimeGit,
selectClaudeAccount,
selectCodexAccount,