fix(parser): lru cache (#753)

Author: tenphi

.changeset/metal-bananas-move.md

@@ -0,0 +1,5 @@
+---
+"@cube-dev/ui-kit": patch
+---
+
+Fix \$label-width definition in Label component.

.changeset/ninety-actors-hear.md

@@ -0,0 +1,5 @@
+---
+"@cube-dev/ui-kit": patch
+---
+
+Set isDismissable for DialogContainer to true by default.

.changeset/weak-parrots-jump.md

@@ -0,0 +1,5 @@
+---
+"@cube-dev/ui-kit": patch
+---
+
+Fix LRU cache error in the style parser.

src/components/overlays/Dialog/DialogContainer.tsx

@@ -41,7 +41,7 @@ export function DialogContainer(props: CubeDialogContainerProps) {
     children,
     type = 'modal',
     onDismiss,
-    isDismissable,
+    isDismissable = true,
     isKeyboardDismissDisabled,
     isOpen,
     hideOnClose,

src/tasty/parser/lru.ts

@@ -3,7 +3,14 @@ export class Lru<K, V> {
   private head: K | null = null;
   private tail: K | null = null;
 
-  constructor(private limit = 1000) {}
+  constructor(private limit = 1000) {
+    // Normalize limit; fall back to sensible default (1000) to keep caching enabled
+    let normalized = Number.isFinite(this.limit)
+      ? Math.floor(this.limit)
+      : 1000;
+    if (normalized <= 0) normalized = 1000;
+    this.limit = normalized;
+  }
 
   get(key: K): V | undefined {
     const node = this.map.get(key);
@@ -20,7 +27,10 @@ export class Lru<K, V> {
       return;
     }
     node = { prev: null, next: this.head, value };
-    if (this.head) this.map.get(this.head)!.prev = key;
+    if (this.head) {
+      const headNode = this.map.get(this.head);
+      if (headNode) headNode.prev = key;
+    }
     this.head = key;
     if (!this.tail) this.tail = key;
     this.map.set(key, node);
@@ -31,23 +41,42 @@ export class Lru<K, V> {
     if (this.head === key) return; // already MRU
 
     // detach
-    if (node.prev) this.map.get(node.prev)!.next = node.next;
-    if (node.next) this.map.get(node.next)!.prev = node.prev;
+    if (node.prev) {
+      const prevNode = this.map.get(node.prev);
+      if (prevNode) prevNode.next = node.next;
+    }
+    if (node.next) {
+      const nextNode = this.map.get(node.next);
+      if (nextNode) nextNode.prev = node.prev;
+    }
     if (this.tail === key) this.tail = node.prev;
 
     // move to head
     node.prev = null;
     node.next = this.head;
-    if (this.head) this.map.get(this.head)!.prev = key;
+    if (this.head) {
+      const headNode = this.map.get(this.head);
+      if (headNode) headNode.prev = key;
+    }
     this.head = key;
   }
 
   private evict() {
     const old = this.tail;
     if (!old) return;
-    const node = this.map.get(old)!;
-    if (node.prev) this.map.get(node.prev)!.next = null;
+    const node = this.map.get(old);
+    if (!node) {
+      // Tail pointer is stale; reset pointers conservatively
+      if (this.head === old) this.head = null;
+      this.tail = null;
+      return;
+    }
+    if (node.prev) {
+      const prevNode = this.map.get(node.prev);
+      if (prevNode) prevNode.next = null;
+    }
     this.tail = node.prev;
+    if (this.head === old) this.head = null;
     this.map.delete(old);
   }