Skip to content

feat(kleur-to-util-styletext): introduce recipe#485

Open
Herrtian wants to merge 4 commits into
nodejs:mainfrom
Herrtian:kleur-to-util-styletext-479
Open

feat(kleur-to-util-styletext): introduce recipe#485
Herrtian wants to merge 4 commits into
nodejs:mainfrom
Herrtian:kleur-to-util-styletext-479

Conversation

@Herrtian

@Herrtian Herrtian commented Jun 5, 2026

Copy link
Copy Markdown

Fixes #479

Summary

  • add a kleur-to-util-styletext migration recipe
  • convert kleur default, namespace, and CommonJS style calls to util.styleText
  • convert kleur/colors named style imports, including nested calls like bgRed(white("FAIL"))
  • remove the kleur dependency from package.json fixtures

Validation

  • npm.cmd test --workspace @nodejs/kleur-to-util-styletext
  • npm.cmd run lint
  • npm.cmd run type-check

@AugustinMauroy

Copy link
Copy Markdown
Member

@Herrtian is voluntary draft ?

@Herrtian Herrtian marked this pull request as ready for review June 6, 2026 12:48
@Herrtian

Herrtian commented Jun 6, 2026

Copy link
Copy Markdown
Author

It was left as draft by mistake. I marked it ready for review.

@AugustinMauroy AugustinMauroy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • add jsdocs
  • use correct tests structure describe on CONTRIBUTING.md
  • add more tests cases such as dynamic import with await & then methode or esm alias

Comment thread recipes/kleur-to-util-styletext/src/workflow.ts Outdated
Comment thread recipes/kleur-to-util-styletext/src/workflow.ts Outdated
Comment thread recipes/kleur-to-util-styletext/src/workflow.ts Outdated
Comment thread recipes/kleur-to-util-styletext/src/workflow.ts Outdated
Comment thread recipes/kleur-to-util-styletext/src/workflow.ts Outdated
Comment thread recipes/kleur-to-util-styletext/README.md Outdated
Comment thread recipes/kleur-to-util-styletext/README.md Outdated
@AugustinMauroy AugustinMauroy added awaiting author Reviewer has requested something from the author ⚠️ fully-AI-generated labels Jun 6, 2026
Comment thread recipes/kleur-to-util-styletext/src/workflow.ts Outdated
@AugustinMauroy AugustinMauroy added awaiting reviewer Author has responded and needs action from the reviewer and removed awaiting author Reviewer has requested something from the author labels Jun 7, 2026
@AugustinMauroy AugustinMauroy requested a review from a team June 7, 2026 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting reviewer Author has responded and needs action from the reviewer ⚠️ fully-AI-generated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

spec(kleur): migrate kleur to node:util styleText

2 participants