selective-ui 1.4.0 → 1.4.2

package/src/ts/core/base/adapter.ts

@@ -62,9 +62,12 @@ import { LifecycleState } from "src/ts/types/core/base/lifecycle.type";
  * @see {@link ModelContract}
  */
 export class Adapter<
-    TItem extends ModelContract<any, any> & { view: TViewer | null; isInit: boolean },
-    TViewer extends ViewContract<any>
-> extends Lifecycle implements AdapterContract<TItem> {
+    TItem extends ModelContract<any, any> & { view?: TViewer; isInit: boolean },
+    TViewer extends ViewContract<any>,
+>
+    extends Lifecycle
+    implements AdapterContract<TItem>
+{
     /**
      * Current list of items managed by the adapter.
      *
@@ -92,6 +95,17 @@ export class Adapter<
      */
     recyclerView: any;
 
+    /**
+     * Tracks all scheduler keys registered by this adapter instance via
+     * {@link onPropChanging} and {@link onPropChanged}.
+     *
+     * Used during {@link destroy} to clean up all associated pipelines
+     * from the global {@link Libs.callbackScheduler}.
+     *
+     * Keys are deduplicated automatically by Set semantics.
+     */
+    callbackSchedulerList: Set<string> = new Set();
+
     /**
      * Creates an adapter with an optional initial item list and initializes its lifecycle.
      *
@@ -118,7 +132,11 @@ export class Adapter<
      * @param {number} position - Index of the item within the adapter item list.
      * @returns {void}
      */
-    public onViewHolder(item: TItem, viewer: TViewer | null, position: number): void {
+    public onViewHolder(
+        item: TItem,
+        viewer?: TViewer,
+        position?: number,
+    ): void {
         void position;
 
         const v = viewer;
@@ -142,8 +160,17 @@ export class Adapter<
      * @returns {void}
      * @see {@link changingProp}
      */
-    public onPropChanging(propName: string, callback: (...args: unknown[]) => void): void {
-        Libs.callbackScheduler.on(`${propName}ing_${this.adapterKey}`, callback, { debounce: 0 });
+    public onPropChanging(
+        propName: string,
+        callback: (...args: unknown[]) => void,
+    ): void {
+        const key = `${propName}ing_${this.adapterKey}`;
+        Libs.callbackScheduler.on(
+            key,
+            callback,
+            { debounce: 0 },
+        );
+        this.callbackSchedulerList.add(key);
     }
 
     /**
@@ -159,8 +186,15 @@ export class Adapter<
      * @returns {void}
      * @see {@link changeProp}
      */
-    public onPropChanged(propName: string, callback: (...args: unknown[]) => void): void {
-        Libs.callbackScheduler.on(`${propName}_${this.adapterKey}`, callback, { debounce: 0 });
+    public onPropChanged(
+        propName: string,
+        callback: (...args: unknown[]) => void,
+    ): void {
+        const key = `${propName}_${this.adapterKey}`;
+        Libs.callbackScheduler.on(key, callback, {
+            debounce: 0,
+        });
+        this.callbackSchedulerList.add(key);
     }
 
     /**
@@ -174,7 +208,10 @@ export class Adapter<
      * @returns {Promise<void>} Resolves when scheduled callbacks complete.
      */
     public changeProp(propName: string, ...params: unknown[]): Promise<void> {
-        return Libs.callbackScheduler.run(`${propName}_${this.adapterKey}`, ...params);
+        return Libs.callbackScheduler.run(
+            `${propName}_${this.adapterKey}`,
+            ...params,
+        );
     }
 
     /**
@@ -188,7 +225,10 @@ export class Adapter<
      * @returns {Promise<void>} Resolves when scheduled callbacks complete.
      */
     public changingProp(propName: string, ...params: unknown[]): Promise<void> {
-        return Libs.callbackScheduler.run(`${propName}ing_${this.adapterKey}`, ...params);
+        return Libs.callbackScheduler.run(
+            `${propName}ing_${this.adapterKey}`,
+            ...params,
+        );
     }
 
     /**
@@ -198,9 +238,9 @@ export class Adapter<
      *
      * @param {HTMLElement} parent - Container element that will host the viewer.
      * @param {TItem} item - The model for which the viewer is created.
-     * @returns {TViewer | null} The created viewer instance; `null` by default.
+     * @returns {TViewer} The created viewer instance; `null` by default.
      */
-    public viewHolder(parent: HTMLElement, item: TItem): TViewer | null {
+    public viewHolder?(parent: HTMLElement, item: TItem): TViewer {
         void parent;
         void item;
         return null;
@@ -316,10 +356,17 @@ export class Adapter<
             return;
         }
 
+        this.callbackSchedulerList.forEach((key) => {
+            Libs.callbackScheduler.off(key);
+        });
+
+        this.callbackSchedulerList.clear();
         this.recyclerView = null;
-        this.items.forEach(item => {
+        this.items.forEach((item) => {
             item?.destroy?.();
         });
         this.items = [];
+
+        super.destroy();
     }
-}
+}

package/src/ts/core/base/fenwick.ts

@@ -131,7 +131,8 @@ export class Fenwick extends Lifecycle {
      * Returns 0 if the first element already exceeds `target`.
      */
     public lowerBoundPrefix(target: number): number {
-        let idx = 0, bitMask = 1;
+        let idx = 0,
+            bitMask = 1;
         while (bitMask << 1 <= this.stackNum) bitMask <<= 1;
 
         let cur = 0;
@@ -144,4 +145,4 @@ export class Fenwick extends Lifecycle {
         }
         return idx;
     }
-}
+}

package/src/ts/core/base/lifecycle.ts

@@ -1,4 +1,8 @@
-import { LifecycleHookContext, LifecycleHooks, LifecycleState } from "src/ts/types/core/base/lifecycle.type";
+import {
+    LifecycleHookContext,
+    LifecycleHooks,
+    LifecycleState,
+} from "src/ts/types/core/base/lifecycle.type";
 
 type LifecycleHookName = keyof LifecycleHooks;
 
@@ -59,7 +63,10 @@ export class Lifecycle {
      * This map is initialized with keys for all supported hooks in the constructor.
      * Callbacks are cleared on {@link destroy}.
      */
-    private hooks: Map<LifecycleHookName, Set<(ctx: LifecycleHookContext) => void>> = new Map();
+    private hooks: Map<
+        LifecycleHookName,
+        Set<(ctx: LifecycleHookContext) => void>
+    > = new Map();
 
     /**
      * Constructs the lifecycle manager and pre-registers hook containers.
@@ -98,7 +105,10 @@ export class Lifecycle {
      * @param {(ctx: LifecycleHookContext) => void} fn - Callback to remove.
      * @returns {this} The current instance (chainable).
      */
-    off(hook: LifecycleHookName, fn: (ctx: LifecycleHookContext) => void): this {
+    off(
+        hook: LifecycleHookName,
+        fn: (ctx: LifecycleHookContext) => void,
+    ): this {
         this.hooks.get(hook)!.delete(fn);
         return this;
     }
@@ -255,4 +265,4 @@ export class Lifecycle {
             set.clear();
         }
     }
-}
+}

package/src/ts/core/base/model.ts

@@ -43,16 +43,18 @@ export class Model<
     TTarget extends HTMLElement,
     TTags extends Record<string, HTMLElement>,
     TView extends ViewContract<TTags>,
-    TOptions = unknown
-> extends Lifecycle implements ModelContract<TTarget, TView> {
-
+    TOptions = unknown,
+>
+    extends Lifecycle
+    implements ModelContract<TTarget, TView>
+{
     /**
      * The currently bound target DOM element.
      *
      * This element typically represents the source-of-truth node in the host DOM (e.g., a native `<option>`).
      * May be replaced via {@link updateTarget} during reconciliation.
      */
-    public targetElement: TTarget | null = null;
+    public targetElement?: TTarget;
 
     /**
      * Configuration options supplied at construction time.
@@ -66,25 +68,25 @@ export class Model<
      * Ownership: this model will destroy the view on {@link destroy}.
      * The view may be attached/assigned by external orchestrators (Adapter/RecyclerView) after construction.
      */
-    public view: TView | null = null;
+    public view?: TView;
 
     /**
      * Position index used by list infrastructure for ordering/tracking.
      * Semantics are library-specific (e.g., top-level index or adapter position).
      */
-    public position = -1;
+    public position: number = -1;
 
     /**
      * Indicates whether this model has completed its initial binding step.
      * Typically set by the adapter/view binding layer to prevent duplicate listener wiring.
      */
-    public isInit = false;
+    public isInit: boolean = false;
 
     /**
      * Indicates whether this model has been removed/destroyed from the active dataset.
      * Set to `true` during {@link destroy}.
      */
-    public isRemoved = false;
+    public isRemoved: boolean = false;
 
     /**
      * Returns the current "value" associated with the bound target element.
@@ -107,13 +109,13 @@ export class Model<
      * - Calls {@link Lifecycle.init} immediately (`NEW → INITIALIZED`).
      *
      * @param {TOptions} options - Configuration options for the model.
-     * @param {TTarget | null} [targetElement=null] - Optional DOM element to bind.
-     * @param {TView | null} [view=null] - Optional view responsible for rendering this model.
+     * @param {TTarget} [targetElement=null] - Optional DOM element to bind.
+     * @param {TView} [view=null] - Optional view responsible for rendering this model.
      */
     public constructor(
         options: TOptions,
-        targetElement: TTarget | null = null,
-        view: TView | null = null
+        targetElement?: TTarget,
+        view?: TView,
     ) {
         super();
         this.options = options;
@@ -134,10 +136,10 @@ export class Model<
      * - Assigns {@link targetElement}.
      * - Calls {@link Lifecycle.update} (guarded by lifecycle state).
      *
-     * @param {TTarget | null} targetElement - The new DOM element to associate with this model.
+     * @param {TTarget} targetElement - The new DOM element to associate with this model.
      * @returns {void}
      */
-    public updateTarget(targetElement: TTarget | null): void {
+    public updateTarget(targetElement?: TTarget): void {
         this.targetElement = targetElement;
         this.update();
     }
@@ -167,4 +169,4 @@ export class Model<
 
         super.destroy();
     }
-}
+}

package/src/ts/core/base/recyclerview.ts

@@ -22,9 +22,11 @@ import { LifecycleState } from "src/ts/types/core/base/lifecycle.type";
  */
 export class RecyclerView<
     TItem extends ModelContract<any, any>,
-    TAdapter extends AdapterContract<TItem>
-> extends Lifecycle implements RecyclerViewContract<TAdapter> {
-
+    TAdapter extends AdapterContract<TItem>,
+>
+    extends Lifecycle
+    implements RecyclerViewContract<TAdapter>
+{
     /** Root container that hosts rendered item views. */
     public viewElement: HTMLDivElement | null = null;
 
@@ -113,10 +115,10 @@ export class RecyclerView<
         if (this.is(LifecycleState.DESTROYED)) {
             return;
         }
-        
+
         this.viewElement = null;
         this.adapter = null;
 
         super.destroy();
     }
-}
+}

package/src/ts/core/base/view.ts

@@ -39,13 +39,16 @@ import { LifecycleState } from "src/ts/types/core/base/lifecycle.type";
  * @see {@link ViewContract}
  * @see {@link LifecycleState}
  */
-export class View<TTags extends Record<string, HTMLElement>> extends Lifecycle implements ViewContract<TTags> {
+export class View<TTags extends Record<string, HTMLElement>>
+    extends Lifecycle
+    implements ViewContract<TTags>
+{
     /**
      * Host container element into which this view's root element is rendered/attached.
      *
      * This reference is captured at construction time and cleared on {@link destroy}.
      */
-    public parent: HTMLElement | null = null;
+    public parent?: HTMLElement;
 
     /**
      * Mounted view result containing:
@@ -54,7 +57,7 @@ export class View<TTags extends Record<string, HTMLElement>> extends Lifecycle i
      *
      * This is expected to be assigned by subclasses (or a mount helper) before {@link getView} is called.
      */
-    public view: MountViewResult<TTags> | null = null;
+    public view?: MountViewResult<TTags>;
 
     /**
      * Creates a View bound to the specified parent container and initializes lifecycle state.
@@ -79,7 +82,9 @@ export class View<TTags extends Record<string, HTMLElement>> extends Lifecycle i
      */
     public getView(): HTMLElement {
         if (!this.view?.view) {
-            throw new Error("View is not mounted. Did you forget to set this.view?");
+            throw new Error(
+                "View is not mounted. Did you forget to set this.view?",
+            );
         }
         return this.view.view;
     }
@@ -108,4 +113,4 @@ export class View<TTags extends Record<string, HTMLElement>> extends Lifecycle i
 
         super.destroy();
     }
-}
+}