use git log know a documents last modified date

.gitignore

@@ -70,3 +70,5 @@ yarn-error.log*
 fake-v1-api/
 mdn-yari-*.tgz
 function.zip
+
+testing/content/files/en-us/_githistory.json

build/git-history.js

@@ -20,7 +20,8 @@ function getFromGit(contentRoot = CONTENT_ROOT) {
     ],
     {
       cwd: repoRoot,
-    }
+    },
+    repoRoot
   );
 
   const map = new Map();
@@ -40,6 +41,7 @@ function getFromGit(contentRoot = CONTENT_ROOT) {
   }
   return map;
 }
+
 function gather(contentRoot, previousFile = null) {
   const map = new Map();
   if (previousFile) {

content/utils.js

@@ -84,10 +84,19 @@ function execGit(args, opts = {}, root = null) {
     args,
     {
       cwd: gitRoot,
+      // Default is 1MB
+      // That's rarely big enough for what we're using Yari for.
+      maxBuffer: 1024 * 1024 * 100, // 100MB
     }
   );
   if (error || status !== 0) {
-    throw new Error(`git command failed:\n${stderr.toString()}`);
+    if (stderr) {
+      console.error(stderr);
+    }
+    if (error) {
+      throw error;
+    }
+    throw new Error(`git command failed: ${error.toString()}`);
   }
   return stdout.toString().trim();
 }

package.json

@@ -13,8 +13,8 @@
     "test": "yarn prettier-check && yarn test:client && yarn test:kumascript && yarn test:testing",
     "build:client": "cd client && cross-env INLINE_RUNTIME_CHUNK=false react-scripts build",
     "build:ssr": "cd ssr && webpack --config webpack.config.js --mode=production",
-    "prepare-build": "yarn build:client && yarn build:ssr && yarn tool gather-git-history",
-    "build": "cd build && cross-env NODE_ENV=production node cli.js",
+    "prepare-build": "yarn build:client && yarn build:ssr && yarn tool gather-git-history --verbose",
+    "build": "cross-env NODE_ENV=production node build/cli.js",
     "start": "(test -f client/build/index.html || yarn build:client) && (test -f ssr/dist/main.js || yarn build:ssr) && nf -j Procfile.start start",
     "dev": "yarn build:client && yarn build:ssr && nf -j Procfile.dev start",
     "start:client": "cd client && cross-env BROWSER=none PORT=3000 react-scripts start",

testing/.env

@@ -5,8 +5,8 @@
 # The relative path trick is necessary because `yarn build` will
 # be running from the `./build/` directory.
 
-CONTENT_ROOT=../testing/content/files
-CONTENT_ARCHIVED_ROOT=../testing/archived-content/files
+CONTENT_ROOT=testing/content/files
+CONTENT_ARCHIVED_ROOT=testing/archived-content/files
 
 # Because you never benefit from the progressbar when doing mass builds in
 # testing environment. And in CI, it's always disabled anyway.

tool/cli.js

@@ -1,4 +1,3 @@
-#!/usr/bin/env node
 const fs = require("fs");
 const path = require("path");
 const os = require("os");
@@ -11,7 +10,7 @@ const open = require("open");
 
 const { DEFAULT_LOCALE, VALID_LOCALES } = require("../libs/constants");
 const { CONTENT_ROOT, Redirect, Document, buildURL } = require("../content");
-const { gatherGitHistory } = require("../build");
+const { buildDocument, gatherGitHistory } = require("../build");
 
 const PORT = parseInt(process.env.SERVER_PORT || "5000");
 
@@ -303,12 +302,49 @@ program
       );
       console.log(
         chalk.green(
-          `Wrote ${Object.keys(
+          `Saved ${Object.keys(
             allHistory
           ).length.toLocaleString()} paths into ${saveHistory}`
         )
       );
     })
+  )
+
+  .command("flaws", "Find (and fix) flaws in a document")
+  .argument("<slug>", "Slug of the document in question")
+  .argument("[locale]", "Locale", {
+    default: DEFAULT_LOCALE,
+    validator: [...VALID_LOCALES.values()],
+  })
+  .option("-y, --yes", "Assume yes", { default: false })
+  .action(
+    tryOrExit(async ({ args, options }) => {
+      const { slug, locale } = args;
+      const { yes } = options;
+      const document = Document.findByURL(buildURL(locale, slug));
+      if (!document) {
+        throw new Error(`Slug ${slug} does not exist for ${locale}`);
+      }
+      const { doc } = await buildDocument(document, {
+        fixFlaws: true,
+        fixFlawsDryRun: true,
+      });
+
+      const flaws = Object.values(doc.flaws)
+        .map((a) => a.filter((f) => f.fixable).length || 0)
+        .reduce((a, b) => a + b);
+      const { run } = yes
+        ? { run: true }
+        : await prompts({
+            type: "confirm",
+            message: `Proceed fixing ${flaws} flaws?`,
+            name: "run",
+            initial: true,
+          });
+      if (run) {
+        buildDocument(document, { fixFlaws: true, fixFlawsVerbose: true });
+      }
+    })
   );
 
 program.run();