Fix blank project name prompt submit (#472)

Author: tannerlinsley

.changeset/quick-humans-sort.md

@@ -0,0 +1,5 @@
+---
+'@tanstack/cli': patch
+---
+
+Fix blank project name submissions in interactive create prompts.

packages/cli/src/ui-prompts.ts

@@ -67,12 +67,16 @@ export async function selectInstall(): Promise<boolean> {
 export async function getProjectName(): Promise<string> {
   const value = await text({
     message: 'Project name (leave empty to use current directory)',
+    // Clack prints `undefined` on blank submit when placeholder is omitted.
+    placeholder: '',
     validate(value) {
-      if (isCurrentDirectoryProjectNameInput(value)) {
+      const projectName = value ?? ''
+
+      if (isCurrentDirectoryProjectNameInput(projectName)) {
         return
       }
 
-      const { valid, error } = validateProjectName(value)
+      const { valid, error } = validateProjectName(projectName)
       if (!valid) {
         return error
       }
@@ -84,7 +88,7 @@ export async function getProjectName(): Promise<string> {
     process.exit(0)
   }
 
-  return value.trim()
+  return (value ?? '').trim()
 }
 
 export async function selectPackageManager(): Promise<PackageManager> {

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

@@ -43,11 +43,26 @@ describe('getProjectName', () => {
     expect(textOptions.message).toBe(
       'Project name (leave empty to use current directory)',
     )
-    expect(textOptions.placeholder).toBeUndefined()
+    expect(textOptions.placeholder).toBe('')
     expect(textOptions.validate?.('')).toBeUndefined()
     expect(textOptions.validate?.('.')).toBeUndefined()
   })
 
+  it('should handle undefined project name values as current directory creation', async () => {
+    const textSpy = vi
+      .spyOn(clack, 'text')
+      .mockImplementation(async () => undefined as unknown as string)
+    vi.spyOn(clack, 'isCancel').mockImplementation(() => false)
+
+    const projectName = await getProjectName()
+    const textOptions = textSpy.mock.calls[0]![0] as {
+      validate?: (value?: string) => string | undefined
+    }
+
+    expect(projectName).toBe('')
+    expect(textOptions.validate?.(undefined)).toBeUndefined()
+  })
+
   it('should exit on cancel', async () => {
     vi.spyOn(clack, 'text').mockImplementation(async () => 'Cancelled')
     vi.spyOn(clack, 'isCancel').mockImplementation(() => true)