fix(devtools): harden wizard invocation

Author: danielroe

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

@@ -1,10 +1,5 @@
-importprocessfrom'node:process'
-
-import{styleText}from'node:util'
-
 import{defineCommand}from'citty'
 import{x}from'tinyexec'
-import{logger}from'../utils/logger'
 
 import{resolveRootDir}from'../utils/paths'
 import{rootDirArgs}from'./_shared'
@@ -15,7 +10,6 @@ export default defineCommand({
 description: 'Enable or disable devtools in a Nuxt project',
 },
 args: {
-// `command` has to precede the `dir` positional supplied by `rootDirArgs`
 command: {
 type: 'positional',
 description: 'Command to run',
@@ -27,14 +21,13 @@ export default defineCommand({
 constcwd=resolveRootDir(ctx.args)
 constcommand=ctx.args.command
 
-if(!command||!['enable','disable'].includes(command)){
-logger.error(`Unknown command ${styleText('cyan',command||'')}.`)
-process.exit(1)
+if(command!=='enable'&&command!=='disable'){
+thrownewError(`Unknown devtools command \`${command}\`. Expected \`enable\` or \`disable\`.`)
 }
 
 awaitx(
 'npx',
-['@nuxt/devtools-wizard@latest',command,cwd],
+['--yes','@nuxt/devtools-wizard@latest',command],
 {
 throwOnError: true,
 nodeOptions: {

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

@@ -0,0 +1,40 @@
+import{resolve}from'pathe'
+import{beforeEach,describe,expect,it,vi}from'vitest'
+
+import{runCommand}from'../../../src/run'
+
+const{ x }=vi.hoisted(()=>({
+x: vi.fn(()=>Promise.resolve({exitCode: 0})),
+}))
+
+vi.mock('tinyexec',()=>({ x }))
+
+describe('nuxt devtools command',()=>{
+beforeEach(()=>{
+vi.clearAllMocks()
+})
+
+it.each(['enable','disable'])('runs the devtools wizard to %s devtools',async(action)=>{
+awaitrunCommand('devtools',[action,'apps/web'])
+
+expect(x).toHaveBeenCalledOnce()
+expect(x).toHaveBeenCalledWith(
+'npx',
+['--yes','@nuxt/devtools-wizard@latest',action],
+{
+throwOnError: true,
+nodeOptions: {
+stdio: 'inherit',
+cwd: resolve('apps/web'),
+},
+},
+)
+})
+
+it('rejects unknown actions without terminating programmatic callers',async()=>{
+awaitexpect(runCommand('devtools',['toggle'])).rejects.toThrow(
+'Unknown devtools command `toggle`. Expected `enable` or `disable`.',
+)
+expect(x).not.toHaveBeenCalled()
+})
+})