fix-repl-window-size-bug #454

Merged

navicore merged 2 commits from fix-repl-window-size-bug into main

2026-05-05 05:13:01 +00:00

navicore commented

2026-05-05 04:58:54 +00:00

Owner

No description provided.

navicore added 2 commits

2026-05-05 04:58:54 +00:00

Crash fix. crates/repl/src/app.rs::render_completions now clamps the popup to the 9bedba09ea

actual frame bounds and skips rendering if there isn't room for at least one item (3
rows) plus the border (4 cols). Repro for the panic: terminal of any size where
popup_height > frame_area.height - y. With the take(10) cap the natural popup height
is up to 12 rows, so any terminal under 13 rows can trigger it depending on cursor
position. The 78×7 case you hit was right in the wedge.

Regression test. test_render_completion_popup_does_not_panic_on_tiny_terminal — types
id, hits Tab to populate completions (10 items), then renders against a 78×7
TestBackend. The naive code panics in Clear; the clamped code renders cleanly.

Pre-included stdlibs. REPL_TEMPLATE now includes all 13 stdlib modules (alphabetical)

CI - Linux / CI - Linux x86_64 (pull_request) Successful in 4m23s

Details

03964a6513

with a header comment explaining why and how to prune. Tab completion will surface
stdlib words from a fresh session — int->bits-fmt-8 is now reachable without :include
std:imath first.

navicore merged commit c1a3e6d419 into main

2026-05-05 05:13:01 +00:00

navicore deleted branch fix-repl-window-size-bug

2026-05-05 05:13:01 +00:00

navicore referenced this pull request from a commit

2026-05-05 05:13:02 +00:00

Merge pull request 'fix-repl-window-size-bug' (#454) from fix-repl-window-size-bug into main