Skip to content

chore: externalize fast-string-width & fast-wrap-ansi#481

Merged
43081j merged 2 commits intobombshell-dev:mainfrom
hyperz111:externalize
Mar 7, 2026
Merged

chore: externalize fast-string-width & fast-wrap-ansi#481
43081j merged 2 commits intobombshell-dev:mainfrom
hyperz111:externalize

Conversation

@hyperz111
Copy link
Contributor

Because we now on ESM-only package, maybe we can externalize these dependencies:

  • fast-string-width
  • fast-wrap-ansi

@changeset-bot
Copy link

changeset-bot bot commented Mar 6, 2026

🦋 Changeset detected

Latest commit: 194da69

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

This PR includes changesets to release 2 packages
Name Type
@clack/prompts Minor
@clack/core Minor

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

@43081j
Copy link
Collaborator

43081j commented Mar 6, 2026

Can you add a change set?

Generally looks good to me, and means we can lean into npm de-dupe since these two packages are quickly growing in usage

@hyperz111
Copy link
Contributor Author

Can you add a change set?

I will do it tomorrow (I'm on GMT+7)

@hyperz111
Copy link
Contributor Author

I think this is can be a minor change.

Copy link
Member

@dreyfus92 dreyfus92 left a comment

Choose a reason for hiding this comment

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

thank you @hyperz111 🫡

@43081j 43081j merged commit 9786226 into bombshell-dev:main Mar 7, 2026
5 checks passed
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.

3 participants