@fluixi/core 1.0.0-alpha.82 → 1.0.0-alpha.84

package/dist/src/lib/render/index.js

@@ -1,5 +1,4 @@
 /**
- * @fileoverview Client-Side Rendering for Fluixi
  * @module @fluixi/core/render
  *
  * Client-side rendering:
@@ -8,27 +7,22 @@
  *
  * Following SolidJS structure: render logic separated from component creation
  */
+import { isServer } from '@fluixi/dom';
 import { createRoot } from '@fluixi/reactive/signal';
 import { stampVersions, warnOnVersionSkew } from './versions.js';
 import { isDOMNode, delegateEvents, insert, } from '@fluixi/dom';
-const isServer = typeof window === 'undefined';
 // ============================================================================
-// SECTION 7 — RENDERING (CLIENT)
+// RENDERING (CLIENT)
 // ============================================================================
 /**
  * Render a component into a DOM element
  *
- * Section 7 — CLIENT:
+ * On the client:
  * - Real DOM (no Virtual DOM)
  * - Fine-grained mutation (no reconciliation)
  *
- * Section 1: Component executes once during render
- * Section 5: Creates reactive owner (root)
- *
- * @param {() => any} code - Component or function to render
- * @param {HTMLElement} container - Target DOM element
- * @param {RenderOptions} options - Render options
- * @returns {() => void} Dispose function
+ * Component executes once during render
+ * Creates reactive owner (root)
  *
  * @example
  * ```typescript
@@ -45,7 +39,7 @@ const isServer = typeof window === 'undefined';
  */
 export function render(code, container, options = {}) {
     delegateEvents(['click', 'input', 'change', 'submit']);
-    if (isServer) {
+    if (isServer()) {
         throw new Error('render() is for client-side only. Use renderToString() on server.');
     }
     if (!container) {
@@ -55,11 +49,11 @@ export function render(code, container, options = {}) {
     // domRender(code, container)
     createRoot((dispose) => {
         disposer = dispose;
-        // Section 1: Execute code function once — Providers run here with Owner = root
+        // Execute code function once. Providers run here with Owner = root
         // so "also stamp" in createProvider sets context on root.contexts, making
         // it reachable from every render effect created by insert() below.
         const result = typeof code === 'function' ? code() : code;
-        // Section 7: Insert into real DOM using the SAME root as code() so that
+        // Insert into real DOM using the same root as code() so that
         // render effects share the Owner chain with Provider context nodes.
         // Using domRender() would create a second createRoot, severing the chain.
         container.textContent = '';
@@ -68,8 +62,8 @@ export function render(code, container, options = {}) {
         stampVersions(container);
         warnOnVersionSkew();
         // Wrap the (already-computed) result in a STABLE thunk so insert() creates a
-        // top-level render effect. Without this, a static array result — e.g. the
-        // imperative backend's `[createMemo(() => createComponent(App)), " "]` — is
+        // top-level render effect. Without this, a static array result: e.g. the
+        // imperative backend's `[createMemo(() => createComponent(App)), " "]`: is
         // flattened ONCE with no enclosing effect, so the nested memos are read
         // untracked and reactive boundaries deep in the tree (Suspense fallback ->
         // content, Outlet, Show/For) never update. The thunk returns the same result
@@ -99,11 +93,6 @@ function isLitTemplateResult(value) {
 /**
  * Render into a different part of the DOM tree
  *
- * @param {() => any} code - Component to render
- * @param {HTMLElement} target - Target element (outside current tree)
- * @param {RenderOptions} options - Render options
- * @returns {() => void} Dispose function
- *
  * @example
  * ```typescript
  * import { portal } from '@fluixi/core';
@@ -133,9 +122,6 @@ export function portal(code, target, options = {}) {
 // ============================================================================
 /**
  * Check if a value can be rendered
- *
- * @param {any} value - Value to check
- * @returns {boolean} True if renderable
  */
 export function isRenderable(value) {
     if (value == null)
@@ -168,7 +154,7 @@ export function isRenderable(value) {
  * ```
  */
 export function initDelegation() {
-    if (isServer)
+    if (isServer())
         return;
     delegateEvents(['click', 'input', 'change', 'submit']);
 }
@@ -177,7 +163,7 @@ export function initDelegation() {
  * Should be called when app unmounts
  */
 export function cleanupDelegation() {
-    if (isServer)
+    if (isServer())
         return;
     // Clear delegated events manually if needed
 }