Skip to content

Commit 8a41f9f

Browse files
committed
fix: treat setup script failure as non-blocking in bootstrap turn start
Setup script failures in dispatchBootstrapTurnStart were re-thrown as RouteRequestError, causing the entire bootstrap to fail and the already-created thread to be deleted (along with the user's message). This was inconsistent with GitManager.preparePullRequestThread, which intentionally swallows the same kind of error via Effect.catch + Effect.logWarning. Remove the Effect.fail re-throw so the error handler only logs the 'setup-script.failed' activity and allows the bootstrap to continue, matching the non-blocking semantics used elsewhere.
1 parent e3372a4 commit 8a41f9f

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

apps/server/src/wsServer.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -733,10 +733,7 @@ export const createServer = Effect.fn(function* (): Effect.fn.Return<
733733
worktreePath: targetWorktreePath,
734734
},
735735
tone: "error",
736-
}).pipe(
737-
Effect.ignoreCause({ log: false }),
738-
Effect.flatMap(() => Effect.fail(toBootstrapRouteRequestError(error))),
739-
),
736+
}).pipe(Effect.ignoreCause({ log: false })),
740737
),
741738
);
742739
})()

0 commit comments

Comments
 (0)