fix: lanczos throwing when destination dimensions are not round (#50)

* fix: lanczos throwing when destination dimensions are not round

Images over `MAX_WIDTH` that don't scale to a rounded width or height
were causing lanczos to blow up with `RangeError: byte length of Uint32Array should be a multiple of 4`

This change ensures that the target dimensions are rounded and the resulting target `Uint32Array`
has a length that is a multiple of 4

* chore: add changeset

---------

Co-authored-by: Jack Works <[email protected]>

Author: carlmw

.changeset/lemon-penguins-bathe.md

@@ -0,0 +1,5 @@
+---
+'@masknet/stego-js': patch
+---
+
+fix: lanczos throwing when destination dimensions are not round

src/utils/image.ts

@@ -15,7 +15,7 @@ export function preprocessImage(
   if (imageData.width <= MAX_WIDTH && imageData.height <= MAX_WIDTH) return imageData
   const scale = MAX_WIDTH / Math.max(imageData.width, imageData.height)
   const [w, h] = [imageData.width * scale, imageData.height * scale]
-  const scaled = getScaled(w, h)
+  const scaled = getScaled(Math.round(w), Math.round(h))
   if (scaled) {
     lanczos(imageData, scaled)
     return scaled

tests/image.ts

@@ -0,0 +1,23 @@
+import { expect, test, vi } from 'vitest'
+import { preprocessImage } from '../src/utils/image.js'
+import { MAX_WIDTH } from '../src/constant.js'
+
+test('preprocessImage rounds the dimensions of the scaled image', async () => {
+  const getScaled = vi.fn().mockImplementation((width, height) => ({
+    height,
+    width,
+    colorSpace: 'srgb',
+    data: new Uint8ClampedArray(width * height * 4),
+  }))
+  preprocessImage(
+    {
+      width: 1980,
+      height: 1024,
+      colorSpace: 'srgb',
+      data: new Uint8ClampedArray(1980 * 1024 * 4),
+    },
+    getScaled,
+  )
+
+  expect(getScaled).toHaveBeenCalledWith(MAX_WIDTH, 1014)
+})