-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
missing import in svelte prod #240
Comments
I've seen something similar during
Never payed much attention to it, as usually a simple server restart helped. Or did I understand your issue wrong? Is there a real runtime error or something? |
Sorry for the late reply. Yes, it is a real runtime error. Worked fine in dev locally but i guess for prod it strips everything unnecessary and this thing is wrongly removed. I only managed to reproduce it using this dockerfile which i use in prod ( FROM node:20 AS builder
WORKDIR /app
COPY package*.json .
RUN npm ci
COPY . .
ARG PGHOST PGPORT PGDATABASE PGUSER PGPASSWORD
RUN PGHOST=$PGHOST PGPORT=$PGPORT PGDATABASE=$PGDATABASE PGUSER=$PGUSER PGPASSWORD=$PGPASSWORD \
npx pgtyped -c pgtyped.json
RUN NODE_OPTIONS="--max-old-space-size=2048" npm run build
RUN npm prune --production
FROM node:20
WORKDIR /app
COPY --from=builder /app/build build/
COPY --from=builder /app/node_modules node_modules/
COPY package.json .
EXPOSE 3000
ENV NODE_ENV=production
CMD [ "node", "build" ] |
Not sure what to do here. I don't even think I fully understand the issue. Any help would be appreciated and PRs welcome. |
In simple terms, when i build for prod using the Dockerfile above, I get a dependency error at runtime. It can be bypassed by doing |
@joleeee sounds like a builder's issue. Or a Docker one. This definitely doesn't happen in simpler cases. If you were able to reproduce it somewhere where I'd have an access, I could take a look at it. But I'm not sure if there's something like repl/stackblitz/you-name-it available which supports Docker. Otherwise, in Vite-based projects without Docker, I can't confirm that something like this happens. |
Feel free to reopen when there's a repro. |
I think #246 mat be relatable here. The issue should be fixed in v0.7.2. |
I can navigate to the page which uses this library, but if i do a refresh or type the url directly, then i get an internal server error and this is spat out:
Working workaround (breaks typing!):
Perhaps everything needed is an
export
part in thepackage.json
? I am unable to test this because if I use a local version of the library, then I don't get the error.This seems very relevant: katspaugh/wavesurfer.js#3066 (reply in thread) and katspaugh/wavesurfer.js@2313458
Note that this bug does not occur when using
npm run dev
orpreview
for me.The text was updated successfully, but these errors were encountered: