@letsrunit/playwright 0.3.8 → 0.3.9

package/package.json

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

package/src/{locator.ts → fuzzy-locator.ts}

@@ -5,16 +5,18 @@ type LocatorOptions = Parameters<Page['locator']>[1];
 /**
  * Locates an element using Playwright selectors, with fallbacks.
  */
-export async function locator(page: Page, selector: string): Promise<Locator> {
+export async function fuzzyLocator(page: Page, selector: string): Promise<Locator> {
   const primary = page.locator(selector).first();
   if (await primary.count()) return primary;
 
-  return await tryRelaxNameToHasText(page, selector)
-    || await tryTagInsteadOfRole(page, selector)
-    || await tryRoleNameProximity(page, selector)
-    || await tryFieldAlternative(page, selector)
-    || await tryAsField(page, selector)
-    || primary; // Nothing found, return the original locator (so caller can still wait/assert)
+  return (
+    (await tryRelaxNameToHasText(page, selector)) ||
+    (await tryTagInsteadOfRole(page, selector)) ||
+    (await tryRoleNameProximity(page, selector)) ||
+    (await tryFieldAlternative(page, selector)) ||
+    (await tryAsField(page, selector)) ||
+    primary
+  ); // Nothing found, return the original locator (so caller can still wait/assert)
 }
 
 async function firstMatch(page: Page, sel: string | string[], opts: LocatorOptions = {}): Promise<Locator | null> {
@@ -65,11 +67,13 @@ async function tryRoleNameProximity(page: Page, selector: string): Promise<Locat
 }
 
 // Try alternatives if field is not found
-// field="foo" → #foo > input
+// field="foo" → #foo > input  (only when name is a valid CSS identifier)
 async function tryFieldAlternative(page: Page, selector: string): Promise<Locator | null> {
   const matchField = selector.match(/^field="([^"]+)"i?$/i);
   if (!matchField) return null;
   const [, field] = matchField;
+  // Skip if the name contains characters invalid in a CSS ID selector
+  if (!/^[a-zA-Z0-9_-]+$/.test(field)) return null;
   return firstMatch(page, `#${field} > input`);
 }
 

package/src/index.ts

@@ -1,7 +1,7 @@
 export * from './browser';
 export * from './field/index';
 export * from './format-html';
-export * from './locator';
+export * from './fuzzy-locator';
 export * from './selector';
 export * from './snapshot';
 export * from './screenshot';

package/src/selector/field-selector.ts

@@ -109,6 +109,14 @@ export const createFieldEngine = () => ({
       const ph = (el as HTMLInputElement).placeholder ?? el.getAttribute('placeholder');
       if (ph) texts.push(ph);
 
+      // 7) Adjacent sibling <label> (handles <input/><label>Text</label> pattern without for/id)
+      for (let sib = el.nextElementSibling; sib; sib = sib.nextElementSibling) {
+        if (sib.tagName === 'LABEL') { texts.push(sib.textContent ?? ''); break; }
+      }
+      for (let sib = el.previousElementSibling; sib; sib = sib.previousElementSibling) {
+        if (sib.tagName === 'LABEL') { texts.push(sib.textContent ?? ''); break; }
+      }
+
       return texts;
     }