From 45ac13cd7d129e7ad5f00e97c0681682c6c2c620 Mon Sep 17 00:00:00 2001 From: Devin Foley Date: Tue, 25 Aug 2026 02:02:17 -0700 Subject: [PATCH] fix(cli): inert --install-service also skips the start prompt On an interactive terminal the stubbed flag still reached the "Start Paperclip now?" prompt, so a PTY-driven automation run could hang waiting for input. Treat an explicit --install-service as an already decided start at both sites: configure, warn, exit. --- cli/src/commands/onboard.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cli/src/commands/onboard.ts b/cli/src/commands/onboard.ts index 1f15a67c3f0..7593bb57534 100644 --- a/cli/src/commands/onboard.ts +++ b/cli/src/commands/onboard.ts @@ -472,7 +472,7 @@ export async function onboard(opts: OnboardOptions): Promise { // fall through to a blocking foreground start: configure and exit so // scripted installs fail fast instead of hanging. let shouldRunNow = !serviceInstalled && opts.installService !== true && (opts.run === true || opts.yes === true); - if (shouldOfferForegroundStart({ serviceInstalled, startAlreadyDecided: shouldRunNow, invokedByRun: opts.invokedByRun === true, interactive: Boolean(process.stdin.isTTY && process.stdout.isTTY) })) { + if (shouldOfferForegroundStart({ serviceInstalled, startAlreadyDecided: shouldRunNow || opts.installService === true, invokedByRun: opts.invokedByRun === true, interactive: Boolean(process.stdin.isTTY && process.stdout.isTTY) })) { const answer = await p.confirm({ message: "Start Paperclip now?", initialValue: true, @@ -750,7 +750,7 @@ export async function onboard(opts: OnboardOptions): Promise { // fall through to a blocking foreground start: configure and exit so // scripted installs fail fast instead of hanging. let shouldRunNow = !serviceInstalled && opts.installService !== true && (opts.run === true || opts.yes === true); - if (shouldOfferForegroundStart({ serviceInstalled, startAlreadyDecided: shouldRunNow, invokedByRun: opts.invokedByRun === true, interactive: Boolean(process.stdin.isTTY && process.stdout.isTTY) })) { + if (shouldOfferForegroundStart({ serviceInstalled, startAlreadyDecided: shouldRunNow || opts.installService === true, invokedByRun: opts.invokedByRun === true, interactive: Boolean(process.stdin.isTTY && process.stdout.isTTY) })) { const answer = await p.confirm({ message: "Start Paperclip now?", initialValue: true,