testronaut 2.0.2 → 2.0.3

package/TODO.md

@@ -12,6 +12,7 @@
 ## Browser Automation Follow-Ups
 
 - [x] Make `click_text` match exact text, prefer the innermost visible element, and fail quickly instead of clicking an ancestor or partial match.
+- [ ] Replace framework/vendor-specific DOM class stripping with a framework-agnostic classifier that removes generated hashes, styling utilities, variants, and arbitrary selectors while preserving semantic component classes. Reconsider broad prefix rules such as `/^cl-/`, cap retained class count and length, and add representative fixtures for Tailwind, MUI, Ant Design, Chakra, Clerk, CSS Modules, and CSS-in-JS.
 
 ## Session and Reporting Follow-Ups
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testronaut",
-  "version": "2.0.2",
+  "version": "2.0.3",
   "main": "./runner/testronaut.js",
   "bin": {
     "testronaut": "bin/cli.js"

package/tests/toolsTests/chromeBrowserClickText.test.js

@@ -42,3 +42,31 @@ describe('ChromeBrowser.click_text', () => {
     expect(browser.page.getByText).toHaveBeenCalledWith('Add', { exact: true });
   });
 });
+
+describe('ChromeBrowser.check_text', () => {
+  it('reports text when at least one matching element is visible', async () => {
+    const matches = {
+      count: vi.fn().mockResolvedValue(2),
+      nth: vi.fn(index => [
+        { isVisible: vi.fn().mockResolvedValue(false) },
+        { isVisible: vi.fn().mockResolvedValue(true) },
+      ][index]),
+    };
+    const browser = Object.create(ChromeBrowser.prototype);
+    browser.page = { getByText: vi.fn().mockReturnValue(matches) };
+
+    await expect(browser.check_text({ text: 'Signed out' })).resolves.toBe('FOUND: Signed out');
+    expect(browser.page.getByText).toHaveBeenCalledWith('Signed out');
+  });
+
+  it('does not report text that only exists in hidden elements', async () => {
+    const matches = {
+      count: vi.fn().mockResolvedValue(1),
+      nth: vi.fn().mockReturnValue({ isVisible: vi.fn().mockResolvedValue(false) }),
+    };
+    const browser = Object.create(ChromeBrowser.prototype);
+    browser.page = { getByText: vi.fn().mockReturnValue(matches) };
+
+    await expect(browser.check_text({ text: 'Hidden message' })).resolves.toBe('NOT FOUND: Hidden message');
+  });
+});

package/tests/toolsTests/domControl.test.js

@@ -285,7 +285,7 @@ describe('domControl helpers', () => {
       const html = `
         <button
           id="btn"
-          class="text-sm bg-red-500 hover:bg-red-600 flex items-center btn-primary custom-class"
+          class="text-sm bg-red-500 hover:bg-red-600 disabled:opacity-50 [&amp;_svg]:size-4 flex items-center btn-primary custom-class"
         >
           Click
         </button>
@@ -301,6 +301,8 @@ describe('domControl helpers', () => {
       expect(classes).not.toContain('text-sm');
       expect(classes).not.toContain('bg-red-500');
       expect(classes).not.toContain('hover:bg-red-600');
+      expect(classes).not.toContain('disabled:opacity-50');
+      expect(classes).not.toContain('[&_svg]:size-4');
       expect(classes).not.toContain('flex');
       expect(classes).not.toContain('items-center');
 

package/tools/chromeBrowser.js

@@ -1135,8 +1135,14 @@ export class ChromeBrowser {
 
   
   async check_text({ text }) {
-    const html = await this.page.content();
-    return html.includes(text) ? `FOUND: ${text}` : `NOT FOUND: ${text}`;
+    const matches = this.page.getByText(text);
+    const count = await matches.count();
+    for (let index = 0; index < count; index++) {
+      if (await matches.nth(index).isVisible().catch(() => false)) {
+        return `FOUND: ${text}`;
+      }
+    }
+    return `NOT FOUND: ${text}`;
   }
 
   async click_text({ text, timeout = 3000 }) {

package/tools/domControl.js

@@ -271,7 +271,8 @@ export function removeAutoGeneratedClasses($) {
     const $el = $(el);
     const classes = ($el.attr('class') || '').split(/\s+/);
     const filtered = classes.filter(cls => {
-      return !utilityClassPatterns.some(pattern => pattern.test(cls));
+      const isVariantOrArbitraryUtility = cls.includes(':') || cls.includes('[');
+      return !isVariantOrArbitraryUtility && !utilityClassPatterns.some(pattern => pattern.test(cls));
     });
     if (filtered.length) {
       $el.attr('class', filtered.join(' '));

package/tools/toolSchema.js

@@ -95,7 +95,7 @@ const toolsSchema = [
     type: 'function',
     function: {
       name: 'check_text',
-      description: 'Search page HTML for specific text',
+      description: 'Search the page for specific visible text',
       parameters: {
         type: 'object',
         properties: {