chore: update deps, fix linting issues

Author: Henrik Fuchs

biome.json

@@ -5,6 +5,9 @@
 	],
 	"linter": {
 		"rules": {
+			"nursery": {
+				"noShadow": "off"
+			},
 			"performance": {
 				"noBarrelFile": "off"
 			}

package-lock.json

@@ -15,7 +15,7 @@
     "semantic-release": "semantic-release"
   },
   "dependencies": {
-    "immer": "11.1.3"
+    "immer": "11.1.4"
   },
   "peerDependencies": {
     "react": ">=16.8.0 <20"
@@ -27,17 +27,17 @@
     "@babel/preset-env": "7.29.0",
     "@babel/preset-react": "7.28.5",
     "@babel/preset-typescript": "7.28.5",
-    "@biomejs/biome": "2.3.14",
+    "@biomejs/biome": "2.4.7",
     "@testing-library/dom": "10.4.1",
     "@testing-library/jest-dom": "6.9.1",
     "@testing-library/react": "16.3.2",
     "@types/jest": "30.0.0",
-    "@types/react": "19.2.13",
+    "@types/react": "19.2.14",
     "@types/react-dom": "19.2.3",
-    "eslint": "9.39.2",
-    "eslint-config-heck": "8.21.0",
-    "jest": "30.2.0",
-    "jest-environment-jsdom": "30.2.0",
+    "eslint": "9.39.4",
+    "eslint-config-heck": "9.2.0",
+    "jest": "30.3.0",
+    "jest-environment-jsdom": "30.3.0",
     "semantic-release": "25.0.3",
     "ts-jest": "29.4.6",
     "tslib": "2.8.1",

package.json

@@ -84,7 +84,7 @@ function useSubscription<TProps, TModel, TMessage extends Message>(
 			const subscriptionResult = subscription(model, props);
 
 			if (subscriptionIsFunctionArray(subscriptionResult)) {
-				const destructors = subscriptionResult.map((sub) => sub(dispatch)).filter((destructor) => destructor !== undefined);
+				const destructors = subscriptionResult.map((sub) => sub(dispatch)).filter((current) => current !== undefined);
 
 				return function combinedDestructor() {
 					for (const destructor of destructors) {
@@ -179,11 +179,7 @@ function runInit<TModel, TProps, TMessage extends Message>(
 	return initModel;
 }
 
-function useDispose<TModel>(
-	dispose: DisposeFunction<TModel> | undefined,
-	model: TModel,
-	reInitOn: unknown[] | undefined,
-): void {
+function useDispose<TModel>(dispose: DisposeFunction<TModel> | undefined, model: TModel, reInitOn: unknown[] | undefined): void {
 	const modelRef = useRef(model);
 
 	modelRef.current = model;
@@ -198,4 +194,15 @@ function useDispose<TModel>(
 	}, reInitOn ?? []);
 }
 
-export { execCmd, logMessage, modelHasChanged, useRedux, useIsMounted, useSubscription, useReInit, useDispose, getDispatch, runInit };
+export {
+	execCmd,
+	getDispatch,
+	logMessage,
+	modelHasChanged,
+	runInit,
+	useDispose,
+	useIsMounted,
+	useRedux,
+	useReInit,
+	useSubscription,
+};

src/Common.ts

@@ -32,7 +32,12 @@ abstract class ElmComponent<TModel, TMessage extends Message, TProps> extends Re
 	 * @param name The name of the component.
 	 * @memberof ElmComponent
 	 */
-	public constructor(props: TProps, init: InitFunction<TProps, TModel, TMessage>, name: string, private readonly disposeHandler?: DisposeFunction<TModel>) {
+	public constructor(
+		props: TProps,
+		init: InitFunction<TProps, TModel, TMessage>,
+		name: string,
+		private readonly disposeHandler?: DisposeFunction<TModel>,
+	) {
 		super(props);
 
 		const fakeOptions = getFakeOptionsOnce<TModel, TMessage>();

src/ElmComponent.ts

@@ -11,6 +11,7 @@ async function asyncResolve(): Promise<void> {
 	// Does nothing
 }
 
+// biome-ignore lint/suspicious/useAwait: Only for testing purpose.
 async function asyncReject(): Promise<void> {
 	throw new Error("error");
 }
@@ -265,7 +266,7 @@ describe("cmd", () => {
 			expect(message.name).toBe("success");
 		});
 
-		it("ignores the error for a sync task", async () => {
+		it("ignores the error for a sync task", () => {
 			// arrange
 			const result = cmd.ofSuccess(syncError, successMsg);
 
@@ -276,7 +277,7 @@ describe("cmd", () => {
 			expect(succeeds).not.toThrow();
 		});
 
-		it("ignores the error for an async task", async () => {
+		it("ignores the error for an async task", () => {
 			// arrange
 			const result = cmd.ofSuccess(asyncReject, successMsg);
 
@@ -420,7 +421,7 @@ describe("cmd", () => {
 			expect(task).toHaveBeenCalledTimes(1);
 		});
 
-		it("ignores the error for a sync task", async () => {
+		it("ignores the error for a sync task", () => {
 			// arrange
 			const result = cmd.ofNone(syncError);
 
@@ -431,7 +432,7 @@ describe("cmd", () => {
 			expect(succeeds).not.toThrow();
 		});
 
-		it("ignores the error for an async task", async () => {
+		it("ignores the error for an async task", () => {
 			// arrange
 			const result = cmd.ofNone(asyncReject);
 

src/cmd.spec.ts

@@ -34,7 +34,12 @@ abstract class ElmComponent<TModel, TMessage extends Message, TProps> extends Re
 	 * @param name The name of the component.
 	 * @memberof ElmComponent
 	 */
-	public constructor(props: TProps, init: InitFunction<TProps, TModel, TMessage>, name: string, private readonly disposeHandler?: DisposeFunction<Immutable<TModel>>) {
+	public constructor(
+		props: TProps,
+		init: InitFunction<TProps, TModel, TMessage>,
+		name: string,
+		private readonly disposeHandler?: DisposeFunction<Immutable<TModel>>,
+	) {
 		super(props);
 
 		const fakeOptions = getFakeOptionsOnce<TModel, TMessage>();

src/immutable/ElmComponent.ts

@@ -95,7 +95,7 @@ describe("Playground", () => {
 	});
 
 	describe("doubleDefer", () => {
-		it("computes the values", async () => {
+		it("computes the values", () => {
 			const args = getUpdateArgs(Msg.doubleDefer());
 
 			const [model, ...commands] = updateFn(...args);

src/immutable/testing/playground.spec.tsx

@@ -1,6 +1,16 @@
 import { castImmutable, type Draft, enablePatches, freeze, type Immutable, produce } from "immer";
 import { useCallback, useRef, useState } from "react";
-import { execCmd, getDispatch, logMessage, runInit, useDispose, useIsMounted, useRedux, useReInit, useSubscription } from "../Common";
+import {
+	execCmd,
+	getDispatch,
+	logMessage,
+	runInit,
+	useDispose,
+	useIsMounted,
+	useRedux,
+	useReInit,
+	useSubscription,
+} from "../Common";
 import { getFakeOptionsOnce } from "../fakeOptions";
 import { Services } from "../Init";
 import type { Cmd, Dispatch, DisposeFunction, InitFunction, Message, Nullable } from "../Types";

src/immutable/useElmish.ts

@@ -1,4 +1,3 @@
-/** biome-ignore-all lint/nursery/noUselessUndefined: Tests only */
 import { cmd } from "../cmd";
 import { execCmd } from ".";
 
@@ -90,6 +89,7 @@ describe("execCmd", () => {
 
 	it("executes all ofPromise commands, fail", async () => {
 		// arrange
+		// biome-ignore lint/suspicious/useAwait: Only for testing purpose.
 		const asyncFunc = async (): Promise<void> => {
 			throw new Error("error");
 		};
@@ -127,6 +127,7 @@ describe("execCmd", () => {
 
 	it("resolves for async attempt, fail", async () => {
 		// arrange
+		// biome-ignore lint/suspicious/useAwait: Only for testing purpose.
 		const asyncFunc = async (): Promise<void> => {
 			throw new Error("error");
 		};
@@ -157,6 +158,7 @@ describe("execCmd", () => {
 
 	it("does not throw when async perform fails", async () => {
 		// arrange
+		// biome-ignore lint/suspicious/useAwait: Only for testing purpose.
 		const asyncFunc = async (): Promise<void> => {
 			throw new Error("fail");
 		};