config.kit.target is no longer required, and should be removed

[davehorner]

Describe the bug

pnpm run sdev

> [email protected] sdev C:\Users\dhorner\work\tauri\c
> svelte-kit dev

> config.kit.target is no longer required, and should be removed
    at file:///C:/Users/dhorner/work/tauri/c/node_modules/.pnpm/@[email protected][email protected]/node_modules/@sveltejs/kit/dist/cli.js:874:10
    at file:///C:/Users/dhorner/work/tauri/c/node_modules/.pnpm/@[email protected][email protected]/node_modules/@sveltejs/kit/dist/cli.js:766:43
    at file:///C:/Users/dhorner/work/tauri/c/node_modules/.pnpm/@[email protected][email protected]/node_modules/@sveltejs/kit/dist/cli.js:752:18
    at file:///C:/Users/dhorner/work/tauri/c/node_modules/.pnpm/@[email protected][email protected]/node_modules/@sveltejs/kit/dist/cli.js:752:18
    at validate_config (file:///C:/Users/dhorner/work/tauri/c/node_modules/.pnpm/@[email protected][email protected]/node_modules/@sveltejs/kit/dist/cli.js:936:9)
    at load_config (file:///C:/Users/dhorner/work/tauri/c/node_modules/.pnpm/@[email protected][email protected]/node_modules/@sveltejs/kit/dist/cli.js:913:20)
    at async file:///C:/Users/dhorner/work/tauri/c/node_modules/.pnpm/@[email protected][email protected]/node_modules/@sveltejs/kit/dist/cli.js:1013:19
 ELIFECYCLE  Command failed with exit code 1.

Reproduction

https://github.com/tauri-apps/tauri/tree/dev/tooling/create-tauri-app creates a package.json that has sdev and sbuild. I was trying to run them on my machine. pnpm run sdev

config.kit.target is no longer required, and should be removed
I don't know how to remove this setting.

Logs

No response

System Info

System:
    OS: Windows 10 10.0.19044
  Binaries:
    Node: 15.14.0 - C:\ProgramData\Miniconda3\node.EXE
    Yarn: 1.22.10 - C:\ProgramData\Miniconda3\yarn.CMD
    npm: 7.19.1 - C:\ProgramData\Miniconda3\npm.CMD
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.17
    @sveltejs/kit: next => 1.0.0-next.259
    svelte: ^3.44.0 => 3.46.3

Severity

blocking all usage of SvelteKit

Additional Information

config.kit.target is no longer required, and should be removed

[ignatiusmb]

As the error says, remove target option from the config

[IamSheikh]

Thank you sooo much

[nberlette]

@davehorner try removing the target option under kit in your project's root svelte.config.js.

Maybe the error message should clarify that it's talking about svelte.config.js? The config.kit.target path might be obvious to seasoned users, but with no reference to a filename or error stack, many users might be confused on exactly which config file needs amending.

P.S. wouldn't it be... nicer to spit out a deprecation warning - but continuing to build - rather than a hard failure? SvelteKit has done this before with many other methods / properties. Seems odd to change the pattern now.

[davehorner]

I think if "svelte.config.js" was added to that message it would be clear. I'm new :)

still running into other errors. looks like the code generated is old from tauri.

request in handle has been replaced with event. See #3384 for details Error: request in handle has been replaced with event. See #3384 for details

$page.path has been replaced by $page.url.pathname Error: $page.path has been replaced by $page.url.pathname

[Yakumwamba]

Hi , so I have a question regarding the removal of the config.kit.target, how does SvelteKit hydrate the

element in src/app.html?

Thanks, Im new here.

[xpat]

@Yakumwamba This may be the answer to your question: #3674

use data-hydrate instead of explicit target https://github.com/sveltejs/kit/pull/3674/commits/f9d529fc558e34e3842af0f28da8e1b106c23644

[Yakumwamba]

Hey xpat, Thanks for the anwer. I removed config.target and I have no errors