Release v0.31.2

CI: Only pin Python version on x64 macOS and Windows
See commit 18be09f
2026-02-04 13:46:19 +01:00 · 2022-11-04 09:44:37 +00:00 · 2022-11-03 14:49:12 +00:00 · 2022-11-03 14:40:16 +00:00 · 2022-11-03 14:09:23 +00:00 · 2022-11-03 12:50:58 +00:00
26 changed files with 367 additions and 189 deletions
--- a/.github/workflows/ci-darwin-arm64v8.yml
+++ b/.github/workflows/ci-darwin-arm64v8.yml
@@ -2,29 +2,33 @@ name: CI (MacStadium)
 on:
  - push
  - pull_request
+permissions: {}
 jobs:
  CI:
+    permissions:
+      contents: write # for npx prebuild to make release
+    name: Node.js ${{ matrix.nodejs_version }} ${{ matrix.nodejs_arch }} ${{ matrix.prebuild && '- prebuild' }}
    runs-on: macos-m1
    strategy:
      fail-fast: false
      matrix:
        include:
          - nodejs_version: 14
-            nodejs_architecture: x64
+            nodejs_arch: x64
          - nodejs_version: 18
-            nodejs_architecture: arm64
+            nodejs_arch: arm64
            prebuild: true
    defaults:
      run:
        shell: /usr/bin/arch -arch arm64e /bin/bash -l {0}
    steps:
-      - name: Dependencies
-        uses: actions/setup-node@v2
+      - name: Dependencies (Node.js)
+        uses: actions/setup-node@v3
        with:
          node-version: ${{ matrix.nodejs_version }}
-          architecture: ${{ matrix.nodejs_architecture }}
+          architecture: ${{ matrix.nodejs_arch }}
      - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
      - name: Install
        run: npm install --build-from-source --unsafe-perm
      - name: Test
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -2,31 +2,34 @@ name: CI (GitHub)
 on:
  - push
  - pull_request
+permissions: {}
 jobs:
  CI:
+    permissions:
+      contents: write # for npx prebuild to make release
+    name: ${{ matrix.container || matrix.os }} - Node.js ${{ matrix.nodejs_version }} ${{ matrix.nodejs_arch }} ${{ matrix.prebuild && '- prebuild' }}
    runs-on: ${{ matrix.os }}
    container: ${{ matrix.container }}
    strategy:
      fail-fast: false
      matrix:
        include:
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
            container: centos:7
            nodejs_version: 14
-            coverage: true
            prebuild: true
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
            container: centos:7
            nodejs_version: 16
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
            container: rockylinux:8
            nodejs_version: 18
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
            container: node:14-alpine3.12
            prebuild: true
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
            container: node:16-alpine3.12
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
            container: node:18-alpine3.14
          - os: macos-11
            nodejs_version: 14
@@ -75,14 +78,19 @@ jobs:
      - name: Dependencies (Linux musl)
        if: contains(matrix.container, 'alpine')
        run: apk add build-base git python3 font-noto --update-cache
-      - name: Dependencies (macOS, Windows)
+      - name: Dependencies (Python 3.10 - macOS, Windows)
+        if: contains(matrix.os, 'macos') || contains(matrix.os, 'windows')
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.10'
+      - name: Dependencies (Node.js - macOS, Windows)
        if: contains(matrix.os, 'macos') || contains(matrix.os, 'windows')
        uses: actions/setup-node@v3
        with:
          node-version: ${{ matrix.nodejs_version }}
          architecture: ${{ matrix.nodejs_arch }}
      - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
      - name: Fix working directory ownership
        if: matrix.container
        run: chown root.root .
@@ -90,11 +98,6 @@ jobs:
        run: npm install --build-from-source --unsafe-perm
      - name: Test
        run: npm test
-      - name: Coverage
-        if: matrix.coverage
-        uses: coverallsapp/github-action@v1.1.2
-        with:
-          github-token: ${{ secrets.GITHUB_TOKEN }}
      - name: Prebuild
        if: matrix.prebuild && startsWith(github.ref, 'refs/tags/')
        env:
--- a/README.md
+++ b/README.md
@@ -98,7 +98,6 @@ readableStream
 A [guide for contributors](https://github.com/lovell/sharp/blob/main/.github/CONTRIBUTING.md)
 covers reporting bugs, requesting features and submitting code changes.

-[![Test Coverage](https://coveralls.io/repos/lovell/sharp/badge.svg?branch=main)](https://coveralls.io/r/lovell/sharp?branch=main)
 [![Node-API v5](https://img.shields.io/badge/Node--API-v5-green.svg)](https://nodejs.org/dist/latest/docs/api/n-api.html#n_api_n_api_version_matrix)

 ## Licensing
--- a/docs/api-resize.md
+++ b/docs/api-resize.md
@@ -14,7 +14,7 @@ When both a `width` and `height` are provided, the possible methods by which the

 Some of these values are based on the [object-fit][1] CSS property.

-When using a `fit` of `cover` or `contain`, the default **position** is `centre`. Other options are:
+When using a **fit** of `cover` or `contain`, the default **position** is `centre`. Other options are:

 *   `sharp.position`: `top`, `right top`, `right`, `right bottom`, `bottom`, `left bottom`, `left`, `left top`.
 *   `sharp.gravity`: `north`, `northeast`, `east`, `southeast`, `south`, `southwest`, `west`, `northwest`, `center` or `centre`.
--- a/docs/changelog.md
+++ b/docs/changelog.md
@@ -2,7 +2,37 @@

 ## v0.31 - *eagle*

-Requires libvips v8.13.1
+Requires libvips v8.13.3
+
+### v0.31.2 - 4th November 2022
+
+* Upgrade to libvips v8.13.3 for upstream bug fixes.
+
+* Ensure manual flip, rotate, resize operation ordering (regression in 0.31.1)
+  [#3391](https://github.com/lovell/sharp/issues/3391)
+
+* Ensure auto-rotation works without resize (regression in 0.31.1)
+  [#3422](https://github.com/lovell/sharp/issues/3422)
+
+### v0.31.1 - 29th September 2022
+
+* Upgrade to libvips v8.13.2 for upstream bug fixes.
+
+* Ensure `close` event occurs after `end` event for Stream-based output.
+  [#3313](https://github.com/lovell/sharp/issues/3313)
+
+* Ensure `limitInputPixels` constructor option uses uint64.
+  [#3349](https://github.com/lovell/sharp/pull/3349)
+  [@marcosc90](https://github.com/marcosc90)
+
+* Ensure auto-rotation works with shrink-on-load and extract (regression in 0.31.0).
+  [#3352](https://github.com/lovell/sharp/issues/3352)
+
+* Ensure AVIF output is always 8-bit.
+  [#3358](https://github.com/lovell/sharp/issues/3358)
+
+* Ensure greyscale images can be trimmed (regression in 0.31.0).
+  [#3386](https://github.com/lovell/sharp/issues/3386)

 ### v0.31.0 - 5th September 2022

--- a/docs/humans.txt
+++ b/docs/humans.txt
@@ -260,3 +260,6 @@ GitHub: https://github.com/brahima

 Name: Anton Marsden
 GitHub: https://github.com/antonmarsden
+
+Name:  Marcos Casagrande
+GitHub: https://github.com/marcosc90
--- a/lib/libvips.js
+++ b/lib/libvips.js
@@ -89,6 +89,7 @@ const removeVendoredLibvips = function () {
  rm(vendorPath, { recursive: true, maxRetries: 3, force: true });
 };

+/* istanbul ignore next */
 const pkgConfigPath = function () {
  if (process.platform !== 'win32') {
    const brewPkgConfigPath = spawnSync(
--- a/lib/output.js
+++ b/lib/output.js
@@ -1205,7 +1205,7 @@ function _pipeline (callback) {
            this.push(data);
          }
          this.push(null);
-          this.emit('close');
+          this.on('end', () => this.emit('close'));
        });
      });
      if (this.streamInFinished) {
@@ -1221,7 +1221,7 @@ function _pipeline (callback) {
          this.push(data);
        }
        this.push(null);
-        this.emit('close');
+        this.on('end', () => this.emit('close'));
      });
    }
    return this;
--- a/lib/resize.js
+++ b/lib/resize.js
@@ -111,7 +111,7 @@ function isResizeExpected (options) {
 *
 * Some of these values are based on the [object-fit](https://developer.mozilla.org/en-US/docs/Web/CSS/object-fit) CSS property.
 *
- * When using a `fit` of `cover` or `contain`, the default **position** is `centre`. Other options are:
+ * When using a **fit** of `cover` or `contain`, the default **position** is `centre`. Other options are:
 * - `sharp.position`: `top`, `right top`, `right`, `right bottom`, `bottom`, `left bottom`, `left`, `left top`.
 * - `sharp.gravity`: `north`, `northeast`, `east`, `southeast`, `south`, `southwest`, `west`, `northwest`, `center` or `centre`.
 * - `sharp.strategy`: `cover` only, dynamically crop using either the `entropy` or `attention` strategy.
@@ -428,7 +428,7 @@ function extend (extend) {
 * @throws {Error} Invalid parameters
 */
 function extract (options) {
-  const suffix = isResizeExpected(this.options) || isRotationExpected(this.options) ? 'Post' : 'Pre';
+  const suffix = isResizeExpected(this.options) || this.options.widthPre !== -1 ? 'Post' : 'Pre';
  if (this.options[`width${suffix}`] !== -1) {
    this.options.debuglog('ignoring previous extract options');
  }
@@ -511,15 +511,12 @@ function trim (trim) {
    }
  } else if (is.object(trim)) {
    this._setBackgroundColourOption('trimBackground', trim.background);
-
    if (!is.defined(trim.threshold)) {
      this.options.trimThreshold = 10;
-    } else if (is.number(trim.threshold)) {
-      if (trim.threshold >= 0) {
-        this.options.trimThreshold = trim.threshold;
-      } else {
-        throw is.invalidParameterError('threshold', 'positive number', trim);
-      }
+    } else if (is.number(trim.threshold) && trim.threshold >= 0) {
+      this.options.trimThreshold = trim.threshold;
+    } else {
+      throw is.invalidParameterError('threshold', 'positive number', trim);
    }
  } else {
    throw is.invalidParameterError('trim', 'string, number or object', trim);
--- a/package.json
+++ b/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.31.0",
+  "version": "0.31.2",
  "author": "Lovell Fuller <npm@lovell.info>",
  "homepage": "https://github.com/lovell/sharp",
  "contributors": [
@@ -92,9 +92,8 @@
    "clean": "rm -rf node_modules/ build/ vendor/ .nyc_output/ coverage/ test/fixtures/output.*",
    "test": "npm run test-lint && npm run test-unit && npm run test-licensing",
    "test-lint": "semistandard && cpplint",
-    "test-unit": "nyc --reporter=lcov --branches=99 mocha --slow=1000 --timeout=60000 ./test/unit/*.js",
+    "test-unit": "nyc --reporter=lcov --reporter=text --check-coverage --branches=100 mocha --slow=1000 --timeout=30000 ./test/unit/*.js",
    "test-licensing": "license-checker --production --summary --onlyAllow=\"Apache-2.0;BSD;ISC;MIT\"",
-    "test-coverage": "./test/coverage/report.sh",
    "test-leak": "./test/leak/leak.sh",
    "docs-build": "documentation lint lib && node docs/build && node docs/search-index/build",
    "docs-serve": "cd docs && npx serve",
@@ -134,7 +133,7 @@
    "detect-libc": "^2.0.1",
    "node-addon-api": "^5.0.0",
    "prebuild-install": "^7.1.1",
-    "semver": "^7.3.7",
+    "semver": "^7.3.8",
    "simple-get": "^4.0.1",
    "tar-fs": "^2.1.1",
    "tunnel-agent": "^0.6.0"
@@ -147,8 +146,8 @@
    "extract-zip": "^2.0.1",
    "icc": "^2.0.0",
    "license-checker": "^25.0.1",
-    "mocha": "^10.0.0",
-    "mock-fs": "^5.1.4",
+    "mocha": "^10.1.0",
+    "mock-fs": "^5.2.0",
    "nyc": "^15.1.0",
    "prebuild": "^11.0.4",
    "rimraf": "^3.0.2",
@@ -156,19 +155,19 @@
  },
  "license": "Apache-2.0",
  "config": {
-    "libvips": "8.13.1",
+    "libvips": "8.13.3",
    "integrity": {
-      "darwin-arm64v8": "sha512-JdpGTx67RDbvRkg3ljFvTzqoq+oBXmMdDFEp0expDYXmP5HLH+GCkikmsROlGltgfKE2KqL/qwpxTEhIwMK/3A==",
-      "darwin-x64": "sha512-0Oh4/hEDnzV+X8MiiyUQ4G/Zh/MHw9rKstfuX0P1czgaxS2hX8Pxdbzdk1oqwTOEYVEGO/hMm9ItCVZ3RVPPaA==",
-      "linux-arm64v8": "sha512-9pSlPzEojt6ue5vXfASNMhQO1YS1p4i4Wydu+bzOfMtIPSBRXbu/+y8WELbbo03Ts7pftm9KtrMHitCVdy5EXw==",
-      "linux-armv6": "sha512-sv2FqS/ggpQly7h5/+nh8txQDulolE5ptaE90PO7iwfTont8N42pudeqootWKsuf0fRmkW4M92184VfVVYCvGw==",
-      "linux-armv7": "sha512-LmQIB8FDfasK6BsFhnE7ZI3LMlxh/rF5tZRNQ/uoTbF2xrtWQqqgiZgCifJByiEM+1tR7RxwNdnjxZhWvM9WmQ==",
-      "linux-x64": "sha512-JBRf8WBnlVw/K1jpSvmeZpnGZGjeqhG2NDEiQV/hUze3zgDGwDza4oiworaQExQmKcDrc2LJKF14Nsz1qQSNJw==",
-      "linuxmusl-arm64v8": "sha512-yzUQO5isDwsRpEUxbMXBeWp0sKhWghebrSK46SUF5mvB/kq6hZ7JbRuJ2aZjE84K/HUTyuCc0kE+M3m8naOs+g==",
-      "linuxmusl-x64": "sha512-H3Vz1QaaZ6X5iEbfPST7TPFwDO01tI8dk1osLm6l4a17BWCaOMaBQlqxgTgYrtd09JJ9CvGoq5fo5j5TPxUc4Q==",
-      "win32-arm64v8": "sha512-b5Ver+uwOJhdOGqvZVM+qF2KLKcowcac/wKK5Fg0czqlSMqP/KxDF2kxw2eKXUJNgfqe4eDH1QG/yTg2pQSetQ==",
-      "win32-ia32": "sha512-h/SJ/Yfn0ce9H70vt1wS8rZ4PfHnguCCTsOGik7e6O/e2AlBQOM0mKsPIB9jSOquoCP8rP0qF6AOPOjXKnCk+w==",
-      "win32-x64": "sha512-p9qpdWdhZooPteib92Kk+qF1vvzcScxvOwdIP8muhgo/A8uDI4/mqXCpEbMBw6vjETKlS3qo2JUbVF6+0/lyWQ=="
+      "darwin-arm64v8": "sha512-xFgYt7CtQSZcWoyUdzPTDNHbioZIrZSEU+gkMxzH4Cgjhi4/N49UsonnIZhKQoTBGloAqEexHeMx4rYTQ2Kgvw==",
+      "darwin-x64": "sha512-6SivWKzu15aUMMohe0wg7sNYMPETVnOe40BuWsnKOgzl3o5FpQqNSgs+68Mi8Za3Qti9/DaR+H/fyD0x48Af2w==",
+      "linux-arm64v8": "sha512-b+iI9V/ehgDabXYRQcvqa5CEysh+1FQsgFmYc358StCrJCDahwNmsQdsiH1GOVd5WaWh5wHUGByPwMmFOO16Aw==",
+      "linux-armv6": "sha512-zRP2F+EiustLE4bXSH8AHCxwfemh9d+QuvmPjira/HL6uJOUuA7SyQgVV1TPwTQle2ioCNnKPm7FEB/MAiT+ug==",
+      "linux-armv7": "sha512-6OCChowE5lBXXXAZrnGdA9dVktg7UdODEBpE5qTroiAJYZv4yXRMgyDFYajok7du2NTgoklhxGk8d9+4vGv5hg==",
+      "linux-x64": "sha512-OTmlmP2r8ozGKdB96X+K5oQE1ojVZanqLqqKlwDpEnfixyIaDGYbVzcjWBNGU3ai/26bvkaCkjynnc2ecYcsuA==",
+      "linuxmusl-arm64v8": "sha512-Qh5Wi+bkKTohFYHzSPssfjMhIkD6z6EHbVmnwmWSsgY9zsUBStFp6+mKcNTQfP5YM5Mz06vJOkLHX2OzEr5TzA==",
+      "linuxmusl-x64": "sha512-DwB4Fs3+ISw9etaLCANkueZDdk758iOS+wNp4TKZkHdq0al6B/3Pk7OHLR8a9E3H6wYDD328u++dcJzip5tacA==",
+      "win32-arm64v8": "sha512-96r3W+O4BtX602B1MtxU5Ru4lKzRRTZqM4OQEBJ//TNL3fiCZdd9agD+RQBjaeR4KFIyBSt3F7IE425ZWmxz+w==",
+      "win32-ia32": "sha512-qfN1MsfQGek1QQd1UNW7JT+5K5Ne1suFQ2GpgpYm3JLSpIve/tz2vOGEGzvTVssOBADJvAkTDFt+yIi3PgU9pA==",
+      "win32-x64": "sha512-eb3aAmjbVVBVRbiYgebQwoxkAt69WI8nwmKlilSQ3kWqoc0pXfIe322rF2UR8ebbISCGvYRUfzD2r1k92RXISQ=="
    },
    "runtime": "napi",
    "target": 7
--- a/src/common.cc
+++ b/src/common.cc
@@ -507,9 +507,10 @@ namespace sharp {
        }
      }
    }
+
    // Limit input images to a given number of pixels, where pixels = width * height
    if (descriptor->limitInputPixels > 0 &&
-      static_cast<uint64_t>(image.width() * image.height()) > descriptor->limitInputPixels) {
+      static_cast<uint64_t>(image.width()) * image.height() > descriptor->limitInputPixels) {
      throw vips::VError("Input image exceeds pixel limit");
    }
    return std::make_tuple(image, imageType);
@@ -556,6 +557,7 @@ namespace sharp {
  VImage RemoveExifOrientation(VImage image) {
    VImage copy = image.copy();
    copy.remove(VIPS_META_ORIENTATION);
+    copy.remove("exif-ifd0-Orientation");
    return copy;
  }

--- a/src/common.h
+++ b/src/common.h
@@ -26,8 +26,8 @@

 #if (VIPS_MAJOR_VERSION < 8) || \
  (VIPS_MAJOR_VERSION == 8 && VIPS_MINOR_VERSION < 13) || \
-  (VIPS_MAJOR_VERSION == 8 && VIPS_MINOR_VERSION == 13 && VIPS_MICRO_VERSION < 1)
-#error "libvips version 8.13.1+ is required - please see https://sharp.pixelplumbing.com/install"
+  (VIPS_MAJOR_VERSION == 8 && VIPS_MINOR_VERSION == 13 && VIPS_MICRO_VERSION < 3)
+#error "libvips version 8.13.3+ is required - please see https://sharp.pixelplumbing.com/install"
 #endif

 #if ((!defined(__clang__)) && defined(__GNUC__) && (__GNUC__ < 4 || (__GNUC__ == 4 && __GNUC_MINOR__ < 6)))
--- a/src/operations.cc
+++ b/src/operations.cc
@@ -289,17 +289,20 @@ namespace sharp {
      background = image.extract_area(0, 0, 1, 1)(0, 0);
      multiplier = 1.0;
    }
-    if (background.size() == 4) {
+    if (HasAlpha(image) && background.size() == 4) {
      // Just discard the alpha because flattening the background colour with
      // itself (effectively what find_trim() does) gives the same result
      backgroundAlpha[0] = background[3] * multiplier;
    }
-    background = {
-      background[0] * multiplier,
-      background[1] * multiplier,
-      background[2] * multiplier
-    };
-
+    if (image.bands() > 2) {
+      background = {
+        background[0] * multiplier,
+        background[1] * multiplier,
+        background[2] * multiplier
+      };
+    } else {
+      background[0] = background[0] * multiplier;
+    }
    int left, top, width, height;
    left = image.find_trim(&top, &width, &height, VImage::option()
      ->set("background", background)
--- a/src/pipeline.cc
+++ b/src/pipeline.cc
@@ -81,32 +81,47 @@ class PipelineWorker : public Napi::AsyncWorker {
      int pageHeight = sharp::GetPageHeight(image);

      // Calculate angle of rotation
-      VipsAngle rotation;
-      bool flip = FALSE;
-      bool flop = FALSE;
+      VipsAngle rotation = VIPS_ANGLE_D0;
+      VipsAngle autoRotation = VIPS_ANGLE_D0;
+      bool autoFlip = FALSE;
+      bool autoFlop = FALSE;
      if (baton->useExifOrientation) {
        // Rotate and flip image according to Exif orientation
-        std::tie(rotation, flip, flop) = CalculateExifRotationAndFlip(sharp::ExifOrientation(image));
+        std::tie(autoRotation, autoFlip, autoFlop) = CalculateExifRotationAndFlip(sharp::ExifOrientation(image));
+        image = sharp::RemoveExifOrientation(image);
      } else {
        rotation = CalculateAngleRotation(baton->angle);
      }

      // Rotate pre-extract
-      if (baton->rotateBeforePreExtract) {
+      bool const shouldRotateBefore = baton->rotateBeforePreExtract &&
+        (rotation != VIPS_ANGLE_D0 || autoRotation != VIPS_ANGLE_D0 ||
+          autoFlip || baton->flip || autoFlop || baton->flop ||
+          baton->rotationAngle != 0.0);
+
+      if (shouldRotateBefore) {
+        if (autoRotation != VIPS_ANGLE_D0) {
+          image = image.rot(autoRotation);
+          autoRotation = VIPS_ANGLE_D0;
+        }
+        if (autoFlip) {
+          image = image.flip(VIPS_DIRECTION_VERTICAL);
+          autoFlip = FALSE;
+        } else if (baton->flip) {
+          image = image.flip(VIPS_DIRECTION_VERTICAL);
+          baton->flip = FALSE;
+        }
+        if (autoFlop) {
+          image = image.flip(VIPS_DIRECTION_HORIZONTAL);
+          autoFlop = FALSE;
+        } else if (baton->flop) {
+          image = image.flip(VIPS_DIRECTION_HORIZONTAL);
+          baton->flop = FALSE;
+        }
        if (rotation != VIPS_ANGLE_D0) {
          image = image.rot(rotation);
+          rotation = VIPS_ANGLE_D0;
        }
-        if (flip) {
-          image = image.flip(VIPS_DIRECTION_VERTICAL);
-        }
-        if (flop) {
-          image = image.flip(VIPS_DIRECTION_HORIZONTAL);
-        }
-        if (rotation != VIPS_ANGLE_D0 || flip || flop) {
-          image = sharp::RemoveExifOrientation(image);
-        }
-        flop = FALSE;
-        flip = FALSE;
        if (baton->rotationAngle != 0.0) {
          MultiPageUnsupported(nPages, "Rotate");
          std::vector<double> background;
@@ -147,7 +162,9 @@ class PipelineWorker : public Napi::AsyncWorker {
      int targetResizeHeight = baton->height;

      // Swap input output width and height when rotating by 90 or 270 degrees
-      bool swap = !baton->rotateBeforePreExtract && (rotation == VIPS_ANGLE_D90 || rotation == VIPS_ANGLE_D270);
+      bool swap = !baton->rotateBeforePreExtract &&
+        (rotation == VIPS_ANGLE_D90 || rotation == VIPS_ANGLE_D270 ||
+          autoRotation == VIPS_ANGLE_D90 || autoRotation == VIPS_ANGLE_D270);

      // Shrink to pageHeight, so we work for multi-page images
      std::tie(hshrink, vshrink) = sharp::ResolveShrink(
@@ -167,7 +184,7 @@ class PipelineWorker : public Napi::AsyncWorker {
      //  - input colourspace is not specified;
      bool const shouldPreShrink = (targetResizeWidth > 0 || targetResizeHeight > 0) &&
        baton->gamma == 0 && baton->topOffsetPre == -1 && baton->trimThreshold == 0.0 &&
-        baton->colourspaceInput == VIPS_INTERPRETATION_LAST;
+        baton->colourspaceInput == VIPS_INTERPRETATION_LAST && !shouldRotateBefore;

      if (shouldPreShrink) {
        // The common part of the shrink: the bit by which both axes must be shrunk
@@ -364,30 +381,21 @@ class PipelineWorker : public Napi::AsyncWorker {
          ->set("kernel", baton->kernel));
      }

+      // Auto-rotate post-extract
+      if (autoRotation != VIPS_ANGLE_D0) {
+        image = image.rot(autoRotation);
+      }
      // Flip (mirror about Y axis)
-      if (baton->flip || flip) {
+      if (baton->flip || autoFlip) {
        image = image.flip(VIPS_DIRECTION_VERTICAL);
-        image = sharp::RemoveExifOrientation(image);
      }
-
      // Flop (mirror about X axis)
-      if (baton->flop || flop) {
+      if (baton->flop || autoFlop) {
        image = image.flip(VIPS_DIRECTION_HORIZONTAL);
-        image = sharp::RemoveExifOrientation(image);
      }
-
      // Rotate post-extract 90-angle
-      if (!baton->rotateBeforePreExtract && rotation != VIPS_ANGLE_D0) {
+      if (rotation != VIPS_ANGLE_D0) {
        image = image.rot(rotation);
-        if (flip) {
-          image = image.flip(VIPS_DIRECTION_VERTICAL);
-          flip = FALSE;
-        }
-        if (flop) {
-          image = image.flip(VIPS_DIRECTION_HORIZONTAL);
-          flop = FALSE;
-        }
-        image = sharp::RemoveExifOrientation(image);
      }

      // Join additional color channels to the image
@@ -899,12 +907,13 @@ class PipelineWorker : public Napi::AsyncWorker {
        } else if (baton->formatOut == "heif" ||
          (baton->formatOut == "input" && inputImageType == sharp::ImageType::HEIF)) {
          // Write HEIF to buffer
-          image = sharp::RemoveAnimationProperties(image);
+          image = sharp::RemoveAnimationProperties(image).cast(VIPS_FORMAT_UCHAR);
          VipsArea *area = reinterpret_cast<VipsArea*>(image.heifsave_buffer(VImage::option()
            ->set("strip", !baton->withMetadata)
            ->set("Q", baton->heifQuality)
            ->set("compression", baton->heifCompression)
            ->set("effort", baton->heifEffort)
+            ->set("bitdepth", 8)
            ->set("subsample_mode", baton->heifChromaSubsampling == "4:4:4"
              ? VIPS_FOREIGN_SUBSAMPLE_OFF : VIPS_FOREIGN_SUBSAMPLE_ON)
            ->set("lossless", baton->heifLossless)));
@@ -1070,12 +1079,13 @@ class PipelineWorker : public Napi::AsyncWorker {
        } else if (baton->formatOut == "heif" || (mightMatchInput && isHeif) ||
          (willMatchInput && inputImageType == sharp::ImageType::HEIF)) {
          // Write HEIF to file
-          image = sharp::RemoveAnimationProperties(image);
+          image = sharp::RemoveAnimationProperties(image).cast(VIPS_FORMAT_UCHAR);
          image.heifsave(const_cast<char*>(baton->fileOut.data()), VImage::option()
            ->set("strip", !baton->withMetadata)
            ->set("Q", baton->heifQuality)
            ->set("compression", baton->heifCompression)
            ->set("effort", baton->heifEffort)
+            ->set("bitdepth", 8)
            ->set("subsample_mode", baton->heifChromaSubsampling == "4:4:4"
              ? VIPS_FOREIGN_SUBSAMPLE_OFF : VIPS_FOREIGN_SUBSAMPLE_ON)
            ->set("lossless", baton->heifLossless));
--- a/test/bench/package.json
+++ b/test/bench/package.json
@@ -10,11 +10,12 @@
  "devDependencies": {
    "@squoosh/cli": "0.7.2",
    "@squoosh/lib": "0.4.0",
+    "@tensorflow/tfjs-node": "3.20.0",
    "async": "3.2.4",
    "benchmark": "2.1.4",
-    "gm": "1.23.1",
+    "gm": "1.24.0",
    "imagemagick": "0.1.3",
-    "jimp": "0.16.1",
+    "jimp": "0.16.2",
    "mapnik": "4.5.9",
    "semver": "7.3.7"
  },
--- a/test/bench/perf.js
+++ b/test/bench/perf.js
@@ -5,7 +5,6 @@ const fs = require('fs');
 const { exec } = require('child_process');

 const async = require('async');
-const assert = require('assert');
 const Benchmark = require('benchmark');

 // Contenders
@@ -15,6 +14,8 @@ const imagemagick = require('imagemagick');
 const mapnik = require('mapnik');
 const jimp = require('jimp');
 const squoosh = require('@squoosh/lib');
+process.env.TF_CPP_MIN_LOG_LEVEL = 1;
+const tfjs = require('@tensorflow/tfjs-node');

 const fixtures = require('../fixtures');

@@ -212,11 +213,10 @@ async.series({
          .filter('Lanczos')
          .resize(width, height)
          .quality(80)
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -243,16 +243,33 @@ async.series({
          .filter('Lanczos')
          .resize(width, height)
          .quality(80)
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
      }
    });
+    // tfjs
+    jpegSuite.add('tfjs-node-buffer-buffer', {
+      defer: true,
+      fn: function (deferred) {
+        const decoded = tfjs.node.decodeJpeg(inputJpgBuffer);
+        const resized = tfjs.image.resizeBilinear(decoded, [height, width]);
+        tfjs
+          .node
+          .encodeJpeg(resized, 'rgb', 80)
+          .then(function () {
+            deferred.resolve();
+            tfjs.disposeVariables();
+          })
+          .catch(function (err) {
+            throw err;
+          });
+      }
+    });
    // sharp
    jpegSuite.add('sharp-buffer-file', {
      defer: true,
@@ -272,11 +289,10 @@ async.series({
      fn: function (deferred) {
        sharp(inputJpgBuffer)
          .resize(width, height)
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -311,11 +327,10 @@ async.series({
      fn: function (deferred) {
        sharp(fixtures.inputJpg)
          .resize(width, height)
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -326,8 +341,7 @@ async.series({
        sharp(inputJpgBuffer)
          .resize(width, height)
          .toBuffer()
-          .then(function (buffer) {
-            assert.notStrictEqual(null, buffer);
+          .then(function () {
            deferred.resolve();
          })
          .catch(function (err) {
@@ -350,11 +364,10 @@ async.series({
        sharp(inputJpgBuffer)
          .resize(width, height)
          .sharpen()
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -365,11 +378,10 @@ async.series({
        sharp(inputJpgBuffer)
          .resize(width, height)
          .sharpen(3, 1, 3)
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -380,11 +392,10 @@ async.series({
        sharp(inputJpgBuffer)
          .resize(width, height)
          .blur()
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -395,11 +406,10 @@ async.series({
        sharp(inputJpgBuffer)
          .resize(width, height)
          .blur(3)
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -410,11 +420,10 @@ async.series({
        sharp(inputJpgBuffer)
          .resize(width, height)
          .gamma()
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -425,11 +434,10 @@ async.series({
        sharp(inputJpgBuffer)
          .resize(width, height)
          .normalise()
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -440,11 +448,10 @@ async.series({
        sharp(inputJpgBuffer)
          .resize(width, height)
          .greyscale()
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -456,11 +463,10 @@ async.series({
          .resize(width, height)
          .gamma()
          .greyscale()
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -471,11 +477,10 @@ async.series({
        sharp(inputJpgBuffer)
          .resize(width, height)
          .jpeg({ progressive: true })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -486,11 +491,10 @@ async.series({
        sharp(inputJpgBuffer)
          .resize(width, height)
          .jpeg({ chromaSubsampling: '4:4:4' })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -501,11 +505,10 @@ async.series({
        sharp(inputJpgBuffer)
          .rotate(90)
          .resize(width, height)
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -516,12 +519,11 @@ async.series({
        sharp.simd(false);
        sharp(inputJpgBuffer)
          .resize(width, height)
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            sharp.simd(true);
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -531,11 +533,10 @@ async.series({
      fn: function (deferred) {
        sharp(inputJpgBuffer, { sequentialRead: true })
          .resize(width, height)
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -548,11 +549,10 @@ async.series({
            fit: 'cover',
            position: sharp.strategy.entropy
          })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -565,11 +565,10 @@ async.series({
            fit: 'cover',
            position: sharp.strategy.attention
          })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -588,11 +587,10 @@ async.series({
      fn: function (deferred) {
        sharp(inputJpgBuffer)
          .resize(width, height, { kernel: 'cubic' })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -602,11 +600,10 @@ async.series({
      fn: function (deferred) {
        sharp(inputJpgBuffer)
          .resize(width, height, { kernel: 'lanczos2' })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -616,11 +613,10 @@ async.series({
      fn: function (deferred) {
        sharp(inputJpgBuffer)
          .resize(width, height, { kernel: 'lanczos3' })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -774,11 +770,10 @@ async.series({
          .resize(width, height)
          .define('PNG:compression-level=6')
          .define('PNG:compression-filter=0')
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -807,11 +802,10 @@ async.series({
        sharp(inputPngBuffer)
          .resize(width, height)
          .png({ compressionLevel: 6 })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -838,11 +832,10 @@ async.series({
        sharp(fixtures.inputPngAlphaPremultiplicationLarge)
          .resize(width, height)
          .png({ compressionLevel: 6 })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -854,11 +847,10 @@ async.series({
        sharp(inputPngBuffer)
          .resize(width, height)
          .png({ compressionLevel: 6, progressive: true })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -870,11 +862,10 @@ async.series({
        sharp(inputPngBuffer)
          .resize(width, height)
          .png({ adaptiveFiltering: true, compressionLevel: 6 })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -886,11 +877,10 @@ async.series({
        sharp(inputPngBuffer)
          .resize(width, height)
          .png({ compressionLevel: 9 })
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -923,11 +913,10 @@ async.series({
      fn: function (deferred) {
        sharp(inputWebPBuffer)
          .resize(width, height)
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -950,11 +939,10 @@ async.series({
      fn: function (deferred) {
        sharp(fixtures.inputWebP)
          .resize(width, height)
-          .toBuffer(function (err, buffer) {
+          .toBuffer(function (err) {
            if (err) {
              throw err;
            } else {
-              assert.notStrictEqual(null, buffer);
              deferred.resolve();
            }
          });
@@ -966,7 +954,9 @@ async.series({
    }).run();
  }
 }, function (err, results) {
-  assert(!err, err);
+  if (err) {
+    throw err;
+  }
  Object.keys(results).forEach(function (format) {
    if (results[format].toString().substr(0, 5) !== 'sharp') {
      console.log('sharp was slower than ' + results[format] + ' for ' + format);
--- a/test/coverage/report.sh
+++ b/test/coverage/report.sh
@@ -1,6 +0,0 @@
-#!/bin/sh
-
-CPPFLAGS="--coverage" LDFLAGS="--coverage" npm rebuild
-npm test
-geninfo --no-external --base-directory src --output-file coverage/sharp.info build/Release/obj.target/sharp/src
-genhtml --title sharp --demangle-cpp --output-directory coverage/sharp coverage/*.info
--- a/test/fixtures/65536-uint32-limit.png
+++ b/test/fixtures/65536-uint32-limit.png
--- a/test/fixtures/index.js
+++ b/test/fixtures/index.js
@@ -98,6 +98,7 @@ module.exports = {
  inputPngTrimSpecificColour: getPath('Flag_of_the_Netherlands.png'), // https://commons.wikimedia.org/wiki/File:Flag_of_the_Netherlands.svg
  inputPngTrimSpecificColour16bit: getPath('Flag_of_the_Netherlands-16bit.png'), // convert Flag_of_the_Netherlands.png -depth 16 Flag_of_the_Netherlands-16bit.png
  inputPngTrimSpecificColourIncludeAlpha: getPath('Flag_of_the_Netherlands-alpha.png'), // convert Flag_of_the_Netherlands.png -alpha set -background none -channel A -evaluate multiply 0.5 +channel Flag_of_the_Netherlands-alpha.png
+  inputPngUint32Limit: getPath('65536-uint32-limit.png'), // https://alexandre.alapetite.fr/doc-alex/large-image/

  inputWebP: getPath('4.webp'), // http://www.gstatic.com/webp/gallery/4.webp
  inputWebPWithTransparency: getPath('5_webp_a.webp'), // http://www.gstatic.com/webp/gallery3/5_webp_a.webp
--- a/test/unit/avif.js
+++ b/test/unit/avif.js
@@ -103,4 +103,28 @@ describe('AVIF', () => {
      width: 10
    });
  });
+
+  it('should cast to uchar', async () => {
+    const data = await sharp(inputJpg)
+      .resize(32)
+      .sharpen()
+      .avif({ effort: 0 })
+      .toBuffer();
+    const { size, ...metadata } = await sharp(data)
+      .metadata();
+    assert.deepStrictEqual(metadata, {
+      channels: 3,
+      compression: 'av1',
+      depth: 'uchar',
+      format: 'heif',
+      hasAlpha: false,
+      hasProfile: false,
+      height: 26,
+      isProgressive: false,
+      pagePrimary: 0,
+      pages: 1,
+      space: 'srgb',
+      width: 32
+    });
+  });
 });
--- a/test/unit/extract.js
+++ b/test/unit/extract.js
@@ -300,7 +300,7 @@ describe('Partial image extraction', function () {
      const s = sharp();
      s.on('warning', function (msg) { warningMessage = msg; });
      const options = { top: 0, left: 0, width: 1, height: 1 };
-      s.extract(options);
+      s.extract(options).extract(options);
      assert.strictEqual(warningMessage, '');
      s.extract(options);
      assert.strictEqual(warningMessage, 'ignoring previous extract options');
@@ -311,7 +311,7 @@ describe('Partial image extraction', function () {
      const s = sharp().rotate();
      s.on('warning', function (msg) { warningMessage = msg; });
      const options = { top: 0, left: 0, width: 1, height: 1 };
-      s.extract(options);
+      s.extract(options).extract(options);
      assert.strictEqual(warningMessage, '');
      s.extract(options);
      assert.strictEqual(warningMessage, 'ignoring previous extract options');
--- a/test/unit/io.js
+++ b/test/unit/io.js
@@ -745,6 +745,17 @@ describe('Input/output', function () {
        })
    );

+    it('Enabling default limit works and fails for an image with resolution higher than uint32 limit', () =>
+      sharp(fixtures.inputPngUint32Limit, { limitInputPixels: true })
+        .toBuffer()
+        .then(() => {
+          assert.fail('Expected to fail');
+        })
+        .catch(err => {
+          assert.strictEqual(err.message, 'Input image exceeds pixel limit');
+        })
+    );
+
    it('Smaller than input fails', () =>
      sharp(fixtures.inputJpg)
        .metadata()
--- a/test/unit/png.js
+++ b/test/unit/png.js
@@ -128,11 +128,26 @@ describe('PNG', function () {
    assert.strictEqual(alphaMeanAfter, alphaMeanBefore);
  });

-  it('palette decode/encode roundtrip', () =>
-    sharp(fixtures.inputPngPalette)
+  it('palette decode/encode roundtrip', async () => {
+    const data = await sharp(fixtures.inputPngPalette)
      .png({ effort: 1, palette: true })
-      .toBuffer()
-  );
+      .toBuffer();
+
+    const { size, ...metadata } = await sharp(data).metadata();
+    assert.deepStrictEqual(metadata, {
+      format: 'png',
+      width: 68,
+      height: 68,
+      space: 'srgb',
+      channels: 3,
+      density: 72,
+      depth: 'uchar',
+      isProgressive: false,
+      paletteBitDepth: 8,
+      hasProfile: false,
+      hasAlpha: false
+    });
+  });

  it('Valid PNG libimagequant palette value does not throw error', function () {
    assert.doesNotThrow(function () {
--- a/test/unit/rotate.js
+++ b/test/unit/rotate.js
@@ -8,16 +8,43 @@ const fixtures = require('../fixtures');
 describe('Rotation', function () {
  ['Landscape', 'Portrait'].forEach(function (orientation) {
    [1, 2, 3, 4, 5, 6, 7, 8].forEach(function (exifTag) {
-      it('Input image has Orientation EXIF tag value of (' + exifTag + '), auto-rotate', function (done) {
-        sharp(fixtures['inputJpgWith' + orientation + 'Exif' + exifTag])
+      const input = fixtures[`inputJpgWith${orientation}Exif${exifTag}`];
+      const expectedOutput = fixtures.expected(`${orientation}_${exifTag}-out.jpg`);
+      it(`Auto-rotate ${orientation} image with EXIF Orientation ${exifTag}`, function (done) {
+        const [expectedWidth, expectedHeight] = orientation === 'Landscape' ? [600, 450] : [450, 600];
+        sharp(input)
          .rotate()
-          .resize(320)
          .toBuffer(function (err, data, info) {
            if (err) throw err;
-            assert.strictEqual('jpeg', info.format);
-            assert.strictEqual(320, info.width);
-            assert.strictEqual(orientation === 'Landscape' ? 240 : 427, info.height);
-            fixtures.assertSimilar(fixtures.expected(orientation + '_' + exifTag + '-out.jpg'), data, done);
+            assert.strictEqual(info.width, expectedWidth);
+            assert.strictEqual(info.height, expectedHeight);
+            fixtures.assertSimilar(expectedOutput, data, done);
+          });
+      });
+      it(`Auto-rotate then resize ${orientation} image with EXIF Orientation ${exifTag}`, function (done) {
+        const [expectedWidth, expectedHeight] = orientation === 'Landscape' ? [320, 240] : [320, 427];
+        sharp(input)
+          .rotate()
+          .resize({ width: 320 })
+          .toBuffer(function (err, data, info) {
+            if (err) throw err;
+            assert.strictEqual(info.width, expectedWidth);
+            assert.strictEqual(info.height, expectedHeight);
+            fixtures.assertSimilar(expectedOutput, data, done);
+          });
+      });
+      it(`Resize then auto-rotate ${orientation} image with EXIF Orientation ${exifTag}`, function (done) {
+        const [expectedWidth, expectedHeight] = orientation === 'Landscape'
+          ? (exifTag < 5) ? [320, 240] : [320, 240]
+          : [320, 427];
+        sharp(input)
+          .resize({ width: 320 })
+          .rotate()
+          .toBuffer(function (err, data, info) {
+            if (err) throw err;
+            assert.strictEqual(info.width, expectedWidth);
+            assert.strictEqual(info.height, expectedHeight);
+            fixtures.assertSimilar(expectedOutput, data, done);
          });
      });
    });
@@ -406,4 +433,41 @@ describe('Rotation', function () {
        fixtures.assertSimilar(fixtures.expected('rotate-and-flop.jpg'), data, done);
      });
  });
+
+  it('Auto-rotate and shrink-on-load', async () => {
+    const [r, g, b] = await sharp(fixtures.inputJpgWithLandscapeExif3)
+      .rotate()
+      .resize(8)
+      .raw()
+      .toBuffer();
+
+    assert.strictEqual(r, 60);
+    assert.strictEqual(g, 73);
+    assert.strictEqual(b, 52);
+  });
+
+  it('Flip and rotate ordering', async () => {
+    const [r, g, b] = await sharp(fixtures.inputJpgWithPortraitExif5)
+      .flip()
+      .rotate(90)
+      .raw()
+      .toBuffer();
+
+    assert.strictEqual(r, 55);
+    assert.strictEqual(g, 65);
+    assert.strictEqual(b, 31);
+  });
+
+  it('Flip, rotate and resize ordering', async () => {
+    const [r, g, b] = await sharp(fixtures.inputJpgWithPortraitExif5)
+      .flip()
+      .rotate(90)
+      .resize(449)
+      .raw()
+      .toBuffer();
+
+    assert.strictEqual(r, 54);
+    assert.strictEqual(g, 64);
+    assert.strictEqual(b, 30);
+  });
 });
--- a/test/unit/text.js
+++ b/test/unit/text.js
@@ -175,12 +175,13 @@ describe('Text to image', () => {
  });

  it('fontfile input', function () {
-    // Added for code coverage
-    sharp({
-      text: {
-        text: 'text',
-        fontfile: 'UnknownFont.ttf'
-      }
+    assert.doesNotThrow(function () {
+      sharp({
+        text: {
+          text: 'text',
+          fontfile: 'UnknownFont.ttf'
+        }
+      });
    });
  });

--- a/test/unit/trim.js
+++ b/test/unit/trim.js
@@ -124,6 +124,32 @@ describe('Trim borders', function () {
    assert.strictEqual(trimOffsetLeft, -13);
  });

+  it('Ensure greyscale image can be trimmed', async () => {
+    const greyscale = await sharp({
+      create: {
+        width: 16,
+        height: 8,
+        channels: 3,
+        background: 'silver'
+      }
+    })
+      .extend({ left: 12, right: 24, background: 'gray' })
+      .toColourspace('b-w')
+      .png({ compressionLevel: 0 })
+      .toBuffer();
+
+    const { info } = await sharp(greyscale)
+      .trim()
+      .raw()
+      .toBuffer({ resolveWithObject: true });
+
+    const { width, height, trimOffsetTop, trimOffsetLeft } = info;
+    assert.strictEqual(width, 16);
+    assert.strictEqual(height, 8);
+    assert.strictEqual(trimOffsetTop, 0);
+    assert.strictEqual(trimOffsetLeft, -12);
+  });
+
  it('Ensure trim of image with all pixels same is no-op', async () => {
    const { info } = await sharp({
      create: {
Author	SHA1	Message	Date
Lovell Fuller	9fa516e849	Release v0.31.2	2022-11-04 09:44:37 +00:00
Lovell Fuller	12f472126d	CI: Only pin Python version on x64 macOS and Windows See commit `18be09f`	2022-11-03 14:49:12 +00:00
Lovell Fuller	18be09f1d7	CI: Pin Python to 3.10 Python 3.11 removes support for opening files in 'universal newline' mode (e.g. 'rU'), however older versions of node-gyp such as v6 still use it.	2022-11-03 14:40:16 +00:00
Lovell Fuller	b3c3290f90	Upgrade to libvips v8.13.3	2022-11-03 14:09:23 +00:00
Lovell Fuller	123f95c85a	Bump devDeps	2022-11-03 12:50:58 +00:00
Lovell Fuller	5b0fba4c01	Ensure auto-rotate always works without resize #3422	2022-11-02 13:59:34 +00:00
Lovell Fuller	37f7ccfff4	CI: upgrade to checkout v3	2022-10-17 16:05:30 +01:00
Lovell Fuller	51811d06e2	Bump deps	2022-10-17 16:05:03 +01:00
Lovell Fuller	181731f8f4	Tests: increase timeout to 30s Font discovery appears to be slooow on Windows	2022-10-17 15:54:43 +01:00
Gino Emiliozzi	ae79d26ead	Docs: help clarify 'fit' is option name, not value (#3410 )	2022-10-17 15:26:59 +01:00
Lovell Fuller	eacb8337fa	Ensure manual flip, rotate, resize op order #3391	2022-10-01 11:55:29 +01:00
Lovell Fuller	99bf279de8	Release v0.31.1	2022-09-29 14:51:45 +01:00
Lovell Fuller	1b0eb6ab53	Tests: add assertion to existing scenario #3357	2022-09-29 14:21:37 +01:00
Lovell Fuller	891cf67d0b	Upgrade to libvips v8.13.2	2022-09-29 14:19:58 +01:00
Lovell Fuller	eaf8a86bf2	Tests: increase timeout to 20s Ignore unit coverage of fn used at install time	2022-09-27 14:25:34 +01:00
Lovell Fuller	3400976d61	Tests: ignore a branch for coverage check	2022-09-27 14:10:49 +01:00
Lovell Fuller	2d49f0e93e	Tests: require 100% branch coverage to pass Remove old coverage tooling, coveralls	2022-09-27 13:49:42 +01:00
Lovell Fuller	b0c69f1ee9	CI: use clearer job names	2022-09-27 10:02:20 +01:00
Lovell Fuller	27d0c35a01	CI: use clearer job names	2022-09-27 09:23:30 +01:00
Alex	32a22b5420	CI: GitHub Workflows security hardening (#3377 ) Signed-off-by: Alex <aleksandrosansan@gmail.com>	2022-09-26 11:25:49 +01:00
Lovell Fuller	d1004eed02	Ensure greyscale images can be trimmed #3386	2022-09-26 10:15:25 +01:00
Lovell Fuller	70e6bb0162	Ensure close event occurs after end event #3313	2022-09-20 08:52:40 +01:00
Lovell Fuller	32aa3b4b20	Tests: bump/pin benchmark dependencies	2022-09-19 16:21:24 +01:00
Lovell Fuller	df24b30755	Tests: add tfjs to benchmark tests	2022-09-19 16:20:23 +01:00
Lovell Fuller	4de74bea94	Tests: remove assertions from benchmark code	2022-09-19 16:09:31 +01:00
Lovell Fuller	28b87db760	Ensure AVIF output is always 8-bit #3358	2022-09-14 13:33:47 +01:00
Lovell Fuller	fbd4970b57	Ensure auto-rotation works with shrink-on-load #3352 Fixes regression in 0.31.0	2022-09-07 14:17:40 +01:00
Lovell Fuller	f5da147a58	Docs: changelog and credit for #3349	2022-09-07 13:31:26 +01:00
Marcos Casagrande	eee0dd36d9	Ensure limitInputPixels uses uint64 (#3349 )	2022-09-06 09:05:51 +01:00