Merge pull request #14341 from artsy/DIA-793-generate-bundle-size-rep…

…ort-for-pull-requests

feat: adds relative-ci agent

.circleci/config.yml

@@ -216,6 +216,17 @@ jobs:
       - store_artifacts:
           path: cypress/screenshots
 
+  bundle-stats:
+    docker:
+      - image: 585031190124.dkr.ecr.us-east-1.amazonaws.com/force:$CIRCLE_SHA1-builder
+        aws_auth:
+          aws_access_key_id: $AWS_ACCESS_KEY_ID
+          aws_secret_access_key: $AWS_SECRET_ACCESS_KEY
+    working_directory: /app
+    steps:
+      - run: yarn bundle-report:client --json webpack-stats.json
+      - run: npx relative-ci-agent
+
 not_main_or_staging_or_release: &not_main_or_staging_or_release
   filters:
     branches:
@@ -330,6 +341,12 @@ workflows:
           requires:
             - electron-runner-image-push
 
+      - bundle-stats:
+          <<: *not_staging_or_release
+          context: hokusai
+          requires:
+            - builder-image-push
+
       # Staging
       - artsy-remote-docker/buildkit-build:
           <<: *only_main
@@ -349,6 +366,7 @@ workflows:
             - type-check
             - relay-check
             - acceptance
+            - bundle-stats
             - production-image-build
 
       - hokusai/deploy-staging:

.gitignore

@@ -51,6 +51,7 @@ report.[0-9]*.[0-9]*.[0-9]*.[0-9]*.json
 
 # Webpack stats
 stats*.json
+webpack-stats.json
 
 # DLL
 manifest-legacy-artwork-dll.json

Dockerfile

@@ -106,6 +106,7 @@ COPY data ./data
 COPY patches ./patches
 COPY src ./src
 COPY webpack ./webpack
+COPY .git ./.git
 COPY .env.oss \
   .env.test \
   .eslintrc.js \
@@ -118,6 +119,7 @@ COPY .env.oss \
   jest.config.js \
   package.json \
   relay.config.js \
+  relativeci.config.js \
   tsconfig.json \
   yarn.lock \
   ./

package.json

@@ -228,6 +228,7 @@
     "@loadable/babel-plugin": "5.13.2",
     "@loadable/webpack-plugin": "5.15.2",
     "@pmmmwh/react-refresh-webpack-plugin": "^0.5.10",
+    "@relative-ci/agent": "^4.2.10",
     "@storybook/addon-actions": "6.5.13",
     "@storybook/addon-essentials": "6.5.13",
     "@storybook/addon-links": "6.5.13",

relativeci.config.js

@@ -0,0 +1,11 @@
+module.exports = {
+  // Get current commit message (`git log -1 --pretty=%B`) and send it to RelativeCI as part of the build informatin
+  includeCommitMessage: true,
+  // Save agent payload to disk for debugging
+  // @example './relative-ci-payload.json',
+  payloadFilepath: undefined,
+  webpack: {
+    // Path to Webpack stats JSON file
+    stats: "./webpack-stats.json",
+  },
+}

webpack/envs/clientProductionConfig.js

@@ -4,10 +4,9 @@ import HtmlWebpackPlugin from "html-webpack-plugin"
 import LoadablePlugin from "@loadable/webpack-plugin"
 import { BundleAnalyzerPlugin } from "webpack-bundle-analyzer"
 import { WebpackManifestPlugin } from "webpack-manifest-plugin"
-import { basePath, webpackEnv } from "../webpackEnv"
+import { basePath } from "../webpackEnv"
 import { sharedPlugins } from "../sharedPlugins"
 import { splitChunks } from "../bundleSplitting"
-
 import { babelLoader, ejsLoader, mjsLoader } from "../sharedLoaders"
 
 import {
@@ -43,10 +42,18 @@ const clientProductionConfig = () => {
       path: path.resolve(basePath, "public/assets"),
       publicPath: "/assets/",
     },
+    stats: process.env.RELATIVE_CI_KEY
+      ? {
+          assets: true,
+          chunks: true,
+          modules: true,
+        }
+      : stats,
     plugins: [
       ...sharedPlugins(),
       new LoadablePlugin({
         filename: "loadable-stats.json",
+        // @ts-ignore
         path: path.resolve(basePath, "public", "assets"),
       }),
       new WebpackManifestPlugin({
@@ -63,15 +70,13 @@ const clientProductionConfig = () => {
         },
         template: path.resolve(basePath, "src/html.ejs"),
       }),
-
       process.env.WEBPACK_BUNDLE_REPORT &&
         new BundleAnalyzerPlugin({
           analyzerMode: "static",
           defaultSizes: "gzip",
         }),
     ].filter(Boolean),
     resolve,
-    stats,
   }
 }