libretto 0.6.12 → 0.6.13

package/src/cli/core/daemon/daemon.ts

@@ -63,7 +63,7 @@ import {
 } from "../browser.js";
 import { handlePages } from "./pages.js";
 import { handleExec, handleReadonlyExec } from "./exec.js";
-import { handleCompactSnapshot, handleSnapshot } from "./snapshot.js";
+import { handleCompactSnapshot } from "./snapshot.js";
 import { librettoCommand } from "../../../shared/package-manager.js";
 import type { Snapshot } from "../../../shared/snapshot/types.js";
 import { snapshot } from "../../../shared/snapshot/capture-snapshot.js";
@@ -271,9 +271,7 @@ class BrowserDaemon {
       });
     }
 
-    if (experiments["compact-snapshot-format"]) {
-      await context.addInitScript(installPageStabilityWaiter);
-    }
+    await context.addInitScript(installPageStabilityWaiter);
 
     // IPC server — typed handlers are attached per client connection so one
     // daemon lifetime can serve multiple CLI invocations.
@@ -295,19 +293,15 @@ class BrowserDaemon {
       wrapPageForActionLogging(p, session);
       daemon.trackPage(p);
     }
-    if (experiments["compact-snapshot-format"]) {
-      await Promise.all(
-        initialPages.map((initialPage) =>
-          daemon.installCompactSnapshotWaiter(initialPage),
-        ),
-      );
-    }
+    await Promise.all(
+      initialPages.map((initialPage) =>
+        daemon.installCompactSnapshotWaiter(initialPage),
+      ),
+    );
     context.on("page", (newPage) => {
       wrapPageForActionLogging(newPage, session);
       daemon.trackPage(newPage);
-      if (experiments["compact-snapshot-format"]) {
-        void daemon.installCompactSnapshotWaiter(newPage);
-      }
+      void daemon.installCompactSnapshotWaiter(newPage);
     });
 
     // Navigate after telemetry is installed (so we capture the initial
