@browserless/screenshot 13.6.10 → 13.6.11

package/package.json

@@ -2,7 +2,7 @@
   "name": "@browserless/screenshot",
   "description": "Capture high-quality screenshots of websites with overlay support, device emulation, and automated image optimization.",
   "homepage": "https://browserless.js.org/#/?id=screenshoturl-options",
-  "version": "13.6.10",
+  "version": "13.6.11",
   "main": "src/index.js",
   "author": {
     "email": "hello@microlink.io",
@@ -52,7 +52,7 @@
     "svg-gradient": "~1.0.4"
   },
   "devDependencies": {
-    "@browserless/test": "13.6.10",
+    "@browserless/test": "13.6.11",
     "ava": "5"
   },
   "engines": {
@@ -71,5 +71,5 @@
     "timeout": "2m",
     "workerThreads": false
   },
-  "gitHead": "0bdae87d4aff5b26de301b2947b70d8ae6c0dcfd"
+  "gitHead": "a1336c7b1837cd9fa0918a859f441ff5f74c1920"
 }

package/src/index.js

@@ -3,6 +3,7 @@
 const debug = require('debug-logfmt')('browserless:screenshot')
 const { isContextDestroyed } = require('@browserless/errors')
 const createGoto = require('@browserless/goto')
+const { extname } = require('node:path')
 const pReflect = require('p-reflect')
 
 const isWhiteScreenshot = require('./is-white-screenshot')
@@ -96,6 +97,14 @@ const SCREENSHOT_DEFAULT_OPTS = {
   isPageReady: defaultIsPageReady
 }
 
+const LOSSY_TYPES = new Set(['jpeg', 'webp'])
+const LOSSY_EXTENSIONS = new Set(['.jpg', '.jpeg', '.webp'])
+
+const isLossy = ({ type, path }) =>
+  type !== undefined
+    ? LOSSY_TYPES.has(type)
+    : LOSSY_EXTENSIONS.has(extname(path ?? '').toLowerCase())
+
 module.exports = ({ goto, ...gotoOpts }) => {
   goto = goto || createGoto(gotoOpts)
 
@@ -110,6 +119,8 @@ module.exports = ({ goto, ...gotoOpts }) => {
         ...opts
       } = {}
     ) => {
+      if (opts.quality !== undefined && !isLossy(opts)) opts.quality = undefined
+
       let screenshot
       let response