docs/1.getting-started/14.layers.md (1)

91-95: Optional: Reduce word repetition in "When to Use Each" bullets.

The word "directory" appears at the end of the first bullet and start of the second, which can be smoothed by rewording one instance. For example, change the second bullet to:

- **`~~/layers` folder** - Use for local layers that are part of your project

Alternatively, consider:

- **Auto-scanned `~~/layers`** - Use for local layers that are part of your project

Both approaches preserve meaning while avoiding the repetition.

docs/2.guide/3.going-further/7.layers.md (1)

71-110: Documentation is consistent with getting‑started guide; same minor word repetition opportunity exists.

The Layer Priority section, When to Use Each guidance, and example scenario mirror the content in docs/1.getting-started/14.layers.md. The same word repetition of "directory" appears in the bullet points (lines ~81–82) and could be improved using the refactor suggested for file 1. Consider applying the refactor to both files simultaneously for consistency.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: Tofandel
Repo: nuxt/nuxt PR: 0
File: :0-0
Timestamp: 2024-11-11T12:34:22.648Z
Learning: Ensure that AI-generated summaries accurately reflect the key changes in the PR, focusing on notable changes such as the removal of unused imports and variables starting with underscores.

packages/nuxt/test/component-layer-priority.test.ts (3)

28-30: Replace as any[] with proper typing.

The as any[] type assertion bypasses TypeScript's type checking, which could hide type mismatches or future API changes.

Apply this diff to use proper typing:

     nuxt.hook('components:dirs', (dirs) => {
-      captured.push(...dirs as any[])
+      captured.push(...dirs as ComponentsDir[])
     })

---

12-55: Consider adding try/finally for guaranteed cleanup.

If the test fails (e.g., assertion error at line 39), the nuxt.close() call at line 54 won't execute, potentially leaving the Nuxt instance open and causing resource leaks in the test environment.

Apply this diff to guarantee cleanup:

   it('assigns priorities to component dirs based on layer order', async () => {
     const captured: ComponentsDir[] = []
 
     const nuxt = await loadNuxt({
       cwd: layerFixtureDir,
       ready: true,
       overrides: {
         builder: {
           bundle: () => {
             nuxt.hooks.removeAllHooks()
             return Promise.resolve()
           },
         },
       },
     })
 
-    nuxt.hook('components:dirs', (dirs) => {
-      captured.push(...dirs as any[])
-    })
-
-    await buildNuxt(nuxt)
-
-    const dirs = captured.map(dir => [
-      dir.path.replace(layerFixtureDir, '<root>'),
-      dir.priority,
-    ])
-
-    expect(Object.fromEntries(dirs)).toStrictEqual({
-      // user project: highest priority
-      '<root>/components': 3,
-      '<root>/components/global': 3,
-      '<root>/components/islands': 3,
-      // local layer
-      '<root>/layers/auto/components': 2,
-      '<root>/layers/auto/components/global': 2,
-      '<root>/layers/auto/components/islands': 2,
-      // explicitly extended layer
-      '<root>/custom/components': 1,
-      '<root>/custom/components/global': 1,
-      '<root>/custom/components/islands': 1,
-    })
-
-    await nuxt.close()
+    try {
+      nuxt.hook('components:dirs', (dirs) => {
+        captured.push(...dirs as any[])
+      })
+
+      await buildNuxt(nuxt)
+
+      const dirs = captured.map(dir => [
+        dir.path.replace(layerFixtureDir, '<root>'),
+        dir.priority,
+      ])
+
+      expect(Object.fromEntries(dirs)).toStrictEqual({
+        // user project: highest priority
+        '<root>/components': 3,
+        '<root>/components/global': 3,
+        '<root>/components/islands': 3,
+        // local layer
+        '<root>/layers/auto/components': 2,
+        '<root>/layers/auto/components/global': 2,
+        '<root>/layers/auto/components/islands': 2,
+        // explicitly extended layer
+        '<root>/custom/components': 1,
+        '<root>/custom/components/global': 1,
+        '<root>/custom/components/islands': 1,
+      })
+    } finally {
+      await nuxt.close()
+    }
   })

---

39-52: Test coverage validates the core fix.

The assertions correctly validate the priority ordering described in the PR: root project (3) > auto-scanned layers (2) > extended layers (1).

For more comprehensive coverage, consider adding tests for edge cases in follow-up work:

• No layers (only root project)
• Multiple auto-scanned layers (verifying alphabetical sort)
• Multiple extended layers (verifying order)
• Only extended layers (no auto-scanned)

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: huang-julien
Repo: nuxt/nuxt PR: 29366
File: packages/nuxt/src/app/components/nuxt-root.vue:16-19
Timestamp: 2024-12-12T12:36:34.871Z
Learning: In `packages/nuxt/src/app/components/nuxt-root.vue`, when optimizing bundle size by conditionally importing components based on route metadata, prefer using inline conditional imports like:

```js
const IsolatedPage = route?.meta?.isolate ? defineAsyncComponent(() => import('#build/isolated-page.mjs')) : null
```

instead of wrapping the import in a computed property or importing the component unconditionally.

Learnt from: CR
Repo: nuxt/nuxt PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-07-18T16:46:07.446Z
Learning: Applies to **/*.{test,spec}.{ts,tsx,js,jsx} : Write unit tests for core functionality using `vitest`

Learnt from: CR
Repo: nuxt/nuxt PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-07-18T16:46:07.446Z
Learning: Applies to **/e2e/**/*.{ts,js} : Write end-to-end tests using Playwright and `nuxt/test-utils`

Learnt from: GalacticHypernova
Repo: nuxt/nuxt PR: 26468
File: packages/nuxt/src/components/plugins/loader.ts:24-24
Timestamp: 2024-11-05T15:22:54.759Z
Learning: In `packages/nuxt/src/components/plugins/loader.ts`, the references to `resolve` and `distDir` are legacy code from before Nuxt used the new unplugin VFS and will be removed.

Learnt from: huang-julien
Repo: nuxt/nuxt PR: 29366
File: packages/nuxt/src/app/components/nuxt-root.vue:16-19
Timestamp: 2024-12-12T12:36:34.871Z
Learning: In `packages/nuxt/src/app/components/nuxt-root.vue`, when optimizing bundle size by conditionally importing components based on route metadata, prefer using inline conditional imports like:

```js
const IsolatedPage = route?.meta?.isolate ? defineAsyncComponent(() => import('#build/isolated-page.mjs')) : null
```

instead of wrapping the import in a computed property or importing the component unconditionally.