lilact 0.27.0 → 0.27.2

package/src/hooks.jsx

@@ -87,8 +87,12 @@ export function useState(initialValue)
  */
 export function useCallback(callback, deps=undefined)
 {
-	if( deps!==undefined && (typeof(deps)!=='object' || deps.constructor.name!=='Array') ) {
-		throw new Error("Callback dependencies must be an array or omitted.");
+	if(
+			deps !== undefined && !Array.isArray(deps) &&
+			deps !== null && typeof deps !== "object"
+		) 
+	{
+		throw new Error("Callback dependencies must be an array, object  or omitted.");
 	}
 
 	const hk = useHook();
@@ -133,22 +137,17 @@ export function createContext(defaultValue)
  * @param {any} context - Context object created by `createContext`.
  * @returns {any} Current context value.
  */
-export function useContext(context)
-{
-	let core = Lilact.current_component[0].parent;
+export function useContext(context) {
+	let core = Lilact.current_component[0]?.parent;
 
-	while(core.entity!==context.Provider && core.parent) {
+	while (core && core.entity !== context.Provider) {
 		core = core.parent;
 	}
 
-	if(core.parent) {
-		let v = core.props?.value;
-		return v??=context.default;
-	}
-
-	return context.default;
+	return core
+		? core.props?.value ?? context.default
+		: context.default;
 }
-
 /**
  * Returns a stable, unique ID that is consistent across renders.
  *
@@ -178,26 +177,24 @@ export function useTransition()
 	if( isEmpty(hk) ) {
 		hk.count=0;
 
-		hk.func =
-
-		(async function(core, hk, fn) {
-
-			if(hk.count===0) {
+		hk.func = (async function(core, hk, fn) {
+			if (hk.count === 0) {
 				core.component.forceUpdate();
 			}
 
 			hk.count++;
 
-			await fn();
+			try {
+				return await fn();
+			} finally {
+				hk.count--;
 
-			hk.count--;
-
-			if(hk.count===0) {
-				core.component.forceUpdate();
+				if (hk.count === 0) {
+					core.component.forceUpdate();
+				}
 			}
-		}
+		}).bind(undefined, Lilact.current_component[0], hk);
 
-		).bind(undefined, Lilact.current_component[0], hk);
 	}
 
 	return [ hk.count!=0, hk.func ];
@@ -275,7 +272,11 @@ export function useRef(initialValue = null)
  */
 export async function useLayoutEffect(effect, deps=undefined)
 {
-	if( deps!==undefined && (typeof(deps)!=='object' && deps.constructor.name!=='Array') ) {
+	if(
+			deps !== undefined && !Array.isArray(deps) &&
+			deps !== null && typeof deps !== "object"
+		) 
+	{
 		throw new Error("Layout effect dependencies must be an array, object or omitted.");
 	}
 
@@ -287,8 +288,8 @@ export async function useLayoutEffect(effect, deps=undefined)
 
 	if(hk?.cleanup) {
 		await hk.cleanup();
-  		hk.cleanup = undefined;
-  	}
+			hk.cleanup = undefined;
+		}
 
 	hk.deps = deps;
 	Lilact.layout_effects.add( async ()=>{ hk.cleanup = await effect(); });
@@ -307,7 +308,11 @@ export async function useLayoutEffect(effect, deps=undefined)
  */
 export async function useEffect(effect, deps=undefined)
 {
-	if( deps!==undefined && (typeof(deps)!=='object' && deps.constructor.name!=='Array') ) {
+	if(
+			deps !== undefined && !Array.isArray(deps) &&
+			deps !== null && typeof deps !== "object"
+		) 
+	{
 		throw new Error("Effect dependencies must be an array, object or omitted.");
 	}
 
@@ -319,7 +324,7 @@ export async function useEffect(effect, deps=undefined)
 
 	if(hk?.cleanup) {
 		await hk.cleanup();
-  		hk.cleanup = undefined;
+			hk.cleanup = undefined;
 	}
 
 	hk.deps = deps;
@@ -340,7 +345,11 @@ export async function useEffect(effect, deps=undefined)
  */
 export async function useInsertionEffect(effect, deps=undefined)
 {
-	if( deps!==undefined && (typeof(deps)!=='object' && deps.constructor.name!=='Array') ) {
+	if(
+			deps !== undefined && !Array.isArray(deps) &&
+			deps !== null && typeof deps !== "object"
+		) 
+	{
 		throw new Error("Insertion effect dependencies must be an array, object, or omitted.");
 	}
 
@@ -352,7 +361,7 @@ export async function useInsertionEffect(effect, deps=undefined)
 
 	if(hk?.cleanup) {
 		await hk.cleanup();
-  		hk.cleanup = undefined;
+			hk.cleanup = undefined;
 	}
 
 	hk.deps = deps;
@@ -373,7 +382,11 @@ export async function useInsertionEffect(effect, deps=undefined)
  */
 export function useMemo(factory,deps=undefined)
 {
-	if( deps!==undefined && (typeof(deps)!=='object' && deps.constructor.name!=='Array') ) {
+	if(
+			deps !== undefined && !Array.isArray(deps) &&
+			deps !== null && typeof deps !== "object"
+		) 
+	{
 		throw new Error("Memo dependencies must be an array or omitted.");
 	}
 
@@ -407,12 +420,12 @@ export function useActionState(action, initialState)
 
 		hk.state = initialState;
 
-		hk.form_action = (sub)=>{
+		hk.form_action = (event)=>{
 			event.preventDefault();
 
 			tran_start_func(
 					async ()=> {
-						const form_data = new FormData(sub.target, sub.submitter);
+						const form_data = new FormData(event.target, event.submitter);
 						hk.state = await action(hk.state, form_data);
 					},
 					[]
@@ -433,27 +446,30 @@ export function useActionState(action, initialState)
  * @param {Function} [init] - Optional initializer function for lazy initial state.
  * @returns {any} Hook result `[state, dispatch]`.
  */
-export function useReducer(reducer, initialArg, init)
-{
+export function useReducer(reducer, initialArg, init) {
 	const hk = useHook();
 
-	if( isEmpty(hk) ) {
-		hk.reducer = reducer;
-		hk.state = init?init(initialArg):initialArg;
+	if(isEmpty(hk)) {
+		hk.state = init ? init(initialArg) : initialArg;
+
 		hk.dispatch = function(core, hk, action) {
-			const newst = hk.reducer(hk.state, action);
-			if(!Lilact.defaultIsEqual(newst,hk.state)) {
-				hk.state = newst;
+			const newState = hk.reducer(hk.state, action);
+
+			if (!Lilact.defaultIsEqual(newState, hk.state)) {
+				hk.state = newState;
 				core.component.forceUpdate();
 			}
 		}.bind(undefined, Lilact.current_component[0], hk);
 	}
 
+	hk.reducer = reducer;
+
 	return [hk.state, hk.dispatch];
 }
 
 
 
+
 // todo: very simple implementation. and not lilactish at all! must be modified
 
 /**
@@ -526,7 +542,7 @@ export function useImperativeHandle(ref, factory, deps=undefined)
 	if(typeof ref?.current !== 'object') {
 		ref.current = {};
 	}
-	Object.assign( ref.current, factory(), 0 );	
+	Object.assign( ref.current, factory());	
 
 }
 

package/src/lilact.jsx

@@ -143,8 +143,10 @@ globalThis.createComponent = Lilact.createComponent;
 globalThis.Fragment = Lilact.Fragment;
 globalThis.require = Lilact.require;
 
-document.addEventListener('DOMContentLoaded', () => {
-  Lilact.runScripts();
+document.addEventListener("DOMContentLoaded", () => {
+  Lilact.runScripts().catch(error => {
+    Lilact.globalErrorHandler?.(error);
+  });
 });
 
 if(DEBUG) {

package/src/misc.jsx

@@ -54,9 +54,12 @@ const typeOf = (input) => {
  * @param value - Value to inspect.
  * @returns True if the value is a class component; otherwise false.
  */
-export const isValidComponent = (value) => {
-	return value[CORE]!==undefined || value[TEXT]!==undefined;
-}
+export const isValidComponent = value => {
+  return Boolean(
+    value &&
+    (value[CORE] !== undefined || value[TEXT] !== undefined)
+  );
+};
 
 /**
  * Checks whether a value is a Lilact component. It is the same as `isValidComponent`.
@@ -359,23 +362,48 @@ export function isEmpty(value)  {
  * @returns True if shallowly equal; otherwise false.
  */
 export const shallowEqual = (source, target, ignore) => {
-	if (typeOf(source) !== typeOf(target)) {
-		return false;
-	}
+  if (Object.is(source, target)) {
+    return true;
+  }
 
-	if (typeOf(source) === "array") {
-		if (source.length !== target.length) {
-			return false;
-		}
-		return source.every((el, index) => el === target[index] || index===ignore );
-	} else if (typeOf(source) === "object") {
-		return Object.keys(source).every((key) => source[key] === target[key] || key===ignore );
-	} else if (typeOf(source) === "date") {
-		return source.getTime() === target.getTime();
-	}
+  if (typeOf(source) !== typeOf(target)) {
+    return false;
+  }
 
-	return source === target;
-}
+  if (typeOf(source) === "array") {
+    if (source.length !== target.length) {
+      return false;
+    }
+
+    return source.every(
+      (value, index) =>
+        index === ignore || Object.is(value, target[index])
+    );
+  }
+
+  if (typeOf(source) === "object") {
+    const sourceKeys = Object.keys(source)
+      .filter(key => key !== ignore);
+
+    const targetKeys = Object.keys(target)
+      .filter(key => key !== ignore);
+
+    if (sourceKeys.length !== targetKeys.length) {
+      return false;
+    }
+
+    return sourceKeys.every(key =>
+      Object.prototype.hasOwnProperty.call(target, key) &&
+      Object.is(source[key], target[key])
+    );
+  }
+
+  if (typeOf(source) === "date") {
+    return source.getTime() === target.getTime();
+  }
+
+  return Object.is(source, target);
+};
 
 
 /**