playwright-intelligence-engine 2.0.4 → 2.1.0

package/src/trace/archive-reader.ts

@@ -24,6 +24,8 @@ export interface TraceEntryDiscovery {
   /** All streams, ordered with explicit failure streams first. */
   traceEntryNames?: string[] | undefined;
   networkEntryNames?: string[] | undefined;
+  hasFailure?: boolean | undefined;
+  failureScore?: number | undefined;
 }
 
 export class ArchiveReader {
@@ -52,16 +54,23 @@ export class ArchiveReader {
           const text = entry.getData().toString('utf8');
           const lines = text.split(/\r?\n/);
           let failureScore = 0;
+          let hasRealError = false;
           for (const line of lines) {
             if (!line.includes('"type":"')) continue;
-            if (line.includes('"type":"after"') && line.includes('"error"')) failureScore += 100;
-            if (line.includes('"type":"error"') && (line.includes('"message"') || line.includes('"error"'))) failureScore += 80;
+            if (line.includes('"type":"after"') && line.includes('"error"')) {
+              failureScore += 100;
+              hasRealError = true;
+            }
+            if (line.includes('"type":"error"') && (line.includes('"message"') || line.includes('"error"'))) {
+              failureScore += 80;
+              hasRealError = true;
+            }
             if (line.includes('"type":"log"')) failureScore += 1;
           }
           if (path.basename(entry.entryName).toLowerCase() === 'test.trace' && failureScore > 0) {
             failureScore += 10;
           }
-          return { entry, failureScore };
+          return { entry, failureScore, hasRealError };
         })
         .sort((a, b) => b.failureScore - a.failureScore || b.entry.header.size - a.entry.header.size);
 
@@ -88,11 +97,17 @@ export class ArchiveReader {
         bestNetwork = prefixMatch ? prefixMatch.entryName : networkCandidates.sort((a, b) => b.header.size - a.header.size)[0]?.entryName;
       }
 
+      const topCandidate = traceCandidates[0];
+      const hasFailure = Boolean(topCandidate?.hasRealError);
+      const topFailureScore = topCandidate?.failureScore ?? 0;
+
       return {
         traceEntryName: bestTrace,
         networkEntryName: bestNetwork,
         traceEntryNames: orderedTraceNames,
         networkEntryNames,
+        hasFailure,
+        failureScore: topFailureScore,
       };
     } catch {
       return {};

package/src/trace/snapshot-resolver.ts

@@ -3,6 +3,13 @@ export type NodeSnapshot =
   | number
   | [string, Record<string, string>, ...NodeSnapshot[]];
 
+export interface BoundingBox {
+  x: number;
+  y: number;
+  width: number;
+  height: number;
+}
+
 export interface ResolvedElement {
   tag: string;
   attributes: Record<string, string>;
@@ -10,6 +17,7 @@ export interface ResolvedElement {
   children: ResolvedElement[];
   role?: string | undefined;
   accessibleName?: string | undefined;
+  boundingBox?: BoundingBox | undefined;
 }
 
 export type SnapshotCompleteness = 'COMPLETE' | 'PARTIAL' | 'UNAVAILABLE';
@@ -115,6 +123,17 @@ export class SnapshotResolver {
       inputValue ??
       textAccumulator.trim();
 
+    let boundingBox: BoundingBox | undefined;
+    const boxAttr = attributes['data-bbox'] ?? attributes['__playwright_box_'] ?? attributes['data-box'];
+    if (boxAttr) {
+      try {
+        const parts = boxAttr.split(',').map(Number);
+        if (parts.length === 4 && parts.every((p) => !isNaN(p))) {
+          boundingBox = { x: parts[0]!, y: parts[1]!, width: parts[2]!, height: parts[3]! };
+        }
+      } catch {}
+    }
+
     return {
       tag,
       attributes,
@@ -122,6 +141,7 @@ export class SnapshotResolver {
       children,
       role,
       accessibleName: accessibleName || undefined,
+      boundingBox,
     };
   }
 
@@ -222,6 +242,17 @@ export class SnapshotResolver {
       inputValue ??
       textAccumulator.trim();
 
+    let boundingBox: BoundingBox | undefined;
+    const boxAttr = attributes['data-bbox'] ?? attributes['__playwright_box_'] ?? attributes['data-box'];
+    if (boxAttr) {
+      try {
+        const parts = boxAttr.split(',').map(Number);
+        if (parts.length === 4 && parts.every((p) => !isNaN(p))) {
+          boundingBox = { x: parts[0]!, y: parts[1]!, width: parts[2]!, height: parts[3]! };
+        }
+      } catch {}
+    }
+
     return {
       tag,
       attributes,
@@ -229,6 +260,7 @@ export class SnapshotResolver {
       children,
       role,
       accessibleName: accessibleName || undefined,
+      boundingBox,
     };
   }