skipEffects flag value does not return to original value

[mo36924]

Throw a Promise in a function component with the Suspense.
SkipEffects flag value does not return to original value.

As an example, the following code causes an error.

import { equal } from "node:assert";
import { options } from "preact";
import { createElement, lazy, Suspense } from "preact/compat";
import prepass from "preact-ssr-prepass";

const skipEffects = "__s";
const Component = lazy(() => Promise.resolve({ default: () => null }));
const App = () => createElement(Suspense, { fallback: null }, createElement(Component, {}));
equal(options[skipEffects], undefined);

prepass(createElement(App, {})).then(() => {
  // This assertion fails because the `skipEffects` option is set to `true` in the `preact-ssr-prepass` module.
  equal(options[skipEffects], undefined);
});

[mo36924]

preact-ssr-prepass/src/index.js

Lines 88 to 109 in 2f63c40

	try {
	const previousSkipEffects = options[_skipEffects];
	options[_skipEffects] = true;
	// options.render was renamed to _render (mangled to __r)
	if (options.render) options.render(vnode);
	if (options.__r) {
	options.__r(vnode);
	}
	const renderResult = Promise.resolve(
	nodeName.call(vnode.__c, props, cctx)
	);
	options[_skipEffects] = previousSkipEffects;
	return renderResult;
	} catch (e) {
	if (e && e.then) {
	return e.then(doRender, doRender);
	}
	return Promise.reject(e);
	}

It might be better to return the value of the flag when an error is thrown.

const previousSkipEffects = options[_skipEffects]; 
options[_skipEffects] = true;
try {
  ...
} catch(e) {
  ...
} finally {
  options[_skipEffects] = previousSkipEffects; 
}