fix(appium): resolve "Unsupported helper type: unknown" in fillField

Appium extends WebDriver but WebElement._detectHelperType only checked
constructor.name === 'WebDriver', so Appium helpers fell through to
'unknown'. fillRichEditor (added in #5527) wraps elements in WebElement
and calls evaluate(), which then threw on every fillField/appendField
call from Appium.

- WebElement now walks the prototype chain so any subclass of
  WebDriver/Playwright/Puppeteer is detected correctly.
- WebDriver.fillField skips fillRichEditor when isWeb === false, since
  rich-editor detection needs a DOM that doesn't exist in Appium native.
- Appium constructor: appiumV2 defaulted via `|| true` (always truthy);
  the v1-deprecation banner then fired on every default-config user.
  Fixed both: explicit `appiumV2: false` is now honored, and the banner
  only prints when the user opts into v1.
- Added regression tests for prototype-chain detection.
- Enabled the Android Appium workflow on pull_request to 4.x so the fix
  can be verified end-to-end via Sauce Labs from the PR.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: DavertMik

.github/workflows/appium_Android.yml

@@ -5,6 +5,9 @@ on:
     branches:
       - 4.x
       - appium-esm-migration
+  pull_request:
+    branches:
+      - 4.x
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}

lib/element/WebElement.js

@@ -15,10 +15,13 @@ class WebElement {
   _detectHelperType(helper) {
     if (!helper) return 'unknown'
 
-    const className = helper.constructor.name
-    if (className === 'Playwright') return 'playwright'
-    if (className === 'WebDriver') return 'webdriver'
-    if (className === 'Puppeteer') return 'puppeteer'
+    let ctor = helper.constructor
+    while (ctor && ctor.name) {
+      if (ctor.name === 'Playwright') return 'playwright'
+      if (ctor.name === 'WebDriver') return 'webdriver'
+      if (ctor.name === 'Puppeteer') return 'puppeteer'
+      ctor = Object.getPrototypeOf(ctor)
+    }
 
     return 'unknown'
   }

lib/helper/Appium.js

@@ -179,13 +179,13 @@ class Appium extends Webdriver {
     super(config)
 
     this.isRunning = false
-    this.appiumV2 = config.appiumV2 || true
+    this.appiumV2 = config.appiumV2 !== false
     this.axios = axios.create()
 
-    if (!config.appiumV2) {
-      console.log('The Appium core team does not maintain Appium 1.x anymore since the 1st of January 2022. Appium 2.x is used by default.')
+    if (this.appiumV2 === false) {
+      console.log('Appium 1.x is no longer maintained by the Appium team (since 2022-01-01).')
       console.log('More info: https://bit.ly/appium-v2-migration')
-      console.log('This Appium 1.x support will be removed in next major release.')
+      console.log('Appium 1.x support will be removed in the next major CodeceptJS release.')
     }
   }
 

lib/helper/WebDriver.js

@@ -1264,7 +1264,7 @@ class WebDriver extends Helper {
     const elem = selectElement(res, field, this)
     highlightActiveElement.call(this, elem)
 
-    if (await fillRichEditor(this, elem, value)) {
+    if (this.isWeb !== false && await fillRichEditor(this, elem, value)) {
       return
     }
 

test/unit/WebElement_test.js

@@ -36,6 +36,22 @@ describe('WebElement', () => {
 
       expect(webElement.helperType).to.equal('unknown')
     })
+
+    it('should detect Appium as webdriver via prototype chain', () => {
+      class WebDriver {}
+      class Appium extends WebDriver {}
+      const webElement = new WebElement({}, new Appium())
+
+      expect(webElement.helperType).to.equal('webdriver')
+    })
+
+    it('should detect subclasses of Playwright as playwright', () => {
+      class Playwright {}
+      class CustomPlaywright extends Playwright {}
+      const webElement = new WebElement({}, new CustomPlaywright())
+
+      expect(webElement.helperType).to.equal('playwright')
+    })
   })
 
   describe('getText()', () => {