Skip to content

Commit 7a96d61

Browse files
gololdf1shclaude
andcommitted
fix(plugin): handle simplified step objects in retryFailedStep for workers
In `run-workers` mode, step events are serialized via `step.simplify()` which maps `step.name` to `title` (not `name`). The `retryFailedStep` plugin only checked `step.name`, which is `undefined` for simplified step objects, causing a TypeError crash in the `ignoredSteps` loop. Use `step.name || step.title` so the plugin works with both full Step objects (in-process) and simplified ones (from workers). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 6ec6690 commit 7a96d61

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

lib/plugin/retryFailedStep.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -111,11 +111,13 @@ export default function (config) {
111111
}
112112

113113
event.dispatcher.on(event.step.started, step => {
114+
const stepName = step.name || step.title
115+
if (!stepName) return
114116
for (const ignored of config.ignoredSteps) {
115-
if (step.name === ignored) return
117+
if (stepName === ignored) return
116118
if (ignored instanceof RegExp) {
117-
if (step.name.match(ignored)) return
118-
} else if (ignored.indexOf('*') && step.name.startsWith(ignored.slice(0, -1))) return
119+
if (stepName.match(ignored)) return
120+
} else if (ignored.indexOf('*') && stepName.startsWith(ignored.slice(0, -1))) return
119121
}
120122
enableRetry = true
121123
})

0 commit comments

Comments
 (0)