@rlabs-inc/tui 0.5.0 → 0.6.0

package/index.ts

@@ -21,8 +21,20 @@ export type { Context } from './src/state/context'
 
 // State modules - Input handling
 export { keyboard, lastKey, lastEvent } from './src/state/keyboard'
-export { mouse, hitGrid, lastMouseEvent, mouseX, mouseY, isMouseDown } from './src/state/mouse'
-export { focusManager, focusedIndex } from './src/state/focus'
+export {
+  mouse,
+  hitGrid,
+  lastMouseEvent,
+  mouseX,
+  mouseY,
+  isMouseDown,
+  onMouseDown,
+  onMouseUp,
+  onClick,
+  onScroll,
+  onComponent,
+} from './src/state/mouse'
+export { focusManager, focusedIndex, pushFocusTrap, popFocusTrap, isFocusTrapped, getFocusTrapContainer } from './src/state/focus'
 export { scroll } from './src/state/scroll'
 export { globalKeys } from './src/state/global-keys'
 export { cursor } from './src/state/cursor'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rlabs-inc/tui",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "description": "The Terminal UI Framework for TypeScript/Bun - Blazing-fast, fine-grained reactive terminal UI with complete flexbox layout",
   "module": "index.ts",
   "main": "index.ts",

package/src/api/mount.ts

@@ -37,6 +37,13 @@ import { resetRegistry } from '../engine/registry'
 import { hitGrid, clearHitGrid, mouse } from '../state/mouse'
 import { globalKeys } from '../state/global-keys'
 
+// =============================================================================
+// MODULE STATE
+// =============================================================================
+
+// Track if global error handlers have been registered (only once per process)
+let globalErrorHandlersRegistered = false
+
 // =============================================================================
 // MOUNT
 // =============================================================================
@@ -149,13 +156,16 @@ export async function mount(
   // Create the component tree
   root()
 
-  // Global error handlers for debugging
-  process.on('uncaughtException', (err) => {
-    console.error('[TUI] Uncaught exception:', err)
-  })
-  process.on('unhandledRejection', (err) => {
-    console.error('[TUI] Unhandled rejection:', err)
-  })
+  // Global error handlers for debugging (register only once per process)
+  if (!globalErrorHandlersRegistered) {
+    globalErrorHandlersRegistered = true
+    process.on('uncaughtException', (err) => {
+      console.error('[TUI] Uncaught exception:', err)
+    })
+    process.on('unhandledRejection', (err) => {
+      console.error('[TUI] Unhandled rejection:', err)
+    })
+  }
 
   // THE ONE RENDER EFFECT
   // This is where the magic happens - reactive rendering!

package/src/pipeline/frameBuffer.ts

@@ -39,7 +39,6 @@ import {
   getInheritedBg,
   getBorderColors,
   getBorderStyles,
-  hasBorder,
   getEffectiveOpacity,
 } from '../engine/inheritance'
 
@@ -222,7 +221,8 @@ function renderComponent(
   // Get border configuration
   const borderStyles = getBorderStyles(index)
   const borderColors = getBorderColors(index)
-  const hasAnyBorder = hasBorder(index)
+  // Inline hasAnyBorder check - avoids redundant getBorderStyles call (was 12 array reads, now 8)
+  const hasAnyBorder = borderStyles.top > 0 || borderStyles.right > 0 || borderStyles.bottom > 0 || borderStyles.left > 0
 
   // Draw borders
   if (hasAnyBorder && w >= 2 && h >= 2) {

package/src/pipeline/layout/titan-engine.ts

@@ -217,7 +217,8 @@ export function computeLayoutTitan(
       if (firstChild[parent] === -1) {
         firstChild[parent] = i
       } else {
-        nextSibling[lastChild[parent]!] = i
+        const last = lastChild[parent] ?? -1
+        if (last !== -1) nextSibling[last] = i
       }
       lastChild[parent] = i
     } else {
@@ -231,10 +232,10 @@ export function computeLayoutTitan(
   let head = 0
   while (head < bfsQueue.length) {
     const parent = bfsQueue[head++]!
-    let child = firstChild[parent]!
+    let child = firstChild[parent] ?? -1
     while (child !== -1) {
       bfsQueue.push(child)
-      child = nextSibling[child]!
+      child = nextSibling[child] ?? -1
     }
   }
 
@@ -319,7 +320,7 @@ export function computeLayoutTitan(
       const overflow = layout.overflow[i] ?? Overflow.VISIBLE
       const isScrollable = overflow === Overflow.SCROLL || overflow === Overflow.AUTO
 
-      let kid = firstChild[i]!
+      let kid = firstChild[i] ?? -1
       if (kid !== -1 && !isScrollable) {
         // Normal containers: intrinsic size includes all children
         const dir = layout.flexDirection[i] ?? FLEX_COLUMN
@@ -368,7 +369,7 @@ export function computeLayoutTitan(
             sumMain += kidH + kidMarginMain + gap
             maxCross = Math.max(maxCross, kidW)
           }
-          kid = nextSibling[kid]!
+          kid = nextSibling[kid] ?? -1
         }
 
         if (childCount > 0) sumMain -= gap
@@ -429,12 +430,12 @@ export function computeLayoutTitan(
     let childrenMaxCross = 0
 
     // Collect flow children
-    let kid = firstChild[parent]!
+    let kid = firstChild[parent] ?? -1
     while (kid !== -1) {
       if ((layout.position[kid] ?? POS_RELATIVE) !== POS_ABSOLUTE) {
         flowKids.push(kid)
       }
-      kid = nextSibling[kid]!
+      kid = nextSibling[kid] ?? -1
     }
 
     if (flowKids.length === 0) return

package/src/pipeline/layout/utils/text-measure.ts

@@ -33,7 +33,8 @@ export function measureTextHeight(content: string, maxWidth: number): number {
   // Split by existing newlines first
   const paragraphs = content.split('\n')
 
-  for (const paragraph of paragraphs) {
+  for (let i = 0; i < paragraphs.length; i++) {
+    const paragraph = paragraphs[i]!
     if (paragraph === '') {
       lines++
       continue
@@ -53,7 +54,7 @@ export function measureTextHeight(content: string, maxWidth: number): number {
 
     // Reset for next paragraph
     currentLineWidth = 0
-    if (paragraphs.indexOf(paragraph) < paragraphs.length - 1) {
+    if (i < paragraphs.length - 1) {
       lines++
     }
   }

package/src/primitives/box.ts

@@ -38,6 +38,7 @@ import {
 import { cleanupIndex as cleanupKeyboardListeners } from '../state/keyboard'
 import { getVariantStyle } from '../state/theme'
 import { getActiveScope } from './scope'
+import { enumSource } from './utils'
 
 // Import arrays
 import * as core from '../engine/arrays/core'
@@ -118,28 +119,6 @@ function alignSelfToNum(align: string | undefined): number {
   }
 }
 
-/**
- * Create a slot source for enum props - returns getter for reactive, value for static.
- * For use with slotArray.setSource()
- * Handles: static values, signals/bindings ({ value: T }), and getter functions (() => T)
- */
-function enumSource<T extends string>(
-  prop: T | { value: T } | (() => T) | undefined,
-  converter: (val: T | undefined) => number
-): number | (() => number) {
-  // Handle getter function (inline derived)
-  if (typeof prop === 'function') {
-    return () => converter(prop())
-  }
-  // Handle object with .value (signal/binding/derived)
-  if (prop !== undefined && typeof prop === 'object' && prop !== null && 'value' in prop) {
-    const reactiveSource = prop as { value: T }
-    return () => converter(reactiveSource.value)
-  }
-  // Static value
-  return converter(prop as T | undefined)
-}
-
 /** Get static boolean for visible prop */
 function getStaticBool(prop: unknown, defaultVal: boolean): boolean {
   if (prop === undefined) return defaultVal

package/src/primitives/scope.ts

@@ -22,8 +22,7 @@
 
 import { effectScope, onScopeDispose } from '@rlabs-inc/signals'
 import type { EffectScope } from '@rlabs-inc/signals'
-
-export type Cleanup = () => void
+import type { Cleanup } from './types'
 
 // =============================================================================
 // ACTIVE SCOPE TRACKING

package/src/primitives/show.ts

@@ -58,14 +58,15 @@ export function show(
     // Initial render
     update(conditionGetter())
 
-    // Effect for updates - skip first run (don't read condition to avoid double tracking)
+    // Effect for updates - reads condition to establish dependency
+    // but skips the update on first run since we already rendered above
     let initialized = false
     effect(() => {
+      const condition = conditionGetter() // Must read to track dependency!
       if (!initialized) {
         initialized = true
         return
       }
-      const condition = conditionGetter()
       update(condition)
     })
 

package/src/primitives/text.ts

@@ -34,6 +34,7 @@ import {
 import { cleanupIndex as cleanupKeyboardListeners } from '../state/keyboard'
 import { getVariantStyle } from '../state/theme'
 import { getActiveScope } from './scope'
+import { enumSource } from './utils'
 
 // Import arrays
 import * as core from '../engine/arrays/core'
@@ -79,8 +80,6 @@ function alignSelfToNum(alignSelf: string | undefined): number {
   }
 }
 
-// bindEnumProp removed - using enumSource instead
-
 /**
  * Convert content prop (string | number) to string source for setSource.
  * Handles: static values, signals, and getters.
@@ -101,28 +100,6 @@ function contentToStringSource(
   return String(content)
 }
 
-/**
- * Create a slot source for enum props - returns getter for reactive, value for static.
- * For use with slotArray.setSource()
- * Handles: static values, signals/bindings ({ value: T }), and getter functions (() => T)
- */
-function enumSource<T extends string>(
-  prop: T | { value: T } | (() => T) | undefined,
-  converter: (val: T | undefined) => number
-): number | (() => number) {
-  // Handle getter function (inline derived)
-  if (typeof prop === 'function') {
-    return () => converter(prop())
-  }
-  // Handle object with .value (signal/binding/derived)
-  if (prop !== undefined && typeof prop === 'object' && prop !== null && 'value' in prop) {
-    const reactiveSource = prop as { value: T }
-    return () => converter(reactiveSource.value)
-  }
-  // Static value
-  return converter(prop as T | undefined)
-}
-
 // =============================================================================
 // TEXT COMPONENT
 // =============================================================================

package/src/primitives/types.ts

@@ -232,41 +232,6 @@ export interface InputProps extends StyleProps, BorderProps, DimensionProps, Spa
   onBlur?: () => void
 }
 
-// =============================================================================
-// SELECT PROPS
-// =============================================================================
-
-export interface SelectOption {
-  value: string
-  label: string
-}
-
-export interface SelectProps extends StyleProps, BorderProps, DimensionProps {
-  /** Selected value (two-way bound) */
-  value: WritableSignal<string> | Binding<string>
-  /** Available options */
-  options: SelectOption[]
-  /** Placeholder when nothing selected */
-  placeholder?: string
-  /** Is visible */
-  visible?: Reactive<boolean>
-  /** Called when selection changes */
-  onChange?: (value: string) => void
-}
-
-// =============================================================================
-// PROGRESS PROPS
-// =============================================================================
-
-export interface ProgressProps extends StyleProps, DimensionProps {
-  /** Progress value 0-1 */
-  value: Reactive<number>
-  /** Show percentage text */
-  showPercent?: boolean
-  /** Is visible */
-  visible?: Reactive<boolean>
-}
-
 // =============================================================================
 // COMPONENT RETURN TYPE
 // =============================================================================

package/src/primitives/utils.ts

@@ -35,3 +35,29 @@ export function getValue<T>(prop: T | { value: T } | (() => T) | undefined, defa
   if (typeof prop === 'object' && prop !== null && 'value' in prop) return (prop as { value: T }).value
   return prop as T
 }
+
+// =============================================================================
+// ENUM SOURCE
+// =============================================================================
+
+/**
+ * Create a slot source for enum props - returns getter for reactive, value for static.
+ * For use with slotArray.setSource()
+ * Handles: static values, signals/bindings ({ value: T }), and getter functions (() => T)
+ */
+export function enumSource<T extends string>(
+  prop: T | { value: T } | (() => T) | undefined,
+  converter: (val: T | undefined) => number
+): number | (() => number) {
+  // Handle getter function (inline derived)
+  if (typeof prop === 'function') {
+    return () => converter(prop())
+  }
+  // Handle object with .value (signal/binding/derived)
+  if (prop !== undefined && typeof prop === 'object' && prop !== null && 'value' in prop) {
+    const reactiveSource = prop as { value: T }
+    return () => converter(reactiveSource.value)
+  }
+  // Static value
+  return converter(prop as T | undefined)
+}

package/src/state/drawnCursor.ts

@@ -111,6 +111,11 @@ function getBlinkClock(fps: number): BlinkRegistry {
 }
 
 function subscribeToBlink(fps: number): () => void {
+  // Guard against invalid fps (0 or negative would cause Infinity interval)
+  if (fps <= 0) {
+    return () => {} // No-op unsubscribe
+  }
+
   const registry = getBlinkClock(fps)
   registry.subscribers++