fix: making nitro the default

Author: jherr

packages/cta-cli/src/command-line.ts

@@ -93,6 +93,10 @@ export async function normalizeOptions(
         selectedAddOns.add(cliOptions.host)
       }
 
+      if (!cliOptions.host) {
+        selectedAddOns.add('nitro')
+      }
+
       return await finalizeAddOns(framework, mode, Array.from(selectedAddOns))
     }
 

packages/cta-cli/src/ui-prompts.ts

@@ -16,6 +16,7 @@ import {
 import type { AddOn, PackageManager } from '@tanstack/cta-engine'
 
 import type { Framework } from '@tanstack/cta-engine/dist/types/types.js'
+import { InitialData } from '../../cta-ui/src/types'
 
 export async function getProjectName(): Promise<string> {
   const value = await text({
@@ -189,6 +190,7 @@ export async function selectHost(
   host?: string,
 ): Promise<string | undefined> {
   const hosts = new Set<AddOn>()
+  let initialValue: string | undefined = undefined
   for (const addOn of framework
     .getAddOns()
     .sort((a, b) => a.name.localeCompare(b.name))) {
@@ -197,6 +199,9 @@ export async function selectHost(
       if (host && addOn.id === host) {
         return host
       }
+      if (addOn.default) {
+        initialValue = addOn.id
+      }
     }
   }
 
@@ -212,7 +217,7 @@ export async function selectHost(
         label: h.name,
       })),
     ],
-    initialValue: undefined,
+    initialValue: initialValue,
   })
 
   if (isCancel(hp)) {

packages/cta-ui-base/src/store/add-ons.ts

@@ -15,6 +15,14 @@ export function getAddOnStatus(
     }
   >()
 
+  const getAddOnType = (addOn: string) =>
+    availableAddOns.find((a) => a.id === addOn)!.type
+
+  const isMultiSelect = (addOn: string) => getAddOnType(addOn) === 'host'
+
+  const areAddonsOfThisTypeCurrentlyChosen = (addOn: string) =>
+    chosenAddOns.some((a) => getAddOnType(a) === getAddOnType(addOn))
+
   for (const addOn of availableAddOns) {
     addOnMap.set(addOn.id, {
       selected: false,
@@ -40,7 +48,7 @@ export function getAddOnStatus(
           const dependsOnAddOn = addOnMap.get(dependsOnId)
           if (dependsOnAddOn) {
             dependsOnAddOn.selected = true
-            dependsOnAddOn.enabled = false
+            dependsOnAddOn.enabled = isMultiSelect(dependsOnId) ? true : false
             dependsOnAddOn.dependedUpon = true
             selectAndDisableDependsOn(dependsOnId)
           }
@@ -60,11 +68,23 @@ export function getAddOnStatus(
   for (const addOn of originalAddOns) {
     const addOnInfo = addOnMap.get(addOn)
     if (addOnInfo) {
-      addOnInfo.selected = true
-      addOnInfo.enabled = false
-      addOnInfo.dependedUpon = true
+      if (isMultiSelect(addOn)) {
+        if (areAddonsOfThisTypeCurrentlyChosen(addOn)) {
+          addOnInfo.selected = false
+          addOnInfo.enabled = true
+          addOnInfo.dependedUpon = false
+        } else {
+          addOnInfo.selected = true
+          addOnInfo.enabled = true
+          addOnInfo.dependedUpon = true
+        }
+      } else {
+        addOnInfo.selected = true
+        addOnInfo.enabled = false
+        addOnInfo.dependedUpon = true
+        cycleGuardedSelectAndDisableDependsOn(addOn)
+      }
     }
-    cycleGuardedSelectAndDisableDependsOn(addOn)
   }
 
   for (const addOnId of chosenAddOns) {