afterbefore 0.1.9 → 0.1.11

package/dist/cli.js

@@ -819,7 +819,8 @@ function findRoutesForFile(file, graph) {
 }
 
 // src/stages/worktree.ts
-import { execSync as execSync2 } from "child_process";
+import { exec as execCb, execSync as execSync2 } from "child_process";
+import { promisify } from "util";
 import { rm, mkdtemp } from "fs/promises";
 import { join as join4 } from "path";
 import { tmpdir } from "os";
@@ -848,14 +849,12 @@ function pmExec(pm) {
 }
 
 // src/stages/worktree.ts
+var exec = promisify(execCb);
 async function createWorktree(base, cwd) {
   const worktreeDir = await mkdtemp(join4(tmpdir(), "afterbefore-wt-"));
   logger.dim(`Creating worktree for ${base} at ${worktreeDir}`);
   try {
-    execSync2(`git worktree add "${worktreeDir}" "${base}"`, {
-      cwd,
-      stdio: ["pipe", "pipe", "pipe"]
-    });
+    await exec(`git worktree add "${worktreeDir}" "${base}"`, { cwd });
   } catch (err) {
     throw new AfterbeforeError(
       `Failed to create worktree for ref "${base}".`,
@@ -864,10 +863,7 @@ async function createWorktree(base, cwd) {
   }
   const pm = detectPackageManager(cwd);
   logger.dim(`Installing dependencies with ${pm}`);
-  execSync2(`${pm} install`, {
-    cwd: worktreeDir,
-    stdio: ["pipe", "pipe", "pipe"]
-  });
+  await exec(`${pm} install`, { cwd: worktreeDir });
   const cleanup = async () => {
     logger.dim(`Cleaning up worktree at ${worktreeDir}`);
     try {
@@ -912,7 +908,7 @@ function waitForServer(url, timeoutMs) {
         await fetch(url);
         resolve5();
       } catch {
-        setTimeout(poll, 500);
+        setTimeout(poll, 150);
       }
     };
     poll();
@@ -921,8 +917,8 @@ function waitForServer(url, timeoutMs) {
 async function startServer(projectDir, port) {
   const url = `http://localhost:${port}`;
   const pm = detectPackageManager(projectDir);
-  const exec = pmExec(pm);
-  const [cmd, ...baseArgs] = exec.split(" ");
+  const exec2 = pmExec(pm);
+  const [cmd, ...baseArgs] = exec2.split(" ");
   const lockFile = join5(projectDir, ".next", "dev", "lock");
   if (existsSync5(lockFile)) {
     throw new AfterbeforeError(
@@ -1009,6 +1005,7 @@ async function detectSections(page, maxSections) {
       if (!container || tagged.has(container)) continue;
       if (container.scrollHeight > document.documentElement.scrollHeight * 0.9) continue;
       container.setAttribute("data-ab-section", String(idx));
+      heading.setAttribute("data-ab-heading", String(idx));
       tagged.add(container);
       results.push({
         label: (heading.textContent ?? "").trim(),
@@ -1067,6 +1064,7 @@ async function tagSectionOnPage(page, headingText, index) {
       }
       if (el) {
         el.setAttribute("data-ab-section", String(idx));
+        match.setAttribute("data-ab-heading", String(idx));
         return true;
       }
       return false;
@@ -1074,12 +1072,33 @@ async function tagSectionOnPage(page, headingText, index) {
     { text: headingText, idx: index }
   );
 }
+async function hideSectionHeading(page, sectionIndex) {
+  await page.evaluate((idx) => {
+    const heading = document.querySelector(`[data-ab-heading="${idx}"]`);
+    if (heading instanceof HTMLElement) {
+      heading.style.display = "none";
+    }
+  }, sectionIndex);
+}
+async function showSectionHeading(page, sectionIndex) {
+  await page.evaluate((idx) => {
+    const heading = document.querySelector(`[data-ab-heading="${idx}"]`);
+    if (heading instanceof HTMLElement) {
+      heading.style.display = "";
+    }
+  }, sectionIndex);
+}
 async function cleanupSectionTags(page) {
   await page.evaluate(() => {
-    const tagged = document.querySelectorAll("[data-ab-section]");
-    for (const el of tagged) {
+    for (const el of document.querySelectorAll("[data-ab-section]")) {
       el.removeAttribute("data-ab-section");
     }
+    for (const el of document.querySelectorAll("[data-ab-heading]")) {
+      el.removeAttribute("data-ab-heading");
+      if (el instanceof HTMLElement) {
+        el.style.display = "";
+      }
+    }
   });
 }
 function sanitizeSectionLabel(label) {
@@ -1395,12 +1414,16 @@ async function captureAutoSections(afterPage, beforePage, parentPrefix, parentLa
     const sectionAfterPath = join6(outputDir, `${sectionPrefix}-after.png`);
     const sectionBeforePath = join6(outputDir, `${sectionPrefix}-before.png`);
     try {
+      await hideSectionHeading(afterPage, section.index);
       const afterEl = afterPage.locator(`[data-ab-section="${section.index}"]`).first();
       await afterEl.screenshot({ path: sectionAfterPath });
+      await showSectionHeading(afterPage, section.index);
       const found = await tagSectionOnPage(beforePage, section.label, section.index);
       if (found) {
+        await hideSectionHeading(beforePage, section.index);
         const beforeEl = beforePage.locator(`[data-ab-section="${section.index}"]`).first();
         await beforeEl.screenshot({ path: sectionBeforePath });
+        await showSectionHeading(beforePage, section.index);
       } else {
         continue;
       }
@@ -1879,7 +1902,7 @@ async function runPipeline(options) {
   const outputDir = resolve4(cwd, output, sessionName);
   const startTime = Date.now();
   try {
-    const version = true ? "0.1.9" : "dev";
+    const version = true ? "0.1.11" : "dev";
     console.log(`
 afterbefore v${version} \xB7 Comparing against ${base}
 `);
@@ -2024,7 +2047,7 @@ afterbefore v${version} \xB7 Comparing against ${base}
 var program = new Command();
 program.name("afterbefore").description(
   "Automatic before/after screenshot capture for PRs. Git diff is the config."
-).version("0.1.9").option("--base <ref>", "Base branch or ref to compare against", "main").option("--output <dir>", "Output directory for screenshots", ".afterbefore").option("--post", "Post results as a PR comment via gh CLI", false).option(
+).version("0.1.11").option("--base <ref>", "Base branch or ref to compare against", "main").option("--output <dir>", "Output directory for screenshots", ".afterbefore").option("--post", "Post results as a PR comment via gh CLI", false).option(
   "--threshold <percent>",
   "Diff threshold percentage (changes below this are ignored)",
   "0.1"