From 979a4d56ae3d66208238dea2ddd43b3598d6a837 Mon Sep 17 00:00:00 2001
From: Malte Ubl <malte.ubl@gmail.com>
Date: Thu, 1 Oct 2020 18:38:49 -0700
Subject: [PATCH 1/6] Shell out to git for a better last modified date

---
 .eleventy.js | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/.eleventy.js b/.eleventy.js
index f68f173..b922096 100644
--- a/.eleventy.js
+++ b/.eleventy.js
@@ -89,9 +89,22 @@ module.exports = function (eleventyConfig) {
       .catch((error) => callback(error));
   });
 
-  eleventyConfig.addFilter("lastModifiedDate", function (filename) {
-    const stats = fs.statSync(filename);
-    return stats.mtime; // Date
+  eleventyConfig.addNunjucksAsyncFilter("lastModifiedDate", function (
+    filename,
+    callback
+  ) {
+    require("child_process").execFile(
+      "git",
+      ["log", "-1", "--format=%cd", filename],
+      (error, stdout) => {
+        if (error) {
+          console.error(error);
+          const stats = fs.statSync(filename);
+          callback(null, stats.mtime); // Date
+        }
+        callback(null, new Date(stdout));
+      }
+    );
   });
 
   eleventyConfig.addFilter("encodeURIComponent", function (str) {

From 3cb86a025f26d552c712184945cf8a4e68373159 Mon Sep 17 00:00:00 2001
From: Malte Ubl <malte.ubl@gmail.com>
Date: Thu, 1 Oct 2020 18:40:58 -0700
Subject: [PATCH 2/6] Avoid hardcoding image dimensions in test

Fixes #42
---
 test/test-generic-post.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/test-generic-post.js b/test/test-generic-post.js
index b01ed52..928f384 100644
--- a/test/test-generic-post.js
+++ b/test/test-generic-post.js
@@ -168,8 +168,8 @@ describe("check build output for a generic post", () => {
         //expect(avif.type).to.equal("image/avif");
         expect(jpg.sizes).to.equal("(max-width: 608px) 100vw, 608px");
         expect(webp.sizes).to.equal("(max-width: 608px) 100vw, 608px");
-        expect(img.height).to.equal(850);
-        expect(img.width).to.equal(1280);
+        expect(img.height).to.match(/^\d+$/);
+        expect(img.width).to.match(/^\d+$/);
         expect(img.getAttribute("loading")).to.equal("lazy");
         expect(img.getAttribute("decoding")).to.equal("async");
         // JSDom fails to parse the style attribute properly

From ab0cb0a63f0a2da0a23abb72a5516d867a6b8c9a Mon Sep 17 00:00:00 2001
From: Malte Ubl <malte.ubl@gmail.com>
Date: Thu, 1 Oct 2020 19:57:25 -0700
Subject: [PATCH 3/6] Explicit liberal indexing

---
 _includes/layouts/base.njk | 1 +
 1 file changed, 1 insertion(+)

diff --git a/_includes/layouts/base.njk b/_includes/layouts/base.njk
index 99ae89d..f0135b7 100644
--- a/_includes/layouts/base.njk
+++ b/_includes/layouts/base.njk
@@ -15,6 +15,7 @@
     <link rel="icon" href="{{ '/img/favicon/favicon-192x192.png' | addHash }}" type="image/png">
     {% endif %}
     <meta name="theme-color" content="#f9c412">
+    <meta name="robots" content="max-snippet:-1, max-image-preview: large, max-video-preview: -1">
     <title>{{ renderData.title or title or metadata.title | escape }}</title>
     <meta property="og:title" content="{{ renderData.title or title or metadata.title | escape  }}">
     {% set desc = renderData.description or description %}

From 5607091bceec56b89ddc4f9bc9a0cd339e1d67bc Mon Sep 17 00:00:00 2001
From: Bao Nguyen <nlbao@users.noreply.github.com>
Date: Sat, 3 Oct 2020 23:05:47 +0800
Subject: [PATCH 4/6] Make updating the test "should have script elements"
 easier (#43)

---
 test/test-generic-post.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/test/test-generic-post.js b/test/test-generic-post.js
index 928f384..bd0fba0 100644
--- a/test/test-generic-post.js
+++ b/test/test-generic-post.js
@@ -63,7 +63,8 @@ describe("check build output for a generic post", () => {
 
     it("should have script elements", () => {
       const scripts = doc.querySelectorAll("script[src]");
-      expect(scripts).to.have.length(GA_ID ? 2 : 1);
+      let has_ga_id = GA_ID ? 1 : 0;
+      expect(scripts).to.have.length(has_ga_id + 1); // NOTE: update this when adding more <script>
       expect(scripts[0].getAttribute("src")).to.match(
         /^\/js\/min\.js\?hash=\w+/
       );

From 9e0b7eabc0e4bb1643a6adf4f48834433e7e0f1e Mon Sep 17 00:00:00 2001
From: Malte Ubl <malte.ubl@gmail.com>
Date: Sat, 3 Oct 2020 11:54:37 -0700
Subject: [PATCH 5/6] Cache the lastModifiedDate git invocation

---
 .eleventy.js | 46 +++++++++++++++++++++++++++++++++-------------
 1 file changed, 33 insertions(+), 13 deletions(-)

diff --git a/.eleventy.js b/.eleventy.js
index b922096..a193533 100644
--- a/.eleventy.js
+++ b/.eleventy.js
@@ -44,7 +44,9 @@ const { DateTime } = require("luxon");
 const { promisify } = require("util");
 const fs = require("fs");
 const hasha = require("hasha");
-const readFile = promisify(require("fs").readFile);
+const readFile = promisify(fs.readFile);
+const stat = promisify(fs.stat);
+const execFile = promisify(require("child_process").execFile);
 const pluginRss = require("@11ty/eleventy-plugin-rss");
 const pluginSyntaxHighlight = require("@11ty/eleventy-plugin-syntaxhighlight");
 const pluginNavigation = require("@11ty/eleventy-navigation");
@@ -89,22 +91,40 @@ module.exports = function (eleventyConfig) {
       .catch((error) => callback(error));
   });
 
+  async function lastModifiedDate(filename) {
+    try {
+      const { stdout } = await execFile("git", [
+        "log",
+        "-1",
+        "--format=%cd",
+        filename,
+      ]);
+      return new Date(stdout);
+    } catch (e) {
+      console.error(e.message);
+      // Fallback to stat if git isn't working.
+      const stats = await stat(filename);
+      return stats.mtime; // Date
+    }
+  }
+  // Cache the lastModifiedDate call because shelling out to git is expensive.
+  // This means the lastModifiedDate will never change per single eleventy invocation.
+  const lastModifiedDateCache = new Map();
   eleventyConfig.addNunjucksAsyncFilter("lastModifiedDate", function (
     filename,
     callback
   ) {
-    require("child_process").execFile(
-      "git",
-      ["log", "-1", "--format=%cd", filename],
-      (error, stdout) => {
-        if (error) {
-          console.error(error);
-          const stats = fs.statSync(filename);
-          callback(null, stats.mtime); // Date
-        }
-        callback(null, new Date(stdout));
-      }
-    );
+    const call = (result) => {
+      result.then((date) => callback(null, date));
+      result.catch((error) => callback(error));
+    };
+    const cached = lastModifiedDateCache.get(filename);
+    if (cached) {
+      return call(cached);
+    }
+    const promise = lastModifiedDate(filename);
+    lastModifiedDateCache.set(filename, promise);
+    call(promise);
   });
 
   eleventyConfig.addFilter("encodeURIComponent", function (str) {

From 6e9425312495353ac579e6387885fecac4edd106 Mon Sep 17 00:00:00 2001
From: Malte Ubl <cramforce@users.noreply.github.com>
Date: Mon, 5 Oct 2020 19:24:04 -0700
Subject: [PATCH 6/6] Always scale the input image to desired output size.
 (#45)

Fixes #44
---
 _11ty/blurry-placeholder.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/_11ty/blurry-placeholder.js b/_11ty/blurry-placeholder.js
index c9a24ce..b1bc6be 100644
--- a/_11ty/blurry-placeholder.js
+++ b/_11ty/blurry-placeholder.js
@@ -109,7 +109,7 @@ module.exports = async function (src) {
                       <feFuncA type="discrete" tableValues="1 1"></feFuncA>
                     </feComponentTransfer>
                   </filter>
-                  <image filter="url(#b)" x="0" y="0"
+                  <image filter="url(#b)" preserveAspectRatio="none"
                     height="100%" width="100%"
                     xlink:href="${dataURI.src}">
                   </image>