@pixeyedev/playwright 0.4.6 → 0.4.7

package/dist/codemods/from-argos.d.ts

@@ -1,10 +1 @@
-/**
- * Transform Argos CI code to Pixeye.
- * Usage: import and call transformArgos(sourceCode) -> transformed code
- *
- * Handles:
- * - import { argosScreenshot } from '@argos-ci/playwright' -> import { pixeye } from '@pixeyedev/playwright'
- * - argosScreenshot(page, name, opts) -> pixeye.screenshot(page, name, opts)
- * - { argosCSS: '...' } -> { css: '...' }
- */
 export declare const transformArgos: (source: string) => string;

package/dist/codemods/from-argos.js

@@ -1,19 +1,7 @@
-/**
- * Transform Argos CI code to Pixeye.
- * Usage: import and call transformArgos(sourceCode) -> transformed code
- *
- * Handles:
- * - import { argosScreenshot } from '@argos-ci/playwright' -> import { pixeye } from '@pixeyedev/playwright'
- * - argosScreenshot(page, name, opts) -> pixeye.screenshot(page, name, opts)
- * - { argosCSS: '...' } -> { css: '...' }
- */
 export const transformArgos = (source) => {
     let result = source;
-    // 1. Transform import
     result = result.replace(/import\s*\{\s*argosScreenshot\s*\}\s*from\s*['"]@argos-ci\/playwright['"]/g, "import { pixeye } from '@pixeyedev/playwright'");
-    // 2. Transform function calls
     result = result.replace(/argosScreenshot\s*\(/g, 'pixeye.screenshot(');
-    // 3. Transform option names
     result = result.replace(/argosCSS\s*:/g, 'css:');
     return result;
 };

package/dist/codemods/from-percy.d.ts

@@ -1,10 +1 @@
-/**
- * Transform Percy code to Pixeye.
- * Usage: import and call transformPercy(sourceCode) -> transformed code
- *
- * Handles:
- * - import percySnapshot from '@percy/playwright' -> import { pixeye } from '@pixeyedev/playwright'
- * - percySnapshot(page, name, opts) -> pixeye.screenshot(page, name, opts)
- * - { percyCSS: '...' } -> { css: '...' }
- */
 export declare const transformPercy: (source: string) => string;

package/dist/codemods/from-percy.js

@@ -1,19 +1,7 @@
-/**
- * Transform Percy code to Pixeye.
- * Usage: import and call transformPercy(sourceCode) -> transformed code
- *
- * Handles:
- * - import percySnapshot from '@percy/playwright' -> import { pixeye } from '@pixeyedev/playwright'
- * - percySnapshot(page, name, opts) -> pixeye.screenshot(page, name, opts)
- * - { percyCSS: '...' } -> { css: '...' }
- */
 export const transformPercy = (source) => {
     let result = source;
-    // 1. Transform import (default import)
     result = result.replace(/import\s+percySnapshot\s+from\s*['"]@percy\/playwright['"]/g, "import { pixeye } from '@pixeyedev/playwright'");
-    // 2. Transform function calls
     result = result.replace(/percySnapshot\s*\(/g, 'pixeye.screenshot(');
-    // 3. Transform option names
     result = result.replace(/percyCSS\s*:/g, 'css:');
     return result;
 };

package/dist/fixture.d.ts

@@ -1,17 +1,3 @@
-/**
- * Playwright test fixture that provides `pixeye` in every test.
- *
- * Usage:
- *   import { test, expect } from '@pixeyedev/playwright';
- *
- *   test('login page', async ({ page, pixeye }) => {
- *     await page.goto('/login');
- *     await pixeye.screenshot(page, 'login-page');
- *   });
- *
- * The fixture automatically populates testFile and testName from
- * Playwright's testInfo, so metadata is always accurate.
- */
 import { expect } from '@playwright/test';
 import { screenshot, type ScreenshotOptions } from './screenshot.js';
 export interface PixeyeFixture {

package/dist/fixture.js

@@ -1,22 +1,7 @@
-/**
- * Playwright test fixture that provides `pixeye` in every test.
- *
- * Usage:
- *   import { test, expect } from '@pixeyedev/playwright';
- *
- *   test('login page', async ({ page, pixeye }) => {
- *     await page.goto('/login');
- *     await pixeye.screenshot(page, 'login-page');
- *   });
- *
- * The fixture automatically populates testFile and testName from
- * Playwright's testInfo, so metadata is always accurate.
- */
 import { join } from 'node:path';
 import { expect, test as base } from '@playwright/test';
 import { screenshot, setContext } from './screenshot.js';
 export const test = base.extend({
-    // eslint-disable-next-line no-empty-pattern
     pixeye: async ({}, use, testInfo) => {
         const outputDir = join(process.cwd(), '.pixeye', 'screenshots');
         const testFile = testInfo.file ? testInfo.file.replace(process.cwd() + '/', '') : 'unknown';

package/dist/index.js

@@ -1,5 +1,4 @@
 export { expect, test } from './fixture.js';
 export { getContext, PIXEYE_BROWSER_ARGS, screenshot, setContext } from './screenshot.js';
-// ── Convenience namespace (backwards compat with design docs) ───────────────
 import { screenshot as screenshotFn } from './screenshot.js';
 export const pixeye = { screenshot: screenshotFn };

package/dist/reporter.d.ts

@@ -3,25 +3,13 @@ interface ReporterOptions {
     token?: string;
     server?: string;
     uploadConcurrency?: number;
-    /**
-     * Fail the test run (non-zero exit) when the upload is rejected for a
-     * CUSTOMER misconfiguration (bad token, lapsed plan/quota, permissions) so a
-     * run can't pass green with zero screenshots uploaded. Default `true`. Our
-     * transient faults (5xx/network) never fail the run regardless. Opt out with
-     * `failOnError: false` or `PIXEYE_FAIL_ON_ERROR=false`.
-     */
     failOnError?: boolean;
 }
 declare const detectShard: (config: FullConfig) => {
     index: number;
     total: number;
 };
-/** The tests that failed after their retries, by the id the sidecar carries
- *  (`testInfo.testId` = `TestCase.id`, unique across files, describes and
- *  projects); a flaky test's last files are from the attempt that passed. */
 export declare const failedTestIds: (suite: Suite | null) => Set<string>;
-/** The server reads `testStatus` and keeps such a shot out of the review
- *  and out of the baselines (`failed` diff status). */
 export declare const markFailedTests: (metadata: Record<string, unknown> | undefined, failed: Set<string>) => Record<string, unknown> | undefined;
 declare class PixeyeReporter implements Reporter {
     private options;

package/dist/reporter.js

@@ -1,15 +1,12 @@
 import { readFile } from 'node:fs/promises';
 import { join } from 'node:path';
-import { detectCIEnvironment, discoverScreenshots, isUploadMisconfig, PixeyeClient, renderUploadFailureBanner, resolveFailOnError, runParallel, swapImageExtension, uploadErrorSummary } from '@pixeyedev/core';
+import { detectCIEnvironment, discoverScreenshots, PixeyeClient, renderUploadFailureBanner, resolveFailOnError, resolveUploadConcurrency, runParallel, swapImageExtension } from '@pixeyedev/core';
 import { CONTENT_TYPE_BY_FORMAT } from '@pixeyedev/shared';
 import { getContext } from './screenshot.js';
-const DEFAULT_UPLOAD_CONCURRENCY = 32;
 const detectShard = (config) => {
-    // 1. Playwright config (1-indexed → 0-indexed)
     if (config.shard) {
         return { index: config.shard.current - 1, total: config.shard.total };
     }
-    // 2. Environment variables (0-indexed)
     const envIndex = process.env.PIXEYE_SHARD_INDEX;
     const envTotal = process.env.PIXEYE_SHARD_TOTAL;
     if (envIndex !== undefined) {
@@ -18,20 +15,14 @@ const detectShard = (config) => {
             total: envTotal ? parseInt(envTotal, 10) : 1
         };
     }
-    // 3. Default: no sharding
     return { index: 0, total: 1 };
 };
 const detectRunner = () => {
     return 'playwright';
 };
-/** The tests that failed after their retries, by the id the sidecar carries
- *  (`testInfo.testId` = `TestCase.id`, unique across files, describes and
- *  projects); a flaky test's last files are from the attempt that passed. */
 export const failedTestIds = (suite) => new Set((suite ? suite.allTests() : [])
     .filter(test => test.outcome() === 'unexpected')
     .map(test => test.id));
-/** The server reads `testStatus` and keeps such a shot out of the review
- *  and out of the baselines (`failed` diff status). */
 export const markFailedTests = (metadata, failed) => metadata && typeof metadata.testId === 'string' && failed.has(metadata.testId)
     ? { ...metadata, testStatus: 'failed' }
     : metadata;
@@ -56,8 +47,6 @@ class PixeyeReporter {
         this.shard = detectShard(config);
     }
     onTestEnd(_test, _result) {
-        // Screenshots are already saved to disk by pixeye.screenshot().
-        // Nothing to do per-test.
     }
     async onEnd(_result) {
         if (this.skipUpload) {
@@ -68,7 +57,6 @@ class PixeyeReporter {
         if (!token) {
             return;
         }
-        // Discover screenshots saved by pixeye.screenshot()
         const ctx = getContext();
         const outputDir = ctx?.outputDir ?? join(process.cwd(), '.pixeye', 'screenshots');
         let files;
@@ -76,8 +64,6 @@ class PixeyeReporter {
             files = await discoverScreenshots(outputDir);
         }
         catch (err) {
-            // A missing directory yields an empty list, so a THROW here is always a
-            // file we could not read — say which, instead of "no screenshots".
             console.error(`[pixeye] Could not read screenshots in ${outputDir}.`);
             console.error(`[pixeye] ${err.message}`);
             return;
@@ -95,11 +81,10 @@ class PixeyeReporter {
             return;
         }
         const runner = detectRunner();
-        const concurrency = this.options.uploadConcurrency ?? DEFAULT_UPLOAD_CONCURRENCY;
+        let observes = false;
         try {
+            const concurrency = resolveUploadConcurrency(this.options.uploadConcurrency);
             const client = new PixeyeClient({ serverUrl: server, token });
-            // 1. Create build. A run with a failed test is partial: the shots it
-            // never reached are not removed, and keep their baseline.
             const failed = failedTestIds(this.suite);
             const prNumber = process.env.PIXEYE_PR ? parseInt(process.env.PIXEYE_PR, 10) : ci?.prNumber;
             const buildResponse = await client.createBuild({
@@ -108,15 +93,12 @@ class PixeyeReporter {
                 branch,
                 prNumber: prNumber && !isNaN(prNumber) ? prNumber : undefined,
                 shardCount: this.shard.total,
-                partial: failed.size > 0,
                 runner,
                 baseBranch: ci?.baseBranch
             });
             const buildId = buildResponse.build.id;
-            // 2. Presign upload URLs
+            observes = client.observes;
             const hashes = files.map(f => f.fileHash);
-            // Only non-PNG needs declaring: the server defaults to png. Filtering
-            // on 'webp' alone stored a sniffed jpeg under a .png key.
             const formats = Object.fromEntries(files.filter(f => f.format !== 'png').map(f => [f.fileHash, f.format]));
             const presign = await client.presignUpload({
                 buildId,
@@ -128,7 +110,6 @@ class PixeyeReporter {
                     console.warn(`[pixeye] ${w}`);
                 }
             }
-            // 3. Upload new files
             const hashUrlPairs = Object.entries(presign.urls);
             const hashToFile = new Map(files.map(f => [f.fileHash, f]));
             let uploaded = 0;
@@ -144,9 +125,7 @@ class PixeyeReporter {
                     uploaded++;
                 });
             }
-            // 4. Read metadata sidecars and build manifest
             const manifest = await Promise.all(files.map(async (f) => {
-                // Try to read companion .meta.json
                 const metaPath = swapImageExtension(f.filePath, '.meta.json');
                 let metadata;
                 try {
@@ -154,7 +133,6 @@ class PixeyeReporter {
                     metadata = JSON.parse(metaContent);
                 }
                 catch {
-                    // No sidecar — that's fine
                 }
                 return {
                     name: f.name,
@@ -165,7 +143,6 @@ class PixeyeReporter {
                     metadata: markFailedTests(metadata, failed)
                 };
             }));
-            // 5. Submit shard
             await client.submitShard({
                 buildId,
                 shardIndex: this.shard.index,
@@ -174,23 +151,10 @@ class PixeyeReporter {
             console.log(`[pixeye] ${files.length} screenshots (${uploaded} new, ${presign.existing.length} cached) — ${server}/builds/${buildId}`);
         }
         catch (err) {
-            // P0-2: a CUSTOMER misconfiguration (bad token, lapsed plan/quota,
-            // permissions) must NOT pass silently — surface a loud banner and fail
-            // the run by default so a green CI can't hide zero uploaded screenshots.
-            // Our transient faults (5xx/network/R2) only warn — never break their CI.
-            if (isUploadMisconfig(err)) {
-                console.error(renderUploadFailureBanner(err, { willFail: this.failOnError }));
-                if (this.failOnError) {
-                    // Playwright IGNORES `process.exitCode` — its runner calls
-                    // `process.exit(status === 'passed' ? 0 : 1)`, overriding it. The ONLY
-                    // reporter-side lever is returning a status from onEnd, which the
-                    // multiplexer applies (`result.status = outResult.status`). Return
-                    // 'failed' so a passing run with a rejected upload still exits non-zero.
-                    return { status: 'failed' };
-                }
-            }
-            else {
-                console.warn(`[pixeye] Upload failed (transient — CI not failed): ${uploadErrorSummary(err)}`);
+            const willFail = this.failOnError && !observes;
+            console.error(renderUploadFailureBanner(err, { willFail }));
+            if (willFail) {
+                return { status: 'failed' };
             }
         }
     }

package/dist/screenshot.d.ts

@@ -1,5 +1,4 @@
 import type { Locator, Page } from '@playwright/test';
-/** Browser args recommended for deterministic screenshots. */
 export declare const PIXEYE_BROWSER_ARGS: string[];
 export interface ScreenshotOptions {
     fullPage?: boolean;

package/dist/screenshot.js

@@ -2,21 +2,16 @@ import { createHash } from 'node:crypto';
 import { mkdir, readFile, writeFile } from 'node:fs/promises';
 import { dirname, join } from 'node:path';
 import { fileURLToPath } from 'node:url';
-/** Browser args recommended for deterministic screenshots. */
 export const PIXEYE_BROWSER_ARGS = [
     '--force-device-scale-factor=1',
     '--font-render-hinting=none',
     '--disable-lcd-text'
 ];
-// Shared context set by the reporter or test setup
 let _context = null;
 export const setContext = (ctx) => {
     _context = ctx;
 };
 export const getContext = () => _context;
-// ── IIFE loader ──────────────────────────────────────────────────────────────
-// The browser IIFE is copied into dist/ at build time (see package.json build script).
-// This avoids a runtime dependency on @pixeyedev/browser, which is private/internal.
 const THIS_DIR = dirname(fileURLToPath(import.meta.url));
 let _iifeCache = null;
 const loadIIFE = async () => {
@@ -26,13 +21,9 @@ const loadIIFE = async () => {
     _iifeCache = await readFile(join(THIS_DIR, 'browser.iife.js'), 'utf-8');
     return _iifeCache;
 };
-// ── Helpers ──────────────────────────────────────────────────────────────────
 const injectBrowserScripts = async (page) => {
     const iife = await loadIIFE();
-    // Check if already injected to avoid double-injection
-    const alreadyInjected = await page.evaluate(
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    () => typeof window.__pixeye !== 'undefined');
+    const alreadyInjected = await page.evaluate(() => typeof window.__pixeye !== 'undefined');
     if (!alreadyInjected) {
         await page.evaluate(iife);
     }
@@ -49,7 +40,6 @@ const isLocator = (target) => {
 const sanitizePath = (str) => {
     return str.replace(/[^a-zA-Z0-9._/-]/g, '_');
 };
-// ── Mask resolution ──────────────────────────────────────────────────────────
 const resolveMaskRegions = async (page, masks) => {
     const regions = [];
     for (const selectorOrLocator of masks) {
@@ -72,7 +62,6 @@ const resolveMaskRegions = async (page, masks) => {
             }
         }
         else {
-            // Locator
             const box = await selectorOrLocator.boundingBox();
             if (box) {
                 regions.push({
@@ -90,7 +79,6 @@ const resolveMaskRegions = async (page, masks) => {
     }
     return regions;
 };
-// ── Capture with retry ───────────────────────────────────────────────────────
 const RETRY_DELAY_MS = 100;
 const captureOnce = async (target, options) => {
     const screenshotOpts = {};
@@ -115,7 +103,6 @@ const captureStable = async (target, options) => {
     if (retries <= 0) {
         return captureOnce(target, options);
     }
-    // Capture retries+1 times, find first consecutive duplicate (stable frame)
     const captures = [];
     const page = getPage(target);
     for (let i = 0; i <= retries; i++) {
@@ -130,13 +117,10 @@ const captureStable = async (target, options) => {
             return captures[i];
         }
     }
-    // No stable frame found — return the last capture
     return captures[captures.length - 1];
 };
-// ── Main screenshot function ─────────────────────────────────────────────────
 export const screenshot = async (target, name, options = {}) => {
     const page = getPage(target);
-    // Determine output directory
     const outputDir = _context?.outputDir ?? join(process.cwd(), '.pixeye', 'screenshots');
     const testFile = _context?.testFile ?? 'unknown';
     const testName = _context?.testName ?? '';
@@ -145,38 +129,27 @@ export const screenshot = async (target, name, options = {}) => {
     await mkdir(dir, { recursive: true });
     const pngPath = join(dir, `${name}.png`);
     const metaPath = join(dir, `${name}.meta.json`);
-    // 1. Inject browser scripts
     await injectBrowserScripts(page);
-    // 2. Stabilize the page
     await page.evaluate((opts) => {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         window.__pixeye.stabilize({
             css: opts.css,
             colorScheme: opts.colorScheme
         });
     }, { css: options.css, colorScheme: options.colorScheme });
-    // 3. Wait for page to be ready
     await page.evaluate((timeout) => {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         return window.__pixeye.waitForReady({ timeout });
     }, options.waitForTimeout ?? 10_000);
-    // 4. Custom waitFor selector
     if (options.waitFor) {
         await page.waitForSelector(options.waitFor, {
             timeout: options.waitForTimeout ?? 10_000
         });
     }
-    // 5. Scan DOM for region attributes
     const scannedRegions = await page.evaluate(() => {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         return window.__pixeye.scanRegions();
     });
-    // 6. Resolve mask selectors/locators to region bounds
     const maskRegions = options.mask ? await resolveMaskRegions(page, options.mask) : [];
     const allRegions = [...scannedRegions, ...maskRegions];
-    // 7. Capture screenshot (with optional retry for stability)
     const buffer = await captureStable(target, options);
-    // 8. Get component bounds for element-level screenshots
     let componentBounds = null;
     if (isLocator(target)) {
         const box = await target.boundingBox();
@@ -189,10 +162,8 @@ export const screenshot = async (target, name, options = {}) => {
             };
         }
     }
-    // 9. Gather viewport + browser info
     const viewportSize = page.viewportSize();
     const browserName = page.context().browser()?.browserType().name() ?? 'unknown';
-    // 10. Build metadata sidecar
     const metadata = {
         version: 1,
         viewport: viewportSize ? { width: viewportSize.width, height: viewportSize.height } : undefined,
@@ -205,7 +176,6 @@ export const screenshot = async (target, name, options = {}) => {
         regions: allRegions.length > 0 ? allRegions : undefined,
         componentBounds
     };
-    // 11. Write .png and .meta.json
     await writeFile(pngPath, buffer);
     await writeFile(metaPath, JSON.stringify(metadata, null, 2) + '\n');
     return pngPath;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pixeyedev/playwright",
-  "version": "0.4.6",
+  "version": "0.4.7",
   "license": "MIT",
   "private": false,
   "type": "module",
@@ -35,8 +35,8 @@
     "ai-check": "exit 0"
   },
   "dependencies": {
-    "@pixeyedev/core": "^0.4.0",
-    "@pixeyedev/shared": "^0.4.0"
+    "@pixeyedev/core": "^0.4.7",
+    "@pixeyedev/shared": "^0.4.7"
   },
   "peerDependencies": {
     "@playwright/test": ">=1.40.0"