terminalhire 0.42.33 → 0.42.35

package/dist/bin/capture/captureEntry.js

@@ -6,8 +6,8 @@
  * repository's. The repository's code has already run: its build wrote files into the
  * clone, or its serve script is listening in a container this one shares a network
  * namespace with. This process serves or reaches that app over loopback, drives a
- * headless Chromium through each route × viewport × colour scheme, and writes PNGs
- * plus `manifest.json` to /out.
+ * headless Chromium through each planned shot (with no spec, one per route: desktop,
+ * light), and writes PNGs plus `manifest.json` to /out.
  *
  * No network in either mode: static capture runs under `--network=none`, and server
  * capture joins an app container that itself has `--network=none`. Whatever the page
@@ -27,7 +27,16 @@ export const VIEWPORTS = {
     desktop: { width: 1280, height: 800, isMobile: false },
     mobile: { width: 390, height: 844, isMobile: true },
 };
-const SCHEMES = ['light', 'dark'];
+export const SCHEMES = ['light', 'dark'];
+/**
+ * What gets shot. With no spec, one picture per route at desktop size in light mode
+ * (TERM-1272). The desktop/mobile × light/dark matrix this replaces suited one claim
+ * and cost every other one three more renders of the same screen, most often a
+ * loading state; mobile and dark are for a spec to ask for, shot by shot.
+ */
+export function plannedShots(req) {
+    return req.routes.map((route) => ({ route, viewport: 'desktop', scheme: 'light' }));
+}
 // ---- which directory is the site -------------------------------------------
 /**
  * When the build started, read off the marker the build step wrote. Taken from the
@@ -287,53 +296,49 @@ async function shoot(origin, basePath, req, out) {
     const items = [];
     const failures = [];
     try {
-        for (const route of req.routes) {
+        for (const { route, viewport, scheme } of plannedShots(req)) {
             const url = `${origin}${basePath.replace(/\/$/, '')}${route.startsWith('/') ? route : `/${route}`}`;
-            for (const viewport of Object.keys(VIEWPORTS)) {
-                for (const scheme of SCHEMES) {
-                    const v = VIEWPORTS[viewport];
-                    const context = await browser.newContext({
-                        viewport: { width: v.width, height: v.height },
-                        isMobile: v.isMobile,
-                        hasTouch: v.isMobile,
-                        colorScheme: scheme,
-                        deviceScaleFactor: 1,
-                    });
-                    try {
-                        const page = await context.newPage();
-                        await page.goto(url, { waitUntil: 'load', timeout: 20_000 });
-                        // A page with no backend may keep retrying forever, so idle is
-                        // waited for briefly and never required.
-                        await page.waitForLoadState('networkidle', { timeout: 5_000 }).catch(() => undefined);
-                        await page.waitForTimeout(300);
-                        // Frozen, so two shots of the same page are the same bytes: a spinner
-                        // caught on different frames would otherwise hide that light and dark
-                        // render identically, which is the one comparison the notes rely on.
-                        const png = await page.screenshot({ animations: 'disabled', caret: 'hide' });
-                        const file = shotName(route, viewport, scheme);
-                        writeFileSync(join(out, file), png);
-                        items.push({
-                            route,
-                            viewport,
-                            scheme,
-                            file,
-                            sha256: createHash('sha256').update(png).digest('hex'),
-                            bytes: png.length,
-                            ...PNG_SIZE(png),
-                        });
-                    }
-                    catch (err) {
-                        failures.push({
-                            route,
-                            viewport,
-                            scheme,
-                            error: String(err?.message ?? err).slice(0, 300),
-                        });
-                    }
-                    finally {
-                        await context.close();
-                    }
-                }
+            const v = VIEWPORTS[viewport];
+            const context = await browser.newContext({
+                viewport: { width: v.width, height: v.height },
+                isMobile: v.isMobile,
+                hasTouch: v.isMobile,
+                colorScheme: scheme,
+                deviceScaleFactor: 1,
+            });
+            try {
+                const page = await context.newPage();
+                await page.goto(url, { waitUntil: 'load', timeout: 20_000 });
+                // A page with no backend may keep retrying forever, so idle is
+                // waited for briefly and never required.
+                await page.waitForLoadState('networkidle', { timeout: 5_000 }).catch(() => undefined);
+                await page.waitForTimeout(300);
+                // Frozen, so two shots of the same page are the same bytes: a spinner
+                // caught on different frames would otherwise hide that light and dark
+                // render identically, which is the one comparison the notes rely on.
+                const png = await page.screenshot({ animations: 'disabled', caret: 'hide' });
+                const file = shotName(route, viewport, scheme);
+                writeFileSync(join(out, file), png);
+                items.push({
+                    route,
+                    viewport,
+                    scheme,
+                    file,
+                    sha256: createHash('sha256').update(png).digest('hex'),
+                    bytes: png.length,
+                    ...PNG_SIZE(png),
+                });
+            }
+            catch (err) {
+                failures.push({
+                    route,
+                    viewport,
+                    scheme,
+                    error: String(err?.message ?? err).slice(0, 300),
+                });
+            }
+            finally {
+                await context.close();
             }
         }
     }

package/dist/bin/jpi-bounties.js

@@ -12592,6 +12592,11 @@ function wrapIndented(text, indent, width) {
   lines.push(pad + line);
   return lines;
 }
+function projectMarkerLine(bounty) {
+  const tasks = bounty?.project?.tasks;
+  if (typeof tasks !== "number" || !Number.isInteger(tasks) || tasks <= 0) return null;
+  return tasks > 1 ? `project \xB7 ${tasks} tasks` : "project";
+}
 function printBounty(i, job, score, reason, matchedTags, claimedIds = /* @__PURE__ */ new Set(), continuityNote = null) {
   const b = job.bounty ?? {};
   const stars = b.repoStars != null ? ` \xB7 ${b.repoStars}\u2605` : "";
@@ -12607,6 +12612,8 @@ ${i + 1}. ${linkTitle(job.title, job.url)} [${ref}]`);
   console.log(
     `   ${formatAmount(b)}${effort} \xB7 ${sanitizeText(b.repoFullName ?? job.company)}${stars}${scoreStr}${contend}${badge}`
   );
+  const marker = projectMarkerLine(b);
+  if (marker) console.log(`   ${marker}`);
   if (reason) console.log(`   ${reason}`);
   if (continuityNote) console.log(`   ${continuityNote}`);
   if (b.publicSummary) {
@@ -12862,6 +12869,7 @@ export {
   getBounties,
   isPinnedFounderBounty,
   printBounty,
+  projectMarkerLine,
   rankBounties,
   run,
   wrapIndented

package/dist/bin/jpi-chat.js

@@ -5978,10 +5978,17 @@ var init_profile = __esm({
 // src/open-url.js
 var open_url_exports = {};
 __export(open_url_exports, {
-  openInBrowser: () => openInBrowser
+  openInBrowser: () => openInBrowser,
+  shouldOpenBrowser: () => shouldOpenBrowser
 });
-import { spawn } from "child_process";
-function openInBrowser(url) {
+import { spawn as nodeSpawn } from "child_process";
+function shouldOpenBrowser(env = process.env) {
+  if (env.NODE_TEST_CONTEXT) return false;
+  const off = String(env.TERMINALHIRE_NO_BROWSER ?? "").trim().toLowerCase();
+  return !(off === "1" || off === "true");
+}
+function openInBrowser(url, { env = process.env, spawn = nodeSpawn } = {}) {
+  if (!shouldOpenBrowser(env)) return;
   let cmd;
   let args;
   if (process.platform === "darwin") {