feat: enable custom cwd for vite plugin
Context
When using sveltekit in a mono-repository context, the build tends to fail with the following error message:
src/app.html does not exist
The error persists if you change the file location from the svelte config.
This is due to the file resolutions being made from the current directory instead of the project's root directory. While it works totally fine in a standalone project, it can't work in a monorepository context.
The objective of this pull request is to provide an option that specifies the root of the project in the vite adapter and to base the file resolution on the new option if it is provided while keeping the default process.cwd() directory if not provided
Related issues
- https://github.com/nxext/nx-extensions/issues/1058
Note that the code still needs a bit of refactoring and testing
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
- [x] 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
- [x] 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 testand lint the project withpnpm lintandpnpm check
Changesets
- [x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running
pnpm changesetand following the prompts. Changesets that add features should beminorand those that fix bugs should bepatch. Please prefix changeset messages withfeat:,fix:, orchore:.
Edits
- [x] Please ensure that 'Allow edits from maintainers' is checked. PRs without this option may be closed.
🦋 Changeset detected
Latest commit: ff5db2c0e98fd45ee8ea291c514cb73ff350012c
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
Did you see there was already a PR sent for this a few days ago? https://github.com/sveltejs/kit/pull/12410
closing as we concluded it is the responsibility of the monorepo tooling https://github.com/sveltejs/kit/issues/12499#issuecomment-2320091512