@@ -659,40 +653,23 @@ class BrowserDaemon {
   private async runSnapshot(
     args: Parameters<CliToDaemonApi["snapshot"]>[0],
   ): Promise<ReturnType<CliToDaemonApi["snapshot"]>> {
-    if (args.mode === "compact") {
-      if (!this.experiments["compact-snapshot-format"]) {
-        throw new Error(
-          `The compact-snapshot-format experiment is not enabled for session "${this.session}". ` +
-            `Close and reopen the session after running ${librettoCommand("experiments enable compact-snapshot-format")}.`,
-        );
-      }
-      const targetPage = this.resolveTargetPage(args.pageId);
-      const result = await this.withRequestTimeout(() =>
-        handleCompactSnapshot(
-          targetPage,
-          this.session,
-          this.logger,
-          {
-            pageId: args.pageId,
-            cachedSnapshot: this.latestCompactSnapshotByPage.get(targetPage),
-            useCachedSnapshot: args.useCachedSnapshot,
-          },
-        ),
-      );
-      if (!args.useCachedSnapshot) {
-        this.latestCompactSnapshotByPage.set(targetPage, result.snapshot);
-      }
-      return result;
-    }
-
-    return this.withRequestTimeout(() =>
-      handleSnapshot(
-        this.resolveTargetPage(args.pageId),
+    const targetPage = this.resolveTargetPage(args.pageId);
+    const result = await this.withRequestTimeout(() =>
+      handleCompactSnapshot(
+        targetPage,
         this.session,
         this.logger,
-        args.pageId,
+        {
+          pageId: args.pageId,
+          cachedSnapshot: this.latestCompactSnapshotByPage.get(targetPage),
+          useCachedSnapshot: args.useCachedSnapshot,
+        },
       ),
     );
+    if (!args.useCachedSnapshot) {
+      this.latestCompactSnapshotByPage.set(targetPage, result.snapshot);
+    }
+    return result;
   }
 
   private async withRequestTimeout<T>(
@@ -720,26 +697,7 @@ class BrowserDaemon {
   private async runExec(
     args: Parameters<CliToDaemonApi["exec"]>[0],
   ): Promise<DaemonExecResult> {
-    if (this.experiments["compact-snapshot-format"]) {
-      return this.runCompactExec(args);
-    }
-
-    try {
-      const data = await this.withRequestTimeout(() =>
-        handleExec(
-          this.resolveTargetPage(args.pageId),
-          args.code,
-          this.context,
-          this.browser,
-          this.execState,
-          this.session,
-          args.visualize,
-        ),
-      );
-      return { ok: true, data };
-    } catch (error) {
-      return this.createExecErrorResult(error);
-    }
+    return this.runCompactExec(args);
   }
 
   private async runCompactExec(
@@ -831,17 +789,17 @@ class BrowserDaemon {
       context: this.context,
       logger: this.logger,
       onLog: (event) => {
-        this.broadcast("workflowOutput", event);
+        void this.broadcast("workflowOutput", event);
       },
       onOutcome: (outcome) => {
         if (outcome.state === "paused") {
-          this.broadcast("workflowPaused", {
+          void this.broadcast("workflowPaused", {
             pausedAt: outcome.pausedAt,
             url: outcome.url,
           });
           return;
         }
-        this.broadcast(
+        void this.broadcast(
           "workflowFinished",
           outcome.result === "completed"
             ? { result: "completed", completedAt: outcome.completedAt }

package/src/cli/core/daemon/ipc.ts

@@ -25,9 +25,10 @@ export type DaemonExecArgs = {
 
 export type DaemonReadonlyExecArgs = { code: string; pageId?: string };
 
-export type DaemonSnapshotArgs =
-  | { mode?: "legacy"; pageId?: string; useCachedSnapshot?: false }
-  | { mode: "compact"; pageId?: string; useCachedSnapshot?: boolean };
+export type DaemonSnapshotArgs = {
+  pageId?: string;
+  useCachedSnapshot?: boolean;
+};
 
 export type DaemonExecSuccess = {
   result: unknown;
@@ -35,24 +36,12 @@ export type DaemonExecSuccess = {
   snapshotDiff?: SnapshotDiff;
 };
 
-export type DaemonLegacySnapshotResult = {
-  pngPath: string;
-  htmlPath: string;
-  snapshotRunId: string;
-  pageUrl: string;
-  title: string;
-};
-
-export type DaemonCompactSnapshotResult = {
+export type DaemonSnapshotResult = {
   mode: "compact";
   pngPath: string;
   snapshot: Snapshot;
 };
 
-export type DaemonSnapshotResult =
-  | DaemonLegacySnapshotResult
-  | DaemonCompactSnapshotResult;
-
 export type DaemonCloseResult = { replayUrl?: string };
 
 export type DaemonCommandResult<T> =

package/src/cli/core/daemon/snapshot.ts

@@ -1,5 +1,5 @@
 import type { Page } from "playwright";
-import { mkdirSync, writeFileSync } from "node:fs";
+import { mkdirSync } from "node:fs";
 import type { LoggerApi } from "../../../shared/logger/index.js";
 import { getSessionSnapshotRunDir } from "../context.js";
 import {
@@ -25,48 +25,6 @@ type SnapshotScreenshot = {
   title: string;
 };
 
-export async function handleSnapshot(
-  targetPage: Page,
-  session: string,
-  logger: LoggerApi,
-  pageId?: string,
-): Promise<{
-  pngPath: string;
-  htmlPath: string;
-  snapshotRunId: string;
-  pageUrl: string;
-  title: string;
-}> {
-  const screenshot = await captureSnapshotScreenshot(
-    targetPage,
-    session,
-    logger,
-    pageId,
-  );
-  const htmlPath = `${getSessionSnapshotRunDir(
-    session,
-    screenshot.snapshotRunId,
-  )}/page.html`;
-
-  // Capture HTML content.
-  const htmlContent = await targetPage.content();
-  writeFileSync(htmlPath, htmlContent);
-
-  logger.info("screenshot-success", {
-    session,
-    pageUrl: screenshot.pageUrl,
-    title: screenshot.title,
-    pngPath: screenshot.pngPath,
-    htmlPath,
-    snapshotRunId: screenshot.snapshotRunId,
-  });
-
-  return {
-    ...screenshot,
-    htmlPath,
-  };
-}
-
 export async function handleCompactSnapshot(
   targetPage: Page,
   session: string,

package/src/cli/core/experiments.ts

@@ -4,45 +4,16 @@ import {
   writeLibrettoConfig,
 } from "./config.js";
 
-export const EXPERIMENTS = {
-  "compact-snapshot-format": {
-    title: "Compact snapshot format",
-    oneSentenceDescription:
-      "Use compact accessibility snapshots and exec page-change diffs without an AI sub-agent.",
-    docs: [
-      "Compact snapshot format changes how agents should use snapshot and exec after the experiment is enabled.",
-      "",
-      "Compared with the skill's documented behavior:",
-      "  - Run libretto snapshot --session <name> without --objective or --context.",
-      "  - Snapshot output is a screenshot path plus a compact accessibility tree; it does not use the PNG + HTML + AI analysis path.",
-      "  - Run libretto snapshot <ref> --session <name> to inspect a subtree from the latest full compact snapshot.",
-      "  - Run libretto exec normally; after successful mutations, Libretto prints page-change diffs from compact snapshots without AI analysis.",
-      "  - If a session was already open before enabling the experiment, close and reopen it before relying on this behavior.",
-      "",
-      "Full compact snapshot:",
-      "  libretto snapshot --session <name>",
-      "",
-      "Cached subtree snapshot:",
-      "  libretto snapshot <ref> --session <name>",
-      "",
-      "Run an unscoped snapshot before using refs. Subtree snapshots capture a fresh screenshot but reuse the latest cached tree.",
-      "",
-      "Notes:",
-      "  - Use ref forms printed in the tree, such as l16. Numeric-suffix aliases such as e16 also match l16.",
-    ].join("\n"),
-    defaultValue: false,
-  },
-} as const satisfies Record<
-  string,
-  {
-    title: string;
-    oneSentenceDescription: string;
-    docs?: string;
-    defaultValue: boolean;
-  }
->;
+export type ExperimentMetadata = {
+  title: string;
+  oneSentenceDescription: string;
+  docs?: string;
+  defaultValue: boolean;
+};
+
+export const EXPERIMENTS: Readonly<Record<string, ExperimentMetadata>> = {};
 
-export type ExperimentName = keyof typeof EXPERIMENTS;
+export type ExperimentName = string;
 export type Experiments = Record<ExperimentName, boolean>;
 
 export function isExperimentName(name: string): name is ExperimentName {

package/src/cli/core/resolve-model.ts

@@ -105,6 +105,7 @@ async function getProviderModel(
       if (!apiKey) {
         throw new Error(missingProviderCredentialsMessage(provider));
       }
+      // @lintc-ignore Human-approved: we don't want to import unless the user is using that subagent.
       const { createGoogleGenerativeAI } = await import("@ai-sdk/google");
       const google = createGoogleGenerativeAI({ apiKey });
       return google(modelId);
@@ -114,6 +115,7 @@ async function getProviderModel(
       if (!project) {
         throw new Error(missingProviderCredentialsMessage(provider));
       }
+      // @lintc-ignore Human-approved: we don't want to import unless the user is using that subagent.
       const { createVertex } = await import("@ai-sdk/google-vertex");
       const vertex = createVertex({
         project,
@@ -126,6 +128,7 @@ async function getProviderModel(
       if (!apiKey) {
         throw new Error(missingProviderCredentialsMessage(provider));
       }
+      // @lintc-ignore Human-approved: we don't want to import unless the user is using that subagent.
       const { createAnthropic } = await import("@ai-sdk/anthropic");
       const anthropic = createAnthropic({ apiKey });
       return anthropic(modelId);
@@ -135,6 +138,7 @@ async function getProviderModel(
       if (!apiKey) {
         throw new Error(missingProviderCredentialsMessage(provider));
       }
+      // @lintc-ignore Human-approved: we don't want to import unless the user is using that subagent.
       const { createOpenAI } = await import("@ai-sdk/openai");
       const openai = createOpenAI({ apiKey });
       return openai(modelId);
@@ -144,6 +148,7 @@ async function getProviderModel(
       if (!apiKey) {
         throw new Error(missingProviderCredentialsMessage(provider));
       }
+      // @lintc-ignore Human-approved: we don't want to import unless the user is using that subagent.
       const { createOpenAI } = await import("@ai-sdk/openai");
       const openrouter = createOpenAI({
         apiKey,

package/src/cli/core/workflow-runtime.ts

@@ -37,6 +37,7 @@ export async function loadDefaultWorkflow(
 ): Promise<ExportedLibrettoWorkflow> {
   let loadedModule: Record<string, unknown>;
   try {
+    // @lintc-ignore Human-approved: user workflow files must be loaded dynamically from the CLI argument.
     loadedModule = (await import(pathToFileURL(absolutePath).href)) as Record<
       string,
       unknown

package/src/cli/index.ts

@@ -1,7 +1,6 @@
 #!/usr/bin/env node
 import { runLibrettoCLI } from "./cli.js";
 
-export { runClose } from "./commands/browser.js";
 export { runLibrettoCLI };
 
 void runLibrettoCLI();

package/src/cli/router.ts

@@ -1,4 +1,3 @@
-import { aiCommands } from "./commands/ai.js";
 import { authCommands } from "./commands/auth.js";
 import { billingCommands } from "./commands/billing.js";
 import { browserCommands } from "./commands/browser.js";
@@ -16,7 +15,6 @@ export const cliRoutes = {
   deploy: deployCommand,
   experiments: experimentsCommand,
   ...executionCommands,
-  ai: aiCommands,
   auth: authCommands,
   billing: billingCommands,
   setup: setupCommand,

package/src/shared/instrumentation/instrument.ts

@@ -131,12 +131,12 @@ function wrapLocatorActions(
       try {
         const result = await orig(...args);
         if (opts.visualize) {
-          enqueue(page, () => visualizeAfterAction(page));
+          void enqueue(page, () => visualizeAfterAction(page));
         }
         return result;
       } catch (err: any) {
         if (opts.visualize) {
-          enqueue(page, () => visualizeAfterAction(page));
+          void enqueue(page, () => visualizeAfterAction(page));
         }
         // Enrich timeout errors for pointer actions
         if (POINTER_ACTIONS.has(method) && isTimeoutError(err)) {
@@ -323,12 +323,12 @@ export async function installInstrumentation(
       try {
         const result = await orig(...args);
         if (visualize) {
-          enqueue(page, () => visualizeAfterAction(page));
+          void enqueue(page, () => visualizeAfterAction(page));
         }
         return result;
       } catch (err: any) {
         if (visualize) {
-          enqueue(page, () => visualizeAfterAction(page));
+          void enqueue(page, () => visualizeAfterAction(page));
         }
         if (
           POINTER_ACTIONS.has(method) &&

package/dist/cli/commands/ai.js

@@ -1,110 +0,0 @@
-import { z } from "zod";
-import {
-  readSnapshotModel,
-  writeSnapshotModel,
-  clearSnapshotModel
-} from "../core/config.js";
-import { LIBRETTO_CONFIG_PATH } from "../core/context.js";
-import { DEFAULT_SNAPSHOT_MODELS } from "../core/ai-model.js";
-import { librettoCommand } from "../../shared/package-manager.js";
-import { SimpleCLI } from "../framework/simple-cli.js";
-const PROVIDER_ALIASES = {
-  claude: DEFAULT_SNAPSHOT_MODELS.anthropic,
-  gemini: DEFAULT_SNAPSHOT_MODELS.google,
-  google: DEFAULT_SNAPSHOT_MODELS.google
-};
-const CONFIGURE_PROVIDERS = [
-  "openai",
-  "anthropic",
-  "gemini",
-  "vertex"
-];
-function formatConfigureProviders(separator = " | ") {
-  return CONFIGURE_PROVIDERS.join(separator);
-}
-function printSnapshotModelConfig(model, configPath) {
-  console.log(`Snapshot model: ${model}`);
-  console.log(`Config file: ${configPath}`);
-}
-function resolveModelFromInput(input) {
-  const trimmed = input.trim();
-  if (!trimmed) return null;
-  if (trimmed.includes("/")) return trimmed;
-  const normalized = trimmed.toLowerCase();
-  return DEFAULT_SNAPSHOT_MODELS[normalized] ?? PROVIDER_ALIASES[normalized] ?? null;
-}
-function runAiConfigure(input, options = {}) {
-  const configureCommandName = options.configureCommandName ?? librettoCommand("ai configure");
-  const configPath = options.configPath ?? LIBRETTO_CONFIG_PATH;
-  const presetArg = input.preset?.trim();
-  if (!presetArg && !input.clear) {
-    const model2 = readSnapshotModel(configPath);
-    if (!model2) {
-      console.log(
-        `No snapshot model set. Choose a default model: ${configureCommandName} ${formatConfigureProviders()}`
-      );
-      console.log(
-        "Provider credentials still come from your shell or .env file."
-      );
-      return;
-    }
-    printSnapshotModelConfig(model2, configPath);
-    return;
-  }
-  if (input.clear) {
-    const removed = clearSnapshotModel(configPath);
-    if (removed) {
-      console.log(`Cleared snapshot model config: ${configPath}`);
-    } else {
-      console.log("No snapshot model was set.");
-    }
-    return;
-  }
-  const model = resolveModelFromInput(presetArg);
-  if (!model) {
-    console.log(
-      `Usage: ${configureCommandName} <${CONFIGURE_PROVIDERS.join("|")}|provider/model-id>
-       ${configureCommandName}
-       ${configureCommandName} --clear`
-    );
-    throw new Error(
-      `Invalid provider or model. Use one of: ${formatConfigureProviders()}, or a full model string like "openai/gpt-4o".`
-    );
-  }
-  writeSnapshotModel(model, configPath);
-  console.log("Snapshot model saved.");
-  printSnapshotModelConfig(model, configPath);
-}
-const aiConfigureInput = SimpleCLI.input({
-  positionals: [
-    SimpleCLI.positional("preset", z.string().optional(), {
-      help: "Provider shorthand or provider/model-id"
-    })
-  ],
-  named: {
-    clear: SimpleCLI.flag({ help: "Clear existing AI config" })
-  }
-});
-const aiCommands = SimpleCLI.group({
-  description: "AI commands",
-  routes: {
-    configure: SimpleCLI.command({
-      description: "Configure AI runtime"
-    }).input(aiConfigureInput).handle(async ({ input }) => {
-      runAiConfigure(
-        {
-          clear: input.clear,
-          preset: input.preset
-        },
-        {
-          configureCommandName: librettoCommand("ai configure")
-        }
-      );
-    })
-  }
-});
-export {
-  aiCommands,
-  aiConfigureInput,
-  runAiConfigure
-};