From b9208877bab3ec082250f3557bf01b6e193d1b7d Mon Sep 17 00:00:00 2001
From: Ryan Christian <rchristian@ryanchristian.dev>
Date: Fri, 19 May 2023 17:00:48 -0500
Subject: [PATCH 01/21] fix: Add types for <dialog>'s close & cancel events

---
 src/jsx.d.ts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/jsx.d.ts b/src/jsx.d.ts
index 46ab9b6ac4..5de268da3e 100644
--- a/src/jsx.d.ts
+++ b/src/jsx.d.ts
@@ -1144,6 +1144,10 @@ export namespace JSXInternal {
 		// Details Events
 		onToggle?: GenericEventHandler<Target> | undefined;
 
+		// Dialog Events
+		onClose?: GenericEventHandler<Target> | undefined;
+		onCancel?: GenericEventHandler<Target> | undefined;
+
 		// Focus Events
 		onFocus?: FocusEventHandler<Target> | undefined;
 		onFocusCapture?: FocusEventHandler<Target> | undefined;

From 0b881e9972225ff3be964d1665dc26a53a24bdfc Mon Sep 17 00:00:00 2001
From: 1o1w1 <610744313@qq.com>
Date: Sat, 20 May 2023 15:08:58 +0800
Subject: [PATCH 02/21] fix: keep hooks index with  useEffect (#4016)

* fix: keep hooks index with  useEffect

* opt code

---------

Co-authored-by: tyrael <1o1w1mail@gmail.com>
---
 debug/src/debug.js                   |  8 +++++
 hooks/src/index.js                   |  1 +
 hooks/test/browser/useEffect.test.js | 47 +++++++++++++++++++++++++++-
 3 files changed, 55 insertions(+), 1 deletion(-)

diff --git a/debug/src/debug.js b/debug/src/debug.js
index 60baab0031..4e1fc03940 100644
--- a/debug/src/debug.js
+++ b/debug/src/debug.js
@@ -32,6 +32,7 @@ export function initDebug() {
 	let oldBeforeDiff = options._diff;
 	let oldDiffed = options.diffed;
 	let oldVnode = options.vnode;
+	let oldRender = options._render;
 	let oldCatchError = options._catchError;
 	let oldRoot = options._root;
 	let oldHook = options._hook;
@@ -252,6 +253,13 @@ export function initDebug() {
 		if (oldBeforeDiff) oldBeforeDiff(vnode);
 	};
 
+	options._render = vnode => {
+		if (oldRender) {
+			oldRender(vnode);
+		}
+		hooksAllowed = true;
+	};
+
 	options._hook = (comp, index, type) => {
 		if (!comp || !hooksAllowed) {
 			throw new Error('Hook can only be invoked from render methods.');
diff --git a/hooks/src/index.js b/hooks/src/index.js
index 56f1aa11d3..3b611b4933 100644
--- a/hooks/src/index.js
+++ b/hooks/src/index.js
@@ -53,6 +53,7 @@ options._render = vnode => {
 			hooks._pendingEffects.forEach(invokeCleanup);
 			hooks._pendingEffects.forEach(invokeEffect);
 			hooks._pendingEffects = [];
+			currentIndex = 0;
 		}
 	}
 	previousComponent = currentComponent;
diff --git a/hooks/test/browser/useEffect.test.js b/hooks/test/browser/useEffect.test.js
index 2075e5f2a1..ce4bb73ab3 100644
--- a/hooks/test/browser/useEffect.test.js
+++ b/hooks/test/browser/useEffect.test.js
@@ -1,4 +1,4 @@
-import { act } from 'preact/test-utils';
+import { act, teardown as teardownAct } from 'preact/test-utils';
 import { createElement, render, Fragment, Component } from 'preact';
 import { useEffect, useState, useRef } from 'preact/hooks';
 import { setupScratch, teardown } from '../../../test/_util/helpers';
@@ -369,6 +369,51 @@ describe('useEffect', () => {
 		);
 	});
 
+	it('hooks should be called in right order', async () => {
+		teardownAct();
+
+		let increment;
+
+		const Counter = () => {
+			const [count, setCount] = useState(0);
+			useState('binggo!!');
+			const renderRoot = useRef();
+			useEffect(() => {
+				const div = renderRoot.current;
+				render(<Dummy />, div);
+			}, [count]);
+
+			increment = () => {
+				setCount(x => x + 1);
+				return Promise.resolve().then(() => setCount(x => x + 1));
+			};
+
+			return (
+				<div>
+					<div>Count: {count}</div>
+					<div ref={renderRoot} />
+				</div>
+			);
+		};
+
+		const Dummy = () => {
+			useState();
+			return <div>dummy</div>;
+		};
+
+		render(<Counter />, scratch);
+
+		expect(scratch.innerHTML).to.equal(
+			'<div><div>Count: 0</div><div></div></div>'
+		);
+		/** Using the act function will affect the timing of the useEffect */
+		await increment();
+
+		expect(scratch.innerHTML).to.equal(
+			'<div><div>Count: 2</div><div><div>dummy</div></div></div>'
+		);
+	});
+
 	it('handles errors correctly', () => {
 		class ErrorBoundary extends Component {
 			constructor(props) {

From d84ddf3e285d2b7931cee706efb47f851d1dc362 Mon Sep 17 00:00:00 2001
From: Alexander Zaytsev <zaytsev126@gmail.com>
Date: Sat, 20 May 2023 16:46:55 +0300
Subject: [PATCH 03/21] Add missing types of TransitionEvent (#4019)

---
 src/jsx.d.ts | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/jsx.d.ts b/src/jsx.d.ts
index 5de268da3e..3b0ca202b2 100644
--- a/src/jsx.d.ts
+++ b/src/jsx.d.ts
@@ -1321,8 +1321,14 @@ export namespace JSXInternal {
 		onAnimationIterationCapture?: AnimationEventHandler<Target> | undefined;
 
 		// Transition Events
+		onTransitionCancel?: TransitionEventHandler<Target>;
+		onTransitionCancelCapture?: TransitionEventHandler<Target>;
 		onTransitionEnd?: TransitionEventHandler<Target>;
 		onTransitionEndCapture?: TransitionEventHandler<Target>;
+		onTransitionRun?: TransitionEventHandler<Target>;
+		onTransitionRunCapture?: TransitionEventHandler<Target>;
+		onTransitionStart?: TransitionEventHandler<Target>;
+		onTransitionStartCapture?: TransitionEventHandler<Target>;
 	}
 
 	// All the WAI-ARIA 1.1 attributes from https://www.w3.org/TR/wai-aria-1.1/

From 87024e8480c6a60bf114d6c69791a87cec0c553a Mon Sep 17 00:00:00 2001
From: Alexander Zaytsev <alexanderz@wix.com>
Date: Sat, 20 May 2023 17:18:50 +0300
Subject: [PATCH 04/21] Add types of `PictureInPictureEvent` (#4020)

* Add types of `PictureInPictureEvent`

* Add `onresize` event types

* Lint

* Update typescript to v4.9.5
---
 package-lock.json | 129 +++-------------------------------------------
 package.json      |   2 +-
 src/jsx.d.ts      |  15 ++++++
 3 files changed, 23 insertions(+), 123 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index 82fea89825..c01f06e78f 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -59,7 +59,7 @@
         "sade": "^1.7.4",
         "sinon": "^9.2.3",
         "sinon-chai": "^3.5.0",
-        "typescript": "4.4.2",
+        "typescript": "^4.9.5",
         "undici": "^4.12.0"
       },
       "funding": {
@@ -2563,21 +2563,6 @@
         "node": ">=10"
       }
     },
-    "node_modules/@wdio/config/node_modules/typescript": {
-      "version": "4.9.5",
-      "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
-      "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
-      "dev": true,
-      "optional": true,
-      "peer": true,
-      "bin": {
-        "tsc": "bin/tsc",
-        "tsserver": "bin/tsserver"
-      },
-      "engines": {
-        "node": ">=4.2.0"
-      }
-    },
     "node_modules/@wdio/logger": {
       "version": "7.26.0",
       "resolved": "https://registry.npmjs.org/@wdio/logger/-/logger-7.26.0.tgz",
@@ -2746,21 +2731,6 @@
         }
       }
     },
-    "node_modules/@wdio/utils/node_modules/typescript": {
-      "version": "4.9.5",
-      "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
-      "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
-      "dev": true,
-      "optional": true,
-      "peer": true,
-      "bin": {
-        "tsc": "bin/tsc",
-        "tsserver": "bin/tsserver"
-      },
-      "engines": {
-        "node": ">=4.2.0"
-      }
-    },
     "node_modules/accepts": {
       "version": "1.3.8",
       "resolved": "https://registry.npmjs.org/accepts/-/accepts-1.3.8.tgz",
@@ -5367,21 +5337,6 @@
         }
       }
     },
-    "node_modules/devtools/node_modules/typescript": {
-      "version": "4.9.5",
-      "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
-      "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
-      "dev": true,
-      "optional": true,
-      "peer": true,
-      "bin": {
-        "tsc": "bin/tsc",
-        "tsserver": "bin/tsserver"
-      },
-      "engines": {
-        "node": ">=4.2.0"
-      }
-    },
     "node_modules/devtools/node_modules/ua-parser-js": {
       "version": "1.0.35",
       "resolved": "https://registry.npmjs.org/ua-parser-js/-/ua-parser-js-1.0.35.tgz",
@@ -15279,9 +15234,9 @@
       }
     },
     "node_modules/typescript": {
-      "version": "4.4.2",
-      "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.4.2.tgz",
-      "integrity": "sha512-gzP+t5W4hdy4c+68bfcv0t400HVJMMd2+H9B7gae1nQlBzCqvrXX+6GL/b3GAgyTH966pzrZ70/fRjwAtZksSQ==",
+      "version": "4.9.5",
+      "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
+      "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
       "dev": true,
       "bin": {
         "tsc": "bin/tsc",
@@ -15608,21 +15563,6 @@
         }
       }
     },
-    "node_modules/webdriver/node_modules/typescript": {
-      "version": "4.9.5",
-      "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
-      "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
-      "dev": true,
-      "optional": true,
-      "peer": true,
-      "bin": {
-        "tsc": "bin/tsc",
-        "tsserver": "bin/tsserver"
-      },
-      "engines": {
-        "node": ">=4.2.0"
-      }
-    },
     "node_modules/webdriverio": {
       "version": "7.30.2",
       "resolved": "https://registry.npmjs.org/webdriverio/-/webdriverio-7.30.2.tgz",
@@ -15739,21 +15679,6 @@
         "url": "https://github.com/sponsors/sindresorhus"
       }
     },
-    "node_modules/webdriverio/node_modules/typescript": {
-      "version": "4.9.5",
-      "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
-      "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
-      "dev": true,
-      "optional": true,
-      "peer": true,
-      "bin": {
-        "tsc": "bin/tsc",
-        "tsserver": "bin/tsserver"
-      },
-      "engines": {
-        "node": ">=4.2.0"
-      }
-    },
     "node_modules/webidl-conversions": {
       "version": "3.0.1",
       "resolved": "https://registry.npmjs.org/webidl-conversions/-/webidl-conversions-3.0.1.tgz",
@@ -18010,14 +17935,6 @@
           "requires": {
             "brace-expansion": "^2.0.1"
           }
-        },
-        "typescript": {
-          "version": "4.9.5",
-          "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
-          "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
-          "dev": true,
-          "optional": true,
-          "peer": true
         }
       }
     },
@@ -18140,14 +18057,6 @@
             "@types/node": "^18.0.0",
             "got": "^11.8.1"
           }
-        },
-        "typescript": {
-          "version": "4.9.5",
-          "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
-          "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
-          "dev": true,
-          "optional": true,
-          "peer": true
         }
       }
     },
@@ -20183,14 +20092,6 @@
             "got": "^11.8.1"
           }
         },
-        "typescript": {
-          "version": "4.9.5",
-          "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
-          "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
-          "dev": true,
-          "optional": true,
-          "peer": true
-        },
         "ua-parser-js": {
           "version": "1.0.35",
           "resolved": "https://registry.npmjs.org/ua-parser-js/-/ua-parser-js-1.0.35.tgz",
@@ -27597,9 +27498,9 @@
       }
     },
     "typescript": {
-      "version": "4.4.2",
-      "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.4.2.tgz",
-      "integrity": "sha512-gzP+t5W4hdy4c+68bfcv0t400HVJMMd2+H9B7gae1nQlBzCqvrXX+6GL/b3GAgyTH966pzrZ70/fRjwAtZksSQ==",
+      "version": "4.9.5",
+      "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
+      "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
       "dev": true
     },
     "ua-parser-js": {
@@ -27833,14 +27734,6 @@
             "@types/node": "^18.0.0",
             "got": "^11.8.1"
           }
