zanachka · pull · Oct 20, 2025 · Oct 19, 2025 · Oct 19, 2025
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [10.9.8](https://github.com/microlinkhq/browserless/compare/v10.9.7...v10.9.8) (2025-10-19)
+
+### Bug Fixes
+
+* **screenshot:** white sampling ([#636](https://github.com/microlinkhq/browserless/issues/636)) ([e12515a](https://github.com/microlinkhq/browserless/commit/e12515a2c435620bedec26f62f91296b96d6d2b1))
+
 ## [10.9.7](https://github.com/microlinkhq/browserless/compare/v10.9.6...v10.9.7) (2025-10-07)
 
 **Note:** Version bump only for package browserless

diff --git a/lerna.json b/lerna.json
@@ -2,7 +2,7 @@
   "packages": [
     "packages/*"
   ],
-  "version": "10.9.7",
+  "version": "10.9.8",
   "command": {
     "bootstrap": {
       "npmClientArgs": [

diff --git a/packages/browserless/CHANGELOG.md b/packages/browserless/CHANGELOG.md
@@ -3,6 +3,10 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [10.9.8](https://github.com/microlinkhq/browserless/compare/v10.9.7...v10.9.8) (2025-10-19)
+
+**Note:** Version bump only for package browserless
+
 ## [10.9.7](https://github.com/microlinkhq/browserless/compare/v10.9.6...v10.9.7) (2025-10-07)
 
 **Note:** Version bump only for package browserless

diff --git a/packages/browserless/package.json b/packages/browserless/package.json
@@ -2,7 +2,7 @@
   "name": "browserless",
   "description": "The headless Chrome/Chromium performance driver for Node.js",
   "homepage": "https://browserless.js.org",
-  "version": "10.9.7",
+  "version": "10.9.8",
   "main": "src/index.js",
   "author": {
     "email": "hello@microlink.io",
@@ -32,9 +32,9 @@
   ],
   "dependencies": {
     "@browserless/errors": "^10.9.7",
-    "@browserless/goto": "^10.9.7",
-    "@browserless/pdf": "^10.9.7",
-    "@browserless/screenshot": "^10.9.7",
+    "@browserless/goto": "^10.9.8",
+    "@browserless/pdf": "^10.9.8",
+    "@browserless/screenshot": "^10.9.8",
     "debug-logfmt": "~1.4.5",
     "kill-process-group": "~1.0.13",
     "p-reflect": "~2.1.0",
@@ -44,7 +44,7 @@
     "superlock": "~1.2.4"
   },
   "devDependencies": {
-    "@browserless/test": "^10.9.7",
+    "@browserless/test": "^10.9.8",
     "ava": "5",
     "ps-list": "7",
     "tinyspawn": "latest"

diff --git a/packages/cli/CHANGELOG.md b/packages/cli/CHANGELOG.md
@@ -3,6 +3,10 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [10.9.8](https://github.com/microlinkhq/browserless/compare/v10.9.7...v10.9.8) (2025-10-19)
+
+**Note:** Version bump only for package @browserless/cli
+
 ## [10.9.7](https://github.com/microlinkhq/browserless/compare/v10.9.6...v10.9.7) (2025-10-07)
 
 **Note:** Version bump only for package @browserless/cli

diff --git a/packages/cli/package.json b/packages/cli/package.json
@@ -2,7 +2,7 @@
   "name": "@browserless/cli",
   "description": "CLI to interact with Browserless capabilities",
   "homepage": "https://browserless.js.org",
-  "version": "10.9.7",
+  "version": "10.9.8",
   "bin": {
     "browserless": "src/index.js"
   },
@@ -32,7 +32,7 @@
   ],
   "dependencies": {
     "beauty-error": "~1.2.21",
-    "browserless": "^10.9.7",
+    "browserless": "^10.9.8",
     "dark-mode": "~3.0.0",
     "dset": "~3.1.4",
     "mri": "~1.2.0",

diff --git a/packages/function/CHANGELOG.md b/packages/function/CHANGELOG.md
@@ -3,6 +3,10 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [10.9.8](https://github.com/microlinkhq/browserless/compare/v10.9.7...v10.9.8) (2025-10-19)
+
+**Note:** Version bump only for package @browserless/function
+
 ## [10.9.7](https://github.com/microlinkhq/browserless/compare/v10.9.6...v10.9.7) (2025-10-07)
 
 **Note:** Version bump only for package @browserless/function

diff --git a/packages/function/package.json b/packages/function/package.json
@@ -2,7 +2,7 @@
   "name": "@browserless/function",
   "description": "Run abritrary JavaScript inside a browser sandbox",
   "homepage": "https://browserless.js.org",
-  "version": "10.9.7",
+  "version": "10.9.8",
   "main": "src/index.js",
   "author": {
     "email": "hello@microlink.io",
@@ -34,7 +34,7 @@
     "require-one-of": "~1.0.24"
   },
   "devDependencies": {
-    "@browserless/test": "^10.9.7",
+    "@browserless/test": "^10.9.8",
     "acorn": "~8.15.0",
     "acorn-walk": "~8.3.4",
     "ava": "5",

diff --git a/packages/goto/CHANGELOG.md b/packages/goto/CHANGELOG.md
@@ -3,6 +3,10 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [10.9.8](https://github.com/microlinkhq/browserless/compare/v10.9.7...v10.9.8) (2025-10-19)
+
+**Note:** Version bump only for package @browserless/goto
+
 ## [10.9.7](https://github.com/microlinkhq/browserless/compare/v10.9.6...v10.9.7) (2025-10-07)
 
 **Note:** Version bump only for package @browserless/goto

diff --git a/packages/goto/package.json b/packages/goto/package.json
@@ -2,7 +2,7 @@
   "name": "@browserless/goto",
   "description": "Go to a page aborting unnecessary requests",
   "homepage": "https://browserless.js.org/#/?id=gotopage-options",
-  "version": "10.9.7",
+  "version": "10.9.8",
   "main": "src/index.js",
   "author": {
     "email": "hello@microlink.io",
@@ -40,7 +40,7 @@
     "tough-cookie": "~6.0.0"
   },
   "devDependencies": {
-    "@browserless/test": "^10.9.7",
+    "@browserless/test": "^10.9.8",
     "ava": "5",
     "p-wait-for": "3"
   },

diff --git a/packages/lighthouse/CHANGELOG.md b/packages/lighthouse/CHANGELOG.md
@@ -3,6 +3,10 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [10.9.8](https://github.com/microlinkhq/browserless/compare/v10.9.7...v10.9.8) (2025-10-19)
+
+**Note:** Version bump only for package @browserless/lighthouse
+
 ## [10.9.7](https://github.com/microlinkhq/browserless/compare/v10.9.6...v10.9.7) (2025-10-07)
 
 **Note:** Version bump only for package @browserless/lighthouse

diff --git a/packages/lighthouse/package.json b/packages/lighthouse/package.json
@@ -2,7 +2,7 @@
   "name": "@browserless/lighthouse",
   "description": "Browserless Lighthouse integration using puppeteer.",
   "homepage": "https://browserless.js.org",
-  "version": "10.9.7",
+  "version": "10.9.8",
   "main": "src",
   "repository": {
     "type": "git",
@@ -27,7 +27,7 @@
     "lighthouse": "~12.8.2"
   },
   "devDependencies": {
-    "@browserless/test": "^10.9.7",
+    "@browserless/test": "^10.9.8",
     "ava": "5"
   },
   "engines": {

diff --git a/packages/pdf/CHANGELOG.md b/packages/pdf/CHANGELOG.md
@@ -3,6 +3,10 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [10.9.8](https://github.com/microlinkhq/browserless/compare/v10.9.7...v10.9.8) (2025-10-19)
+
+**Note:** Version bump only for package @browserless/pdf
+
 ## [10.9.7](https://github.com/microlinkhq/browserless/compare/v10.9.6...v10.9.7) (2025-10-07)
 
 **Note:** Version bump only for package @browserless/pdf

diff --git a/packages/pdf/package.json b/packages/pdf/package.json
@@ -2,7 +2,7 @@
   "name": "@browserless/pdf",
   "description": "Sensible good defaults for exporting a website as PDF",
   "homepage": "https://browserless.js.org/#/?id=pdfurl-options",
-  "version": "10.9.7",
+  "version": "10.9.8",
   "main": "src",
   "repository": {
     "directory": "packages/pdf",
@@ -24,8 +24,8 @@
     "screen"
   ],
   "dependencies": {
-    "@browserless/goto": "^10.9.7",
-    "@browserless/screenshot": "^10.9.7",
+    "@browserless/goto": "^10.9.8",
+    "@browserless/screenshot": "^10.9.8",
     "@kikobeats/time-span": "~1.0.9",
     "debug-logfmt": "~1.4.5",
     "pretty-ms": "~7.0.1"

diff --git a/packages/screencast/CHANGELOG.md b/packages/screencast/CHANGELOG.md
@@ -3,6 +3,10 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [10.9.8](https://github.com/microlinkhq/browserless/compare/v10.9.7...v10.9.8) (2025-10-19)
+
+**Note:** Version bump only for package @browserless/screencast
+
 ## [10.9.7](https://github.com/microlinkhq/browserless/compare/v10.9.6...v10.9.7) (2025-10-07)
 
 **Note:** Version bump only for package @browserless/screencast

diff --git a/packages/screencast/package.json b/packages/screencast/package.json
@@ -2,7 +2,7 @@
   "name": "@browserless/screencast",
   "description": "Browserless frame capturing using puppeteer",
   "homepage": "https://browserless.js.org/#/?id=screencasturl-options",
-  "version": "10.9.7",
+  "version": "10.9.8",
   "main": "src/index.js",
   "author": {
     "email": "hello@microlink.io",
@@ -36,7 +36,7 @@
     "null-prototype-object": "~1.2.3"
   },
   "devDependencies": {
-    "@browserless/test": "^10.9.7",
+    "@browserless/test": "^10.9.8",
     "@kikobeats/time-span": "latest",
     "@skyra/gifenc": "latest",
     "ava": "5",

diff --git a/packages/screenshot/CHANGELOG.md b/packages/screenshot/CHANGELOG.md
@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [10.9.8](https://github.com/microlinkhq/browserless/compare/v10.9.7...v10.9.8) (2025-10-19)
+
+### Bug Fixes
+
+* **screenshot:** white sampling ([#636](https://github.com/microlinkhq/browserless/issues/636)) ([e12515a](https://github.com/microlinkhq/browserless/commit/e12515a2c435620bedec26f62f91296b96d6d2b1))
+
 ## [10.9.7](https://github.com/microlinkhq/browserless/compare/v10.9.6...v10.9.7) (2025-10-07)
 
 **Note:** Version bump only for package @browserless/screenshot

diff --git a/packages/screenshot/package.json b/packages/screenshot/package.json
@@ -2,7 +2,7 @@
   "name": "@browserless/screenshot",
   "description": "Take a clean screenshot of any website",
   "homepage": "https://browserless.js.org/#/?id=screenshoturl-options",
-  "version": "10.9.7",
+  "version": "10.9.8",
   "main": "src/index.js",
   "author": {
     "email": "hello@microlink.io",
@@ -28,7 +28,7 @@
     "screenshot"
   ],
   "dependencies": {
-    "@browserless/goto": "^10.9.7",
+    "@browserless/goto": "^10.9.8",
     "@kikobeats/content-type": "~1.0.1",
     "@kikobeats/time-span": "~1.0.9",
     "debug-logfmt": "~1.4.5",
@@ -46,7 +46,7 @@
     "svg-gradient": "~1.0.3"
   },
   "devDependencies": {
-    "@browserless/test": "^10.9.7",
+    "@browserless/test": "^10.9.8",
     "ava": "5",
     "cheerio": "latest"
   },

diff --git a/packages/screenshot/src/is-white-screenshot.js b/packages/screenshot/src/is-white-screenshot.js
@@ -3,20 +3,30 @@
 const { Jimp } = require('jimp')
 
 module.exports = async uint8array => {
-  const image = await Jimp.fromBuffer(Buffer.from(uint8array))
-  const firstPixel = image.getPixelColor(0, 0)
-  const height = image.bitmap.height
-  const width = image.bitmap.width
+  try {
+    const image = await Jimp.fromBuffer(Buffer.from(uint8array))
+    const firstPixel = image.getPixelColor(0, 0)
+    const height = image.bitmap.height
+    const width = image.bitmap.width
 
-  const samplePercentage = 0.25 // Sample 25% of the image
-  const sampleSize = Math.floor(width * height * samplePercentage) // Calculate sample size based on percentage
-  const stepSize = Math.max(1, Math.floor((width * height) / sampleSize)) // Calculate step size based on sample size
+    // For 2D grid sampling, calculate stepSize to achieve approximately the target sample percentage.
+    // When sampling every 'stepSize' pixels in both dimensions, actual samples = (height/stepSize) * (width/stepSize).
+    // To achieve samplePercentage, we need: (h*w)/(stepSize²) ≈ samplePercentage*(h*w)
+    // Therefore: stepSize ≈ sqrt(1 / samplePercentage)
+    const samplePercentage = 0.25 // Sample ~25% of the image
+    const stepSize = Math.max(1, Math.ceil(Math.sqrt(1 / samplePercentage)))
 
-  for (let i = 0; i < height; i += stepSize) {
-    for (let j = 0; j < width; j += stepSize) {
-      if (firstPixel !== image.getPixelColor(j, i)) return false
+    for (let i = 0; i < height; i += stepSize) {
+      for (let j = 0; j < width; j += stepSize) {
+        if (firstPixel !== image.getPixelColor(j, i)) return false
+      }
     }
-  }
 
-  return true
+    return true
+  } catch (error) {
+    if (error.message.includes('maxMemoryUsageInMB')) {
+      return false
+    }
+    throw error
+  }
 }
diff --git a/packages/screenshot/test/is-white-screenshot.js b/packages/screenshot/test/is-white-screenshot.js
@@ -3,8 +3,36 @@
 const test = require('ava')
 const { readFile } = require('fs/promises')
 
+const { Jimp } = require('jimp')
+
 const isWhite = require('../src/is-white-screenshot')
 
+const createJimpSpy = () => {
+  const originalFromBuffer = Jimp.fromBuffer
+  const spy = { callCount: 0 }
+
+  const wrappedFromBuffer = async function (buffer, options) {
+    const image = await originalFromBuffer.call(this, buffer, options)
+    const originalGetPixelColor = image.getPixelColor.bind(image)
+
+    image.getPixelColor = function (x, y) {
+      spy.callCount++
+      return originalGetPixelColor(x, y)
+    }
+
+    return image
+  }
+
+  Jimp.fromBuffer = wrappedFromBuffer
+
+  return {
+    spy,
+    restore: () => {
+      Jimp.fromBuffer = originalFromBuffer
+    }
+  }
+}
+
 test('true', async t => {
   t.true(await isWhite(await readFile('./test/fixtures/white-5k.jpg')))
   t.true(await isWhite(await readFile('./test/fixtures/white-5k.png')))
@@ -14,3 +42,38 @@ test('false', async t => {
   t.false(await isWhite(await readFile('./test/fixtures/no-white-5k.jpg')))
   t.false(await isWhite(await readFile('./test/fixtures/no-white-5k.png')))
 })
+
+test('sampling algorithm correctly samples ~25% of pixels', async t => {
+  const imageBuffer = await readFile('./test/fixtures/white-5k.png')
+  const tempImage = await Jimp.fromBuffer(imageBuffer)
+  const totalPixels = tempImage.bitmap.width * tempImage.bitmap.height
+  const { spy, restore } = createJimpSpy()
+
+  await isWhite(imageBuffer)
+  restore()
+
+  const percentageChecked = (spy.callCount / totalPixels) * 100
+
+  t.true(
+    percentageChecked >= 20 && percentageChecked <= 30,
+    `Expected to check ~25% of pixels, but checked ${percentageChecked.toFixed(2)}% (${
+      spy.callCount
+    }/${totalPixels})`
+  )
+})
+
+test('handles memory errors gracefully on very large images', async t => {
+  const { createCanvas } = require('canvas')
+
+  const width = 10000
+  const height = 10000
+  const canvas = createCanvas(width, height)
+  const ctx = canvas.getContext('2d')
+
+  ctx.fillStyle = '#ffffff'
+  ctx.fillRect(0, 0, width, height)
+
+  const largeImageBuffer = canvas.toBuffer('image/jpeg', { quality: 0.9 })
+  const result = await isWhite(largeImageBuffer)
+  t.is(typeof result, 'boolean', 'Should return a boolean, not throw an error')
+})