@tempo-ai/visual-review 0.0.114 → 0.0.115

package/dist/bin.js

@@ -6,7 +6,7 @@ async function main(argv = process.argv.slice(2)) {
   const [command, ...commandArgs] = argv;
   switch (command) {
     case "capture":
-      return await (await import("./cli-BJVATWX4.js")).main(commandArgs);
+      return await (await import("./cli-HEA4IYKH.js")).main(commandArgs);
     case "claim":
       return await (await import("./claim-J654JAS6.js")).main(commandArgs);
     case "report":

package/dist/{chunk-RY7ZWRV5.js → chunk-BXSN22AO.js}

@@ -1503,18 +1503,23 @@ function runAbortableShellCommand(options) {
       exitCode: null,
       stdout: "",
       stderr: "",
+      output: "",
       aborted: true,
+      timedOut: false,
       spawnError: null
     });
   }
   return new Promise((resolve2) => {
     let stdout = "";
     let stderr = "";
+    let output = "";
     let aborted = false;
+    let timedOut = false;
     let settled = false;
     let terminationComplete = true;
     let closeResult;
     let forceTimer = null;
+    let idleTimer = null;
     let child;
     try {
       child = spawn2(options.command, {
@@ -1530,7 +1535,9 @@ function runAbortableShellCommand(options) {
         exitCode: null,
         stdout,
         stderr,
+        output,
         aborted: false,
+        timedOut: false,
         spawnError: error instanceof Error ? error : new Error(String(error))
       });
       return;
@@ -1540,18 +1547,23 @@ function runAbortableShellCommand(options) {
       if (settled || closeResult === void 0 || !terminationComplete) return;
       settled = true;
       if (forceTimer) clearTimeout(forceTimer);
+      if (idleTimer) clearTimeout(idleTimer);
       options.signal?.removeEventListener("abort", onAbort);
       resolve2({
         exitCode: closeResult,
         stdout,
         stderr,
+        output,
         aborted,
+        timedOut,
         spawnError
       });
     };
-    const terminateTree = () => {
-      if (aborted) return;
-      aborted = true;
+    const terminateTree = (reason) => {
+      if (aborted || timedOut || closeResult !== void 0) return;
+      aborted = reason === "abort";
+      timedOut = reason === "timeout";
+      if (idleTimer) clearTimeout(idleTimer);
       terminationComplete = false;
       const pid = child.pid;
       if (pid != null && process.platform !== "win32") {
@@ -1602,19 +1614,41 @@ function runAbortableShellCommand(options) {
         terminationComplete = true;
       }
     };
-    const onAbort = () => terminateTree();
+    const onAbort = () => terminateTree("abort");
     options.signal?.addEventListener("abort", onAbort, { once: true });
-    if (options.signal?.aborted) terminateTree();
+    if (options.signal?.aborted) onAbort();
+    const resetIdleTimer = () => {
+      if (idleTimer) clearTimeout(idleTimer);
+      if (aborted || timedOut || closeResult !== void 0 || !options.idleTimeoutMs)
+        return;
+      idleTimer = setTimeout(
+        () => terminateTree("timeout"),
+        options.idleTimeoutMs
+      );
+    };
+    const forwardOutput = (chunk) => {
+      if (settled || String(chunk).length === 0) return;
+      output = appendTail(output, chunk, outputLimit);
+      resetIdleTimer();
+      try {
+        options.onOutput?.(String(chunk));
+      } catch {
+      }
+    };
+    resetIdleTimer();
     child.stdout?.on("data", (chunk) => {
       stdout = appendTail(stdout, chunk, outputLimit);
+      forwardOutput(chunk);
     });
     child.stderr?.on("data", (chunk) => {
       stderr = appendTail(stderr, chunk, outputLimit);
+      forwardOutput(chunk);
     });
     child.on("error", (error) => {
       spawnError = error;
     });
     child.on("close", (exitCode) => {
+      if (idleTimer) clearTimeout(idleTimer);
       closeResult = exitCode;
       finishIfComplete();
     });
@@ -4876,6 +4910,34 @@ async function parcelEventToFileEvent(event, rootInfo) {
   };
 }
 
+// ../tempo-devserver/install-feedback.ts
+var INSTALL_IDLE_TIMEOUT_MS = 5 * 6e4;
+var CONNECTION_CODE = /\b(?:ETIMEDOUT|ESOCKETTIMEDOUT|ECONNRESET|ECONNREFUSED|ENOTFOUND|EAI_AGAIN|ENETUNREACH|EHOSTUNREACH|ERR_SOCKET_TIMEOUT|UND_ERR_CONNECT_TIMEOUT|UND_ERR_SOCKET)\b/;
+var CERTIFICATE_CODE = /\b(?:CERT_HAS_EXPIRED|CERT_NOT_YET_VALID|SELF_SIGNED_CERT_IN_CHAIN|DEPTH_ZERO_SELF_SIGNED_CERT|UNABLE_TO_VERIFY_LEAF_SIGNATURE|UNABLE_TO_GET_ISSUER_CERT_LOCALLY|ERR_TLS_CERT_ALTNAME_INVALID)\b/;
+function describeInstallConnectivity(output) {
+  const relevant = output.replace(/\u001b\[[0-9;]*m/g, "").split(/\r?\n/).filter((line) => !/\b(?:warn|warning)\s+deprecated\b/i.test(line)).join("\n");
+  const terminal = [
+    ...relevant.matchAll(/npm (?:error|ERR!) code\s+(\w+)|\bERR_PNPM_(\w+)/g)
+  ].at(-1);
+  const terminalCode = terminal?.[1] ?? terminal?.[2];
+  const failureDetail = terminal?.[2] === "META_FETCH_FAIL" ? relevant.slice(terminal.index) : terminalCode ?? relevant;
+  if (terminalCode && terminalCode !== "META_FETCH_FAIL" && !CONNECTION_CODE.test(terminalCode) && !CERTIFICATE_CODE.test(terminalCode))
+    return null;
+  const certificate = failureDetail.match(CERTIFICATE_CODE)?.[0];
+  if (certificate)
+    return {
+      code: certificate,
+      hint: "Check your proxy or registry certificate trust, then retry."
+    };
+  const connection = failureDetail.match(CONNECTION_CODE)?.[0];
+  if (connection)
+    return {
+      code: connection,
+      hint: "Check your internet connection, VPN, or proxy, then retry."
+    };
+  return null;
+}
+
 // ../tempo-sdk/package.json
 var package_default = {
   name: "tempo-sdk",
@@ -5554,10 +5616,11 @@ import path8 from "path";
 
 // ../tempo-devserver/errors.ts
 var TempoDevServerError = class extends Error {
-  constructor(code, message) {
+  constructor(code, message, recoveryHint) {
     super(message);
     this.code = code;
     this.name = "TempoDevServerError";
+    if (recoveryHint) this.recoveryHint = recoveryHint;
   }
 };
 var ConfigMissingError = class extends TempoDevServerError {
@@ -5577,6 +5640,32 @@ ${output}` : "";
     this.name = "InstallFailedError";
   }
 };
+var InstallNetworkError = class extends TempoDevServerError {
+  constructor(command, output, problem) {
+    super(
+      "install_network_error",
+      `${problem.hint}
+
+Package download failed (${problem.code}): ${command}
+${output.trim()}`
+    );
+    this.name = "InstallNetworkError";
+    this.recoveryHint = problem.hint;
+  }
+};
+var InstallTimeoutError = class extends TempoDevServerError {
+  constructor(command, output) {
+    super(
+      "install_timeout",
+      `Dependency installation produced no output for 5 minutes and was stopped. Check the install logs, your connection, or a waiting install script before retrying.
+
+Command: ${command}
+${output.trim()}`
+    );
+    this.recoveryHint = "No install output for 5 minutes. Check the install logs and your connection, then retry.";
+    this.name = "InstallTimeoutError";
+  }
+};
 var ToolMissingError = class extends TempoDevServerError {
   constructor(toolName, cwd) {
     super(
@@ -7904,13 +7993,15 @@ function buildUnsatisfiedNodeHint(args) {
 // ../tempo-devserver/sdk-upgrade-plan.ts
 import { access as access3, readFile as readFile7 } from "fs/promises";
 import path11 from "path";
-var LOCKFILES = [
+var INSTALL_MARKERS = [
   // Order matters, same rationale as pm-detect: a tree carrying both bun.lock
   // and package-lock.json is a bun tree with a stale npm lockfile.
   { file: "bun.lock", pm: "bun" },
   { file: "bun.lockb", pm: "bun" },
   { file: "pnpm-lock.yaml", pm: "pnpm" },
   { file: "yarn.lock", pm: "yarn" },
+  // A pnpm workspace owns its install even before its first lockfile exists.
+  { file: "pnpm-workspace.yaml", pm: "pnpm" },
   { file: "package-lock.json", pm: "npm" }
 ];
 async function fileExists2(filePath) {
@@ -7932,11 +8023,11 @@ function parsePackageManagerField2(value) {
 function isYarnBerryLockfile(contents) {
   return /^__metadata:/m.test(contents);
 }
-function buildCommands(pm, yarnBerry, metadataCacheDir, scopedToTempoRoot) {
+function buildCommands(pm, yarnBerry, metadataCacheDir, ignorePnpmWorkspace) {
   switch (pm) {
     case "pnpm": {
       const cacheFlag = metadataCacheDir ? ` --config.cacheDir=${metadataCacheDir}` : "";
-      const workspaceFlag = scopedToTempoRoot ? " --ignore-workspace" : "";
+      const workspaceFlag = ignorePnpmWorkspace ? " --ignore-workspace" : "";
       return {
         updateCommand: `pnpm update tempo-sdk --latest${workspaceFlag}${cacheFlag}`,
         remediationCommand: `pnpm update tempo-sdk --latest${workspaceFlag}${cacheFlag}`
@@ -7971,9 +8062,10 @@ async function resolveTempoSdkUpgradePlan(tempoRoot, options = {}) {
     const parsed = JSON.parse(raw);
     return parsePackageManagerField2(parsed.packageManager);
   }).catch(() => null);
-  const owner = await findLockfileOwner(resolvedRoot);
+  const owner = await findInstallOwner(resolvedRoot);
   const resolvedPm = declared ?? owner?.pm ?? "npm";
   const governedByAncestorWorkspace = owner !== null && owner.dir !== resolvedRoot;
+  const hasOwnPnpmWorkspace = resolvedPm === "pnpm" && await fileExists2(path11.join(resolvedRoot, "pnpm-workspace.yaml"));
   return {
     packageManager: resolvedPm,
     cwd: resolvedRoot,
@@ -7982,20 +8074,20 @@ async function resolveTempoSdkUpgradePlan(tempoRoot, options = {}) {
       resolvedPm,
       owner?.yarnLockContents !== void 0 && isYarnBerryLockfile(owner.yarnLockContents),
       options.metadataCacheDir ?? null,
-      !governedByAncestorWorkspace
+      !governedByAncestorWorkspace && !hasOwnPnpmWorkspace
     )
   };
 }
-async function findLockfileOwner(startDir) {
+async function findInstallOwner(startDir) {
   for (let dir = startDir; ; dir = path11.dirname(dir)) {
-    for (const candidate of LOCKFILES) {
-      const lockPath = path11.join(dir, candidate.file);
-      if (!await fileExists2(lockPath)) continue;
+    for (const candidate of INSTALL_MARKERS) {
+      const markerPath = path11.join(dir, candidate.file);
+      if (!await fileExists2(markerPath)) continue;
       return {
         dir,
         pm: candidate.pm,
         ...candidate.pm === "yarn" ? {
-          yarnLockContents: await readFile7(lockPath, "utf-8").catch(
+          yarnLockContents: await readFile7(markerPath, "utf-8").catch(
             () => ""
           )
         } : {}
@@ -9345,7 +9437,8 @@ var TempoDevServer = class _TempoDevServer {
       if (options?.forceInstall || !this.state.dependenciesInstalled) {
         await this.installDependenciesForStart(
           options?.forceInstall === true,
-          options?.signal
+          options?.signal,
+          options?.onInstallOutput
         );
       }
       throwIfLaunchAborted(options?.signal);
@@ -9379,7 +9472,12 @@ var TempoDevServer = class _TempoDevServer {
               }
               await invalidateInstallCompletionMarkers(root);
             }
-            await runShellCommand(command, cwd, signal);
+            await runShellCommand(
+              command,
+              cwd,
+              signal,
+              options?.onInstallOutput
+            );
             sdkCommandCompleted = true;
           }
         }
@@ -9586,7 +9684,7 @@ var TempoDevServer = class _TempoDevServer {
       }
     };
   }
-  async installDependenciesForStart(forceInstall, signal) {
+  async installDependenciesForStart(forceInstall, signal, onOutput) {
     const installTimer = devTimer("supervisor:install", {
       tempoRoot: this.tempoRoot,
       commandCwd: this.commandCwd,
@@ -9604,7 +9702,7 @@ var TempoDevServer = class _TempoDevServer {
         try {
           for (const step of steps) {
             throwIfLaunchAborted(signal);
-            await runInstallStep(step, signal);
+            await runInstallStep(step, signal, onOutput);
           }
         } catch (error) {
           throwIfLaunchAborted(signal);
@@ -9617,7 +9715,7 @@ var TempoDevServer = class _TempoDevServer {
           );
           for (const step of localSteps) {
             throwIfLaunchAborted(signal);
-            await runInstallStep(step, signal);
+            await runInstallStep(step, signal, onOutput);
           }
         }
       };
@@ -11490,6 +11588,9 @@ function classifyInstallFailure(command, output) {
   if (INSTALL_LOCK_PATTERN.test(output)) {
     return new InstallPermissionDeniedError(command, output);
   }
+  const connectivity = describeInstallConnectivity(output);
+  if (connectivity)
+    return new InstallNetworkError(command, output, connectivity);
   return null;
 }
 function isIgnoredBuildScriptsOnlyFailure(output) {
@@ -11503,6 +11604,7 @@ async function runInstallCommandWithLockRetry(command, cwd, runCommandFn = runSh
       await runCommandFn(command, cwd, signal);
       return;
     } catch (error) {
+      if (error instanceof InstallTimeoutError) throw error;
       const output = error instanceof Error ? error.message : String(error);
       const delay = retryDelaysMs[attempt];
       if (delay === void 0 || !isTransientFsLockOutput(output)) throw error;
@@ -11511,7 +11613,7 @@ async function runInstallCommandWithLockRetry(command, cwd, runCommandFn = runSh
     }
   }
 }
-async function runInstallStep(step, signal) {
+async function runInstallStep(step, signal, onOutput) {
   throwIfLaunchAborted(signal);
   await Promise.all(
     [
@@ -11538,12 +11640,13 @@ async function runInstallStep(step, signal) {
     await runInstallCommandWithLockRetry(
       step.command,
       step.cwd,
-      runShellCommand,
+      (command, cwd, signal2) => runShellCommand(command, cwd, signal2, onOutput),
       INSTALL_LOCK_RETRY_DELAYS_MS,
       signal
     );
   } catch (error) {
     throwIfLaunchAborted(signal);
+    if (error instanceof InstallTimeoutError) throw error;
     const output = error instanceof Error ? error.message : String(error);
     if (isIgnoredBuildScriptsOnlyFailure(output) && await nodeModulesHasMeaningfulEntries(step.cwd)) {
       console.warn(
@@ -12011,7 +12114,7 @@ async function runBoundedShellCommand(options) {
     );
   });
 }
-async function runShellCommand(command, cwd, signal) {
+async function runShellCommand(command, cwd, signal, onOutput) {
   throwIfLaunchAborted(signal);
   const projectNode = toProjectNodeEnvOption(
     await resolveProjectNodeOverrideForDir(cwd)
@@ -12021,12 +12124,15 @@ async function runShellCommand(command, cwd, signal) {
     command: lifted.command,
     cwd,
     signal,
-    env: { ...buildManagedCommandEnv({ projectNode }), ...lifted.env }
+    env: { ...buildManagedCommandEnv({ projectNode }), ...lifted.env },
+    onOutput,
+    idleTimeoutMs: INSTALL_IDLE_TIMEOUT_MS
   });
   if (result.aborted) throw launchAbortError();
+  if (result.timedOut) throw new InstallTimeoutError(command, result.output);
   if (result.spawnError) throw result.spawnError;
   if (result.exitCode === 0) return;
-  const output = (result.stderr.trim() || result.stdout.trim()).slice(0, 4e3);
+  const output = result.output.trim();
   throw new InstallFailedError(command, result.exitCode, output);
 }
 async function ensureTempoSdkLatestSpec(cwd, signal, options = {}) {
@@ -14067,4 +14173,4 @@ export {
   buildBaselineCaptureArgs,
   main
 };
-//# sourceMappingURL=chunk-RY7ZWRV5.js.map
+//# sourceMappingURL=chunk-BXSN22AO.js.map