-        },
-        "typescript": {
-          "version": "4.9.5",
-          "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
-          "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
-          "dev": true,
-          "optional": true,
-          "peer": true
         }
       }
     },
@@ -27927,14 +27820,6 @@
           "resolved": "https://registry.npmjs.org/type-fest/-/type-fest-0.20.2.tgz",
           "integrity": "sha512-Ne+eE4r0/iWnpAxD852z3A+N0Bt5RN//NjJwRd2VFHEmrywxf5vsZlh4R6lixl6B+wz/8d+maTSAkN1FIkI3LQ==",
           "dev": true
-        },
-        "typescript": {
-          "version": "4.9.5",
-          "resolved": "https://registry.npmjs.org/typescript/-/typescript-4.9.5.tgz",
-          "integrity": "sha512-1FXk9E2Hm+QzZQ7z+McJiHL4NW1F2EzMu9Nq9i3zAaGqibafqYwCVU6WyWAuyQRRzOlxou8xZSyXLEN8oKj24g==",
-          "dev": true,
-          "optional": true,
-          "peer": true
         }
       }
     },
diff --git a/package.json b/package.json
index ad697cb303..96bc3ea55c 100644
--- a/package.json
+++ b/package.json
@@ -305,7 +305,7 @@
     "sade": "^1.7.4",
     "sinon": "^9.2.3",
     "sinon-chai": "^3.5.0",
