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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reshotdev/screenshot",
-  "version": "0.0.1-beta.17",
+  "version": "0.0.1-beta.18",
   "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/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(