halo-agent 2.4.1 → 2.4.3

package/detectFormErrors.js

@@ -429,12 +429,19 @@ async function detectRequiredEmptiesInRoot(page) {
       //      control (not a shared group container with multiple inputs).
       const labelTextOf = (lblEl) => (lblEl && (lblEl.innerText || lblEl.textContent || '')).trim();
       const marks = (txt) => /\*\s*$/.test(txt) || /\(\s*required\s*\)/i.test(txt) || /\brequired\b\s*\*?$/i.test(txt);
+      // Class-marker required detection. Ashby renders the asterisk via a
+      // CSS ::after pseudo-element on a label whose className includes
+      // "_required_" / "required". The text-based `marks()` check above can't
+      // see pseudo-content, so we also accept a className match. Same idea
+      // covers many other React component-libraries (Workable, custom
+      // tailwind builds) that don't put the literal "*" in label text.
+      const labelHasRequiredClass = (lblEl) => !!(lblEl && lblEl.className && /(^|[\s_-])required($|[\s_-])/i.test(lblEl.className));
 
       if (el.id) {
         const lbl = document.querySelector(`label[for="${el.id}"]`);
-        if (lbl) return marks(labelTextOf(lbl));
+        if (lbl) return marks(labelTextOf(lbl)) || labelHasRequiredClass(lbl);
       }
-      if (el.labels && el.labels[0]) return marks(labelTextOf(el.labels[0]));
+      if (el.labels && el.labels[0]) return marks(labelTextOf(el.labels[0])) || labelHasRequiredClass(el.labels[0]);
 
       // Closest wrapping label/legend, but ONLY if that container holds a
       // single control (so we don't inherit a group asterisk meant for a
@@ -444,7 +451,7 @@ async function detectRequiredEmptiesInRoot(page) {
         const labelEl = p.querySelector(':scope > label, :scope > legend, :scope > [class*="label"]');
         if (labelEl) {
           const controlsInScope = p.querySelectorAll('input:not([type=hidden]), textarea, select, [role="combobox"], [contenteditable="true"]').length;
-          if (controlsInScope <= 1) return marks(labelTextOf(labelEl));
+          if (controlsInScope <= 1) return marks(labelTextOf(labelEl)) || labelHasRequiredClass(labelEl);
           // Shared container with multiple controls — the asterisk isn't
           // necessarily ours. Stop; treat as not-required unless a
           // programmatic marker said otherwise (handled above).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "halo-agent",
-  "version": "2.4.1",
+  "version": "2.4.3",
   "description": "HALO local apply agent — auto-fills job applications using your real Chrome session",
   "main": "index.js",
   "bin": {