libretto 0.5.0 → 0.5.2

package/src/cli/core/session.ts

@@ -23,9 +23,17 @@ import {
 
 const SESSION_NAME_PATTERN = /^[a-zA-Z0-9._-]+$/;
 
-export const SESSION_DEFAULT = "default";
 export const SESSION_DEV_SERVER = "dev-server";
 export const SESSION_BROWSER_AGENT = "browser-agent";
+
+export function generateSessionName(): string {
+  const chars = "abcdefghijklmnopqrstuvwxyz0123456789";
+  let id = "";
+  for (let i = 0; i < 4; i++) {
+    id += chars[Math.floor(Math.random() * chars.length)];
+  }
+  return `ses-${id}`;
+}
 export { SESSION_STATE_VERSION };
 export type { SessionStatus, SessionState };
 
@@ -136,7 +144,10 @@ export function readSessionStateOrThrow(session: string): SessionState {
   }
 
   try {
-    return parseSessionStateContent(readFileSync(stateFile, "utf-8"), stateFile);
+    return parseSessionStateContent(
+      readFileSync(stateFile, "utf-8"),
+      stateFile,
+    );
   } catch (err) {
     throw new Error(
       `Could not read session state for "${session}": ${err instanceof Error ? err.message : String(err)}`,
@@ -186,10 +197,13 @@ export function setSessionStatus(
   const state = readSessionState(session, logger);
   if (!state) return;
   if (state.status === status) return;
-  writeSessionState({
-    ...state,
-    status,
-  }, logger);
+  writeSessionState(
+    {
+      ...state,
+      status,
+    },
+    logger,
+  );
 }
 
 export function assertSessionAvailableForStart(
@@ -198,7 +212,7 @@ export function assertSessionAvailableForStart(
 ): void {
   const existingState = readSessionState(session, logger);
   if (!existingState) return;
-  if (!isPidRunning(existingState.pid)) {
+  if (existingState.pid == null || !isPidRunning(existingState.pid)) {
     setSessionStatus(session, "exited", logger);
     return;
   }

package/src/cli/core/snapshot-analyzer.ts

@@ -8,15 +8,10 @@
  * to the CLI-agent approach if needed.
  *
  * Shared types and utilities (InterpretResultSchema, buildInlinePromptSelection,
- * formatInterpretationOutput, etc.) are still actively used by the API analyzer.
+ * etc.) are still actively used by the API analyzer.
  */
 
-import {
-  existsSync,
-  mkdtempSync,
-  readFileSync,
-  rmSync,
-} from "node:fs";
+import { existsSync, mkdtempSync, readFileSync, rmSync } from "node:fs";
 import { extname, isAbsolute, join, resolve } from "node:path";
 import { spawn } from "node:child_process";
 import { tmpdir } from "node:os";
@@ -147,7 +142,12 @@ abstract class UserCodingAgent {
     logger: LoggerApi,
     stdinText?: string,
   ): Promise<ExternalCommandResult> {
-    const result = await runExternalCommand(this.command, args, logger, stdinText);
+    const result = await runExternalCommand(
+      this.command,
+      args,
+      logger,
+      stdinText,
+    );
     if (result.exitCode !== 0) {
       throw new Error(
         `Analyzer command failed (${[this.command, ...args].join(" ")}).\n${stripAnsi(result.stderr).trim() || stripAnsi(result.stdout).trim() || "No error output."}`,
@@ -586,17 +586,18 @@ function estimateTokensFromChars(chars: number): number {
   return Math.ceil(chars / 4);
 }
 
-function inferContextWindowTokens(
-  model: string,
-): { contextWindowTokens: number; source: string } {
+function inferContextWindowTokens(model: string): {
+  contextWindowTokens: number;
+  source: string;
+} {
   const normalized = model.trim().toLowerCase();
   if (normalized.includes("claude")) {
     return { contextWindowTokens: 200_000, source: "model:claude" };
   }
   if (
-    normalized.includes("gpt-5")
-    || normalized.includes("o3")
-    || normalized.includes("o4")
+    normalized.includes("gpt-5") ||
+    normalized.includes("o3") ||
+    normalized.includes("o4")
   ) {
     return { contextWindowTokens: 200_000, source: "model:openai" };
   }
@@ -699,7 +700,9 @@ export function buildInlinePromptSelection(
     fullDomChars: fullHtmlContent.length,
     fullDomEstimatedTokens: estimateTokensFromChars(fullHtmlContent.length),
     condensedDomChars: condensedHtmlContent.length,
-    condensedDomEstimatedTokens: estimateTokensFromChars(condensedHtmlContent.length),
+    condensedDomEstimatedTokens: estimateTokensFromChars(
+      condensedHtmlContent.length,
+    ),
     configuredModel: model,
   };
 
@@ -740,8 +743,7 @@ export function buildInlinePromptSelection(
     false,
   );
   if (fullCandidate.promptEstimatedTokens <= budget.promptBudgetTokens) {
-    const selectionReason =
-      `Full DOM fits within the estimated prompt budget (~${fullCandidate.promptEstimatedTokens.toLocaleString()} <= ${budget.promptBudgetTokens.toLocaleString()} tokens), so the analyzer receives the uncondensed page HTML.`;
+    const selectionReason = `Full DOM fits within the estimated prompt budget (~${fullCandidate.promptEstimatedTokens.toLocaleString()} <= ${budget.promptBudgetTokens.toLocaleString()} tokens), so the analyzer receives the uncondensed page HTML.`;
     const prompt = buildInlineHtmlPrompt(args, {
       htmlContent: fullHtmlContent,
       domLabel: "full DOM",
@@ -784,7 +786,10 @@ export function buildInlinePromptSelection(
     2_000,
     budget.promptBudgetTokens - estimateTokensFromChars(basePrompt.length),
   );
-  const truncatedHtml = truncateText(condensedHtmlContent, availableHtmlTokens * 4);
+  const truncatedHtml = truncateText(
+    condensedHtmlContent,
+    availableHtmlTokens * 4,
+  );
 
   return buildCandidate(
     "condensed",
@@ -794,31 +799,6 @@ export function buildInlinePromptSelection(
   );
 }
 
-export function formatInterpretationOutput(
-  parsed: InterpretResult,
-  header: string = "Interpretation:",
-): string {
-  const outputLines: string[] = [];
-  outputLines.push(header);
-  outputLines.push(`Answer: ${parsed.answer}`);
-  outputLines.push("");
-  if (parsed.selectors.length === 0) {
-    outputLines.push("Selectors: none found.");
-  } else {
-    outputLines.push("Selectors:");
-    parsed.selectors.forEach((selector, index) => {
-      outputLines.push(`  ${index + 1}. ${selector.label}`);
-      outputLines.push(`     selector: ${selector.selector}`);
-      outputLines.push(`     rationale: ${selector.rationale}`);
-    });
-  }
-  if (parsed.notes && parsed.notes.trim()) {
-    outputLines.push("");
-    outputLines.push(`Notes: ${parsed.notes.trim()}`);
-  }
-  return outputLines.join("\n");
-}
-
 export async function runInterpret(
   args: InterpretArgs,
   logger: LoggerApi,
@@ -860,14 +840,14 @@ export async function runInterpret(
   // re-enabled, the caller must supply a valid provider/model-id string.
   throw new Error(
     "The CLI-agent snapshot analysis path is not active. " +
-    "Update your config to the current format with `npx libretto ai configure <provider>`, " +
-    "or set API credentials in .env for direct API analysis.",
+      "Update your config to the current format with `npx libretto ai configure <provider>`, " +
+      "or set API credentials in .env for direct API analysis.",
   );
 
   // Preserved for reference — to re-enable, remove the throw above and:
   // const selection = buildInlinePromptSelection(args, fullHtmlContent, condensedHtmlContent, model);
   // const parsed = await configuredAgent.analyzeSnapshot(selection.prompt, pngPath, logger);
-  // console.log(formatInterpretationOutput(parsed));
+  // console.log(parsed.answer);
 }
 
 export function canAnalyzeSnapshots(): boolean {

package/src/cli/core/snapshot-api-config.ts

@@ -1,191 +1,188 @@
 import { existsSync, readFileSync } from "node:fs";
 import { dirname, join, resolve } from "node:path";
-import {
-	type AiConfig,
-	readAiConfig,
-} from "./ai-config.js";
+import { type AiConfig, readAiConfig } from "./ai-config.js";
 import { LIBRETTO_CONFIG_PATH, REPO_ROOT } from "./context.js";
 import {
-	hasProviderCredentials,
-	parseModel,
-	type Provider,
+  hasProviderCredentials,
+  parseModel,
+  type Provider,
 } from "../../shared/llm/client.js";
 
 const DEFAULT_SNAPSHOT_MODELS = {
-	openai: "openai/gpt-5.4",
-	anthropic: "anthropic/claude-sonnet-4-6",
-	google: "google/gemini-3-flash-preview",
-	vertex: "vertex/gemini-2.5-pro",
+  openai: "openai/gpt-5.4",
+  anthropic: "anthropic/claude-sonnet-4-6",
+  google: "google/gemini-3-flash-preview",
+  vertex: "vertex/gemini-2.5-pro",
 } as const satisfies Record<Provider, string>;
 
 export type SnapshotApiModelSelection = {
-	model: string;
-	provider: Provider;
-	source:
-		| "config"
-		| "env:auto-openai"
-		| "env:auto-anthropic"
-		| "env:auto-google"
-		| "env:auto-vertex";
+  model: string;
+  provider: Provider;
+  source:
+    | "config"
+    | "env:auto-openai"
+    | "env:auto-anthropic"
+    | "env:auto-google"
+    | "env:auto-vertex";
 };
 
 export class SnapshotApiUnavailableError extends Error {
-	constructor(message: string) {
-		super(message);
-		this.name = "SnapshotApiUnavailableError";
-	}
+  constructor(message: string) {
+    super(message);
+    this.name = "SnapshotApiUnavailableError";
+  }
 }
 
 function providerSetupSentence(provider: Provider): string {
-	switch (provider) {
-		case "openai":
-			return "Add OPENAI_API_KEY to .env or as a shell environment variable.";
-		case "anthropic":
-			return "Add ANTHROPIC_API_KEY to .env or as a shell environment variable.";
-		case "google":
-			return "Add GEMINI_API_KEY or GOOGLE_GENERATIVE_AI_API_KEY to .env or as a shell environment variable.";
-		case "vertex":
-			return "Add GOOGLE_CLOUD_PROJECT or GCLOUD_PROJECT to .env or as a shell environment variable, and make sure application default credentials are configured.";
-	}
+  switch (provider) {
+    case "openai":
+      return "Add OPENAI_API_KEY to .env or as a shell environment variable.";
+    case "anthropic":
+      return "Add ANTHROPIC_API_KEY to .env or as a shell environment variable.";
+    case "google":
+      return "Add GEMINI_API_KEY or GOOGLE_GENERATIVE_AI_API_KEY to .env or as a shell environment variable.";
+    case "vertex":
+      return "Add GOOGLE_CLOUD_PROJECT or GCLOUD_PROJECT to .env or as a shell environment variable, and make sure application default credentials are configured.";
+  }
 }
 
 function defaultModelCommandLine(): string {
-	return "npx libretto ai configure openai | anthropic | gemini | vertex";
+  return "npx libretto ai configure openai | anthropic | gemini | vertex";
 }
 
 function providerMissingCredentialSummary(provider: Provider): string {
-	switch (provider) {
-		case "openai":
-			return "OPENAI_API_KEY is missing";
-		case "anthropic":
-			return "ANTHROPIC_API_KEY is missing";
-		case "google":
-			return "GEMINI_API_KEY and GOOGLE_GENERATIVE_AI_API_KEY are missing";
-		case "vertex":
-			return "GOOGLE_CLOUD_PROJECT and GCLOUD_PROJECT are missing";
-	}
+  switch (provider) {
+    case "openai":
+      return "OPENAI_API_KEY is missing";
+    case "anthropic":
+      return "ANTHROPIC_API_KEY is missing";
+    case "google":
+      return "GEMINI_API_KEY and GOOGLE_GENERATIVE_AI_API_KEY are missing";
+    case "vertex":
+      return "GOOGLE_CLOUD_PROJECT and GCLOUD_PROJECT are missing";
+  }
 }
 
 function noSnapshotApiConfiguredMessage(): string {
-	return [
-		"Failed to analyze snapshot because no snapshot analyzer is configured.",
-		`Add OPENAI_API_KEY, ANTHROPIC_API_KEY, GEMINI_API_KEY or GOOGLE_GENERATIVE_AI_API_KEY, or GOOGLE_CLOUD_PROJECT to .env or as a shell environment variable, or choose a default model with \`${defaultModelCommandLine()}\`.`,
-		"For more info, run `npx libretto init`.",
-	].join(" ");
+  return [
+    "Failed to analyze snapshot because no snapshot analyzer is configured.",
+    `Add OPENAI_API_KEY, ANTHROPIC_API_KEY, GEMINI_API_KEY or GOOGLE_GENERATIVE_AI_API_KEY, or GOOGLE_CLOUD_PROJECT to .env or as a shell environment variable, or choose a default model with \`${defaultModelCommandLine()}\`.`,
+    "For more info, run `npx libretto init`.",
+  ].join(" ");
 }
 
 function missingProviderSnapshotMessage(
-	selection: SnapshotApiModelSelection,
+  selection: SnapshotApiModelSelection,
 ): string {
-	const configuredSource =
-		selection.source === "config"
-			? ` in ${LIBRETTO_CONFIG_PATH}`
-			: " from process env or .env";
-	return [
-		`Failed to analyze snapshot because ${selection.provider} is configured${configuredSource}, but ${providerMissingCredentialSummary(selection.provider)}.`,
-		providerSetupSentence(selection.provider),
-		"For more info, run `npx libretto init`.",
-	].join(" ");
+  const configuredSource =
+    selection.source === "config"
+      ? ` in ${LIBRETTO_CONFIG_PATH}`
+      : " from process env or .env";
+  return [
+    `Failed to analyze snapshot because ${selection.provider} is configured${configuredSource}, but ${providerMissingCredentialSummary(selection.provider)}.`,
+    providerSetupSentence(selection.provider),
+    "For more info, run `npx libretto init`.",
+  ].join(" ");
 }
 
 function readWorktreeEnvPath(): string | null {
-	const gitPath = join(REPO_ROOT, ".git");
-	if (!existsSync(gitPath)) return null;
-
-	try {
-		const gitPointer = readFileSync(gitPath, "utf-8").trim();
-		const match = gitPointer.match(/^gitdir:\s*(.+)$/i);
-		if (!match?.[1]) return null;
-		const worktreeGitDir = resolve(REPO_ROOT, match[1].trim());
-		const commonGitDir = resolve(worktreeGitDir, "..", "..");
-		return join(dirname(commonGitDir), ".env");
-	} catch {
-		return null;
-	}
+  const gitPath = join(REPO_ROOT, ".git");
+  if (!existsSync(gitPath)) return null;
+
+  try {
+    const gitPointer = readFileSync(gitPath, "utf-8").trim();
+    const match = gitPointer.match(/^gitdir:\s*(.+)$/i);
+    if (!match?.[1]) return null;
+    const worktreeGitDir = resolve(REPO_ROOT, match[1].trim());
+    const commonGitDir = resolve(worktreeGitDir, "..", "..");
+    return join(dirname(commonGitDir), ".env");
+  } catch {
+    return null;
+  }
 }
 
 export function loadSnapshotEnv(): void {
-	if (process.env.LIBRETTO_DISABLE_DOTENV?.trim() === "1") return;
-
-	const envPathCandidates = [
-		join(REPO_ROOT, ".env"),
-		readWorktreeEnvPath(),
-	].filter((value): value is string => Boolean(value));
-
-	const envPath = envPathCandidates.find((candidate) => existsSync(candidate));
-	if (!envPath) return;
-
-	for (const line of readFileSync(envPath, "utf-8").split("\n")) {
-		const parsed = parseDotEnvAssignment(line);
-		if (!parsed) continue;
-		if (!(parsed.key in process.env)) {
-			process.env[parsed.key] = parsed.value;
-		}
-	}
+  if (process.env.LIBRETTO_DISABLE_DOTENV?.trim() === "1") return;
+
+  const envPathCandidates = [
+    join(REPO_ROOT, ".env"),
+    readWorktreeEnvPath(),
+  ].filter((value): value is string => Boolean(value));
+
+  const envPath = envPathCandidates.find((candidate) => existsSync(candidate));
+  if (!envPath) return;
+
+  for (const line of readFileSync(envPath, "utf-8").split("\n")) {
+    const parsed = parseDotEnvAssignment(line);
+    if (!parsed) continue;
+    if (!(parsed.key in process.env)) {
+      process.env[parsed.key] = parsed.value;
+    }
+  }
 }
 
 export function parseDotEnvAssignment(
-	line: string,
+  line: string,
 ): { key: string; value: string } | null {
-	const trimmed = line.trim();
-	if (!trimmed || trimmed.startsWith("#")) return null;
-
-	const withoutExport = trimmed.startsWith("export ")
-		? trimmed.slice("export ".length).trimStart()
-		: trimmed;
-	const eqIdx = withoutExport.indexOf("=");
-	if (eqIdx < 1) return null;
-
-	const key = withoutExport.slice(0, eqIdx).trim();
-	if (!key) return null;
-
-	const rawValue = withoutExport.slice(eqIdx + 1).trimStart();
-	if (!rawValue) {
-		return { key, value: "" };
-	}
-
-	if (rawValue.startsWith('"')) {
-		const closeIdx = rawValue.indexOf('"', 1);
-		if (closeIdx > 0) {
-			return { key, value: rawValue.slice(1, closeIdx) };
-		}
-		return { key, value: rawValue.slice(1) };
-	}
-
-	if (rawValue.startsWith("'")) {
-		const closeIdx = rawValue.indexOf("'", 1);
-		if (closeIdx > 0) {
-			return { key, value: rawValue.slice(1, closeIdx) };
-		}
-		return { key, value: rawValue.slice(1) };
-	}
-
-	const inlineCommentIndex = rawValue.search(/\s#/);
-	const value =
-		inlineCommentIndex >= 0
-			? rawValue.slice(0, inlineCommentIndex).trimEnd()
-			: rawValue.trim();
-	return { key, value };
+  const trimmed = line.trim();
+  if (!trimmed || trimmed.startsWith("#")) return null;
+
+  const withoutExport = trimmed.startsWith("export ")
+    ? trimmed.slice("export ".length).trimStart()
+    : trimmed;
+  const eqIdx = withoutExport.indexOf("=");
+  if (eqIdx < 1) return null;
+
+  const key = withoutExport.slice(0, eqIdx).trim();
+  if (!key) return null;
+
+  const rawValue = withoutExport.slice(eqIdx + 1).trimStart();
+  if (!rawValue) {
+    return { key, value: "" };
+  }
+
+  if (rawValue.startsWith('"')) {
+    const closeIdx = rawValue.indexOf('"', 1);
+    if (closeIdx > 0) {
+      return { key, value: rawValue.slice(1, closeIdx) };
+    }
+    return { key, value: rawValue.slice(1) };
+  }
+
+  if (rawValue.startsWith("'")) {
+    const closeIdx = rawValue.indexOf("'", 1);
+    if (closeIdx > 0) {
+      return { key, value: rawValue.slice(1, closeIdx) };
+    }
+    return { key, value: rawValue.slice(1) };
+  }
+
+  const inlineCommentIndex = rawValue.search(/\s#/);
+  const value =
+    inlineCommentIndex >= 0
+      ? rawValue.slice(0, inlineCommentIndex).trimEnd()
+      : rawValue.trim();
+  return { key, value };
 }
 
 function inferAutoSnapshotModel(): SnapshotApiModelSelection | null {
-	const providersInPriorityOrder: Provider[] = [
-		"openai",
-		"anthropic",
-		"google",
-		"vertex",
-	];
-
-	for (const provider of providersInPriorityOrder) {
-		if (!hasProviderCredentials(provider)) continue;
-		return {
-			model: DEFAULT_SNAPSHOT_MODELS[provider],
-			provider,
-			source: `env:auto-${provider}` as SnapshotApiModelSelection["source"],
-		};
-	}
-
-	return null;
+  const providersInPriorityOrder: Provider[] = [
+    "openai",
+    "anthropic",
+    "google",
+    "vertex",
+  ];
+
+  for (const provider of providersInPriorityOrder) {
+    if (!hasProviderCredentials(provider)) continue;
+    return {
+      model: DEFAULT_SNAPSHOT_MODELS[provider],
+      provider,
+      source: `env:auto-${provider}` as SnapshotApiModelSelection["source"],
+    };
+  }
+
+  return null;
 }
 
 /**
@@ -196,41 +193,39 @@ function inferAutoSnapshotModel(): SnapshotApiModelSelection | null {
  * 2. Auto-detect from available API credentials in env
  */
 export function resolveSnapshotApiModel(
-	config: AiConfig | null = readAiConfig(),
+  config: AiConfig | null = readAiConfig(),
 ): SnapshotApiModelSelection | null {
-	loadSnapshotEnv();
-
-	if (config?.model) {
-		const { provider } = parseModel(config.model);
-		return {
-			model: config.model,
-			provider,
-			source: "config",
-		};
-	}
-
-	return inferAutoSnapshotModel();
+  loadSnapshotEnv();
+
+  if (config?.model) {
+    const { provider } = parseModel(config.model);
+    return {
+      model: config.model,
+      provider,
+      source: "config",
+    };
+  }
+
+  return inferAutoSnapshotModel();
 }
 
 export function resolveSnapshotApiModelOrThrow(
-	config: AiConfig | null = readAiConfig(),
+  config: AiConfig | null = readAiConfig(),
 ): SnapshotApiModelSelection {
-	const selection = resolveSnapshotApiModel(config);
-	if (!selection) {
-		throw new SnapshotApiUnavailableError(
-			noSnapshotApiConfiguredMessage(),
-		);
-	}
-
-	if (!hasProviderCredentials(selection.provider)) {
-		throw new SnapshotApiUnavailableError(
-			missingProviderSnapshotMessage(selection),
-		);
-	}
-
-	return selection;
+  const selection = resolveSnapshotApiModel(config);
+  if (!selection) {
+    throw new SnapshotApiUnavailableError(noSnapshotApiConfiguredMessage());
+  }
+
+  if (!hasProviderCredentials(selection.provider)) {
+    throw new SnapshotApiUnavailableError(
+      missingProviderSnapshotMessage(selection),
+    );
+  }
+
+  return selection;
 }
 
 export function isSnapshotApiUnavailableError(error: unknown): boolean {
-	return error instanceof SnapshotApiUnavailableError;
+  return error instanceof SnapshotApiUnavailableError;
 }