Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Should be provided by compiler" Error #77

Closed
sabercoy opened this issue Jan 1, 2024 · 9 comments
Closed

"Should be provided by compiler" Error #77

sabercoy opened this issue Jan 1, 2024 · 9 comments

Comments

@sabercoy
Copy link

sabercoy commented Jan 1, 2024

Initializing a basic SolidStart project, then running build and start will result in
image

This also occurs with vinxi version 0.0.62

@brenelz
Copy link

brenelz commented Jan 1, 2024

Guessing this is using npm instead of pnpm?

@sabercoy
Copy link
Author

sabercoy commented Jan 1, 2024

Guessing this is using npm instead of pnpm?

Yes! should I be using pnpm?

@brenelz
Copy link

brenelz commented Jan 1, 2024

I know there was a similar issue with npm previously but i thought it was fixed. pnpm seems to work i think.

Ideally it would work with npm as well

@sabercoy
Copy link
Author

sabercoy commented Jan 1, 2024

I know there was a similar issue with npm previously but i thought it was fixed. pnpm seems to work i think.

Ideally it would work with npm as well

Yes it looks like pnpm works.
I believe I have other npm related bugs to report, so I wanted to make sure it was not expected to only use pnpm.

@edivados
Copy link
Contributor

edivados commented Jan 2, 2024

For whatever reason npm installs two vinxi versions. Maybe because solid-start doesn't declare vinxi as peer dep?

  1. The one in your projects package.json (whatever that is)
    • /node_modules/vinxi
  2. The one solid-start pulls in (0.0.54 - assuming you are using the latest release)
    • /node_modules/@solidjs/start/node_modules/vinxi

Number 2 is what is being used during build and leads to no routes config being generated.

One thing you could do is change vinxi to 0.0.54 in your package.json or use overrides to force a specific vinxi version.

@sabercoy
Copy link
Author

sabercoy commented Jan 2, 2024

For whatever reason npm installs two vinxi versions. Maybe because solid-start doesn't declare vinxi as peer dep?

  1. The one in your projects package.json (whatever that is)

    • /node_modules/vinxi
  2. The one solid-start declares in devDependencies (0.0.54 - assuming you are using the latest release)

    • /node_modules/@solidjs/start/node_modules/vinxi

Number 2 is what is being used during build and leads to no routes config being generated.

One thing you could do is change vinxi to 0.0.54 in your package.json or use overrides to force a specific vinxi version.

This does look to be the case. There is a mismatch in versions between those two locations and the behavior stays the same no matter which version I npm install (meaning it only uses the one solid-start declares).

I wanted to use greater than 0.0.54 because better CSS Modules support was added. I am not sure about overrides, but pnpm seems to be working great at the moment.

@LexSwed
Copy link

LexSwed commented Jan 2, 2024

overrides should work for now, otherwise we need to wait for Ryan to cut a new release of @solidjs/start with vinxi updates.

@nksaraf
Copy link
Owner

nksaraf commented Jan 2, 2024

yup a new solid-start release is needed for this

@nksaraf
Copy link
Owner

nksaraf commented Jan 11, 2024

Should be fixed in latest versions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants