@crouton-kit/capture 1.4.23 → 1.4.24

package/bin/capture

@@ -10944,7 +10944,7 @@ function validateHarAppendBatch(value, source) {
   return { entries, incompleteLifecycles };
 }
 function captureVersion() {
-  const version = "1.4.23";
+  const version = "1.4.24";
   return typeof version === "string" && version.length > 0 ? version : "development";
 }
 function token() {
@@ -13046,7 +13046,7 @@ function wallTime(value) {
   }
 }
 function captureVersion2() {
-  const version = "1.4.23";
+  const version = "1.4.24";
   return typeof version === "string" && version.length > 0 ? version : "development";
 }
 function deepFreeze(value) {
@@ -240317,7 +240317,8 @@ function checkSnapshot(ref, requested) {
       if (!e) continue;
       const point = sample.points?.find((p) => {
         const receiver = p.result?.topReceiver;
-        const receiverElement = receiver && elements.find((x) => receiver.backendNodeId != null && x.backendNodeId === receiver.backendNodeId);
+        if (!receiver) return false;
+        const receiverElement = elements.find((x) => receiver.backendNodeId != null && x.backendNodeId === receiver.backendNodeId);
         return receiverElement !== void 0 && !isSelfOrDescendant(e, receiverElement);
       })?.result;
       if (!point?.topReceiver) continue;
@@ -240495,11 +240496,12 @@ async function cmdMeasureCheck(parsed, _args) {
   } catch (err) {
     const resolution = err;
     const detail = err instanceof Error ? err.message : "unknown artifact read failure";
+    const unresolved = resolution.name === "ArtifactResolutionError";
     emitResult({
       tag: "error",
-      attrs: { command: "measure check", status: resolution.name === "ArtifactResolutionError" ? "artifact_unavailable" : "check_failed" },
-      summary: fact`Measure check could not read the requested artifact: ${detail}`,
-      followUp: text`Create a settled snapshot with capture measure snap <url>, then pass its id or absolute path.`
+      attrs: { command: "measure check", status: unresolved ? "artifact_unavailable" : "check_failed" },
+      summary: unresolved ? fact`Measure check could not read the requested artifact: ${detail}` : fact`Measure check failed while measuring the snapshot: ${detail}`,
+      followUp: unresolved ? text`Create a settled snapshot with capture measure snap <url>, then pass its id or absolute path.` : text`The snapshot resolved; this is an internal capture check error, not a missing snapshot. Narrow --for to the checks you need, and report the error with the snapshot path.`
     }, { json: parsed.json });
     process.exitCode = 1;
   }
@@ -267516,7 +267518,7 @@ I/O contract: flags and positionals on input; one rendered prose block on stdout
 A CDP-enabled browser must be running; \`capture tab list\` is the probe, and \`capture tab launch\` starts one capture owns and reaps when nothing is (never hand-roll a detached browser \u2014 nothing reaps that). Capture never signals a browser it did not start, so attaching to your own with --port is unchanged. CDP_PORT / CDP_TARGET pin the browser + tab for target-selecting commands \u2014 precedence: explicit flag > active session > env. \`session start\` adopts only an explicit --target. When CRTR_NODE_ID is set (a crtr agent node), a browser command with no --port, no --target and no active session of that node is refused instead of falling back to CDP_PORT, CDP_TARGET or localhost discovery, because the browser found could belong to another node: run \`capture session start --url <url>\`, or pass --port or --target. \`capture tab list\` and commands that only read a recorded artifact are not refused.`;
 }
 function printVersion() {
-  const declared = "1.4.23";
+  const declared = "1.4.24";
   if (declared) {
     console.log(declared);
     return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@crouton-kit/capture",
-  "version": "1.4.23",
+  "version": "1.4.24",
   "description": "Browser automation and UI validation via CDP — session-based screenshots, HAR, a11y, click/type, and JS exec",
   "bin": {
     "capture": "./bin/capture"

package/src/cdp/commands/measure/check.ts

@@ -152,11 +152,16 @@ export async function cmdMeasureCheck(parsed: ParsedArgs, _args: string[]): Prom
   } catch (err) {
     const resolution = err as Partial<ArtifactResolutionError>;
     const detail = err instanceof Error ? err.message : 'unknown artifact read failure';
+    const unresolved = resolution.name === 'ArtifactResolutionError';
     emitResult({
       tag: 'error',
-      attrs: { command: 'measure check', status: resolution.name === 'ArtifactResolutionError' ? 'artifact_unavailable' : 'check_failed' },
-      summary: fact`Measure check could not read the requested artifact: ${detail}`,
-      followUp: text`Create a settled snapshot with capture measure snap <url>, then pass its id or absolute path.`,
+      attrs: { command: 'measure check', status: unresolved ? 'artifact_unavailable' : 'check_failed' },
+      summary: unresolved
+        ? fact`Measure check could not read the requested artifact: ${detail}`
+        : fact`Measure check failed while measuring the snapshot: ${detail}`,
+      followUp: unresolved
+        ? text`Create a settled snapshot with capture measure snap <url>, then pass its id or absolute path.`
+        : text`The snapshot resolved; this is an internal capture check error, not a missing snapshot. Narrow --for to the checks you need, and report the error with the snapshot path.`,
     }, { json: parsed.json });
     process.exitCode = 1;
   }

package/src/cdp/measure/check.ts

@@ -448,7 +448,9 @@ export function checkSnapshot(ref: SnapRef, requested: readonly CheckName[]): {
       if (!e) continue;
       const point = sample.points?.find((p) => {
         const receiver = p.result?.topReceiver;
-        const receiverElement = receiver && elements.find((x) => receiver.backendNodeId != null && x.backendNodeId === receiver.backendNodeId);
+        // The collector records `topReceiver: null` for a point with an empty hit stack (e.g. a sample outside the viewport); no receiver means nothing occludes it.
+        if (!receiver) return false;
+        const receiverElement = elements.find((x) => receiver.backendNodeId != null && x.backendNodeId === receiver.backendNodeId);
         return receiverElement !== undefined && !isSelfOrDescendant(e, receiverElement);
       })?.result;
       if (!point?.topReceiver) continue;