@reshotdev/screenshot 0.0.1-beta.21 → 0.0.1-beta.23

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reshotdev/screenshot",
-  "version": "0.0.1-beta.21",
+  "version": "0.0.1-beta.23",
   "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/publish.js

@@ -1569,6 +1569,12 @@ async function publishCommand(options = {}) {
   // Validate storage configuration
   const validation = validateStorageConfig(storageConfig);
 
+  // Print informational notices (e.g. "using platform mode") as gray info, not
+  // alarming ⚠ warnings (audit run-15 F4).
+  for (const note of validation.info || []) {
+    console.log(chalk.gray(`  ℹ ${note}`));
+  }
+
   // Print warnings
   for (const warning of validation.warnings) {
     console.log(chalk.yellow(`  ⚠ ${warning}`));

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/storage-providers.js

@@ -32,7 +32,14 @@ function validateStorageConfig(config) {
   const warnings = [];
   
   if (!config || !config.type) {
-    return { valid: true, errors: [], warnings: ['No storage configuration - using platform mode (requires auth)'] };
+    // Platform mode is the default/happy path (Reshot-hosted R2), not a
+    // misconfiguration — surface it as info, not a ⚠ warning (audit run-15 F4).
+    return {
+      valid: true,
+      errors: [],
+      warnings: [],
+      info: ["Using Reshot-hosted storage (platform mode)."],
+    };
   }
   
   switch (config.type) {