-    "typescript": "4.4.2",
+    "typescript": "^4.9.5",
     "undici": "^4.12.0"
   },
   "overrides": {
diff --git a/src/jsx.d.ts b/src/jsx.d.ts
index 3b0ca202b2..a2b714ea02 100644
--- a/src/jsx.d.ts
+++ b/src/jsx.d.ts
@@ -1073,6 +1073,10 @@ export namespace JSXInternal {
 		Target,
 		WheelEvent
 	>;
+	export type TargetedPictureInPictureEvent<Target extends EventTarget> = TargetedEvent<
+		Target,
+		PictureInPictureEvent
+	>;
 
 	export interface EventHandler<E extends TargetedEvent> {
 		(this: void, event: E): void;
@@ -1116,6 +1120,9 @@ export namespace JSXInternal {
 	export type WheelEventHandler<Target extends EventTarget> = EventHandler<
 		TargetedWheelEvent<Target>
 	>;
+	export type PictureInPictureEventHandler<Target extends EventTarget> = EventHandler<
+		TargetedPictureInPictureEvent<Target>
+	>;
 
 	export interface DOMAttributes<Target extends EventTarget>
 		extends PreactDOMAttributes {
@@ -1329,6 +1336,14 @@ export namespace JSXInternal {
 		onTransitionRunCapture?: TransitionEventHandler<Target>;
 		onTransitionStart?: TransitionEventHandler<Target>;
 		onTransitionStartCapture?: TransitionEventHandler<Target>;
+
+		// PictureInPicture Events
+		onEnterPictureInPicture?: PictureInPictureEventHandler<Target>;
+		onEnterPictureInPictureCapture?: PictureInPictureEventHandler<Target>;
+		onLeavePictureInPicture?: PictureInPictureEventHandler<Target>;
+		onLeavePictureInPictureCapture?: PictureInPictureEventHandler<Target>;
+		onResize?: PictureInPictureEventHandler<Target>;
+		onResizeCapture?: PictureInPictureEventHandler<Target>;
 	}
 
 	// All the WAI-ARIA 1.1 attributes from https://www.w3.org/TR/wai-aria-1.1/

From ea49f7a0f9d1ff2c98c0bdd66aa0cbc583055246 Mon Sep 17 00:00:00 2001
From: Jovi De Croock <decroockjovi@gmail.com>
Date: Sun, 21 May 2023 20:37:35 +0200
Subject: [PATCH 05/21] 10.15.0 (#4011)

---
 devtools/src/devtools.js | 2 +-
 package-lock.json        | 4 ++--
 package.json             | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/devtools/src/devtools.js b/devtools/src/devtools.js
index 5f61152334..4a07c23b31 100644
--- a/devtools/src/devtools.js
+++ b/devtools/src/devtools.js
@@ -2,7 +2,7 @@ import { options, Fragment, Component } from 'preact';
 
 export function initDevTools() {
 	if (typeof window != 'undefined' && window.__PREACT_DEVTOOLS__) {
-		window.__PREACT_DEVTOOLS__.attachPreact('10.14.1', options, {
+		window.__PREACT_DEVTOOLS__.attachPreact('10.15.0', options, {
 			Fragment,
 			Component
 		});
diff --git a/package-lock.json b/package-lock.json
index c01f06e78f..2c46c749d4 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "preact",
-  "version": "10.14.1",
+  "version": "10.15.0",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "preact",
-      "version": "10.14.1",
+      "version": "10.15.0",
       "license": "MIT",
       "devDependencies": {
         "@actions/github": "^5.0.0",
diff --git a/package.json b/package.json
index 96bc3ea55c..edebe7cb53 100644
--- a/package.json
+++ b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "preact",
   "amdName": "preact",
-  "version": "10.14.1",
+  "version": "10.15.0",
   "private": false,
   "description": "Fast 3kb React-compatible Virtual DOM library.",
   "main": "dist/preact.js",

From 0e9b319470297474930a117532b40ab8f13c75c8 Mon Sep 17 00:00:00 2001
From: Ryan Christian <rchristian@ryanchristian.dev>
Date: Tue, 23 May 2023 00:42:06 -0500
Subject: [PATCH 06/21] fix: TS FunctionComponent w/ exactOptionalPropertyTypes

---
 src/index.d.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/index.d.ts b/src/index.d.ts
index eb192aa15c..c57fcb6f6f 100644
--- a/src/index.d.ts
+++ b/src/index.d.ts
@@ -89,7 +89,7 @@ export type ComponentProps<
 export interface FunctionComponent<P = {}> {
 	(props: RenderableProps<P>, context?: any): VNode<any> | null;
 	displayName?: string;
-	defaultProps?: Partial<P>;
+	defaultProps?: Partial<P> | undefined;
 }
 export interface FunctionalComponent<P = {}> extends FunctionComponent<P> {}
 

From 7cf471abac838173932ff720bca7937a99403f71 Mon Sep 17 00:00:00 2001
From: shoonia <alexanderz@wix.com>
Date: Tue, 23 May 2023 20:11:57 +0300
Subject: [PATCH 07/21] Add W3C aria attributes

---
 src/jsx.d.ts | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/src/jsx.d.ts b/src/jsx.d.ts
index a2b714ea02..4528221da5 100644
--- a/src/jsx.d.ts
+++ b/src/jsx.d.ts
@@ -1359,6 +1359,16 @@ export namespace JSXInternal {
 		'aria-autocomplete'?: Signalish<
 			'none' | 'inline' | 'list' | 'both' | undefined
 		>;
+		/**
+		 * Defines a string value that labels the current element, which is intended to be converted into Braille.
+		 * @see aria-label.
+		 */
+		'aria-braillelabel'?: Signalish<string | undefined>;
+		/**
+		 * Defines a human-readable, author-localized abbreviated description for the role of an element, which is intended to be converted into Braille.
+		 * @see aria-roledescription.
+		 */
+		'aria-brailleroledescription'?: Signalish<string | undefined>;
 		/** Indicates an element is being modified and that assistive technologies MAY want to wait until the modifications are complete before exposing them to the user. */
 		'aria-busy'?: Signalish<Booleanish | undefined>;
 		/**

From f8118eb1be3baf2028dbd3fbfe66583b9974ca95 Mon Sep 17 00:00:00 2001
From: Scott Fischer <Scott-Fischer@users.noreply.github.com>
Date: Wed, 24 May 2023 19:04:08 +0000
Subject: [PATCH 08/21] adds missing react types to compat

---
 compat/src/index.d.ts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/compat/src/index.d.ts b/compat/src/index.d.ts
index 00f47d0def..54990f6a94 100644
--- a/compat/src/index.d.ts
+++ b/compat/src/index.d.ts
@@ -30,6 +30,7 @@ declare namespace React {
 	export import useReducer = _hooks.useReducer;
 	export import useRef = _hooks.useRef;
 	export import useState = _hooks.useState;
+	export type Dispatch<A> = (value: A) => void;
 	// React 18 hooks
 	export import useInsertionEffect = _hooks.useLayoutEffect;
 	export function useTransition(): [false, typeof startTransition];
@@ -40,6 +41,7 @@ declare namespace React {
 	): T;
 
 	// Preact Defaults
+	export import Context = preact.Context;
 	export import ContextType = preact.ContextType;
 	export import RefObject = preact.RefObject;
 	export import Component = preact.Component;

From 5577b4997aa443d3c5302110ba897b8883842149 Mon Sep 17 00:00:00 2001
From: Scott Fischer <Scott-Fischer@users.noreply.github.com>
Date: Wed, 24 May 2023 17:31:13 -0400
Subject: [PATCH 09/21] Update compat/src/index.d.ts

Co-authored-by: Ryan Christian <33403762+rschristian@users.noreply.github.com>
---
 compat/src/index.d.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/compat/src/index.d.ts b/compat/src/index.d.ts
index 54990f6a94..b57674688b 100644
--- a/compat/src/index.d.ts
+++ b/compat/src/index.d.ts
@@ -30,7 +30,7 @@ declare namespace React {
 	export import useReducer = _hooks.useReducer;
 	export import useRef = _hooks.useRef;
 	export import useState = _hooks.useState;
-	export type Dispatch<A> = (value: A) => void;
+	export type Dispatch<A> = (action: A) => void;
 	// React 18 hooks
 	export import useInsertionEffect = _hooks.useLayoutEffect;
 	export function useTransition(): [false, typeof startTransition];

From bdb27ebf9241763fb0b29b8fe9abc32b619b8002 Mon Sep 17 00:00:00 2001
From: Scott Fischer <Scott-Fischer@users.noreply.github.com>
Date: Fri, 26 May 2023 16:28:05 +0000
Subject: [PATCH 10/21] moves dispatch type to hooks

---
 compat/src/index.d.ts | 2 +-
 hooks/src/index.d.ts  | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/compat/src/index.d.ts b/compat/src/index.d.ts
index b57674688b..28125420bc 100644
--- a/compat/src/index.d.ts
+++ b/compat/src/index.d.ts
@@ -17,6 +17,7 @@ declare namespace React {
 	export import Inputs = _hooks.Inputs;
 	export import PropRef = _hooks.PropRef;
 	export import Reducer = _hooks.Reducer;
+	export import Dispatch = _hooks.Dispatch;
 	export import Ref = _hooks.Ref;
 	export import StateUpdater = _hooks.StateUpdater;
 	export import useCallback = _hooks.useCallback;
@@ -30,7 +31,6 @@ declare namespace React {
 	export import useReducer = _hooks.useReducer;
 	export import useRef = _hooks.useRef;
 	export import useState = _hooks.useState;
-	export type Dispatch<A> = (action: A) => void;
 	// React 18 hooks
 	export import useInsertionEffect = _hooks.useLayoutEffect;
 	export function useTransition(): [false, typeof startTransition];
diff --git a/hooks/src/index.d.ts b/hooks/src/index.d.ts
index dfec612eaa..561f034943 100644
--- a/hooks/src/index.d.ts
+++ b/hooks/src/index.d.ts
@@ -15,6 +15,7 @@ export function useState<S = undefined>(): [
 ];
 
 export type Reducer<S, A> = (prevState: S, action: A) => S;
+export type Dispatch<A> = (action: A) => void;
 /**
  * An alternative to `useState`.
  *
@@ -27,7 +28,7 @@ export type Reducer<S, A> = (prevState: S, action: A) => S;
 export function useReducer<S, A>(
 	reducer: Reducer<S, A>,
 	initialState: S
-): [S, (action: A) => void];
+): [S, Dispatch<A>];
 
 /**
  * An alternative to `useState`.
@@ -43,7 +44,7 @@ export function useReducer<S, A, I>(
 	reducer: Reducer<S, A>,
 	initialArg: I,
 	init: (arg: I) => S
-): [S, (action: A) => void];
+): [S, Dispatch<A>];
 
 /** @deprecated Use the `Ref` type instead. */
 type PropRef<T> = MutableRef<T>;

From aef6087f592c8a6542524998e8b08cb608566d7a Mon Sep 17 00:00:00 2001
From: Jovi De Croock <decroockjovi@gmail.com>
Date: Sat, 27 May 2023 11:24:38 +0200
Subject: [PATCH 11/21] 10.15.1 (#4030)

---
 devtools/src/devtools.js | 2 +-
 package-lock.json        | 4 ++--
 package.json             | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/devtools/src/devtools.js b/devtools/src/devtools.js
index 4a07c23b31..c4470482b6 100644
--- a/devtools/src/devtools.js
+++ b/devtools/src/devtools.js
@@ -2,7 +2,7 @@ import { options, Fragment, Component } from 'preact';
 
 export function initDevTools() {
 	if (typeof window != 'undefined' && window.__PREACT_DEVTOOLS__) {
-		window.__PREACT_DEVTOOLS__.attachPreact('10.15.0', options, {
+		window.__PREACT_DEVTOOLS__.attachPreact('10.15.1', options, {
 			Fragment,
 			Component
 		});
diff --git a/package-lock.json b/package-lock.json
index 2c46c749d4..46fcee73cb 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "preact",
-  "version": "10.15.0",
+  "version": "10.15.1",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "preact",
-      "version": "10.15.0",
+      "version": "10.15.1",
       "license": "MIT",
       "devDependencies": {
         "@actions/github": "^5.0.0",
diff --git a/package.json b/package.json
index edebe7cb53..5d435f6f9d 100644
--- a/package.json
+++ b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "preact",
   "amdName": "preact",
-  "version": "10.15.0",
+  "version": "10.15.1",
   "private": false,
   "description": "Fast 3kb React-compatible Virtual DOM library.",
   "main": "dist/preact.js",

From 3efd4c6817bf7490b50d128445093832834eaaa7 Mon Sep 17 00:00:00 2001
From: shoonia <alexanderz@wix.com>
Date: Sun, 28 May 2023 17:00:48 +0300
Subject: [PATCH 12/21] Add missing aria attributes

---
 src/jsx.d.ts | 26 +++++++++++++++++++-------
 1 file changed, 19 insertions(+), 7 deletions(-)

diff --git a/src/jsx.d.ts b/src/jsx.d.ts
index 4528221da5..26b91cd81a 100644
--- a/src/jsx.d.ts
+++ b/src/jsx.d.ts
@@ -1073,10 +1073,8 @@ export namespace JSXInternal {
 		Target,
 		WheelEvent
 	>;
-	export type TargetedPictureInPictureEvent<Target extends EventTarget> = TargetedEvent<
-		Target,
-		PictureInPictureEvent
-	>;
+	export type TargetedPictureInPictureEvent<Target extends EventTarget> =
+		TargetedEvent<Target, PictureInPictureEvent>;
 
 	export interface EventHandler<E extends TargetedEvent> {
 		(this: void, event: E): void;
@@ -1120,9 +1118,8 @@ export namespace JSXInternal {
 	export type WheelEventHandler<Target extends EventTarget> = EventHandler<
 		TargetedWheelEvent<Target>
 	>;
-	export type PictureInPictureEventHandler<Target extends EventTarget> = EventHandler<
-		TargetedPictureInPictureEvent<Target>
-	>;
+	export type PictureInPictureEventHandler<Target extends EventTarget> =
+		EventHandler<TargetedPictureInPictureEvent<Target>>;
 
 	export interface DOMAttributes<Target extends EventTarget>
 		extends PreactDOMAttributes {
@@ -1388,6 +1385,11 @@ export namespace JSXInternal {
 		 * @see aria-colspan.
 		 */
 		'aria-colindex'?: Signalish<number | undefined>;
+		/**
+		 * Defines a human readable text alternative of aria-colindex.
+		 * @see aria-rowindextext.
+		 */
+		'aria-colindextext'?: Signalish<string | undefined>;
 		/**
 		 * Defines the number of columns spanned by a cell or gridcell within a table, grid, or treegrid.
 		 * @see aria-colindex
@@ -1408,6 +1410,11 @@ export namespace JSXInternal {
 		 * @see aria-labelledby
 		 */
 		'aria-describedby'?: Signalish<string | undefined>;
+		/**
+		 * Defines a string value that describes or annotates the current element.
+		 * @see related aria-describedby.
+		 */
+		'aria-description'?: Signalish<string | undefined>;
 		/**
 		 * Identifies the element that provides a detailed, extended description for the object.
 		 * @see aria-describedby.
@@ -1541,6 +1548,11 @@ export namespace JSXInternal {
 		 * @see aria-rowspan.
 		 */
 		'aria-rowindex'?: Signalish<number | undefined>;
+		/**
+		 * Defines a human readable text alternative of aria-rowindex.
+		 * @see aria-colindextext.
+		 */
+		'aria-rowindextext'?: Signalish<string | undefined>;
 		/**
 		 * Defines the number of rows spanned by a cell or gridcell within a table, grid, or treegrid.
 		 * @see aria-rowindex

From 2833c8e4f51318d95a456237149915bf89271ad7 Mon Sep 17 00:00:00 2001
From: Marvin Hagemeister <hello@marvinh.dev>
Date: Thu, 8 Jun 2023 09:46:35 +0200
Subject: [PATCH 13/21] Include HTML above render root for DOM validation

This fixes debug warnings when Islands are used where the preact
tree is embedded into an existing HTML structure
---
 debug/src/debug.js               | 121 ++++++++++++++++++++-----------
 debug/test/browser/debug.test.js |  14 ++++
 2 files changed, 94 insertions(+), 41 deletions(-)

diff --git a/debug/src/debug.js b/debug/src/debug.js
index 4e1fc03940..d76a60ca45 100644
--- a/debug/src/debug.js
+++ b/debug/src/debug.js
@@ -15,12 +15,22 @@ import { assign, isNaN } from './util';
 
 const isWeakMapSupported = typeof WeakMap == 'function';
 
-function getClosestDomNodeParent(parent) {
-	if (!parent) return {};
+/**
+ * @param {import('./internal').VNode} parent
+ * @returns {string}
+ */
+function getClosestDomNodeParentName(parent) {
+	if (!parent) return '';
 	if (typeof parent.type == 'function') {
-		return getClosestDomNodeParent(parent._parent);
+		if (parent._parent === null) {
+			if (parent._dom !== null && parent._dom.parentNode !== null) {
+				return parent._dom.parentNode.localName;
+			}
+			return '';
+		}
+		return getClosestDomNodeParentName(parent._parent);
 	}
-	return parent;
+	return /** @type {string} */ (parent.type);
 }
 
 export function initDebug() {
@@ -36,6 +46,7 @@ export function initDebug() {
 	let oldCatchError = options._catchError;
 	let oldRoot = options._root;
 	let oldHook = options._hook;
+	let oldCommit = options._commit;
 	const warnedComponents = !isWeakMapSupported
 		? null
 		: {
@@ -44,6 +55,8 @@ export function initDebug() {
 				lazyPropTypes: new WeakMap()
 		  };
 	const deprecations = [];
+	/** @type {import("./internal.d.ts").VNode[]} */
+	let checkVNodeDom = [];
 
 	options._catchError = (error, vnode, oldVNode, errorInfo) => {
 		let component = vnode && vnode._component;
@@ -116,8 +129,18 @@ export function initDebug() {
 	};
 
 	options._diff = vnode => {
-		let { type, _parent: parent } = vnode;
-		let parentVNode = getClosestDomNodeParent(parent);
+		let { type } = vnode;
+		if (
+			typeof type === 'string' &&
+			(type === 'thead' ||
+				type === 'tfoot' ||
+				type === 'tbody' ||
+				type === 'tr' ||
+				type === 'td' ||
+				type === 'th')
+		) {
+			checkVNodeDom.push(vnode);
+		}
 
 		hooksAllowed = true;
 
@@ -146,41 +169,6 @@ export function initDebug() {
 			);
 		}
 
-		if (
-			(type === 'thead' || type === 'tfoot' || type === 'tbody') &&
-			parentVNode.type !== 'table'
-		) {
-			console.error(
-				'Improper nesting of table. Your <thead/tbody/tfoot> should have a <table> parent.' +
-					serializeVNode(vnode) +
-					`\n\n${getOwnerStack(vnode)}`
-			);
-		} else if (
-			type === 'tr' &&
-			parentVNode.type !== 'thead' &&
-			parentVNode.type !== 'tfoot' &&
-			parentVNode.type !== 'tbody' &&
-			parentVNode.type !== 'table'
-		) {
-			console.error(
-				'Improper nesting of table. Your <tr> should have a <thead/tbody/tfoot/table> parent.' +
-					serializeVNode(vnode) +
-					`\n\n${getOwnerStack(vnode)}`
-			);
-		} else if (type === 'td' && parentVNode.type !== 'tr') {
-			console.error(
-				'Improper nesting of table. Your <td> should have a <tr> parent.' +
-					serializeVNode(vnode) +
-					`\n\n${getOwnerStack(vnode)}`
-			);
-		} else if (type === 'th' && parentVNode.type !== 'tr') {
-			console.error(
-				'Improper nesting of table. Your <th> should have a <tr>.' +
-					serializeVNode(vnode) +
-					`\n\n${getOwnerStack(vnode)}`
-			);
-		}
-
 		if (
 			vnode.ref !== undefined &&
 			typeof vnode.ref != 'function' &&
@@ -387,6 +375,57 @@ export function initDebug() {
 			}
 		}
 	};
+
+	options._commit = (root, queue) => {
+		for (let i = 0; i < checkVNodeDom.length; i++) {
+			const vnode = checkVNodeDom[i];
+
+			// Check if HTML nesting is valid. We need to do it in `options.diffed`
+			// so that we can optionally traverse outside the vdom root in case
+			// it's an island embedded in an existing (and valid) HTML tree.
+			const { type, _parent: parent } = vnode;
+
+			let domParentName = getClosestDomNodeParentName(parent);
+
+			if (
+				(type === 'thead' || type === 'tfoot' || type === 'tbody') &&
+				domParentName !== 'table'
+			) {
+				console.error(
+					'Improper nesting of table. Your <thead/tbody/tfoot> should have a <table> parent.' +
+						serializeVNode(vnode) +
+						`\n\n${getOwnerStack(vnode)}`
+				);
+			} else if (
+				type === 'tr' &&
+				domParentName !== 'thead' &&
+				domParentName !== 'tfoot' &&
+				domParentName !== 'tbody' &&
+				domParentName !== 'table'
+			) {
+				console.error(
+					'Improper nesting of table. Your <tr> should have a <thead/tbody/tfoot/table> parent.' +
+						serializeVNode(vnode) +
+						`\n\n${getOwnerStack(vnode)}`
+				);
+			} else if (type === 'td' && domParentName !== 'tr') {
+				console.error(
+					'Improper nesting of table. Your <td> should have a <tr> parent.' +
+						serializeVNode(vnode) +
+						`\n\n${getOwnerStack(vnode)}`
+				);
+			} else if (type === 'th' && domParentName !== 'tr') {
+				console.error(
+					'Improper nesting of table. Your <th> should have a <tr>.' +
+						serializeVNode(vnode) +
+						`\n\n${getOwnerStack(vnode)}`
+				);
+			}
+		}
+		checkVNodeDom = [];
+
+		if (oldCommit) oldCommit(root, queue);
+	};
 }
 
 const setState = Component.prototype.setState;
diff --git a/debug/test/browser/debug.test.js b/debug/test/browser/debug.test.js
index e72668be2a..cbcd01b063 100644
--- a/debug/test/browser/debug.test.js
+++ b/debug/test/browser/debug.test.js
@@ -15,6 +15,7 @@ const h = createElement;
 /** @jsx createElement */
 
 describe('debug', () => {
+	/** @type {HTMLDivElement} */
 	let scratch;
 	let errors = [];
 	let warnings = [];
@@ -513,6 +514,19 @@ describe('debug', () => {
 			render(<Table />, scratch);
 			expect(console.error).to.not.be.called;
 		});
+
+		it('should include DOM parents outside of root node', () => {
+			const Table = () => (
+				<tr>
+					<td>Head</td>
+				</tr>
+			);
+
+			const table = document.createElement('table');
+			scratch.appendChild(table);
+			render(<Table />, table);
+			expect(console.error).to.not.be.called;
+		});
 	});
 
 	describe('PropTypes', () => {

From d8875391b865d6edcf9e1b2c7a4009b218fe966e Mon Sep 17 00:00:00 2001
From: Jovi De Croock <decroockjovi@gmail.com>
Date: Wed, 14 Jun 2023 11:14:44 +0200
Subject: [PATCH 14/21] Port skew based child diffing (#4010)

* first attempt at porting skew-based

* down to only failing fragment test

* reduce complexity of placeChild

* 2 failing tests left

* port skew based diff

* add tests for other fixed issues

* simplify children diffing

* experiment

* remove more nextDom st uff

* one more try

* remove unused import

* simplify suspension check, i.e. whether dom is connected by replacing it with _original === null

* simplify if check

* remove lastDom helper

* simplify more checks

* always use insertBefore

* experiment

* add hyrating check back

* simplify matcher

* try removing isHydrating

* Revert "try removing isHydrating"

This reverts commit 787ad174d1dfc9e79a6d247400b54e663cda3ea5.

* remove isHydrating

* add the !isMounting back
---
 src/diff/children.js           | 244 ++++++++++++++++-----------------
 test/_util/logCall.js          |  25 +++-
 test/browser/fragments.test.js | 118 ++++++++++++++--
 test/browser/keys.test.js      | 154 +++++++++++++++++++--
 test/browser/render.test.js    |  45 ++++++
 5 files changed, 429 insertions(+), 157 deletions(-)

diff --git a/src/diff/children.js b/src/diff/children.js
index 4c388ed6f3..2a59a0cc2a 100644
--- a/src/diff/children.js
+++ b/src/diff/children.js
@@ -1,7 +1,6 @@
 import { diff, unmount, applyRef } from './index';
 import { createVNode, Fragment } from '../create-element';
 import { EMPTY_OBJ, EMPTY_ARR } from '../constants';
-import { getDomSibling } from '../component';
 import { isArray } from '../util';
 
 /**
@@ -36,16 +35,25 @@ export function diffChildren(
 	oldDom,
 	isHydrating
 ) {
-	let i, j, oldVNode, childVNode, newDom, firstChildDom, refs;
+	let i,
+		j,
+		oldVNode,
+		childVNode,
+		newDom,
+		firstChildDom,
+		refs,
+		skew = 0;
 
 	// This is a compression of oldParentVNode!=null && oldParentVNode != EMPTY_OBJ && oldParentVNode._children || EMPTY_ARR
 	// as EMPTY_OBJ._children should be `undefined`.
 	let oldChildren = (oldParentVNode && oldParentVNode._children) || EMPTY_ARR;
 
-	let oldChildrenLength = oldChildren.length;
+	let oldChildrenLength = oldChildren.length,
+		remainingOldChildren = oldChildrenLength,
+		newChildrenLength = renderResult.length;
 
 	newParentVNode._children = [];
-	for (i = 0; i < renderResult.length; i++) {
+	for (i = 0; i < newChildrenLength; i++) {
 		childVNode = renderResult[i];
 
 		if (
@@ -104,40 +112,22 @@ export function diffChildren(
 		childVNode._parent = newParentVNode;
 		childVNode._depth = newParentVNode._depth + 1;
 
-		// Check if we find a corresponding element in oldChildren.
-		// If found, delete the array item by setting to `undefined`.
-		// We use `undefined`, as `null` is reserved for empty placeholders
-		// (holes).
-		oldVNode = oldChildren[i];
+		let skewedIndex = i + skew;
+		const matchingIndex = findMatchingIndex(
+			childVNode,
+			oldChildren,
+			skewedIndex,
+			remainingOldChildren
+		);
 
-		if (
-			oldVNode === null ||
-			(oldVNode &&
-				childVNode.key == oldVNode.key &&
-				childVNode.type === oldVNode.type)
-		) {
-			oldChildren[i] = undefined;
+		if (matchingIndex === -1) {
+			oldVNode = EMPTY_OBJ;
 		} else {
-			// Either oldVNode === undefined or oldChildrenLength > 0,
-			// so after this loop oldVNode == null or oldVNode is a valid value.
-			for (j = 0; j < oldChildrenLength; j++) {
-				oldVNode = oldChildren[j];
-				// If childVNode is unkeyed, we only match similarly unkeyed nodes, otherwise we match by key.
-				// We always match by type (in either case).
-				if (
-					oldVNode &&
-					childVNode.key == oldVNode.key &&
-					childVNode.type === oldVNode.type
-				) {
-					oldChildren[j] = undefined;
-					break;
-				}
-				oldVNode = null;
-			}
+			oldVNode = oldChildren[matchingIndex] || EMPTY_OBJ;
+			oldChildren[matchingIndex] = undefined;
+			remainingOldChildren--;
 		}
 
-		oldVNode = oldVNode || EMPTY_OBJ;
-
 		// Morph the old element into the new one, but don't append it to the dom yet
 		diff(
 			parentDom,
@@ -164,24 +154,60 @@ export function diffChildren(
 				firstChildDom = newDom;
 			}
 
+			let isMounting = oldVNode === EMPTY_OBJ || oldVNode._original === null;
+			let hasMatchingIndex = !isMounting && matchingIndex === skewedIndex;
+			if (isMounting) {
+				if (matchingIndex == -1) {
+					skew--;
+				}
+			} else if (matchingIndex !== skewedIndex) {
+				if (matchingIndex === skewedIndex + 1) {
+					skew++;
+					hasMatchingIndex = true;
+				} else if (matchingIndex > skewedIndex) {
+					if (remainingOldChildren > newChildrenLength - skewedIndex) {
+						skew += matchingIndex - skewedIndex;
+						hasMatchingIndex = true;
+					} else {
+						// ### Change from keyed: I think this was missing from the algo...
+						skew--;
+					}
+				} else if (matchingIndex < skewedIndex) {
+					if (matchingIndex == skewedIndex - 1) {
+						skew = matchingIndex - skewedIndex;
+					} else {
+						skew = 0;
+					}
+				} else {
+					skew = 0;
+				}
+			}
+
+			skewedIndex = i + skew;
+			hasMatchingIndex =
+				hasMatchingIndex || (matchingIndex == i && !isMounting);
+
 			if (
 				typeof childVNode.type == 'function' &&
-				childVNode._children === oldVNode._children
+				(matchingIndex !== skewedIndex ||
+					oldVNode._children === childVNode._children)
 			) {
-				childVNode._nextDom = oldDom = reorderChildren(
-					childVNode,
-					oldDom,
-					parentDom
-				);
+				oldDom = reorderChildren(childVNode, oldDom, parentDom);
+			} else if (typeof childVNode.type != 'function' && !hasMatchingIndex) {
+				oldDom = placeChild(parentDom, newDom, oldDom);
+			} else if (childVNode._nextDom !== undefined) {
+				// Only Fragments or components that return Fragment like VNodes will
+				// have a non-undefined _nextDom. Continue the diff from the sibling
+				// of last DOM child of this child VNode
+				oldDom = childVNode._nextDom;
+
+				// Eagerly cleanup _nextDom. We don't need to persist the value because
+				// it is only used by `diffChildren` to determine where to resume the diff after
+				// diffing Components and Fragments. Once we store it the nextDOM local var, we
+				// can clean up the property
+				childVNode._nextDom = undefined;
 			} else {
-				oldDom = placeChild(
-					parentDom,
-					childVNode,
-					oldVNode,
-					oldChildren,
-					newDom,
-					oldDom
-				);
+				oldDom = newDom.nextSibling;
 			}
 
 			if (typeof newParentVNode.type == 'function') {
@@ -194,14 +220,6 @@ export function diffChildren(
 				// node's nextSibling.
 				newParentVNode._nextDom = oldDom;
 			}
-		} else if (
-			oldDom &&
-			oldVNode._dom == oldDom &&
-			oldDom.parentNode != parentDom
-		) {
-			// The above condition is to handle null placeholders. See test in placeholder.test.js:
-			// `efficiently replace null placeholders in parent rerenders`
-			oldDom = getDomSibling(oldVNode);
 		}
 	}
 
@@ -218,7 +236,8 @@ export function diffChildren(
 				// If the newParentVNode.__nextDom points to a dom node that is about to
 				// be unmounted, then get the next sibling of that vnode and set
 				// _nextDom to it
-				newParentVNode._nextDom = getLastDom(oldParentVNode).nextSibling;
+
+				newParentVNode._nextDom = oldChildren[i]._dom.nextSibling;
 			}
 
 			unmount(oldChildren[i], oldChildren[i]);
@@ -236,6 +255,7 @@ export function diffChildren(
 function reorderChildren(childVNode, oldDom, parentDom) {
 	// Note: VNodes in nested suspended trees may be missing _children.
 	let c = childVNode._children;
+
 	let tmp = 0;
 	for (; c && tmp < c.length; tmp++) {
 		let vnode = c[tmp];
@@ -249,7 +269,7 @@ function reorderChildren(childVNode, oldDom, parentDom) {
 			if (typeof vnode.type == 'function') {
 				oldDom = reorderChildren(vnode, oldDom, parentDom);
 			} else {
-				oldDom = placeChild(parentDom, vnode, vnode, c, vnode._dom, oldDom);
+				oldDom = placeChild(parentDom, vnode._dom, oldDom);
 			}
 		}
 	}
@@ -276,81 +296,59 @@ export function toChildArray(children, out) {
 	return out;
 }
 
-function placeChild(
-	parentDom,
-	childVNode,
-	oldVNode,
-	oldChildren,
-	newDom,
-	oldDom
-) {
-	let nextDom;
-	if (childVNode._nextDom !== undefined) {
-		// Only Fragments or components that return Fragment like VNodes will
-		// have a non-undefined _nextDom. Continue the diff from the sibling
-		// of last DOM child of this child VNode
-		nextDom = childVNode._nextDom;
-
-		// Eagerly cleanup _nextDom. We don't need to persist the value because
-		// it is only used by `diffChildren` to determine where to resume the diff after
-		// diffing Components and Fragments. Once we store it the nextDOM local var, we
-		// can clean up the property
-		childVNode._nextDom = undefined;
-	} else if (
-		oldVNode == null ||
-		newDom != oldDom ||
-		newDom.parentNode == null
-	) {
-		outer: if (oldDom == null || oldDom.parentNode !== parentDom) {
-			parentDom.appendChild(newDom);
-			nextDom = null;
-		} else {
-			// `j<oldChildrenLength; j+=2` is an alternative to `j++<oldChildrenLength/2`
-			for (
-				let sibDom = oldDom, j = 0;
-				(sibDom = sibDom.nextSibling) && j < oldChildren.length;
-				j += 1
-			) {
-				if (sibDom == newDom) {
-					break outer;
-				}
-			}
-			parentDom.insertBefore(newDom, oldDom);
-			nextDom = oldDom;
-		}
-	}
-
-	// If we have pre-calculated the nextDOM node, use it. Else calculate it now
-	// Strictly check for `undefined` here cuz `null` is a valid value of `nextDom`.
-	// See more detail in create-element.js:createVNode
-	if (nextDom !== undefined) {
-		oldDom = nextDom;
-	} else {
-		oldDom = newDom.nextSibling;
+function placeChild(parentDom, newDom, oldDom) {
+	if (oldDom == null || oldDom.parentNode !== parentDom) {
+		parentDom.insertBefore(newDom, null);
+	} else if (newDom != oldDom || newDom.parentNode == null) {
+		parentDom.insertBefore(newDom, oldDom);
 	}
 
-	return oldDom;
+	return newDom.nextSibling;
 }
 
 /**
- * @param {import('../internal').VNode} vnode
+ * @param {import('../internal').VNode | string} childVNode
+ * @param {import('../internal').VNode[]} oldChildren
+ * @param {number} skewedIndex
+ * @param {number} remainingOldChildren
+ * @returns {number}
  */
-function getLastDom(vnode) {
-	if (vnode.type == null || typeof vnode.type === 'string') {
-		return vnode._dom;
-	}
+function findMatchingIndex(
+	childVNode,
+	oldChildren,
+	skewedIndex,
+	remainingOldChildren
+) {
+	const key = childVNode.key;
+	const type = childVNode.type;
+	let x = skewedIndex - 1;
+	let y = skewedIndex + 1;
+	let oldVNode = oldChildren[skewedIndex];
+
+	if (
+		oldVNode === null ||
+		(oldVNode && key == oldVNode.key && type === oldVNode.type)
+	) {
+		return skewedIndex;
+	} else if (remainingOldChildren > (oldVNode != null ? 1 : 0)) {
+		while (x >= 0 || y < oldChildren.length) {
+			if (x >= 0) {
+				oldVNode = oldChildren[x];
+				if (oldVNode && key == oldVNode.key && type === oldVNode.type) {
+					return x;
+				}
+				x--;
+			}
 
-	if (vnode._children) {
-		for (let i = vnode._children.length - 1; i >= 0; i--) {
-			let child = vnode._children[i];
-			if (child) {
-				let lastDom = getLastDom(child);
-				if (lastDom) {
-					return lastDom;
+			if (y < oldChildren.length) {
+				oldVNode = oldChildren[y];
+				if (oldVNode && key == oldVNode.key && type === oldVNode.type) {
+					return y;
 				}
+				y++;
 			}
 		}
 	}
 
-	return null;
+	return -1;
 }
diff --git a/test/_util/logCall.js b/test/_util/logCall.js
index a9ce2e877b..d6d928e7ed 100644
--- a/test/_util/logCall.js
+++ b/test/_util/logCall.js
@@ -29,11 +29,26 @@ export function logCall(obj, method) {
 			c += serialize(args[i]);
 		}
 
-		// Normalize removeChild -> remove to keep output clean and readable
-		const operation =
-			method != 'removeChild'
-				? `${serialize(this)}.${method}(${c})`
-				: `${serialize(c)}.remove()`;
+		let operation;
+		switch (method) {
+			case 'removeChild': {
+				operation = `${serialize(c)}.remove()`;
+				break;
+			}
+			case 'insertBefore': {
+				if (args[1] === null && args.length === 2) {
+					operation = `${serialize(this)}.appendChild(${serialize(args[0])})`;
+				} else {
+					operation = `${serialize(this)}.${method}(${c})`;
+				}
+				break;
+			}
+			default: {
+				operation = `${serialize(this)}.${method}(${c})`;
+				break;
+			}
+		}
+
 		log.push(operation);
 		return old.apply(this, args);
 	};
diff --git a/test/browser/fragments.test.js b/test/browser/fragments.test.js
index e6df094d73..55842b952d 100644
--- a/test/browser/fragments.test.js
+++ b/test/browser/fragments.test.js
@@ -687,7 +687,8 @@ describe('Fragment', () => {
 		expect(scratch.innerHTML).to.equal(htmlForFalse);
 		expectDomLogToBe(
 			[
-				'<div>fooHellobeep.appendChild(<div>Hello)',
+				'<div>barHellobeep.insertBefore(<div>bar, <div>beep)',
+				'<div>Hellobarbeep.appendChild(<div>Hello)',
 				'<div>barbeepHello.appendChild(<div>bar)'
 			],
 			'rendering true to false'
@@ -1277,9 +1278,7 @@ describe('Fragment', () => {
 		);
 		expectDomLogToBe([
 			'<ol>012345.insertBefore(<li>4, <li>0)',
-			'<ol>401235.insertBefore(<li>5, <li>0)',
-			// TODO: Hmmm why does this extra append happen?
-			'<ol>453012.appendChild(<li>3)'
+			'<ol>401235.insertBefore(<li>5, <li>0)'
 		]);
 
 		clearLog();
@@ -1478,7 +1477,8 @@ describe('Fragment', () => {
 		);
 		expectDomLogToBe(
 			[
-				'<div>fooHellobeepboop.insertBefore(<div>Hello, <div>boop)',
+				'<div>barHellobeepboop.insertBefore(<div>bar, <div>beep)',
+				'<div>Hellobarbeepboop.insertBefore(<div>Hello, <div>boop)',
 				'<div>barbeepHelloboop.insertBefore(<div>bar, <div>boop)',
 				'<div>boop.remove()'
 			],
@@ -1564,7 +1564,8 @@ describe('Fragment', () => {
 		);
 		expectDomLogToBe(
 			[
-				'<div>fooHellobeepbeepbeep.appendChild(<div>Hello)',
+				'<div>barHellobeepbeepbeep.insertBefore(<div>bar, <div>beep)',
+				'<div>Hellobarbeepbeepbeep.appendChild(<div>Hello)',
 				'<div>barbeepbeepbeepHello.appendChild(<div>bar)'
 			],
 			'rendering from true to false'
@@ -1580,9 +1581,9 @@ describe('Fragment', () => {
 		);
 		expectDomLogToBe(
 			[
-				'<div>beepbeepbeepHellofoo.insertBefore(<div>foo, <div>beep)',
-				'<div>foobeepbeepbeepHello.insertBefore(<div>Hello, <div>beep)',
-				'<div>fooHelloboopboopboop.appendChild(<div>boop)'
+				'<div>beepbeepbeepHellofoo.appendChild(<div>Hello)',
+				'<div>beepbeepbeepfooHello.insertBefore(<div>foo, <div>beep)',
+				'<div>foobeepbeepbeepHello.insertBefore(<div>Hello, <div>beep)'
 			],
 			'rendering from false to true'
 		);
@@ -2608,8 +2609,10 @@ describe('Fragment', () => {
 		rerender();
 		expect(scratch.innerHTML).to.equal(bottom);
 		expectDomLogToBe([
+			'<div>top panelNavigationContent.insertBefore(<div>Navigation, <div>top panel)',
+			'<div>Navigationtop panelContent.insertBefore(<div>Content, <div>top panel)',
 			'<div>.appendChild(#text)',
-			'<div>top panelNavigationContent.appendChild(<div>bottom panel)',
+			'<div>NavigationContenttop panel.insertBefore(<div>bottom panel, <div>top panel)',
 			'<div>top panel.remove()'
 		]);
 
@@ -2703,6 +2706,96 @@ describe('Fragment', () => {
 		expectDomLogToBe(['<div>2.remove()', '<div>3.remove()']);
 	});
 
+	it('should properly unmount Fragment children around an unmounting null placeholder #2987', () => {
+		const arrayOf = (n, fill = 0) => new Array(n).fill(fill);
+
+		class App extends Component {
+			constructor(props) {
+				super(props);
+
+				this.state = {
+					renderFirstElement: true,
+					renderLastElement: true,
+					childrenProps: [
+						{
+							arrayData: arrayOf(10).map((_, index) => ({
+								key: index + 5,
+								text: index + 5
+							}))
+						},
+						{
+							arrayData: arrayOf(10).map((_, index) => ({
+								key: index + 15,
+								text: index + 15
+							}))
+						}
+					]
+				};
+			}
+
+			componentDidMount() {
+				// eslint-disable-next-line react/no-did-mount-set-state
+				this.setState({
+					renderFirstElement: false,
+					renderLastElement: true,
+					childrenProps: [
+						{
+							arrayData: arrayOf(15).map((_, index) => ({
+								key: index,
+								text: index
+							}))
+						},
+						{
+							arrayData: arrayOf(5).map((_, index) => ({
+								key: index + 15,
+								text: index + 15
+							}))
+						}
+					]
+				});
+			}
+
+			render() {
+				const { renderFirstElement, renderLastElement, childrenProps } =
+					this.state;
+				return (
+					<div>
+						{renderFirstElement && <div>This is the first div</div>}
+						{childrenProps.map(({ arrayData }) => {
+							return arrayData.map(({ key, text }) => (
+								<div key={key}>{text}</div>
+							));
+						})}
+						{renderLastElement && <div>This is the last div</div>}
+					</div>
+				);
+			}
+		}
+
+		render(<App />, scratch);
+		expect(scratch.innerHTML).to.equal(
+			div([
+				div('This is the first div'),
+				arrayOf(20)
+					.map((_, i) => div(i + 5)) // 5 - 24
+					.join(''),
+				div('This is the last div')
+			])
+		);
+
+		// Flush CDM setState call
+		rerender();
+		expect(scratch.innerHTML).to.equal(
+			div([
+				// "This is the first div" is unmounted using a null placeholder pattern
+				arrayOf(20)
+					.map((_, i) => div(i)) // 0 - 19 (0 - 4 are inserted, 20 - 24 are unmounted)
+					.join(''),
+				div('This is the last div')
+			])
+		);
+	});
+
 	it('should efficiently place new children and unmount nested Fragment children', () => {
 		// <div>4 is added and Fragment sibling unmounts. Does <div>4 get correct _nextDom pointer?
 		function App({ condition }) {
@@ -2952,9 +3045,6 @@ describe('Fragment', () => {
 		rerender();
 
 		expect(scratch.innerHTML).to.equal([div('A'), div(1), div(2)].join(''));
-		expectDomLogToBe([
-			'<div>B.remove()',
-			'<div>1A2.insertBefore(<div>1, <div>2)'
-		]);
+		expectDomLogToBe(['<div>B.remove()', '<div>2A1.appendChild(<div>2)']);
 	});
 });
diff --git a/test/browser/keys.test.js b/test/browser/keys.test.js
index bb99932cf1..118efbcf82 100644
--- a/test/browser/keys.test.js
+++ b/test/browser/keys.test.js
@@ -1,7 +1,8 @@
-import { createElement, Component, render } from 'preact';
+import { createElement, Component, render, createRef } from 'preact';
 import { setupScratch, teardown } from '../_util/helpers';
 import { logCall, clearLog, getLog } from '../_util/logCall';
 import { div } from '../_util/dom';
+import { setupRerender } from 'preact/test-utils';
 
 /** @jsx createElement */
 
@@ -9,6 +10,8 @@ describe('keys', () => {
 	/** @type {HTMLDivElement} */
 	let scratch;
 
+	let rerender;
+
 	/** @type {string[]} */
 	let ops;
 
@@ -70,6 +73,7 @@ describe('keys', () => {
 	});
 
 	beforeEach(() => {
+		rerender = setupRerender();
 		scratch = setupScratch();
 		ops = [];
 	});
@@ -319,11 +323,7 @@ describe('keys', () => {
 		render(<List values={values} />, scratch);
 		expect(scratch.textContent).to.equal('abcd', 'move to beginning');
 		expect(getLog()).to.deep.equal(
-			[
-				'<ol>bcda.appendChild(<li>b)',
-				'<ol>cdab.appendChild(<li>c)',
-				'<ol>dabc.appendChild(<li>d)'
-			],
+			['<ol>bcda.insertBefore(<li>a, <li>b)'],
 			'move to beginning'
 		);
 	});
@@ -341,15 +341,15 @@ describe('keys', () => {
 		render(<List values={values} />, scratch);
 		expect(scratch.textContent).to.equal(values.join(''));
 		expect(getLog()).to.deep.equal([
-			'<ol>abcdefghij.appendChild(<li>i)',
-			'<ol>abcdefghji.appendChild(<li>h)',
-			'<ol>abcdefgjih.appendChild(<li>g)',
-			'<ol>abcdefjihg.appendChild(<li>f)',
-			'<ol>abcdejihgf.appendChild(<li>e)',
-			'<ol>abcdjihgfe.appendChild(<li>d)',
-			'<ol>abcjihgfed.appendChild(<li>c)',
-			'<ol>abjihgfedc.appendChild(<li>b)',
-			'<ol>ajihgfedcb.appendChild(<li>a)'
+			'<ol>abcdefghij.insertBefore(<li>j, <li>a)',
+			'<ol>jabcdefghi.insertBefore(<li>i, <li>a)',
+			'<ol>jiabcdefgh.insertBefore(<li>h, <li>a)',
+			'<ol>jihabcdefg.insertBefore(<li>g, <li>a)',
+			'<ol>jihgabcdef.insertBefore(<li>f, <li>a)',
+			'<ol>jihgfabcde.insertBefore(<li>e, <li>a)',
+			'<ol>jihgfeabcd.insertBefore(<li>d, <li>a)',
+			'<ol>jihgfedabc.insertBefore(<li>c, <li>a)',
+			'<ol>jihgfedcab.appendChild(<li>a)'
 		]);
 	});
 
@@ -558,6 +558,130 @@ describe('keys', () => {
 		expect(Stateful2Ref).to.equal(Stateful2MovedRef);
 	});
 
+	it('should effectively iterate on large lists', done => {
+		const newItems = () =>
+			Array(100)
+				.fill(0)
+				.map((item, i) => i);
+
+		let set,
+			mutatedNodes = [];
+
+		class App extends Component {
+			constructor(props) {
+				super(props);
+				this.state = { items: newItems() };
+				set = this.set = this.set.bind(this);
+				this.ref = createRef();
+			}
+
+			componentDidMount() {
+				const observer = new MutationObserver(listener);
+				observer.observe(this.ref.current, { childList: true });
+
+				function listener(mutations) {
+					for (const { addedNodes } of mutations) {
+						for (const node of addedNodes) {
+							mutatedNodes.push(node);
+						}
+					}
+				}
+			}
+
+			set() {
+				const currentItems = this.state.items;
+				const items = newItems().filter(id => {
+					const isVisible = currentItems.includes(id);
+					return id >= 20 && id <= 80 ? !isVisible : isVisible;
+				});
+				this.setState({ items });
+			}
+
+			render() {
+				return (
+					<div ref={this.ref}>
+						{this.state.items.map(i => (
+							<div key={i}>{i}</div>
+						))}
+					</div>
+				);
+			}
+		}
+
+		render(<App />, scratch);
+
+		set();
+		rerender();
+
+		setTimeout(() => {
+			expect(mutatedNodes.length).to.equal(0);
+			done();
+		});
+	});
+
+	it('should effectively iterate on large component lists', done => {
+		const newItems = () =>
+			Array(100)
+				.fill(0)
+				.map((item, i) => i);
+
+		let set,
+			mutatedNodes = [];
+
+		const Row = ({ i }) => <p>{i}</p>;
+
+		class App extends Component {
+			constructor(props) {
+				super(props);
+				this.state = { items: newItems() };
+				set = this.set = this.set.bind(this);
+				this.ref = createRef();
+			}
+
+			componentDidMount() {
+				const observer = new MutationObserver(listener);
+				observer.observe(this.ref.current, { childList: true });
+
+				function listener(mutations) {
+					for (const { addedNodes } of mutations) {
+						for (const node of addedNodes) {
+							mutatedNodes.push(node);
+						}
+					}
+				}
+			}
+
+			set() {
+				const currentItems = this.state.items;
+				const items = newItems().filter(id => {
+					const isVisible = currentItems.includes(id);
+					return id >= 20 && id <= 80 ? !isVisible : isVisible;
+				});
+				this.setState({ items });
+			}
+
+			render() {
+				return (
+					<div ref={this.ref}>
+						{this.state.items.map(i => (
+							<Row key={i} i={i} />
+						))}
+					</div>
+				);
+			}
+		}
+
+		render(<App />, scratch);
+
+		set();
+		rerender();
+
+		setTimeout(() => {
+			expect(mutatedNodes.length).to.equal(0);
+			done();
+		});
+	});
+
 	it('should not preserve state when switching between keyed and unkeyed components as children', () => {
 		// React & Preact v8 behavior: https://codesandbox.io/s/8l3p6lz9kj
 
diff --git a/test/browser/render.test.js b/test/browser/render.test.js
index dc04ca3011..308f768185 100644
--- a/test/browser/render.test.js
+++ b/test/browser/render.test.js
@@ -1061,6 +1061,51 @@ describe('render()', () => {
 		expect(scratch.textContent).to.equal('01');
 	});
 
+	it('should not remove iframe', () => {
+		let setState;
+		const Iframe = () => {
+			return <iframe src="https://codesandbox.io/s/runtime-silence-no4zx" />;
+		};
+
+		const Test2 = () => <div>Test2</div>;
+		const Test3 = () => <div>Test3</div>;
+
+		class App extends Component {
+			constructor(props) {
+				super(props);
+				this.state = { value: true };
+				setState = this.setState.bind(this);
+			}
+
+			render(props, state) {
+				return (
+					<div>
+						{state.value ? <Test3 /> : null}
+						{state.value ? <Test2 /> : null}
+						<Iframe key="iframe" />
+					</div>
+				);
+			}
+		}
+
+		render(<App />, scratch);
+
+		expect(scratch.innerHTML).to.equal(
+			'<div><div>Test3</div><div>Test2</div><iframe src="https://codesandbox.io/s/runtime-silence-no4zx"></iframe></div>'
+		);
+		clearLog();
+		setState({ value: false });
+		rerender();
+
+		expect(scratch.innerHTML).to.equal(
+			'<div><iframe src="https://codesandbox.io/s/runtime-silence-no4zx"></iframe></div>'
+		);
+		expect(getLog()).to.deep.equal([
+			'<div>Test2.remove()',
+			'<div>Test3.remove()'
+		]);
+	});
+
 	it('should not cause infinite loop with referentially equal props', () => {
 		let i = 0;
 		let prevDiff = options._diff;

From f1eed0e8711a18a12119d7ba5bfe78468d76eb57 Mon Sep 17 00:00:00 2001
From: dmitrage <dmitrage@gmail.com>
Date: Fri, 16 Jun 2023 18:31:27 +0200
Subject: [PATCH 15/21] Fix forceUpdate edge cases

---
 src/diff/index.js               | 11 +++----
 test/browser/components.test.js | 53 +++++++++++++++++++++++++++++++++
 2 files changed, 57 insertions(+), 7 deletions(-)

diff --git a/src/diff/index.js b/src/diff/index.js
index 477663cd9e..87c0ba21e4 100644
--- a/src/diff/index.js
+++ b/src/diff/index.js
@@ -134,14 +134,14 @@ export function diff(
 				}
 
 				if (
-					(!c._force &&
-						c.shouldComponentUpdate != null &&
+					!c._force &&
+					((c.shouldComponentUpdate != null &&
 						c.shouldComponentUpdate(
 							newProps,
 							c._nextState,
 							componentContext
 						) === false) ||
-					newVNode._original === oldVNode._original
+						newVNode._original === oldVNode._original)
 				) {
 					// More info about this here: https://gist.github.com/JoviDeCroock/bec5f2ce93544d2e6070ef8e0036e4e8
 					if (newVNode._original !== oldVNode._original) {
@@ -154,8 +154,6 @@ export function diff(
 						c._dirty = false;
 					}
 
-					// In cases of bailing due to strict-equality we have to reset force as well
-					c._force = false;
 					newVNode._dom = oldVNode._dom;
 					newVNode._children = oldVNode._children;
 					newVNode._children.forEach(vnode => {
@@ -188,6 +186,7 @@ export function diff(
 			c.context = componentContext;
 			c.props = newProps;
 			c._parentDom = parentDom;
+			c._force = false;
 
 			let renderHook = options._render,
 				count = 0;
@@ -255,8 +254,6 @@ export function diff(
 			if (clearProcessingException) {
 				c._pendingError = c._processingException = null;
 			}
-
-			c._force = false;
 		} else if (
 			excessDomChildren == null &&
 			newVNode._original === oldVNode._original
diff --git a/test/browser/components.test.js b/test/browser/components.test.js
index 1391d52603..3d36e3a29f 100644
--- a/test/browser/components.test.js
+++ b/test/browser/components.test.js
@@ -2660,5 +2660,58 @@ describe('Components', () => {
 
 			expect(scratch.innerHTML).to.equal('<div>bar</div>');
 		});
+
+		it('should skip shouldComponentUpdate when called during render', () => {
+			let isSCUCalled = false;
+			class App extends Component {
+				shouldComponentUpdate() {
+					isSCUCalled = true;
+					return false;
+				}
+				render() {
+					const isUpdated = this.isUpdated;
+					if (!isUpdated) {
+						this.isUpdated = true;
+						this.forceUpdate();
+					}
+					return <div>Updated: {isUpdated ? 'yes' : 'no'}</div>;
+				}
+			}
+			render(<App />, scratch);
+			rerender();
+			expect(isSCUCalled).to.be.false;
+			expect(scratch.innerHTML).to.equal('<div>Updated: yes</div>');
+		});
+
+		it('should break through strict equality optimization', () => {
+			let isSCUCalled = false;
+
+			class Child extends Component {
+				componentDidMount() {
+					this.props.parent.forceUpdate();
+					this.forceUpdate();
+					this.isUpdated = true;
+				}
+				shouldComponentUpdate() {
+					isSCUCalled = true;
+					return false;
+				}
+				render() {
+					return <div>Updated: {this.isUpdated ? 'yes' : 'no'}</div>;
+				}
+			}
+
+			class App extends Component {
+				children = (<Child parent={this} />);
+				render() {
+					return this.children;
+				}
+			}
+
+			render(<App />, scratch);
+			rerender();
+			expect(isSCUCalled).to.be.false;
+			expect(scratch.innerHTML).to.equal('<div>Updated: yes</div>');
+		});
 	});
 });

From 56d8f41c5df69f736463cca33f238c4d24da178d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mika=C3=ABl=20FIMA?= <MikaFima@users.noreply.github.com>
Date: Sat, 24 Jun 2023 10:47:12 +0200
Subject: [PATCH 16/21] Server compat should have a default export (#4047)

---
 compat/server.browser.js | 7 +++++++
 compat/server.mjs        | 7 +++++++
 2 files changed, 14 insertions(+)

diff --git a/compat/server.browser.js b/compat/server.browser.js
index 9e8858e822..51c7a5d332 100644
--- a/compat/server.browser.js
+++ b/compat/server.browser.js
@@ -1,4 +1,11 @@
+import { renderToString } from 'preact-render-to-string';
+
 export {
 	renderToString,
 	renderToString as renderToStaticMarkup
 } from 'preact-render-to-string';
+
+export default {
+	renderToString,
+	renderToStaticMarkup: renderToString
+};
diff --git a/compat/server.mjs b/compat/server.mjs
index 9e8858e822..51c7a5d332 100644
--- a/compat/server.mjs
+++ b/compat/server.mjs
@@ -1,4 +1,11 @@
+import { renderToString } from 'preact-render-to-string';
+
 export {
 	renderToString,
 	renderToString as renderToStaticMarkup
 } from 'preact-render-to-string';
+
+export default {
+	renderToString,
+	renderToStaticMarkup: renderToString
+};

From 0ef740bbaa6c41c9d8b5173443b4960ec6c42523 Mon Sep 17 00:00:00 2001
From: Jovi De Croock <decroockjovi@gmail.com>
Date: Sun, 25 Jun 2023 09:20:30 +0200
Subject: [PATCH 17/21] improve VNode typings when generic is passed in (#4050)

---
 src/index.d.ts | 35 +++++++++++++++++++++++++----------
 1 file changed, 25 insertions(+), 10 deletions(-)

diff --git a/src/index.d.ts b/src/index.d.ts
index c57fcb6f6f..a2a881d2bb 100644
--- a/src/index.d.ts
+++ b/src/index.d.ts
@@ -194,7 +194,11 @@ export function createElement(
 				ClassAttributes<HTMLInputElement>)
 		| null,
 	...children: ComponentChildren[]
-): VNode<any>;
+): VNode<
+	| (JSXInternal.DOMAttributes<HTMLInputElement> &
+			ClassAttributes<HTMLInputElement>)
+	| null
+>;
 export function createElement<
 	P extends JSXInternal.HTMLAttributes<T>,
 	T extends HTMLElement
@@ -202,7 +206,7 @@ export function createElement<
 	type: keyof JSXInternal.IntrinsicElements,
 	props: (ClassAttributes<T> & P) | null,
 	...children: ComponentChildren[]
-): VNode<any>;
+): VNode<(ClassAttributes<T> & P) | null>;
 export function createElement<
 	P extends JSXInternal.SVGAttributes<T>,
 	T extends HTMLElement
@@ -210,7 +214,7 @@ export function createElement<
 	type: keyof JSXInternal.IntrinsicElements,
 	props: (ClassAttributes<T> & P) | null,
 	...children: ComponentChildren[]
-): VNode<any>;
+): VNode<(ClassAttributes<T> & P) | null>;
 export function createElement<T extends HTMLElement>(
 	type: string,
 	props:
@@ -219,12 +223,14 @@ export function createElement<T extends HTMLElement>(
 				JSXInternal.SVGAttributes)
 		| null,
 	...children: ComponentChildren[]
-): VNode<any>;
+): VNode<
+	ClassAttributes<T> & JSXInternal.HTMLAttributes & JSXInternal.SVGAttributes
+>;
 export function createElement<P>(
 	type: ComponentType<P>,
 	props: (Attributes & P) | null,
 	...children: ComponentChildren[]
-): VNode<any>;
+): VNode<P>;
 export namespace createElement {
 	export import JSX = JSXInternal;
 }
@@ -236,7 +242,11 @@ export function h(
 				ClassAttributes<HTMLInputElement>)
 		| null,
 	...children: ComponentChildren[]
-): VNode<any>;
+): VNode<
+	| (JSXInternal.DOMAttributes<HTMLInputElement> &
+			ClassAttributes<HTMLInputElement>)
+	| null
+>;
 export function h<
 	P extends JSXInternal.HTMLAttributes<T>,
 	T extends HTMLElement
@@ -244,7 +254,7 @@ export function h<
 	type: keyof JSXInternal.IntrinsicElements,
 	props: (ClassAttributes<T> & P) | null,
 	...children: ComponentChildren[]
-): VNode<any>;
+): VNode<(ClassAttributes<T> & P) | null>;
 export function h<
 	P extends JSXInternal.SVGAttributes<T>,
 	T extends HTMLElement
@@ -252,7 +262,7 @@ export function h<
 	type: keyof JSXInternal.IntrinsicElements,
 	props: (ClassAttributes<T> & P) | null,
 	...children: ComponentChildren[]
-): VNode<any>;
+): VNode<(ClassAttributes<T> & P) | null>;
 export function h<T extends HTMLElement>(
 	type: string,
 	props:
@@ -261,12 +271,17 @@ export function h<T extends HTMLElement>(
 				JSXInternal.SVGAttributes)
 		| null,
 	...children: ComponentChildren[]
-): VNode<any>;
+): VNode<
+	| (ClassAttributes<T> &
+			JSXInternal.HTMLAttributes &
+			JSXInternal.SVGAttributes)
+	| null
+>;
 export function h<P>(
 	type: ComponentType<P>,
 	props: (Attributes & P) | null,
 	...children: ComponentChildren[]
-): VNode<any>;
+): VNode<(Attributes & P) | null>;
 export namespace h {
 	export import JSX = JSXInternal;
 }

From a12d60e6d7bfafe0f3b776776875a9ad40197e3e Mon Sep 17 00:00:00 2001
From: "xiaofeng.mxf" <xiaofeng.mxf@taobao.com>
Date: Sun, 25 Jun 2023 18:01:48 +0800
Subject: [PATCH 18/21] fix: incorrect actDepth calculation in test-utils

---
 test-utils/src/index.js            | 18 +++++++++++++-----
 test-utils/test/shared/act.test.js | 12 ++++++++++++
 2 files changed, 25 insertions(+), 5 deletions(-)

diff --git a/test-utils/src/index.js b/test-utils/src/index.js
index 4bf7a21f77..f8df33393d 100644
--- a/test-utils/src/index.js
+++ b/test-utils/src/index.js
@@ -32,11 +32,19 @@ export function act(cb) {
 		// been set up.
 		//
 		// If an exception occurs, the outermost `act` will handle cleanup.
-		const result = cb();
-		if (isThenable(result)) {
-			return result.then(() => {
-				--actDepth;
-			});
+		try {
+			const result = cb();
+			if (isThenable(result)) {
+				return result.then(() => {
+					--actDepth;
+				}, (e) => {
+					--actDepth;
+					throw e;
+				})
+			}
+		} catch(e) {
+			--actDepth;
+			throw e;
 		}
 		--actDepth;
 		return Promise.resolve();
diff --git a/test-utils/test/shared/act.test.js b/test-utils/test/shared/act.test.js
index d1f6ef248f..af9823b906 100644
--- a/test-utils/test/shared/act.test.js
+++ b/test-utils/test/shared/act.test.js
@@ -385,6 +385,12 @@ describe('act', () => {
 			} catch (e) {}
 		};
 
+		const tryNestedRenderBroken = () => {
+			act(() => {
+				tryRenderBroken();
+			});
+		}
+
 		describe('synchronously', () => {
 			it('should rethrow the exception', () => {
 				expect(renderBroken).to.throw('BrokenWidget is broken');
@@ -396,6 +402,12 @@ describe('act', () => {
 				expect(scratch.textContent).to.equal('1');
 			});
 
+			it('should not affect state updates in future renders when nested `act` throws an exception', () => {
+				tryNestedRenderBroken();
+				renderWorking();
+				expect(scratch.textContent).to.equal('1');
+			});
+			
 			it('should not affect effects in future renders', () => {
 				tryRenderBroken();
 				renderWorking();

From 76d690973ee40f8fbaf0c687fa9433da49c4b5f3 Mon Sep 17 00:00:00 2001
From: Jovi De Croock <decroockjovi@gmail.com>
Date: Fri, 30 Jun 2023 07:23:08 +0200
Subject: [PATCH 19/21] fix: setting ref to null after updating it with new
 element (#4054)

---
 hooks/test/browser/combinations.test.js    |  79 +++++++++
 hooks/test/browser/errorBoundary.test.js   |  47 +++++-
 hooks/test/browser/useLayoutEffect.test.js |  37 +++++
 src/component.js                           |   9 +-
 src/diff/children.js                       |  22 ++-
 src/diff/index.js                          |  25 ++-
 src/render.js                              |   8 +-
 test/browser/refs.test.js                  | 178 ++++++++++++++++++++-
 8 files changed, 376 insertions(+), 29 deletions(-)

diff --git a/hooks/test/browser/combinations.test.js b/hooks/test/browser/combinations.test.js
index 02db5b74f2..028110e8b9 100644
--- a/hooks/test/browser/combinations.test.js
+++ b/hooks/test/browser/combinations.test.js
@@ -398,4 +398,83 @@ describe('combinations', () => {
 		});
 		expect(scratch.textContent).to.equal('2');
 	});
+
+	it.skip('parent and child refs should be set before all effects', () => {
+		const anchorId = 'anchor';
+		const tooltipId = 'tooltip';
+		const effectLog = [];
+
+		let useRef2 = sinon.spy(init => {
+			const realRef = useRef(init);
+			const ref = useRef(init);
+			Object.defineProperty(ref, 'current', {
+				get: () => realRef.current,
+				set: value => {
+					realRef.current = value;
+					effectLog.push('set ref ' + value?.tagName);
+				}
+			});
+			return ref;
+		});
+
+		function Tooltip({ anchorRef, children }) {
+			// For example, used to manually position the tooltip
+			const tooltipRef = useRef2(null);
+
+			useLayoutEffect(() => {
+				expect(anchorRef.current?.id).to.equal(anchorId);
+				expect(tooltipRef.current?.id).to.equal(tooltipId);
+				effectLog.push('tooltip layout effect');
+			}, [anchorRef, tooltipRef]);
+			useEffect(() => {
+				expect(anchorRef.current?.id).to.equal(anchorId);
+				expect(tooltipRef.current?.id).to.equal(tooltipId);
+				effectLog.push('tooltip effect');
+			}, [anchorRef, tooltipRef]);
+
+			return (
+				<div class="tooltip-wrapper">
+					<div id={tooltipId} ref={tooltipRef}>
+						{children}
+					</div>
+				</div>
+			);
+		}
+
+		function App() {
+			// For example, used to define what element to anchor the tooltip to
+			const anchorRef = useRef2(null);
+
+			useLayoutEffect(() => {
+				expect(anchorRef.current?.id).to.equal(anchorId);
+				effectLog.push('anchor layout effect');
+			}, [anchorRef]);
+			useEffect(() => {
+				expect(anchorRef.current?.id).to.equal(anchorId);
+				effectLog.push('anchor effect');
+			}, [anchorRef]);
+
+			return (
+				<div>
+					<p id={anchorId} ref={anchorRef}>
+						More info
+					</p>
+					<Tooltip anchorRef={anchorRef}>a tooltip</Tooltip>
+				</div>
+			);
+		}
+
+		act(() => {
+			render(<App />, scratch);
+		});
+
+		expect(effectLog).to.deep.equal([
+			'set ref P',
+			'set ref DIV',
+			'tooltip layout effect',
+			'anchor layout effect',
+			'tooltip effect',
+			'anchor effect'
+		]);
+	});
 });
diff --git a/hooks/test/browser/errorBoundary.test.js b/hooks/test/browser/errorBoundary.test.js
index 9412ef421d..0e7de2f625 100644
--- a/hooks/test/browser/errorBoundary.test.js
+++ b/hooks/test/browser/errorBoundary.test.js
@@ -1,6 +1,6 @@
 import { createElement, render } from 'preact';
 import { setupScratch, teardown } from '../../../test/_util/helpers';
-import { useErrorBoundary } from 'preact/hooks';
+import { useErrorBoundary, useLayoutEffect } from 'preact/hooks';
 import { setupRerender } from 'preact/test-utils';
 
 /** @jsx createElement */
@@ -107,4 +107,49 @@ describe('errorBoundary', () => {
 		expect(spy2).to.be.calledWith(error);
 		expect(scratch.innerHTML).to.equal('<p>Error</p>');
 	});
+
+	it('does not invoke old effects when a cleanup callback throws an error and is handled', () => {
+		let throwErr = false;
+		let thrower = sinon.spy(() => {
+			if (throwErr) {
+				throw new Error('test');
+			}
+		});
+		let badEffect = sinon.spy(() => thrower);
+		let goodEffect = sinon.spy();
+
+		function EffectThrowsError() {
+			useLayoutEffect(badEffect);
+			return <span>Test</span>;
+		}
+
+		function Child({ children }) {
+			useLayoutEffect(goodEffect);
+			return children;
+		}
+
+		function App() {
+			const [err] = useErrorBoundary();
+			return err ? (
+				<p>Error</p>
+			) : (
+				<Child>
+					<EffectThrowsError />
+				</Child>
+			);
+		}
+
+		render(<App />, scratch);
+		expect(scratch.innerHTML).to.equal('<span>Test</span>');
+		expect(badEffect).to.be.calledOnce;
+		expect(goodEffect).to.be.calledOnce;
+
+		throwErr = true;
+		render(<App />, scratch);
+		rerender();
+		expect(scratch.innerHTML).to.equal('<p>Error</p>');
+		expect(thrower).to.be.calledOnce;
+		expect(badEffect).to.be.calledOnce;
+		expect(goodEffect).to.be.calledOnce;
+	});
 });
diff --git a/hooks/test/browser/useLayoutEffect.test.js b/hooks/test/browser/useLayoutEffect.test.js
index 5a5e75eb7b..f20fa1198a 100644
--- a/hooks/test/browser/useLayoutEffect.test.js
+++ b/hooks/test/browser/useLayoutEffect.test.js
@@ -476,4 +476,41 @@ describe('useLayoutEffect', () => {
 		expect(calls.length).to.equal(1);
 		expect(calls).to.deep.equal(['doing effecthi']);
 	});
+
+	it('should run layout affects after all refs are invoked', () => {
+		const calls = [];
+		const verifyRef = name => el => {
+			calls.push(name);
+			expect(document.body.contains(el), name).to.equal(true);
+		};
+
+		const App = () => {
+			const ref = useRef();
+			useLayoutEffect(() => {
+				expect(ref.current).to.equalNode(scratch.querySelector('p'));
+
+				calls.push('doing effect');
+				return () => {
+					calls.push('cleaning up');
+				};
+			});
+
+			return (
+				<div ref={verifyRef('callback ref outer')}>
+					<p ref={ref}>
+						<span ref={verifyRef('callback ref inner')}>Hi</span>
+					</p>
+				</div>
+			);
+		};
+
+		act(() => {
+			render(<App />, scratch);
+		});
+		expect(calls).to.deep.equal([
+			'callback ref inner',
+			'callback ref outer',
+			'doing effect'
+		]);
+	});
 });
diff --git a/src/component.js b/src/component.js
index 13a69114e7..2270391120 100644
--- a/src/component.js
+++ b/src/component.js
@@ -125,7 +125,8 @@ function renderComponent(component) {
 		parentDom = component._parentDom;
 
 	if (parentDom) {
-		let commitQueue = [];
+		let commitQueue = [],
+			refQueue = [];
 		const oldVNode = assign({}, vnode);
 		oldVNode._original = vnode._original + 1;
 
@@ -138,9 +139,11 @@ function renderComponent(component) {
 			vnode._hydrating != null ? [oldDom] : null,
 			commitQueue,
 			oldDom == null ? getDomSibling(vnode) : oldDom,
-			vnode._hydrating
+			vnode._hydrating,
+			refQueue
 		);
-		commitRoot(commitQueue, vnode);
+
+		commitRoot(commitQueue, vnode, refQueue);
 
 		if (vnode._dom != oldDom) {
 			updateParentDomPointers(vnode);
diff --git a/src/diff/children.js b/src/diff/children.js
index 2a59a0cc2a..90afbaf65e 100644
--- a/src/diff/children.js
+++ b/src/diff/children.js
@@ -22,6 +22,7 @@ import { isArray } from '../util';
  * render (except when hydrating). Can be a sibling DOM element when diffing
  * Fragments that have siblings. In most cases, it starts out as `oldChildren[0]._dom`.
  * @param {boolean} isHydrating Whether or not we are in hydration
+ * @param {Array<any>} refQueue an array of elements needed to invoke refs
  */
 export function diffChildren(
 	parentDom,
@@ -33,7 +34,8 @@ export function diffChildren(
 	excessDomChildren,
 	commitQueue,
 	oldDom,
-	isHydrating
+	isHydrating,
+	refQueue
 ) {
 	let i,
 		j,
@@ -41,7 +43,6 @@ export function diffChildren(
 		childVNode,
 		newDom,
 		firstChildDom,
-		refs,
 		skew = 0;
 
 	// This is a compression of oldParentVNode!=null && oldParentVNode != EMPTY_OBJ && oldParentVNode._children || EMPTY_ARR
@@ -138,15 +139,17 @@ export function diffChildren(
 			excessDomChildren,
 			commitQueue,
 			oldDom,
-			isHydrating
+			isHydrating,
+			refQueue
 		);
 
 		newDom = childVNode._dom;
 
 		if ((j = childVNode.ref) && oldVNode.ref != j) {
-			if (!refs) refs = [];
-			if (oldVNode.ref) refs.push(oldVNode.ref, null, childVNode);
-			refs.push(j, childVNode._component || newDom, childVNode);
+			if (oldVNode.ref) {
+				applyRef(oldVNode.ref, null, childVNode);
+			}
+			refQueue.push(j, childVNode._component || newDom, childVNode);
 		}
 
 		if (newDom != null) {
@@ -243,13 +246,6 @@ export function diffChildren(
 			unmount(oldChildren[i], oldChildren[i]);
 		}
 	}
-
-	// Set refs only after unmount
-	if (refs) {
-		for (i = 0; i < refs.length; i++) {
-			applyRef(refs[i], refs[++i], refs[++i]);
-		}
-	}
 }
 
 function reorderChildren(childVNode, oldDom, parentDom) {
diff --git a/src/diff/index.js b/src/diff/index.js
index 87c0ba21e4..2537a1b35b 100644
--- a/src/diff/index.js
+++ b/src/diff/index.js
@@ -20,7 +20,8 @@ import options from '../options';
  * element any new dom elements should be placed around. Likely `null` on first
  * render (except when hydrating). Can be a sibling DOM element when diffing
  * Fragments that have siblings. In most cases, it starts out as `oldChildren[0]._dom`.
- * @param {boolean} [isHydrating] Whether or not we are in hydration
+ * @param {boolean} isHydrating Whether or not we are in hydration
+ * @param {Array<any>} refQueue an array of elements needed to invoke refs
  */
 export function diff(
 	parentDom,
@@ -31,7 +32,8 @@ export function diff(
 	excessDomChildren,
 	commitQueue,
 	oldDom,
-	isHydrating
+	isHydrating,
+	refQueue
 ) {
 	let tmp,
 		newType = newVNode.type;
@@ -239,7 +241,8 @@ export function diff(
 				excessDomChildren,
 				commitQueue,
 				oldDom,
-				isHydrating
+				isHydrating,
+				refQueue
 			);
 
 			c.base = newVNode._dom;
@@ -269,7 +272,8 @@ export function diff(
 				isSvg,
 				excessDomChildren,
 				commitQueue,
-				isHydrating
+				isHydrating,
+				refQueue
 			);
 		}
 
@@ -293,7 +297,11 @@ export function diff(
  * which have callbacks to invoke in commitRoot
  * @param {import('../internal').VNode} root
  */
-export function commitRoot(commitQueue, root) {
+export function commitRoot(commitQueue, root, refQueue) {
+	for (let i = 0; i < refQueue.length; i++) {
+		applyRef(refQueue[i], refQueue[++i], refQueue[++i]);
+	}
+
 	if (options._commit) options._commit(root, commitQueue);
 
 	commitQueue.some(c => {
@@ -323,6 +331,7 @@ export function commitRoot(commitQueue, root) {
  * @param {Array<import('../internal').Component>} commitQueue List of components
  * which have callbacks to invoke in commitRoot
  * @param {boolean} isHydrating Whether or not we are in hydration
+ * @param {Array<any>} refQueue an array of elements needed to invoke refs
  * @returns {import('../internal').PreactElement}
  */
 function diffElementNodes(
@@ -333,7 +342,8 @@ function diffElementNodes(
 	isSvg,
 	excessDomChildren,
 	commitQueue,
-	isHydrating
+	isHydrating,
+	refQueue
 ) {
 	let oldProps = oldVNode.props;
 	let newProps = newVNode.props;
@@ -445,7 +455,8 @@ function diffElementNodes(
 				excessDomChildren
 					? excessDomChildren[0]
 					: oldVNode._children && getDomSibling(oldVNode, 0),
-				isHydrating
+				isHydrating,
+				refQueue
 			);
 
 			// Remove children that are not part of any vnode.
diff --git a/src/render.js b/src/render.js
index 3f92be680e..02901b041d 100644
--- a/src/render.js
+++ b/src/render.js
@@ -33,7 +33,8 @@ export function render(vnode, parentDom, replaceNode) {
 		createElement(Fragment, null, [vnode]);
 
 	// List of effects that need to be called after diffing.
-	let commitQueue = [];
+	let commitQueue = [],
+		refQueue = [];
 	diff(
 		parentDom,
 		// Determine the new vnode tree and store it on the DOM element on
@@ -55,11 +56,12 @@ export function render(vnode, parentDom, replaceNode) {
 			: oldVNode
 			? oldVNode._dom
 			: parentDom.firstChild,
-		isHydrating
+		isHydrating,
+		refQueue
 	);
 
 	// Flush all queued effects
-	commitRoot(commitQueue, vnode);
+	commitRoot(commitQueue, vnode, refQueue);
 }
 
 /**
diff --git a/test/browser/refs.test.js b/test/browser/refs.test.js
index a5962ea500..a69b846239 100644
--- a/test/browser/refs.test.js
+++ b/test/browser/refs.test.js
@@ -1,5 +1,5 @@
 import { setupRerender } from 'preact/test-utils';
-import { createElement, render, Component, createRef } from 'preact';
+import { createElement, render, Component, createRef, Fragment } from 'preact';
 import { setupScratch, teardown } from '../_util/helpers';
 
 /** @jsx createElement */
@@ -103,8 +103,8 @@ describe('refs', () => {
 			'called with H1',
 			'called with DIV',
 			'called with null',
-			'called with H1',
 			'called with null',
+			'called with H1',
 			'called with DIV'
 		]);
 	});
@@ -543,4 +543,178 @@ describe('refs', () => {
 		expect(calls.length).to.equal(1);
 		expect(calls[0]).to.equal(scratch.firstChild.firstChild);
 	});
+
+	// Test for #4049
+	it('should first clean-up refs and after apply them', () => {
+		let calls = [];
+		let set;
+		class App extends Component {
+			constructor(props) {
+				super(props);
+				this.state = {
+					phase: 1
+				};
+				set = () => this.setState({ phase: 2 });
+			}
+
+			render(props, { phase }) {
+				return (
+					<Fragment>
+						{phase === 1 ? (
+							<div>
+								<div
+									ref={r =>
+										r
+											? calls.push('adding ref to two')
+											: calls.push('removing ref from two')
+									}
+								>
+									Element two
+								</div>
+								<div
+									ref={r =>
+										r
+											? calls.push('adding ref to three')
+											: calls.push('removing ref from three')
+									}
+								>
+									Element three
+								</div>
+							</div>
+						) : phase === 2 ? (
+							<div class="outer">
+								<div
+									ref={r =>
+										r
+											? calls.push('adding ref to one')
+											: calls.push('removing ref from one')
+									}
+								>
+									Element one
+								</div>
+								<div class="wrapper">
+									<div
+										ref={r =>
+											r
+												? calls.push('adding ref to two')
+												: calls.push('removing ref from two')
+										}
+									>
+										Element two
+									</div>
+									<div
+										ref={r =>
+											r
+												? calls.push('adding ref to three')
+												: calls.push('removing ref from three')
+										}
+									>
+										Element three
+									</div>
+								</div>
+							</div>
+						) : null}
+					</Fragment>
+				);
+			}
+		}
+
+		render(<App />, scratch);
+
+		expect(calls).to.deep.equal(['adding ref to two', 'adding ref to three']);
+		calls = [];
+
+		set();
+		rerender();
+		expect(calls).to.deep.equal([
+			'removing ref from two',
+			'adding ref to one',
+			'adding ref to two',
+			'adding ref to three'
+		]);
+	});
+
+	it('should bind refs before componentDidMount', () => {
+		/** @type {import('preact').RefObject<HTMLSpanElement>[]} */
+		const refs = [];
+
+		class Parent extends Component {
+			componentDidMount() {
+				// Child refs should be set
+				expect(refs.length).to.equal(2);
+				expect(refs[0].current.tagName).to.equal('SPAN');
+				expect(refs[1].current.tagName).to.equal('SPAN');
+			}
+
+			render(props) {
+				return props.children;
+			}
+		}
+
+		class Child extends Component {
+			constructor(props) {
+				super(props);
+
+				this.ref = createRef();
+			}
+
+			componentDidMount() {
+				// SPAN refs should be set
+				expect(this.ref.current.tagName).to.equal('SPAN');
+				expect(document.body.contains(this.ref.current)).to.equal(true);
+				refs.push(this.ref);
+			}
+
+			render() {
+				return <span ref={this.ref}>Hello</span>;
+			}
+		}
+
+		render(
+			<Parent>
+				<Child />
+				<Child />
+			</Parent>,
+			scratch
+		);
+
+		expect(scratch.innerHTML).to.equal('<span>Hello</span><span>Hello</span>');
+		expect(refs[0].current).to.equalNode(scratch.firstChild);
+		expect(refs[1].current).to.equalNode(scratch.lastChild);
+	});
+
+	it('should call refs after element is added to document on initial mount', () => {
+		const verifyRef = name => el =>
+			expect(document.body.contains(el), name).to.equal(true);
+
+		function App() {
+			return (
+				<div ref={verifyRef('div tag')}>
+					<p ref={verifyRef('p tag')}>Hello</p>
+				</div>
+			);
+		}
+
+		render(<App />, scratch);
+	});
+
+	it('should call refs after element is added to document on update', () => {
+		const verifyRef = name => el =>
+			expect(document.body.contains(el), name).to.equal(true);
+
+		function App({ show = false }) {
+			return (
+				<div>
+					{show && (
+						<p ref={verifyRef('p tag')}>
+							<span ref={verifyRef('inner span')}>Hello</span>
+						</p>
+					)}
+				</div>
+			);
+		}
+
+		render(<App />, scratch);
+		render(<App show />, scratch);
+	});
 });

From 0cdc9dfdf0c94ff30e8892514087cd765b1ae144 Mon Sep 17 00:00:00 2001
From: Jovi De Croock <decroockjovi@gmail.com>
Date: Mon, 3 Jul 2023 07:45:51 +0200
Subject: [PATCH 20/21] this was fixed by #4054 (#4058)

---
 test/browser/refs.test.js | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/test/browser/refs.test.js b/test/browser/refs.test.js
index a69b846239..7e4224af57 100644
--- a/test/browser/refs.test.js
+++ b/test/browser/refs.test.js
@@ -509,25 +509,23 @@ describe('refs', () => {
 		expect(ref.current).to.equal(scratch.firstChild.firstChild);
 	});
 
-	// TODO
-	it.skip('should properly call null for memoized components keyed', () => {
-		const calls = [];
+	it('should properly call null for memoized components keyed', () => {
+		let calls = [];
 		const element = <div ref={x => calls.push(x)}>hey</div>;
 		function App(props) {
 			return <div key={props.count}>{element}</div>;
 		}
 
 		render(<App count={0} />, scratch);
+		expect(calls).to.deep.equal([scratch.firstChild.firstChild]);
+		calls = [];
+
 		render(<App count={1} />, scratch);
+		expect(calls).to.deep.equal([null, scratch.firstChild.firstChild]);
+		calls = [];
+
 		render(<App count={2} />, scratch);
-		expect(calls.length).to.equal(5);
-		expect(calls).to.deep.equal([
-			scratch.firstChild.firstChild,
-			null,
-			scratch.firstChild.firstChild,
-			null,
-			scratch.firstChild.firstChild
-		]);
+		expect(calls).to.deep.equal([null, scratch.firstChild.firstChild]);
 	});
 
 	it('should properly call null for memoized components unkeyed', () => {

From 051f10c59d14229520f14a531a4de79162e18c02 Mon Sep 17 00:00:00 2001
From: Jovi De Croock <decroockjovi@gmail.com>
Date: Sun, 9 Jul 2023 09:57:45 +0200
Subject: [PATCH 21/21] 10.16.0 (#4063)

---
 devtools/src/devtools.js | 2 +-
 package-lock.json        | 4 ++--
 package.json             | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/devtools/src/devtools.js b/devtools/src/devtools.js
index c4470482b6..670d45b6df 100644
--- a/devtools/src/devtools.js
+++ b/devtools/src/devtools.js
@@ -2,7 +2,7 @@ import { options, Fragment, Component } from 'preact';
 
 export function initDevTools() {
 	if (typeof window != 'undefined' && window.__PREACT_DEVTOOLS__) {
-		window.__PREACT_DEVTOOLS__.attachPreact('10.15.1', options, {
+		window.__PREACT_DEVTOOLS__.attachPreact('10.16.0', options, {
 			Fragment,
 			Component
 		});
diff --git a/package-lock.json b/package-lock.json
index 46fcee73cb..7876cfe36f 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "preact",
-  "version": "10.15.1",
+  "version": "10.16.0",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "preact",
-      "version": "10.15.1",
+      "version": "10.16.0",
       "license": "MIT",
       "devDependencies": {
         "@actions/github": "^5.0.0",
diff --git a/package.json b/package.json
index 5d435f6f9d..346e6d4c1b 100644
--- a/package.json
+++ b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "preact",
   "amdName": "preact",
-  "version": "10.15.1",
+  "version": "10.16.0",
   "private": false,
   "description": "Fast 3kb React-compatible Virtual DOM library.",
   "main": "dist/preact.js",