From b68da59ef41d4d66b9c4ec5d7da1a3550b5b58b7 Mon Sep 17 00:00:00 2001
From: Denis Badurina <badurinadenis@gmail.com>
Date: Mon, 15 Aug 2022 12:00:44 +0200
Subject: [PATCH] feat: incremental build (#123)

* incremental

* changeset

* also incremental in ts

* omit incremental flag completely
---
 .changeset/cold-moles-remain.md |  5 +++++
 src/commands/build.ts           | 36 +++++++++++++++++++++++++--------
 2 files changed, 33 insertions(+), 8 deletions(-)
 create mode 100644 .changeset/cold-moles-remain.md

diff --git a/.changeset/cold-moles-remain.md b/.changeset/cold-moles-remain.md
new file mode 100644
index 00000000..0953b4ff
--- /dev/null
+++ b/.changeset/cold-moles-remain.md
@@ -0,0 +1,5 @@
+---
+"bob-the-bundler": minor
+---
+
+better performance by incrementally building only packages that had changes
diff --git a/src/commands/build.ts b/src/commands/build.ts
index a92a8ec1..e3953c40 100644
--- a/src/commands/build.ts
+++ b/src/commands/build.ts
@@ -74,11 +74,15 @@ function assertTypeScriptBuildResult(result: execa.ExecaReturnValue<string>) {
   }
 }
 
-async function buildTypeScript(buildPath: string) {
+async function buildTypeScript(
+  buildPath: string,
+  options: { incremental?: boolean } = {}
+) {
   assertTypeScriptBuildResult(
     await execa("npx", [
       "tsc",
       ...compilerOptionsToArgs(typeScriptCompilerOptions("esm")),
+      ...(options.incremental ? ["--incremental"] : []),
       "--outDir",
       join(buildPath, "esm"),
     ])
@@ -88,22 +92,34 @@ async function buildTypeScript(buildPath: string) {
     await execa("npx", [
       "tsc",
       ...compilerOptionsToArgs(typeScriptCompilerOptions("cjs")),
+      ...(options.incremental ? ["--incremental"] : []),
       "--outDir",
       join(buildPath, "cjs"),
     ])
   );
 }
 
-export const buildCommand = createCommand<{}, {}>((api) => {
+export const buildCommand = createCommand<
+  {},
+  {
+    incremental?: boolean;
+  }
+>((api) => {
   const { reporter } = api;
 
   return {
     command: "build",
     describe: "Build",
     builder(yargs) {
-      return yargs.options({});
+      return yargs.options({
+        incremental: {
+          describe:
+            "Better performance by building only packages that had changes.",
+          type: "boolean",
+        },
+      });
     },
-    async handler() {
+    async handler({ incremental }) {
       const cwd = process.cwd();
       const rootPackageJSON = await getRootPackageJSON(cwd);
       const workspaces = getWorkspaces(rootPackageJSON);
@@ -112,8 +128,10 @@ export const buildCommand = createCommand<{}, {}>((api) => {
       if (isSinglePackage) {
         const buildPath = join(cwd, ".bob");
 
-        await fse.remove(buildPath);
-        await buildTypeScript(buildPath);
+        if (!incremental) {
+          await fse.remove(buildPath);
+        }
+        await buildTypeScript(buildPath, { incremental });
         const pkg = await fse.readJSON(resolve(cwd, "package.json"));
         const fullName: string = pkg.name;
 
@@ -150,8 +168,10 @@ export const buildCommand = createCommand<{}, {}>((api) => {
       );
 
       const bobBuildPath = join(cwd, ".bob");
-      await fse.remove(bobBuildPath);
-      await buildTypeScript(bobBuildPath);
+      if (!incremental) {
+        await fse.remove(bobBuildPath);
+      }
+      await buildTypeScript(bobBuildPath, { incremental });
 
       await Promise.all(
         packageInfoList.map(({ cwd, pkg, fullName }) =>