Run & review this pull request inStackBlitz Codeflow.

Open in StackBlitz

npm i https://pkg.pr.new/@nuxt/kit@33858

npm i https://pkg.pr.new/@nuxt/nitro-server@33858

npm i https://pkg.pr.new/nuxt@33858

npm i https://pkg.pr.new/@nuxt/rspack-builder@33858

npm i https://pkg.pr.new/@nuxt/schema@33858

npm i https://pkg.pr.new/@nuxt/vite-builder@33858

npm i https://pkg.pr.new/@nuxt/webpack-builder@33858

commit:21a5c0c

Walkthrough

This pull request modifies the loader configuration inpackages/kit/src/loader/config.ts to adjust trailing slash handling for local layers. The change replaces a single import from ufo with bothwithTrailingSlash andwithoutTrailingSlash utilities. The normalisation oflocalLayers switches from removing trailing slashes to explicitly enforcing them viawithTrailingSlash. Conversely, the construction oflocalRelativePaths now removes trailing slashes fromlocalLayers usingwithoutTrailingSlash, reversing the previous path comparison logic. No public API signatures are affected.

Possibly related PRs

• fix(kit,nuxt,schema): add trailing slash to some dir aliases #32755: Modifies the same trailing-slash normalisation inpackages/kit/src/loader/config.ts usingwithTrailingSlash andwithoutTrailingSlash utilities.
• perf(kit,nuxt): don't resolve paths from local layers/modules #30650: Adjusts how local layer and module paths are normalised and compared through prefix-based matching with trailing-slash normalisation.
• fix(kit): ensure local layers are prioritised alphabetically #33030: Modifiespackages/kit/src/loader/config.ts to alter layer normalisation alongside changes to layer ordering logic.

Pre-merge checks and finishing touches

issue_enrichment:auto_enrich:enabled:false

Thanks for usingCodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment@coderabbitai help to get the list of available commands and usage tips.}

CodSpeed Performance Report

Merging#33858 willimprove performances by 12.17%

_{Comparingrywils:dotted-layer-directories (21a5c0c) withmain (c1b3590)¹}

Summary

⚡ 1 improvement
✅ 9 untouched

Benchmarks breakdown

I'm not sure about tests but is it possible to have test case for numbered layers directories?

Hey@danielroe

Is it possible to change order of layers viaextends so we don't have to rename directories when we want to add new layer which required to be placed between existing layers.

Benefits:

• When using template repo it doesn't break pull updates
• When checking repo history it doesn't confuse

I would love to have this feature, Should I raise a feature request for this?

unfortunately that's not possible with local layers.

if you want that, move them out of thelayers directory and register them withextends instead. we can't mix + match.