libretto 0.6.12 → 0.6.14

package/src/cli/commands/snapshot.ts

@@ -1,22 +1,12 @@
-import { readFileSync, writeFileSync } from "node:fs";
 import { z } from "zod";
 import type { LoggerApi } from "../../shared/logger/index.js";
-import { condenseDom } from "../../shared/condense-dom/condense-dom.js";
 import { readSessionState } from "../core/session.js";
-import {
-  type InterpretArgs,
-  type ScreenshotPair,
-} from "../core/snapshot-analyzer.js";
-import { SimpleCLI } from "../framework/simple-cli.js";
+import { SimpleCLI } from "affordance";
 import {
   pageOption,
   sessionOption,
-  withExperiments,
   withRequiredSession,
 } from "./shared.js";
-import { runApiInterpret } from "../core/api-snapshot-analyzer.js";
-import { readSnapshotModel } from "../core/config.js";
-import { resolveSnapshotApiModelOrThrow } from "../core/ai-model.js";
 import { DaemonClient } from "../core/daemon/ipc.js";
 import { librettoCommand } from "../../shared/package-manager.js";
 import { renderSnapshot } from "../../shared/snapshot/render-snapshot.js";
@@ -116,101 +106,6 @@ export async function forceSnapshotViewport(
   });
 }
 
