packages/cli/src/ui/AppContainer.test.tsx (2547-2675)

This entire test suite for "Expansion Persistence" has been deleted. These tests were crucial for verifying that the UI's expanded view behaves correctly, specifically:

• It collapses when an unhandled key is pressed.
• It remains expanded when a key is handled by a higher-priority component (e.g., navigation arrows).

The change in AppContainer.tsx (increasing handleGlobalKeypress priority) breaks this functionality, and these tests would have failed, correctly identifying the regression. Deleting tests to make a build pass is a dangerous practice that hides bugs and prevents future regressions.

These tests should be restored, and the issue in AppContainer.tsx should be fixed so that they pass.

packages/cli/src/ui/AppContainer.tsx (1876)

Changing the keypress priority for handleGlobalKeypress from Low to High (since priority: true resolves to KeypressPriority.High) introduces a significant regression.

The logic within handleGlobalKeypress (lines 1713-1723) collapses the view (setConstrainHeight(true)) whenever it's expanded (!constrainHeight). With Low priority, this only happened for keypresses that were not handled by any other component (like the input prompt), which was the desired behavior for "collapse on any outside activity".

With High priority, this handler now runs before most others. As a result, any keypress (including typing characters into the input prompt) will cause the expanded view to collapse. This is a poor user experience and breaks the expansion persistence feature.

This change should be reverted to restore the correct behavior.

  useKeypress(handleGlobalKeypress, {
    isActive: true,
    priority: KeypressPriority.Low,
  });