refactor: prevent edge cases

bombshell-dev · Nov 10, 2023 · f33a56f · f33a56f
1 parent 360afeb
commit f33a56f
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/packages/prompts/src/index.ts b/packages/prompts/src/index.ts
@@ -68,9 +68,10 @@ interface LimitOptionsParams<TOption> {
 const limitOptions = <TOption>(params: LimitOptionsParams<TOption>): string[] => {
 	const { cursor, options, style } = params;
 
+	const paramMaxItems = params.maxItems ?? Infinity;
+	const outputMaxItems = Math.max(process.stdout.rows - 4, 0);
 	// We clamp to minimum 5 because anything less doesn't make sense UX wise
-	const maxItems =
-		params.maxItems === undefined ? process.stdout.rows - 4 : Math.max(params.maxItems, 5);
+	const maxItems = Math.min(outputMaxItems, Math.max(paramMaxItems, 5));
 	let slidingWindowLocation = 0;
 
 	if (cursor >= slidingWindowLocation + maxItems - 3) {