@letsrunit/playwright 0.17.1 → 0.18.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@letsrunit/playwright",
-  "version": "0.17.1",
+  "version": "0.18.0",
   "description": "Playwright extensions and utilities for letsrunit",
   "keywords": [
     "testing",
@@ -42,7 +42,7 @@
   },
   "packageManager": "yarn@4.10.3",
   "dependencies": {
-    "@letsrunit/utils": "0.17.1",
+    "@letsrunit/utils": "0.18.0",
     "@playwright/test": "^1.57.0",
     "case": "^1.6.3",
     "diff": "^8.0.3",

package/src/scrub-html.ts

@@ -32,7 +32,7 @@ export type ScrubHtmlOptions = {
   /** Limit lists to max items: -1 mean no limit. Default: -1 */
   limitLists?: number;
   /** Strip utility-framework classes (Tailwind, Bootstrap, UnoCSS, Windi) from class
-   *  attributes. Removes the attribute entirely when all classes are stripped. Default: false */
+   *  attributes. Removes the attribute entirely when all classes are stripped. Default: true */
   dropUtilityClasses?: boolean;
 };
 
@@ -51,7 +51,7 @@ function getDefaults(contentLength: number): Required<ScrubHtmlOptions> {
     dropComments: true,
     replaceBrInHeadings: true,
     limitLists: contentLength >= HTML_LIMIT_LISTS_THRESHOLD ? 20 : -1,
-    dropUtilityClasses: false,
+    dropUtilityClasses: true,
   };
 }
 
@@ -350,10 +350,25 @@ const UTILITY_STANDALONE = new Set([
   'sr-only', 'not-sr-only', 'clearfix', 'row', 'col',
 ]);
 
+const SPACING_ALPHA_VALUES = new Set(['auto', 'px']);
+
+function isSpacingUtility(token: string): boolean {
+  const match = token.match(/^-?([mp][xytblrse]?)-(.*)$/i);
+  if (!match) return false;
+
+  const value = match[2];
+  if (!value) return false;
+
+  if (/^[0-9./[\]-]+$/.test(value)) return true;
+  if (SPACING_ALPHA_VALUES.has(value.toLowerCase())) return true;
+  return false;
+}
+
 function isUtilityClass(token: string): boolean {
   if (UTILITY_VARIANT_RE.test(token)) return true;
   const base = token.startsWith('-') ? token.slice(1) : token;
   if (UTILITY_STANDALONE.has(base)) return true;
+  if (/^-?[mp][xytblrse]?-/i.test(token)) return isSpacingUtility(token);
   return UTILITY_PREFIX_RE.test(token);
 }
 

package/src/snapshot.ts

@@ -6,7 +6,7 @@ import type { Snapshot } from './types';
 import { waitForDomIdle } from './wait';
 
 export type SnapshotOptions = {
-  /** Strip utility-framework classes (Tailwind, Bootstrap, UnoCSS, Windi) from the captured HTML. */
+  /** Strip utility-framework classes (Tailwind, Bootstrap, UnoCSS, Windi) from the captured HTML. Default: true. */
   dropUtilityClasses?: boolean;
 };
 
@@ -16,7 +16,9 @@ export async function snapshot(page: Page, opts: SnapshotOptions = {}): Promise<
 
   const [url, html, file] = await Promise.all([page.url(), getContentWithMarkedHidden(page), screenshot(page)]);
 
-  const finalHtml = opts.dropUtilityClasses
+  const dropUtilityClasses = opts.dropUtilityClasses ?? true;
+
+  const finalHtml = dropUtilityClasses
     ? await realScrubHtml({ html, url }, {
         dropHidden: false, dropHead: false, dropSvg: false, pickMain: false,
         stripAttributes: 0, normalizeWhitespace: false, dropComments: false,