fix(init): print cd step for single-character project directories

Author: danielroe

packages/nuxt-cli/src/commands/init.ts

@@ -85,6 +85,29 @@ async function reportMissingNonInteractiveArgs<T extends ArgsDef>(
     .join(', ')}`)
 }
 
+/**
+ * Commands to print in the closing 'Next steps' box, in the order to run them.
+ *
+ * `dir` is relative to the current working directory, so `.` means the project
+ * was created in place and there is nowhere to `cd` to.
+ */
+export function getNextSteps(options: {
+  dir: string
+  shell: boolean
+  installFailure?: unknown
+  recoveryCommands: string[]
+  packageManager: PackageManagerName
+}): string[] {
+  const { dir, shell, installFailure, recoveryCommands, packageManager } = options
+  const runCmd = packageManager === 'deno' ? 'task' : 'run'
+  return [
+    !shell && dir !== '.' && `cd ${dir}`,
+    installFailure && `${packageManager} install`,
+    ...recoveryCommands,
+    `${packageManager} ${runCmd} dev`,
+  ].filter((step): step is string => typeof step === 'string')
+}
+
 export default defineCommand({
   meta: {
     name: 'init',
@@ -678,16 +701,13 @@ export default defineCommand({
     }
 
     // Display next steps
-    const relativeTemplateDir = relative(process.cwd(), template.dir) || '.'
-    const runCmd = selectedPackageManager === 'deno' ? 'task' : 'run'
-    const nextSteps = [
-      !ctx.args.shell
-      && relativeTemplateDir.length > 1
-      && colors.cyan(`cd ${relativeTemplateDir}`),
-      installFailure && colors.cyan(`${selectedPackageManager} install`),
-      ...recoveryCommands.map(command => colors.cyan(command)),
-      colors.cyan(`${selectedPackageManager} ${runCmd} dev`),
-    ].filter(Boolean)
+    const nextSteps = getNextSteps({
+      dir: relative(process.cwd(), template.dir) || '.',
+      shell: !!ctx.args.shell,
+      installFailure,
+      recoveryCommands,
+      packageManager: selectedPackageManager,
+    }).map(step => colors.cyan(step))
 
     logger.message()
     writeWithGuide(output => box(`\n${nextSteps.map(step => ` › ${step}`).join('\n')}\n`, ` 👉 Next steps `, {

packages/nuxt-cli/test/unit/commands/init.spec.ts

@@ -3,7 +3,41 @@ import { tmpdir } from 'node:os'
 import { join } from 'node:path'
 import { afterEach, beforeEach, describe, expect, it } from 'vitest'
 
-import { detectTemplatePackageManager } from '../../../src/commands/init'
+import { detectTemplatePackageManager, getNextSteps } from '../../../src/commands/init'
+
+describe('getNextSteps', () => {
+  const base = { shell: false, recoveryCommands: [] as string[], packageManager: 'npm' as const }
+
+  it('should tell the user to change into the project directory', () => {
+    expect(getNextSteps({ ...base, dir: 'my-app' })).toEqual(['cd my-app', 'npm run dev'])
+  })
+
+  it('should tell the user to change into a single-character directory', () => {
+    expect(getNextSteps({ ...base, dir: 'a' })).toEqual(['cd a', 'npm run dev'])
+  })
+
+  it('should omit the change of directory when the project was created in place', () => {
+    expect(getNextSteps({ ...base, dir: '.' })).toEqual(['npm run dev'])
+  })
+
+  it('should omit the change of directory when a shell is launched', () => {
+    expect(getNextSteps({ ...base, dir: 'my-app', shell: true })).toEqual(['npm run dev'])
+  })
+
+  it('should ask for a retried install before recovery commands', () => {
+    expect(getNextSteps({
+      ...base,
+      dir: 'my-app',
+      installFailure: { error: new Error('offline') },
+      recoveryCommands: ['pnpm approve-builds'],
+      packageManager: 'pnpm',
+    })).toEqual(['cd my-app', 'pnpm install', 'pnpm approve-builds', 'pnpm run dev'])
+  })
+
+  it('should use `task` for deno', () => {
+    expect(getNextSteps({ ...base, dir: 'my-app', packageManager: 'deno' })).toEqual(['cd my-app', 'deno task dev'])
+  })
+})
 
 describe('detectTemplatePackageManager', () => {
   let dir: string