Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(core): use picocolors instead of chalk in the nx package#34305

Merged
JamesHenry merged 7 commits intomasterfrom
nx-picocolors
Feb 6, 2026
Merged

fix(core): use picocolors instead of chalk in the nx package#34305
JamesHenry merged 7 commits intomasterfrom
nx-picocolors

Conversation

@JamesHenry
Copy link
Collaborator

  • Updated package.json to depend on picocolors instead of chalk
  • Added ESLint rule to prevent new imports of chalk and guide developers to use picocolors instead
  • Replaced all chalk imports with picocolors (aliased as pc) across the codebase
  • Updated all chalk method calls to use picocolors equivalents:
    • chalk.bold()pc.bold()
    • chalk.green()pc.green()
    • chalk.red()pc.red()
    • chalk.yellow()pc.yellow()
    • chalk.white()pc.white()
    • chalk.cyan()pc.cyan()
    • chalk.blue()pc.blue()
    • chalk.magenta()pc.magenta()
    • chalk.dim()pc.dim()
    • chalk.gray()pc.gray()
    • chalk.inverse()pc.inverse()
    • chalk.underline()pc.underline()
    • chalk.reset()pc.reset()
  • Replaced chalk.keyword('orange') calls with the orange utility from utils/output
  • Updated color arrays in project-logger.ts and node-child-process.ts to use available picocolors functions
  • Fixed chained method calls (e.g., chalk.bold.white()) to use simple function composition (e.g., pc.bold(pc.white()))

@netlify
Copy link

netlify bot commented Feb 3, 2026

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit 6823090
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/6985cf7daa1b2600083ff10c
😎 Deploy Preview https://deploy-preview-34305--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud
Copy link
Contributor

nx-cloud bot commented Feb 3, 2026

View your CI Pipeline Execution ↗ for commit 6823090

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 3m 49s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 1m 48s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 8s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 3s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-06 11:33:34 UTC

@JamesHenry JamesHenry marked this pull request as ready for review February 3, 2026 15:45
@JamesHenry JamesHenry requested review from a team, Coly010 and leosvelperez as code owners February 3, 2026 15:45
@JamesHenry JamesHenry self-assigned this Feb 3, 2026
@JamesHenry JamesHenry added the scope: core core nx functionality label Feb 3, 2026
@vercel
Copy link

vercel bot commented Feb 3, 2026

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

Project Deployment Actions Updated (UTC)
nx-dev Error Error Feb 3, 2026 4:17pm

Request Review

nx-cloud[bot]

This comment was marked as outdated.

@netlify
Copy link

netlify bot commented Feb 5, 2026

Deploy Preview for nx-dev ready!

Name Link
🔨 Latest commit 6823090
🔍 Latest deploy log https://app.netlify.com/projects/nx-dev/deploys/6985cf7d9dd7710008514a64
😎 Deploy Preview https://deploy-preview-34305--nx-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

Copy link
Contributor

@nx-cloud nx-cloud bot left a comment

Choose a reason for hiding this comment

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

Important

At least one additional CI pipeline execution has run since the conclusion below was written and it may no longer be applicable.

Nx Cloud has identified a possible root cause for your failed CI:

Our Maven e2e test failed due to an external service outage (Cloudflare 522 timeout from start.spring.io), not from the chalk→picocolors changes in this PR. This should resolve on retry once the Spring Initializr API recovers.

No code changes were suggested for this issue.

🔂 A CI rerun has been triggered by adding an empty commit to this branch.

Nx Cloud View detailed reasoning on Nx Cloud ↗


🎓 Learn more about Self-Healing CI on nx.dev

@JamesHenry JamesHenry merged commit 0d2882e into master Feb 6, 2026
23 checks passed
@JamesHenry JamesHenry deleted the nx-picocolors branch February 6, 2026 15:17
@github-actions
Copy link
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

scope: core core nx functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants