🤦‍♀️ ignore empty lines (#2)

src/unindent.test.ts

@@ -58,4 +58,25 @@ describe("unindent", () => {
     content TEXT NOT NULL
 )`);
     });
+
+    test("it handles empty lines in the middle of the string", () => {
+        expect(unindent`
+
+        CREATE TABLE posts (
+            id UUID NOT NULL PRIMARY KEY,
+
+            title TEXT NOT NULL,
+
+            content TEXT NOT NULL
+        )
+
+
+      `).toEqual(`CREATE TABLE posts (
+    id UUID NOT NULL PRIMARY KEY,
+
+    title TEXT NOT NULL,
+
+    content TEXT NOT NULL
+)`);
+    });
 });

src/unindent.ts

@@ -2,18 +2,24 @@ export const unindent = (strings: TemplateStringsArray) => {
     const joined = strings.join("");
     const lines = joined.split("\n");
 
+    // strip any leading empty lines
     while (lines[0]?.replace(/\s*/, "") === "") {
         lines.splice(0, 1);
     }
 
+    // strip any traling empty lines
     while (lines[lines.length - 1]?.replace(/\s*/, "") === "") {
         lines.splice(lines.length - 1, 1);
     }
 
-    const indent =
-        Math.min(
-            ...lines.map((s) => s.length - s.replace(/^\s+/, "").length),
-        ) ?? 0;
+    const indents = lines
+        // ignore any blank lines when calculating indents
+        .filter((l) => l.replace(/\s*/, "") !== "")
+        // get the number of whitespace characters before the first non-whitespace character
+        .map((s) => s.length - s.replace(/^\s+/, "").length);
+
+    // take the smallest indent: this is how much we're going to unindent by
+    const indent = Math.min(...indents) ?? 0;
 
     return lines.map((s) => s.substring(indent)).join("\n");
 };