fix(core): enhance MeoNodeUnmounter cleanup logic and support additional props cloning

- Refactor MeoNodeUnmounter to use useEffectEvent for stable cleanup on unmount
- Cleanup removes node from BaseNode.elementCache, untracks mount via MountTrackerUtil, unregisters from BaseNode.cacheCleanupRegistry, and clears lastSignature to prevent memory leaks
- Support cloning and forwarding additional props to valid React children elements

Author: l7aromeo

src/components/meonode-unmounter.client.ts

@@ -1,25 +1,27 @@
 'use client'
-import { type ReactNode, useEffect, useEffectEvent } from 'react'
+import { cloneElement, isValidElement, type ReactNode, useEffect, useEffectEvent } from 'react'
 import { MountTrackerUtil } from '@src/util/mount-tracker.util.js'
 import { BaseNode } from '@src/core.node.js'
 import type { NodeInstance } from '@src/types/node.type.js'
 
 /**
- * `MeoNodeUnmounter` is a client-side React component responsible for cleaning up
- * resources associated with a rendered node when it unmounts.
+ * `MeoNodeUnmounter` is a client-side React component responsible for performing cleanup
+ * operations when a `MeoNode` instance is unmounted from the React tree.
  *
- * It uses a `useEffect` hook to register a cleanup function that runs when the component
- * unmounts or when its `stableKey` changes. The cleanup function checks if the node
- * identified by `stableKey` is currently tracked as mounted. If it is, it removes
- * the node from `BaseNode.elementCache` and untracks its mount status using `MountTrackerUtil`.
- * Additionally, it clears the `lastPropsRef` and `lastSignature` of the associated `BaseNode`
- * instance to prevent memory leaks from retained prop objects.
+ * It leverages `useEffectEvent` to create a stable cleanup function that is called
+ * when the component unmounts. This cleanup function performs the following actions:
+ * - Deletes the node from `BaseNode.elementCache` using its `stableKey`.
+ * - Untracks the node's mount status via `MountTrackerUtil.untrackMount`.
+ * - Unregisters the node from `BaseNode.cacheCleanupRegistry` to prevent redundant
+ *   finalization callbacks.
+ * - Clears the `lastSignature` of the associated `BaseNode` instance to help prevent
+ *   memory leaks from retained prop objects.
  * @param {object} props The component's props.
  * @param {NodeInstance} props.node The BaseNode instance associated with this component.
  * @param {ReactNode} [props.children] The children to be rendered by this component.
  * @returns {ReactNode} The `children` passed to the component.
  */
-export default function MeoNodeUnmounter({ node, children }: { node: NodeInstance; children?: ReactNode }): ReactNode {
+export default function MeoNodeUnmounter({ node, children, ...rest }: { node: NodeInstance; children?: ReactNode }): ReactNode {
   const onUnmount = useEffectEvent(() => {
     if (node.stableKey) {
       BaseNode.elementCache.delete(node.stableKey)
@@ -40,5 +42,9 @@ export default function MeoNodeUnmounter({ node, children }: { node: NodeInstanc
     return () => onUnmount()
   }, [])
 
+  if (isValidElement(children)) {
+    return cloneElement(children, rest)
+  }
+
   return children
 }