address CodeRabbit review comments

Author: onmax

packages/nuxi/src/commands/doctor.ts

@@ -238,6 +238,11 @@ function checkConfig(checks: DoctorCheck[], nuxt: Nuxt): void {
 async function checkModuleCompat(checks: DoctorCheck[], nuxt: Nuxt, cwd: string): Promise<void> {
   const nuxtVersion = await resolveNuxtVersion(cwd)
   if (!nuxtVersion) {
+    checks.push({
+      name: 'Modules',
+      status: 'warning',
+      message: 'could not determine Nuxt version for compatibility check',
+    })
     return
   }
 

packages/nuxi/test/unit/commands/doctor.spec.ts

@@ -31,8 +31,9 @@ vi.mock('@clack/prompts', () => ({
 // Mock process.exit to prevent tests from exiting
 const mockExit = vi.spyOn(process, 'exit').mockImplementation(() => undefined as never)
 
-// Store original process.version to restore after tests
+// Store original process versions to restore after tests
 const originalProcessVersion = process.version
+const originalProcessVersionsNode = process.versions.node
 
 describe('nuxt doctor command', () => {
   beforeEach(() => {
@@ -42,6 +43,10 @@ describe('nuxt doctor command', () => {
     // Reset mockCallHook implementation (clearAllMocks only clears history, not implementation)
     mockCallHook.mockReset()
 
+    // Restore process versions in case previous test modified them
+    Object.defineProperty(process, 'version', { value: originalProcessVersion, configurable: true })
+    Object.defineProperty(process.versions, 'node', { value: originalProcessVersionsNode, configurable: true })
+
     // Default mock for loadNuxt
     mockLoadNuxt.mockResolvedValue({
       options: {
@@ -161,6 +166,7 @@ describe('nuxt doctor command', () => {
   it('should error on Node.js < 18', async () => {
     // Mock Node version < 18
     Object.defineProperty(process, 'version', { value: 'v16.20.0', configurable: true })
+    Object.defineProperty(process.versions, 'node', { value: '16.20.0', configurable: true })
 
     mockLoadNuxt.mockResolvedValueOnce({
       options: {
@@ -184,9 +190,6 @@ describe('nuxt doctor command', () => {
 
     // Node < 18 is an error
     expect(mockExit).toHaveBeenCalledWith(1)
-
-    // Restore original version
-    Object.defineProperty(process, 'version', { value: originalProcessVersion, configurable: true })
   })
 
   it('should warn when prerender routes defined with SSR disabled', async () => {
@@ -269,6 +272,7 @@ describe('nuxt doctor command', () => {
 
     // Should only call console.log once with valid JSON
     expect(consoleSpy).toHaveBeenCalledOnce()
+    expect(consoleSpy.mock.calls[0]).toBeDefined()
     const rawOutput = consoleSpy.mock.calls[0]![0]
 
     // Verify output is valid JSON (no intro/outro pollution)
@@ -294,6 +298,7 @@ describe('nuxt doctor command', () => {
     })
 
     expect(consoleSpy).toHaveBeenCalledOnce()
+    expect(consoleSpy.mock.calls[0]).toBeDefined()
     const rawOutput = consoleSpy.mock.calls[0]![0]
     expect(() => JSON.parse(rawOutput)).not.toThrow()
     const output = JSON.parse(rawOutput)
@@ -331,8 +336,10 @@ describe('nuxt doctor command', () => {
       data: undefined,
     })
 
+    expect(consoleSpy.mock.calls[0]).toBeDefined()
     const output = JSON.parse(consoleSpy.mock.calls[0]![0])
     const testCheck = output.find((c: any) => c.name === 'Test')
+    expect(testCheck).toBeDefined()
 
     // Validate required fields
     expect(testCheck).toHaveProperty('name')