Remove user-facing when option from retryFailedStep config

The internal retry predicate (config.when) is still set for the
recorder; only the configurable customWhen hook is removed.

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

Author: DavertMik

docs/plugins/retryFailedStep.md

@@ -29,7 +29,6 @@ Run tests with plugin enabled:
 #### Configuration:
 
 *   `retries` - number of retries (by default 3),
-*   `when` - function, when to perform a retry (accepts error as parameter)
 *   `factor` - The exponential factor to use. Default is 1.5.
 *   `minTimeout` - The number of milliseconds before starting the first retry. Default is 150.
 *   `maxTimeout` - The maximum number of milliseconds between two retries. Default is 10000.

lib/plugin/retryFailedStep.js

@@ -47,7 +47,6 @@ const RETRY_PRIORITIES = {
  * #### Configuration:
  *
  * * `retries` - number of retries (by default 3),
- * * `when` - function, when to perform a retry (accepts error as parameter)
  * * `factor` - The exponential factor to use. Default is 1.5.
  * * `minTimeout` - The number of milliseconds before starting the first retry. Default is 150.
  * * `maxTimeout` - The maximum number of milliseconds between two retries. Default is 10000.
@@ -94,7 +93,6 @@ const RETRY_PRIORITIES = {
 export default function (config) {
   config = Object.assign({}, defaultConfig, config)
   config.ignoredSteps = config.ignoredSteps.concat(config.defaultIgnoredSteps)
-  const customWhen = config.when
 
   let enableRetry = false
 
@@ -104,7 +102,6 @@ export default function (config) {
     if (!store.autoRetries) return false
     if (err && err.isTerminal) return false
     if (err && err.message && (err.message.includes('ERR_ABORTED') || err.message.includes('frame was detached') || err.message.includes('Target page, context or browser has been closed'))) return false
-    if (customWhen) return customWhen(err)
     return true
   }
   config.when = when