Tasks Fire and Forget

Fire-and-Forget: Task Breakdown

Reference: docs/plan-fire-and-forget-v2.md

Phase 1 — Trivial send() conversions

Task 1.1 — Convert `findMyRobot` from `get()` to `send()`

File: src/roborockCommunication/protocol/dispatcher/V10MessageDispatcher.ts line 103

Change: await this.client.get(duid, request) → await this.client.send(duid, request)

Definition of Done:

findMyRobot() calls client.send() not client.get()
npm run type-check exits 0

Task 1.2 — Update `getDeviceStatus` interface return type to `void`

File: src/roborockCommunication/protocol/dispatcher/abstractMessageDispatcher.ts line 10

Change: Promise<DeviceStatus | undefined> → Promise<void>

Definition of Done:

Interface signature is getDeviceStatus(duid: string): Promise<void>
npm run type-check exits 0 (all implementations must also be updated in same step if TS complains)

Task 1.3 — Convert `V10.getDeviceStatus()` from `get()` to `send()`

File: src/roborockCommunication/protocol/dispatcher/V10MessageDispatcher.ts lines 30-40

Change: Remove response handling, call client.send(), return void

Definition of Done:

Method signature is Promise<void>
Calls client.send() not client.get()
CloudMessageResult import removed if unused
debugStringify import removed if unused
npm run type-check exits 0

Task 1.4 — Convert `Q10.getNetworkInfo()` from `get()` to `send()`

File: src/roborockCommunication/protocol/dispatcher/Q10MessageDispatcher.ts lines 36-40

Change: Replace client.get() + response log with client.send()

Definition of Done:

Calls client.send() not client.get()
Response log line removed
debugStringify import removed if unused
npm run type-check exits 0

Task 1.5 — Convert `Q10.getDeviceStatus()` from `get()` to `send()`

File: src/roborockCommunication/protocol/dispatcher/Q10MessageDispatcher.ts lines 47-50

Change: client.get() → client.send(), return type Promise<void>

Definition of Done:

Method signature is Promise<void>
Calls client.send() not client.get()
npm run type-check exits 0

Task 1.6 — Convert `Q10.getMapInfo()` from `get()` to `send()`

File: src/roborockCommunication/protocol/dispatcher/Q10MessageDispatcher.ts lines 58-67

Change: client.get() → client.send(), remove response/log, return stub

Definition of Done:

Calls client.send() not client.get()
Response variable and log line removed
Returns new MapInfo({ max_multi_map: 0, max_bak_map: 0, multi_map_count: 0, map_info: [] })
npm run type-check exits 0

Task 1.7 — Convert `Q10.getRoomMap()` from `get()` to `send()`

File: src/roborockCommunication/protocol/dispatcher/Q10MessageDispatcher.ts lines 70-73

Change: client.get() → client.send(), return []

Definition of Done:

Calls client.send() not client.get()
Returns []
npm run type-check exits 0

Task 1.8 — Convert `Q7.getMapInfo()` from `get()` to `send()`

File: src/roborockCommunication/protocol/dispatcher/Q7MessageDispatcher.ts lines 82-93

Change: client.get() → client.send(), remove response/log, return stub

Definition of Done:

Calls client.send() not client.get()
Response variable and log line removed
Returns new MapInfo({ max_multi_map: 0, max_bak_map: 0, multi_map_count: 0, map_info: [] })
npm run type-check exits 0

Task 1.9 — Convert `Q7.getRoomMap()` from `get()` to `send()`

File: src/roborockCommunication/protocol/dispatcher/Q7MessageDispatcher.ts lines 95-106

Change: client.get() → client.send(), remove response/log, return []

Definition of Done:

Calls client.send() not client.get()
Response variable and log line removed
Returns []
npm run type-check exits 0

Phase 1 Gate

All tasks 1.1–1.9 complete
npm run type-check exits 0
npm test passes

Phase 2 — Introduce OneShotResponseListener + query()

Task 2.1 — Create `OneShotResponseListener`

New file: src/roborockCommunication/routing/listeners/oneShotResponseListener.ts

Definition of Done:

File exists at the path above
Implements AbstractMessageListener (name, duid, onMessage())
Has waitFor(): Promise<T> that resolves on first parseFn match, rejects on timeout
Has settled flag — onMessage() is no-op after resolving or rejecting
Timeout default is MESSAGE_TIMEOUT_MS from src/constants/index.js
npm run type-check exits 0

Task 2.2 — Add `query<T>()` to `Client` interface

File: src/roborockCommunication/routing/client.ts

Definition of Done:

query<T>(duid, request, parseFn, timeoutMs?) method signature present in interface
ResponseMessage imported from '../models/index.js'
npm run type-check exits 0

Task 2.3 — Add `query<T>()` to `AbstractClient`

File: src/roborockCommunication/routing/abstractClient.ts

Definition of Done:

public async query<T>() method implemented
Creates OneShotResponseListener, registers on responseBroadcaster, calls sendInternal(), awaits waitFor()
Errors caught and logged, returns undefined on failure
OneShotResponseListener imported
npm run type-check exits 0

Task 2.4 — Add `query<T>()` to `ClientRouter`

File: src/roborockCommunication/routing/clientRouter.ts

Definition of Done:

public async query<T>() method implemented
Creates OneShotResponseListener, registers on broadcasterFactory, calls send(), awaits waitFor()
Errors caught and logged, returns undefined on failure
OneShotResponseListener imported
npm run type-check exits 0