Tests: fix JP2 suite with global libvips (#4477)

Takes advantage of libvips' improved ICC handling

.cirrus.yml

@@ -1,5 +1,5 @@
 freebsd_instance:
-  image_family: freebsd-15-0-snap
+  image_family: freebsd-15-0
 
 task:
   name: FreeBSD
@@ -9,9 +9,10 @@ task:
   prerequisites_script:
     - pkg update -f
     - pkg upgrade -y
-    - pkg install -y devel/git devel/pkgconf graphics/vips www/node20 www/npm
+    - pkg install -y devel/git devel/pkgconf graphics/vips www/node22 www/npm
     - pkg-config --modversion vips-cpp
   install_script:
-    - npm install --build-from-source
+    - npm install
+    - npm run build
   test_script:
-    - npx mocha --no-config --spec=test/unit/io.js --timeout=30000
+    - node --test test/unit/io.js

.editorconfig

@@ -7,6 +7,4 @@ indent_size = 2
 charset = utf-8
 trim_trailing_whitespace = true
 insert_final_newline = true
-
+max_line_length = 120
-[*.md]
-trim_trailing_whitespace = false

.github/CONTRIBUTING.md

@@ -6,8 +6,6 @@ Hello, thank you for your interest in helping!
 
 Please create a [new issue](https://github.com/lovell/sharp/issues/new) containing the steps to reproduce the problem.
 
-If you're having installation problems, please include the output of running `npm install --verbose sharp`.
-
 New bugs are assigned a `triage` label whilst under investigation.
 
 ## Submit a new feature request
@@ -16,7 +14,7 @@ If a [similar request](https://github.com/lovell/sharp/labels/enhancement) exist
 it's probably fastest to add a comment to it about your requirement.
 
 Implementation is usually straightforward if libvips
-[already supports](https://www.libvips.org/API/current/func-list.html)
+[already supports](https://www.libvips.org/API/current/function-list.html)
 the feature you need.
 
 ## Submit a Pull Request to fix a bug
@@ -27,12 +25,11 @@ Please select the `main` branch as the destination for your Pull Request so your
 
 Please squash your changes into a single commit using a command like `git rebase -i upstream/main`.
 
-To test C++ changes, you can compile the module using `npm install --build-from-source` and then run the tests using `npm test`.
+To test C++ changes, you can compile the module using `npm run build` and then run the tests using `npm test`.
 
 ## Submit a Pull Request with a new feature
 
 Please add JavaScript [unit tests](https://github.com/lovell/sharp/tree/main/test/unit) to cover your new feature.
-A test coverage report for the JavaScript code is generated in the `coverage/lcov-report` directory.
 Please also update the [TypeScript definitions](https://github.com/lovell/sharp/tree/main/lib/index.d.ts), along with the [type definition tests](https://github.com/lovell/sharp/tree/main/test/types/sharp.test-d.ts).
 
 Where possible, the functional tests use gradient-based perceptual hashes

.github/workflows/ci.yml

@@ -4,9 +4,23 @@ on:
   - pull_request
 permissions: {}
 jobs:
+  lint:
+    permissions:
+      contents: read
+    runs-on: ubuntu-24.04
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v5
+        with:
+          node-version: "24"
+      - run: npm install --ignore-scripts
+      - run: npm run lint-cpp
+      - run: npm run lint-js
+      - run: npm run lint-types
   build-native:
     permissions:
       contents: read
+    needs: lint
     name: "build-${{ matrix.platform }} [Node.js ${{ matrix.nodejs_version_major }}] ${{ matrix.package && '[package]' }}"
     runs-on: ${{ matrix.os }}
     container: ${{ matrix.container }}
@@ -59,34 +73,34 @@ jobs:
             nodejs_version: "^20.3.0"
             nodejs_version_major: 20
             platform: linux-arm64
-          - os: macos-13
+          - os: macos-15-intel
             nodejs_arch: x64
             nodejs_version: "^18.17.0"
             nodejs_version_major: 18
             platform: darwin-x64
             package: true
-          - os: macos-13
+          - os: macos-15-intel
             nodejs_arch: x64
             nodejs_version: "^20.3.0"
             nodejs_version_major: 20
             platform: darwin-x64
-          - os: macos-13
+          - os: macos-15-intel
             nodejs_arch: x64
             nodejs_version: "^22.9.0"
             nodejs_version_major: 22
             platform: darwin-x64
-          - os: macos-14
+          - os: macos-15
             nodejs_arch: arm64
             nodejs_version: "^18.17.0"
             nodejs_version_major: 18
             platform: darwin-arm64
             package: true
-          - os: macos-14
+          - os: macos-15
             nodejs_arch: arm64
             nodejs_version: "^20.3.0"
             nodejs_version_major: 20
             platform: darwin-arm64
-          - os: macos-14
+          - os: macos-15
             nodejs_arch: arm64
             nodejs_version: "^22.9.0"
             nodejs_version_major: 22
@@ -150,17 +164,15 @@ jobs:
           python-version: "3.12"
       - name: Dependencies (Node.js)
         if: "!contains(matrix.platform, 'linuxmusl')"
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           node-version: ${{ matrix.nodejs_version }}
           architecture: ${{ matrix.nodejs_arch }}
       - uses: actions/checkout@v4
-      - name: Install
+      - run: npm install
-        run: npm install --build-from-source
+      - run: npm run build
-      - name: Test
+      - run: npm run test-unit
-        run: npm test
+      - if: matrix.package
-      - name: Populate npm package
-        if: matrix.package
         run: npm run package-from-local-build
       - uses: actions/upload-artifact@v4
         if: matrix.package
@@ -172,6 +184,7 @@ jobs:
   build-linuxmusl-arm64:
     permissions:
       contents: read
+    needs: lint
     name: "build-linuxmusl-arm64 [Node.js ${{ matrix.nodejs_version_major }}] ${{ matrix.package && '[package]' }}"
     runs-on: ubuntu-24.04-arm
     container:
@@ -199,12 +212,10 @@ jobs:
       - name: Dependencies
         run: apk add build-base git python3 font-noto --update-cache
       - uses: actions/checkout@v4
-      - name: Install
+      - run: npm install
-        run: npm install --build-from-source
+      - run: npm run build
-      - name: Test
+      - run: npm run test-unit
-        run: npm test
+      - if: matrix.package
-      - name: Populate npm package
-        if: matrix.package
         run: npm run package-from-local-build
       - uses: actions/upload-artifact@v4
         if: matrix.package
@@ -216,6 +227,7 @@ jobs:
   build-qemu:
     permissions:
       contents: read
+    needs: lint
     name: "build-${{ matrix.platform }} [Node.js ${{ matrix.nodejs_version_major }}] [package]"
     runs-on: ubuntu-24.04
     strategy:
@@ -223,40 +235,49 @@ jobs:
       matrix:
         include:
           - platform: linux-arm
-            distro: bullseye
+            base_image: "balenalib/rpi-raspbian:bullseye"
-            run_on_arch: armv6
             nodejs_arch: armv6l
             nodejs_hostname: unofficial-builds.nodejs.org
             nodejs_version: "18.17.0"
             nodejs_version_major: 18
           - platform: linux-s390x
-            distro: bookworm
+            base_image: "--platform=linux/s390x s390x/debian:bookworm"
-            run_on_arch: s390x
             nodejs_arch: s390x
             nodejs_hostname: nodejs.org
             nodejs_version: "18.17.0"
             nodejs_version_major: 18
           - platform: linux-ppc64
-            distro: bookworm
+            base_image: "--platform=linux/ppc64le ppc64le/debian:bookworm"
-            run_on_arch: ppc64le
             nodejs_arch: ppc64le
             nodejs_hostname: nodejs.org
             nodejs_version: "18.17.0"
             nodejs_version_major: 18
+          - platform: linux-riscv64
+            base_image: "--platform=linux/riscv64 riscv64/debian:trixie"
+            compiler_flags: "-march=rv64gc"
+            nodejs_arch: riscv64
+            nodejs_hostname: unofficial-builds.nodejs.org
+            nodejs_version: "20.19.5"
+            nodejs_version_major: 20
     steps:
       - uses: actions/checkout@v4
       - uses: uraimo/run-on-arch-action@v3
         with:
-          arch: ${{ matrix.run_on_arch }}
+          arch: none
-          distro: ${{ matrix.distro }}
+          distro: none
+          base_image: ${{ matrix.base_image }}
+          env: |
+            CFLAGS: "${{ matrix.compiler_flags }}"
+            CXXFLAGS: "${{ matrix.compiler_flags }}"
           run: |
             apt-get update
             apt-get install -y curl g++ git libatomic1 make python3 xz-utils
             mkdir /opt/nodejs
             curl --silent https://${{ matrix.nodejs_hostname }}/download/release/v${{ matrix.nodejs_version}}/node-v${{ matrix.nodejs_version}}-linux-${{ matrix.nodejs_arch }}.tar.xz | tar xJC /opt/nodejs --strip-components=1
             export PATH=$PATH:/opt/nodejs/bin
-            npm install --build-from-source
+            npm install
-            npx mocha --no-config --spec=test/unit/io.js --timeout=30000
+            npm run build
+            node --test test/unit/io.js
             npm run package-from-local-build
       - uses: actions/upload-artifact@v4
         with:
@@ -267,19 +288,20 @@ jobs:
   build-emscripten:
     permissions:
       contents: read
+    needs: lint
     name: "build-wasm32 [package]"
     runs-on: ubuntu-24.04
-    container: "emscripten/emsdk:4.0.14"
+    container: "emscripten/emsdk:4.0.18"
     steps:
       - uses: actions/checkout@v4
       - name: Dependencies
         run: apt-get update && apt-get install -y pkg-config
       - name: Dependencies (Node.js)
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v5
         with:
           node-version: "20"
-      - name: Install
+      - run: npm install
-        run: emmake npm install --build-from-source
+      - run: emmake npm run build
       - name: Verify emscripten versions match
         run: |
           EMSCRIPTEN_VERSION_LIBVIPS=$(node -p "require('@img/sharp-libvips-dev-wasm32/versions').emscripten")
@@ -287,10 +309,8 @@ jobs:
           echo "libvips built with emscripten $EMSCRIPTEN_VERSION_LIBVIPS"
           echo "sharp built with emscripten $EMSCRIPTEN_VERSION_SHARP"
           test "$EMSCRIPTEN_VERSION_LIBVIPS" = "$EMSCRIPTEN_VERSION_SHARP"
-      - name: Test
+      - run: emmake npm run test-unit
-        run: emmake npm test
+      - run: emmake npm run package-from-local-build
-      - name: Populate npm package
-        run: emmake npm run package-from-local-build
       - uses: actions/upload-artifact@v4
         with:
           name: wasm32
@@ -314,7 +334,7 @@ jobs:
           path: npm
       - name: Create npm workspace tarball
         run: tar -vcaf npm-workspace.tar.xz --directory npm --exclude=from-local-build.js .
-      - uses: actions/setup-node@v4
+      - uses: actions/setup-node@v5
         with:
           node-version: '24'
       - name: Create release notes

.github/workflows/npm.yml

@@ -43,30 +43,30 @@ jobs:
             runtime: bun
 
           - name: darwin-x64-node-npm
-            runs-on: macos-13
+            runs-on: macos-15-intel
             runtime: node
             package-manager: npm
           - name: darwin-x64-node-pnpm
-            runs-on: macos-13
+            runs-on: macos-15-intel
             runtime: node
             package-manager: pnpm
           - name: darwin-x64-node-yarn
-            runs-on: macos-13
+            runs-on: macos-15-intel
             runtime: node
             package-manager: yarn
           - name: darwin-x64-node-yarn-pnp
-            runs-on: macos-13
+            runs-on: macos-15-intel
             runtime: node
             package-manager: yarn-pnp
           - name: darwin-x64-node-yarn-v1
-            runs-on: macos-13
+            runs-on: macos-15-intel
             runtime: node
             package-manager: yarn-v1
           - name: darwin-x64-deno
-            runs-on: macos-13
+            runs-on: macos-15-intel
             runtime: deno
           - name: darwin-x64-bun
-            runs-on: macos-13
+            runs-on: macos-15-intel
             runtime: bun
 
           - name: win32-x64-node-npm
@@ -185,7 +185,9 @@ jobs:
 
       - name: Run with Deno
         if: ${{ matrix.runtime == 'deno' }}
-        run: deno run --allow-read --allow-ffi release.mjs
+        run: |
+          deno install
+          deno run --allow-env --allow-ffi --allow-read --allow-sys release.mjs
 
       - name: Run with Bun
         if: ${{ matrix.runtime == 'bun' }}

.mocharc.jsonc

@@ -1,7 +0,0 @@
-{
-  "parallel": true,
-  "slow": 1000,
-  "timeout": 30000,
-  "require": "./test/beforeEach.js",
-  "spec": "./test/unit/*.js"
-}

biome.json

@@ -0,0 +1,26 @@
+{
+  "$schema": "https://biomejs.dev/schemas/2.3.4/schema.json",
+  "vcs": {
+    "enabled": true,
+    "clientKind": "git",
+    "useIgnoreFile": true
+  },
+  "files": {
+    "ignoreUnknown": true
+  },
+  "linter": {
+    "enabled": true,
+    "rules": {
+      "recommended": true
+    }
+  },
+  "formatter": {
+    "enabled": false,
+    "useEditorconfig": true
+  },
+  "javascript": {
+    "formatter": {
+      "quoteStyle": "single"
+    }
+  }
+}

docs/astro.config.mjs

@@ -1,6 +1,6 @@
 // @ts-check
-import { defineConfig } from 'astro/config';
 import starlight from '@astrojs/starlight';
+import { defineConfig } from 'astro/config';
 import starlightAutoSidebar from 'starlight-auto-sidebar';
 
 import { version } from '../package.json';

docs/build.mjs

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 import fs from 'node:fs/promises';
 import path from 'node:path';

docs/package.json

@@ -11,8 +11,8 @@
     "astro": "astro"
   },
   "dependencies": {
-    "@astrojs/starlight": "^0.35.2",
+    "@astrojs/starlight": "^0.36.2",
-    "astro": "^5.13.5",
+    "astro": "^5.15.3",
-    "starlight-auto-sidebar": "^0.1.2"
+    "starlight-auto-sidebar": "^0.1.3"
   }
 }

docs/src/content/docs/api-channel.md

@@ -8,7 +8,7 @@ title: Channel manipulation
 
 Remove alpha channels, if any. This is a no-op if the image does not have an alpha channel.
 
-See also [flatten](/api-operation#flatten).
+See also [flatten](/api-operation/#flatten).
 
 
 **Example**  

docs/src/content/docs/api-colour.md

@@ -80,7 +80,7 @@ as defined by [toColourspace](#tocolourspace).
 
 | Param | Type | Description |
 | --- | --- | --- |
-| [colourspace] | <code>string</code> | pipeline colourspace e.g. `rgb16`, `scrgb`, `lab`, `grey16` [...](https://github.com/libvips/libvips/blob/41cff4e9d0838498487a00623462204eb10ee5b8/libvips/iofuncs/enumtypes.c#L774) |
+| [colourspace] | <code>string</code> | pipeline colourspace e.g. `rgb16`, `scrgb`, `lab`, `grey16` [...](https://www.libvips.org/API/current/enum.Interpretation.html) |
 
 **Example**  
 ```js
@@ -123,7 +123,7 @@ By default output image will be web-friendly sRGB, with additional channels inte
 
 | Param | Type | Description |
 | --- | --- | --- |
-| [colourspace] | <code>string</code> | output colourspace e.g. `srgb`, `rgb`, `cmyk`, `lab`, `b-w` [...](https://github.com/libvips/libvips/blob/3c0bfdf74ce1dc37a6429bed47fa76f16e2cd70a/libvips/iofuncs/enumtypes.c#L777-L794) |
+| [colourspace] | <code>string</code> | output colourspace e.g. `srgb`, `rgb`, `cmyk`, `lab`, `b-w` [...](https://www.libvips.org/API/current/enum.Interpretation.html) |
 
 **Example**  
 ```js

docs/src/content/docs/api-composite.md

@@ -21,7 +21,7 @@ The `blend` option can be one of `clear`, `source`, `over`, `in`, `out`, `atop`,
 `hard-light`, `soft-light`, `difference`, `exclusion`.
 
 More information about blend modes can be found at
-https://www.libvips.org/API/current/libvips-conversion.html#VipsBlendMode
+https://www.libvips.org/API/current/enum.BlendMode.html
 and https://www.cairographics.org/operators/
 
 
@@ -64,8 +64,8 @@ and https://www.cairographics.org/operators/
 | [images[].raw.height] | <code>Number</code> |  |  |
 | [images[].raw.channels] | <code>Number</code> |  |  |
 | [images[].animated] | <code>boolean</code> | <code>false</code> | Set to `true` to read all frames/pages of an animated image. |
-| [images[].failOn] | <code>string</code> | <code>&quot;&#x27;warning&#x27;&quot;</code> | @see [constructor parameters](/api-constructor#parameters) |
+| [images[].failOn] | <code>string</code> | <code>&quot;&#x27;warning&#x27;&quot;</code> | @see [constructor parameters](/api-constructor/) |
-| [images[].limitInputPixels] | <code>number</code> \| <code>boolean</code> | <code>268402689</code> | @see [constructor parameters](/api-constructor#parameters) |
+| [images[].limitInputPixels] | <code>number</code> \| <code>boolean</code> | <code>268402689</code> | @see [constructor parameters](/api-constructor/) |
 
 **Example**  
 ```js

docs/src/content/docs/api-input.md

@@ -13,7 +13,7 @@ It does not take into consideration any operations to be applied to the output i
 such as resize or rotate.
 
 Dimensions in the response will respect the `page` and `pages` properties of the
-[constructor parameters](/api-constructor#parameters).
+[constructor parameters](/api-constructor/).
 
 A `Promise` is returned when `callback` is not provided.
 
@@ -21,9 +21,9 @@ A `Promise` is returned when `callback` is not provided.
 - `size`: Total size of image in bytes, for Stream and Buffer input only
 - `width`: Number of pixels wide (EXIF orientation is not taken into consideration, see example below)
 - `height`: Number of pixels high (EXIF orientation is not taken into consideration, see example below)
-- `space`: Name of colour space interpretation e.g. `srgb`, `rgb`, `cmyk`, `lab`, `b-w` [...](https://www.libvips.org/API/current/VipsImage.html#VipsInterpretation)
+- `space`: Name of colour space interpretation e.g. `srgb`, `rgb`, `cmyk`, `lab`, `b-w` [...](https://www.libvips.org/API/current/enum.Interpretation.html)
 - `channels`: Number of bands e.g. `3` for sRGB, `4` for CMYK
-- `depth`: Name of pixel depth format e.g. `uchar`, `char`, `ushort`, `float` [...](https://www.libvips.org/API/current/VipsImage.html#VipsBandFormat)
+- `depth`: Name of pixel depth format e.g. `uchar`, `char`, `ushort`, `float` [...](https://www.libvips.org/API/current/enum.BandFormat.html)
 - `density`: Number of pixels per inch (DPI), if present
 - `chromaSubsampling`: String containing JPEG chroma subsampling, `4:2:0` or `4:4:4` for RGB, `4:2:0:4` or `4:4:4:4` for CMYK
 - `isProgressive`: Boolean indicating whether the image is interlaced using a progressive scan

docs/src/content/docs/api-operation.md

@@ -179,7 +179,7 @@ When used without parameters, performs a fast, mild sharpen of the output image.
 When a `sigma` is provided, performs a slower, more accurate sharpen of the L channel in the LAB colour space.
 Fine-grained control over the level of sharpening in "flat" (m1) and "jagged" (m2) areas is available.
 
-See [libvips sharpen](https://www.libvips.org/API/current/libvips-convolution.html#vips-sharpen) operation.
+See [libvips sharpen](https://www.libvips.org/API/current/method.Image.sharpen.html) operation.
 
 
 **Throws**:

docs/src/content/docs/api-output.md

@@ -201,7 +201,7 @@ const dataWithMergedExif = await sharp(inputWithExif)
 
 Keep ICC profile from the input image in the output image.
 
-Where necessary, will attempt to convert the output colour space to match the profile.
+When input and output colour spaces differ, use with [toColourspace](/api-colour/#tocolourspace) and optionally [pipelineColourspace](/api-colour/#pipelinecolourspace).
 
 
 **Since**: 0.33.0  
@@ -211,6 +211,14 @@ const outputWithIccProfile = await sharp(inputWithIccProfile)
   .keepIccProfile()
   .toBuffer();
 ```
+**Example**  
+```js
+const cmykOutputWithIccProfile = await sharp(cmykInputWithIccProfile)
+  .pipelineColourspace('cmyk')
+  .toColourspace('cmyk')
+  .keepIccProfile()
+  .toBuffer();
+```
 
 
 ## withIccProfile
@@ -430,7 +438,7 @@ Indexed PNG input at 1, 2 or 4 bits per pixel is converted to 8 bits per pixel.
 Set `palette` to `true` for slower, indexed PNG output.
 
 For 16 bits per pixel output, convert to `rgb16` via
-[toColourspace](/api-colour#tocolourspace).
+[toColourspace](/api-colour/#tocolourspace).
 
 
 **Throws**:
@@ -589,7 +597,7 @@ Use these JP2 options for output image.
 
 Requires libvips compiled with support for OpenJPEG.
 The prebuilt binaries do not include this - see
-[installing a custom libvips](https://sharp.pixelplumbing.com/install#custom-libvips).
+[installing a custom libvips](/install#custom-libvips).
 
 
 **Throws**:
@@ -646,6 +654,7 @@ instead of providing `xres` and `yres` in pixels/mm.
 | [options.quality] | <code>number</code> | <code>80</code> | quality, integer 1-100 |
 | [options.force] | <code>boolean</code> | <code>true</code> | force TIFF output, otherwise attempt to use input format |
 | [options.compression] | <code>string</code> | <code>&quot;&#x27;jpeg&#x27;&quot;</code> | compression options: none, jpeg, deflate, packbits, ccittfax4, lzw, webp, zstd, jp2k |
+| [options.bigtiff] | <code>boolean</code> | <code>false</code> | use BigTIFF variant (has no effect when compression is none) |
 | [options.predictor] | <code>string</code> | <code>&quot;&#x27;horizontal&#x27;&quot;</code> | compression predictor options: none, horizontal, float |
 | [options.pyramid] | <code>boolean</code> | <code>false</code> | write an image pyramid |
 | [options.tile] | <code>boolean</code> | <code>false</code> | write a tiled tiff |
@@ -753,7 +762,7 @@ This feature is experimental, please do not use in production systems.
 
 Requires libvips compiled with support for libjxl.
 The prebuilt binaries do not include this - see
-[installing a custom libvips](https://sharp.pixelplumbing.com/install#custom-libvips).
+[installing a custom libvips](/install/#custom-libvips).
 
 
 **Throws**:

docs/src/content/docs/api-utility.md

@@ -114,7 +114,7 @@ e.g. libaom manages its own 4 threads when encoding AVIF images,
 and these are independent of the value set here.
 
 :::note
-Further [control over performance](/performance) is available.
+Further [control over performance](/performance/) is available.
 :::
 
 

docs/src/content/docs/changelog/v0.34.4.md

@@ -1,5 +1,5 @@
 ---
-title: v0.34.4 - TBD
+title: v0.34.4 - 17th September 2025
 slug: changelog/v0.34.4
 ---
 

docs/src/content/docs/changelog/v0.34.5.md

@@ -0,0 +1,21 @@
+---
+title: v0.34.5 - 6th November 2025
+slug: changelog/v0.34.5
+---
+
+* Upgrade to libvips v8.17.3 for upstream bug fixes.
+
+* Add experimental support for prebuilt Linux RISC-V 64-bit binaries.
+
+* Support building from source with npm v12+, deprecate `--build-from-source` flag.
+  [#4458](https://github.com/lovell/sharp/issues/4458)
+
+* Add support for BigTIFF output.
+  [#4459](https://github.com/lovell/sharp/pull/4459)
+  [@throwbi](https://github.com/throwbi)
+
+* Improve error messaging when only warnings issued.
+  [#4465](https://github.com/lovell/sharp/issues/4465)
+
+* Simplify ICC processing when retaining input profiles.
+  [#4468](https://github.com/lovell/sharp/issues/4468)

docs/src/content/docs/install.md

@@ -20,7 +20,7 @@ npm install sharp
 pnpm add sharp
 ```
 
-When using `pnpm`, you may need to add `sharp` to
+When using `pnpm`, add `sharp` to
 [ignoredBuiltDependencies](https://pnpm.io/settings#ignoredbuiltdependencies)
 to silence warnings.
 
@@ -33,7 +33,8 @@ bun add sharp
 ```
 
 ```sh frame="none"
-deno run --allow-ffi ...
+deno add --quiet npm:sharp
+deno run --allow-env --allow-ffi --allow-read --allow-sys ...
 ```
 
 ## Prerequisites
@@ -48,6 +49,7 @@ Ready-compiled sharp and libvips binaries are provided for use on the most commo
 * macOS ARM64
 * Linux ARM (glibc >= 2.31)
 * Linux ARM64 (glibc >= 2.26, musl >= 1.2.2)
+* Linux RISC-V 64-bit (glibc >= 2.41)
 * Linux ppc64 (glibc >= 2.36)
 * Linux s390x (glibc >= 2.36)
 * Linux x64 (glibc >= 2.26, musl >= 1.2.2, CPU with SSE4.2)
@@ -110,10 +112,11 @@ and on macOS when running Node.js under Rosetta.
 
 ## Building from source
 
-This module will be compiled from source at `npm install` time when:
+This module will be compiled from source when:
 
 * a globally-installed libvips is detected, or
-* when the `npm install --build-from-source` flag is used.
+* using `npm explore sharp -- npm run build`, or
+* using the deprecated `npm run --build-from-source` at `npm install` time.
 
 The logic to detect a globally-installed libvips can be skipped by setting the
 `SHARP_IGNORE_GLOBAL_LIBVIPS` (never try to use it) or

docs/src/content/docs/performance.md

@@ -17,7 +17,8 @@ before the Node.js process starts to increase the thread pool size.
 export UV_THREADPOOL_SIZE="$(lscpu -p | egrep -v "^#" | sort -u -t, -k 2,4 | wc -l)"
 ```
 
-libvips uses a glib-managed thread pool to avoid the overhead of spawning new threads.
+libvips uses a shared thread pool to avoid the overhead of spawning new threads.
+The size of this thread pool will grow on demand and shrink when idle.
 
 The default number of threads used to concurrently process each image is the same as the number of CPU cores,
 except when using glibc-based Linux without jemalloc, where the default is `1` to help reduce memory fragmentation.
@@ -25,7 +26,7 @@ except when using glibc-based Linux without jemalloc, where the default is `1` t
 Use [`sharp.concurrency()`](/api-utility/#concurrency) to manage the number of threads per image.
 
 To reduce memory fragmentation when using the default Linux glibc memory allocator, set the
-[`MALLOC_ARENA_MAX`](https://www.gnu.org/software/libc/manual/html_node/Memory-Allocation-Tunables.html)
+[`MALLOC_ARENA_MAX`](https://sourceware.org/glibc/manual/latest/html_node/Memory-Allocation-Tunables.html)
 environment variable before the Node.js process starts to reduce the number of memory pools.
 
 ```sh frame="none"

install/build.js

@@ -0,0 +1,38 @@
+/*!
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
+
+const {
+  useGlobalLibvips,
+  globalLibvipsVersion,
+  log,
+  spawnRebuild,
+} = require('../lib/libvips');
+
+log('Attempting to build from source via node-gyp');
+log('See https://sharp.pixelplumbing.com/install#building-from-source');
+
+try {
+  const addonApi = require('node-addon-api');
+  log(`Found node-addon-api ${addonApi.version || ''}`);
+} catch (_err) {
+  log('Please add node-addon-api to your dependencies');
+  process.exit(1);
+}
+try {
+  const gyp = require('node-gyp');
+  log(`Found node-gyp ${gyp().version}`);
+} catch (_err) {
+  log('Please add node-gyp to your dependencies');
+  process.exit(1);
+}
+
+if (useGlobalLibvips(log)) {
+  log(`Detected globally-installed libvips v${globalLibvipsVersion()}`);
+}
+
+const status = spawnRebuild();
+if (status !== 0) {
+  process.exit(status);
+}

install/check.js

@@ -1,39 +1,12 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 try {
-  const { useGlobalLibvips, globalLibvipsVersion, log, spawnRebuild } = require('../lib/libvips');
+  const { useGlobalLibvips } = require('../lib/libvips');
-
+  if (useGlobalLibvips() || process.env.npm_config_build_from_source) {
-  const buildFromSource = (msg) => {
+    process.exit(1);
-    log(msg);
-    log('Attempting to build from source via node-gyp');
-    try {
-      const addonApi = require('node-addon-api');
-      log(`Found node-addon-api ${addonApi.version || ''}`);
-    } catch (err) {
-      log('Please add node-addon-api to your dependencies');
-      return;
-    }
-    try {
-      const gyp = require('node-gyp');
-      log(`Found node-gyp ${gyp().version}`);
-    } catch (err) {
-      log('Please add node-gyp to your dependencies');
-      return;
-    }
-    log('See https://sharp.pixelplumbing.com/install#building-from-source');
-    const status = spawnRebuild();
-    if (status !== 0) {
-      process.exit(status);
-    }
-  };
-
-  if (useGlobalLibvips(log)) {
-    buildFromSource(`Detected globally-installed libvips v${globalLibvipsVersion()}`);
-  } else if (process.env.npm_config_build_from_source) {
-    buildFromSource('Detected --build-from-source flag');
   }
 } catch (err) {
   const summary = err.message.split(/\n/).slice(0, 1);

lib/channel.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const is = require('./is');
 
@@ -18,7 +18,7 @@ const bool = {
 /**
  * Remove alpha channels, if any. This is a no-op if the image does not have an alpha channel.
  *
- * See also {@link /api-operation#flatten|flatten}.
+ * See also {@link /api-operation/#flatten flatten}.
  *
  * @example
  * sharp('rgba.png')
@@ -163,7 +163,7 @@ function bandbool (boolOp) {
  * @module Sharp
  * @private
  */
-module.exports = function (Sharp) {
+module.exports = (Sharp) => {
   Object.assign(Sharp.prototype, {
     // Public instance functions
     removeAlpha,

lib/colour.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const color = require('@img/colour');
 const is = require('./is');
@@ -69,7 +69,7 @@ function grayscale (grayscale) {
  *
  * The input image will be converted to the provided colourspace at the start of the pipeline.
  * All operations will use this colourspace before converting to the output colourspace,
- * as defined by {@link #tocolourspace|toColourspace}.
+ * as defined by {@link #tocolourspace toColourspace}.
  *
  * @since 0.29.0
  *
@@ -80,7 +80,7 @@ function grayscale (grayscale) {
  *  .toColourspace('srgb')
  *  .toFile('16bpc-pipeline-to-8bpc-output.png')
  *
- * @param {string} [colourspace] - pipeline colourspace e.g. `rgb16`, `scrgb`, `lab`, `grey16` [...](https://github.com/libvips/libvips/blob/41cff4e9d0838498487a00623462204eb10ee5b8/libvips/iofuncs/enumtypes.c#L774)
+ * @param {string} [colourspace] - pipeline colourspace e.g. `rgb16`, `scrgb`, `lab`, `grey16` [...](https://www.libvips.org/API/current/enum.Interpretation.html)
  * @returns {Sharp}
  * @throws {Error} Invalid parameters
  */
@@ -112,7 +112,7 @@ function pipelineColorspace (colorspace) {
  *  .toColourspace('rgb16')
  *  .toFile('16-bpp.png')
  *
- * @param {string} [colourspace] - output colourspace e.g. `srgb`, `rgb`, `cmyk`, `lab`, `b-w` [...](https://github.com/libvips/libvips/blob/3c0bfdf74ce1dc37a6429bed47fa76f16e2cd70a/libvips/iofuncs/enumtypes.c#L777-L794)
+ * @param {string} [colourspace] - output colourspace e.g. `srgb`, `rgb`, `cmyk`, `lab`, `b-w` [...](https://www.libvips.org/API/current/enum.Interpretation.html)
  * @returns {Sharp}
  * @throws {Error} Invalid parameters
  */
@@ -141,7 +141,10 @@ function toColorspace (colorspace) {
  * @throws {Error} Invalid value
  */
 function _getBackgroundColourOption (value) {
-  if (is.object(value) || is.string(value)) {
+  if (
+    is.object(value) ||
+    (is.string(value) && value.length >= 3 && value.length <= 200)
+  ) {
     const colour = color(value);
     return [
       colour.red(),
@@ -172,7 +175,7 @@ function _setBackgroundColourOption (key, value) {
  * @module Sharp
  * @private
  */
-module.exports = function (Sharp) {
+module.exports = (Sharp) => {
   Object.assign(Sharp.prototype, {
     // Public
     tint,

lib/composite.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const is = require('./is');
 
@@ -56,7 +56,7 @@ const blend = {
  * `hard-light`, `soft-light`, `difference`, `exclusion`.
  *
  * More information about blend modes can be found at
- * https://www.libvips.org/API/current/libvips-conversion.html#VipsBlendMode
+ * https://www.libvips.org/API/current/enum.BlendMode.html
  * and https://www.cairographics.org/operators/
  *
  * @since 0.22.0
@@ -123,8 +123,8 @@ const blend = {
  * @param {Number} [images[].raw.height]
  * @param {Number} [images[].raw.channels]
  * @param {boolean} [images[].animated=false] - Set to `true` to read all frames/pages of an animated image.
- * @param {string} [images[].failOn='warning'] - @see {@link /api-constructor#parameters|constructor parameters}
+ * @param {string} [images[].failOn='warning'] - @see {@link /api-constructor/ constructor parameters}
- * @param {number|boolean} [images[].limitInputPixels=268402689] - @see {@link /api-constructor#parameters|constructor parameters}
+ * @param {number|boolean} [images[].limitInputPixels=268402689] - @see {@link /api-constructor/ constructor parameters}
  * @returns {Sharp}
  * @throws {Error} Invalid parameters
  */
@@ -206,7 +206,7 @@ function composite (images) {
  * @module Sharp
  * @private
  */
-module.exports = function (Sharp) {
+module.exports = (Sharp) => {
   Sharp.prototype.composite = composite;
   Sharp.blend = blend;
 };

lib/constructor.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const util = require('node:util');
 const stream = require('node:stream');
@@ -12,6 +12,10 @@ require('./sharp');
 // Use NODE_DEBUG=sharp to enable libvips warnings
 const debuglog = util.debuglog('sharp');
 
+const queueListener = (queueLength) => {
+  Sharp.queue.emit('change', queueLength);
+};
+
 /**
  * Constructor factory to create an instance of `sharp`, to which further methods are chained.
  *
@@ -205,6 +209,7 @@ const debuglog = util.debuglog('sharp');
  * @throws {Error} Invalid parameters
  */
 const Sharp = function (input, options) {
+  // biome-ignore lint/complexity/noArguments: constructor factory
   if (arguments.length === 1 && !is.defined(input)) {
     throw new Error('Invalid input');
   }
@@ -353,6 +358,7 @@ const Sharp = function (input, options) {
     gifProgressive: false,
     tiffQuality: 80,
     tiffCompression: 'jpeg',
+    tiffBigtiff: false,
     tiffPredictor: 'horizontal',
     tiffPyramid: false,
     tiffMiniswhite: false,
@@ -396,9 +402,7 @@ const Sharp = function (input, options) {
       debuglog(warning);
     },
     // Function to notify of queue length changes
-    queueListener: function (queueLength) {
+    queueListener
-      Sharp.queue.emit('change', queueLength);
-    }
   };
   this.options.input = this._createInputDescriptor(input, options, { allowStream: true });
   return this;

lib/index.d.ts

@@ -27,7 +27,7 @@
 
 /// <reference types="node" />
 
-import { Duplex } from 'stream';
+import type { Duplex } from 'node:stream';
 
 //#region Constructor functions
 
@@ -860,6 +860,7 @@ declare namespace sharp {
                 | JxlOptions
                 | GifOptions
                 | Jp2Options
+                | RawOptions
                 | TiffOptions,
         ): Sharp;
 
@@ -1027,11 +1028,11 @@ declare namespace sharp {
         openSlide?: OpenSlideInputOptions | undefined;
         /** JPEG 2000 specific input options */
         jp2?: Jp2InputOptions | undefined;
-        /** Deprecated: use tiff.subifd instead */
+        /** @deprecated Use {@link SharpOptions.tiff} instead */
         subifd?: number | undefined;
-        /** Deprecated: use pdf.background instead */
+        /** @deprecated Use {@link SharpOptions.pdf} instead */
         pdfBackground?: Colour | Color | undefined;
-        /** Deprecated: use openSlide.level instead */
+        /** @deprecated Use {@link SharpOptions.openSlide} instead */
         level?: number | undefined;
         /** Set to `true` to read all frames/pages of an animated image (equivalent of setting `pages` to `-1`). (optional, default false) */
         animated?: boolean | undefined;
@@ -1181,6 +1182,10 @@ declare namespace sharp {
         'IFD3'?: ExifDir;
     }
 
+    type HeifCompression = 'av1' | 'hevc';
+
+    type Unit = 'inch' | 'cm';
+
     interface WriteableMetadata {
         /** Number of pixels per inch (DPI) */
         density?: number | undefined;
@@ -1259,7 +1264,7 @@ declare namespace sharp {
         /** Buffer containing raw TIFFTAG_PHOTOSHOP data, if present */
         tifftagPhotoshop?: Buffer | undefined;
         /** The encoder used to compress an HEIF file, `av1` (AVIF) or `hevc` (HEIC) */
-        compression?: 'av1' | 'hevc';
+        compression?: HeifCompression | undefined;
         /** Default background colour, if present, for PNG (bKGD) and GIF images */
         background?: { r: number; g: number; b: number } | { gray: number };
         /** Details of each level in a multi-level image provided as an array of objects, requires libvips compiled with support for OpenSlide */
@@ -1267,7 +1272,7 @@ declare namespace sharp {
         /** Number of Sub Image File Directories in an OME-TIFF image */
         subifds?: number | undefined;
         /** The unit of resolution (density) */
-        resolutionUnit?: 'inch' | 'cm' | undefined;
+        resolutionUnit?: Unit | undefined;
         /** String containing format for images loaded via *magick */
         formatMagick?: string | undefined;
         /** Array of keyword/text pairs representing PNG text blocks, if present. */
@@ -1423,7 +1428,7 @@ declare namespace sharp {
         /** quality, integer 1-100 (optional, default 50) */
         quality?: number | undefined;
         /** compression format: av1, hevc (optional, default 'av1') */
-        compression?: 'av1' | 'hevc' | undefined;
+        compression?: HeifCompression | undefined;
         /** use lossless compression (optional, default false) */
         lossless?: boolean | undefined;
         /** Level of CPU effort to reduce file size, between 0 (fastest) and 9 (slowest) (optional, default 4) */
@@ -1460,6 +1465,8 @@ declare namespace sharp {
         quality?: number | undefined;
         /** Compression options: none, jpeg, deflate, packbits, ccittfax4, lzw, webp, zstd, jp2k (optional, default 'jpeg') */
         compression?: string | undefined;
+        /** Use BigTIFF variant (has no effect when compression is none) (optional, default false) */
+        bigtiff?: boolean | undefined;
         /** Compression predictor options: none, horizontal, float (optional, default 'horizontal') */
         predictor?: string | undefined;
         /** Write an image pyramid (optional, default false) */
@@ -1479,7 +1486,7 @@ declare namespace sharp {
         /** Write 1-bit images as miniswhite (optional, default false) */
         miniswhite?: boolean | undefined;
         /** Resolution unit options: inch, cm (optional, default 'inch') */
-        resolutionUnit?: 'inch' | 'cm' | undefined;
+        resolutionUnit?: Unit | undefined;
     }
 
     interface PngOptions extends OutputOptions {
@@ -1604,7 +1611,7 @@ declare namespace sharp {
     }
 
     interface RawOptions {
-        depth?: 'char' | 'uchar' | 'short' | 'ushort' | 'int' | 'uint' | 'float' | 'complex' | 'double' | 'dpcomplex';
+        depth?: keyof DepthEnum;
     }
 
     /** 1 for grayscale, 2 for grayscale + alpha, 3 for sRGB, 4 for CMYK or RGBA */

lib/index.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const Sharp = require('./constructor');
 require('./input')(Sharp);

lib/input.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const is = require('./is');
 const sharp = require('./sharp');
@@ -54,7 +54,7 @@ function _createInputDescriptor (input, inputOptions, containerOptions) {
   const inputDescriptor = {
     autoOrient: false,
     failOn: 'warning',
-    limitInputPixels: Math.pow(0x3FFF, 2),
+    limitInputPixels: 0x3FFF ** 2,
     ignoreIcc: false,
     unlimited: false,
     sequentialRead: true
@@ -150,7 +150,7 @@ function _createInputDescriptor (input, inputOptions, containerOptions) {
     if (is.defined(inputOptions.limitInputPixels)) {
       if (is.bool(inputOptions.limitInputPixels)) {
         inputDescriptor.limitInputPixels = inputOptions.limitInputPixels
-          ? Math.pow(0x3FFF, 2)
+          ? 0x3FFF ** 2
           : 0;
       } else if (is.integer(inputOptions.limitInputPixels) && is.inRange(inputOptions.limitInputPixels, 0, Number.MAX_SAFE_INTEGER)) {
         inputDescriptor.limitInputPixels = inputOptions.limitInputPixels;
@@ -513,7 +513,7 @@ function _createInputDescriptor (input, inputOptions, containerOptions) {
       }
     }
   } else if (is.defined(inputOptions)) {
-    throw new Error('Invalid input options ' + inputOptions);
+    throw new Error(`Invalid input options ${inputOptions}`);
   }
   return inputDescriptor;
 }
@@ -525,10 +525,8 @@ function _createInputDescriptor (input, inputOptions, containerOptions) {
  * @param {string} encoding - unused
  * @param {Function} callback
  */
-function _write (chunk, encoding, callback) {
+function _write (chunk, _encoding, callback) {
-  /* istanbul ignore else */
   if (Array.isArray(this.options.input.buffer)) {
-    /* istanbul ignore else */
     if (is.buffer(chunk)) {
       if (this.options.input.buffer.length === 0) {
         this.on('finish', () => {
@@ -572,7 +570,7 @@ function _isStreamInput () {
  * such as resize or rotate.
  *
  * Dimensions in the response will respect the `page` and `pages` properties of the
- * {@link /api-constructor#parameters|constructor parameters}.
+ * {@link /api-constructor/ constructor parameters}.
  *
  * A `Promise` is returned when `callback` is not provided.
  *
@@ -580,9 +578,9 @@ function _isStreamInput () {
  * - `size`: Total size of image in bytes, for Stream and Buffer input only
  * - `width`: Number of pixels wide (EXIF orientation is not taken into consideration, see example below)
  * - `height`: Number of pixels high (EXIF orientation is not taken into consideration, see example below)
- * - `space`: Name of colour space interpretation e.g. `srgb`, `rgb`, `cmyk`, `lab`, `b-w` [...](https://www.libvips.org/API/current/VipsImage.html#VipsInterpretation)
+ * - `space`: Name of colour space interpretation e.g. `srgb`, `rgb`, `cmyk`, `lab`, `b-w` [...](https://www.libvips.org/API/current/enum.Interpretation.html)
  * - `channels`: Number of bands e.g. `3` for sRGB, `4` for CMYK
- * - `depth`: Name of pixel depth format e.g. `uchar`, `char`, `ushort`, `float` [...](https://www.libvips.org/API/current/VipsImage.html#VipsBandFormat)
+ * - `depth`: Name of pixel depth format e.g. `uchar`, `char`, `ushort`, `float` [...](https://www.libvips.org/API/current/enum.BandFormat.html)
  * - `density`: Number of pixels per inch (DPI), if present
  * - `chromaSubsampling`: String containing JPEG chroma subsampling, `4:2:0` or `4:4:4` for RGB, `4:2:0:4` or `4:4:4:4` for CMYK
  * - `isProgressive`: Boolean indicating whether the image is interlaced using a progressive scan
@@ -794,7 +792,7 @@ function stats (callback) {
  * @module Sharp
  * @private
  */
-module.exports = function (Sharp) {
+module.exports = (Sharp) => {
   Object.assign(Sharp.prototype, {
     // Private
     _inputOptionsFromObject,

lib/is.js

@@ -1,61 +1,49 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 /**
  * Is this value defined and not null?
  * @private
  */
-const defined = function (val) {
+const defined = (val) => typeof val !== 'undefined' && val !== null;
-  return typeof val !== 'undefined' && val !== null;
-};
 
 /**
  * Is this value an object?
  * @private
  */
-const object = function (val) {
+const object = (val) => typeof val === 'object';
-  return typeof val === 'object';
-};
 
 /**
  * Is this value a plain object?
  * @private
  */
-const plainObject = function (val) {
+const plainObject = (val) => Object.prototype.toString.call(val) === '[object Object]';
-  return Object.prototype.toString.call(val) === '[object Object]';
-};
 
 /**
  * Is this value a function?
  * @private
  */
-const fn = function (val) {
+const fn = (val) => typeof val === 'function';
-  return typeof val === 'function';
-};
 
 /**
  * Is this value a boolean?
  * @private
  */
-const bool = function (val) {
+const bool = (val) => typeof val === 'boolean';
-  return typeof val === 'boolean';
-};
 
 /**
  * Is this value a Buffer object?
  * @private
  */
-const buffer = function (val) {
+const buffer = (val) => val instanceof Buffer;
-  return val instanceof Buffer;
-};
 
 /**
  * Is this value a typed array object?. E.g. Uint8Array or Uint8ClampedArray?
  * @private
  */
-const typedArray = function (val) {
+const typedArray = (val) => {
   if (defined(val)) {
     switch (val.constructor) {
       case Uint8Array:
@@ -78,49 +66,37 @@ const typedArray = function (val) {
  * Is this value an ArrayBuffer object?
  * @private
  */
-const arrayBuffer = function (val) {
+const arrayBuffer = (val) => val instanceof ArrayBuffer;
-  return val instanceof ArrayBuffer;
-};
 
 /**
  * Is this value a non-empty string?
  * @private
  */
-const string = function (val) {
+const string = (val) => typeof val === 'string' && val.length > 0;
-  return typeof val === 'string' && val.length > 0;
-};
 
 /**
  * Is this value a real number?
  * @private
  */
-const number = function (val) {
+const number = (val) => typeof val === 'number' && !Number.isNaN(val);
-  return typeof val === 'number' && !Number.isNaN(val);
-};
 
 /**
  * Is this value an integer?
  * @private
  */
-const integer = function (val) {
+const integer = (val) => Number.isInteger(val);
-  return Number.isInteger(val);
-};
 
 /**
  * Is this value within an inclusive given range?
  * @private
  */
-const inRange = function (val, min, max) {
+const inRange = (val, min, max) => val >= min && val <= max;
-  return val >= min && val <= max;
-};
 
 /**
  * Is this value within the elements of an array?
  * @private
  */
-const inArray = function (val, list) {
+const inArray = (val, list) => list.includes(val);
-  return list.includes(val);
-};
 
 /**
  * Create an Error with a message relating to an invalid parameter.
@@ -131,11 +107,9 @@ const inArray = function (val, list) {
  * @returns {Error} Containing the formatted message.
  * @private
  */
-const invalidParameterError = function (name, expected, actual) {
+const invalidParameterError = (name, expected, actual) => new Error(
-  return new Error(
     `Expected ${expected} for ${name} but received ${actual} of type ${typeof actual}`
   );
-};
 
 /**
  * Ensures an Error from C++ contains a JS stack.
@@ -145,7 +119,7 @@ const invalidParameterError = function (name, expected, actual) {
  * @returns {Error} Error with message and stack.
  * @private
  */
-const nativeError = function (native, context) {
+const nativeError = (native, context) => {
   context.message = native.message;
   return context;
 };

lib/libvips.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const { spawnSync } = require('node:child_process');
 const { createHash } = require('node:crypto');
@@ -12,13 +12,13 @@ const detectLibc = require('detect-libc');
 
 const { config, engines, optionalDependencies } = require('../package.json');
 
-const minimumLibvipsVersionLabelled = process.env.npm_package_config_libvips || /* istanbul ignore next */
+/* node:coverage ignore next */
-  config.libvips;
+const minimumLibvipsVersionLabelled = process.env.npm_package_config_libvips || config.libvips;
 const minimumLibvipsVersion = semverCoerce(minimumLibvipsVersionLabelled).version;
 
 const prebuiltPlatforms = [
   'darwin-arm64', 'darwin-x64',
-  'linux-arm', 'linux-arm64', 'linux-ppc64', 'linux-s390x', 'linux-x64',
+  'linux-arm', 'linux-arm64', 'linux-ppc64', 'linux-riscv64', 'linux-s390x', 'linux-x64',
   'linuxmusl-arm64', 'linuxmusl-x64',
   'win32-arm64', 'win32-ia32', 'win32-x64'
 ];
@@ -36,17 +36,16 @@ const log = (item) => {
   }
 };
 
-/* istanbul ignore next */
+/* node:coverage ignore next */
 const runtimeLibc = () => detectLibc.isNonGlibcLinuxSync() ? detectLibc.familySync() : '';
 
 const runtimePlatformArch = () => `${process.platform}${runtimeLibc()}-${process.arch}`;
 
-/* istanbul ignore next */
 const buildPlatformArch = () => {
+  /* node:coverage ignore next 3 */
   if (isEmscripten()) {
     return 'wasm32';
   }
-  /* eslint camelcase: ["error", { allow: ["^npm_config_"] }] */
   const { npm_config_arch, npm_config_platform, npm_config_libc } = process.env;
   const libc = typeof npm_config_libc === 'string' ? npm_config_libc : runtimeLibc();
   return `${npm_config_platform || process.platform}${libc}-${npm_config_arch || process.arch}`;
@@ -56,19 +55,19 @@ const buildSharpLibvipsIncludeDir = () => {
   try {
     return require(`@img/sharp-libvips-dev-${buildPlatformArch()}/include`);
   } catch {
+    /* node:coverage ignore next 5 */
     try {
       return require('@img/sharp-libvips-dev/include');
     } catch {}
   }
-  /* istanbul ignore next */
   return '';
 };
 
 const buildSharpLibvipsCPlusPlusDir = () => {
+  /* node:coverage ignore next 4 */
   try {
     return require('@img/sharp-libvips-dev/cplusplus');
   } catch {}
-  /* istanbul ignore next */
   return '';
 };
 
@@ -76,16 +75,17 @@ const buildSharpLibvipsLibDir = () => {
   try {
     return require(`@img/sharp-libvips-dev-${buildPlatformArch()}/lib`);
   } catch {
+    /* node:coverage ignore next 5 */
     try {
       return require(`@img/sharp-libvips-${buildPlatformArch()}/lib`);
     } catch {}
   }
-  /* istanbul ignore next */
   return '';
 };
 
+/* node:coverage disable */
+
 const isUnsupportedNodeRuntime = () => {
-  /* istanbul ignore next */
   if (process.release?.name === 'node' && process.versions) {
     if (!semverSatisfies(process.versions.node, engines.node)) {
       return { found: process.versions.node, expected: engines.node };
@@ -93,14 +93,12 @@ const isUnsupportedNodeRuntime = () => {
   }
 };
 
-/* istanbul ignore next */
 const isEmscripten = () => {
   const { CC } = process.env;
-  return Boolean(CC && CC.endsWith('/emcc'));
+  return Boolean(CC?.endsWith('/emcc'));
 };
 
 const isRosetta = () => {
-  /* istanbul ignore next */
   if (process.platform === 'darwin' && process.arch === 'x64') {
     const translated = spawnSync('sysctl sysctl.proc_translated', spawnSyncOptions).stdout;
     return (translated || '').trim() === 'sysctl.proc_translated: 1';
@@ -108,6 +106,8 @@ const isRosetta = () => {
   return false;
 };
 
+/* node:coverage enable */
+
 const sha512 = (s) => createHash('sha512').update(s).digest('hex');
 
 const yarnLocator = () => {
@@ -121,7 +121,8 @@ const yarnLocator = () => {
   return '';
 };
 
-/* istanbul ignore next */
+/* node:coverage disable */
+
 const spawnRebuild = () =>
   spawnSync(`node-gyp rebuild --directory=src ${isEmscripten() ? '--nodedir=emscripten' : ''}`, {
     ...spawnSyncOptions,
@@ -137,16 +138,17 @@ const globalLibvipsVersion = () => {
         PKG_CONFIG_PATH: pkgConfigPath()
       }
     }).stdout;
-    /* istanbul ignore next */
     return (globalLibvipsVersion || '').trim();
   } else {
     return '';
   }
 };
 
-/* istanbul ignore next */
+/* node:coverage enable */
+
 const pkgConfigPath = () => {
   if (process.platform !== 'win32') {
+    /* node:coverage ignore next 4 */
     const brewPkgConfigPath = spawnSync(
       'which brew >/dev/null 2>&1 && brew environment --plain | grep PKG_CONFIG_LIBDIR | cut -d" " -f2',
       spawnSyncOptions
@@ -178,13 +180,13 @@ const useGlobalLibvips = (logger) => {
   if (Boolean(process.env.SHARP_FORCE_GLOBAL_LIBVIPS) === true) {
     return skipSearch(true, 'SHARP_FORCE_GLOBAL_LIBVIPS', logger);
   }
-  /* istanbul ignore next */
+  /* node:coverage ignore next 3 */
   if (isRosetta()) {
     return skipSearch(false, 'Rosetta', logger);
   }
   const globalVipsVersion = globalLibvipsVersion();
-  return !!globalVipsVersion && /* istanbul ignore next */
+  /* node:coverage ignore next */
-    semverGreaterThanOrEqualTo(globalVipsVersion, minimumLibvipsVersion);
+  return !!globalVipsVersion && semverGreaterThanOrEqualTo(globalVipsVersion, minimumLibvipsVersion);
 };
 
 module.exports = {

lib/operation.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const is = require('./is');
 
@@ -239,7 +239,7 @@ function affine (matrix, options) {
  * When a `sigma` is provided, performs a slower, more accurate sharpen of the L channel in the LAB colour space.
  * Fine-grained control over the level of sharpening in "flat" (m1) and "jagged" (m2) areas is available.
  *
- * See {@link https://www.libvips.org/API/current/libvips-convolution.html#vips-sharpen|libvips sharpen} operation.
+ * See {@link https://www.libvips.org/API/current/method.Image.sharpen.html libvips sharpen} operation.
  *
  * @example
  * const data = await sharp(input).sharpen().toBuffer();
@@ -485,7 +485,7 @@ function erode (width) {
 /**
  * Merge alpha transparency channel, if any, with a background, then remove the alpha channel.
  *
- * See also {@link /api-channel#removealpha|removeAlpha}.
+ * See also {@link /api-channel#removealpha removeAlpha}.
  *
  * @example
  * await sharp(rgbaInput)
@@ -660,7 +660,7 @@ function normalize (options) {
 
 /**
  * Perform contrast limiting adaptive histogram equalization
- * {@link https://en.wikipedia.org/wiki/Adaptive_histogram_equalization#Contrast_Limited_AHE|CLAHE}.
+ * {@link https://en.wikipedia.org/wiki/Adaptive_histogram_equalization#Contrast_Limited_AHE CLAHE}.
  *
  * This will, in general, enhance the clarity of the image by bringing out darker details.
  *
@@ -742,9 +742,7 @@ function convolve (kernel) {
   }
   // Default scale is sum of kernel values
   if (!is.integer(kernel.scale)) {
-    kernel.scale = kernel.kernel.reduce(function (a, b) {
+    kernel.scale = kernel.kernel.reduce((a, b) => a + b, 0);
-      return a + b;
-    }, 0);
   }
   // Clip scale to a minimum value of 1
   if (kernel.scale < 1) {
@@ -989,7 +987,7 @@ function modulate (options) {
  * @module Sharp
  * @private
  */
-module.exports = function (Sharp) {
+module.exports = (Sharp) => {
   Object.assign(Sharp.prototype, {
     autoOrient,
     rotate,

lib/output.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const path = require('node:path');
 const is = require('./is');
@@ -43,7 +43,7 @@ const bitdepthFromColourCount = (colours) => 1 << 31 - Math.clz32(Math.ceil(Math
  * Note that raw pixel data is only supported for buffer output.
  *
  * By default all metadata will be removed, which includes EXIF-based orientation.
- * See {@link #withmetadata|withMetadata} for control over this.
+ * See {@link #withmetadata withMetadata} for control over this.
  *
  * The caller is responsible for ensuring directory structures and permissions exist.
  *
@@ -97,12 +97,12 @@ function toFile (fileOut, callback) {
  * Write output to a Buffer.
  * JPEG, PNG, WebP, AVIF, TIFF, GIF and raw pixel data output are supported.
  *
- * Use {@link #toformat|toFormat} or one of the format-specific functions such as {@link jpeg}, {@link png} etc. to set the output format.
+ * Use {@link #toformat toFormat} or one of the format-specific functions such as {@link #jpeg jpeg}, {@link #png png} etc. to set the output format.
  *
  * If no explicit format is set, the output format will match the input image, except SVG input which becomes PNG output.
  *
  * By default all metadata will be removed, which includes EXIF-based orientation.
- * See {@link #withmetadata|withMetadata} for control over this.
+ * See {@link #withmetadata withMetadata} for control over this.
  *
  * `callback`, if present, gets three arguments `(err, data, info)` where:
  * - `err` is an error, if any.
@@ -256,7 +256,7 @@ function withExifMerge (exif) {
 /**
  * Keep ICC profile from the input image in the output image.
  *
- * Where necessary, will attempt to convert the output colour space to match the profile.
+ * When input and output colour spaces differ, use with {@link /api-colour/#tocolourspace toColourspace} and optionally {@link /api-colour/#pipelinecolourspace pipelineColourspace}.
  *
  * @since 0.33.0
  *
@@ -265,6 +265,13 @@ function withExifMerge (exif) {
  *   .keepIccProfile()
  *   .toBuffer();
  *
+ * @example
+ * const cmykOutputWithIccProfile = await sharp(cmykInputWithIccProfile)
+ *   .pipelineColourspace('cmyk')
+ *   .toColourspace('cmyk')
+ *   .keepIccProfile()
+ *   .toBuffer();
+ *
  * @returns {Sharp}
  */
 function keepIccProfile () {
@@ -565,7 +572,7 @@ function jpeg (options) {
  * Set `palette` to `true` for slower, indexed PNG output.
  *
  * For 16 bits per pixel output, convert to `rgb16` via
- * {@link /api-colour#tocolourspace|toColourspace}.
+ * {@link /api-colour/#tocolourspace toColourspace}.
  *
  * @example
  * // Convert any input to full colour PNG output
@@ -845,13 +852,12 @@ function gif (options) {
   return this._updateFormatOut('gif', options);
 }
 
-/* istanbul ignore next */
 /**
  * Use these JP2 options for output image.
  *
  * Requires libvips compiled with support for OpenJPEG.
  * The prebuilt binaries do not include this - see
- * {@link https://sharp.pixelplumbing.com/install#custom-libvips installing a custom libvips}.
+ * {@link /install#custom-libvips installing a custom libvips}.
  *
  * @example
  * // Convert any input to lossless JP2 output
@@ -880,6 +886,7 @@ function gif (options) {
  * @throws {Error} Invalid options
  */
 function jp2 (options) {
+  /* node:coverage ignore next 41 */
   if (!this.constructor.format.jp2k.output.buffer) {
     throw errJp2Save();
   }
@@ -959,7 +966,7 @@ function trySetAnimationOptions (source, target) {
 /**
  * Use these TIFF options for output image.
  *
- * The `density` can be set in pixels/inch via {@link #withmetadata|withMetadata}
+ * The `density` can be set in pixels/inch via {@link #withmetadata withMetadata}
  * instead of providing `xres` and `yres` in pixels/mm.
  *
  * @example
@@ -976,6 +983,7 @@ function trySetAnimationOptions (source, target) {
  * @param {number} [options.quality=80] - quality, integer 1-100
  * @param {boolean} [options.force=true] - force TIFF output, otherwise attempt to use input format
  * @param {string} [options.compression='jpeg'] - compression options: none, jpeg, deflate, packbits, ccittfax4, lzw, webp, zstd, jp2k
+ * @param {boolean} [options.bigtiff=false] - use BigTIFF variant (has no effect when compression is none)
  * @param {string} [options.predictor='horizontal'] - compression predictor options: none, horizontal, float
  * @param {boolean} [options.pyramid=false] - write an image pyramid
  * @param {boolean} [options.tile=false] - write a tiled tiff
@@ -1054,6 +1062,10 @@ function tiff (options) {
         throw is.invalidParameterError('compression', 'one of: none, jpeg, deflate, packbits, ccittfax4, lzw, webp, zstd, jp2k', options.compression);
       }
     }
+    // bigtiff
+    if (is.defined(options.bigtiff)) {
+      this._setBooleanOption('tiffBigtiff', options.bigtiff);
+    }
     // predictor
     if (is.defined(options.predictor)) {
       if (is.string(options.predictor) && is.inArray(options.predictor, ['none', 'horizontal', 'float'])) {
@@ -1189,7 +1201,7 @@ function heif (options) {
  *
  * Requires libvips compiled with support for libjxl.
  * The prebuilt binaries do not include this - see
- * {@link https://sharp.pixelplumbing.com/install#custom-libvips installing a custom libvips}.
+ * {@link /install/#custom-libvips installing a custom libvips}.
  *
  * @since 0.31.3
  *
@@ -1502,7 +1514,6 @@ function _setBooleanOption (key, val) {
  * @private
  */
 function _read () {
-  /* istanbul ignore else */
   if (!this.options.streamOut) {
     this.options.streamOut = true;
     const stack = Error();
@@ -1619,7 +1630,7 @@ function _pipeline (callback, stack) {
  * @module Sharp
  * @private
  */
-module.exports = function (Sharp) {
+module.exports = (Sharp) => {
   Object.assign(Sharp.prototype, {
     // Public
     toFile,

lib/resize.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const is = require('./is');
 
@@ -579,7 +579,7 @@ function trim (options) {
  * @module Sharp
  * @private
  */
-module.exports = function (Sharp) {
+module.exports = (Sharp) => {
   Object.assign(Sharp.prototype, {
     resize,
     extend,

lib/sharp.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 // Inspects the runtime environment and exports the relevant sharp.node binary
 
@@ -17,6 +17,8 @@ const paths = [
   '@img/sharp-wasm32/sharp.node'
 ];
 
+/* node:coverage disable */
+
 let path, sharp;
 const errors = [];
 for (path of paths) {
@@ -24,12 +26,10 @@ for (path of paths) {
     sharp = require(path);
     break;
   } catch (err) {
-    /* istanbul ignore next */
     errors.push(err);
   }
 }
 
-/* istanbul ignore next */
 if (sharp && path.startsWith('@img/sharp-linux-x64') && !sharp._isUsingX64V2()) {
   const err = new Error('Prebuilt binaries for linux-x64 require v2 microarchitecture');
   err.code = 'Unsupported CPU';
@@ -37,7 +37,6 @@ if (sharp && path.startsWith('@img/sharp-linux-x64') && !sharp._isUsingX64V2())
   sharp = null;
 }
 
-/* istanbul ignore next */
 if (sharp) {
   module.exports = sharp;
 } else {
@@ -88,7 +87,7 @@ if (sharp) {
         `    Found ${libcFound}`,
         `    Requires ${libcRequires}`
       );
-    } catch (errEngines) {}
+    } catch (_errEngines) {}
   }
   if (isLinux && /\/snap\/core[0-9]{2}/.test(messages)) {
     help.push(

lib/utility.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const events = require('node:events');
 const detectLibc = require('detect-libc');
@@ -57,7 +57,7 @@ const interpolators = {
 let versions = {
   vips: libvipsVersion.semver
 };
-/* istanbul ignore next */
+/* node:coverage ignore next 15 */
 if (!libvipsVersion.isGlobal) {
   if (!libvipsVersion.isWasm) {
     try {
@@ -75,7 +75,7 @@ if (!libvipsVersion.isGlobal) {
 }
 versions.sharp = require('../package.json').version;
 
-/* istanbul ignore next */
+/* node:coverage ignore next 5 */
 if (versions.heif && format.heif) {
   // Prebuilt binaries provide AV1
   format.heif.input.fileSuffix = ['.avif'];
@@ -136,7 +136,7 @@ cache(true);
  * and these are independent of the value set here.
  *
  * :::note
- * Further {@link /performance|control over performance} is available.
+ * Further {@link /performance/ control over performance} is available.
  * :::
  *
  * @example
@@ -150,7 +150,7 @@ cache(true);
 function concurrency (concurrency) {
   return sharp.concurrency(is.integer(concurrency) ? concurrency : null);
 }
-/* istanbul ignore next */
+/* node:coverage ignore next 7 */
 if (detectLibc.familySync() === detectLibc.GLIBC && !sharp._isUsingJemalloc()) {
   // Reduce default concurrency to 1 when using glibc memory allocator
   sharp.concurrency(1);
@@ -277,7 +277,7 @@ function unblock (options) {
  * @module Sharp
  * @private
  */
-module.exports = function (Sharp) {
+module.exports = (Sharp) => {
   Sharp.cache = cache;
   Sharp.concurrency = concurrency;
   Sharp.counters = counters;

npm/darwin-arm64/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-darwin-arm64",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with macOS 64-bit ARM",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",
@@ -15,7 +15,7 @@
   },
   "preferUnplugged": true,
   "optionalDependencies": {
-    "@img/sharp-libvips-darwin-arm64": "1.2.3"
+    "@img/sharp-libvips-darwin-arm64": "1.2.4"
   },
   "files": [
     "lib"

npm/darwin-x64/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-darwin-x64",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with macOS x64",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",
@@ -15,7 +15,7 @@
   },
   "preferUnplugged": true,
   "optionalDependencies": {
-    "@img/sharp-libvips-darwin-x64": "1.2.3"
+    "@img/sharp-libvips-darwin-x64": "1.2.4"
   },
   "files": [
     "lib"

npm/from-local-build.js

@@ -1,7 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 // Populate the npm package for the current platform with the local build
 

npm/linux-arm/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-linux-arm",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with Linux (glibc) ARM (32-bit)",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",
@@ -15,7 +15,7 @@
   },
   "preferUnplugged": true,
   "optionalDependencies": {
-    "@img/sharp-libvips-linux-arm": "1.2.3"
+    "@img/sharp-libvips-linux-arm": "1.2.4"
   },
   "files": [
     "lib"

npm/linux-arm64/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-linux-arm64",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with Linux (glibc) 64-bit ARM",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",
@@ -15,7 +15,7 @@
   },
   "preferUnplugged": true,
   "optionalDependencies": {
-    "@img/sharp-libvips-linux-arm64": "1.2.3"
+    "@img/sharp-libvips-linux-arm64": "1.2.4"
   },
   "files": [
     "lib"

npm/linux-ppc64/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-linux-ppc64",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with Linux (glibc) ppc64",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",
@@ -15,7 +15,7 @@
   },
   "preferUnplugged": true,
   "optionalDependencies": {
-    "@img/sharp-libvips-linux-ppc64": "1.2.3"
+    "@img/sharp-libvips-linux-ppc64": "1.2.4"
   },
   "files": [
     "lib"

npm/linux-riscv64/package.json

@@ -0,0 +1,46 @@
+{
+  "name": "@img/sharp-linux-riscv64",
+  "version": "0.34.5",
+  "description": "Prebuilt sharp for use with Linux (glibc) RISC-V 64-bit",
+  "author": "Lovell Fuller <npm@lovell.info>",
+  "homepage": "https://sharp.pixelplumbing.com",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/lovell/sharp.git",
+    "directory": "npm/linux-riscv64"
+  },
+  "license": "Apache-2.0",
+  "funding": {
+    "url": "https://opencollective.com/libvips"
+  },
+  "preferUnplugged": true,
+  "optionalDependencies": {
+    "@img/sharp-libvips-linux-riscv64": "1.2.4"
+  },
+  "files": [
+    "lib"
+  ],
+  "publishConfig": {
+    "access": "public"
+  },
+  "type": "commonjs",
+  "exports": {
+    "./sharp.node": "./lib/sharp-linux-riscv64.node",
+    "./package": "./package.json"
+  },
+  "engines": {
+    "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
+  },
+  "config": {
+    "glibc": ">=2.41"
+  },
+  "os": [
+    "linux"
+  ],
+  "libc": [
+    "glibc"
+  ],
+  "cpu": [
+    "riscv64"
+  ]
+}

npm/linux-s390x/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-linux-s390x",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with Linux (glibc) s390x",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",
@@ -15,7 +15,7 @@
   },
   "preferUnplugged": true,
   "optionalDependencies": {
-    "@img/sharp-libvips-linux-s390x": "1.2.3"
+    "@img/sharp-libvips-linux-s390x": "1.2.4"
   },
   "files": [
     "lib"

npm/linux-x64/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-linux-x64",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with Linux (glibc) x64",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",
@@ -15,7 +15,7 @@
   },
   "preferUnplugged": true,
   "optionalDependencies": {
-    "@img/sharp-libvips-linux-x64": "1.2.3"
+    "@img/sharp-libvips-linux-x64": "1.2.4"
   },
   "files": [
     "lib"

npm/linuxmusl-arm64/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-linuxmusl-arm64",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with Linux (musl) 64-bit ARM",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",
@@ -15,7 +15,7 @@
   },
   "preferUnplugged": true,
   "optionalDependencies": {
-    "@img/sharp-libvips-linuxmusl-arm64": "1.2.3"
+    "@img/sharp-libvips-linuxmusl-arm64": "1.2.4"
   },
   "files": [
     "lib"

npm/linuxmusl-x64/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-linuxmusl-x64",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with Linux (musl) x64",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",
@@ -15,7 +15,7 @@
   },
   "preferUnplugged": true,
   "optionalDependencies": {
-    "@img/sharp-libvips-linuxmusl-x64": "1.2.3"
+    "@img/sharp-libvips-linuxmusl-x64": "1.2.4"
   },
   "files": [
     "lib"

npm/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "private": "true",
   "workspaces": [
     "darwin-arm64",
@@ -8,6 +8,7 @@
     "linux-arm",
     "linux-arm64",
     "linux-ppc64",
+    "linux-riscv64",
     "linux-s390x",
     "linux-x64",
     "linuxmusl-arm64",

npm/wasm32/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-wasm32",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with wasm32",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",
@@ -31,7 +31,7 @@
     "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
   },
   "dependencies": {
-    "@emnapi/runtime": "^1.5.0"
+    "@emnapi/runtime": "^1.7.0"
   },
   "cpu": [
     "wasm32"

npm/win32-arm64/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-win32-arm64",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with Windows 64-bit ARM",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",

npm/win32-ia32/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-win32-ia32",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with Windows x86 (32-bit)",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",

npm/win32-x64/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@img/sharp-win32-x64",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "description": "Prebuilt sharp for use with Windows x64",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",

package.json

@@ -1,7 +1,7 @@
 {
   "name": "sharp",
   "description": "High performance Node.js image processing, the fastest module to resize JPEG, PNG, WebP, GIF, AVIF and TIFF images",
-  "version": "0.34.4-rc.4",
+  "version": "0.34.5",
   "author": "Lovell Fuller <npm@lovell.info>",
   "homepage": "https://sharp.pixelplumbing.com",
   "contributors": [
@@ -92,14 +92,16 @@
     "Don Denton <don@happycollision.com>"
   ],
   "scripts": {
-    "install": "node install/check.js",
+    "build": "node install/build.js",
+    "install": "node install/check.js || npm run build",
     "clean": "rm -rf src/build/ .nyc_output/ coverage/ test/fixtures/output.*",
-    "test": "npm run test-lint && npm run test-unit && npm run test-licensing && npm run test-types",
+    "test": "npm run lint && npm run test-unit",
-    "test-lint": "semistandard && cpplint",
+    "lint": "npm run lint-cpp && npm run lint-js && npm run lint-types",
-    "test-unit": "nyc --reporter=lcov --reporter=text --check-coverage --branches=100 mocha",
+    "lint-cpp": "cpplint --quiet src/*.h src/*.cc",
-    "test-licensing": "license-checker --production --summary --onlyAllow=\"Apache-2.0;BSD;ISC;LGPL-3.0-or-later;MIT\"",
+    "lint-js": "biome lint",
+    "lint-types": "tsd --files ./test/types/sharp.test-d.ts",
     "test-leak": "./test/leak/leak.sh",
-    "test-types": "tsd",
+    "test-unit": "node --experimental-test-coverage test/unit.mjs",
     "package-from-local-build": "node npm/from-local-build.js",
     "package-release-notes": "node npm/release-notes.js",
     "docs-build": "node docs/build.mjs",
@@ -138,53 +140,52 @@
   ],
   "dependencies": {
     "@img/colour": "^1.0.0",
-    "detect-libc": "^2.1.0",
+    "detect-libc": "^2.1.2",
-    "semver": "^7.7.2"
+    "semver": "^7.7.3"
   },
   "optionalDependencies": {
-    "@img/sharp-darwin-arm64": "0.34.4-rc.4",
+    "@img/sharp-darwin-arm64": "0.34.5",
-    "@img/sharp-darwin-x64": "0.34.4-rc.4",
+    "@img/sharp-darwin-x64": "0.34.5",
-    "@img/sharp-libvips-darwin-arm64": "1.2.3",
+    "@img/sharp-libvips-darwin-arm64": "1.2.4",
-    "@img/sharp-libvips-darwin-x64": "1.2.3",
+    "@img/sharp-libvips-darwin-x64": "1.2.4",
-    "@img/sharp-libvips-linux-arm": "1.2.3",
+    "@img/sharp-libvips-linux-arm": "1.2.4",
-    "@img/sharp-libvips-linux-arm64": "1.2.3",
+    "@img/sharp-libvips-linux-arm64": "1.2.4",
-    "@img/sharp-libvips-linux-ppc64": "1.2.3",
+    "@img/sharp-libvips-linux-ppc64": "1.2.4",
-    "@img/sharp-libvips-linux-s390x": "1.2.3",
+    "@img/sharp-libvips-linux-riscv64": "1.2.4",
-    "@img/sharp-libvips-linux-x64": "1.2.3",
+    "@img/sharp-libvips-linux-s390x": "1.2.4",
-    "@img/sharp-libvips-linuxmusl-arm64": "1.2.3",
+    "@img/sharp-libvips-linux-x64": "1.2.4",
-    "@img/sharp-libvips-linuxmusl-x64": "1.2.3",
+    "@img/sharp-libvips-linuxmusl-arm64": "1.2.4",
-    "@img/sharp-linux-arm": "0.34.4-rc.4",
+    "@img/sharp-libvips-linuxmusl-x64": "1.2.4",
-    "@img/sharp-linux-arm64": "0.34.4-rc.4",
+    "@img/sharp-linux-arm": "0.34.5",
-    "@img/sharp-linux-ppc64": "0.34.4-rc.4",
+    "@img/sharp-linux-arm64": "0.34.5",
-    "@img/sharp-linux-s390x": "0.34.4-rc.4",
+    "@img/sharp-linux-ppc64": "0.34.5",
-    "@img/sharp-linux-x64": "0.34.4-rc.4",
+    "@img/sharp-linux-riscv64": "0.34.5",
-    "@img/sharp-linuxmusl-arm64": "0.34.4-rc.4",
+    "@img/sharp-linux-s390x": "0.34.5",
-    "@img/sharp-linuxmusl-x64": "0.34.4-rc.4",
+    "@img/sharp-linux-x64": "0.34.5",
-    "@img/sharp-wasm32": "0.34.4-rc.4",
+    "@img/sharp-linuxmusl-arm64": "0.34.5",
-    "@img/sharp-win32-arm64": "0.34.4-rc.4",
+    "@img/sharp-linuxmusl-x64": "0.34.5",
-    "@img/sharp-win32-ia32": "0.34.4-rc.4",
+    "@img/sharp-wasm32": "0.34.5",
-    "@img/sharp-win32-x64": "0.34.4-rc.4"
+    "@img/sharp-win32-arm64": "0.34.5",
+    "@img/sharp-win32-ia32": "0.34.5",
+    "@img/sharp-win32-x64": "0.34.5"
   },
   "devDependencies": {
-    "@emnapi/runtime": "^1.5.0",
+    "@biomejs/biome": "^2.3.4",
-    "@img/sharp-libvips-dev": "1.2.3",
+    "@cpplint/cli": "^0.1.0",
-    "@img/sharp-libvips-dev-wasm32": "1.2.3",
+    "@emnapi/runtime": "^1.7.0",
-    "@img/sharp-libvips-win32-arm64": "1.2.3",
+    "@img/sharp-libvips-dev": "1.2.4",
-    "@img/sharp-libvips-win32-ia32": "1.2.3",
+    "@img/sharp-libvips-dev-wasm32": "1.2.4",
-    "@img/sharp-libvips-win32-x64": "1.2.3",
+    "@img/sharp-libvips-win32-arm64": "1.2.4",
+    "@img/sharp-libvips-win32-ia32": "1.2.4",
+    "@img/sharp-libvips-win32-x64": "1.2.4",
     "@types/node": "*",
-    "cc": "^3.0.1",
+    "emnapi": "^1.7.0",
-    "emnapi": "^1.5.0",
     "exif-reader": "^2.0.2",
     "extract-zip": "^2.0.1",
     "icc": "^3.0.0",
-    "jsdoc-to-markdown": "^9.1.2",
+    "jsdoc-to-markdown": "^9.1.3",
-    "license-checker": "^25.0.1",
-    "mocha": "^11.7.2",
     "node-addon-api": "^8.5.0",
-    "node-gyp": "^11.4.2",
+    "node-gyp": "^11.5.0",
-    "nyc": "^17.1.0",
-    "semistandard": "^17.0.0",
     "tar-fs": "^3.1.1",
     "tsd": "^0.33.0"
   },
@@ -193,28 +194,9 @@
     "node": "^18.17.0 || ^20.3.0 || >=21.0.0"
   },
   "config": {
-    "libvips": ">=8.17.2"
+    "libvips": ">=8.17.3"
   },
   "funding": {
     "url": "https://opencollective.com/libvips"
-  },
-  "semistandard": {
-    "env": [
-      "mocha"
-    ]
-  },
-  "cc": {
-    "linelength": "120",
-    "filter": [
-      "build/include"
-    ]
-  },
-  "nyc": {
-    "include": [
-      "lib"
-    ]
-  },
-  "tsd": {
-    "directory": "test/types/"
   }
 }

src/CPPLINT.cfg

@@ -0,0 +1,10 @@
+set noparent
+
+linelength=120
+
+filter=-build/include
+filter=+build/include_alpha
+filter=+build/include_subdir
+filter=+build/include_what_you_use
+
+filter=-whitespace/indent_namespace

src/binding.gyp

@@ -120,7 +120,7 @@
     'conditions': [
       ['use_global_libvips == "true"', {
         # Use pkg-config for include and lib
-        'include_dirs': ['<!@(PKG_CONFIG_PATH="<(pkg_config_path)" pkg-config --cflags-only-I vips-cpp vips glib-2.0 | sed s\/-I//g)'],
+        'include_dirs': ['<!@(PKG_CONFIG_PATH="<(pkg_config_path)" pkg-config --cflags-only-I vips-cpp vips glib-2.0 | sed s/-I//g)'],
         'libraries': ['<!@(PKG_CONFIG_PATH="<(pkg_config_path)" pkg-config --libs vips-cpp)'],
         'defines': [
           'SHARP_USE_GLOBAL_LIBVIPS'

src/common.cc

@@ -1,18 +1,22 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
+#include <algorithm>
 #include <cstdlib>
-#include <string>
-#include <string.h>
-#include <vector>
-#include <queue>
 #include <map>
-#include <mutex>  // NOLINT(build/c++11)
+#include <mutex>
+#include <queue>
+#include <string>
+#include <tuple>
+#include <utility>
+#include <vector>
 
 #include <napi.h>
 #include <vips/vips8>
 
-#include "common.h"
+#include "./common.h"
 
 using vips::VImage;
 

src/common.h

@@ -1,13 +1,16 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
 #ifndef SRC_COMMON_H_
 #define SRC_COMMON_H_
 
+#include <atomic>
 #include <string>
 #include <tuple>
+#include <utility>
 #include <vector>
-#include <atomic>
 
 #include <napi.h>
 #include <vips/vips8>
@@ -16,8 +19,8 @@
 
 #if (VIPS_MAJOR_VERSION < 8) || \
   (VIPS_MAJOR_VERSION == 8 && VIPS_MINOR_VERSION < 17) || \
-  (VIPS_MAJOR_VERSION == 8 && VIPS_MINOR_VERSION == 17 && VIPS_MICRO_VERSION < 2)
+  (VIPS_MAJOR_VERSION == 8 && VIPS_MINOR_VERSION == 17 && VIPS_MICRO_VERSION < 3)
-#error "libvips version 8.17.2+ is required - please see https://sharp.pixelplumbing.com/install"
+#error "libvips version 8.17.3+ is required - please see https://sharp.pixelplumbing.com/install"
 #endif
 
 #if defined(__has_include)
@@ -30,7 +33,7 @@ using vips::VImage;
 
 namespace sharp {
 
-  struct InputDescriptor {  // NOLINT(runtime/indentation_namespace)
+  struct InputDescriptor {
     std::string name;
     std::string file;
     bool autoOrient;

src/emscripten/pre.js

@@ -1,5 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
 /* global Module, ENV, _vips_shutdown, _uv_library_shutdown */
 

src/metadata.cc

@@ -1,15 +1,19 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-#include <numeric>
-#include <vector>
 #include <cmath>
+#include <numeric>
+#include <string>
+#include <utility>
+#include <vector>
 
 #include <napi.h>
 #include <vips/vips8>
 
-#include "common.h"
+#include "./common.h"
-#include "metadata.h"
+#include "./metadata.h"
 
 static void* readPNGComment(VipsImage *image, const char *field, GValue *value, void *p);
 
@@ -215,10 +219,10 @@ class MetadataWorker : public Napi::AsyncWorker {
       if (!baton->levels.empty()) {
         int i = 0;
         Napi::Array levels = Napi::Array::New(env, static_cast<size_t>(baton->levels.size()));
-        for (std::pair<int, int> const &l : baton->levels) {
+        for (const auto& [width, height] : baton->levels) {
           Napi::Object level = Napi::Object::New(env);
-          level.Set("width", l.first);
+          level.Set("width", width);
-          level.Set("height", l.second);
+          level.Set("height", height);
           levels.Set(i++, level);
         }
         info.Set("levels", levels);
@@ -275,10 +279,10 @@ class MetadataWorker : public Napi::AsyncWorker {
       if (baton->comments.size() > 0) {
         int i = 0;
         Napi::Array comments = Napi::Array::New(env, baton->comments.size());
-        for (auto &c : baton->comments) {
+        for (const auto& [keyword, text] : baton->comments) {
           Napi::Object comment = Napi::Object::New(env);
-          comment.Set("keyword", c.first);
+          comment.Set("keyword", keyword);
-          comment.Set("text", c.second);
+          comment.Set("text", text);
           comments.Set(i++, comment);
         }
         info.Set("comments", comments);

src/metadata.h

@@ -1,10 +1,13 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
 #ifndef SRC_METADATA_H_
 #define SRC_METADATA_H_
 
 #include <string>
+#include <vector>
 #include <napi.h>
 
 #include "./common.h"

src/operations.cc

@@ -1,5 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
 #include <algorithm>
 #include <functional>
@@ -8,8 +10,8 @@
 #include <vector>
 #include <vips/vips8>
 
-#include "common.h"
+#include "./common.h"
-#include "operations.h"
+#include "./operations.h"
 
 using vips::VImage;
 using vips::VError;

src/operations.h

@@ -1,5 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
 #ifndef SRC_OPERATIONS_H_
 #define SRC_OPERATIONS_H_
@@ -8,6 +10,7 @@
 #include <functional>
 #include <memory>
 #include <tuple>
+#include <vector>
 #include <vips/vips8>
 
 using vips::VImage;

src/pipeline.cc

@@ -1,9 +1,11 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
 #include <algorithm>
 #include <cmath>
-#include <filesystem>
+#include <filesystem>  // NOLINT(build/c++17)
 #include <map>
 #include <memory>
 #include <numeric>
@@ -17,9 +19,9 @@
 #include <vips/vips8>
 #include <napi.h>
 
-#include "common.h"
+#include "./common.h"
-#include "operations.h"
+#include "./operations.h"
-#include "pipeline.h"
+#include "./pipeline.h"
 
 class PipelineWorker : public Napi::AsyncWorker {
  public:
@@ -453,12 +455,10 @@ class PipelineWorker : public Napi::AsyncWorker {
           std::tie(image, background) = sharp::ApplyAlpha(image, baton->resizeBackground, shouldPremultiplyAlpha);
 
           // Embed
-          int left;
+          const auto& [left, top] = sharp::CalculateEmbedPosition(
-          int top;
-          std::tie(left, top) = sharp::CalculateEmbedPosition(
             inputWidth, inputHeight, baton->width, baton->height, baton->position);
-          int width = std::max(inputWidth, baton->width);
+          const int width = std::max(inputWidth, baton->width);
-          int height = std::max(inputHeight, baton->height);
+          const int height = std::max(inputHeight, baton->height);
 
           image = nPages > 1
             ? sharp::EmbedMultiPage(image,
@@ -477,13 +477,10 @@ class PipelineWorker : public Napi::AsyncWorker {
           // Crop
           if (baton->position < 9) {
             // Gravity-based crop
-            int left;
+            const auto& [left, top] = sharp::CalculateCrop(
-            int top;
-
-            std::tie(left, top) = sharp::CalculateCrop(
               inputWidth, inputHeight, baton->width, baton->height, baton->position);
-            int width = std::min(inputWidth, baton->width);
+            const int width = std::min(inputWidth, baton->width);
-            int height = std::min(inputHeight, baton->height);
+            const int height = std::min(inputHeight, baton->height);
 
             image = nPages > 1
               ? sharp::CropMultiPage(image,
@@ -795,20 +792,14 @@ class PipelineWorker : public Napi::AsyncWorker {
         image = sharp::EnsureAlpha(image, baton->ensureAlpha);
       }
 
-      // Convert image to sRGB, if not already
+      // Ensure output colour space
       if (sharp::Is16Bit(image.interpretation())) {
         image = image.cast(VIPS_FORMAT_USHORT);
       }
       if (image.interpretation() != baton->colourspace) {
-        // Convert colourspace, pass the current known interpretation so libvips doesn't have to guess
         image = image.colourspace(baton->colourspace, VImage::option()->set("source_space", image.interpretation()));
-        // Transform colours from embedded profile to output profile
+        if (inputProfile.first != nullptr && baton->withIccProfile.empty()) {
-        if ((baton->keepMetadata & VIPS_FOREIGN_KEEP_ICC) && baton->colourspacePipeline != VIPS_INTERPRETATION_CMYK &&
+          image = sharp::SetProfile(image, inputProfile);
-          baton->withIccProfile.empty() && sharp::HasProfile(image)) {
-          image = image.icc_transform(processingProfile, VImage::option()
-            ->set("embedded", true)
-            ->set("depth", sharp::Is16Bit(image.interpretation()) ? 16 : 8)
-            ->set("intent", VIPS_INTENT_PERCEPTUAL));
         }
       }
 
@@ -843,8 +834,6 @@ class PipelineWorker : public Napi::AsyncWorker {
         } catch(...) {
           sharp::VipsWarningCallback(nullptr, G_LOG_LEVEL_WARNING, "Invalid profile", nullptr);
         }
-      } else if (baton->keepMetadata & VIPS_FOREIGN_KEEP_ICC) {
-        image = sharp::SetProfile(image, inputProfile);
       }
 
       // Negate the colours in the image
@@ -866,8 +855,8 @@ class PipelineWorker : public Napi::AsyncWorker {
         if (!baton->withExifMerge) {
           image = sharp::RemoveExif(image);
         }
-        for (const auto& s : baton->withExif) {
+        for (const auto& [key, value] : baton->withExif) {
-          image.set(s.first.data(), s.second.data());
+          image.set(key.c_str(), value.c_str());
         }
       }
       // XMP buffer
@@ -1009,6 +998,7 @@ class PipelineWorker : public Napi::AsyncWorker {
             ->set("Q", baton->tiffQuality)
             ->set("bitdepth", baton->tiffBitdepth)
             ->set("compression", baton->tiffCompression)
+            ->set("bigtiff", baton->tiffBigtiff)
             ->set("miniswhite", baton->tiffMiniswhite)
             ->set("predictor", baton->tiffPredictor)
             ->set("pyramid", baton->tiffPyramid)
@@ -1211,6 +1201,7 @@ class PipelineWorker : public Napi::AsyncWorker {
             ->set("Q", baton->tiffQuality)
             ->set("bitdepth", baton->tiffBitdepth)
             ->set("compression", baton->tiffCompression)
+            ->set("bigtiff", baton->tiffBigtiff)
             ->set("miniswhite", baton->tiffMiniswhite)
             ->set("predictor", baton->tiffPredictor)
             ->set("pyramid", baton->tiffPyramid)
@@ -1275,10 +1266,15 @@ class PipelineWorker : public Napi::AsyncWorker {
       char const *what = err.what();
       if (what && what[0]) {
         (baton->err).append(what);
+      } else {
+        if (baton->input->failOn == VIPS_FAIL_ON_WARNING) {
+          (baton->err).append("Warning treated as error due to failOn setting");
+          baton->errUseWarning = true;
         } else {
           (baton->err).append("Unknown error");
         }
       }
+    }
     // Clean up libvips' per-request data and threads
     vips_error_clear();
     vips_thread_shutdown();
@@ -1291,7 +1287,11 @@ class PipelineWorker : public Napi::AsyncWorker {
     // Handle warnings
     std::string warning = sharp::VipsWarningPop();
     while (!warning.empty()) {
+      if (baton->errUseWarning) {
+        (baton->err).append("\n").append(warning);
+      } else {
         debuglog.Call(Receiver().Value(), { Napi::String::New(env, warning) });
+      }
       warning = sharp::VipsWarningPop();
     }
 
@@ -1435,11 +1435,11 @@ class PipelineWorker : public Napi::AsyncWorker {
   std::string
   AssembleSuffixString(std::string extname, std::vector<std::pair<std::string, std::string>> options) {
     std::string argument;
-    for (auto const &option : options) {
+    for (const auto& [key, value] : options) {
       if (!argument.empty()) {
         argument += ",";
       }
-      argument += option.first + "=" + option.second;
+      argument += key + "=" + value;
     }
     return extname + "[" + argument + "]";
   }
@@ -1750,6 +1750,7 @@ Napi::Value pipeline(const Napi::CallbackInfo& info) {
   baton->gifReuse = sharp::AttrAsBool(options, "gifReuse");
   baton->gifProgressive = sharp::AttrAsBool(options, "gifProgressive");
   baton->tiffQuality = sharp::AttrAsUint32(options, "tiffQuality");
+  baton->tiffBigtiff = sharp::AttrAsBool(options, "tiffBigtiff");
   baton->tiffPyramid = sharp::AttrAsBool(options, "tiffPyramid");
   baton->tiffMiniswhite = sharp::AttrAsBool(options, "tiffMiniswhite");
   baton->tiffBitdepth = sharp::AttrAsUint32(options, "tiffBitdepth");

src/pipeline.h

@@ -1,13 +1,15 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
 #ifndef SRC_PIPELINE_H_
 #define SRC_PIPELINE_H_
 
 #include <memory>
 #include <string>
-#include <vector>
 #include <unordered_map>
+#include <vector>
 
 #include <napi.h>
 #include <vips/vips8>
@@ -175,6 +177,7 @@ struct PipelineBaton {
   bool gifProgressive;
   int tiffQuality;
   VipsForeignTiffCompression tiffCompression;
+  bool tiffBigtiff;
   VipsForeignTiffPredictor tiffPredictor;
   bool tiffPyramid;
   int tiffBitdepth;
@@ -197,6 +200,7 @@ struct PipelineBaton {
   bool jxlLossless;
   VipsBandFormat rawDepth;
   std::string err;
+  bool errUseWarning;
   int keepMetadata;
   int withMetadataOrientation;
   double withMetadataDensity;
@@ -350,6 +354,7 @@ struct PipelineBaton {
     gifProgressive(false),
     tiffQuality(80),
     tiffCompression(VIPS_FOREIGN_TIFF_COMPRESSION_JPEG),
+    tiffBigtiff(false),
     tiffPredictor(VIPS_FOREIGN_TIFF_PREDICTOR_HORIZONTAL),
     tiffPyramid(false),
     tiffBitdepth(8),
@@ -371,6 +376,7 @@ struct PipelineBaton {
     jxlEffort(7),
     jxlLossless(false),
     rawDepth(VIPS_FORMAT_UCHAR),
+    errUseWarning(false),
     keepMetadata(0),
     withMetadataOrientation(-1),
     withMetadataDensity(0.0),

src/sharp.cc

@@ -1,16 +1,18 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-#include <mutex>  // NOLINT(build/c++11)
+#include <mutex>
 
 #include <napi.h>
 #include <vips/vips8>
 
-#include "common.h"
+#include "./common.h"
-#include "metadata.h"
+#include "./metadata.h"
-#include "pipeline.h"
+#include "./pipeline.h"
-#include "utilities.h"
+#include "./stats.h"
-#include "stats.h"
+#include "./utilities.h"
 
 Napi::Object init(Napi::Env env, Napi::Object exports) {
   static std::once_flag sharp_vips_init_once;

src/stats.cc

@@ -1,15 +1,18 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-#include <numeric>
-#include <vector>
 #include <iostream>
+#include <numeric>
+#include <string>
+#include <vector>
 
 #include <napi.h>
 #include <vips/vips8>
 
-#include "common.h"
+#include "./common.h"
-#include "stats.h"
+#include "./stats.h"
 
 class StatsWorker : public Napi::AsyncWorker {
  public:

src/stats.h

@@ -1,10 +1,13 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
 #ifndef SRC_STATS_H_
 #define SRC_STATS_H_
 
 #include <string>
+#include <vector>
 #include <napi.h>
 
 #include "./common.h"
@@ -24,7 +27,7 @@ struct ChannelStats {
 
   ChannelStats(int minVal, int maxVal, double sumVal, double squaresSumVal,
     double meanVal, double stdevVal, int minXVal, int minYVal, int maxXVal, int maxYVal):
-    min(minVal), max(maxVal), sum(sumVal), squaresSum(squaresSumVal),
+    min(minVal), max(maxVal), sum(sumVal), squaresSum(squaresSumVal),  // NOLINT(build/include_what_you_use)
     mean(meanVal), stdev(stdevVal), minX(minXVal), minY(minYVal), maxX(maxXVal), maxY(maxYVal) {}
 };
 

src/utilities.cc

@@ -1,17 +1,19 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
 #include <cmath>
-#include <string>
 #include <cstdio>
+#include <string>
 
 #include <napi.h>
 #include <vips/vips8>
 #include <vips/vector.h>
 
-#include "common.h"
+#include "./common.h"
-#include "operations.h"
+#include "./operations.h"
-#include "utilities.h"
+#include "./utilities.h"
 
 /*
   Get and set cache limits

src/utilities.h

@@ -1,5 +1,7 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
 #ifndef SRC_UTILITIES_H_
 #define SRC_UTILITIES_H_

test/beforeEach.js

@@ -1,24 +0,0 @@
-// Copyright 2013 Lovell Fuller and others.
-// SPDX-License-Identifier: Apache-2.0
-
-'use strict';
-
-const sharp = require('../');
-
-const usingCache = !process.env.G_DEBUG;
-const usingSimd = !process.env.VIPS_NOVECTOR;
-const concurrency = Number(process.env.VIPS_CONCURRENCY) || 0;
-
-exports.mochaHooks = {
-  beforeEach () {
-    sharp.cache(usingCache);
-    sharp.simd(usingSimd);
-    sharp.concurrency(concurrency);
-  },
-
-  afterEach () {
-    if (global.gc) {
-      global.gc();
-    }
-  }
-};

test/bench/Dockerfile

@@ -15,7 +15,7 @@ RUN apt-get install -y imagemagick libmagick++-dev graphicsmagick
 # Install sharp
 RUN mkdir /tmp/sharp
 RUN cd /tmp && git clone --single-branch --branch $BRANCH https://github.com/lovell/sharp.git
-RUN cd /tmp/sharp && npm install --build-from-source
+RUN cd /tmp/sharp && npm install && npm run build
 
 # Install benchmark test
 RUN cd /tmp/sharp/test/bench && npm install --omit optional

test/bench/parallel.js

@@ -1,11 +1,11 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 process.env.UV_THREADPOOL_SIZE = 64;
 
-const assert = require('assert');
+const assert = require('node:assert');
 const async = require('async');
 
 const sharp = require('../../');
@@ -16,32 +16,29 @@ const height = 480;
 
 sharp.concurrency(1);
 
-const timer = setInterval(function () {
+const timer = setInterval(() => {
   console.dir(sharp.counters());
 }, 100);
 
-async.mapSeries([1, 1, 2, 4, 8, 16, 32, 64], function (parallelism, next) {
+async.mapSeries([1, 1, 2, 4, 8, 16, 32, 64], (parallelism, next) => {
-  const start = new Date().getTime();
+  const start = Date.now();
   async.times(parallelism,
-    function (id, callback) {
+    (_id, callback) => {
-      /* jslint unused: false */
+      sharp(fixtures.inputJpg).resize(width, height).toBuffer((err, buffer) => {
-      sharp(fixtures.inputJpg).resize(width, height).toBuffer(function (err, buffer) {
         buffer = null;
-        callback(err, new Date().getTime() - start);
+        callback(err, Date.now() - start);
       });
     },
-    function (err, ids) {
+    (err, ids) => {
       assert(!err);
       assert(ids.length === parallelism);
       ids.sort();
-      const mean = ids.reduce(function (a, b) {
+      const mean = ids.reduce((a, b) => a + b) / ids.length;
-        return a + b;
+      console.log(`${parallelism} parallel calls: fastest=${ids[0]}ms slowest=${ids[ids.length - 1]}ms mean=${mean}ms`);
-      }) / ids.length;
-      console.log(parallelism + ' parallel calls: fastest=' + ids[0] + 'ms slowest=' + ids[ids.length - 1] + 'ms mean=' + mean + 'ms');
       next();
     }
   );
-}, function () {
+}, () => {
   clearInterval(timer);
   console.dir(sharp.counters());
 });

test/bench/perf.js

@@ -1,10 +1,10 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const fs = require('node:fs');
-
+const { execSync } = require('node:child_process');
-const fs = require('fs');
-const { execSync } = require('child_process');
 
 const async = require('async');
 const Benchmark = require('benchmark');
@@ -12,7 +12,7 @@ const Benchmark = require('benchmark');
 const safeRequire = (name) => {
   try {
     return require(name);
-  } catch (err) {}
+  } catch (_err) {}
   return null;
 };
 
@@ -45,13 +45,13 @@ console.log(`Detected ${physicalCores} physical cores`);
 sharp.concurrency(physicalCores);
 
 async.series({
-  jpeg: function (callback) {
+  jpeg: (callback) => {
     const inputJpgBuffer = fs.readFileSync(fixtures.inputJpg);
     const jpegSuite = new Benchmark.Suite('jpeg');
     // jimp
     jpegSuite.add('jimp-buffer-buffer', {
       defer: true,
-      fn: async function (deferred) {
+      fn: async (deferred) => {
         const image = await Jimp.read(inputJpgBuffer);
         await image
           .resize({ w: width, h: height, mode: Jimp.RESIZE_BICUBIC })
@@ -60,7 +60,7 @@ async.series({
       }
     }).add('jimp-file-file', {
       defer: true,
-      fn: async function (deferred) {
+      fn: async (deferred) => {
         const image = await Jimp.read(fixtures.inputJpg);
         await image
           .resize({ w: width, h: height, mode: Jimp.RESIZE_BICUBIC })
@@ -71,14 +71,14 @@ async.series({
     // mapnik
     mapnik && jpegSuite.add('mapnik-file-file', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
-        mapnik.Image.open(fixtures.inputJpg, function (err, img) {
+        mapnik.Image.open(fixtures.inputJpg, (err, img) => {
           if (err) throw err;
           img
             .resize(width, height, {
               scaling_method: mapnik.imageScaling.lanczos
             })
-            .save(outputJpg, 'jpeg:quality=80', function (err) {
+            .save(outputJpg, 'jpeg:quality=80', (err) => {
               if (err) throw err;
               deferred.resolve();
             });
@@ -86,14 +86,14 @@ async.series({
       }
     }).add('mapnik-buffer-buffer', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
-        mapnik.Image.fromBytes(inputJpgBuffer, { max_size: 3000 }, function (err, img) {
+        mapnik.Image.fromBytes(inputJpgBuffer, { max_size: 3000 }, (err, img) => {
           if (err) throw err;
           img
             .resize(width, height, {
               scaling_method: mapnik.imageScaling.lanczos
             })
-            .encode('jpeg:quality=80', function (err) {
+            .encode('jpeg:quality=80', (err) => {
               if (err) throw err;
               deferred.resolve();
             });
@@ -103,7 +103,7 @@ async.series({
     // imagemagick
     jpegSuite.add('imagemagick-file-file', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         imagemagick.resize({
           srcPath: fixtures.inputJpg,
           dstPath: outputJpg,
@@ -112,7 +112,7 @@ async.series({
           height,
           format: 'jpg',
           filter: 'Lanczos'
-        }, function (err) {
+        }, (err) => {
           if (err) {
             throw err;
           } else {
@@ -124,12 +124,12 @@ async.series({
     // gm
     jpegSuite.add('gm-buffer-file', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         gm(inputJpgBuffer)
           .filter('Lanczos')
           .resize(width, height)
           .quality(80)
-          .write(outputJpg, function (err) {
+          .write(outputJpg, (err) => {
             if (err) {
               throw err;
             } else {
@@ -139,12 +139,12 @@ async.series({
       }
     }).add('gm-buffer-buffer', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         gm(inputJpgBuffer)
           .filter('Lanczos')
           .resize(width, height)
           .quality(80)
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -154,12 +154,12 @@ async.series({
       }
     }).add('gm-file-file', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         gm(fixtures.inputJpg)
           .filter('Lanczos')
           .resize(width, height)
           .quality(80)
-          .write(outputJpg, function (err) {
+          .write(outputJpg, (err) => {
             if (err) {
               throw err;
             } else {
@@ -169,12 +169,12 @@ async.series({
       }
     }).add('gm-file-buffer', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         gm(fixtures.inputJpg)
           .filter('Lanczos')
           .resize(width, height)
           .quality(80)
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -186,17 +186,17 @@ async.series({
     // tfjs
     tfjs && jpegSuite.add('tfjs-node-buffer-buffer', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         const decoded = tfjs.node.decodeJpeg(inputJpgBuffer);
         const resized = tfjs.image.resizeBilinear(decoded, [height, width]);
         tfjs
           .node
           .encodeJpeg(resized, 'rgb', 80)
-          .then(function () {
+          .then(() => {
             deferred.resolve();
             tfjs.disposeVariables();
           })
-          .catch(function (err) {
+          .catch((err) => {
             throw err;
           });
       }
@@ -204,10 +204,10 @@ async.series({
     // sharp
     jpegSuite.add('sharp-buffer-file', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
-          .toFile(outputJpg, function (err) {
+          .toFile(outputJpg, (err) => {
             if (err) {
               throw err;
             } else {
@@ -217,10 +217,10 @@ async.series({
       }
     }).add('sharp-buffer-buffer', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -230,10 +230,10 @@ async.series({
       }
     }).add('sharp-file-file', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(fixtures.inputJpg)
           .resize(width, height)
-          .toFile(outputJpg, function (err) {
+          .toFile(outputJpg, (err) => {
             if (err) {
               throw err;
             } else {
@@ -243,10 +243,10 @@ async.series({
       }
     }).add('sharp-stream-stream', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         const readable = fs.createReadStream(fixtures.inputJpg);
         const writable = fs.createWriteStream(outputJpg);
-        writable.on('finish', function () {
+        writable.on('finish', () => {
           deferred.resolve();
         });
         const pipeline = sharp()
@@ -255,10 +255,10 @@ async.series({
       }
     }).add('sharp-file-buffer', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(fixtures.inputJpg)
           .resize(width, height)
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -268,34 +268,34 @@ async.series({
       }
     }).add('sharp-promise', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
           .toBuffer()
-          .then(function () {
+          .then(() => {
             deferred.resolve();
           })
-          .catch(function (err) {
+          .catch((err) => {
             throw err;
           });
       }
-    }).on('cycle', function (event) {
+    }).on('cycle', (event) => {
-      console.log('jpeg ' + String(event.target));
+      console.log(`jpeg ${String(event.target)}`);
     }).on('complete', function () {
       callback(null, this.filter('fastest').map('name'));
     }).run();
   },
   // Effect of applying operations
-  operations: function (callback) {
+  operations: (callback) => {
     const inputJpgBuffer = fs.readFileSync(fixtures.inputJpg);
     const operationsSuite = new Benchmark.Suite('operations');
     operationsSuite.add('sharp-sharpen-mild', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
           .sharpen()
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -305,11 +305,11 @@ async.series({
       }
     }).add('sharp-sharpen-radius', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
           .sharpen(3, 1, 3)
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -319,11 +319,11 @@ async.series({
       }
     }).add('sharp-blur-mild', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
           .blur()
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -333,11 +333,11 @@ async.series({
       }
     }).add('sharp-blur-radius', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
           .blur(3)
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -347,11 +347,11 @@ async.series({
       }
     }).add('sharp-gamma', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
           .gamma()
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -361,11 +361,11 @@ async.series({
       }
     }).add('sharp-normalise', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
           .normalise()
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -375,11 +375,11 @@ async.series({
       }
     }).add('sharp-greyscale', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
           .greyscale()
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -389,12 +389,12 @@ async.series({
       }
     }).add('sharp-greyscale-gamma', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
           .gamma()
           .greyscale()
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -404,11 +404,11 @@ async.series({
       }
     }).add('sharp-progressive', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
           .jpeg({ progressive: true })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -418,11 +418,11 @@ async.series({
       }
     }).add('sharp-without-chroma-subsampling', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height)
           .jpeg({ chromaSubsampling: '4:4:4' })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -432,11 +432,11 @@ async.series({
       }
     }).add('sharp-rotate', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .rotate(90)
           .resize(width, height)
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -446,11 +446,11 @@ async.series({
       }
     }).add('sharp-without-simd', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp.simd(false);
         sharp(inputJpgBuffer)
           .resize(width, height)
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             sharp.simd(true);
             if (err) {
               throw err;
@@ -461,10 +461,10 @@ async.series({
       }
     }).add('sharp-random-access-read', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer, { sequentialRead: false })
           .resize(width, height)
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -474,13 +474,13 @@ async.series({
       }
     }).add('sharp-crop-entropy', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height, {
             fit: 'cover',
             position: sharp.strategy.entropy
           })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -490,13 +490,13 @@ async.series({
       }
     }).add('sharp-crop-attention', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height, {
             fit: 'cover',
             position: sharp.strategy.attention
           })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -504,21 +504,21 @@ async.series({
             }
           });
       }
-    }).on('cycle', function (event) {
+    }).on('cycle', (event) => {
-      console.log('operations ' + String(event.target));
+      console.log(`operations ${String(event.target)}`);
     }).on('complete', function () {
       callback(null, this.filter('fastest').map('name'));
     }).run();
   },
   // Comparative speed of kernels
-  kernels: function (callback) {
+  kernels: (callback) => {
     const inputJpgBuffer = fs.readFileSync(fixtures.inputJpg);
     (new Benchmark.Suite('kernels')).add('sharp-cubic', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height, { kernel: 'cubic' })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -528,10 +528,10 @@ async.series({
       }
     }).add('sharp-lanczos2', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height, { kernel: 'lanczos2' })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -541,10 +541,10 @@ async.series({
       }
     }).add('sharp-lanczos3', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height, { kernel: 'lanczos3' })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -554,10 +554,10 @@ async.series({
       }
     }).add('sharp-mks2013', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height, { kernel: 'mks2013' })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -567,10 +567,10 @@ async.series({
       }
     }).add('sharp-mks2021', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputJpgBuffer)
           .resize(width, height, { kernel: 'mks2021' })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -578,21 +578,21 @@ async.series({
             }
           });
       }
-    }).on('cycle', function (event) {
+    }).on('cycle', (event) => {
-      console.log('kernels ' + String(event.target));
+      console.log(`kernels ${String(event.target)}`);
     }).on('complete', function () {
       callback(null, this.filter('fastest').map('name'));
     }).run();
   },
   // PNG
-  png: function (callback) {
+  png: (callback) => {
     const inputPngBuffer = fs.readFileSync(fixtures.inputPngAlphaPremultiplicationLarge);
     const pngSuite = new Benchmark.Suite('png');
     const minSamples = 64;
     // jimp
     pngSuite.add('jimp-buffer-buffer', {
       defer: true,
-      fn: async function (deferred) {
+      fn: async (deferred) => {
         const image = await Jimp.read(inputPngBuffer);
         await image
           .resize({ w: width, h: heightPng, mode: Jimp.RESIZE_BICUBIC })
@@ -601,7 +601,7 @@ async.series({
       }
     }).add('jimp-file-file', {
       defer: true,
-      fn: async function (deferred) {
+      fn: async (deferred) => {
         const image = await Jimp.read(fixtures.inputPngAlphaPremultiplicationLarge);
         await image
           .resize({ w: width, h: heightPng, mode: Jimp.RESIZE_BICUBIC })
@@ -612,18 +612,18 @@ async.series({
     // mapnik
     mapnik && pngSuite.add('mapnik-file-file', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
-        mapnik.Image.open(fixtures.inputPngAlphaPremultiplicationLarge, function (err, img) {
+        mapnik.Image.open(fixtures.inputPngAlphaPremultiplicationLarge, (err, img) => {
           if (err) throw err;
-          img.premultiply(function (err, img) {
+          img.premultiply((err, img) => {
             if (err) throw err;
             img.resize(width, heightPng, {
               scaling_method: mapnik.imageScaling.lanczos
-            }, function (err, img) {
+            }, (err, img) => {
               if (err) throw err;
-              img.demultiply(function (err, img) {
+              img.demultiply((err, img) => {
                 if (err) throw err;
-                img.save(outputPng, 'png32:f=no:z=6', function (err) {
+                img.save(outputPng, 'png32:f=no:z=6', (err) => {
                   if (err) throw err;
                   deferred.resolve();
                 });
@@ -634,18 +634,18 @@ async.series({
       }
     }).add('mapnik-buffer-buffer', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
-        mapnik.Image.fromBytes(inputPngBuffer, { max_size: 3000 }, function (err, img) {
+        mapnik.Image.fromBytes(inputPngBuffer, { max_size: 3000 }, (err, img) => {
           if (err) throw err;
-          img.premultiply(function (err, img) {
+          img.premultiply((err, img) => {
             if (err) throw err;
             img.resize(width, heightPng, {
               scaling_method: mapnik.imageScaling.lanczos
-            }, function (err, img) {
+            }, (err, img) => {
               if (err) throw err;
-              img.demultiply(function (err, img) {
+              img.demultiply((err, img) => {
                 if (err) throw err;
-                img.encode('png32:f=no:z=6', function (err) {
+                img.encode('png32:f=no:z=6', (err) => {
                   if (err) throw err;
                   deferred.resolve();
                 });
@@ -658,7 +658,7 @@ async.series({
     // imagemagick
     pngSuite.add('imagemagick-file-file', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         imagemagick.resize({
           srcPath: fixtures.inputPngAlphaPremultiplicationLarge,
           dstPath: outputPng,
@@ -669,7 +669,7 @@ async.series({
             '-define', 'PNG:compression-level=6',
             '-define', 'PNG:compression-filter=0'
           ]
-        }, function (err) {
+        }, (err) => {
           if (err) {
             throw err;
           } else {
@@ -681,13 +681,13 @@ async.series({
     // gm
     pngSuite.add('gm-file-file', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         gm(fixtures.inputPngAlphaPremultiplicationLarge)
           .filter('Lanczos')
           .resize(width, heightPng)
           .define('PNG:compression-level=6')
           .define('PNG:compression-filter=0')
-          .write(outputPng, function (err) {
+          .write(outputPng, (err) => {
             if (err) {
               throw err;
             } else {
@@ -697,13 +697,13 @@ async.series({
       }
     }).add('gm-file-buffer', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         gm(fixtures.inputPngAlphaPremultiplicationLarge)
           .filter('Lanczos')
           .resize(width, heightPng)
           .define('PNG:compression-level=6')
           .define('PNG:compression-filter=0')
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -716,11 +716,11 @@ async.series({
     pngSuite.add('sharp-buffer-file', {
       defer: true,
       minSamples,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputPngBuffer)
           .resize(width, heightPng)
           .png({ compressionLevel: 6 })
-          .toFile(outputPng, function (err) {
+          .toFile(outputPng, (err) => {
             if (err) {
               throw err;
             } else {
@@ -731,11 +731,11 @@ async.series({
     }).add('sharp-buffer-buffer', {
       defer: true,
       minSamples,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputPngBuffer)
           .resize(width, heightPng)
           .png({ compressionLevel: 6 })
-          .toBuffer(function (err, data) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -746,11 +746,11 @@ async.series({
     }).add('sharp-file-file', {
       defer: true,
       minSamples,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(fixtures.inputPngAlphaPremultiplicationLarge)
           .resize(width, heightPng)
           .png({ compressionLevel: 6 })
-          .toFile(outputPng, function (err) {
+          .toFile(outputPng, (err) => {
             if (err) {
               throw err;
             } else {
@@ -761,11 +761,11 @@ async.series({
     }).add('sharp-file-buffer', {
       defer: true,
       minSamples,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(fixtures.inputPngAlphaPremultiplicationLarge)
           .resize(width, heightPng)
           .png({ compressionLevel: 6 })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -776,11 +776,11 @@ async.series({
     }).add('sharp-progressive', {
       defer: true,
       minSamples,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputPngBuffer)
           .resize(width, heightPng)
           .png({ compressionLevel: 6, progressive: true })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -791,11 +791,11 @@ async.series({
     }).add('sharp-adaptiveFiltering', {
       defer: true,
       minSamples,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputPngBuffer)
           .resize(width, heightPng)
           .png({ adaptiveFiltering: true, compressionLevel: 6 })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -806,11 +806,11 @@ async.series({
     }).add('sharp-compressionLevel=9', {
       defer: true,
       minSamples,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputPngBuffer)
           .resize(width, heightPng)
           .png({ compressionLevel: 9 })
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -819,21 +819,21 @@ async.series({
           });
       }
     });
-    pngSuite.on('cycle', function (event) {
+    pngSuite.on('cycle', (event) => {
-      console.log(' png ' + String(event.target));
+      console.log(` png ${String(event.target)}`);
     }).on('complete', function () {
       callback(null, this.filter('fastest').map('name'));
     }).run();
   },
   // WebP
-  webp: function (callback) {
+  webp: (callback) => {
     const inputWebPBuffer = fs.readFileSync(fixtures.inputWebP);
     (new Benchmark.Suite('webp')).add('sharp-buffer-file', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputWebPBuffer)
           .resize(width, height)
-          .toFile(outputWebP, function (err) {
+          .toFile(outputWebP, (err) => {
             if (err) {
               throw err;
             } else {
@@ -843,10 +843,10 @@ async.series({
       }
     }).add('sharp-buffer-buffer', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(inputWebPBuffer)
           .resize(width, height)
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -856,10 +856,10 @@ async.series({
       }
     }).add('sharp-file-file', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(fixtures.inputWebP)
           .resize(width, height)
-          .toFile(outputWebP, function (err) {
+          .toFile(outputWebP, (err) => {
             if (err) {
               throw err;
             } else {
@@ -869,10 +869,10 @@ async.series({
       }
     }).add('sharp-file-buffer', {
       defer: true,
-      fn: function (deferred) {
+      fn: (deferred) => {
         sharp(fixtures.inputWebP)
           .resize(width, height)
-          .toBuffer(function (err) {
+          .toBuffer((err) => {
             if (err) {
               throw err;
             } else {
@@ -880,19 +880,19 @@ async.series({
             }
           });
       }
-    }).on('cycle', function (event) {
+    }).on('cycle', (event) => {
-      console.log('webp ' + String(event.target));
+      console.log(`webp ${String(event.target)}`);
     }).on('complete', function () {
       callback(null, this.filter('fastest').map('name'));
     }).run();
   }
-}, function (err, results) {
+}, (err, results) => {
   if (err) {
     throw err;
   }
-  Object.keys(results).forEach(function (format) {
+  Object.keys(results).forEach((format) => {
     if (results[format].toString().substr(0, 5) !== 'sharp') {
-      console.log('sharp was slower than ' + results[format] + ' for ' + format);
+      console.log(`sharp was slower than ${results[format]} for ${format}`);
     }
   });
   console.dir(sharp.cache());

test/bench/random.js

@@ -1,11 +1,11 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
-
+  SPDX-License-Identifier: Apache-2.0
-'use strict';
+*/
 
 const imagemagick = require('imagemagick');
 const gm = require('gm');
-const assert = require('assert');
+const assert = require('node:assert');
 const Benchmark = require('benchmark');
 
 const sharp = require('../../');
@@ -16,13 +16,11 @@ sharp.cache(false);
 const min = 320;
 const max = 960;
 
-const randomDimension = function () {
+const randomDimension = () => Math.ceil((Math.random() * (max - min)) + min);
-  return Math.ceil((Math.random() * (max - min)) + min);
-};
 
 new Benchmark.Suite('random').add('imagemagick', {
   defer: true,
-  fn: function (deferred) {
+  fn: (deferred) => {
     imagemagick.resize({
       srcPath: fixtures.inputJpg,
       dstPath: fixtures.path('output.jpg'),
@@ -31,7 +29,7 @@ new Benchmark.Suite('random').add('imagemagick', {
       height: randomDimension(),
       format: 'jpg',
       filter: 'Lanczos'
-    }, function (err) {
+    }, (err) => {
       if (err) {
         throw err;
       } else {
@@ -41,12 +39,12 @@ new Benchmark.Suite('random').add('imagemagick', {
   }
 }).add('gm', {
   defer: true,
-  fn: function (deferred) {
+  fn: (deferred) => {
     gm(fixtures.inputJpg)
       .resize(randomDimension(), randomDimension())
       .filter('Lanczos')
       .quality(80)
-      .toBuffer(function (err, buffer) {
+      .toBuffer((err, buffer) => {
         if (err) {
           throw err;
         } else {
@@ -57,10 +55,10 @@ new Benchmark.Suite('random').add('imagemagick', {
   }
 }).add('sharp', {
   defer: true,
-  fn: function (deferred) {
+  fn: (deferred) => {
     sharp(fixtures.inputJpg)
       .resize(randomDimension(), randomDimension())
-      .toBuffer(function (err, buffer) {
+      .toBuffer((err, buffer) => {
         if (err) {
           throw err;
         } else {
@@ -69,9 +67,9 @@ new Benchmark.Suite('random').add('imagemagick', {
         }
       });
   }
-}).on('cycle', function (event) {
+}).on('cycle', (event) => {
   console.log(String(event.target));
 }).on('complete', function () {
   const winner = this.filter('fastest').map('name');
-  assert.strictEqual('sharp', String(winner), 'sharp was slower than ' + winner);
+  assert.strictEqual('sharp', String(winner), `sharp was slower than ${winner}`);
 }).run();

test/fixtures/index.js

@@ -1,16 +1,14 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const path = require('node:path');
-
-const path = require('path');
 const sharp = require('../../');
 const maxColourDistance = require('../../lib/sharp')._maxColourDistance;
 
 // Helpers
-const getPath = function (filename) {
+const getPath = (filename) => path.join(__dirname, filename);
-  return path.join(__dirname, filename);
-};
 
 // Generates a 64-bit-as-binary-string image fingerprint
 // Based on the dHash gradient method - see http://www.hackerfactor.com/blog/index.php?/archives/529-Kind-of-Like-That.html
@@ -22,7 +20,7 @@ async function fingerprint (image) {
     .resize(9, 8, { fit: sharp.fit.fill })
     .raw()
     .toBuffer()
-    .then(function (data) {
+    .then((data) => {
       let fingerprint = '';
       for (let col = 0; col < 8; col++) {
         for (let row = 0; row < 8; row++) {
@@ -115,6 +113,7 @@ module.exports = {
   inputTiff8BitDepth: getPath('8bit_depth.tiff'),
   inputTifftagPhotoshop: getPath('tifftag-photoshop.tiff'), // https://github.com/lovell/sharp/issues/1600
   inputTiffFogra: getPath('fogra-0-100-100-0.tif'), // https://github.com/lovell/sharp/issues/4045
+  inputTiffGeo: getPath('bonne.geo.tif'), // https://download.osgeo.org/geotiff/samples/intergraph
 
   inputJp2: getPath('relax.jp2'), // https://www.fnordware.com/j2k/relax.jp2
   inputJp2TileParts: getPath('relax_tileparts.jp2'), // kdu_expand -i relax.jp2 -o relax-tmp.tif ; kdu_compress -i relax-tmp.tif -o relax_tileparts.jp2 -jp2_space sRGB Clayers=8 -rate 1.0,0.04 Stiles='{128,128}' ORGtparts=L ; rm relax-tmp.tif
@@ -147,14 +146,12 @@ module.exports = {
   path: getPath,
 
   // Path for expected output images
-  expected: function (filename) {
+  expected: (filename) => getPath(path.join('expected', filename)),
-    return getPath(path.join('expected', filename));
-  },
 
   // Verify similarity of expected vs actual images via fingerprint
   // Specify distance threshold using `options={threshold: 42}`, default
   // `threshold` is 5;
-  assertSimilar: async function (expectedImage, actualImage, options, callback) {
+  assertSimilar: async (expectedImage, actualImage, options, callback) => {
     if (typeof options === 'function') {
       callback = options;
       options = {};
@@ -194,12 +191,12 @@ module.exports = {
     }
   },
 
-  assertMaxColourDistance: function (actualImagePath, expectedImagePath, acceptedDistance) {
+  assertMaxColourDistance: (actualImagePath, expectedImagePath, acceptedDistance) => {
     if (typeof actualImagePath !== 'string') {
-      throw new TypeError('`actualImagePath` must be a string; got ' + actualImagePath);
+      throw new TypeError(`\`actualImagePath\` must be a string; got ${actualImagePath}`);
     }
     if (typeof expectedImagePath !== 'string') {
-      throw new TypeError('`expectedImagePath` must be a string; got ' + expectedImagePath);
+      throw new TypeError(`\`expectedImagePath\` must be a string; got ${expectedImagePath}`);
     }
     if (typeof acceptedDistance !== 'number') {
       // Default threshold
@@ -207,7 +204,7 @@ module.exports = {
     }
     const distance = maxColourDistance(actualImagePath, expectedImagePath);
     if (distance > acceptedDistance) {
-      throw new Error('Expected maximum absolute distance of ' + acceptedDistance + ', actual ' + distance);
+      throw new Error(`Expected maximum absolute distance of ${acceptedDistance}, actual ${distance}`);
     }
   }
 

test/leak/leak.sh

@@ -18,5 +18,5 @@ for test in $TESTS; do
     --show-leak-kinds=definite,indirect \
     --num-callers=20 \
     --trace-children=yes \
-    node --expose-gc --zero-fill-buffers node_modules/.bin/mocha --no-config --slow=60000 --timeout=120000 --require test/beforeEach.js "test/unit/$test";
+    node --zero-fill-buffers --test "test/unit/$test";
 done

test/types/sharp.test-d.ts

@@ -1,6 +1,9 @@
+// biome-ignore-all lint/correctness/noUnusedFunctionParameters: types only test file
+// biome-ignore-all lint/correctness/noUnusedVariables: types only test file
+
 import sharp = require('../../');
 
-import { createReadStream, createWriteStream } from 'fs';
+import { createReadStream, createWriteStream } from 'node:fs';
 
 const input: Buffer = Buffer.alloc(0);
 const readableStream: NodeJS.ReadableStream = createReadStream(input);
@@ -79,7 +82,7 @@ sharp({
 let transformer = sharp()
   .resize(300)
   .on('info', (info: sharp.OutputInfo) => {
-    console.log('Image height is ' + info.height);
+    console.log(`Image height is ${info.height}`);
   });
 readableStream.pipe(transformer).pipe(writableStream);
 
@@ -318,7 +321,7 @@ sharp('input.gif')
 // From https://sharp.pixelplumbing.com/api-output#examples-9
 // Extract raw RGB pixel data from JPEG input
 sharp('input.jpg')
-  .raw()
+  .raw({ depth: 'ushort' })
   .toBuffer({ resolveWithObject: true })
   .then(({ data, info }) => {
     console.log(data);

test/unit.mjs

@@ -0,0 +1,16 @@
+import { readdir } from 'node:fs/promises';
+import { run } from 'node:test';
+import { spec } from 'node:test/reporters';
+
+const files = (await readdir('./test/unit')).map((f) => `./test/unit/${f}`);
+
+run({
+  files,
+  concurrency: true,
+  timeout: 60000,
+  coverage: true,
+  coverageIncludeGlobs: ['lib/*.js'],
+  branchCoverage: 100,
+})
+  .compose(new spec())
+  .pipe(process.stdout);

test/unit/affine.js

@@ -1,9 +1,10 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');

test/unit/alpha.js

@@ -1,18 +1,19 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 const fixtures = require('../fixtures');
 const sharp = require('../../');
 
-describe('Alpha transparency', function () {
+describe('Alpha transparency', () => {
-  it('Flatten to black', function (done) {
+  it('Flatten to black', (_t, done) => {
     sharp(fixtures.inputPngWithTransparency)
       .flatten()
       .resize(400, 300)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(400, info.width);
         assert.strictEqual(300, info.height);
@@ -20,14 +21,14 @@ describe('Alpha transparency', function () {
       });
   });
 
-  it('Flatten to RGB orange', function (done) {
+  it('Flatten to RGB orange', (_t, done) => {
     sharp(fixtures.inputPngWithTransparency)
       .resize(400, 300)
       .flatten({
         background: { r: 255, g: 102, b: 0 }
       })
       .jpeg({ chromaSubsampling: '4:4:4' })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(400, info.width);
         assert.strictEqual(300, info.height);
@@ -35,12 +36,12 @@ describe('Alpha transparency', function () {
       });
   });
 
-  it('Flatten to CSS/hex orange', function (done) {
+  it('Flatten to CSS/hex orange', (_t, done) => {
     sharp(fixtures.inputPngWithTransparency)
       .resize(400, 300)
       .flatten({ background: '#ff6600' })
       .jpeg({ chromaSubsampling: '4:4:4' })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(400, info.width);
         assert.strictEqual(300, info.height);
@@ -48,13 +49,13 @@ describe('Alpha transparency', function () {
       });
   });
 
-  it('Flatten 16-bit PNG with transparency to orange', function (done) {
+  it('Flatten 16-bit PNG with transparency to orange', (_t, done) => {
     const output = fixtures.path('output.flatten-rgb16-orange.jpg');
     sharp(fixtures.inputPngWithTransparency16bit)
       .flatten({
         background: { r: 255, g: 102, b: 0 }
       })
-      .toFile(output, function (err, info) {
+      .toFile(output, (err, info) => {
         if (err) throw err;
         assert.strictEqual(true, info.size > 0);
         assert.strictEqual(32, info.width);
@@ -64,10 +65,10 @@ describe('Alpha transparency', function () {
       });
   });
 
-  it('Do not flatten', function (done) {
+  it('Do not flatten', (_t, done) => {
     sharp(fixtures.inputPngWithTransparency)
       .flatten(false)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, _data, info) => {
         if (err) throw err;
         assert.strictEqual('png', info.format);
         assert.strictEqual(4, info.channels);
@@ -75,10 +76,10 @@ describe('Alpha transparency', function () {
       });
   });
 
-  it('Ignored for JPEG', function (done) {
+  it('Ignored for JPEG', (_t, done) => {
     sharp(fixtures.inputJpg)
       .flatten({ background: '#ff0000' })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, _data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(3, info.channels);
@@ -98,68 +99,60 @@ describe('Alpha transparency', function () {
     });
   });
 
-  it('Enlargement with non-nearest neighbor interpolation shouldn’t cause dark edges', function () {
+  it('Enlargement with non-nearest neighbor interpolation shouldn’t cause dark edges', () => {
     const base = 'alpha-premultiply-enlargement-2048x1536-paper.png';
-    const actual = fixtures.path('output.' + base);
+    const actual = fixtures.path(`output.${base}`);
     const expected = fixtures.expected(base);
     return sharp(fixtures.inputPngAlphaPremultiplicationSmall)
       .resize(2048, 1536)
       .toFile(actual)
-      .then(function () {
+      .then(() => {
         fixtures.assertMaxColourDistance(actual, expected, 102);
       });
   });
 
-  it('Reduction with non-nearest neighbor interpolation shouldn’t cause dark edges', function () {
+  it('Reduction with non-nearest neighbor interpolation shouldn’t cause dark edges', () => {
     const base = 'alpha-premultiply-reduction-1024x768-paper.png';
-    const actual = fixtures.path('output.' + base);
+    const actual = fixtures.path(`output.${base}`);
     const expected = fixtures.expected(base);
     return sharp(fixtures.inputPngAlphaPremultiplicationLarge)
       .resize(1024, 768)
       .toFile(actual)
-      .then(function () {
+      .then(() => {
         fixtures.assertMaxColourDistance(actual, expected, 102);
       });
   });
 
-  it('Removes alpha from fixtures with transparency, ignores those without', function () {
+  it('Removes alpha from fixtures with transparency, ignores those without', () => Promise.all([
-    return Promise.all([
       fixtures.inputPngWithTransparency,
       fixtures.inputPngWithTransparency16bit,
       fixtures.inputWebPWithTransparency,
       fixtures.inputJpg,
       fixtures.inputPng,
       fixtures.inputWebP
-    ].map(function (input) {
+    ].map((input) => sharp(input)
-      return sharp(input)
         .resize(10)
         .removeAlpha()
         .toBuffer({ resolveWithObject: true })
-        .then(function (result) {
+        .then((result) => {
           assert.strictEqual(3, result.info.channels);
-        });
+        }))));
-    }));
-  });
 
-  it('Ensures alpha from fixtures without transparency, ignores those with', function () {
+  it('Ensures alpha from fixtures without transparency, ignores those with', () => Promise.all([
-    return Promise.all([
       fixtures.inputPngWithTransparency,
       fixtures.inputPngWithTransparency16bit,
       fixtures.inputWebPWithTransparency,
       fixtures.inputJpg,
       fixtures.inputPng,
       fixtures.inputWebP
-    ].map(function (input) {
+    ].map((input) => sharp(input)
-      return sharp(input)
         .resize(10)
         .ensureAlpha()
         .png()
         .toBuffer({ resolveWithObject: true })
-        .then(function (result) {
+        .then((result) => {
           assert.strictEqual(4, result.info.channels);
-        });
+        }))));
-    }));
-  });
 
   it('Valid ensureAlpha value used for alpha channel', async () => {
     const background = { r: 255, g: 0, b: 0 };

test/unit/avif.js

@@ -1,9 +1,10 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const { inputAvif, inputJpg, inputGifAnimated } = require('../fixtures');
@@ -20,8 +21,8 @@ describe('AVIF', () => {
       .resize(32)
       .jpeg()
       .toBuffer();
-    const { size, ...metadata } = await sharp(data)
+    const { size, ...metadata } = await sharp(data).metadata();
-      .metadata();
+    void size;
     assert.deepStrictEqual(metadata, {
       autoOrient: {
         height: 13,
@@ -49,8 +50,8 @@ describe('AVIF', () => {
       .resize(32)
       .avif({ effort: 0 })
       .toBuffer();
-    const { size, ...metadata } = await sharp(data)
+    const { size, ...metadata } = await sharp(data).metadata();
-      .metadata();
+    void size;
     assert.deepStrictEqual(metadata, {
       autoOrient: {
         height: 26,
@@ -77,8 +78,8 @@ describe('AVIF', () => {
     const data = await sharp(inputAvif)
       .resize(32)
       .toBuffer();
-    const { size, ...metadata } = await sharp(data)
+    const { size, ...metadata } = await sharp(data).metadata();
-      .metadata();
+    void size;
     assert.deepStrictEqual(metadata, {
       autoOrient: {
         height: 13,
@@ -106,8 +107,8 @@ describe('AVIF', () => {
       .resize(10)
       .avif({ effort: 0 })
       .toBuffer();
-    const { size, ...metadata } = await sharp(data)
+    const { size, ...metadata } = await sharp(data).metadata();
-      .metadata();
+    void size;
     assert.deepStrictEqual(metadata, {
       autoOrient: {
         height: 300,
@@ -136,8 +137,8 @@ describe('AVIF', () => {
       .sharpen()
       .avif({ effort: 0 })
       .toBuffer();
-    const { size, ...metadata } = await sharp(data)
+    const { size, ...metadata } = await sharp(data).metadata();
-      .metadata();
+    void size;
     assert.deepStrictEqual(metadata, {
       autoOrient: {
         height: 26,
@@ -174,9 +175,10 @@ describe('AVIF', () => {
     )
   );
 
-  it('Invalid bitdepth value throws error', async () => {
+  it('Invalid bitdepth value throws error', () =>
-    assert.rejects(
+    assert.throws(
       () => sharp().avif({ bitdepth: 11 }),
-      /Error: Expected 8, 10 or 12 for bitdepth but received 11 of type number/);
+      /Expected 8, 10 or 12 for bitdepth but received 11 of type number/
-  });
+    )
+  );
 });

test/unit/bandbool.js

@@ -1,51 +1,52 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 const fixtures = require('../fixtures');
 const sharp = require('../../');
 
-describe('Bandbool per-channel boolean operations', function () {
+describe('Bandbool per-channel boolean operations', () => {
   [
     sharp.bool.and,
     sharp.bool.or,
     sharp.bool.eor
   ]
-    .forEach(function (op) {
+    .forEach((op) => {
-      it(op + ' operation', function (done) {
+      it(`${op} operation`, (_t, done) => {
         sharp(fixtures.inputPngBooleanNoAlpha)
           .bandbool(op)
           .toColourspace('b-w')
-          .toBuffer(function (err, data, info) {
+          .toBuffer((err, data, info) => {
             if (err) throw err;
             assert.strictEqual(200, info.width);
             assert.strictEqual(200, info.height);
             assert.strictEqual(1, info.channels);
-            fixtures.assertSimilar(fixtures.expected('bandbool_' + op + '_result.png'), data, done);
+            fixtures.assertSimilar(fixtures.expected(`bandbool_${op}_result.png`), data, done);
           });
       });
     });
 
-  it('sRGB image retains 3 channels', function (done) {
+  it('sRGB image retains 3 channels', (_t, done) => {
     sharp(fixtures.inputJpg)
       .bandbool('and')
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, _data, info) => {
         if (err) throw err;
         assert.strictEqual(3, info.channels);
         done();
       });
   });
 
-  it('Invalid operation', function () {
+  it('Invalid operation', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp().bandbool('fail');
     });
   });
 
-  it('Missing operation', function () {
+  it('Missing operation', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp().bandbool();
     });
   });

test/unit/blur.js

@@ -1,19 +1,20 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');
 
-describe('Blur', function () {
+describe('Blur', () => {
-  it('specific radius 1', function (done) {
+  it('specific radius 1', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(320, 240)
       .blur(1)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(320, info.width);
@@ -22,11 +23,11 @@ describe('Blur', function () {
       });
   });
 
-  it('specific radius 10', function (done) {
+  it('specific radius 10', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(320, 240)
       .blur(10)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(320, info.width);
@@ -35,11 +36,11 @@ describe('Blur', function () {
       });
   });
 
-  it('specific options.sigma 10', function (done) {
+  it('specific options.sigma 10', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(320, 240)
       .blur({ sigma: 10 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(320, info.width);
@@ -48,11 +49,11 @@ describe('Blur', function () {
       });
   });
 
-  it('specific radius 0.3', function (done) {
+  it('specific radius 0.3', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(320, 240)
       .blur(0.3)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(320, info.width);
@@ -61,11 +62,11 @@ describe('Blur', function () {
       });
   });
 
-  it('mild blur', function (done) {
+  it('mild blur', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(320, 240)
       .blur()
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(320, info.width);
@@ -74,17 +75,17 @@ describe('Blur', function () {
       });
   });
 
-  it('invalid radius', function () {
+  it('invalid radius', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpg).blur(0.1);
     });
   });
 
-  it('blurred image is smaller than non-blurred', function (done) {
+  it('blurred image is smaller than non-blurred', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(320, 240)
       .blur(false)
-      .toBuffer(function (err, notBlurred, info) {
+      .toBuffer((err, notBlurred, info) => {
         if (err) throw err;
         assert.strictEqual(true, notBlurred.length > 0);
         assert.strictEqual('jpeg', info.format);
@@ -93,7 +94,7 @@ describe('Blur', function () {
         sharp(fixtures.inputJpg)
           .resize(320, 240)
           .blur(true)
-          .toBuffer(function (err, blurred, info) {
+          .toBuffer((err, blurred, info) => {
             if (err) throw err;
             assert.strictEqual(true, blurred.length > 0);
             assert.strictEqual(true, blurred.length < notBlurred.length);
@@ -105,18 +106,18 @@ describe('Blur', function () {
       });
   });
 
-  it('invalid precision', function () {
+  it('invalid precision', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpg).blur({ sigma: 1, precision: 'invalid' });
     }, /Expected one of: integer, float, approximate for precision but received invalid of type string/);
   });
 
-  it('invalid minAmplitude', function () {
+  it('invalid minAmplitude', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpg).blur({ sigma: 1, minAmplitude: 0 });
     }, /Expected number between 0.001 and 1 for minAmplitude but received 0 of type number/);
 
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpg).blur({ sigma: 1, minAmplitude: 1.01 });
     }, /Expected number between 0.001 and 1 for minAmplitude but received 1.01 of type number/);
   });
@@ -149,8 +150,8 @@ describe('Blur', function () {
     await fixtures.assertSimilar(fixtures.expected('blur-10.jpg'), minAmplitudeLow);
   });
 
-  it('options.sigma is required if options object is passed', function () {
+  it('options.sigma is required if options object is passed', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpg).blur({ precision: 'invalid' });
     }, /Expected number between 0.3 and 1000 for options.sigma but received undefined of type undefined/);
   });

test/unit/boolean.js

@@ -1,15 +1,16 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const fs = require('node:fs');
-
+const { describe, it } = require('node:test');
-const fs = require('fs');
+const assert = require('node:assert');
-const assert = require('assert');
 
 const fixtures = require('../fixtures');
 const sharp = require('../../');
 
-describe('Boolean operation between two images', function () {
+describe('Boolean operation between two images', () => {
   const inputJpgBooleanTestBuffer = fs.readFileSync(fixtures.inputJpgBooleanTest);
 
   [
@@ -17,63 +18,63 @@ describe('Boolean operation between two images', function () {
     sharp.bool.or,
     sharp.bool.eor
   ]
-    .forEach(function (op) {
+    .forEach((op) => {
-      it(op + ' operation, file', function (done) {
+      it(`${op} operation, file`, (_t, done) => {
         sharp(fixtures.inputJpg)
           .resize(320, 240)
           .boolean(fixtures.inputJpgBooleanTest, op)
-          .toBuffer(function (err, data, info) {
+          .toBuffer((err, data, info) => {
             if (err) throw err;
             assert.strictEqual(320, info.width);
             assert.strictEqual(240, info.height);
-            fixtures.assertSimilar(fixtures.expected('boolean_' + op + '_result.jpg'), data, done);
+            fixtures.assertSimilar(fixtures.expected(`boolean_${op}_result.jpg`), data, done);
           });
       });
 
-      it(op + ' operation, buffer', function (done) {
+      it(`${op} operation, buffer`, (_t, done) => {
         sharp(fixtures.inputJpg)
           .resize(320, 240)
           .boolean(inputJpgBooleanTestBuffer, op)
-          .toBuffer(function (err, data, info) {
+          .toBuffer((err, data, info) => {
             if (err) throw err;
             assert.strictEqual(320, info.width);
             assert.strictEqual(240, info.height);
-            fixtures.assertSimilar(fixtures.expected('boolean_' + op + '_result.jpg'), data, done);
+            fixtures.assertSimilar(fixtures.expected(`boolean_${op}_result.jpg`), data, done);
           });
       });
 
-      it(op + ' operation, raw', function (done) {
+      it(`${op} operation, raw`, (_t, done) => {
         sharp(fixtures.inputJpgBooleanTest)
           .raw()
-          .toBuffer(function (err, data, info) {
+          .toBuffer((err, data, info) => {
             if (err) throw err;
             sharp(fixtures.inputJpg)
               .resize(320, 240)
               .boolean(data, op, { raw: info })
-              .toBuffer(function (err, data, info) {
+              .toBuffer((err, data, info) => {
                 if (err) throw err;
                 assert.strictEqual(320, info.width);
                 assert.strictEqual(240, info.height);
-                fixtures.assertSimilar(fixtures.expected('boolean_' + op + '_result.jpg'), data, done);
+                fixtures.assertSimilar(fixtures.expected(`boolean_${op}_result.jpg`), data, done);
               });
           });
       });
     });
 
-  it('Invalid operation', function () {
+  it('Invalid operation', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp().boolean(fixtures.inputJpgBooleanTest, 'fail');
     });
   });
 
-  it('Invalid operation, non-string', function () {
+  it('Invalid operation, non-string', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp().boolean(fixtures.inputJpgBooleanTest, null);
     });
   });
 
-  it('Missing input', function () {
+  it('Missing input', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp().boolean();
     });
   });

test/unit/clahe.js

@@ -1,139 +1,140 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../lib');
 const fixtures = require('../fixtures');
 
-describe('Clahe', function () {
+describe('Clahe', () => {
-  it('width 5 width 5 maxSlope 0', function (done) {
+  it('width 5 width 5 maxSlope 0', (_t, done) => {
     sharp(fixtures.inputJpgClahe)
       .clahe({ width: 5, height: 5, maxSlope: 0 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         fixtures.assertSimilar(fixtures.expected('clahe-5-5-0.jpg'), data, { threshold: 10 }, done);
       });
   });
 
-  it('width 5 width 5 maxSlope 5', function (done) {
+  it('width 5 width 5 maxSlope 5', (_t, done) => {
     sharp(fixtures.inputJpgClahe)
       .clahe({ width: 5, height: 5, maxSlope: 5 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         fixtures.assertSimilar(fixtures.expected('clahe-5-5-5.jpg'), data, done);
       });
   });
 
-  it('width 11 width 25 maxSlope 14', function (done) {
+  it('width 11 width 25 maxSlope 14', (_t, done) => {
     sharp(fixtures.inputJpgClahe)
       .clahe({ width: 11, height: 25, maxSlope: 14 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         fixtures.assertSimilar(fixtures.expected('clahe-11-25-14.jpg'), data, done);
       });
   });
 
-  it('width 50 width 50 maxSlope 0', function (done) {
+  it('width 50 width 50 maxSlope 0', (_t, done) => {
     sharp(fixtures.inputJpgClahe)
       .clahe({ width: 50, height: 50, maxSlope: 0 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         fixtures.assertSimilar(fixtures.expected('clahe-50-50-0.jpg'), data, done);
       });
   });
 
-  it('width 50 width 50 maxSlope 14', function (done) {
+  it('width 50 width 50 maxSlope 14', (_t, done) => {
     sharp(fixtures.inputJpgClahe)
       .clahe({ width: 50, height: 50, maxSlope: 14 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         fixtures.assertSimilar(fixtures.expected('clahe-50-50-14.jpg'), data, done);
       });
   });
 
-  it('width 100 width 50 maxSlope 3', function (done) {
+  it('width 100 width 50 maxSlope 3', (_t, done) => {
     sharp(fixtures.inputJpgClahe)
       .clahe({ width: 100, height: 50, maxSlope: 3 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         fixtures.assertSimilar(fixtures.expected('clahe-100-50-3.jpg'), data, done);
       });
   });
 
-  it('width 100 width 100 maxSlope 0', function (done) {
+  it('width 100 width 100 maxSlope 0', (_t, done) => {
     sharp(fixtures.inputJpgClahe)
       .clahe({ width: 100, height: 100, maxSlope: 0 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         fixtures.assertSimilar(fixtures.expected('clahe-100-100-0.jpg'), data, done);
       });
   });
 
-  it('invalid maxSlope', function () {
+  it('invalid maxSlope', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: 100, height: 100, maxSlope: -5 });
     });
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: 100, height: 100, maxSlope: 110 });
     });
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: 100, height: 100, maxSlope: 5.5 });
     });
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: 100, height: 100, maxSlope: 'a string' });
     });
   });
 
-  it('invalid width', function () {
+  it('invalid width', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: 100.5, height: 100 });
     });
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: -5, height: 100 });
     });
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: true, height: 100 });
     });
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: 'string test', height: 100 });
     });
   });
 
-  it('invalid height', function () {
+  it('invalid height', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: 100, height: 100.5 });
     });
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: 100, height: -5 });
     });
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: 100, height: true });
     });
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe({ width: 100, height: 'string test' });
     });
   });
 
-  it('invalid options object', function () {
+  it('invalid options object', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgClahe).clahe(100, 100, 5);
     });
   });
 
-  it('uses default maxSlope of 3', function (done) {
+  it('uses default maxSlope of 3', (_t, done) => {
     sharp(fixtures.inputJpgClahe)
       .clahe({ width: 100, height: 50 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         fixtures.assertSimilar(fixtures.expected('clahe-100-50-3.jpg'), data, done);

test/unit/clone.js

@@ -1,29 +1,30 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const fs = require('node:fs');
-
+const { afterEach, beforeEach, describe, it } = require('node:test');
-const fs = require('fs');
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');
 
-describe('Clone', function () {
+describe('Clone', () => {
-  beforeEach(function () {
+  beforeEach(() => {
     sharp.cache(false);
   });
-  afterEach(function () {
+  afterEach(() => {
     sharp.cache(true);
   });
 
-  it('Read from Stream and write to multiple Streams', function (done) {
+  it('Read from Stream and write to multiple Streams', (_t, done) => {
     let finishEventsExpected = 2;
     // Output stream 1
     const output1 = fixtures.path('output.multi-stream.1.jpg');
     const writable1 = fs.createWriteStream(output1);
-    writable1.on('finish', function () {
+    writable1.on('finish', () => {
-      sharp(output1).toBuffer(function (err, data, info) {
+      sharp(output1).toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(true, data.length > 0);
         assert.strictEqual(data.length, info.size);
@@ -40,8 +41,8 @@ describe('Clone', function () {
     // Output stream 2
     const output2 = fixtures.path('output.multi-stream.2.jpg');
     const writable2 = fs.createWriteStream(output2);
-    writable2.on('finish', function () {
+    writable2.on('finish', () => {
-      sharp(output2).toBuffer(function (err, data, info) {
+      sharp(output2).toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(true, data.length > 0);
         assert.strictEqual(data.length, info.size);
@@ -64,14 +65,14 @@ describe('Clone', function () {
     fs.createReadStream(fixtures.inputJpg).pipe(rotator);
   });
 
-  it('Stream-based input attaches finish event listener to original', function () {
+  it('Stream-based input attaches finish event listener to original', () => {
     const original = sharp();
     const clone = original.clone();
     assert.strictEqual(1, original.listenerCount('finish'));
     assert.strictEqual(0, clone.listenerCount('finish'));
   });
 
-  it('Non Stream-based input does not attach finish event listeners', function () {
+  it('Non Stream-based input does not attach finish event listeners', () => {
     const original = sharp(fixtures.inputJpg);
     const clone = original.clone();
     assert.strictEqual(0, original.listenerCount('finish'));

test/unit/colourspace.js

@@ -1,22 +1,23 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');
 
-describe('Colour space conversion', function () {
+describe('Colour space conversion', () => {
-  it('To greyscale', function (done) {
+  it('To greyscale', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(320, 240)
       .greyscale()
       .toFile(fixtures.path('output.greyscale-gamma-0.0.jpg'), done);
   });
 
-  it('To greyscale with gamma correction', function (done) {
+  it('To greyscale with gamma correction', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(320, 240)
       .gamma()
@@ -24,19 +25,19 @@ describe('Colour space conversion', function () {
       .toFile(fixtures.path('output.greyscale-gamma-2.2.jpg'), done);
   });
 
-  it('Not to greyscale', function (done) {
+  it('Not to greyscale', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(320, 240)
       .greyscale(false)
       .toFile(fixtures.path('output.greyscale-not.jpg'), done);
   });
 
-  it('Greyscale with single channel output', function (done) {
+  it('Greyscale with single channel output', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(320, 240)
       .greyscale()
       .toColourspace('b-w')
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(1, info.channels);
         assert.strictEqual(320, info.width);
@@ -55,10 +56,10 @@ describe('Colour space conversion', function () {
     assert.strictEqual(format, 'webp');
   });
 
-  it('From CMYK to sRGB', function (done) {
+  it('From CMYK to sRGB', (_t, done) => {
     sharp(fixtures.inputJpgWithCmykProfile)
       .resize(320)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(true, data.length > 0);
         assert.strictEqual('jpeg', info.format);
@@ -67,13 +68,13 @@ describe('Colour space conversion', function () {
       });
   });
 
-  it('From CMYK to sRGB with white background, not yellow', function (done) {
+  it('From CMYK to sRGB with white background, not yellow', (_t, done) => {
     sharp(fixtures.inputJpgWithCmykProfile)
       .resize(320, 240, {
         fit: sharp.fit.contain,
         background: 'white'
       })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(320, info.width);
@@ -82,10 +83,10 @@ describe('Colour space conversion', function () {
       });
   });
 
-  it('From profile-less CMYK to sRGB', function (done) {
+  it('From profile-less CMYK to sRGB', (_t, done) => {
     sharp(fixtures.inputJpgWithCmykNoProfile)
       .resize(320)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(320, info.width);
@@ -122,14 +123,14 @@ describe('Colour space conversion', function () {
     );
   });
 
-  it('CMYK profile to CMYK profile with negate', (done) => {
+  it('CMYK profile to CMYK profile with negate', (_t, done) => {
     sharp(fixtures.inputTiffFogra)
       .resize(320, 240)
       .toColourspace('cmyk')
       .pipelineColourspace('cmyk')
       .withIccProfile(fixtures.path('XCMYK 2017.icc'))
       .negate()
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('tiff', info.format);
         assert.strictEqual(320, info.width);
@@ -143,13 +144,13 @@ describe('Colour space conversion', function () {
       });
   });
 
-  it('From sRGB with RGB16 pipeline, resize with gamma, to sRGB', function (done) {
+  it('From sRGB with RGB16 pipeline, resize with gamma, to sRGB', (_t, done) => {
     sharp(fixtures.inputPngGradients)
       .pipelineColourspace('rgb16')
       .resize(320)
       .gamma()
       .toColourspace('srgb')
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(320, info.width);
         fixtures.assertSimilar(fixtures.expected('colourspace-gradients-gamma-resize.png'), data, {
@@ -177,15 +178,15 @@ describe('Colour space conversion', function () {
     assert.strictEqual(b, 34);
   });
 
-  it('Invalid pipelineColourspace input', function () {
+  it('Invalid pipelineColourspace input', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpg)
         .pipelineColorspace(null);
     }, /Expected string for colourspace but received null of type object/);
   });
 
-  it('Invalid toColourspace input', function () {
+  it('Invalid toColourspace input', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpg)
         .toColourspace(null);
     });

test/unit/composite.js

@@ -1,9 +1,10 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const fixtures = require('../fixtures');
 const sharp = require('../../');
@@ -322,7 +323,7 @@ describe('composite', () => {
   describe('string gravity', () => {
     Object.keys(sharp.gravity).forEach(gravity => {
       it(gravity, done => {
-        const expected = fixtures.expected('overlay-gravity-' + gravity + '.jpg');
+        const expected = fixtures.expected(`overlay-gravity-${gravity}.jpg`);
         sharp(fixtures.inputJpg)
           .resize(80)
           .composite([{
@@ -344,7 +345,7 @@ describe('composite', () => {
   describe('tile and gravity', () => {
     Object.keys(sharp.gravity).forEach(gravity => {
       it(gravity, done => {
-        const expected = fixtures.expected('overlay-tile-gravity-' + gravity + '.jpg');
+        const expected = fixtures.expected(`overlay-tile-gravity-${gravity}.jpg`);
         sharp(fixtures.inputJpg)
           .resize(80)
           .composite([{

test/unit/convolve.js

@@ -1,15 +1,16 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');
 
-describe('Convolve', function () {
+describe('Convolve', () => {
-  it('specific convolution kernel 1', function (done) {
+  it('specific convolution kernel 1', (_t, done) => {
     sharp(fixtures.inputPngStripesV)
       .convolve({
         width: 3,
@@ -22,7 +23,7 @@ describe('Convolve', function () {
           10, 20, 10
         ]
       })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('png', info.format);
         assert.strictEqual(320, info.width);
@@ -31,7 +32,7 @@ describe('Convolve', function () {
       });
   });
 
-  it('specific convolution kernel 2', function (done) {
+  it('specific convolution kernel 2', (_t, done) => {
     sharp(fixtures.inputPngStripesH)
       .convolve({
         width: 3,
@@ -42,7 +43,7 @@ describe('Convolve', function () {
           1, 0, 1
         ]
       })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('png', info.format);
         assert.strictEqual(320, info.width);
@@ -51,7 +52,7 @@ describe('Convolve', function () {
       });
   });
 
-  it('horizontal Sobel operator', function (done) {
+  it('horizontal Sobel operator', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(320, 240)
       .convolve({
@@ -63,7 +64,7 @@ describe('Convolve', function () {
           -1, 0, 1
         ]
       })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(320, info.width);
@@ -72,14 +73,14 @@ describe('Convolve', function () {
       });
   });
 
-  describe('invalid kernel specification', function () {
+  describe('invalid kernel specification', () => {
-    it('missing', function () {
+    it('missing', () => {
-      assert.throws(function () {
+      assert.throws(() => {
         sharp(fixtures.inputJpg).convolve({});
       });
     });
-    it('incorrect data format', function () {
+    it('incorrect data format', () => {
-      assert.throws(function () {
+      assert.throws(() => {
         sharp(fixtures.inputJpg).convolve({
           width: 3,
           height: 3,
@@ -87,8 +88,8 @@ describe('Convolve', function () {
         });
       });
     });
-    it('incorrect dimensions', function () {
+    it('incorrect dimensions', () => {
-      assert.throws(function () {
+      assert.throws(() => {
         sharp(fixtures.inputJpg).convolve({
           width: 3,
           height: 4,

test/unit/dilate.js

@@ -1,15 +1,14 @@
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');
 
-describe('Dilate', function () {
+describe('Dilate', () => {
-  it('dilate 1 png', function (done) {
+  it('dilate 1 png', (_t, done) => {
     sharp(fixtures.inputPngDotAndLines)
       .dilate(1)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('png', info.format);
         assert.strictEqual(100, info.width);
@@ -18,10 +17,10 @@ describe('Dilate', function () {
       });
   });
 
-  it('dilate 1 png - default width', function (done) {
+  it('dilate 1 png - default width', (_t, done) => {
     sharp(fixtures.inputPngDotAndLines)
       .dilate()
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('png', info.format);
         assert.strictEqual(100, info.width);
@@ -30,8 +29,8 @@ describe('Dilate', function () {
       });
   });
 
-  it('invalid dilation width', function () {
+  it('invalid dilation width', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpg).dilate(-1);
     });
   });

test/unit/erode.js

@@ -1,15 +1,14 @@
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');
 
-describe('Erode', function () {
+describe('Erode', () => {
-  it('erode 1 png', function (done) {
+  it('erode 1 png', (_t, done) => {
     sharp(fixtures.inputPngDotAndLines)
       .erode(1)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('png', info.format);
         assert.strictEqual(100, info.width);
@@ -18,10 +17,10 @@ describe('Erode', function () {
       });
   });
 
-  it('erode 1 png - default width', function (done) {
+  it('erode 1 png - default width', (_t, done) => {
     sharp(fixtures.inputPngDotAndLines)
       .erode()
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('png', info.format);
         assert.strictEqual(100, info.width);
@@ -30,8 +29,8 @@ describe('Erode', function () {
       });
   });
 
-  it('invalid erosion width', function () {
+  it('invalid erosion width', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpg).erode(-1);
     });
   });

test/unit/extend.js

@@ -1,20 +1,21 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');
 
-describe('Extend', function () {
+describe('Extend', () => {
-  describe('extend all sides equally via a single value', function () {
+  describe('extend all sides equally via a single value', () => {
-    it('JPEG', function (done) {
+    it('JPEG', (_t, done) => {
       sharp(fixtures.inputJpg)
         .resize(120)
         .extend(10)
-        .toBuffer(function (err, data, info) {
+        .toBuffer((err, data, info) => {
           if (err) throw err;
           assert.strictEqual(140, info.width);
           assert.strictEqual(118, info.height);
@@ -22,11 +23,11 @@ describe('Extend', function () {
         });
     });
 
-    it('Animated WebP', function (done) {
+    it('Animated WebP', (_t, done) => {
       sharp(fixtures.inputWebPAnimated, { pages: -1 })
         .resize(120)
         .extend(10)
-        .toBuffer(function (err, data, info) {
+        .toBuffer((err, data, info) => {
           if (err) throw err;
           assert.strictEqual(140, info.width);
           assert.strictEqual(140 * 9, info.height);
@@ -36,7 +37,7 @@ describe('Extend', function () {
   });
 
   ['background', 'copy', 'mirror', 'repeat'].forEach(extendWith => {
-    it(`extends all sides with animated WebP (${extendWith})`, function (done) {
+    it(`extends all sides with animated WebP (${extendWith})`, (_t, done) => {
       sharp(fixtures.inputWebPAnimated, { pages: -1 })
         .resize(120)
         .extend({
@@ -46,7 +47,7 @@ describe('Extend', function () {
           left: 40,
           right: 40
         })
-        .toBuffer(function (err, data, info) {
+        .toBuffer((err, data, info) => {
           if (err) throw err;
           assert.strictEqual(200, info.width);
           assert.strictEqual(200 * 9, info.height);
@@ -54,7 +55,7 @@ describe('Extend', function () {
         });
     });
 
-    it(`extend all sides equally with RGB (${extendWith})`, function (done) {
+    it(`extend all sides equally with RGB (${extendWith})`, (_t, done) => {
       sharp(fixtures.inputJpg)
         .resize(120)
         .extend({
@@ -65,7 +66,7 @@ describe('Extend', function () {
           right: 10,
           background: { r: 255, g: 0, b: 0 }
         })
-        .toBuffer(function (err, data, info) {
+        .toBuffer((err, data, info) => {
           if (err) throw err;
           assert.strictEqual(140, info.width);
           assert.strictEqual(118, info.height);
@@ -73,7 +74,7 @@ describe('Extend', function () {
         });
     });
 
-    it(`extend sides unequally with RGBA (${extendWith})`, function (done) {
+    it(`extend sides unequally with RGBA (${extendWith})`, (_t, done) => {
       sharp(fixtures.inputPngWithTransparency16bit)
         .resize(120)
         .extend({
@@ -83,7 +84,7 @@ describe('Extend', function () {
           right: 35,
           background: { r: 0, g: 0, b: 0, alpha: 0 }
         })
-        .toBuffer(function (err, data, info) {
+        .toBuffer((err, data, info) => {
           if (err) throw err;
           assert.strictEqual(165, info.width);
           assert.strictEqual(170, info.height);
@@ -91,7 +92,7 @@ describe('Extend', function () {
         });
     });
 
-    it(`PNG with 2 channels (${extendWith})`, function (done) {
+    it(`PNG with 2 channels (${extendWith})`, (_t, done) => {
       sharp(fixtures.inputPngWithGreyAlpha)
         .extend({
           extendWith,
@@ -101,7 +102,7 @@ describe('Extend', function () {
           right: 80,
           background: 'transparent'
         })
-        .toBuffer(function (err, data, info) {
+        .toBuffer((err, data, info) => {
           if (err) throw err;
           assert.strictEqual(true, data.length > 0);
           assert.strictEqual('png', info.format);
@@ -146,13 +147,13 @@ describe('Extend', function () {
     assert.strictEqual(1470, height);
   });
 
-  it('missing parameter fails', function () {
+  it('missing parameter fails', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp().extend();
     });
   });
-  it('negative fails', function () {
+  it('negative fails', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp().extend(-1);
     });
   });
@@ -190,7 +191,7 @@ describe('Extend', function () {
     assert.doesNotThrow(() => sharp().extend({ top: 1, left: 2, bottom: 3 }));
   });
 
-  it('should add alpha channel before extending with a transparent Background', function (done) {
+  it('should add alpha channel before extending with a transparent Background', (_t, done) => {
     sharp(fixtures.inputJpgWithLandscapeExif1)
       .extend({
         bottom: 10,
@@ -198,7 +199,7 @@ describe('Extend', function () {
         background: { r: 0, g: 0, b: 0, alpha: 0 }
       })
       .toFormat(sharp.format.png)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(610, info.width);
         assert.strictEqual(460, info.height);

test/unit/extract.js

@@ -1,18 +1,19 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');
 
-describe('Partial image extraction', function () {
+describe('Partial image extraction', () => {
-  it('JPEG', function (done) {
+  it('JPEG', (_t, done) => {
     sharp(fixtures.inputJpg)
       .extract({ left: 2, top: 2, width: 20, height: 20 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(20, info.width);
         assert.strictEqual(20, info.height);
@@ -20,10 +21,10 @@ describe('Partial image extraction', function () {
       });
   });
 
-  it('PNG', function (done) {
+  it('PNG', (_t, done) => {
     sharp(fixtures.inputPng)
       .extract({ left: 200, top: 300, width: 400, height: 200 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(400, info.width);
         assert.strictEqual(200, info.height);
@@ -31,10 +32,10 @@ describe('Partial image extraction', function () {
       });
   });
 
-  it('WebP', function (done) {
+  it('WebP', (_t, done) => {
     sharp(fixtures.inputWebP)
       .extract({ left: 100, top: 50, width: 125, height: 200 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(125, info.width);
         assert.strictEqual(200, info.height);
@@ -42,12 +43,12 @@ describe('Partial image extraction', function () {
       });
   });
 
-  describe('Animated WebP', function () {
+  describe('Animated WebP', () => {
-    it('Before resize', function (done) {
+    it('Before resize', (_t, done) => {
       sharp(fixtures.inputWebPAnimated, { pages: -1 })
         .extract({ left: 0, top: 30, width: 80, height: 20 })
         .resize(320, 80)
-        .toBuffer(function (err, data, info) {
+        .toBuffer((err, data, info) => {
           if (err) throw err;
           assert.strictEqual(320, info.width);
           assert.strictEqual(80 * 9, info.height);
@@ -55,11 +56,11 @@ describe('Partial image extraction', function () {
         });
     });
 
-    it('After resize', function (done) {
+    it('After resize', (_t, done) => {
       sharp(fixtures.inputWebPAnimated, { pages: -1 })
         .resize(320, 320)
         .extract({ left: 0, top: 120, width: 320, height: 80 })
-        .toBuffer(function (err, data, info) {
+        .toBuffer((err, data, info) => {
           if (err) throw err;
           assert.strictEqual(320, info.width);
           assert.strictEqual(80 * 9, info.height);
@@ -68,10 +69,10 @@ describe('Partial image extraction', function () {
     });
   });
 
-  it('TIFF', function (done) {
+  it('TIFF', (_t, done) => {
     sharp(fixtures.inputTiff)
       .extract({ left: 34, top: 63, width: 341, height: 529 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(341, info.width);
         assert.strictEqual(529, info.height);
@@ -79,11 +80,11 @@ describe('Partial image extraction', function () {
       });
   });
 
-  it('Before resize', function (done) {
+  it('Before resize', (_t, done) => {
     sharp(fixtures.inputJpg)
       .extract({ left: 10, top: 10, width: 10, height: 500 })
       .resize(100, 100)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(100, info.width);
         assert.strictEqual(100, info.height);
@@ -91,13 +92,13 @@ describe('Partial image extraction', function () {
       });
   });
 
-  it('After resize and crop', function (done) {
+  it('After resize and crop', (_t, done) => {
     sharp(fixtures.inputJpg)
       .resize(500, 500, {
         position: sharp.gravity.north
       })
       .extract({ left: 10, top: 10, width: 100, height: 100 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(100, info.width);
         assert.strictEqual(100, info.height);
@@ -105,14 +106,14 @@ describe('Partial image extraction', function () {
       });
   });
 
-  it('Before and after resize and crop', function (done) {
+  it('Before and after resize and crop', (_t, done) => {
     sharp(fixtures.inputJpg)
       .extract({ left: 0, top: 0, width: 700, height: 700 })
       .resize(500, 500, {
         position: sharp.gravity.north
       })
       .extract({ left: 10, top: 10, width: 100, height: 100 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(100, info.width);
         assert.strictEqual(100, info.height);
@@ -120,12 +121,12 @@ describe('Partial image extraction', function () {
       });
   });
 
-  it('Extract then rotate', function (done) {
+  it('Extract then rotate', (_t, done) => {
     sharp(fixtures.inputPngWithGreyAlpha)
       .extract({ left: 20, top: 10, width: 380, height: 280 })
       .rotate(90)
       .jpeg()
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(280, info.width);
         assert.strictEqual(380, info.height);
@@ -133,11 +134,11 @@ describe('Partial image extraction', function () {
       });
   });
 
-  it('Rotate then extract', function (done) {
+  it('Rotate then extract', (_t, done) => {
     sharp(fixtures.inputPngWithGreyAlpha)
       .rotate(90)
       .extract({ left: 20, top: 10, width: 280, height: 380 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(280, info.width);
         assert.strictEqual(380, info.height);
@@ -145,12 +146,12 @@ describe('Partial image extraction', function () {
       });
   });
 
-  it('Extract then rotate then extract', function (done) {
+  it('Extract then rotate then extract', (_t, done) => {
     sharp(fixtures.inputPngWithGreyAlpha)
       .extract({ left: 20, top: 10, width: 180, height: 280 })
       .rotate(90)
       .extract({ left: 20, top: 10, width: 200, height: 100 })
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(200, info.width);
         assert.strictEqual(100, info.height);
@@ -158,12 +159,12 @@ describe('Partial image extraction', function () {
       });
   });
 
-  it('Extract then rotate non-90 anagle', function (done) {
+  it('Extract then rotate non-90 anagle', (_t, done) => {
     sharp(fixtures.inputPngWithGreyAlpha)
       .extract({ left: 20, top: 10, width: 380, height: 280 })
       .rotate(45)
       .jpeg()
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(467, info.width);
         assert.strictEqual(467, info.height);
@@ -171,12 +172,12 @@ describe('Partial image extraction', function () {
       });
   });
 
-  it('Rotate then extract non-90 angle', function (done) {
+  it('Rotate then extract non-90 angle', (_t, done) => {
     sharp(fixtures.inputPngWithGreyAlpha)
       .rotate(45)
       .extract({ left: 20, top: 10, width: 380, height: 280 })
       .jpeg()
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(380, info.width);
         assert.strictEqual(280, info.height);
@@ -219,11 +220,11 @@ describe('Partial image extraction', function () {
         image: fixtures.inputJpgWithLandscapeExif8
       }
     ].forEach(({ name, image }) => {
-      it(name, function (done) {
+      it(name, (_t, done) => {
         sharp(image)
           .rotate()
           .extract({ left: 0, top: 208, width: 60, height: 40 })
-          .toBuffer(function (err, data) {
+          .toBuffer((err, data) => {
             if (err) throw err;
             fixtures.assertSimilar(fixtures.expected('rotate-mirror-extract.jpg'), data, done);
           });
@@ -231,67 +232,67 @@ describe('Partial image extraction', function () {
     });
   });
 
-  describe('Invalid parameters', function () {
+  describe('Invalid parameters', () => {
-    describe('using the legacy extract(top,left,width,height) syntax', function () {
+    describe('using the legacy extract(top,left,width,height) syntax', () => {
-      it('String top', function () {
+      it('String top', () => {
-        assert.throws(function () {
+        assert.throws(() => {
           sharp(fixtures.inputJpg).extract('spoons', 10, 10, 10);
         });
       });
 
-      it('Non-integral left', function () {
+      it('Non-integral left', () => {
-        assert.throws(function () {
+        assert.throws(() => {
           sharp(fixtures.inputJpg).extract(10, 10.2, 10, 10);
         });
       });
 
-      it('Negative width - negative', function () {
+      it('Negative width - negative', () => {
-        assert.throws(function () {
+        assert.throws(() => {
           sharp(fixtures.inputJpg).extract(10, 10, -10, 10);
         });
       });
 
-      it('Null height', function () {
+      it('Null height', () => {
-        assert.throws(function () {
+        assert.throws(() => {
           sharp(fixtures.inputJpg).extract(10, 10, 10, null);
         });
       });
     });
 
-    it('Undefined', function () {
+    it('Undefined', () => {
-      assert.throws(function () {
+      assert.throws(() => {
         sharp(fixtures.inputJpg).extract();
       });
     });
 
-    it('String top', function () {
+    it('String top', () => {
-      assert.throws(function () {
+      assert.throws(() => {
         sharp(fixtures.inputJpg).extract({ left: 10, top: 'spoons', width: 10, height: 10 });
       });
     });
 
-    it('Non-integral left', function () {
+    it('Non-integral left', () => {
-      assert.throws(function () {
+      assert.throws(() => {
         sharp(fixtures.inputJpg).extract({ left: 10.2, top: 10, width: 10, height: 10 });
       });
     });
 
-    it('Negative width - negative', function () {
+    it('Negative width - negative', () => {
-      assert.throws(function () {
+      assert.throws(() => {
         sharp(fixtures.inputJpg).extract({ left: 10, top: 10, width: -10, height: 10 });
       });
     });
 
-    it('Null height', function () {
+    it('Null height', () => {
-      assert.throws(function () {
+      assert.throws(() => {
         sharp(fixtures.inputJpg).extract({ left: 10, top: 10, width: 10, height: null });
       });
     });
 
-    it('Bad image area', function (done) {
+    it('Bad image area', (_t, done) => {
       sharp(fixtures.inputJpg)
         .extract({ left: 3000, top: 10, width: 10, height: 10 })
-        .toBuffer(function (err) {
+        .toBuffer((err) => {
           assert(err instanceof Error);
           assert.strictEqual(err.message, 'extract_area: bad extract area');
           done();
@@ -301,7 +302,7 @@ describe('Partial image extraction', function () {
     it('Multiple extract emits warning', () => {
       let warningMessage = '';
       const s = sharp();
-      s.on('warning', function (msg) { warningMessage = msg; });
+      s.on('warning', (msg) => { warningMessage = msg; });
       const options = { top: 0, left: 0, width: 1, height: 1 };
       s.extract(options).extract(options);
       assert.strictEqual(warningMessage, '');
@@ -312,7 +313,7 @@ describe('Partial image extraction', function () {
     it('Multiple rotate+extract emits warning', () => {
       let warningMessage = '';
       const s = sharp().rotate();
-      s.on('warning', function (msg) { warningMessage = msg; });
+      s.on('warning', (msg) => { warningMessage = msg; });
       const options = { top: 0, left: 0, width: 1, height: 1 };
       s.extract(options).extract(options);
       assert.strictEqual(warningMessage, '');
@@ -323,7 +324,7 @@ describe('Partial image extraction', function () {
     it('Multiple extract+resize emits warning', () => {
       let warningMessage = '';
       const s = sharp();
-      s.on('warning', function (msg) { warningMessage = msg; });
+      s.on('warning', (msg) => { warningMessage = msg; });
       const options = { top: 0, left: 0, width: 1, height: 1 };
       s.extract(options).extract(options);
       assert.strictEqual(warningMessage, '');

test/unit/extractChannel.js

@@ -1,19 +1,20 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');
 
-describe('Image channel extraction', function () {
+describe('Image channel extraction', () => {
-  it('Red channel', function (done) {
+  it('Red channel', (_t, done) => {
     sharp(fixtures.inputJpg)
       .extractChannel('red')
       .resize(320, 240)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(320, info.width);
         assert.strictEqual(240, info.height);
@@ -21,11 +22,11 @@ describe('Image channel extraction', function () {
       });
   });
 
-  it('Green channel', function (done) {
+  it('Green channel', (_t, done) => {
     sharp(fixtures.inputJpg)
       .extractChannel('green')
       .resize(320, 240)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(320, info.width);
         assert.strictEqual(240, info.height);
@@ -33,11 +34,11 @@ describe('Image channel extraction', function () {
       });
   });
 
-  it('Blue channel', function (done) {
+  it('Blue channel', (_t, done) => {
     sharp(fixtures.inputJpg)
       .extractChannel('blue')
       .resize(320, 240)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(320, info.width);
         assert.strictEqual(240, info.height);
@@ -45,11 +46,11 @@ describe('Image channel extraction', function () {
       });
   });
 
-  it('Blue channel by number', function (done) {
+  it('Blue channel by number', (_t, done) => {
     sharp(fixtures.inputJpg)
       .extractChannel(2)
       .resize(320, 240)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(320, info.width);
         assert.strictEqual(240, info.height);
@@ -66,23 +67,23 @@ describe('Image channel extraction', function () {
     assert.strictEqual(chroma, 104);
   });
 
-  it('Alpha from 16-bit PNG', function (done) {
+  it('Alpha from 16-bit PNG', (_t, done) => {
     const output = fixtures.path('output.extract-alpha-16bit.png');
     sharp(fixtures.inputPngWithTransparency16bit)
       .resize(16)
       .extractChannel(3)
-      .toFile(output, function (err) {
+      .toFile(output, (err) => {
         if (err) throw err;
         fixtures.assertMaxColourDistance(output, fixtures.expected('extract-alpha-16bit.png'));
         done();
       });
   });
 
-  it('Alpha from 2-channel input', function (done) {
+  it('Alpha from 2-channel input', (_t, done) => {
     const output = fixtures.path('output.extract-alpha-2-channel.png');
     sharp(fixtures.inputPngWithGreyAlpha)
       .extractChannel('alpha')
-      .toFile(output, function (err, info) {
+      .toFile(output, (err, info) => {
         if (err) throw err;
         assert.strictEqual(1, info.channels);
         fixtures.assertMaxColourDistance(output, fixtures.expected('extract-alpha-2-channel.png'));
@@ -90,15 +91,15 @@ describe('Image channel extraction', function () {
       });
   });
 
-  it('Invalid channel number', function () {
+  it('Invalid channel number', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpg)
         .extractChannel(-1);
     });
   });
 
-  it('No arguments', function () {
+  it('No arguments', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpg)
         .extractChannel();
     });

test/unit/failOn.js

@@ -1,19 +1,20 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
+const fs = require('node:fs');
-const fs = require('fs');
 
 const sharp = require('../../lib');
 const fixtures = require('../fixtures');
 
 describe('failOn', () => {
-  it('handles truncated JPEG', function (done) {
+  it('handles truncated JPEG', (_t, done) => {
     sharp(fixtures.inputJpgTruncated, { failOn: 'none' })
       .resize(32, 24)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(32, info.width);
@@ -22,17 +23,17 @@ describe('failOn', () => {
       });
   });
 
-  it('handles truncated PNG, emits warnings', function (done) {
+  it('handles truncated PNG, emits warnings', (_t, done) => {
     let isWarningEmitted = false;
     sharp(fixtures.inputPngTruncated, { failOn: 'none' })
-      .on('warning', function (warning) {
+      .on('warning', (warning) => {
         assert.ok(
           ['read gave 2 warnings', 'not enough data', 'end of stream']
             .some(m => warning.includes(m)));
         isWarningEmitted = true;
       })
       .resize(32, 24)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, _data, info) => {
         if (err) throw err;
         assert.strictEqual(true, isWarningEmitted);
         assert.strictEqual('png', info.format);
@@ -70,8 +71,8 @@ describe('failOn', () => {
     );
   });
 
-  it('returns errors to callback for truncated JPEG', function (done) {
+  it('returns errors to callback for truncated JPEG', (_t, done) => {
-    sharp(fixtures.inputJpgTruncated, { failOn: 'truncated' }).toBuffer(function (err, data, info) {
+    sharp(fixtures.inputJpgTruncated, { failOn: 'truncated' }).toBuffer((err, data, info) => {
       assert.ok(err.message.includes('VipsJpeg: premature end of'), err);
       assert.strictEqual(data, undefined);
       assert.strictEqual(info, undefined);
@@ -79,8 +80,8 @@ describe('failOn', () => {
     });
   });
 
-  it('returns errors to callback for truncated PNG', function (done) {
+  it('returns errors to callback for truncated PNG', (_t, done) => {
-    sharp(fixtures.inputPngTruncated, { failOn: 'truncated' }).toBuffer(function (err, data, info) {
+    sharp(fixtures.inputPngTruncated, { failOn: 'truncated' }).toBuffer((err, data, info) => {
       assert.ok(err.message.includes('read error'), err);
       assert.strictEqual(data, undefined);
       assert.strictEqual(info, undefined);
@@ -88,7 +89,7 @@ describe('failOn', () => {
     });
   });
 
-  it('rejects promises for truncated JPEG', function (done) {
+  it('rejects promises for truncated JPEG', (_t, done) => {
     sharp(fixtures.inputJpgTruncated, { failOn: 'error' })
       .toBuffer()
       .then(() => {
@@ -104,4 +105,11 @@ describe('failOn', () => {
     fs.createReadStream(fixtures.inputJpgTruncated).pipe(writable);
     return writable.toBuffer();
   });
+
+  it('converts warnings to error for GeoTIFF', async () => {
+    await assert.rejects(
+      sharp(fixtures.inputTiffGeo).toBuffer(),
+      /Tag 34737/
+    );
+  });
 });

test/unit/fixtures.js

@@ -1,28 +1,29 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 const fixtures = require('../fixtures');
 
-describe('Test fixtures', function () {
+describe('Test fixtures', () => {
-  describe('assertMaxColourDistance', function () {
+  describe('assertMaxColourDistance', () => {
-    it('should throw an Error when images have a different number of channels', function () {
+    it('should throw an Error when images have a different number of channels', () => {
-      assert.throws(function () {
+      assert.throws(() => {
         fixtures.assertMaxColourDistance(fixtures.inputPngOverlayLayer1, fixtures.inputJpg);
       });
     });
-    it('should throw an Error when images have different dimensions', function () {
+    it('should throw an Error when images have different dimensions', () => {
-      assert.throws(function () {
+      assert.throws(() => {
         fixtures.assertMaxColourDistance(fixtures.inputJpg, fixtures.inputJpgWithExif);
       });
     });
-    it('should accept a zero threshold when comparing an image to itself', function () {
+    it('should accept a zero threshold when comparing an image to itself', () => {
       const image = fixtures.inputPngOverlayLayer0;
       fixtures.assertMaxColourDistance(image, image, 0);
     });
-    it('should accept a numeric threshold for two different images', function () {
+    it('should accept a numeric threshold for two different images', () => {
       fixtures.assertMaxColourDistance(fixtures.inputPngOverlayLayer0, fixtures.inputPngOverlayLayer1, 100);
     });
   });

test/unit/gamma.js

@@ -1,18 +1,19 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const { describe, it } = require('node:test');
-
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');
 
-describe('Gamma correction', function () {
+describe('Gamma correction', () => {
-  it('value of 0.0 (disabled)', function (done) {
+  it('value of 0.0 (disabled)', (_t, done) => {
     sharp(fixtures.inputJpgWithGammaHoliness)
       .resize(129, 111)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(129, info.width);
@@ -21,11 +22,11 @@ describe('Gamma correction', function () {
       });
   });
 
-  it('value of 2.2 (default)', function (done) {
+  it('value of 2.2 (default)', (_t, done) => {
     sharp(fixtures.inputJpgWithGammaHoliness)
       .resize(129, 111)
       .gamma()
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(129, info.width);
@@ -34,11 +35,11 @@ describe('Gamma correction', function () {
       });
   });
 
-  it('value of 3.0', function (done) {
+  it('value of 3.0', (_t, done) => {
     sharp(fixtures.inputJpgWithGammaHoliness)
       .resize(129, 111)
       .gamma(3)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(129, info.width);
@@ -47,11 +48,11 @@ describe('Gamma correction', function () {
       });
   });
 
-  it('input value of 2.2, output value of 3.0', function (done) {
+  it('input value of 2.2, output value of 3.0', (_t, done) => {
     sharp(fixtures.inputJpgWithGammaHoliness)
       .resize(129, 111)
       .gamma(2.2, 3.0)
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(129, info.width);
@@ -60,12 +61,12 @@ describe('Gamma correction', function () {
       });
   });
 
-  it('alpha transparency', function (done) {
+  it('alpha transparency', (_t, done) => {
     sharp(fixtures.inputPngOverlayLayer1)
       .resize(320)
       .gamma()
       .jpeg()
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual('jpeg', info.format);
         assert.strictEqual(320, info.width);
@@ -73,14 +74,14 @@ describe('Gamma correction', function () {
       });
   });
 
-  it('invalid first parameter value', function () {
+  it('invalid first parameter value', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgWithGammaHoliness).gamma(4);
     });
   });
 
-  it('invalid second parameter value', function () {
+  it('invalid second parameter value', () => {
-    assert.throws(function () {
+    assert.throws(() => {
       sharp(fixtures.inputJpgWithGammaHoliness).gamma(2.2, 4);
     });
   });

test/unit/gif.js

@@ -1,10 +1,11 @@
-// Copyright 2013 Lovell Fuller and others.
+/*!
-// SPDX-License-Identifier: Apache-2.0
+  Copyright 2013 Lovell Fuller and others.
+  SPDX-License-Identifier: Apache-2.0
+*/
 
-'use strict';
+const fs = require('node:fs');
-
+const { describe, it } = require('node:test');
-const fs = require('fs');
+const assert = require('node:assert');
-const assert = require('assert');
 
 const sharp = require('../../');
 const fixtures = require('../fixtures');
@@ -198,11 +199,11 @@ describe('GIF input', () => {
     );
   });
 
-  it('should work with streams when only animated is set', function (done) {
+  it('should work with streams when only animated is set', (_t, done) => {
     fs.createReadStream(fixtures.inputGifAnimated)
       .pipe(sharp({ animated: true }))
       .gif()
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(true, data.length > 0);
         assert.strictEqual('gif', info.format);
@@ -210,11 +211,11 @@ describe('GIF input', () => {
       });
   });
 
-  it('should work with streams when only pages is set', function (done) {
+  it('should work with streams when only pages is set', (_t, done) => {
     fs.createReadStream(fixtures.inputGifAnimated)
       .pipe(sharp({ pages: -1 }))
       .gif()
-      .toBuffer(function (err, data, info) {
+      .toBuffer((err, data, info) => {
         if (err) throw err;
         assert.strictEqual(true, data.length > 0);
         assert.strictEqual('gif', info.format);