@reshotdev/screenshot 0.0.1-beta.14 → 0.0.1-beta.15

package/package.json

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

package/src/commands/publish.js

@@ -629,6 +629,24 @@ function buildPublishMetadata({
   };
 }
 
+/**
+ * Read pixel dimensions from a captured image so the platform can store and
+ * expose them (used by `reshot pull` for CLS-safe `<img width height>`
+ * embedding). Returns nulls for non-images or on any error — never throws.
+ */
+async function getImageDimensions(filePath, contentType) {
+  if (!filePath || !(contentType || "").startsWith("image/")) {
+    return { width: null, height: null };
+  }
+  try {
+    const sharp = require("sharp");
+    const meta = await sharp(filePath).metadata();
+    return { width: meta.width ?? null, height: meta.height ?? null };
+  } catch {
+    return { width: null, height: null };
+  }
+}
+
 /**
  * Publish using transactional flow (direct R2 upload with presigned URLs)
  */
@@ -915,12 +933,18 @@ async function publishWithTransactionalFlow(
         assets: [],
       });
     }
+    const dimensions = await getImageDimensions(
+      result.file.path,
+      result.file.contentType,
+    );
     groupMap.get(groupKey).assets.push({
       key: result.file.key,
       s3Path: result.s3Path,
       hash: result.file.hash,
       visualKey: result.file.visualKey,
       size: result.file.size,
+      width: dimensions.width,
+      height: dimensions.height,
       contentType: result.file.contentType,
       // Include diff data from CLI analysis
       diffPercentage: result.file.diffData?.diffPercentage ?? null,
@@ -1014,7 +1038,15 @@ async function publishWithTransactionalFlow(
         viewUrl = batchResult.viewUrl;
       }
     } catch (error) {
-      console.log(chalk.red(`     ✖ Batch request failed: ${error.message}`));
+      // Surface the server's descriptive error body (e.g. a validation
+      // message) instead of just axios's opaque "Request failed with status
+      // code 400" — otherwise a 1-line fix turns into a support ticket.
+      const serverMsg =
+        error.response?.data?.error || error.response?.data?.message;
+      const detail = serverMsg
+        ? `${error.message} — ${serverMsg}`
+        : error.message;
+      console.log(chalk.red(`     ✖ Batch request failed: ${detail}`));
       failCount += chunk.length;
     }
   }