fix: Remove improper effect() usage to fix runtime error

Removed effect() calls from constructors that were tracking @input properties
instead of signals, which caused 'Cannot read properties of null' errors.
Moved logic to ngOnChanges() and ngOnInit() lifecycle hooks.

Author: ishansasika

demo/src/app/components/playground/component-playground/component-playground.component.ts

@@ -1,4 +1,4 @@
-import { Component, Input, signal, computed, OnInit, effect } from '@angular/core';
+import { Component, Input, signal, computed, OnInit } from '@angular/core';
 import { CommonModule } from '@angular/common';
 import { ComponentMetadata, CodeExample } from '../../../models/playground.types';
 import { SELECT_EXAMPLES } from '../../../data/select-examples';
@@ -38,12 +38,7 @@ export class ComponentPlaygroundComponent implements OnInit {
   });
 
   constructor(private codeGenerator: CodeGeneratorService) {
-    // Initialize props when metadata is available
-    effect(() => {
-      if (this.metadata && Object.keys(this.currentProps()).length === 0) {
-        this.currentProps.set({ ...this.metadata.defaultProps });
-      }
-    });
+    // Initialization happens in ngOnInit
   }
 
   ngOnInit(): void {

src/lib/components/perfect-select/perfect-select.component.ts

@@ -13,7 +13,6 @@ import {
   SimpleChanges,
   ElementRef,
   ViewChild,
-  effect,
   inject,
   DestroyRef
 } from '@angular/core';
@@ -282,19 +281,19 @@ export class PerfectSelectComponent implements ControlValueAccessor, OnInit, OnC
   private onTouched: any = () => {};
 
   constructor() {
-    // Update closeMenuOnSelect based on isMulti
-    effect(() => {
-      if (this.isMulti && this.closeMenuOnSelect === true) {
-        this.closeMenuOnSelect = false;
-      }
-    });
+    // Constructor intentionally empty - initialization happens in ngOnInit
   }
 
   ngOnChanges(changes: SimpleChanges): void {
     // Update internal options when the options input changes
     if (changes['options'] && this.options.length > 0 && !this.loadOptions) {
       this.internalOptions.set([...this.options]);
     }
+
+    // Update closeMenuOnSelect based on isMulti
+    if (changes['isMulti'] && this.isMulti && this.closeMenuOnSelect === true) {
+      this.closeMenuOnSelect = false;
+    }
   }
 
   writeValue(value: any): void {
@@ -319,6 +318,11 @@ export class PerfectSelectComponent implements ControlValueAccessor, OnInit, OnC
       this.internalOptions.set([...this.options]);
     }
 
+    // Update closeMenuOnSelect based on isMulti (initial setup)
+    if (this.isMulti && this.closeMenuOnSelect === true) {
+      this.closeMenuOnSelect = false;
+    }
+
     // Load default options if async
     if (this.loadOptions && this.defaultOptions) {
       this.handleLoadOptions('');