@alphacoder-v0/pi-loops 0.24.0 → 0.24.1

package/src/pi-loops.ts

@@ -16,7 +16,7 @@
 import * as path from "node:path";
 import { fileURLToPath } from "node:url";
 import type { ExtensionAPI, ExtensionCommandContext, ExtensionContext, ToolDefinition } from "@earendil-works/pi-coding-agent";
-import { ProjectTrustStore, SessionManager, VERSION as PI_VERSION, getAgentDir, readStoredCredential } from "@earendil-works/pi-coding-agent";
+import { ProjectTrustStore, SessionManager, VERSION as PI_VERSION, copyToClipboard, getAgentDir, readStoredCredential } from "@earendil-works/pi-coding-agent";
 import { ARCHIVE_EXT, type ActivateMode, applyImport, automationOf, defaultExportPath, exportSession, importSession, offerToEnable, parseActivateMode } from "./archive.ts";
 import { Box, Text } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
@@ -38,7 +38,8 @@ import { cardLines, cardOrigin, claimPrompt, dismissOutcome, dismissTail, inboxC
 import { shouldEmitSnapshot, snapshotFingerprint } from "./snapshot.ts";
 import { FRESH_RUNS_READ, burstLine, freshRuns, panelRuns, placeRunCards, runCardDelivery, runCardLines, runCardsHere, runLine } from "./run-card.ts";
 import { type ShareMessage, renderShare, shareSummary, shareableMessages } from "./share.ts";
-import { installLauncherWithConfirm, installedByPi, launcherSource, launcherState, launcherTarget, refreshLauncher, stopAndWait, stopService } from "./cli.ts";
+import { UsageError, addLauncher, installLauncherWithConfirm, installedByPi, launcherAction, launcherSource, launcherState, launcherTarget, openPage, pageLaunchHost, refreshLauncher, slashDir, stopAndWait, stopService, uninstallLauncher, webPageRequest } from "./cli.ts";
+import { copyText, isRemoteTty } from "./browser-open.ts";
 import { execFile } from "node:child_process";
 import { createHash } from "node:crypto";
 import { computeDue, computeNext, formatLocal, formatSchedule, localOffset, parseSchedule, stamp } from "./schedule.ts";
@@ -2464,7 +2465,7 @@ export default function piLoops(pi: ExtensionAPI) {
 	const ARCHIVE_WARNING = `warning: ${ARCHIVE_EXT} archives include transcript and tool history. They do not include separate auth stores, provider credentials, OAuth tokens, MCP config, or the inbox.`;
 
 	pi.registerCommand("pi-loops", {
-		description: "About this extension; start | stop | restart | status the part that runs loops while no pi is open; install-launcher puts the `pi-loops` command on your PATH",
+		description: "About this extension; start | stop | restart | status the part that runs loops while no pi is open; web opens this conversation in the browser page; install-launcher puts the `pi-loops` command on your PATH, uninstall-launcher removes it",
 		handler: async (args, ctx) => {
 			lastCtx = ctx;
 			const sub = args.trim().split(/\s+/)[0] ?? "";
@@ -2491,6 +2492,37 @@ export default function piLoops(pi: ExtensionAPI) {
 				await drainRequests({ tick: true });
 				return;
 			}
+			if (sub === "web") {
+				// The page on this conversation with no `pi-loops` on PATH: the same page-open and opener
+				// the command line uses. Shown as a notice, not written into the conversation: the link
+				// carries a one-use key.
+				const lines: string[] = [];
+				const host = { ...pageLaunchHost(dir), packageDir: PACKAGE_DIR, piPackage: piPackageDir(), startedBy: `the pi window pid ${process.pid}`, myPiVersion: PI_VERSION };
+				const req = webPageRequest({ cwd: session.cwd, sessionId: binding.here().sessionId ?? session.sessionId, sessionFile: ctx.sessionManager.getSessionFile() });
+				// The link is shown and copied before the browser is tried: the wait for a browser to spend
+				// the key can take seconds, and the person may paste the link sooner.
+				const onLink = async (link: string) => {
+					// Said only when a clipboard command took the link. pi's copy can still get there through
+					// OSC 52, which no terminal confirms, so that try says nothing either way.
+					if (await copyText(link)) lines.push("  copied to the clipboard");
+					else await copyToClipboard(link).catch((err: any) => log.info(`/pi-loops web: copy: ${err?.message ?? err}`));
+					ctx.ui.notify(lines.join("\n"), "info");
+				};
+				const opened = await openPage(dir, req, { key: true, browser: isRemoteTty() ? "remote" : "yes", host }, (l) => lines.push(l), undefined, onLink);
+				// pi shows one notice at a time, so the last one repeats the link with the outcome.
+				ctx.ui.notify(lines.join("\n"), opened.code === 0 ? "info" : "warning");
+				refreshBadge();
+				return;
+			}
+			if (sub === "install-launcher" || sub === "uninstall-launcher") {
+				try {
+					slashDir(tokenize(splitCommand(args).rest).map((t) => t.value));
+				} catch (err: any) {
+					if (!(err instanceof UsageError)) throw err;
+					ctx.ui.notify(`${err.message}. usage: /pi-loops ${sub} [--dir <dir>]`, "warning");
+					return;
+				}
+			}
 			if (sub === "install-launcher") {
 				// The command line's own `install-launcher` cannot be reached until it has run once —
 				// `pi install` does not put the package on your PATH. You are already inside pi with
@@ -2505,12 +2537,20 @@ export default function piLoops(pi: ExtensionAPI) {
 				show(ctx, code === 0 ? "installed the pi-loops launcher" : "could not install the launcher", lines.map((l) => `  ${l.trim()}`));
 				return;
 			}
+			if (sub === "uninstall-launcher") {
+				const lines: string[] = [];
+				const code = uninstallLauncher(slashDir(tokenize(splitCommand(args).rest).map((t) => t.value)), (l) => lines.push(l));
+				show(ctx, code === 0 ? "the pi-loops launcher is gone" : "left the launcher alone", lines.map((l) => `  ${l.trim()}`));
+				return;
+			}
 			show(ctx, `pi-loops ${PI_LOOPS_VERSION}`, [
 				`  loops directory: ${homeRel(dir)}`,
 				`  package: ${homeRel(PACKAGE_DIR)}`,
 				"",
 				"  /pi-loops start | stop | restart | status",
+				"  /pi-loops web                              open this conversation in the browser page",
 				"  /pi-loops install-launcher [--dir <dir>]   put the `pi-loops` command on your PATH",
+				"  /pi-loops uninstall-launcher [--dir <dir>] remove it, so no session writes it again",
 				"  /cron · /triggers · /inbox · /goal",
 			]);
 		},
@@ -3160,7 +3200,7 @@ export default function piLoops(pi: ExtensionAPI) {
 	/* -------------------------------------------------------- lifecycle */
 
 	/**
-	 * The `pi-loops` command, kept on the PATH of whoever asked for it there.
+	 * The `pi-loops` command, put on the PATH and kept working there.
 	 *
 	 * `pi install` puts this package under pi's directory and nothing on the PATH, so until 0.22.1
 	 * the first thing a new user had to do was type a bare `node …/cli-entry.mjs install-launcher`
@@ -3169,40 +3209,47 @@ export default function piLoops(pi: ExtensionAPI) {
 	 * it runs and where a launcher would go, so the step belongs here.
 	 *
 	 * This is the extension's only write outside the loops directory (AGENTS.md). Its limit is one
-	 * file, `pi-loops` in a directory already on the PATH, and only where a launcher of ours already
-	 * is or after a yes. A file with no marker of ours is somebody's own wrapper and is left alone;
-	 * one naming another copy of this package that is still on disk is that copy's to keep current,
-	 * so an install and a checkout both being opened never trade the file back and forth; a no is
-	 * remembered in `ui.json`, so it is asked once rather than in every window; and a checkout is
-	 * never asked for a launcher that is not there at all — nobody ran `pi install`, so nothing is
-	 * missing from their PATH — though one written from a checkout is still kept working.
+	 * file, `pi-loops` in a directory already on the PATH. With nothing there, the first interactive
+	 * session of a copy `pi install` put in place writes it with no question and says so in one line,
+	 * as pi's own installer does for `pi` (#180); after that only a launcher of ours is rewritten. A
+	 * file that does not open with our header is somebody's own wrapper and is left alone; one naming another
+	 * copy of this package that is still on disk is that copy's to keep current, so an install and a
+	 * checkout both being opened never trade the file back and forth; `uninstall-launcher` is
+	 * remembered in `ui.json`, so no later session writes it again; a write that failed is
+	 * remembered with its directory, so it is not tried in every window; and a checkout never writes
+	 * a launcher that is not there at all — nobody ran `pi install`, so nothing is missing from their
+	 * PATH — though one written from a checkout is still kept working. The decision is
+	 * `launcherAction` in src/cli.ts.
 	 */
 	async function keepLauncher(ctx: ExtensionContext): Promise<void> {
 		const { entryDir, node, agentDir } = launcherSource();
 		const target = launcherTarget(undefined);
-		// Nothing on the PATH to write into. `/pi-loops install-launcher`, run by hand, is where that
-		// is explained; a session start saying it unprompted would be a warning about nothing.
-		if (!target) return;
-		const { state } = launcherState(entryDir, node, agentDir, target);
-		if (state === "stale") {
+		const prefs = readUiPrefs(dir);
+		const state = target ? launcherState(entryDir, node, agentDir, target).state : undefined;
+		const action = launcherAction({ target, state, hasUI: ctx.hasUI, declined: prefs.launcher === "declined", failedIn: prefs.launcher === "failed" ? prefs.launcherFailedIn : undefined, installedByPi: state === "absent" && installedByPi() });
+		if (!target || action === "none") return;
+		if (action === "refresh") {
 			const done = refreshLauncher(entryDir, node, agentDir, target);
 			if (!done) return;
 			log.info(`launcher: ${done.message}`);
-			// No question: they asked for a launcher once, and keeping that one working is what they
-			// asked for. The line is so that a file changing under them is never a silent change.
+			// They have a launcher of ours, and keeping that one working is what it is for. The line is
+			// so that a file changing under them is never a silent change.
 			if (ctx.hasUI) ctx.ui.notify(`[pi-loops] ${done.message}`, done.ok ? "info" : "warning");
 			return;
 		}
-		if (state !== "absent") return; // ours and current, another copy's, or somebody else's entirely
-		if (!ctx.hasUI || readUiPrefs(dir).launcher === "declined" || !installedByPi()) return;
-		const lines: string[] = [];
-		const code = await installLauncherWithConfirm([], (title, body) => ctx.ui.confirm(title, body), (l) => lines.push(l));
-		if (code === undefined) {
-			writeUiPref(dir, "launcher", "declined");
-			ctx.ui.notify("[pi-loops] not installed — `/pi-loops install-launcher` writes one whenever you want it", "info");
+		const added = addLauncher(entryDir, node, agentDir, target);
+		if (added.kind === "none") return;
+		if (added.kind === "added") {
+			// A failed write into another directory is not news any more.
+			clearUiPref(dir, "launcher");
+			clearUiPref(dir, "launcherFailedIn");
+			log.info(`launcher: wrote ${added.file}`);
+			ctx.ui.notify(`[pi-loops] added pi-loops to ${homeRel(target)}; /pi-loops uninstall-launcher removes it`, "info");
 			return;
 		}
-		ctx.ui.notify(`[pi-loops] ${lines.map((l) => l.trim()).join(" · ")}`, code === 0 ? "info" : "warning");
+		writeUiPref(dir, "launcher", "failed");
+		writeUiPref(dir, "launcherFailedIn", target);
+		ctx.ui.notify(`[pi-loops] ${added.message}; /pi-loops install-launcher --dir <dir> writes it elsewhere`, "warning");
 	}
 
 	/**
@@ -3277,8 +3324,7 @@ export default function piLoops(pi: ExtensionAPI) {
 			loadMcpConfig(windowTrusted(getAgentDir(), ctx.cwd, ctx.isProjectTrusted(), APPROVED));
 			for (const d of mcpDiagnostics) if (ctx.hasUI) ctx.ui.notify(`[pi-loops] ${d}`, "warning");
 			await startMcpSources(); // tools for every process; pushes are consumed by interactive processes only
-			// Not awaited: the question below is a dialog, and the clock must not wait behind someone who
-			// has walked away from it.
+			// A launcher that cannot be written or kept is logged, and never stops the session starting.
 			void keepLauncher(ctx).catch((err: any) => log.info(`launcher: ${err?.message ?? err}`));
 			// tui and rpc processes stay alive and run the clock; `PI_LOOPS_HOST=1` starts it in a `pi -p`
 			// run too, which queues its conversation's due prompts but hands none to pi (the window

package/src/ui-prefs.ts

@@ -25,10 +25,13 @@ export interface UiPrefs {
 	model?: string;
 	thinking?: string;
 	/**
-	 * `"declined"` once someone has said no to the launcher a session start offers, so the offer is
-	 * made once and not in every window. `/pi-loops install-launcher` clears it when it writes one.
+	 * `"declined"` once someone has removed the launcher with `uninstall-launcher`, so no session
+	 * start writes it again; `"failed"` once a session start could not write it. `/pi-loops
+	 * install-launcher` clears it when it writes one.
 	 */
 	launcher?: string;
+	/** The directory a session start could not write the launcher into: tried again only elsewhere. */
+	launcherFailedIn?: string;
 	/** `"stopped"` once a person stopped pi-loops by hand: no window starts it again until `pi-loops start`. */
 	host?: string;
 	/** `"shown"` once a session start has said that closed conversations' plain jobs now run in the host. */
@@ -65,6 +68,7 @@ export function readUiPrefs(dir: string): UiPrefs {
 	if (typeof doc.model === "string") prefs.model = doc.model;
 	if (typeof doc.thinking === "string") prefs.thinking = doc.thinking;
 	if (typeof doc.launcher === "string") prefs.launcher = doc.launcher;
+	if (typeof doc.launcherFailedIn === "string") prefs.launcherFailedIn = doc.launcherFailedIn;
 	if (typeof doc.host === "string") prefs.host = doc.host;
 	if (typeof doc.heldJobsNotice === "string") prefs.heldJobsNotice = doc.heldJobsNotice;
 	if (typeof doc.toolsCapNotice === "string") prefs.toolsCapNotice = doc.toolsCapNotice;

package/src/web.mjs

@@ -11,7 +11,8 @@ export const DOOR = `<!doctype html><meta charset="utf-8"><title>pi-loops</title
 <meta name="color-scheme" content="light dark">
 <body style="font:15px/1.6 system-ui,sans-serif;max-width:34rem;margin:12vh auto;padding:0 1.5rem;background:Canvas;color:CanvasText">
 <h2 style="font-weight:600">This browser has not been here before.</h2>
-<p>Run <code>pi-loops</code> in a terminal on this machine, or type the code from <b>add device</b> on a browser that is signed in:</p>
+<p>Run <code>pi-loops open</code> in a terminal on this machine for a link that signs this browser in.</p>
+<p>Pairing another device? <code>pi-loops pair</code> shows a code.</p>
 <form method="get" style="display:flex;gap:.5rem">
   <input name="pair" inputmode="numeric" pattern="[0-9]*" autocomplete="one-time-code" maxlength="6"
          placeholder="000000" style="font:inherit;letter-spacing:.3em;padding:.55rem .7rem;flex:1;min-width:0;border:1px solid #8886;border-radius:6px;background:transparent;color:inherit">