@reshotdev/screenshot 0.0.1-beta.17 → 0.0.1-beta.19

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reshotdev/screenshot",
-  "version": "0.0.1-beta.17",
+  "version": "0.0.1-beta.19",
   "description": "Screenshot and video capture CLI",
   "author": "Reshot <hello@reshot.dev>",
   "license": "MIT",

package/src/commands/doctor-target.js

@@ -2,6 +2,7 @@
 
 const chalk = require("chalk");
 const { runDoctorTarget } = require("../lib/certification");
+const config = require("../lib/config");
 
 async function doctorTargetCommand(options = {}) {
   const scenarioKeys = options.scenarios
@@ -11,6 +12,32 @@ async function doctorTargetCommand(options = {}) {
         .filter(Boolean)
     : null;
 
+  // Check for a config BEFORE the banner so we don't print "scenarios: all
+  // certified" immediately above a "Config file not found" error (audit run-11
+  // F5). Without a config there is nothing to certify.
+  let hasConfig = false;
+  try {
+    hasConfig = config.configExists();
+  } catch {
+    hasConfig = false;
+  }
+  if (!hasConfig) {
+    if (options.json) {
+      console.log(
+        JSON.stringify({ ok: false, error: "Config file not found" }, null, 2),
+      );
+    } else {
+      console.error(
+        chalk.red("\n  ✖ Target doctor aborted: Config file not found."),
+      );
+      console.error(
+        chalk.gray("  Run `reshot init` (or `reshot setup`) to create one."),
+      );
+    }
+    process.exitCode = 1;
+    return { ok: false, error: "Config file not found" };
+  }
+
   // Emit an immediate banner BEFORE any async work so the command is never
   // silent — previously it produced zero output while preparing fixtures and
   // launching a browser, which read as a hang.

package/src/commands/publish.js

@@ -1847,7 +1847,7 @@ async function publishCommand(options = {}) {
           if (keptLive > 0) {
             console.log(
               chalk.gray(
-                `    ${keptLive} unchanged visual${keptLive === 1 ? "" : "s"} stayed live (no re-approval needed).`,
+                `    ${keptLive} unchanged visual${keptLive === 1 ? "" : "s"} kept current state (live pointers preserved; no re-approval needed).`,
               ),
             );
           }
@@ -1862,7 +1862,7 @@ async function publishCommand(options = {}) {
         } else {
           console.log(
             chalk.cyan(
-              `\n  ℹ No new captures to review — all ${successCount} published visual${successCount === 1 ? "" : "s"} were unchanged and stayed live.`,
+              `\n  ℹ No new captures to review — all ${successCount} published visual${successCount === 1 ? "" : "s"} were unchanged; existing live pointers are preserved (already-approved stay live, pending stay pending).`,
             ),
           );
         }

package/src/commands/pull.js

@@ -120,7 +120,13 @@ function deriveUsageSample(assets) {
       const contexts = Object.keys(assets[group][visualKey] || {});
       if (contexts.length === 0) continue;
       const context = contexts.includes("default") ? "default" : contexts[0];
-      return { group, visualKey, context };
+      // A capture entry is either flat ({src,width,height}) or step-based
+      // ({steps:[{src,...}]}). Tell the caller which, so the printed example
+      // references the right path (`.steps[0].src` vs `.src`) and doesn't
+      // produce an undefined src (audit run-12 MED-1).
+      const entry = assets[group][visualKey][context];
+      const stepped = !!(entry && Array.isArray(entry.steps));
+      return { group, visualKey, context, stepped };
     }
   }
   return null;
@@ -479,6 +485,9 @@ async function pullCommand(options = {}) {
     const sGroup = sample?.group ?? "dashboard";
     const sVisual = sample?.visualKey ?? "mainView";
     const sCtx = sample?.context ?? "default";
+    // Step-based capture entries nest the fields under `steps[0]`; reference it
+    // so the printed example's src/width/height aren't undefined (run-12 MED-1).
+    const sStep = sample?.stepped ? ".steps[0]" : "";
 
     // Show format-specific usage instructions
     if (format === "ts") {
@@ -499,10 +508,10 @@ async function pullCommand(options = {}) {
       console.log(chalk.cyan('  import assets from "./reshot-assets.json";\n'));
       console.log(chalk.white("Access assets with dot notation:\n"));
       console.log(chalk.cyan("  <img"));
-      console.log(chalk.cyan(`    src={assets.assets.${sGroup}.${sVisual}.${sCtx}.src}`));
-      console.log(chalk.cyan(`    width={assets.assets.${sGroup}.${sVisual}.${sCtx}.width}`));
-      console.log(chalk.cyan(`    height={assets.assets.${sGroup}.${sVisual}.${sCtx}.height}`));
-      console.log(chalk.cyan(`    alt={assets.assets.${sGroup}.${sVisual}.${sCtx}.alt}`));
+      console.log(chalk.cyan(`    src={assets.assets.${sGroup}.${sVisual}.${sCtx}${sStep}.src}`));
+      console.log(chalk.cyan(`    width={assets.assets.${sGroup}.${sVisual}.${sCtx}${sStep}.width}`));
+      console.log(chalk.cyan(`    height={assets.assets.${sGroup}.${sVisual}.${sCtx}${sStep}.height}`));
+      console.log(chalk.cyan(`    alt={assets.assets.${sGroup}.${sVisual}.${sCtx}${sStep}.alt}`));
       console.log(chalk.cyan("  />\n"));
     } else if (format === "csv") {
       console.log(chalk.blue("━━━ CSV Usage ━━━\n"));

package/src/commands/setup-wizard.js

@@ -225,6 +225,26 @@ async function setupWizard(options = {}) {
     );
   }
 
+  // The interactive wizard below needs a TTY. In CI / piped-stdin contexts
+  // (and when no `--project/--token` were supplied to drive it non-
+  // interactively), bail out with clear guidance instead of letting inquirer
+  // throw an unhandled `ERR_USE_AFTER_CLOSE: readline` (audit run-11 F4).
+  if (!isInteractive() && !didLinkFromOptions) {
+    console.log(
+      chalk.yellow("\n⚠ Non-interactive environment detected (no TTY)."),
+    );
+    console.log(
+      chalk.gray("  Run setup with your project credentials instead:"),
+    );
+    console.log(
+      chalk.cyan("    npx reshot setup --project <projectId> --token <token>"),
+    );
+    console.log(
+      chalk.gray("  or set RESHOT_PROJECT_ID and RESHOT_API_KEY, then re-run.\n"),
+    );
+    return;
+  }
+
   // If already set up and not forcing, show status and offer options
   if ((isAlreadyAuthed || existingConfig) && !force && !didLinkFromOptions) {
     console.log(