sygnal 5.1.4 → 5.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sygnal",
-  "version": "5.1.4",
+  "version": "5.2.0",
   "description": "An intuitive framework for building fast and small components or applications based on Cycle.js",
   "main": "./dist/index.cjs.js",
   "types": "./dist/index.d.ts",

package/src/component.ts

@@ -26,6 +26,7 @@ const ENVIRONMENT: any =
 const BOOTSTRAP_ACTION = 'BOOTSTRAP';
 const INITIALIZE_ACTION = 'INITIALIZE';
 const HYDRATE_ACTION = 'HYDRATE';
+const DISPOSE_ACTION = 'DISPOSE';
 const PARENT_SINK_NAME = 'PARENT';
 const CHILD_SOURCE_NAME = 'CHILD';
 const READY_SINK_NAME = 'READY';
@@ -446,8 +447,14 @@ class Component {
   }
 
   dispose(): void {
-    // Signal disposal to the component via dispose$ stream
-    // This fires FIRST so CLEANUP actions in the model can process
+    // Fire the DISPOSE built-in action so model handlers can run cleanup logic
+    const hasDispose = this.model && (this.model[DISPOSE_ACTION] || Object.keys(this.model).some(k => k.includes('|') && k.split('|')[0].trim() === DISPOSE_ACTION))
+    if (hasDispose && this.action$ && typeof this.action$.shamefullySendNext === 'function') {
+      try {
+        this.action$.shamefullySendNext({ type: DISPOSE_ACTION })
+      } catch (_) {}
+    }
+    // Signal disposal to the component via dispose$ stream (for advanced use cases)
     if (this._disposeListener) {
       try {
         this._disposeListener.next(true)
@@ -455,7 +462,7 @@ class Component {
       } catch (_) {}
       this._disposeListener = null
     }
-    // Tear down streams on next microtask to allow CLEANUP actions to process
+    // Tear down streams on next microtask to allow DISPOSE/cleanup actions to process
     setTimeout(() => {
       // Complete the action$ stream to stop the entire component cycle
       if (this.action$ && typeof this.action$.shamefullySendComplete === 'function') {
@@ -1598,7 +1605,7 @@ class Component {
         if (vdom$.length === 0) return xs.of(root)
 
         // Track READY state on the component instance (persists across folds)
-        for (const [id, val] of entries) {
+        for (const [id, val] of entries as [string, any]) {
           if (this._childReadyState[id] !== undefined) continue // already tracking
           const readySink = val.sink$[READY_SINK_NAME]
           if (readySink) {
@@ -2117,11 +2124,11 @@ function __sortFunctionFromObj(item: Record<string, any>): ((a: any, b: any) =>
   }
   let ascending = true
   if (typeof directionRaw === 'string') {
-    if (!['asc', 'dec'].includes(directionRaw.toLowerCase())) {
-      console.error('Sort object string values must be asc or dec:', item)
+    if (!['asc', 'desc'].includes(directionRaw.toLowerCase())) {
+      console.error('Sort object string values must be asc or desc:', item)
       return undefined
     }
-    ascending = directionRaw.toLowerCase() === 'asc'
+    ascending = directionRaw.toLowerCase() !== 'desc'
   }
   if (typeof directionRaw === 'number') {
     if (directionRaw !== 1 && directionRaw !== -1) {
@@ -2139,9 +2146,9 @@ function sortFunctionFromProp(sortProp: any): ((a: any, b: any) => number) | und
   // if function do nothing
   if (propType === 'function') return sortProp
   if (propType === 'string') {
-    // if passed either 'asc' or 'dec' sort on the entire item
-    if (sortProp.toLowerCase() === 'asc' || sortProp.toLowerCase() === 'dec') {
-      const ascending = sortProp.toLowerCase() === 'asc'
+    // if passed either 'asc' or 'desc' sort on the entire item
+    if (sortProp.toLowerCase() === 'asc' || sortProp.toLowerCase() === 'desc') {
+      const ascending = sortProp.toLowerCase() !== 'desc'
       return (a, b) => __baseSort(a, b, ascending)
     }
     // assume it's a field/property name, and sort it ascending
@@ -2150,7 +2157,7 @@ function sortFunctionFromProp(sortProp: any): ((a: any, b: any) => number) | und
   } else if (Array.isArray(sortProp)) {
     const sorters = sortProp.map(item => {
       if (typeof item === 'function') return item
-      if (typeof item === 'string' && !['asc', 'dec'].includes(item.toLowerCase())) return (a: any, b: any) => __baseSort(a[item], b[item], true)
+      if (typeof item === 'string' && !['asc', 'desc'].includes(item.toLowerCase())) return (a: any, b: any) => __baseSort(a[item], b[item], true)
       if (isObj(item)) {
         return __sortFunctionFromObj(item)
       }

package/src/extra/run.ts

@@ -118,6 +118,10 @@ export default function run(
       (sources as any).STATE.stream.removeListener(persistListener);
       persistListener = null;
     }
+    // Trigger the component's dispose() which fires the DISPOSE action and dispose$ stream
+    if (typeof (sinks as any).__dispose === 'function') {
+      try { (sinks as any).__dispose(); } catch (_) {}
+    }
     rawDispose();
   };
 

package/src/extra/testing.ts

@@ -249,6 +249,10 @@ export function renderComponent(
       } catch (_) {}
       stateListener = null;
     }
+    // Trigger the component's dispose() which fires the DISPOSE action and dispose$ stream
+    if (typeof (sinks as any).__dispose === 'function') {
+      try { (sinks as any).__dispose(); } catch (_) {}
+    }
     rawDispose();
   };
 

package/src/index.d.ts

@@ -117,6 +117,7 @@ type WithDefaultActions<STATE, ACTIONS> = ACTIONS & {
   BOOTSTRAP?: never;
   INITIALIZE?: STATE;
   HYDRATE?: any;
+  DISPOSE?: never;
 }
 
 type ComponentModel<STATE, PROPS, DRIVERS, ACTIONS, CALCULATED, SINK_RETURNS extends NonStateSinkReturns = {}> = keyof ACTIONS extends never
@@ -239,7 +240,7 @@ export type Filter<ITEM = any> = (item: ITEM) => boolean
 export type SortFunction<ITEM = any> = (a: ITEM, b: ITEM) => number
 
 export type SortObject<ITEM = any> = {
-  [field: string]: 'asc' | 'dec' | SortFunction<ITEM>
+  [field: string]: 'asc' | 'desc' | SortFunction<ITEM>
 }
 
 /**