From d10c875d88fe4c8dacbc308fcc138f5cdbd660aa Mon Sep 17 00:00:00 2001 From: nrobi144 Date: Thu, 18 Jun 2026 11:14:28 +0300 Subject: [PATCH] docs(desktop): refresh launch-opt plan pending list with App() blocker MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Phase 1.4 / 2.4 / the four Phase 5.2 regression tests are all blocked on the same broader App() dependency-injection refactor — relayManager, localCache, localRelayStore, and subscriptionsCoordinator are still constructed inside App() via remember { … }. The torManager slot has been loosened to ITorManager in preparation, but the rest is wider work than this session can absorb. --- ...06-17-feat-app-launch-optimization-plan.md | 20 ++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/desktopApp/plans/2026-06-17-feat-app-launch-optimization-plan.md b/desktopApp/plans/2026-06-17-feat-app-launch-optimization-plan.md index 3f67e6726b..be7215cf7d 100644 --- a/desktopApp/plans/2026-06-17-feat-app-launch-optimization-plan.md +++ b/desktopApp/plans/2026-06-17-feat-app-launch-optimization-plan.md @@ -26,11 +26,16 @@ deepened: 2026-06-17 **Still pending after this session:** -- **Phase 1.4** — `App()` Compose UI smoke test. Hard-blocked on mocking the - six App() dependencies that have substantial real implementations - (DeckState, WorkspaceManager, DesktopTorManager, TorType / port flows, - initialTorSettings). The `BenchmarkRelayConnectionManager` subclass added - in 3.2 is the seam the future App() harness will use. +- **Phase 1.4** — `App()` Compose UI smoke test. Blocked on broader App() + dependency injection: `accountManager`, `deckState`, `workspaceManager` + and `torManager` are already ctor parameters (the `torManager` slot was + loosened to `ITorManager` in commit `d2d044a63` for this purpose), but + `relayManager`, `localCache`, `localRelayStore` and + `subscriptionsCoordinator` are still constructed inside App() via + `remember { … }` and so cannot be swapped for the in-process / fixture + versions added in Phase 2 without a wider refactor. The + `BenchmarkRelayConnectionManager` subclass added in 3.2 is the seam the + future App() harness will use once those dependencies become injectable. - **Phase 2.4** — wire fixture relay into `AppStateMachineTest` (depends on Phase 1.4). - **Cold-fork shell driver** for the benchmark (per-sample JVM fork). The @@ -43,6 +48,11 @@ deepened: 2026-06-17 does not drive the App() gate, so the gate fix is validated by the `SubscribeBeforeConnectTest` invariant rather than by a benchmark delta in this report. +- The four Phase 5.2 regression tests + (`slowRelayDoesNotStallFeed`, `noRelaysAvailableShowsErrorState`, + `relayListArrivesLateStartsSubscriptionThen`, + `relaysAddedMidLoadDoNotDoubleSubscribe`) — each requires driving the + App() composable, so blocked on the Phase 1.4 harness above. ## Enhancement Summary