@reshotdev/screenshot 0.0.1-beta.22 → 0.0.1-beta.24

package/package.json

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

package/src/commands/doctor-target.js

@@ -90,10 +90,32 @@ async function doctorTargetCommand(options = {}) {
         : chalk.red("  ✖ Target contract check failed"),
     );
     for (const audit of report.readinessAudits) {
+      if (audit.ok) {
+        console.log(chalk.green(`  ✔ ${audit.scenario}`));
+        continue;
+      }
+      // Build an actionable reason from whatever failed, instead of a bare ✖
+      // with no explanation (audit run-16 F4).
+      const reasons = [];
+      if (audit.contractFailure) reasons.push(audit.contractFailure);
+      if (typeof audit.status === "number" && audit.status >= 400) {
+        reasons.push(`HTTP ${audit.status}`);
+      }
+      if (audit.missingSelectors && audit.missingSelectors.length > 0) {
+        reasons.push(`missing selector(s): ${audit.missingSelectors.join(", ")}`);
+      }
+      if (audit.readyFailure) reasons.push(audit.readyFailure);
+      if (audit.ready === false && !audit.readyFailure) {
+        reasons.push("readiness signal not satisfied");
+      }
+      const reason = reasons.length > 0 ? reasons.join("; ") : "readiness check failed";
+      console.log(chalk.red(`  ✖ ${audit.scenario} — ${reason}`));
+    }
+    if (!report.ok) {
       console.log(
-        audit.ok
-          ? chalk.green(`  ✔ ${audit.scenario}`)
-          : chalk.red(`  ✖ ${audit.scenario}${audit.contractFailure ? ` — ${audit.contractFailure}` : ""}`),
+        chalk.gray(
+          "    (Target doctor audits the opt-in certified-target pipeline; it does not block `run`/`publish`.)",
+        ),
       );
     }
   }

package/src/commands/status.js

@@ -134,8 +134,12 @@ async function displayConfig(configSummary) {
 
 function renderJobs(jobs) {
   if (jobs.length === 0) {
-    console.log(chalk.gray("  No sync jobs found."));
-    console.log(chalk.gray("  Run `reshot sync` to upload traces."));
+    console.log(chalk.gray("  No trace-sync jobs found."));
+    console.log(
+      chalk.gray(
+        "  (This section tracks `reshot sync` trace jobs — your publishes appear in the dashboard Activity feed / `reshot pull`.)",
+      ),
+    );
     return;
   }
 

package/src/lib/certification.js

@@ -464,11 +464,17 @@ async function runDoctorTarget(options = {}) {
       );
     }
 
-    let readyContractOk = hasDeterministicReadyContract(scenario);
+    // A strict app-owned readiness CONTRACT is only required for the "certified"
+    // tier. For self-serve/local (custom) targets, the actual page-readiness
+    // audit below is what matters — don't fail the scenario (with no reason)
+    // just because it lacks a deterministic ready contract (audit run-18 R18-1).
+    const requireReadyContract = target.tier === "certified";
+    let readyContractOk = !requireReadyContract || hasDeterministicReadyContract(scenario);
     let contractFailure = null;
-    if (target.tier === "certified" && !readyContractOk) {
+    if (requireReadyContract && !hasDeterministicReadyContract(scenario)) {
+      readyContractOk = false;
       contractFailure = "Scenario is missing an app-owned readiness contract.";
-    } else if (target.tier === "certified" && usesSleepOnlyReadiness(scenario)) {
+    } else if (requireReadyContract && usesSleepOnlyReadiness(scenario)) {
       readyContractOk = false;
       contractFailure = "Scenario relies on waits without a deterministic ready contract.";
     }