Skip to content

deps(create-catalyst): replace chalk with consola colorize utility function#2591

Merged
matthewvolk merged 2 commits intocanaryfrom
CATALYST-1454-lock-chalk
Sep 8, 2025
Merged

deps(create-catalyst): replace chalk with consola colorize utility function#2591
matthewvolk merged 2 commits intocanaryfrom
CATALYST-1454-lock-chalk

Conversation

@matthewvolk
Copy link
Copy Markdown
Contributor

@matthewvolk matthewvolk commented Sep 8, 2025

What/Why?

Removes chalk dependency in favor of consola "colorize" utility function (which only depends on node:tty)

Testing

Text is still colorized appropriately:

Screenshot 2025-09-08 at 12 58 11 PM

Migration

N/A

Closes #2592

@matthewvolk matthewvolk requested a review from a team September 8, 2025 17:31
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Sep 8, 2025

🦋 Changeset detected

Latest commit: 78def1b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bigcommerce/create-catalyst Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 8, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
catalyst-b2b Ready Ready Preview Comment Sep 8, 2025 6:02pm
catalyst-canary Ready Ready Preview Comment Sep 8, 2025 6:02pm
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
catalyst Ignored Ignored Sep 8, 2025 6:02pm
catalyst-au Ignored Ignored Sep 8, 2025 6:02pm
catalyst-uk Ignored Ignored Sep 8, 2025 6:02pm

Copy link
Copy Markdown
Contributor

@jorgemoya jorgemoya left a comment

Choose a reason for hiding this comment

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

Is there a particular reason for this?

@adamdyche
Copy link
Copy Markdown

Is there a particular reason for this?

https://www.aikido.dev/blog/npm-debug-and-chalk-packages-compromised

@chanceaclark
Copy link
Copy Markdown
Contributor

chanceaclark commented Sep 8, 2025

The only thing that would be affected is the github actions, which doesn’t affect any users. That compromised version is already removed off of npm registries so we don’t need to worry about it being installed.

@matthewvolk matthewvolk changed the title deps(create-catalyst): lock chalk to 5.4.1 deps(create-catalyst): replace chalk with consola colorize utility function Sep 8, 2025
@matthewvolk
Copy link
Copy Markdown
Contributor Author

I opted to just completely remove chalk in the latest commit, and replace it with what we're now using in the @bigcommerce/catalyst CLI, which is the colorize utility from consola: https://github.com/unjs/consola/blob/main/src/utils/color.ts

@matthewvolk matthewvolk added this pull request to the merge queue Sep 8, 2025
Merged via the queue into canary with commit f791fef Sep 8, 2025
13 checks passed
@matthewvolk matthewvolk deleted the CATALYST-1454-lock-chalk branch September 8, 2025 18:14
Copy link
Copy Markdown
Contributor

@jorgemoya jorgemoya left a comment

Choose a reason for hiding this comment

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

Nice!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants