Skip to content

Commit 63ea914

Browse files
gololdf1shclaude
andcommitted
fix(step): preserve step.name in simplify() for worker IPC
step.simplify() maps name → title, dropping name from the serialized object. Plugins listening to step events in the master process (e.g. retryFailedStep) crash on step.name being undefined. Add name to simplify() output alongside title (kept for BC). Add null-guard in retryFailedStep for edge cases where step.name is undefined on the full Step object (hook steps). Closes testomatio/e2e-tests#139 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 6ec6690 commit 63ea914

2 files changed

Lines changed: 2 additions & 0 deletions

File tree

lib/plugin/retryFailedStep.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ export default function (config) {
111111
}
112112

113113
event.dispatcher.on(event.step.started, step => {
114+
if (!step.name) return
114115
for (const ignored of config.ignoredSteps) {
115116
if (step.name === ignored) return
116117
if (ignored instanceof RegExp) {

lib/step/base.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -223,6 +223,7 @@ class Step {
223223

224224
return {
225225
opts: step.opts || {},
226+
name: step.name,
226227
title: step.name,
227228
args: args,
228229
status: step.status,

0 commit comments

Comments
 (0)