-async function captureSnapshot(
-  session: string,
-  logger: LoggerApi,
-  daemonSocketPath: string,
-  pageId?: string,
-): Promise<ScreenshotPair> {
-  logger.info("snapshot-via-daemon", { session, pageId });
-  const client = await DaemonClient.connect(daemonSocketPath);
-  let snapshotResult: Awaited<ReturnType<DaemonClient["snapshot"]>>;
-  try {
-    snapshotResult = await client.snapshot({ pageId });
-  } finally {
-    client.destroy();
-  }
-  if (!("htmlPath" in snapshotResult)) {
-    throw new Error("Daemon returned a compact snapshot for a legacy request.");
-  }
-  const { pngPath, htmlPath, snapshotRunId, pageUrl, title } = snapshotResult;
-
-  // condenseDom runs in the CLI process, not the daemon.
-  const htmlContent = readFileSync(htmlPath, "utf8");
-  const condenseResult = condenseDom(htmlContent);
-  const condensedHtmlPath = htmlPath.replace(/\.html$/, ".condensed.html");
-  writeFileSync(condensedHtmlPath, condenseResult.html);
-
-  logger.info("snapshot-daemon-success", {
-    session,
-    pageUrl,
-    title,
-    pngPath,
-    htmlPath,
-    condensedHtmlPath,
-    snapshotRunId,
-    domCondenseStats: {
-      originalLength: condenseResult.originalLength,
-      condensedLength: condenseResult.condensedLength,
-      reductions: condenseResult.reductions,
-    },
-  });
-
-  return { pngPath, htmlPath, condensedHtmlPath, baseName: snapshotRunId };
-}
-
-async function runSnapshot(
-  session: string,
-  logger: LoggerApi,
-  pageId: string | undefined,
-  objective: string | undefined,
-  context: string | undefined,
-): Promise<void> {
-  if (objective === undefined) {
-    throw new Error("Missing required option --objective.");
-  }
-  if (context === undefined) {
-    throw new Error("Missing required option --context.");
-  }
-
-  const normalizedObjective = objective.trim();
-  const normalizedContext = context.trim();
-
-  const snapshotModel = readSnapshotModel();
-  resolveSnapshotApiModelOrThrow(snapshotModel);
-
-  const state = readSessionState(session, logger);
-  if (!state?.daemonSocketPath) {
-    throw new Error(
-      `Session "${session}" has no daemon socket. The browser daemon may have crashed. ` +
-        `Close and reopen the session: ${librettoCommand(`close --session ${session}`)}`,
-    );
-  }
-
-  const { pngPath, htmlPath, condensedHtmlPath } =
-    await captureSnapshot(session, logger, state.daemonSocketPath, pageId);
-
-  console.log("Screenshot saved:");
-  console.log(`  PNG:             ${pngPath}`);
-  console.log(`  HTML:            ${htmlPath}`);
-  console.log(`  Condensed HTML:  ${condensedHtmlPath}`);
-
-  const interpretArgs: InterpretArgs = {
-    objective: normalizedObjective,
-    session,
-    context: normalizedContext,
-    pngPath,
-    htmlPath,
-    condensedHtmlPath,
-  };
-
-  // Analysis uses direct API calls via the Vercel AI SDK (see api-snapshot-analyzer.ts).
-  // The legacy CLI-agent path (spawning codex/claude/gemini as a subprocess) is preserved
-  // in snapshot-analyzer.ts — to switch back, replace this call with:
-  //   await runInterpret(interpretArgs, logger);
-  await runApiInterpret(interpretArgs, logger, snapshotModel);
-}
-
 async function runCompactSnapshot(
   args: {
     session: string;
@@ -237,17 +132,12 @@ async function runCompactSnapshot(
   let result: Awaited<ReturnType<DaemonClient["snapshot"]>>;
   try {
     result = await client.snapshot({
-      mode: "compact",
       pageId: args.pageId,
       useCachedSnapshot: args.ref !== undefined,
     });
   } finally {
     client.destroy();
   }
-  if (!("mode" in result) || result.mode !== "compact") {
-    throw new Error("Daemon returned a legacy snapshot for a compact request.");
-  }
-
   console.log(`Screenshot at ${result.pngPath}`);
   console.log(renderSnapshot(result.snapshot, args.ref));
   console.log(
@@ -270,34 +160,16 @@ export const snapshotInput = SimpleCLI.input({
 });
 
 export const snapshotCommand = SimpleCLI.command({
-  description: "Capture PNG + HTML and analyze with --objective and --context",
+  description: "Capture a screenshot and compact accessibility snapshot",
 })
   .input(snapshotInput)
   .use(withRequiredSession())
-  .use(withExperiments())
   .handle(async ({ input, ctx }) => {
-    if (ctx.experiments["compact-snapshot-format"]) {
-      await runCompactSnapshot({
-        session: ctx.session,
-        daemonSocketPath: ctx.sessionState.daemonSocketPath,
-        logger: ctx.logger,
-        pageId: input.page,
-        ref: input.ref,
-      });
-      return;
-    }
-
-    if (input.ref) {
-      throw new Error(
-        `Snapshot refs require the compact-snapshot-format experiment. Enable it with ${librettoCommand("experiments enable compact-snapshot-format")}.`,
-      );
-    }
-
-    await runSnapshot(
-      ctx.session,
-      ctx.logger,
-      input.page,
-      input.objective,
-      input.context,
-    );
+    await runCompactSnapshot({
+      session: ctx.session,
+      daemonSocketPath: ctx.sessionState.daemonSocketPath,
+      logger: ctx.logger,
+      pageId: input.page,
+      ref: input.ref,
+    });
   });

package/src/cli/commands/status.ts

@@ -1,50 +1,5 @@
-import { LIBRETTO_CONFIG_PATH } from "../core/context.js";
-import { type AiSetupStatus, resolveAiSetupStatus } from "../core/ai-model.js";
-import { librettoCommand } from "../../shared/package-manager.js";
 import { listRunningSessions, type SessionState } from "../core/session.js";
-import { SimpleCLI } from "../framework/simple-cli.js";
-
-// ── AI status printing ──────────────────────────────────────────────────────
-
-function printAiStatus(status: AiSetupStatus): void {
-  console.log("AI configuration:");
-
-  switch (status.kind) {
-    case "ready":
-      console.log(`  ✓ Snapshot model: ${status.model}`);
-      if (status.source === "config") {
-        console.log(`  Config: ${LIBRETTO_CONFIG_PATH}`);
-      } else {
-        console.log(`  Source: ${status.source}`);
-      }
-      console.log(
-        `  To change: ${librettoCommand("ai configure openai | anthropic | gemini | vertex | openrouter")}`,
-      );
-      break;
-
-    case "configured-missing-credentials":
-      console.log(
-        `  ✗ ${status.provider} is configured (model: ${status.model}), but credentials are missing.`,
-      );
-      console.log(`  Run \`${librettoCommand("setup")}\` to repair.`);
-      break;
-
-    case "invalid-config":
-      console.log("  ✗ Config is invalid:");
-      for (const line of status.message.split("\n")) {
-        console.log(`    ${line}`);
-      }
-      console.log(`  Run \`${librettoCommand("setup")}\` to reconfigure.`);
-      break;
-
-    case "unconfigured":
-      console.log("  ✗ No AI model configured.");
-      console.log(
-        `  Run \`${librettoCommand("setup")}\` or \`${librettoCommand("ai configure")}\` to set up.`,
-      );
-      break;
-  }
-}
+import { SimpleCLI } from "affordance";
 
 // ── Session status printing ─────────────────────────────────────────────────
 
@@ -68,13 +23,10 @@ function printOpenSessions(sessions: SessionState[]): void {
 // ── Command ─────────────────────────────────────────────────────────────────
 
 export const statusCommand = SimpleCLI.command({
-  description: "Show workspace status: AI configuration and open sessions",
+  description: "Show workspace status and open sessions",
 })
   .input(SimpleCLI.input({ positionals: [], named: {} }))
   .handle(async () => {
-    const aiStatus = resolveAiSetupStatus();
-    printAiStatus(aiStatus);
-
     const sessions = listRunningSessions();
     printOpenSessions(sessions);
   });

package/src/cli/core/auth-fetch.ts

@@ -11,7 +11,11 @@
 
 import { readAuthState, writeAuthState, type AuthState } from "./auth-storage.js";
 
-export const HOSTED_API_URL = "https://api.libretto.sh";
+export const DEFAULT_HOSTED_API_URL = "https://api.libretto.sh";
+
+export function resolveHostedApiUrl(): string {
+  return process.env.LIBRETTO_API_URL?.trim() || DEFAULT_HOSTED_API_URL;
+}
 
 /**
  * Shared "you have no usable credential" message. Pointed at the two
@@ -19,8 +23,9 @@ export const HOSTED_API_URL = "https://api.libretto.sh";
  */
 export const NOT_AUTHENTICATED_MESSAGE = [
   "Not authenticated.",
-  "  • Cookie expired or never set: run `libretto experimental auth login` to refresh it.",
-  "  • Or set LIBRETTO_API_KEY in your .env (issue one with `libretto experimental auth api-key issue --label <label>` after logging in).",
+  "  • New account: run `libretto cloud auth signup`.",
+  "  • Existing account: run `libretto cloud auth login`.",
+  "  • Automation: set LIBRETTO_API_KEY in your env (issue one with `libretto cloud auth api-key issue --label <label>` after signing in).",
 ].join("\n");
 
 export type CredentialSource = "env-api-key" | "cookie" | "none";
@@ -41,7 +46,7 @@ export function pickCredential(state: AuthState | null): CredentialChoice {
 }
 
 export function resolveApiUrl(_state: AuthState | null): string {
-  return HOSTED_API_URL;
+  return resolveHostedApiUrl();
 }
 
 type FetchOptions = {

package/src/cli/core/browser.ts

@@ -6,15 +6,20 @@ import {
   type Page,
 } from "playwright";
 import { existsSync, readFileSync, writeFileSync, unlinkSync } from "node:fs";
+import { mkdir, writeFile } from "node:fs/promises";
 import { dirname, join } from "node:path";
 import { createServer } from "node:net";
+import { isWindowsNamedPipePath } from "../../shared/ipc/socket-transport.js";
 import type { LoggerApi } from "../../shared/logger/index.js";
 import type { SessionAccessMode } from "../../shared/state/index.js";
 import type { Experiments } from "./experiments.js";
 import { getSessionProviderClosePath, PROFILES_DIR } from "./context.js";
 import { readLibrettoConfig } from "./config.js";
 import { librettoCommand } from "../../shared/package-manager.js";
-import { getCloudProviderApi } from "./providers/index.js";
+import {
+  getCloudProviderApi,
+  getProviderStartupTimeoutMs,
+} from "./providers/index.js";
 import {
   assertSessionAvailableForStart,
   clearSessionState,
@@ -88,7 +93,8 @@ export function normalizeUrl(url: string): URL {
   if (
     parsedUrl.protocol === "http:" ||
     parsedUrl.protocol === "https:" ||
-    parsedUrl.protocol === "file:"
+    parsedUrl.protocol === "file:" ||
+    parsedUrl.href === "about:blank"
   ) {
     return parsedUrl;
   }
@@ -98,7 +104,7 @@ export function normalizeUrl(url: string): URL {
   }
 
   throw new Error(
-    `Unsupported URL protocol: ${parsedUrl.protocol}. Use http://, https://, or file://.`,
+    `Unsupported URL protocol: ${parsedUrl.protocol}. Use http://, https://, file://, or about:blank.`,
   );
 }
 
@@ -547,8 +553,8 @@ export async function runOpenWithProvider(
     },
     logger,
     logPath: runLogPath,
-    // Remote CDP connection + navigation; must cover both.
-    startupTimeoutMs: 60_000,
+    // Remote provider creation can wait for cloud capacity before CDP exists.
+    startupTimeoutMs: getProviderStartupTimeoutMs(providerName),
   });
   client.destroy();
 
@@ -657,9 +663,8 @@ export async function runSave(
     }
 
     const state = { cookies, origins };
-    const fs = await import("node:fs/promises");
-    await fs.mkdir(dirname(profilePath), { recursive: true });
-    await fs.writeFile(profilePath, JSON.stringify(state, null, 2));
+    await mkdir(dirname(profilePath), { recursive: true });
+    await writeFile(profilePath, JSON.stringify(state, null, 2));
 
     logger.info("save-success", {
       domain,
@@ -974,6 +979,8 @@ function unlinkDaemonSocket(
   session: string,
 ): void {
   if (!socketPath) return;
+  if (isWindowsNamedPipePath(socketPath)) return;
+
   try {
     unlinkSync(socketPath);
   } catch (err) {

package/src/cli/core/config.ts

@@ -3,7 +3,6 @@ import { dirname } from "node:path";
 import { z } from "zod";
 import { SessionAccessModeSchema } from "../../shared/state/index.js";
 import { LIBRETTO_CONFIG_PATH } from "./context.js";
-import { librettoCommand } from "../../shared/package-manager.js";
 
 export const CURRENT_CONFIG_VERSION = 1;
 
@@ -42,7 +41,6 @@ function formatExpectedConfigExample(): string {
   return JSON.stringify(
     {
       version: CURRENT_CONFIG_VERSION,
-      snapshotModel: "openai/gpt-5.4",
       viewport: {
         width: 1280,
         height: 800,
@@ -66,10 +64,9 @@ function invalidConfigError(configPath: string, detail?: string): Error {
       "Expected config example:",
       formatExpectedConfigExample(),
       "Notes:",
-      '  - "snapshotModel", "viewport", "windowPosition", and "sessionMode" are optional.',
-      '  - "snapshotModel" must be a provider/model string like "openai/gpt-5.4" or "anthropic/claude-sonnet-4-6".',
-      "Fix the file to match this shape, or delete it and rerun:",
-      `  ${librettoCommand("ai configure openai | anthropic | gemini | vertex | openrouter")}`,
+      '  - "viewport", "windowPosition", and "sessionMode" are optional.',
+      '  - "snapshotModel" is deprecated and ignored by snapshot.',
+      "Fix the file to match this shape, or delete it and rerun setup.",
     ]
       .filter(Boolean)
       .join("\n"),