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

npm run start:prod is failed, unnecessary modules required (newly created project) v5 #594

Closed
VMois opened this issue Apr 23, 2018 · 18 comments

Comments

@VMois
Copy link

VMois commented Apr 23, 2018

I'm submitting a...


[ ] Regression 
[x] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

Current behavior

After setting up a project with CLI as described in docs (without changing code), I have tried to run a command 'npm run start:prod' and get this error:

> test@0.0.1 prestart:prod /home/vmois/Projects/Portal/test
> rm -rf dist && tsc

node_modules/@nestjs/common/interfaces/microservices/microservice-configuration.interface.d.ts(3,32): error TS2307: Cannot find module 'mqtt'.
node_modules/@nestjs/common/interfaces/microservices/microservice-configuration.interface.d.ts(4,35): error TS2307: Cannot find module 'grpc'.

npm ERR! Linux 3.16.0-5-amd64
npm ERR! argv "/usr/bin/nodejs" "/usr/local/bin/npm" "run" "start:prod"
npm ERR! node v8.4.0
npm ERR! npm  v4.0.5
npm ERR! code ELIFECYCLE
npm ERR! test@0.0.1 prestart:prod: `rm -rf dist && tsc`
npm ERR! Exit status 2
npm ERR! 
npm ERR! Failed at the test@0.0.1 prestart:prod script 'rm -rf dist && tsc'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the test package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     rm -rf dist && tsc
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs test
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls test
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/vmois/Projects/Portal/test/npm-debug.log

Of course the easiest solution is to install these packages, but what if I don't use them and don't want to install. Thanks.

Expected behavior

Compile and server in dist/main.js

Minimal reproduction of the problem with instructions

Try to setup up project with CLI, and then run 'npm run start:prod'

Environment


[Nest Information]
microservices version : 5.0.0-beta.3
websockets version    : 5.0.0-beta.3
testing version       : 5.0.0-beta.3
common version        : 5.0.0-beta.3
core version          : 5.0.0-beta.3

 
For Tooling issues:
- Node version: 8.4.0
- Platform: Linux

Others:
Package manager: yarn
@sgomanf
Copy link

sgomanf commented Apr 23, 2018

same issue

@VMois
Copy link
Author

VMois commented Apr 24, 2018

npm run start:prod command works in NestJS v4.5.9, so I think it is an issue related to v5.

@VMois VMois changed the title npm run start:prod is failed after CLI new project setup npm run start:prod is failed, unnecessary modules required (newly create project) v5 Apr 24, 2018
@VMois VMois changed the title npm run start:prod is failed, unnecessary modules required (newly create project) v5 npm run start:prod is failed, unnecessary modules required (newly created project) v5 Apr 24, 2018
@kamilmysliwiec
Copy link
Member

This issue comes from Nest v5.0.0. Thanks for reporting!

@kamilmysliwiec
Copy link
Member

Fixed in v5.0.0-beta.4. Let me know if you face any issue. 🙂

@sgomanf
Copy link

sgomanf commented Apr 29, 2018

It works thanks

@robertmain
Copy link

I'm having this issue with 5.1.0

@arfaWong
Copy link

same issue with 5.1.0.

@moltar
Copy link

moltar commented Nov 5, 2018

Same here!

@kamilmysliwiec
Copy link
Member

I would recommend updating your packages to 5.4.0, there were no breaking changes.

@moltar
Copy link

moltar commented Nov 6, 2018

Updated to 5.4.0, but still experiencing this issue.

@kamilmysliwiec
Copy link
Member

The issue is definitely not present in 5.4.0. Please, ensure that all packages were updated. If you prefer, you can create a dedicated issue on StackOverflow.

@kumanan12
Copy link

kumanan12 commented Dec 17, 2018

I am still seeing the same issue mentioned above. Its very easy to reproduce. I am running 5.4 version.

Just create a project and run "npm run start:prod"

@y12studio
Copy link

Just for someone else experimenting this issue, my workaround was to modify the path of main.js :

sed -i.bak 's|dist/main.js|dist/src/main.js|g' package.json

@baddlan
Copy link

baddlan commented Feb 13, 2019

If you happen to build your project in a Docker container and then run it in a different one for use in production, then don't use npm run start:prod because it counter-intuitively deletes you dist directory.

It must have been an idea for an April fools' day prank. 🤔

Just call node dist/main.js directly.

@robertmain
Copy link

The idea is to clean the built assets out before building new ones. This is to ensure you aren't running old code

@nigelvon
Copy link

nigelvon commented Jul 4, 2019

Same issue with v6.4.1

@marluanespiritusanto
Copy link

If you happen to build your project in a Docker container and then run it in a different one for use in production, then don't use npm run start:prod because it counter-intuitively deletes you dist directory.

It must have been an idea for an April fools' day prank. 🤔

Just call node dist/main.js directly.

It works for me :)

@lock
Copy link

lock bot commented Oct 16, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests