@navios/commander-tui 1.5.1 → 1.6.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@navios/commander-tui",
-  "version": "1.5.1",
+  "version": "1.6.1",
   "license": "MIT",
   "author": {
     "name": "Oleksandr Hanzha",

package/src/adapters/interface.ts

@@ -15,8 +15,9 @@ export interface AdapterRoot {
 
   /**
    * Unmount and cleanup the root.
+   * Can be async to allow waiting for cleanup to complete.
    */
-  unmount(): void
+  unmount(): void | Promise<void>
 }
 
 /**

package/src/keyboard/index.ts

@@ -1,2 +1,2 @@
-export * from './keyboard_manager.ts'
 export * from './create_bindings.ts'
+export * from './keyboard_manager.ts'

package/src/schemas/screen-options.ts

@@ -10,7 +10,7 @@ export const ScreenOptionsSchema = z.object({
   /** Whether the screen is hidden */
   hidden: z.boolean().optional().default(false),
   /** Whether the screen is static (ignored in auto-close calculations) */
-  static: z.boolean().optional().default(false),
+  static: z.boolean().optional().default(true),
 })
 
 export type ScreenOptions = z.infer<typeof ScreenOptionsSchema>

package/src/services/logger.ts

@@ -42,7 +42,6 @@ export class ScreenLoggerInstance implements LoggerService {
       typeof options.screen === 'string'
         ? {
             name: options.screen,
-            static: true,
           }
         : options.screen,
     )

package/src/services/screen.ts

@@ -43,6 +43,7 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
   // Prompt queue system
   private promptQueue: PendingPrompt[] = []
   private activePrompt: PendingPrompt | null = null
+  private promptVersion: number = 0
 
   constructor(id: string, options: ScreenOptions) {
     super()
@@ -63,6 +64,14 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
     return this.version
   }
 
+  private incrementPromptVersion(): void {
+    this.promptVersion++
+  }
+
+  getPromptVersion(): number {
+    return this.promptVersion
+  }
+
   /**
    * Internal: Set the manager reference
    */
@@ -343,10 +352,22 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
   }
 
   /**
-   * Get the currently active prompt (for rendering)
+   * Get the currently active prompt (for rendering).
+   * Returns a shallow clone to ensure React detects changes when prompt state mutates.
    */
   getActivePrompt(): PromptData | null {
-    return this.activePrompt?.data ?? null
+    if (!this.activePrompt?.data) return null
+    const prompt = this.activePrompt.data
+
+    // Clone with proper Set handling for multiChoice
+    if (prompt.type === 'multiChoice') {
+      return {
+        ...prompt,
+        selectedIndices: new Set(prompt.selectedIndices),
+      }
+    }
+
+    return { ...prompt }
   }
 
   /**
@@ -372,6 +393,7 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
     } else if (prompt.type === 'confirm') {
       ;(prompt as ConfirmPromptData).selectedValue = index === 0
     }
+    this.incrementPromptVersion()
     this.emit('prompt:updated')
   }
 
@@ -415,6 +437,7 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
     const prompt = this.activePrompt.data
     if (prompt.type === 'confirm') {
       ;(prompt as ConfirmPromptData).selectedValue = true
+      this.incrementPromptVersion()
       this.emit('prompt:updated')
     }
   }
@@ -424,6 +447,7 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
     const prompt = this.activePrompt.data
     if (prompt.type === 'confirm') {
       ;(prompt as ConfirmPromptData).selectedValue = false
+      this.incrementPromptVersion()
       this.emit('prompt:updated')
     }
   }
@@ -441,6 +465,7 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
       } else if (p.selectedIndices.size < p.maxSelect) {
         p.selectedIndices.add(p.focusedIndex)
       }
+      this.incrementPromptVersion()
       this.emit('prompt:updated')
     }
   }
@@ -457,6 +482,7 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
       const choice = prompt.choices[prompt.selectedIndex]
       if (choice?.input) {
         ;(prompt as ChoicePromptData).inputMode = true
+        this.incrementPromptVersion()
         this.emit('prompt:updated')
         return true
       }
@@ -476,6 +502,7 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
     const prompt = this.activePrompt.data
     if (prompt.type === 'choice' && prompt.inputMode) {
       ;(prompt as ChoicePromptData).inputMode = false
+      this.incrementPromptVersion()
       this.emit('prompt:updated')
     }
     // Input prompts cannot exit input mode
@@ -505,9 +532,11 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
     const prompt = this.activePrompt.data
     if (prompt.type === 'choice' && prompt.inputMode) {
       ;(prompt as ChoicePromptData).inputValue = value
+      this.incrementPromptVersion()
       this.emit('prompt:updated')
     } else if (prompt.type === 'input') {
       ;(prompt as InputPromptData).value = value
+      this.incrementPromptVersion()
       this.emit('prompt:updated')
     }
   }
@@ -521,9 +550,11 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
     const prompt = this.activePrompt.data
     if (prompt.type === 'choice' && prompt.inputMode) {
       ;(prompt as ChoicePromptData).inputValue += char
+      this.incrementPromptVersion()
       this.emit('prompt:updated')
     } else if (prompt.type === 'input') {
       ;(prompt as InputPromptData).value += char
+      this.incrementPromptVersion()
       this.emit('prompt:updated')
     }
   }
@@ -537,9 +568,11 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
     const prompt = this.activePrompt.data
     if (prompt.type === 'choice' && prompt.inputMode) {
       ;(prompt as ChoicePromptData).inputValue = prompt.inputValue.slice(0, -1)
+      this.incrementPromptVersion()
       this.emit('prompt:updated')
     } else if (prompt.type === 'input') {
       ;(prompt as InputPromptData).value = prompt.value.slice(0, -1)
+      this.incrementPromptVersion()
       this.emit('prompt:updated')
     }
   }
@@ -606,6 +639,7 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
     this.activePrompt = null
     this.activateNextPrompt()
     this.incrementVersion()
+    this.incrementPromptVersion()
     this.emit('prompt:resolved')
   }
 
@@ -618,6 +652,7 @@ export class ScreenInstance extends EventEmitter<ScreenEventMap> {
       // Notify manager to focus this screen
       this.manager?.onScreenPromptActivated(this)
       this.incrementVersion()
+      this.incrementPromptVersion()
       this.emit('prompt:activated')
     }
   }

package/src/services/screen_manager.ts

@@ -74,6 +74,8 @@ export class ScreenManagerInstance
     // Set first visible screen as active by default
     if (!this.activeScreenId && !screen.isHidden()) {
       this.activeScreenId = id
+      // Emit activeScreen:changed so subscribers know about the new active screen
+      this.emit('activeScreen:changed', id)
     }
 
     this.checkAutoClose()
@@ -125,6 +127,12 @@ export class ScreenManagerInstance
   async bind(options?: BindOptions): Promise<void> {
     if (this.mode !== RenderMode.UNBOUND) return
 
+    // In non-interactive environments (no TTY), stay in UNBOUND mode
+    // This allows prompts to return defaults and screens to print on completion
+    if (!process.stdout.isTTY) {
+      return
+    }
+
     this.bindOptions = options ?? {}
 
     // Resolve theme from options
@@ -266,15 +274,15 @@ export class ScreenManagerInstance
     return this.globalLogLevels ? Array.from(this.globalLogLevels) : null
   }
 
-  onServiceDestroy(): void {
-    this.unbind()
+  async onServiceDestroy(): Promise<void> {
+    await this.unbind()
   }
 
   /**
    * Stop TUI rendering and cleanup
    * Flushes screens to stdout/stderr based on mode
    */
-  unbind(): void {
+  async unbind(): Promise<void> {
     if (this.mode === RenderMode.UNBOUND) {
       // Even in unbound mode, flush any remaining screens on destroy
       this.flushRemainingScreens()
@@ -292,7 +300,7 @@ export class ScreenManagerInstance
     // Cleanup TUI if active
     if (previousMode === RenderMode.TUI_ACTIVE) {
       if (this.root) {
-        this.root.unmount()
+        await this.root.unmount()
       }
 
       // Explicitly disable mouse tracking before destroy to prevent escape sequence leakage
@@ -504,7 +512,7 @@ export class ScreenManagerInstance
     // Start auto-close timer (either all non-static screens succeeded, or only static screens exist)
     const delay = typeof autoClose === 'number' ? autoClose : 5000
     this.autoCloseTimer = setTimeout(() => {
-      this.unbind()
+      void this.unbind()
     }, delay)
   }