fix: use Vite `root` config setting instead of current working directory by teemingc · Pull Request #15469 · sveltejs/kit

teemingc · 2026-03-02T21:01:40Z

closes #11356
closes #12140
closes #15188

This PR standardises the use of process.cwd() to prefer the Vite config root setting when available similar to VPS. This helps us resolve paths correctly before reading files or generating tsconfig paths. It eases lots of monorepo tooling usage, particularly Vitest workspaces. This also allows us to use the Vitest IDE extension to run unit tests (which didn't work before because of the aforementioned issues).

We could backport these changes to v2 but it's a bit easier to base it on v3 which already makes use of import.meta.dirname in many places thanks to #15434

I've only made changes to kit itself for now, so other packages still default to process.cwd() but those are probably less problematic since they won't run for Vitest tests

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
This message body should clearly illustrate what problems it solves.
Ideally, include a test that fails without this PR but passes with it.

Tests

Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Edits

Please ensure that 'Allow edits from maintainers' is checked. PRs without this option may be closed.

changeset-bot · 2026-03-02T21:01:48Z

🦋 Changeset detected

Latest commit: 4728f85

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@sveltejs/kit	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

svelte-docs-bot · 2026-03-02T21:02:56Z

Preview: https://svelte-dev-git-preview-kit-15469-svelte.vercel.app/

teemingc · 2026-03-03T09:17:23Z

 */
-export function get_page_options(filepath) {
+export function get_page_options(filepath, root) {
+	const input = read(path.resolve(root, filepath));


Moved this out of the try...catch to avoid silent errors for files it can't find when calling read. The real issue was that the route filepaths weren't being resolved correctly. That's not an issue anymore now that we resolve against the root.

teemingc · 2026-03-03T09:18:54Z

 			id: '/blog.json',
 			pattern: '/^/blog.json/?$/',
-			endpoint: { file: 'samples/basic/blog.json/+server.js', page_options: null }
+			endpoint: { file: 'samples/basic/blog.json/+server.js', page_options: {} }


This fixes the test results to match the change from https://github.com/sveltejs/kit/pull/15469/changes#diff-8701fabc771d319105f7401163b7bc28189c5a11e30706215a62fb303385797a . Previously, the page options were incorrectly computed as null (unable to statically analyse). However, the correct value is {} (which means no page options exported).

teemingc · 2026-03-04T13:24:09Z


 fs.writeFileSync(
-	'./src/version.js',
+	path.join(import.meta.dirname, '..', 'src', 'version.js'),


Helps the version generation test run from the Vitest extension by avoiding resolving to the wrong path

teemingc · 2026-03-04T13:27:41Z

-		const process = await import('node:process');

-		relative = (file) => path.relative(process.cwd(), file);
+		relative = (file) => path.relative('.', file);


Apparently '.' is equivalent to process.cwd() so that helps simplify things here

teemingc · 2026-03-04T13:28:56Z

+	it('should be the exact version from package.json', async () => {
+		const { VERSION } = await import(join(import.meta.dirname, 'version.js'));
+		const pkg = JSON.parse(
+			readFileSync(fileURLToPath(new URL('../package.json', import.meta.url)), 'utf-8')
+		);
+		assert.equal(
+			VERSION,
+			pkg.version,
+			'VERSION export in src/version.js does not equal version in package.json'
+		);
+	});


Had to put the test logic inside of it so Vitest would stop skipping it from the IDE extension

teemingc · 2026-03-04T13:29:21Z

+
+		typescript: {
+			config: (config) => {
+				config.include.push('../unit-test');


Avoids a typescript error from displaying in the unit test file

benmccann

needs a changeset, but besides that it looks good to me. will this result in any changes that users of vite can see or only vitest? it might be good to call this a feat:/minor so that it's more prominent in the changelog as I imagine it will be a semi-impactful change for a subset of users

Co-authored-by: Tee Ming <chewteeming01@gmail.com>

This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to version-3, this PR will be updated. ⚠️

⚠️

⚠️ `version-3` is currently in **pre mode** so this branch has prereleases rather than normal releases. If you want to exit prereleases, run `changeset pre exit` on `version-3`. ⚠️

⚠️

⚠️ # Releases ## @sveltejs/adapter-auto@8.0.0-next.0 ### Major Changes - breaking: require SvelteKit 3 ([#15506](#15506)) ### Patch Changes - Updated dependencies [[`fa335bd`](fa335bd), [`3031d89`](3031d89), [`cb9d416`](cb9d416), [`caf3a18`](caf3a18), [`4777827`](4777827), [`a2792e2`](a2792e2), [`ba36148`](ba36148), [`48e8710`](48e8710), [`e2f3075`](e2f3075), [`047d6a0`](047d6a0), [`87603d1`](87603d1), [`096962c`](096962c), [`d545970`](d545970), [`e2f3075`](e2f3075), [`d06affc`](d06affc), [`8af47eb`](8af47eb), [`e2f3075`](e2f3075), [`5c4d130`](5c4d130), [`3f11f35`](3f11f35), [`caf3a18`](caf3a18), [`8823037`](8823037), [`1d76212`](1d76212), [`0dc0548`](0dc0548), [`00d81fa`](00d81fa)]: - @sveltejs/kit@3.0.0-next.0 ## @sveltejs/adapter-cloudflare@8.0.0-next.0 ### Major Changes - breaking: upgrade `@cloudflare/workers-types` to 4.20260219.0 ([#15347](#15347)) - breaking: upgrade minimum `wrangler` version to ^4.67.0 ([#15347](#15347)) - breaking: remove `platform.context` in favour of `platform.ctx` ([#15347](#15347)) - breaking: require SvelteKit 3 ([#15506](#15506)) ### Patch Changes - chore: check the `WORKERS_CI` environment variable to determine if we're building for Cloudflare Workers ([#13733](#13733)) - Updated dependencies [[`fa335bd`](fa335bd), [`3031d89`](3031d89), [`cb9d416`](cb9d416), [`caf3a18`](caf3a18), [`4777827`](4777827), [`a2792e2`](a2792e2), [`ba36148`](ba36148), [`48e8710`](48e8710), [`e2f3075`](e2f3075), [`047d6a0`](047d6a0), [`87603d1`](87603d1), [`096962c`](096962c), [`d545970`](d545970), [`e2f3075`](e2f3075), [`d06affc`](d06affc), [`8af47eb`](8af47eb), [`e2f3075`](e2f3075), [`5c4d130`](5c4d130), [`3f11f35`](3f11f35), [`caf3a18`](caf3a18), [`8823037`](8823037), [`1d76212`](1d76212), [`0dc0548`](0dc0548), [`00d81fa`](00d81fa)]: - @sveltejs/kit@3.0.0-next.0 ## @sveltejs/adapter-netlify@7.0.0-next.0 ### Major Changes - chore: use `rolldown` for edge function bundling ([#15432](#15432)) - breaking: write output that conforms to the stable [Netlify Frameworks API](https://docs.netlify.com/build/frameworks/frameworks-api/). ([#15294](#15294)) Deploying and previewing with Netlify CLI now requires [v17.31.0](https://github.com/netlify/cli/releases/tag/v17.31.0) or later. Run `npm i -g netlify-cli@latest` to upgrade. - breaking: require SvelteKit 3 ([#15506](#15506)) - breaking: edge function build target is now `es2022` ([#15432](#15432)) ### Patch Changes - Updated dependencies [[`fa335bd`](fa335bd), [`3031d89`](3031d89), [`cb9d416`](cb9d416), [`caf3a18`](caf3a18), [`4777827`](4777827), [`a2792e2`](a2792e2), [`ba36148`](ba36148), [`48e8710`](48e8710), [`e2f3075`](e2f3075), [`047d6a0`](047d6a0), [`87603d1`](87603d1), [`096962c`](096962c), [`d545970`](d545970), [`e2f3075`](e2f3075), [`d06affc`](d06affc), [`8af47eb`](8af47eb), [`e2f3075`](e2f3075), [`5c4d130`](5c4d130), [`3f11f35`](3f11f35), [`caf3a18`](caf3a18), [`8823037`](8823037), [`1d76212`](1d76212), [`0dc0548`](0dc0548), [`00d81fa`](00d81fa)]: - @sveltejs/kit@3.0.0-next.0 ## @sveltejs/adapter-node@6.0.0-next.0 ### Major Changes - chore: migrate from rollup to rolldown ([#15297](#15297)) - breaking: require SvelteKit 3 ([#15506](#15506)) ### Patch Changes - Updated dependencies [[`fa335bd`](fa335bd), [`3031d89`](3031d89), [`cb9d416`](cb9d416), [`caf3a18`](caf3a18), [`4777827`](4777827), [`a2792e2`](a2792e2), [`ba36148`](ba36148), [`48e8710`](48e8710), [`e2f3075`](e2f3075), [`047d6a0`](047d6a0), [`87603d1`](87603d1), [`096962c`](096962c), [`d545970`](d545970), [`e2f3075`](e2f3075), [`d06affc`](d06affc), [`8af47eb`](8af47eb), [`e2f3075`](e2f3075), [`5c4d130`](5c4d130), [`3f11f35`](3f11f35), [`caf3a18`](caf3a18), [`8823037`](8823037), [`1d76212`](1d76212), [`0dc0548`](0dc0548), [`00d81fa`](00d81fa)]: - @sveltejs/kit@3.0.0-next.0 ## @sveltejs/adapter-static@4.0.0-next.0 ### Major Changes - breaking: require SvelteKit 3 ([#15506](#15506)) ### Patch Changes - Updated dependencies [[`fa335bd`](fa335bd), [`3031d89`](3031d89), [`cb9d416`](cb9d416), [`caf3a18`](caf3a18), [`4777827`](4777827), [`a2792e2`](a2792e2), [`ba36148`](ba36148), [`48e8710`](48e8710), [`e2f3075`](e2f3075), [`047d6a0`](047d6a0), [`87603d1`](87603d1), [`096962c`](096962c), [`d545970`](d545970), [`e2f3075`](e2f3075), [`d06affc`](d06affc), [`8af47eb`](8af47eb), [`e2f3075`](e2f3075), [`5c4d130`](5c4d130), [`3f11f35`](3f11f35), [`caf3a18`](caf3a18), [`8823037`](8823037), [`1d76212`](1d76212), [`0dc0548`](0dc0548), [`00d81fa`](00d81fa)]: - @sveltejs/kit@3.0.0-next.0 ## @sveltejs/adapter-vercel@7.0.0-next.0 ### Major Changes - chore: use `rolldown` for edge function bundling ([#15432](#15432)) - breaking: edge function build target is now `es2022` ([#15432](#15432)) ### Patch Changes - Updated dependencies [[`fa335bd`](fa335bd), [`3031d89`](3031d89), [`cb9d416`](cb9d416), [`caf3a18`](caf3a18), [`4777827`](4777827), [`a2792e2`](a2792e2), [`ba36148`](ba36148), [`48e8710`](48e8710), [`e2f3075`](e2f3075), [`047d6a0`](047d6a0), [`87603d1`](87603d1), [`096962c`](096962c), [`d545970`](d545970), [`e2f3075`](e2f3075), [`d06affc`](d06affc), [`8af47eb`](8af47eb), [`e2f3075`](e2f3075), [`5c4d130`](5c4d130), [`3f11f35`](3f11f35), [`caf3a18`](caf3a18), [`8823037`](8823037), [`1d76212`](1d76212), [`0dc0548`](0dc0548), [`00d81fa`](00d81fa)]: - @sveltejs/kit@3.0.0-next.0 ## @sveltejs/enhanced-img@1.0.0-next.0 ### Major Changes - breaking: require Node 22 or newer ([#12548](#12548)) ### Minor Changes - breaking: require Vite 8 and `vite-plugin-svelte` 7 ([#15542](#15542)) ## @sveltejs/kit@3.0.0-next.0 ### Major Changes - breaking: TypeScript 6 is now the minimum required version ([#15930](#15930)) - breaking: upgrade to cookie v1. Cookie names must now contain only ASCII characters ([#13386](#13386)) - breaking: require Node 22 or newer ([#12548](#12548)) - breaking: remove the `preloadStrategy` option. `modulepreload` will always be used ([#15256](#15256)) - breaking: default the cookie `path` option to `'/'` ([#15398](#15398)) - breaking: remove `@sveltejs/kit/node/polyfills` ([#15430](#15430)) - breaking: add `config.kit.output.linkHeaderPreload` to preload using the `Link` header ([#15939](#15939)) - breaking: require `@sveltejs/vite-plugin-svelte` v7 ([#15371](#15371)) - breaking: remove `createEntries` from the `Builder` object passed to adapter functions ([#15509](#15509)) - breaking: remove the deprecated CSRF `checkOrigin` option in favor of `trustedOrigins` ([#15437](#15437)) - breaking: the `delta` property now only exists for `popstate` navigation events ([#15522](#15522)) - breaking: remove deprecated `pragma` header in version polling for improved CORS support ([#15428](#15428)) - breaking: require Svelte 5.48.0 or newer ([#15371](#15371)) - chore: change `error`, `isHttpError`, `redirect`, and `isRedirect` to refer to public type instead of internal class ([#15250](#15250)) - breaking: require Vite 8. Provides new functionality even for existing Vite 8 users such as faster builds with Vite hook filters and more powerful SvelteKit adapters with the Vite environment API ([#15371](#15371)) - breaking: remove `data-sveltekit-*` option `'off'` in favour of `false` ([#15907](#15907)) ### Minor Changes - feat: resolve paths using the Vite config `root` option instead of `process.cwd()` to better support monorepo configurations such as Vitest workspaces ([#15469](#15469)) - chore: deprecate `Response` helpers in favor of platform-provided alternatives ([#15448](#15448)) - feat: explicit env vars ([#15934](#15934)) ### Patch Changes - fix: remove check for svelte.config.js before running `sync` ([#15946](#15946)) - fix: generate a placeholder tsconfig.json to squelch sync-time warnings ([#15948](#15948)) - chore: remove dependency on kleur ([#12548](#12548)) - chore: remove dependency on `set-cookie-parser` ([#15384](#15384)) - fix: allow use of `$app/env/public` in service workers ([#15950](#15950)) ## @sveltejs/package@3.0.0-next.0 ### Major Changes - breaking: require Node 22 or newer ([#12548](#12548)) ### Patch Changes - chore: remove dependency on kleur ([#12548](#12548)) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

teemingc added 2 commits March 3, 2026 04:41

fix

7d3ab36

uncomment

abee42a

teemingc changed the base branch from main to version-3 March 2, 2026 21:01

teemingc added 9 commits March 3, 2026 05:20

more resolves to root

9b09666

fix sync-all

1d9d811

revert unrelated change

7a6f301

revert unnessecary change

5ae0182

use path joiner

32aef73

normalized node modules

7d6eca9

fix build-errors

9656454

fix other tests

be9f93b

add extends

2360bfc

teemingc commented Mar 3, 2026

View reviewed changes

teemingc added 8 commits March 3, 2026 17:49

move comment

02d6397

fix up vitest extension run

221cd2d

fix types

5a227e1

remove console.log

07bc942

add explanation

cde2bf6

more explanations

46356c5

add configResolved hook

4a4957d

revert package changes

8477013

teemingc marked this pull request as ready for review March 4, 2026 13:22

teemingc commented Mar 4, 2026

View reviewed changes

rename to root

9ca38ee

teemingc commented Mar 4, 2026

View reviewed changes

rename to root

732df04

vercel Bot reviewed Mar 4, 2026

View reviewed changes

Comment thread packages/kit/src/core/utils.js Outdated

teemingc added 7 commits March 4, 2026 21:36

don't cache

7cfdb10

fix logic

d1f3c37

restore old server guard logic

3002be7

fix vitest configs

856f82e

fix vitest config issues

6102baf

Merge branch 'version-3' into fix-vite-workspaces

8b03c90

fix format

79057c6

Rich-Harris approved these changes Mar 5, 2026

View reviewed changes

benmccann reviewed Mar 5, 2026

View reviewed changes

Comment thread packages/adapter-auto/vitest.config.js Outdated

teemingc added 2 commits March 6, 2026 03:36

single line

f891fe2

changeset

d96f13f

teemingc requested a review from benmccann March 5, 2026 19:37

benmccann reviewed Mar 5, 2026

View reviewed changes

Comment thread .changeset/nasty-impalas-wear.md Outdated

Update .changeset/nasty-impalas-wear.md

4728f85

Co-authored-by: Tee Ming <chewteeming01@gmail.com>

benmccann approved these changes Mar 5, 2026

View reviewed changes

teemingc merged commit 8af47eb into version-3 Mar 5, 2026
18 of 19 checks passed

teemingc deleted the fix-vite-workspaces branch March 5, 2026 20:16

github-actions Bot mentioned this pull request Mar 5, 2026

Version Packages (next) #15311

Merged

axel7083 mentioned this pull request Mar 8, 2026

Sveltekit plugin breaks vitest workspaces #11356

Closed

teemingc mentioned this pull request Mar 10, 2026

Ability to use Sveltekit in a mono-repository context such as with Nx #12499

Closed

Rich-Harris mentioned this pull request Apr 27, 2026

feat: fetchable dev environments #15574

Open

6 tasks

teemingc mentioned this pull request May 4, 2026

fix: use root for kit.env.dir fallback #15792

Open

6 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: use Vite `root` config setting instead of current working directory#15469

fix: use Vite `root` config setting instead of current working directory#15469
teemingc merged 31 commits into
version-3from
fix-vite-workspaces

teemingc commented Mar 2, 2026 •

edited

Loading

Uh oh!

changeset-bot Bot commented Mar 2, 2026 •

edited

Loading

Uh oh!

svelte-docs-bot Bot commented Mar 2, 2026

Uh oh!

teemingc Mar 3, 2026

Uh oh!

teemingc Mar 3, 2026

Uh oh!

teemingc Mar 4, 2026

Uh oh!

teemingc Mar 4, 2026

Uh oh!

teemingc Mar 4, 2026

Uh oh!

teemingc Mar 4, 2026

Uh oh!

Uh oh!

benmccann left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

teemingc commented Mar 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

Tests

Changesets

Edits

Uh oh!

changeset-bot Bot commented Mar 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

svelte-docs-bot Bot commented Mar 2, 2026

Uh oh!

teemingc Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

teemingc Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

teemingc Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

teemingc Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

teemingc Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

teemingc Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

benmccann left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

teemingc commented Mar 2, 2026 •

edited

Loading

changeset-bot Bot commented Mar 2, 2026 •

edited

Loading