Skip to content

feat(hooks): pass in Listr2 context to simple hooks#4174

Open
erickzhao wants to merge 2 commits intonextfrom
listr2-hooks
Open

feat(hooks): pass in Listr2 context to simple hooks#4174
erickzhao wants to merge 2 commits intonextfrom
listr2-hooks

Conversation

@erickzhao
Copy link
Copy Markdown
Member

@erickzhao erickzhao commented Mar 17, 2026

Allows userland config/hooks to read into the Listr2 context so logs can get written to task.output.

@erickzhao erickzhao marked this pull request as ready for review March 18, 2026 05:45
@erickzhao erickzhao requested a review from a team as a code owner March 18, 2026 05:45
@erickzhao erickzhao changed the title feat(hooks): pass Listr2 context feat(hooks): pass in Listr2 context Mar 18, 2026
@erickzhao erickzhao changed the title feat(hooks): pass in Listr2 context feat(hooks): pass in Listr2 context to simple hooks Mar 18, 2026
@erickzhao erickzhao added the next label Mar 19, 2026
Copy link
Copy Markdown
Member

@MarshallOfSound MarshallOfSound left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is already possible today via the exported namedHookWithTaskFn helper. Does that not solve this problem?

Copy link
Copy Markdown
Member

@MarshallOfSound MarshallOfSound left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Block on question

@erickzhao
Copy link
Copy Markdown
Member Author

erickzhao commented Mar 31, 2026

This is already possible today via the exported namedHookWithTaskFn helper. Does that not solve this problem?

That helper gives a static name to the hook for Listr2, but I specifically want to take a slew of console.log statements that are currently in a userland hook and pass it to the Task output to show progress on a task.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants