snap-ally 0.2.1-beta → 0.2.3-beta

package/README.md

@@ -7,6 +7,19 @@ A powerful, developer-friendly Playwright reporter for **Accessibility testing**
 
 ---
 
+## 🤔 Why the name "Snap-Ally"?
+
+- **Snap**: Like a snapshot, it provides an instant picture of a website's accessibility state at the moment the tests are executed.
+- **Ally**: It resembles **a11y** (the abbreviation for accessibility) and serves as an ally that allows you to create bugs in Azure DevOps more easily.
+
+## 💡 Motivation
+
+I have seen closely how much people with disabilities struggle with something as fundamental as finding a job.
+
+I believe that with relatively simple changes in HTML, good color contrast, among other things, systems should work and help all people equally. Since about 15% of the world's population lives with some form of disability.
+
+---
+
 ## 📺 Demo
 
 **[▶️ Watch the Demo Video](https://www.loom.com/share/853c04f1f76242a699e8f82e54733007)**

package/dist/A11yAuditOverlay.d.ts

@@ -1,4 +1,4 @@
-import { Page, TestInfo } from '@playwright/test';
+import type { Page, TestInfo } from '@playwright/test';
 /**
  * Handles visual feedback and Playwright annotations during an accessibility audit.
  *

package/dist/A11yAuditOverlay.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.A11yAuditOverlay = void 0;
-const test_1 = require("@playwright/test");
 /**
  * Handles visual feedback and Playwright annotations during an accessibility audit.
  *
@@ -261,7 +260,9 @@ class A11yAuditOverlay {
      * @returns The raw screenshot buffer.
      */
     async captureAndAttachScreenshot(fileName, testInfo) {
-        return await test_1.test.step('Capture A11y screenshot', async () => {
+        // eslint-disable-next-line @typescript-eslint/no-require-imports
+        const { test } = require('@playwright/test');
+        return await test.step('Capture A11y screenshot', async () => {
             const screenshot = await this.page.screenshot({ fullPage: false });
             await testInfo.attach(fileName, { contentType: 'image/png', body: screenshot });
             return screenshot;

package/dist/A11yScanner.d.ts

@@ -1,4 +1,4 @@
-import { Page, Locator, TestInfo } from '@playwright/test';
+import type { Page, Locator, TestInfo } from '@playwright/test';
 export interface A11yScannerOptions {
     /** Specific selector or locator to include in the scan. */
     include?: string | Locator;

package/dist/A11yScanner.js

@@ -6,7 +6,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.checkAccessibility = void 0;
 exports.scanA11y = scanA11y;
 const playwright_1 = __importDefault(require("@axe-core/playwright"));
-const test_1 = require("@playwright/test");
 const A11yAuditOverlay_1 = require("./A11yAuditOverlay");
 const models_1 = require("./models");
 const A11yTimeUtils_1 = require("./A11yTimeUtils");
@@ -96,7 +95,10 @@ async function scanA11y(page, testInfo, options = {}) {
         }
     }
     // Fail the test if violations found (softly)
-    test_1.expect
+    // Dynamically require to avoid eager loading @playwright/test during config evaluation
+    // eslint-disable-next-line @typescript-eslint/no-require-imports
+    const { expect } = require('@playwright/test');
+    expect
         .soft(violationCount, `Accessibility audit failed with ${violationCount} violations.`)
         .toBe(0);
     // Run Axe Audit
@@ -167,7 +169,6 @@ async function scanA11y(page, testInfo, options = {}) {
         pageUrl: page.url(),
         accessibilityScore: 0, // No longer used, derivation from Lighthouse removed
         a11yErrors: errors,
-        video: 'a11y-scan-video.webm', // Reference name for reporter
         criticalColor: models_1.Severity.critical,
         seriousColor: models_1.Severity.serious,
         moderateColor: models_1.Severity.moderate,

package/dist/models/index.d.ts

@@ -2,7 +2,7 @@ export interface ReportData {
     pageKey: string;
     pageUrl?: string;
     accessibilityScore: number;
-    video: string;
+    video?: string;
     a11yErrors: A11yError[];
     criticalColor: string;
     seriousColor: string;

package/dist/templates/report-app.js

@@ -894,10 +894,15 @@ async function submitFinalBug() {
     let videoUrl = null;
     if (videoPath) {
         try {
-            const videoBlob = await fetch(videoPath).then((res) => res.blob());
-            videoUrl = await uploadAttachment(videoBlob, 'session-recording.webm');
+            const res = await fetch(videoPath);
+            if (res.ok) {
+                const videoBlob = await res.blob();
+                videoUrl = await uploadAttachment(videoBlob, 'session-recording.webm');
+            } else {
+                console.warn(`[SnapAlly] Video not found or access denied: ${videoPath}. Skipping video upload.`);
+            }
         } catch (e) {
-            console.error('Failed to video upload', e);
+            console.warn(`[SnapAlly] Failed to load video: ${e.message}. Skipping video upload.`);
         }
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "snap-ally",
-    "version": "0.2.1-beta",
+    "version": "0.2.3-beta",
     "description": "A custom Playwright reporter for Accessibility testing using Axe, with HTML reporting and Azure DevOps integration.",
     "main": "dist/index.js",
     "types": "dist/index.d.ts",