[GH-ISSUE #145] fix(application): avoid partial client setup by handling later clients after early failures #91

Closed
opened 2026-10-15 18:21:03 +00:00 by bsakaguchi · 0 comments
Owner

Originally created by @morluto on GitHub (Jul 14, 2026).
Original GitHub issue: https://github.com/morluto/rea/issues/145

Description

configureDetectedClients returns immediately on the first client setup failure and does not attempt to configure remaining detected clients.

Impact

Setup becomes partial-and-ambiguous: some clients may be configured while later clients are skipped without a clear recovery strategy.

Reproduction

  1. Provide configuration where one middle-detected client fails during setup.
  2. Observe function exits early before later clients are processed.

Expected behavior

Either continue and return per-client outcomes (with optional rollback policy) or enforce explicit all-or-nothing semantics.

Files

  • src/application/SetupClients.ts (around lines 26-37)
Originally created by @morluto on GitHub (Jul 14, 2026). Original GitHub issue: https://github.com/morluto/rea/issues/145 ## Description `configureDetectedClients` returns immediately on the first client setup failure and does not attempt to configure remaining detected clients. ## Impact Setup becomes partial-and-ambiguous: some clients may be configured while later clients are skipped without a clear recovery strategy. ## Reproduction 1. Provide configuration where one middle-detected client fails during setup. 2. Observe function exits early before later clients are processed. ## Expected behavior Either continue and return per-client outcomes (with optional rollback policy) or enforce explicit all-or-nothing semantics. ## Files - `src/application/SetupClients.ts` (around lines 26-37)
bsakaguchi 2026-10-15 18:21:03 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
morluto/rea#91
No description provided.