Files
PaperClipAI/cli/src/onboard-service.ts
T
f987f95d6a fix(cli): guard service install against a missing managed shim (#13985)
## Thinking Path

> - Paperclip is the open source app people use to manage AI agents for
work
> - The `paperclipai` CLI can run Paperclip as a background service
(launchd on macOS, a systemd user unit on Linux), and that service
definition always launches the managed-install shim
> - #12148 made onboarding create the managed install first
(`ensureServiceShim()`), or refuse with a repair path, when the shim
does not exist
> - The standalone `paperclipai service install` command never got that
guard: it calls `manager.install()` directly, reports `installed: true`,
and leaves the supervisor crash-looping on a missing binary (for example
after a global npm install)
> - This pull request runs the same onboarding guard in `service
install` before any service definition is written
> - The benefit is that `service install` either produces a service that
can start, or fails with the same repair instructions onboarding gives,
and never reports success for a service that cannot run

## Linked Issues or Issue Description

- Fixes: #13841
- Refs #12148 (the onboarding-path fix this extends), #11755 and #11620
(related global-npm / service issues, not addressed here)

## What Changed

- `cli/src/onboard-service.ts`: moved the existing onboarding shim
guard, unchanged, into an exported `ensureServiceShim()` so both entry
points share it. It takes an optional `installIfMissing` flag that
defaults to `true`, so onboarding behaves exactly as before. When only
the shim is missing (a managed install is recorded and its current
payload is still present), the guard now runs the installer's existing
`smokePayload()` check (the payload must run and report its recorded
version), then rewrites the shim in place with the existing
`writeManagedShim()` instead of reinstalling. A payload that fails the
check gets no new shim; the command stops with the repair hint. The
recorded version and update channel stay unchanged, with no download and
no payload pruning. Otherwise the existing onboarding fallback
(reinstall the recorded git revision, or install this CLI's release
version) is unchanged.
- `cli/src/commands/service.ts`: `service install` now calls
`ensureServiceShim()` after the service manager is resolved and before
`manager.install()`. If the shim cannot be made available, the command
throws `Background service not installed: <reason>. Run \`paperclipai
install\`, then \`paperclipai service install\`.` (exit code 1) and
writes no service definition.
- With `--json`, the command only checks for the shim and does not start
a managed install. `installCommand()` prints progress to stdout, which
would corrupt the machine-readable result. JSON callers get a clean
non-zero exit with the repair hint; interactive callers get the same
behavior as onboarding (create the managed install, then the service).
The JSON output shape and all existing flags are unchanged.
- Docs: no change needed. `doc/INSTALLING.md` already says the service
uses the managed-install shim; this makes the command enforce that.
- Tests: a new command-level test (`service-install-command.test.ts`)
and one `ensureServiceShim` case in `onboard-service.test.ts`.

## Verification

- `pnpm exec vitest run
cli/src/__tests__/service-install-command.test.ts` on unmodified
`master` (efce9356): 3/3 tests fail (the guard is never called, and the
missing-shim command resolves instead of rejecting). With this change:
pass.
- `pnpm exec vitest run
cli/src/__tests__/service-install-command.test.ts
cli/src/__tests__/onboard-service.test.ts
cli/src/__tests__/service-health-check.test.ts`: 36/36 pass.
- The tests cover: the guard runs before `manager.install` (call order);
`--no-start-now` / `--no-start-on-login` are still forwarded;
`installed: true` JSON is printed only on success; a failed guard
rejects with the repair message, never calls `manager.install`, and
prints nothing; `--json` uses check-only mode; check-only mode never
calls `installCommand`; with a managed payload present, the repair
smoke-tests the payload, then restores the shim in place without calling
the installer, and rechecks the executable; a payload that fails the
smoke test gets no shim and no reinstall.
- `pnpm --filter paperclipai typecheck` and `pnpm --filter paperclipai
build`: pass.
- Repo-wide `pnpm -r typecheck` and `pnpm build` stop at the unchanged
`paperclip-runner` package because `cargo` is not installed locally.
They fail the same way on unmodified `master`, so CI is the authority
there.
- `pnpm test:run` locally: 32 server failures. 30 reproduce on
unmodified `master` (AI-connection/auth and worktree-provisioning
cases). The other 2 pass when run alone on both `master` and this
branch. None are in `cli/`.
- Manual check for reviewers: with no `~/.local/bin/paperclipai`,
`paperclipai service install --json` now exits 1 with the repair hint
and writes no LaunchAgent/unit. Without `--json`, it creates the managed
install first, then the service.

## Risks

- Low risk. Behavior changes only when the managed shim is missing,
which is exactly the case that used to produce a crash-looping service.
- Behavior shift: in that case, `service install --json` now fails
instead of reporting a service that cannot start. Scripts that relied on
the old false success will see exit code 1 and the repair instructions.
- Onboarding is unchanged. The guard body moved into a named function,
and onboarding calls it with the same defaults.

> For core feature work, check [`ROADMAP.md`](ROADMAP.md) first and
discuss it in `#dev` before opening the PR. Feature PRs that overlap
with planned core work may need to be redirected — check the roadmap
first. See `CONTRIBUTING.md`.

## Model Used

- Implementation and tests: OpenAI Codex, model `gpt-6-astra` (agentic
coding with tool use and code execution; context window not reported by
the runtime).
- Plan critique before implementation: OpenAI `gpt-5.6-sol` at high
reasoning effort (read-only).
- Coordination, scoping and PR description: Anthropic Claude Opus 5.5
(`claude-opus-5-5`, 1M context, tool use).
- Independent code review of the exact head: Anthropic `claude-sonnet-5`
at medium effort (read-only), no findings.
- A human contributor directed the work and owns the change.

## Checklist

- [x] I have included a thinking path that traces from project context
to this change
- [x] I have specified the model used (with version and capability
details)
- [x] I have checked ROADMAP.md and confirmed this PR does not duplicate
planned core work
- [x] I have searched GitHub for duplicate or related PRs and linked
them above
- [x] I have either (a) linked existing issues with `Fixes: #` / `Closes
#` / `Refs #` OR (b) described the issue in-PR following the relevant
issue template
- [x] I have not referenced internal/instance-local Paperclip issues or
links (only public GitHub `#NNN` / `github.com/paperclipai/paperclip`
URLs)
- [x] My branch name describes the change (e.g. `docs/...`, `fix/...`)
and contains no internal Paperclip ticket id or instance-derived details
- [x] I have run tests locally and they pass
- [x] I have added or updated tests where applicable
- [x] I have updated relevant documentation to reflect my changes
- [x] I have considered and documented any risks above
- [ ] All Paperclip CI gates are green
- [ ] Greptile is 5/5 with no open P2s, recommendations, or follow-ups
- [x] I will address all Greptile and reviewer comments before
requesting merge

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-authored-by: devinfoley <139239+devinfoley@users.noreply.github.com>
2026-10-13 17:20:36 -07:00

292 lines
11 KiB
TypeScript

import fs from "node:fs";
import path from "node:path";
import * as p from "@clack/prompts";
import pc from "picocolors";
import type { PaperclipConfig } from "./config/schema.js";
import { openUrl } from "./client/board-auth.js";
import { installCommand, runCommandWithDiagnostics, smokePayload } from "./commands/install.js";
import { resolvePaperclipInstanceId } from "./config/home.js";
import { readRuntimeInfo, type PaperclipRuntimeInfo } from "./runtime-info.js";
import {
readInstallManifest,
resolveInstallStorePaths,
writeManagedShim,
type InstallManifest,
} from "./install-store.js";
import {
detectServiceManager,
isExecutableFile,
resolveServiceShimPath,
type ServiceManagerDetection,
} from "./services/service-manager.js";
import { buildLocalAppUrl, buildLocalHealthUrl } from "./utils/health-url.js";
import { packageVersion } from "./version.js";
export type OnboardServiceOptions = {
yes?: boolean;
installService?: boolean;
};
type EnsureShimResult = { ok: boolean; installedNow: boolean; reason?: string };
type OnboardServiceDashboardConfig = {
auth: Pick<PaperclipConfig["auth"], "baseUrlMode" | "publicBaseUrl">;
server: Pick<PaperclipConfig["server"], "host" | "port">;
};
type OnboardServiceDashboardDependencies = {
isInteractive: () => boolean;
waitUntilReady: () => Promise<PaperclipRuntimeInfo | null>;
openDashboard: (url: string) => Promise<boolean>;
info: (message: string) => void;
success: (message: string) => void;
warn: (message: string) => void;
};
function envDisablesBrowser(value = process.env.PAPERCLIP_NO_BROWSER): boolean {
const normalized = value?.trim().toLowerCase();
return normalized === "1" || normalized === "true" || normalized === "yes";
}
async function waitUntilDashboardReady(timeoutMs = 60_000): Promise<PaperclipRuntimeInfo | null> {
const instanceId = resolvePaperclipInstanceId();
const detection = await detectServiceManager({ instanceId });
if (!detection.supported) return null;
const deadline = Date.now() + timeoutMs;
while (Date.now() < deadline) {
const info = readRuntimeInfo(instanceId);
if (info) {
const status = await detection.manager.status().catch(() => null);
if (status?.active && status.pid === info.pid) {
try {
const response = await fetch(buildLocalHealthUrl(info.host, info.port), {
signal: AbortSignal.timeout(2_000),
});
const body = await response.json() as { status?: unknown };
if (response.ok && body.status === "ok") return info;
} catch {}
}
}
await new Promise((resolve) => setTimeout(resolve, 500));
}
return null;
}
const defaultDashboardDependencies: OnboardServiceDashboardDependencies = {
isInteractive: () => process.stdin.isTTY === true && process.stdout.isTTY === true,
waitUntilReady: waitUntilDashboardReady,
openDashboard: openUrl,
info: (message) => p.log.info(message),
success: (message) => p.log.success(message),
warn: (message) => p.log.warn(message),
};
export function resolveOnboardServiceDashboardUrl(
config: OnboardServiceDashboardConfig,
runtime?: Pick<PaperclipRuntimeInfo, "dashboardUrl"> | null,
): string {
if (runtime?.dashboardUrl.trim()) return runtime.dashboardUrl.trim().replace(/\/+$/, "");
if (config.auth.baseUrlMode === "explicit" && config.auth.publicBaseUrl?.trim()) {
return config.auth.publicBaseUrl.trim().replace(/\/+$/, "");
}
return buildLocalAppUrl(config.server.host, config.server.port);
}
export async function handoffToOnboardedService(
config: OnboardServiceDashboardConfig,
dependencies: Partial<OnboardServiceDashboardDependencies> = {},
): Promise<void> {
const deps = { ...defaultDashboardDependencies, ...dependencies };
const runtime = await deps.waitUntilReady();
const dashboardUrl = resolveOnboardServiceDashboardUrl(config, runtime);
deps.info(`Paperclip dashboard: ${pc.cyan(dashboardUrl)}`);
if (!runtime) {
deps.warn(
`The background service started, but the dashboard is not ready yet. ` +
`Open ${dashboardUrl} after checking \`paperclipai service logs\`.`,
);
return;
}
if (!deps.isInteractive() || envDisablesBrowser()) return;
if (await deps.openDashboard(dashboardUrl)) {
deps.success("Sent the Paperclip dashboard to your browser.");
} else {
deps.warn(`Could not open a browser automatically. Open ${dashboardUrl} manually.`);
}
}
// Source checkouts carry the repository placeholder version; installing
// that as an npm spec would fetch an ancient release (or nothing) instead
// of the running code. Only real calendar versions are installable.
export function isInstallableReleaseVersion(version: string): boolean {
return /^\d{4}\.\d{1,4}\.\d+(?:-[0-9A-Za-z.-]+)?$/.test(version);
}
type OnboardServiceDependencies = {
detect: (instanceId: string) => Promise<ServiceManagerDetection>;
ensureServiceShim: () => Promise<EnsureShimResult>;
confirm: () => Promise<boolean>;
confirmLinger: () => Promise<boolean>;
isInteractive: () => boolean;
info: (message: string) => void;
success: (message: string) => void;
warn: (message: string) => void;
};
// The service definition targets the managed shim. An ephemeral run (npx)
// never lays it down, so installing the service without this step creates
// a definition that crash-loops on a missing binary.
export async function ensureServiceShim(options: { installIfMissing?: boolean } = {}): Promise<EnsureShimResult> {
const shimPath = resolveServiceShimPath();
if (await isExecutableFile(shimPath)) {
return { ok: true, installedNow: false };
}
if (options.installIfMissing === false) {
return { ok: false, installedNow: false, reason: `no executable exists at ${shimPath}` };
}
const storeShimPath = resolveInstallStorePaths().shimPath;
if (path.resolve(shimPath) !== path.resolve(storeShimPath)) {
return {
ok: false,
installedNow: false,
reason: `no executable exists at ${shimPath} (PAPERCLIP_SHIM_PATH), and it is outside the managed install store`,
};
}
let manifest: InstallManifest | null = null;
try {
manifest = readInstallManifest();
} catch {}
try {
if (
manifest &&
fs.existsSync(path.join(resolveInstallStorePaths().currentPath, "node_modules", "paperclipai", "dist", "index.js"))
) {
// Only the shim is missing: smoke-test and restore it for the recorded payload,
// keeping its version and update channel instead of reinstalling.
await smokePayload(resolveInstallStorePaths().currentPath, manifest.version, runCommandWithDiagnostics);
writeManagedShim();
} else if (manifest?.source === "git" && manifest.repo) {
// A managed git payload must be preserved as-is: reinstall the
// exact revision the manifest records, not an npm release.
await installCommand({ repo: manifest.repo, ref: manifest.sha ?? manifest.ref, yes: true });
} else if (isInstallableReleaseVersion(packageVersion)) {
// packageVersion, not cliVersion: a managed executable's cliVersion
// carries provenance text that is not an installable npm spec.
await installCommand({ version: packageVersion, yes: true });
} else {
return {
ok: false,
installedNow: false,
reason:
`this build reports version ${packageVersion}, which is not an installable release; ` +
"run `paperclipai install` (or `paperclipai install --repo <repo> --ref <ref>` for source builds) first",
};
}
} catch (error) {
return {
ok: false,
installedNow: false,
reason: error instanceof Error ? error.message : String(error),
};
}
if (await isExecutableFile(shimPath)) {
return { ok: true, installedNow: true };
}
return {
ok: false,
installedNow: false,
reason: `the managed install completed but no executable shim appeared at ${shimPath}`,
};
}
const defaultDependencies: OnboardServiceDependencies = {
detect: (instanceId) => detectServiceManager({ instanceId }),
ensureServiceShim: () => ensureServiceShim(),
confirm: async () => {
const answer = await p.confirm({
message: "Install Paperclip as a background service?",
initialValue: true,
});
return !p.isCancel(answer) && answer === true;
},
confirmLinger: async () => {
const answer = await p.confirm({
message: "Allow Paperclip to keep running after logout? This may request system authorization.",
initialValue: false,
});
return !p.isCancel(answer) && answer === true;
},
isInteractive: () => process.stdin.isTTY === true && process.stdout.isTTY === true,
info: (message) => p.log.message(pc.dim(message)),
success: (message) => p.log.success(message),
warn: (message) => p.log.warn(message),
};
export async function handleOnboardService(
options: OnboardServiceOptions,
dependencies: Partial<OnboardServiceDependencies> = {},
): Promise<boolean> {
const deps = { ...defaultDependencies, ...dependencies };
if (options.installService === false) return false;
const explicitlyRequested = options.installService === true;
const canPrompt = options.yes !== true && deps.isInteractive();
if (!explicitlyRequested && !canPrompt) {
deps.info(
"Background service not installed. Use `paperclipai onboard --install-service` or `paperclipai service install` to opt in.",
);
return false;
}
const instanceId = resolvePaperclipInstanceId();
const detection = await deps.detect(instanceId);
if (!detection.supported) {
if (explicitlyRequested) deps.warn(detection.reason);
return false;
}
if (!explicitlyRequested && !(await deps.confirm())) return false;
// A definition pointing at a missing binary crash-loops in the platform
// supervisor's penalty box while doctor blames a port conflict.
// Materialize the managed install first, or decline with the repair path
// instead of installing a corpse.
const shim = await deps.ensureServiceShim();
if (!shim.ok) {
deps.warn(
`Background service not installed: ${shim.reason ?? "the managed install could not be completed"}. ` +
"Run `paperclipai install`, then `paperclipai service install`.",
);
return false;
}
if (shim.installedNow) {
deps.success("Installed the managed paperclipai payload and command shim for the service.");
}
await detection.manager.install({ startNow: true, startOnLogin: true });
if (!explicitlyRequested && detection.manager.enableLinger && await deps.confirmLinger()) {
await detection.manager.enableLinger();
}
deps.success(`Installed and started ${detection.manager.serviceName}.`);
return true;
}
// Onboarding falls back to offering a foreground start when nothing else
// will serve. A just-installed service is already serving, so offering the
// start would only run the user into the already-running instance guard.
export function shouldOfferForegroundStart(options: {
serviceInstalled: boolean;
startAlreadyDecided: boolean;
invokedByRun: boolean;
interactive: boolean;
}): boolean {
return (
!options.startAlreadyDecided &&
!options.serviceInstalled &&
!options.invokedByRun &&
options.interactive
);
}