[v5] Error on build adonis with 'pino-pretty'

[satheler]

Dependencies versions

@adonisjs/ace: 6.0.0
@adonisjs/auth: 4.3.1
@adonisjs/core: 5.0.0-preview-rc-1.12
@adonisjs/fold: 6.0.0
@adonisjs/lucid: 8.5.0
@adonisjs/mail: 4.2.2
@adonisjs/view: 2.0.9

Steps (when all dependencies installed):

1. node ace build --production
2. Move build folder to another location (outside the project folder) and remove node_modules folder
3. yarn install --production
4. node server.js

Error: Missing `pino-pretty` module: `pino-pretty` must be installed separately
    at getPrettyStream (/home/satheler/Documents/my_project/repos/build/node_modules/pino/lib/tools.js:188:11)
    at normalizeArgs (/home/satheler/Documents/my_project/repos/build/node_modules/pino/lib/tools.js:351:16)
    at Object.pino [as default] (/home/satheler/Documents/my_project/repos/build/node_modules/pino/pino.js:78:28)
    at Object.getPino (/home/satheler/Documents/my_project/repos/build/node_modules/@adonisjs/logger/build/src/getPino.js:22:89)
    at new Logger (/home/satheler/Documents/my_project/repos/build/node_modules/@adonisjs/logger/build/src/Logger.js:49:43)
    at Object.callback (/home/satheler/Documents/my_project/repos/build/node_modules/@adonisjs/logger/build/providers/LoggerProvider.js:32:20)
    at Ioc.resolveBinding (/home/satheler/Documents/my_project/repos/build/node_modules/@adonisjs/fold/build/src/Ioc/index.js:94:51)
    at Ioc.resolve (/home/satheler/Documents/my_project/repos/build/node_modules/@adonisjs/fold/build/src/Ioc/index.js:133:29)
    at Ioc.use (/home/satheler/Documents/my_project/repos/build/node_modules/@adonisjs/fold/build/src/Ioc/index.js:290:26)
    at Object.callback (/home/satheler/Documents/my_project/repos/build/node_modules/@adonisjs/lucid/build/providers/DatabaseProvider.js:24:43)

Why do you need to move build folder to another location?

This example to move folder is like happening working with docker.
I'm creating a docker container with adonis project and I am copy only build folder to inside docker.
After, I run command to install dependencies and start server.

Dockerfile example

FROM node:lts
ADD ./build ./
RUN yarn install --production
CMD node server.js

[rrmesquita]

Hey @satheler I've fixed this by moving the pino-pretty package from devDependencies to production dependencies.

I'm also using Docker, I can share my Dockerfile if you want.

[satheler]

Share you Dockerfile with me

[rrmesquita]

@satheler

FROM node:12 AS basic
RUN mkdir -p /home/node/app/node_modules
WORKDIR /home/node/app
EXPOSE $PORT

FROM basic AS builder
COPY package.json ./
COPY . .
RUN chown -R node:node /home/node
RUN npm install
USER node
RUN npm run build
CMD [ "npm", "run", "dev" ]

FROM basic as production
COPY --from=builder /home/node/app/build .
COPY .env .
RUN chown -R node:node /home/node
RUN npm install
USER node
CMD [ "npm", "run", "start" ]

[thetutlage]

If you are running your app in production, then you have to tell the code as well. As there is no way for Javascript to know if it is running in prod machine or a dev machine.

Set the NODE_ENV = 'production' before starting the server and the need of using pino-pretty will disappear as well

[satheler]

@thetutlage Thank you! That was missing this environment variable. Now it's working!

[ebauger]

Thank @thetutlag, it was useful for me too.