Updated build process

Author: clintonwoo

.dockerignore

@@ -1,5 +1,7 @@
+.git
+.next
+build
+dist
+docs
 node_modules
 npm-debug.log
-build/static
-docs
-src/.next

.gitignore

@@ -4,8 +4,6 @@
 .vscode
 *.DS_Store
 *.log
-build/app
-build/static
 coverage
 dist
 logs

Dockerfile

@@ -1,37 +1,46 @@
 # DEV BUILD STEP
 FROM node:12.16.1-alpine3.11 as devBuild
-WORKDIR /usr/src/app
+WORKDIR /app
 
 # Log the settings for NPM and Environment variables
+ENV NODE_ENV=production
 RUN npm config ls
 RUN env
 
 # Copy the source code and build
 COPY . .
-RUN npm install
-RUN npm run build
+RUN ls -a
+RUN npm install --production=false
+RUN npm run build:prod
+RUN ls -a
 
 # PROD BUILD STEP
 # Using latest LTS release of Node
 FROM node:12.16.1-alpine3.11
 
 # Create an app directory on the container
-WORKDIR /usr/src/app
+WORKDIR /app
 ENV NODE_ENV=production
 
 # Project copy build, install only prod dependencies
-COPY --from=devBuild /usr/src/app/dist ./dist
-COPY package.json package-lock.json README.md ./
+COPY --from=devBuild /app/dist ./dist
+COPY --from=devBuild /app/.next ./.next
+COPY --from=devBuild /app/public ./public
+COPY --from=devBuild /app/next.config.js ./next.config.js
+COPY package.json package-lock.json healthcheck.js ./
+
+RUN ls -a
+
 RUN npm install --only=prod
 
-# Install curl to do healthchecks
-RUN apk add curl --no-cache
+RUN npx next telemetry disable
 
 # Expose the container port to the OS
 # docker run takes -p argument to forward this port to network
 EXPOSE 3000
 
 # Start the application
-CMD npm run start:production
+CMD npm run start:prod
 
-HEALTHCHECK CMD curl --silent --fail http://localhost:3000/ || exit 1
+HEALTHCHECK --interval=30s --timeout=12s --start-period=30s \  
+  CMD node /healthcheck.js

babel.config.js

@@ -3,18 +3,7 @@ module.exports = (api) => {
   api.cache(false);
 
   return {
-    presets: ['next/babel' /* , { 'preset-env': { modules: 'commonjs' } } */],
-    plugins: [
-      // '@babel/plugin-syntax-dynamic-import',
-      // '@babel/plugin-proposal-export-default-from',
-      // '@babel/proposal-class-properties',
-      // '@babel/proposal-object-rest-spread',
-      // [
-      //   'transform-define',
-      //   {
-      //     'process.env.NODE_ENV': process.env.NODE_ENV,
-      //   },
-      // ],
-    ],
+    presets: ['next/babel'],
+    plugins: [],
   };
 };

build/README.md

@@ -0,0 +1,25 @@
+// This file is used to run health checks with Node in the Docker container
+
+var http = require('http');
+
+var options = {
+  host: 'localhost',
+  port: '3000',
+  timeout: 2000,
+};
+
+var request = http.request(options, (res) => {
+  console.log(`STATUS: ${res.statusCode}`);
+  if (res.statusCode == 200) {
+    process.exit(0);
+  } else {
+    process.exit(1);
+  }
+});
+
+request.on('error', function (err) {
+  console.log('ERROR');
+  process.exit(1);
+});
+
+request.end();

healthcheck.js

@@ -3,9 +3,7 @@ module.exports = {
   testEnvironment: 'node',
   setupFilesAfterEnv: ['<rootDir>/test-setup.js'],
   snapshotSerializers: ['enzyme-to-json/serializer'],
-  transform: {
-    '^.+\\.tsx?$': 'ts-jest',
-  },
+  transform: { '^.+\\.tsx?$': 'ts-jest' },
   testMatch: ['**/__tests__/**/*.ts?(x)', '**/?(*.)+(test).ts?(x)'],
   verbose: true,
 };

jest.config.js

@@ -1,54 +1,8 @@
 module.exports = {
-  // async exportPathMap() {
-  //   return {
-  //     // '/': { page: '/' },
-  //     // '/news': { page: '/' },
-  //     // '/active': { page: '/active' },
-  //     // '/ask': { page: '/ask' },
-  //     // '/best': { page: '/best' },
-  //     // '/bestcomments': { page: '/bestcomments' },
-  //     // '/bookmarklet': { page: '/bookmarklet' },
-  //     // '/dmca': { page: '/dmca' },
-  //     // '/formatdoc': { page: '/formatdoc' },
-  //     // '/front': { page: '/front' },
-  //     // '/item': { page: '/item' },
-  //     // '/jobs': { page: '/jobs' },
-  //     // '/leaders': { page: '/leaders' },
-  //     // '/lists': { page: '/lists' },
-  //     // '/login': { page: '/login' },
-  //     // '/newcomments': { page: '/newcomments' },
-  //     // '/newest': { page: '/newest' },
-  //     // '/newpoll': { page: '/newpoll' },
-  //     // '/newsfaq': { page: '/newsfaq' },
-  //     // '/newsguidelines': { page: '/newsguidelines' },
-  //     // '/newswelcome': { page: '/newswelcome' },
-  //     // '/noobcomments': { page: '/noobcomments' },
-  //     // '/security': { page: '/security' },
-  //     // '/show': { page: '/show' },
-  //     // '/submit': { page: '/submit' },
-  //     // '/threads': { page: '/threads' },
-  //     //     '/p/975': { page: '/post', query: { id: '975' } },
-  //     //     '/p/481': { page: '/post', query: { id: '481' } },
-  //   };
-  // },
-  webpack: (config, { dev, defaultLoaders }) => {
+  webpack: (config) => {
     // Perform customizations to webpack config
-    // if (!dev) {
-    //   config.module.rules.push({
-    //     test: /\.(css|ico|gif)$/,
-    //     use: [
-    //       {
-    //         loader: 'file-loader',
-    //         options: {
-    //           outputPath: 'static/',
-    //         },
-    //       },
-    //     ],
-    //   });
-    // }
 
-    config.node = { fs: 'empty', net: 'empty' };
+    // config.node = { fs: 'empty', net: 'empty' };
     return config;
   },
-  // cssModules: false,
 };