@afixt/test-utils 1.1.0 → 1.1.1

package/.claude/settings.local.json

@@ -12,7 +12,9 @@
       "Bash(grep:*)",
       "Bash(touch:*)",
       "Bash(git stash:*)",
-      "Bash(npm publish:*)"
+      "Bash(npm publish:*)",
+      "Bash(git commit:*)",
+      "Bash(git push:*)"
     ]
   },
   "enableAllProjectMcpServers": false

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@afixt/test-utils",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Various utilities for accessibility testing",
   "main": "src/index.js",
   "scripts": {

package/repairs-needed.md

@@ -0,0 +1,84 @@
+# Required Changes for @afixt/test-utils Library
+
+## Issue
+
+The browser extension is throwing "Invalid input: element must be a DOM Element,
+got: null" errors when viewing dynamic sites like CNN.com. The error originates
+from the @afixt/test-utils library functions.
+
+## Root Cause
+
+The `isVisible()` function (and potentially other functions) in
+@afixt/test-utils throws an error when passed null or non-Element arguments.
+This happens frequently during DOM mutations when:
+
+1. Elements are removed from the DOM during repair operations
+2. Mutation observers fire on disconnected elements
+3. Elements become stale between detection and processing
+
+## Required Changes
+
+### 1. isVisible() Function
+
+The function needs to handle null/invalid inputs gracefully:
+
+```javascript
+function isVisible(element, strict = false) {
+  // Add null check at the beginning
+  if (!element || !(element instanceof Element)) {
+    return false;
+  }
+
+  // Check if element is still connected to the DOM
+  if (!element.isConnected) {
+    return false;
+  }
+
+  // ... rest of the existing function
+}
+```
+
+### 2. getAccessibleText() Function
+
+Should handle null elements:
+
+```javascript
+function getAccessibleText(element) {
+  if (!element || !(element instanceof Element)) {
+    return '';
+  }
+
+  if (!element.isConnected) {
+    return '';
+  }
+
+  // ... rest of the existing function
+}
+```
+
+### 3. getAccessibleName() Function
+
+Should handle null elements:
+
+```javascript
+function getAccessibleName(element) {
+  if (!element || !(element instanceof Element)) {
+    return '';
+  }
+
+  if (!element.isConnected) {
+    return '';
+  }
+
+  // ... rest of the existing function
+}
+```
+
+## Why These Changes Are Needed
+
+1. **Defensive Programming**: DOM elements can become null or disconnected at
+   any time, especially in dynamic web applications
+2. **Mutation Observer Behavior**: When observing DOM changes, elements can be
+   removed between the time a mutation is detected and when it's processed
+3. **Library Robustness**: A utility library should handle invalid inputs
+   gracefully rather than throwing errors

package/src/getAccessibleName.js

@@ -7,7 +7,13 @@ const { getAccessibleText } = require("./getAccessibleText.js");
  * @returns {string|boolean} The accessible name or false if none exists
  */
 function getAccessibleName(element) {
-    if (!element) return false;
+    if (!element || !(element instanceof Element)) {
+        return false;
+    }
+
+    if (!element.isConnected) {
+        return false;
+    }
 
     // These are elements which are totally not able to be labeled at all.
     // Even if the title attribute is valid per HTML for these elements,

package/src/getAccessibleText.js

@@ -6,7 +6,13 @@ const { isEmpty } = require("./stringUtils.js");
  * @returns {string} The accessible text.
  */
 function getAccessibleText(el) {
-    if (!el) return "";
+    if (!el || !(el instanceof Element)) {
+        return '';
+    }
+
+    if (!el.isConnected) {
+        return '';
+    }
     
     let textContent = "";
     

package/src/isVisible.js

@@ -5,7 +5,15 @@
  * @returns {boolean}
  */
 function isVisible(element, strict = false) {
-    if (!element) return false;
+    // Add null check at the beginning
+    if (!element || !(element instanceof Element)) {
+        return false;
+    }
+
+    // Check if element is still connected to the DOM
+    if (!element.isConnected) {
+        return false;
+    }
 
     const id = element.id;
     let visible = true;

package/todo.md

@@ -1 +1,2 @@
 # Test-Utils Todo
+