fix(cli): avoid paginated add-on multiselect glitch

Set multiselect maxItems to the full selectable add-on count so Clack pagination markers are not used for this prompt, preventing duplicated option rendering near the bottom of the list.

Fixes #352

Author: tannerlinsley

.changeset/fair-trees-grab.md

@@ -0,0 +1,5 @@
+---
+"@tanstack/cli": patch
+---
+
+Prevent add-on multiselect options from rendering with pagination markers by showing the full list, which avoids a Clack navigation glitch that could duplicate the second-to-last entry while moving between the bottom options.

packages/cli/src/ui-prompts.ts

@@ -86,15 +86,22 @@ export async function selectAddOns(
   }
 
   if (allowMultiple) {
+    const selectableAddOns = addOns.filter(
+      (addOn) => !forcedAddOns.includes(addOn.id),
+    )
+
+    if (selectableAddOns.length === 0) {
+      return []
+    }
+
     const value = await multiselect({
       message,
-      options: addOns
-        .filter((addOn) => !forcedAddOns.includes(addOn.id))
-        .map((addOn) => ({
+      options: selectableAddOns.map((addOn) => ({
           value: addOn.id,
           label: addOn.name,
           hint: addOn.description,
         })),
+      maxItems: selectableAddOns.length,
       required: false,
     })
 

packages/cli/tests/ui-prompts.test.ts

@@ -57,7 +57,9 @@ describe('selectPackageManager', () => {
 describe('selectAddOns', () => {
   it('should show keyboard shortcuts help and select add-ons', async () => {
     const noteSpy = vi.spyOn(clack, 'note').mockImplementation(() => {})
-    vi.spyOn(clack, 'multiselect').mockImplementation(async () => ['add-on-1'])
+    const multiselectSpy = vi
+      .spyOn(clack, 'multiselect')
+      .mockImplementation(async () => ['add-on-1'])
     vi.spyOn(clack, 'isCancel').mockImplementation(() => false)
 
     const packageManager = await selectAddOns(
@@ -83,6 +85,9 @@ describe('selectAddOns', () => {
       'Use ↑/↓ to navigate • Space to select/deselect • Enter to confirm',
       'Keyboard Shortcuts',
     )
+    expect(multiselectSpy).toHaveBeenCalledWith(
+      expect.objectContaining({ maxItems: 1 }),
+    )
   })
 
   it('should exit on cancel', async () => {